Done.
May I point out, however, that most of the `wrdac` files are with tabs and not spaces.


The question is the same here as below: why use both single and double underscore conventions?


Maybe we should use BinDeps for this?


@mro95: See L40


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.2.0-checkProjectionSignature+1](http://build.prxm.it:8811/viewLog.html?buildId=5525&buildTypeId=CqrsFramework_CiGitFlow) outcome was **FAILURE**
Summary: Tests failed: 13 (13 new), passed: 125, ignored: 4 Build time: 00:18:45
### Failed tests
```
Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.ConcurrentCheckpointTrackerTests.Verify_check_of_generic_projection_checkpoint_error: <no details avaliable>

Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.ConcurrentCheckpointTrackerTests.Verify_check_of_new_projection: <no details avaliable>

Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.ConcurrentCheckpointTrackerTests.Verify_check_of_new_projection_ok_when_rebuild: <no details avaliable>

Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.V2.BasicRebuild: <no details avaliable>

Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.V2.BasicRebuildWithNewProjectionDifferentSlot: <no details avaliable>

Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.V2.BasicRebuildWithNewProjectionSameSlot: <no details avaliable>

Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.V2.PollerCanStopAndRestart: <no details avaliable>

Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.V2.ProjectionEnginePriorityTests.verify_priority_is_maintained: <no details avaliable>

Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.V2.ProjectionEngineTestBasic.run_poll_and_wait: <no details avaliable>

Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.V2.ProjectionEngineTestsCheckpoints.run_poller: <no details avaliable>

Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.V2.ProjectionEngineTestsPoller.run_poller: <no details avaliable>

##### there are 3 more failed tests, see build details
```


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11538](https://teamcity.mesosphere.io/viewLog.html?buildId=105136&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:08:20


This allows you to access results of previous middleware on the original request while decorating the outgoing request. For example, cookies:

```
var proxy = require("express-http-proxy"),
    cookieParser = require("cookie-parser"),
    app = require("express")();

app.use(cookieParser());
app.use("/proxy", proxy("www.google.com", {
    decorateRequest: function(reqOut, reqOrg) {
        if (reqOrg.cookies["bearer"]) {
            reqOut.header["Authorization"] = "Bearer " + reqOrg.cookies["bearer"];
        }
    }
}));
```


Need a function to change this


welcome to sandist


Vil ikke den da alltid bli kjørt når man starter containeren?


I'm guilty: https://github.com/RigsOfRods/rigs-of-rods/commit/415f7f513ab9d2fefdc245a52288608f1efb04df#diff-a52911fe3e9c40ab98b42726250e304eL3876
It worked fine before: https://github.com/RigsOfRods/rigs-of-rods/blob/415f7f513ab9d2fefdc245a52288608f1efb04df/source/rig_file_input_output/RigDef_File.h#L1804
Sorry :blush:


Because I manually merged one commit :-) this is a change in a file which
is not used! I will remove it soon. Binary_server_asyncio.py is used

On Thu, Jan 14, 2016, 09:48 Andrew <notifications@github.com> wrote:

> Any idea why zerox1212@2b25287
> <https://github.com/zerox1212/python-opcua/commit/2b2528797293f9751c82e62c405e649a0ae5d12b>
> didn't get merged with this pull request? I thought it would get merged as
> well.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FreeOpcUa/python-opcua/commit/8972304a22ef4241f0032126efb13bcc2182f461#commitcomment-15444399>
> .
>


Please check this out


nid-ul asta va fi diferit in productie


resent


Please make all functions static which need not be exported.


OK, I see.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1158](http://teamcity.monogame.net/viewLog.html?buildId=13577&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


각줄의 comment?


Hey! This has some issues with detecting html pages encoding.

Html can provide encoding in page text itself, requests detects it with apparent_encoding, but does not use because this detection has higher priority, still worse quality


I'm also interested in the answer ;-)


Well... @svddries created the file, and I added a line for you. We didn't want you to find out about it this way but...


Why removing `useCurrent`, particularly from `created_at`? It solves [this][1] issue.

[1]: https://github.com/laravel/framework/issues/11806


Indeed, and one day we should really consider polishing the repo in this respect.
For the meantime, it's also neater not to introduce more tabs.


你想实现什么效果？


No, this commit is in the 3.2 betas.


TeamCity Jarvis Framework :: CI-GitFlow [Build 79](http://build.prxm.it:8811/viewLog.html?buildId=5526&buildTypeId=CqrsFramework_CiGitFlow) is now running


toumingdenahan


I know, but I needed a Docker file that I could experiment on, inserting steps along the way, and removing others :)


Fix #5 click recipe is bad


Just a quick fix @collinsnji  I don't like the mOptions here. Could change this to migrationOptions? This should be for consistency. With this everything should be fine.


missing assignment or return


Hi @Assimilator,

Which January 2015 cashier version are you referring to? Rather which cashier version (2.1.0?) is the issue coming up? Thanks


LGTM


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.2.0-checkProjectionSignature+2](http://build.prxm.it:8811/viewLog.html?buildId=5526&buildTypeId=CqrsFramework_CiGitFlow) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 129 Build time: 00:03:50
### Failed tests
```
Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.ProjectionEngineTests.ConcurrentCheckpointTrackerTests.Verify_check_of_signature_change: <no details avaliable>

```


test


Please see #2086 for a discussion of this issue.


Only the first word of a sentence is printing on the panel


Hi, if you would like this to be merged into the npm `cg` package, you can open a pull request! Looks good to me!


basic app


UGH NO THIS IS WRONG


Maybe we should do the change for all strings in the opposite direction. As an array the type has information about the length and compiler could emit additional warnings when something is wrong (although there are currently not very good at this).


This is not how we pascal case things, we have an epic String library in the core to handle all sorts of casing.


`renderTeams` prioritizes teams for the currently selected format over `curTeamIndex`.

There's a reason we intentionally pass `teamIndex` into `renderTeams`..........


@cvvergara you forgot to add "group: edge" as requested by @BanzaiMan. See https://github.com/gildegoma/travis-ci-inspector/commit/47ef64ed187d975433777db8ae6135e120e3da74


Ah, this warning is about QObject parent reparenting (`QObject::setParent`)!

I was able to verify that putting QObjects into the data slot of an Item calls `setParent` on that object (`void QQuickItemPrivate::data_append` is also pretty clear about this). This branch does not change anything w.r.t. to this. I assume that an event filter filtering for `QChildEvent` could solve this issue eventually, but for now I'd vote for keeping the log info.


tremendous*


You're right, I derped.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11539](https://teamcity.mesosphere.io/viewLog.html?buildId=105137&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:08:58


No wonder people were reporting that teams would randomly reset when I thought I fixed that half a year ago.


It might be because I started with the output, and not the input.
As far I can see, in memory, the subgroup is "owned" by the quest subgroup, not the DIAL record.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11540](https://teamcity.mesosphere.io/viewLog.html?buildId=105138&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:04


Sollte man hier nicht noch einmal g definieren falls die erste Zelle nicht ausgeführt wurde?


TeamCity Jarvis Framework :: CI-GitFlow [Build 80](http://build.prxm.it:8811/viewLog.html?buildId=5527&buildTypeId=CqrsFramework_CiGitFlow) is now running


:+1: Makes sense and looks good to me. I'm not sure why you didn't create a JIRA issue though.


*focus* and *blur* are synchronous events. That means you should check digest phase:
```
lxTextField.isActive = true;
lxTextField.isFocus = true;
if (!scope.$root.$$phase) scope.$apply();
```
See [how angular do it](https://github.com/angular/angular.js/blob/v1.4.8/src/ng/directive/ngEventDirs.js#L44). And see [the issue](https://github.com/lumapps/lumX/issues/381).


I don't know what you are putting in at notesRequest, but I suspect the issue may be that you are putting spaces between notes (like "C3, C#3, D3"), when the string is only being split at ",".
I just tested it and it works fine if I put "C3,C#3,D3,D#3" with no spaces.
However, if I put spaces between the notes, note_number returns None since " C#3" and " D3" etc. (with a space in front) aren't found in the dictionary.


Where is that? Thanks


上一个版本，在session获取的登录user没有和岗位相关的数据，即：导致JudgePrivilegeUtils#hasPrivilegeByName不能进入for循环的判断，即user.getRoles() = null


Looks good for me


@peff:
Do you think there is a chance to convince Junio to enable Travis for gitster/git? The current TravisCI implementation already proved that it will not bug anyone with emails. I wouldn't worry about the load at this point as this is what TravisCI is made for.

I also played a bit with the RAM disk idea. Unfortunately I was not able to see the huge speed gains mentioned in the [readme](https://github.com/git/git/blob/fc10eb5b87a75af2cb93a3932897572f2c544915/t/README#L169). Consider the following example. First, I test write performance of my RAM and my SSD. Then I run the same test against both. The wall clock time is almost the same. Do you see what is wrong with my setup?

```
lars@ubuntu:~/git/t$ dd bs=1k count=1000000 if=/dev/zero of=/tmp/ssd-test conv=fdatasync
1000000+0 records in
1000000+0 records out
1024000000 bytes (1.0 GB) copied, 7.47684 s, 137 MB/s

lars@ubuntu:~/git/t$ dd bs=1k count=1000000 if=/dev/zero of=/dev/shm/ram-test conv=fdatasync
1000000+0 records in
1000000+0 records out
1024000000 bytes (1.0 GB) copied, 2.45845 s, 417 MB/s

lars@ubuntu:~/git/t$ time ./t2013-checkout-submodule.sh --root=/tmp
...
# still have 6 known breakage(s)
# passed all remaining 28 test(s)
1..34

real    0m3.722s
user    0m0.144s
sys 0m0.440s

lars@ubuntu:~/git/t$ time ./t2013-checkout-submodule.sh --root=/dev/shm
...
# still have 6 known breakage(s)
# passed all remaining 28 test(s)
1..34

real    0m3.371s
user    0m0.132s
sys 0m0.304s
```


Cool.


It's not a big deal, but I'd like to understand the rationale here. `Validators.validate(request)` is a stream, so it seems more logical not to split that across lines. If the rationale is simply to align some periods vertically, then shouldn't we also split `this.spaceId` in the list method below? If you hate the latter, which you should, why don't you also hate the former?


Thanks, I missed that!


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.2.0-checkProjectionSignature+3](http://build.prxm.it:8811/viewLog.html?buildId=5527&buildTypeId=CqrsFramework_CiGitFlow) outcome was **FAILURE**
Summary: Tests passed: 131; exit code 1 Build time: 00:04:00


The first command should probably be:

```
pip install -r requirements.txt
```

But even than it's not working, because some requirements are only available on Galaxy depot. In this case the following command would be useful.

```
pip install -r requirements.txt --extra-index-url https://wheels.galaxyproject.org/simple
```

But even that is not working for me, because a few packages could not be found. @gregvonkuster is it working for you?


@pminutillo, Peter, I'm not sure why we always set inline to true (line #207).


Yes, that's better actually.  If we were out of date and a user wants to use a newer enum, they can simply `"foo".asInstanceOf`.


@jcalovski: Sorry I don't understand your comment. Who needs what?


Doh. Yes. :)


fixed (MailboxCreationException)


@pb00067 Only master branch has been set to 8MB hash.
new_options	Hash=128
base_options	Hash=8


TeamCity Jarvis Framework :: CI-GitFlow [Build 81](http://build.prxm.it:8811/viewLog.html?buildId=5528&buildTypeId=CqrsFramework_CiGitFlow) is now running


Why
 tag always applied ? Are you assuming that everyone who will implement this library needs a
 if that is an assumption than, that it is a wrong expectation.

You need to modify css to make it work


maybe better:
public static List<String> parseFile(File file)
would be easier to use


This method doesn't make sense to me because its implementation doesn't match its name. It's supposed to "commit if successful", but there's not actual check to see if the result is successful before calling #commitChanges on the callback. If the MutatorCallback method were named #commitIfSuccessful, then I would understand that you are delegating the responsibility to the implementor to verify success before committing. But I don't think that's a good route to take. #commitIfSuccessful should really be

```
public final void commitIfSuccessful ()
{
  if (isSuccessful () && callback.isPresent ()) callback.get ().commitChanges ();
}
```


br tag always applied why ?


hej @jp5000 are you sure the CT power here is 0?


Ops, sorry!


I think it's still defined https://url.spec.whatwg.org/#api, although confusingly the name has changed, but this is speculation, agreed.


just
return parseKeyWords(linedFile);


Nice work! Before I merge it in, can you please add one more space on `SYSTEM_PACKAGE_TYPE=` lines so it stays aligned with other lines around.


Ok, never mind. I see that a MutatorCallback can only be passed to #success, so if it is present, then the result was successful. Honestly though, even though it might be slightly more efficient to write it the way you did, it makes the code much less readable in my opinion.


`$scope.$apply` calls `$rootScope.$digest`. That means you should reduce `$scope.$apply` calls to one call only.


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.2.0-checkProjectionSignature+3](http://build.prxm.it:8811/viewLog.html?buildId=5528&buildTypeId=CqrsFramework_CiGitFlow) outcome was **FAILURE**
Summary: Tests passed: 131; exit code 1 Build time: 00:04:26


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1158](http://teamcity.monogame.net/viewLog.html?buildId=13577&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:28:23


Done


I would really like to see a few tests to demonstrate this code in action. I think it has potential.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11541](https://teamcity.mesosphere.io/viewLog.html?buildId=105139&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:23


My prefered way of fixing this would be to switch to encryption algorithms that are not vulnerable to this kind of attack (adaptive chosen ciphertext attack) like RSA-OAEP and AES-GCM and disable support for the vulnerable PKCS#1 v1.5 encryption and CBC-* mode algorithms. This way we solve the problem at the origin.

This seems like the best way forward. Open question is then of course what the software support for those algorithms out there is. I have no clue since we do not support encrypted assertions currently so we don't have many IdP's using them.

@surfnet-niels: Do you have a concrete IdP you want to use this with? What software does it use and which algorithms does it support?


This shouldn't have chamged to the ja.resx.


Please note that there is a PR against edx-configuration that touches upon deployment of comprehensive theming (edx/configuration#2676). It currently makes no assumptions about what `EDXAPP_COMPREHENSIVE_THEME_DIR` is. But just mentioning it here so the issue is cross-referenced.


Actually, ASCII-Betical Order.


You fixed br tag in last commit - neat. But answer these questions below

 1.  $('.sdls_spinner').css("background-color", spinnerColor); < -- what does this do for you, is it tested  ?
 2. Why you '-spinner' instead of just 'slds'
 3. I told you replace sdls everywhere with slds , which is 'not' fixed

Fix both and provide an evidence that 1'st works, send me pull request again once this is fixed.


Should we not just refer to 'the graphical configuration frontend' as SysConfig?


As the strategy is close to our testee, i prefer to use a concrete class.


README.md shouldn't be changed, as it is displayed on the main page of xenadmin project in github


please include the sha512 hash so that it is verified


Ah yes, I actually meant the order of records and subrecords too inside other groups. Physically topic children group 7 is inside quest children group 10 right after the DIAL record.
I atually thought about solving this from the other side, as I mentioned when adding new DIAL records to QUST (using right click Add on QUST record), it adds DIAL properly only for the first time, the second time it creates new quest children group and places DIAL there instead of using already existing one, so the problem occures somewhere around that code not able to find already existing children group. Second addition looks fine in xEdit GUI, but if you save and reload plugin there will be orphaned children group.


Addresses pootlepress/ppb-live-editor#12


This simple change is enough for client side security.
It seems that there is no need to involve `RouteController` when checking if the user is authenticated.
What do you think @rhyslbw and @DominikGuzei


correction taille url image jeu


Sure, you can upload it.

If you would like to add the Spanish translation to the master branch, you are welcome.


Done


What does this even fix? `curTeamIndex` shouldn't be `0` anyway, it should be `-1`


I think the literal option is better, because it's an entire command, not just an file name.


done


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11542](https://teamcity.mesosphere.io/viewLog.html?buildId=105140&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:37


"inside" sounds a bit strange, isn't it more like "can only be used **after** a COLLECT statement"? Or more precisely: "can immediately follow after a COLLECT statement only"


Same argument here about the literal.


getLanguageList() is used in templates/includes/header.php


Not all targets are affected. For example Sparky uses different AF mapping for USART2 ..


Do you know how widespread this issue is now? If it's not a problem anymore, I think one should consider removing this block of code.


Used in templates/includes/header.php


Hmm, maybe you're right. I was afraid someone will decide it's not needed but attaching unneeded unload handlers would be wrong as it may always have unwanted side effects so I guess it may stay as it is.


What about:

Don't forget to set the <filename>.fetchmailrc</filename> permissions to 0710.

and using literal also here for chmod


Where is the test?


この説明だけでなく、 https://github.com/project-hatohol/website/compare/16.01#diff-a3875c579f71ddd4a2bb035e68720cf8R48 からもQpidの項目を消しておくべきです。逆に、HAPI2ではrabbitmq-serverのインストールと設定が必要になるのでこれを新たに書く必要があるように思います。


@ajithcj : resubmitted test.


It's a one-time const allocation that persists for the entire runtime. It is freed on exit.


hier gibt es bestimmt ein paar mehr markets: https://www.enterprisecarshare.ca/ca/en/home.html


Good point.


Done by Colleague Risto Risti by glen request.

![fix chosen](https://cloud.githubusercontent.com/assets/12842560/12320797/473e6362-bab3-11e5-9e0f-ef95339136a8.png)


This PR is marked WIP


RabbitMQ本家の説明ではrabbitmq-serverのstandaloneのrpmを使って説明がなされているのでこれを元に書くといいのではないでしょうか: https://www.rabbitmq.com/install-rpm.html


***Code Quality: Awesome***

Its always a good practice to start a selector and declaration on a new line


Is not this js overkill?  It could not be done with spans with attrs?


Done


Missing a closing para in my opinion.


After this change, gradle is giving me 403 errors when fetching dependencies. Misleadingly, it says "403 error when fetching from libraries.minecraft.net/...", which means it wasn't able to find the file on the MinecraftForge server and fell back to the other dep servers.

The failing dependencies are also marked in this part of the output of `gradle :dependencies`:

```
+--- codechicken:CodeChickenLib:1.7.10-1.1.3.136 -> 1.7.10-1.1.3.138
+--- codechicken:CodeChickenCore:1.7.10-1.0.6.44 -> 1.7.10-1.0.7.47
+--- codechicken:NotEnoughItems:1.7.10-1.0.4.107 -> 1.7.10-1.0.5.111
+--- appeng:appliedenergistics2:rv3-beta-1
|    +--- appeng:RotaryCraft:V6e
|    +--- pneumaticCraft:PneumaticCraft-1.7.10:1.9.15-105 FAILED
|    +--- api:rf:2
|    +--- api:mfr:1
|    +--- api:railcraft:1
|    +--- appeng:mekanism:1.7.10-8.0.1.198
|    +--- com.mod-buildcraft:buildcraft:7.0.9 FAILED
|    +--- li.cil.oc:OpenComputers:MC1.7.10-1.5.12.26 FAILED
|    +--- codechicken:CodeChickenLib:1.7.10-1.1.3.138
|    +--- api:immibis:1
|    +--- api:coloredlightscore:1
|    +--- net.industrial-craft:industrialcraft-2:2.2.717-experimental
|    +--- net.mcft.copy.betterstorage:BetterStorage:1.7.10-0.11.3.123.20
|    +--- api:craftguide:1
|    +--- codechicken:ForgeMultipart:1.7.10-1.2.0.345
|    +--- appeng:InventoryTweaks:1.58
|    +--- codechicken:NotEnoughItems:1.7.10-1.0.5.111
|    +--- codechicken:CodeChickenCore:1.7.10-1.0.7.47
|    \--- appeng:Waila:1.5.10_1.7.10
+--- :MineTweaker3:Dev-1.7.10-3.0.9B
+--- :forgeBin:1.7.10-10.13.4.1481-1.7.10
```

I tried running `gradle --refresh-dependencies` to clear the dep cache, but the same thing happens. If the build works for you, is there something else you had to do to get it going?


warning 방지


Unnecessary space added.


Thanks for your submission Balázs Úr! We are looking forward to further work from you, and your comments in response to mine.


Oh boy - did I get the fold functions in the wrong order? Should the folded value - the `c` be first?


Nadam se da je sad ok.
Tražio sam i po forumima na internetu kako se računa visina u pointima i mislim da je ovako u redu.


wstring -> utf-8 변환 알고리즘을 STL로 대체


njm,m,


can you make this method `private`?


I think it's fine to mark low_watermark as u32 instead.


c'est dommage, à peu de choses prêt on aurait pu directement passer `config.logs.logstash`


Why do you add sheet?  I removed it...


Duh! This block should not have appeared in this commit.


Wouldn't it be better to pass one more argument to the function: the output file name? This way you don't have to open it later again and write as sorted specific file.


It'd be nice to move it under the hod namespace too.


Do we still need this? Isn't it redundant with `:rbenv_ruby_version`?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11543](https://teamcity.mesosphere.io/viewLog.html?buildId=105141&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:08:34


Remember spaces between each definition in these classes or Sébastien will grumble :)


arf, il y a un bug "config.logs.logstash.name" devrait être `config.logstash.name` je fais la modif


Let's use a variant (Global | Local or something) instead of the boolean.


Hi. I tried the tool last night but It doesn't seem to install on x64 Win10.


Is this still needed?


I'm pretty sure this change is wrong


sd


should remain !strcasecmp


Needs quotes around `"$4"`


**This file contains unused source code.**
*PyUnusedCodeBear, severity 1, section `python`.* *Related to file `coalib/tests/settings/ConfigurationGatheringTest.py`, line 6.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/tests/settings/ConfigurationGatheringTest.py
+++ b/coalib/tests/settings/ConfigurationGatheringTest.py
@@ -3,7 +3,6 @@
 import re
 import tempfile
 import unittest
-from contextlib import contextmanager
 from pyprint.NullPrinter import NullPrinter
 from pyprint.ClosableObject import close_objects

```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/tests/settings/ConfigurationGatheringTest.py`, line 15.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/tests/settings/ConfigurationGatheringTest.py
+++ b/coalib/tests/settings/ConfigurationGatheringTest.py
@@ -13,6 +13,7 @@
                                                     find_user_config)
 from coalib.output.printers.LogPrinter import LogPrinter
 from coalib.misc.ContextManagers import make_temp
+

 class ConfigurationGatheringTest(unittest.TestCase):

```


@bparees we had Secret before, but I don't really like it because it does not say when the secret is used. Also it conflicts with the Secrets in the CustomStrategy. I really like to have this more explicit and related to the "build" where the secrets are used.


done


Also any temporaly output files is better to store under each plugin output folder. For example for cfa plugin under: "./cfa_plugin/output"


hard to mock a PublicKey and all its behaviour


... @allandegnan yeah, it's on my list of things to do to ask you for a repo to be made for it :)


Annotation keywords should be all upper case, followed by a colon and a space, then a note describing the problem.


...it's id normally... this will cause problems for designer


This is the `std::forward` that's missing from ARMCCs standard lib?
Could we move it to the compiler-polyfill module?


***Code Quality: Nitpick***

It might have been better if font-size and and font-weight are defined in one single declaration. css offers shorthand properties that are useful for code efficiency and provides better understanding  this can be re-written as
```font: normal 100%```


I think you shouldn't authenticate here, you should get the user model from the DB directly instead, like ModelBackend does. ModelBackend.authenticate will also check_password and will return None if the password doesn't match.
If the app uses some other Backend to check passwords ModelBackend.authenticate will always return None and your checks below will never get called.

Also, any reason to make UserExpiryBackend a subclass of ModelBackend?


fixed


We definitely still need the task, and I think it makes sense to link all the required files earlier in the process


Conda FAQ:

1. You can absolutely not install planemo via conda and run Galaxy inside the conda environment currently.

    galaxy-lib is a dependency of planemo and having this on the Python path prevents correct operation of Galaxy.

1. How do I enable conda dependency resolution within Galaxy tools (install conda dependencies corresponding to ``requirement`` tags)?

   Just install planemo normally via pip into a virtual environment or via brew and use the appropriate commands and options:

   ```
    $ planemo conda_init
    $ planemo conda_install .
    $ planemo test --conda_dependency_resolution .
    $ planemo serve --conda_dependency_resolution .
    ```

    The test and serve commands above require the target Galaxy to be 16.01 or higher, the default target is still 15.10 so the above command is assuming galaxy_root has been overridden to ``~/.planemo.yml``.

1. How do I manually install some conda dependencies and test my tools using them in Galaxy.

    This is an advanced technique and I'd frankly discourage it currently, but should be possible.

    planemo must be installed outside the conda environment - via pip into a virtual environment or via homebrew. In the former case, don't place the virtualenv's ``bin`` directory on your PATH - it will conflict with conda. Just reference planemo directly ``/path/to/planemo_venv/bin/planemo test``.

    To run Galaxy from within the environment you will need to install Galaxy dependencies into the conda environment. I'd absolutely recommend targeting the development branch of Galaxy for this since it has "unpinned" dependencies that are easier to fullfill for conda.

    ```
    (conda-env-test) $ conda install numpy bx-python pysam # install the hard ones using conda
    (conda-env-test) $ cd $GALAXY_ROOT
    (conda-env-test) $ ./scripts/common_startup.sh --skip-venv --dev-wheels # install remaining ones using pip via Galaxy
    (conda-env-test) $ cd /path/to/my/tools
    (conda-env-test) $ /path/to/planemo_venv/bin/planemo test --skip_venv  .
    (conda-env-test) $ /path/to/planemo_venv/bin/planemo serve --skip_venv  .
    ```


Yeah you're right, for simple scenarios there is no need for complex authentication processing :+1:


Vu l'âge du commit (plus de 2 ans), je te suggère de faire une PR en master avec les éventuelles corrections nécessaires.

En attendant, voici un tractopelle pour le déterrage.
![image](https://cloud.githubusercontent.com/assets/7300760/12321211/38785ed8-baad-11e5-8b8f-98a9fcff2d0c.png)


Quand tu wrap des éléments indentes les éléments enfants.

BAD :

GOOD:


Ack, yes, you're right.  In fact on Window, `self` is just a pointer back to the same window, and not a `ServiceWorkerGlobalScope` at all, dunno where I got that from.

Technically it inherits `WorkerGlobalScope`, but that's not in the web workers api yet- https://www.w3.org/TR/workers/#workerglobalscope.  Inheriting `js.GlobalScope` would probably be nice though.


Thanks!


BuildSecrets sounds good to me, @smarterclayton ?


Happy to, if it's needed anywhere else!


> Do you think there is a chance to convince Junio to enable Travis for gitster/git?

I don't know what his thoughts are. If it is not much work and does not bother him, perhaps. I think the best thing would be to ask him on the list (and probably provide short instructions for doing so).

> I also played a bit with the RAM disk idea. Unfortunately I was not able to see the huge speed gains mentioned in the readme.

In your example, you're only running a single script. Some of them are more disk-heavy (and especially `fsync`-heavy) than others. It may also make more difference if you're running tests in parallel (that's just a guess, but it seems plausible that fsyncing in one script may stall another). Here are my numbers across the whole test suite:

```
$ best-of-five make GIT_PROVE_OPTS='-j16 --state=slow,save' GIT_TEST_OPTS='' prove
Attempt 1: 59.042
Attempt 2: 57.441
Attempt 3: 64.325
Attempt 4: 62.451
Attempt 5: 59.208

real    0m57.441s
user    1m10.720s
sys     0m26.012s

$ best-of-five make GIT_PROVE_OPTS='-j16 --state=slow,save' GIT_TEST_OPTS='--root=/var/ram/git-tests' prove
Attempt 1: 48.657
Attempt 2: 50.532
Attempt 3: 50.656
Attempt 4: 50.666
Attempt 5: 50.711

real    0m48.657s
user    1m10.868s
sys     0m20.888s
```

So that's about 15% on the best-of-five time, but notice also that the times are a lot more consistent in the RAM-disk case.

My initial timings (from f423ef5f2bec35efbc328f02e468a18612970242) were more like a 50% speedup, but that's probably beacuse I wasn't using `prove` to order the tests (so we would end up waiting at the end for a small number of disk-heavy tests to complete).


here we did this on purpose.


@bgruening Absolutely - I missed up those pip calls are not a good way to get Galaxy dependencies into your virtualenv. I create a short FAQ abov, which should land up in planemo's docs at some point once we all like it based on the problems we discussed out of band and here. In the FAQ I recommend installing numpy and a couple other big dependencies using conda directly and then install the rest using ``common_startup.sh`` so the correct index is available.


I changed this, because there is "filename" tag at line 173. If literal is better, than change at line 173, too.


Quand tu wrap des éléments indentes les éléments enfants

Mais


Ok, souhlas, pozd2ji to předělám.


again, this is real valid reference equality


Looks good. The [`sysvinit-passenger.example`](https://github.com/mysociety/alaveteli/blob/develop/config/sysvinit-passenger.example#L17) and [`sysvinit-thin.example`](https://github.com/mysociety/alaveteli/blob/develop/config/sysvinit-thin.example#L17) do this, but use the `$SITE_HOME` variable. I do prefer the name `$APPDIR`, but I think we should change these to `$SITE_HOME` for consistency.


Forcing all users of the django-userlog use a custom auth model seems a bit harsh.

Can we also offer a solution to users who have chosen to extend the User model (https://docs.djangoproject.com/en/1.9/topics/auth/customizing/#extending-the-existing-user-model)?

The implications of changing to custom auth model for an existing app might be stoping users altogether to adopt django-userlog.


Remove it from here


Issues mentioned in this commit (4f7cec2390a43f90951b20bd5c1f95d4756d53f7):

* [Issue #9001](https://bugs.mtasa.com/view.php?id=9001)


Issues mentioned in this commit (1d9474b3a8d4381f40da6eb9e85ca5fb7980899b):

* [Issue #9002](https://bugs.mtasa.com/view.php?id=9002)
* [Issue #9003](https://bugs.mtasa.com/view.php?id=9003)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11616](https://teamcity.mesosphere.io/viewLog.html?buildId=105599&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:03:56


111624768_HWRadiantConvBB (Nigusse) - x86_64-MacOS-10.9-clang: OK (2049 of 2049 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Not fully functional yet, it's just client side


How about this: https://docs.oracle.com/javase/tutorial/collections/algorithms/#sorting?


It may look fine here, but it's actually not correct. Apparently the bot doesn't check to see if the PR status is still accepted before doing the final merge...


Can you please put braces around the single line if statement? See coding convention here: https://github.com/Microsoft/ChakraCore/wiki/Coding-Convention


Done


de Julio Cesar Perez Cuellar de 5B hahaaha :D

2016-01-14 17:43 GMT-06:00 ffranjavierutez <notifications@github.com>:

> Mmm el commit me marca de esta cuenta o de quien es esta cuenta?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ffranjavierutez/gitdesdeapps3/commit/6dc4550729347cc83c514c0644f7c76b024b3c83#commitcomment-15463966>
> .
>


how about google-map-elements.html


@diegoamc They will be when we finish it, it's a single commit just to avoid keeping patches just between us as @marcheing said.


As a bit of UX feedback, I like to stay on the same page after approving a connection, since it gives me a chance to select which collection my new connection goes into.

If there was some kind of drop-down to select which collection a person should go into within the popup modal, though, this would be a non-issue.


'google-map-marker#marker2' -> #marker2


If this assertion is not true, isn't this a bug?  We use non-64-bit depth all over the place, including protentially in production some day.


LGTM, except I want to improve the test coverage for all depths.


Warning! New migration added! @thekordy is there anything extra to do with the migration? I mean when a user updates to a new version, how will be the new migration migrated?


If this is a temporary debug line in the first place, let's just remove this line.


> Furthermore, I can no longer use chaining syntax or aliases

Aliases should still work with the plugin ;)


LGTM, except request for nuking commented out debug code.


Damn, you got me


develop (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (954 of 954 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-26.64%25-green.svg)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11617](https://teamcity.mesosphere.io/viewLog.html?buildId=105602&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:04:18


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11618](https://teamcity.mesosphere.io/viewLog.html?buildId=105603&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:17


Shouldn't they be marked 'invalid' instead?


Issues mentioned in this commit (4f7cec2390a43f90951b20bd5c1f95d4756d53f7):

* [Issue #9001](https://bugs.mtasa.com/view.php?id=9001)


Issues mentioned in this commit (1d9474b3a8d4381f40da6eb9e85ca5fb7980899b):

* [Issue #9002](https://bugs.mtasa.com/view.php?id=9002)
* [Issue #9003](https://bugs.mtasa.com/view.php?id=9003)


I know :-)


:ticket:  You can configure your editor to add an automatic break line at the end of a file


Value isn't used???


Danke 👌


This looks good thanks Aaron!


What does this todo mean? I don't understand it.


Yap


This, um, hopefully isn't needed anymore, again.


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 202](https://teamcity.labkey.org/viewLog.html?buildId=296827&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **FAILURE**
Summary: Snapshot dependency failed to start: LabKey :: Trunk :: Premium :: build Build time: 00:00:01


Sure, the update should be there now.

Am 1/14/2016 um 3:54 PM schrieb Alistair Francis:
>
> Ok, can you cast ram_kernel_size to an int and then compare with less
> then 0? That sounds like the best option to me.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/liebharc/qemu/commit/cb94f0957eb0bd4e61275d359a623abd4c7dda11#commitcomment-15464150>.
>


Done


Seems like this commit added a dependency on the xalan directory, but doesn't include it, so now I get an error trying to build:

make clean
/bin/sh: line 0: cd: xalan: No such file or directory
make: *** [clean] Error 1


I think it was very intuitive that helm takes over the default `find-file`. As for my own solution, I just did `customize-variable` and took out the two values as mentioned above.


You need to add a `getDependencyMigrateClassNameArray()` with `SF_migrate_igive_addkeywords` here so that this class is run after `SF_migrate_igive_addkeywords`. When lines 13-17 are run Propel expects to be able to query all the fields on that table, some of which (`keywords`) aren't there because the migration hasn't run.


I think just you should pass silent mode as a parameter to `READER_MODE` from the javascript.  This would simplify some things


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11619](https://teamcity.mesosphere.io/viewLog.html?buildId=105604&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:03:58


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11620](https://teamcity.mesosphere.io/viewLog.html?buildId=105605&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:27


I guess this should also depend on `SF_migrate_igive_addlanguages`.


Change this to `function (silentMode, win, fail)` or `function(settings, win, fail)` where `settings: {silentMode:bool}`


This looks decent -- I had a similar diff in my local repo.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11621](https://teamcity.mesosphere.io/viewLog.html?buildId=105606&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:10


I wasn't going to get too far testing with this so 25ef74df09784f3583e86e33e9d3aaf585855f3d :stuck_out_tongue_winking_eye:


이건 왜 만드시는거


develop (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1505 of 1505 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.67%25-green.svg)


Please please please, let me know when you've added the cropcover to include/widgets.php! I'm eager to implement cover photo functionality in my themes.


Switched over to all async functions in b31e4550f904533a577a752c990b758557dc95b9 and 894da0d4bc332a5e0a4bfb9c8fd2dbcf2eb502cb.

On Jan 14, 2016, at 3:11 PM, Nicholas Hallahan <notifications@github.com<mailto:notifications@github.com>> wrote:

We can't have anything synchronous, because it will jam up the Node process when doing the given task.

—
Reply to this email directly or view it on GitHub<https://github.com/AmericanRedCross/OpenMapKitServer/commit/aa652cb567e547f12bb6aa013df3c8887d26699f#commitcomment-15463399>.


It does but only on the internal side... What's wrong with it? If you send a fix or a revert I can stamp it.


I had not realized that there was a base class to PolyCollectionBase.


You guys, feel free to improve it. For instance, the original implementation uses octaves: it might be useful to introduce them here. Also, maximizing the activation of just one channel in a representation layer would be interesting (e.g. identify which channel in the last layer stands for "flower", then focus on that one rather than summing all channels).


Ah, my bad. I've checked against divine-pride (is this considered valid source?), found out it dropped from 2320, 2321 and 2322 which doesn't make too much sense as Bakonawa itself is unreachable in melee range. Any thoughts?


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1927 of 2054 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-93.82%25-yellow.svg)


I think this one is intentional


Also why is this under AutoSystemInfo::Data.SSE2Available()? and ENABLE_NATIVE_CODEGEN?


This is a good question.  I’m not sure either.  They can still attach to a process even without creating the semaphores.  I guess that case depends on the permissions needed to look at the /proc/xxx/maps to get the module list for a process.


111624768_HWRadiantConvBB (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


thanks, hadn't picked up on the FQN-style children, handy..


Is it intentional that I can search for a Spanish title with English selected as my language but still pull up that offering (albeit with its English title)?


Added testing for python 3.5 support.


I've reverted the change to the readme in a later commit, pending if you'd want to have a 2x support in this project or maintain in another,,,


I don't see CloneToScriptContext implemented for support iframe marshalling.


@valib I'm not native either, but I would rather say:
```html
<html><strong>Default:</strong> Auto detect<br><strong>Notes:</strong> You can find the list of supported character encodings at the <a href="http://userguide.icu-project.org/conversion/detection#TOC-Detected-Encodings">ICU4J page</a>. If your character encoding isn't supported by automatic detection, you can select the character encoding manually.<br>Warning: If you set the character encoding manually, any subtitles that use other character encodings will not be displayed correctly.</html>
```

I've replaced ```codepage``` with ```character encoding``` since codepage is strictly a Microsoft term. Microsoft (and IBM used to I think) calls their character encodings for "codepages", but that only applies to the Windows ones. The ISO and the other encodings are not called codepages. The name of the field should probably be changed as well.

But, instead of making yet another commit we should probably have some natives look over it.


CloneToScriptContext is required for all the static type objects inherited from RecyclableObject


ENV['RAILS_ENV'] || 'development'

为什么默认default呢


NSInteger


commit


CGFloat


This is pretty clever :wink:, but what if in the query you did `->addAsColumn( "Title", $language . "_title" )` and/or `SF_propel_helper::selectAs()`? Then you wouldn't need to `postProcess()` the results.


It's getting a lot closer. They're being scaled appropriately and stored now, and notifications sent out. (Requires changes I completed seconds ago).  You can get an idea of what's going on by pulling an update and visiting /cover_photo on your site. That will let you create one. Next we'll have to start using them for stuff... I'll start on a function to get_cover_photo() in a little bit after some more coffee.


space before =, initWith..


Also are you planning a seperate check-in for debugger support?


I'm unsure why post filter is used rather than a filtered query.  I think using post filter means filters will not be cached.  Any reason why it should be post filter?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11622](https://teamcity.mesosphere.io/viewLog.html?buildId=105609&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:13


Indeed, thanks :)


Good catch! This mistake is still in mainline QEMU. Do you feel like sending a patch to them? If not I can send it on your behalf for you.


most of the changes here are migrating from deprecated FilterBuilder to QueryBuilder.


This is so nice!


Ah I see what you mean here now. The start is ImpactStart, not the beginning of the enum, and mImpactSoundId values are supposed to range from 0 to 3, the same as the footstep sounds.


teste here not 100%


sorry about the indentation changes, i should have been more careful so changes would be more apparent.  I can update the merge with spaces fixed if you request it.


Some of these examples can be combined if someone adds mire examples


reusing createConnection params so cluster in this test will have name as set in properties.


Are you interested in telemetry for individual built-ins? CHAKRATEL_LANGSTATS_INC_BUILTINCOUNT macro is a way to do it. I guess you may not want individual API telemetry as there are over 300 of them. Something to think about.


used bulk insert.


This could raise an `AttributeError` as there is a chance that there will be no `username` or custom username field on a custom user model named other than `username`.


I saw elsewhere a sleep was used to make sure data was ready to be queried, refresh does this same thing.


This is great, though I don't suppose anyone would ever notice if this test failed.


does python have issues with mixed indentation?


this was weird, the test db was over 400 MB with this setting.  So I removed the spec on tree_levels.


You mentioned that you wish to keep Introverts public methods inline with the commands and also that the functionality would be better suited in a private method.  I just took those verbatim without thinking for myself and that was the result.


@nanakiwurtz Well, official kRO site states that this 4591 (Bakonawa_Card) being dropped from 2335 (MA_TREASURE) is correct.
source : http://ro.gnjoy.com/guide/runemidgarts/popup/monsterview.asp?monsterID=MA_TREASURE


Instead of checking the arg.Info.Count for every iteration of the loop why not bound the loop by that variable and fill undefined for the remaining one in a separate loop?


Why to bother to check `AUTHENTICATION_METHOD` when allauth [AuthenticationBackend](https://github.com/pennersr/django-allauth/blob/master/allauth/account/auth_backends.py#L12) already takes care of it.


`vi` you fickle beast.


In 2x, two mappings in the same index may not have different types for the same property name.  Since geo is shape here and point in active mappings, and this geo is not used in inactive features, I removed the type.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11623](https://teamcity.mesosphere.io/viewLog.html?buildId=105612&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 212 Build time: 00:03:48


111624768_HWRadiantConvBB (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2087 of 2087 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11624](https://teamcity.mesosphere.io/viewLog.html?buildId=105613&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 211 Build time: 00:04:03


its the best font ever.


it seems you want the gt-elasticsearch to tract the geo-tools version?


I have to test that… I changed that just to be safe …


若用户没有设置头像，则把用户名的第一个字母当做头像


I think we'll need a version qualifier here as boost::simd appears to use some newer boost::function stuff (I needed to update BH to get it to compile)


We could potentially use a timeout on the Jenkins job; cdunn does this for Blasr cram tests.  The runtime with these inputs before my change was around 20 hours (and since that was run from smrtlink, it should at least have had a reserved slot), so it's not hard to tell when it breaks.


StreamFilteringTests_AQ and StreamFilteringTests_SMS are reusing the same config (fixture). Nevertheless, that was the main reason to have multiple classes, because their configs are and should be different. Checkout the diff here: https://github.com/dotnet/orleans/compare/master...galvesribeiro:xunit-migration#diff-d7c28c0597f46b920f3d20bb75a6b486R325


> How did you make that commit?

Similar to instructions here: https://github.com/hashbang/admin-tools#install-a-package
On ny1 I ran:
```
sudo apt install libgumbo-dev
dpkg --get-selections | sudo tee /etc/packages.txt
sudo etckeeper commit -m 'Install libgumbo-dev
```


need to keep guava out of the gs-geoserver jar.  and also update the guava to v18 in geoserver.war/webinf/lib


sphinx is warning about bad rst here


修正done!


remove this class entirely, each class/collection fixture knows how to cleanup


I guess this should end with `::` ?


Oh I would be fine if you do that :). Thanks

Am 1/14/2016 um 4:37 PM schrieb Alistair Francis:
>
> Good catch! This mistake is still in mainline QEMU. Do you feel like
> sending a patch to them? If not I can send it on your behalf for you.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/liebharc/qemu/commit/c023c9ce491129946f0d4428eece857658be633f#commitcomment-15464788>.
>


For all the people, that's the fix for random item in the Crafting Table output slot ex. Lapis

Regards


I would move this first sentence up above the example, replacing the existing sentence.

Instead of 'Vars can be defined and used as follows', have 'Vars can be defined by the platform, in the config at the top level, inside the triggers, or in a remove config, as shown in this example:'.


我这已经解决了，不过还是非常感谢开涛前辈


TeamCity LabKey :: LabkeyRelease15.3 :: Git postgres 9.4 [Build 123](https://teamcity.labkey.org/viewLog.html?buildId=296834&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:01:02


The second sentence in this paragraph would be the first sentence after the example. I suggest elaborating on this through the example. Change the example to have a var that is identical in two different places, and explain what the value would be based on this. It's a lot clearer to hammer home these kinds of exceptions using the example you already have.


The next four list items could go away if you take on my previous suggestion.


If I understand correctly: this is still O(N^2), you're just shortening the inner loop by limiting it to the contig of interest.  This seems fine.  If it passes your tests I will just merge it.

[This code (which I wrote) ain't pretty, if we want this to scale properly we probably want an interval tree or something like that so we don't have to do an O(N) lookup inside the loop.  In fact it would be quite trivial if w e could commit to a standard convention for block sizes, then we quickly identify the  right "summary" object for a variant call]


I like that idea.  We could try it, with a loose timeout bound.


alert 함수 대신 notify 라이브러리를 사용하는게 좋을듯 합니다.


Hmmm, this is a dangerous and easy to make mistake. If we have a test that instantiates `TestingSiloHost` in the constructor (instead of using an `IDisposable` fixture, it is extremely easy to forget to implement `IDisposable` in this class itself and leave the cluster running, interfering with subsequent tests.

A potential solution is to have a `HostedTestClusterPerTest` base class as we used to have, and have a virtual `CreateSiloHost` method that you can override (or not if you just want the default config), and that base class is responsible for implementing IDisposable also and stopping `HostedCluster`


수정하겠습니다.


Ops! Forgot that... Was commented out just to check the behaviour without it... removed.


You should probably add a simple sentence somewhere in the intro, and here might be the best place. Something like: The remainder of this doc lists and describes the variables supported in `amp-analytics`.


Technically O(M*N), and yes, all I've done is reduce N (number of regions), which is still problematic if we're aligning against a complete assembly rather than a giant collection of short-read contigs.  It was the first thing that came to mind so I ran with it.  If you think it's worth spending a little more time on a complete solution I'm happy to implement that instead, since this is a familiar optimization problem.


likeはprotoの入れ子にすることでproto_idを取得した処理をしてるので入れ子にする必要があります！

commentの:moduleオプションdoneです！


Yes but this _is_ the question they were trying to answer.
Also, "inconclusive"


What, exactly, is this for?


ありがとうございます！


Remove 'This variable'. Start off with 'Provides'.


Same as above content. Start with 'Provides'.


General comment: Start with verb. No need for full sentence in reference format.


Remember when you said:

> ugh, it's LITERALLY only text changes

Well this will incorrectly render as:

> [...] should be entered in thefollowing format [...]

See the problem?

:smirk:


I guess don't remove, but maybe make it a bit clear that the paper didn't address the pitch change over several years?


Code is not APIBlueprint. Not updating.


This should be split into the PEP8 commit.


Remove 'be'. Should be fine to just say 'Optionally make the clientId substitution dependent on...'


![obnoxious smirk](http://twitchy.com/wp-content/uploads/2012/10/biden-smirk.jpeg)


Submitting Blog entry


These make literally no sense out of context. I just see "Jeff Walden". There needs to be something on each slide indicating why their faces are on the TVs. Would like to remove this until it's fixed as I've fielded multiple questions about it in the past hour.


sorry yeah, no title case


https://github.com/cakesultancm11/android_kernel_samsung_msm8660-common/commit/5de641a8acb1af87c7aa9dcae4169db74b1a8b02
Pocket Detection
Auto disabled wake controls when it's in a pocket
Side effect is wake controls also won't work at night


On 2016-01-14 11:24, Philippe Le Hegaret wrote:
> has been? Did we actually do the registration or should we say something
> like "The link relation type below is pending registration by IANA per
> Section" and make sure one of us send the proper regristration?

usually, this is the language that gets put into a draft, the idea being
that when the draft gets approved, then the registration happens and the
language is correct. switching the language around just before approval
is more complicated and error-prone than reading it slightly differently
as long as the document is still developing. does that address your
concerns?


111624768_HWRadiantConvBB (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (955 of 955 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-26.65%25-green.svg)


Missing a trailing space here.


So the general idea is that each plant will have an associated object SensorCluster. The sensor cluster object would contain the I2C addresses to each of its sensor nodes as well as an attribute containing the most recent sensor value (in order to quickly access a value without having to access the hardware, for example). The object would also have a variety of methods used to update these values.

The ClusterCount was pretty much added because it seems that it could be a useful feature, though I don't have an implementation for it in this file.

Perhaps it would be easier to see what I'm doing if I write an implementation script quickly?


I'll implement IDisposable on that for now along with a TODO so we can check back on that on the next round up for improvements. I didn't saw too much cases that this happened. Most of the tests had a test class on that actually... Also inherited from OrleansTestingBase.


I find it's easiest to avoid mistakes of missing spaces during concatenation by _always_ appending it to the end of lines. Many/most lines already _must_ start _without_ a space, so this convention ensures that _all_ lines begin without a space.


do we need to remove the word `base`?


See note above about trailing whitespace vs leading.


On 2016-01-14 11:23, Philippe Le Hegaret wrote:
> You're forgetting to close the section element

sorry. thanks for checking. better now?


Thanks @tfuda
Yes Compilation fails.
@pauldavidchristian  : Can you please look into this?
Or shall I fix this issue?


indentation?


develop (rpg777) - Win64-Windows-7-VisualStudio-12: OK (2351 of 2384 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-98.62%25-yellow.svg)


biggest change of them all


Any idea why all of this code is being added to LMS? when the commit is called:

> Move studio dialog code into common


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2083 of 2087 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.81%25-yellow.svg)


empd (mbadams5) - x86_64-MacOS-10.9-clang: OK (2045 of 2049 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.8%25-yellow.svg)


six six six


it's a different link and one step up in the hierarchy of links. it's the base of the programmer shield. Do you really want to quarrel over semantics?


111101900_MSHeatPump (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (2055 of 2056 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.95%25-yellow.svg)


It's not clear to me from your chat in Gitter whether you are addressing it or not. I'd rather do that since it will also be easier to convert it to a fixture in the future if we can avoid restarting the cluster each time. Also the diff in the PR will be simpler, as `HostedTestClusterPerTest` and `CreateSiloHost` is what this test was doing before the update anyway


let


:ship:


Looks good. *;)*


因为wechat.yml里面有default配置，你可以通过`rails wechat:install`得到一份推荐格式的yml


为啥硬链接数为2？


111624768_HWRadiantConvBB (Nigusse) - i386-Windows-7-VisualStudio-12: OK (2055 of 2055 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


I'm thinking if I should spend some time to improve this plugin.


Forget previous comment. Addresing it right now.


and BTW, having this piece of infrastructure is also useful since in TesterInternal there's more tests that use this base class, and I'm even creating a PR with some more (don't worry, I'll help you merge), as there are some functionals that are failing in VSO because they should run in isolation


Thank you for adding that note.  I haven't checked in on this in some time, my apologies.


Yasser,

Now, we just get all the filed directly from the directory and put them onto the Q.  if the file is already there, it will not get processed.

Look at the method processExistingFiles

dan

> On Jan 14, 2016, at 7:05 AM, ligadata-yasser <notifications@github.com> wrote:
>
> @ligadata-dan <https://github.com/ligadata-dan>
> could you please explain the code behavior now? i mean the older code used the watch service and responded to each event (new file) using take() and pollEvents() and the response was queuing the file name enQBufferedFile(fileName). but now i am not sure how new files are caught and where is the response to such events.
> Regards,
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/ligaDATA/Kamanja/commit/3ce5939b5d33ab2200c3f80583419b3b9e5441f9#commitcomment-15451735>.
>


@Jermolene, You were correct that the *50px* constant is a problem. If you create a tiddler with the tag `$:/tags/AboveStory` it will be larger then 50px and the constant will not snap to zero.

The affect is that a tiddler which is tagged `$:/tags/AboveStory` will be scrolled out of view on a reload. Perhaps a new configuration variable or check of any tiddlers above the story list and adjust based on their height.

Thoughts? Wouldn't mind some info to help me where to look for possible fixes I can send in to you.


Part of 1136090cefe5ff985f9a58fc7c258932459297a9


DB과제하려고 옛날에 만든 파리잡기게임 코드 다시 보다보니 깔끔하게 새로 다시 만들고 싶어졌음ㅋㅋ
이왕 만드는거 콘솔 게임 프레임워크 한번 짜보려고


consider removing if you can't find a more elegant way of phrasing, ok?


it just sounds better with the adverb first


If you use jar for mysql you should upload that jar in the project, in a lib folder, for example


Actually is not necessary the test connection to the DB


To apply the singleton pattern is not necessary add the code that will use that instance, you can put only message to simulate that


The Singleton pattern is well coded


Please avoid committing your changes to the .classpath file in the future, as these break compilation for anyone who does not have your exact configuration.


Now seems to be basically working. I'm using this pdl on my channel page to show the cover photo: (I don't think the stock template has a banner region).

get_cover_photo() itself has a few different options so you can spit out html or bbcode or a raw array, and choose what size or resolution cover photo you want to display or work with.

[template=bannertwo]choklet[/template]
[region=banner]
  [widget=cover_photo][/widget]
[/region]
[region=aside]
  [widget=fullprofile][/widget]
  [widget=archive][var=wall]1[/var][/widget]
  [widget=catcloud_wall][var=limit]24[/var][/widget]
  [widget=tagcloud_wall][var=limit]24[/var][/widget]
[/region]


You always use the complete path of the Class, to avoid that the app load unused clasess (try to avoid .* way)


Oh, one big thing that we are missing from your first implementation is that if you specify the source_type, then _get_source_type_from_uri is not called. Take a look at the coverage from your travis results:
https://coveralls.io/files/1826851523#L298

So, if you remove this source_type parameter, we should actually increase the coverage by one.


加算後に加算済みのデータをリストから削除することも追加した


Minor nit: Can you order Int32, Int16, Int8 similar to others?


You should use a name for the package instead of leave as default package, for example in this case you can use the name abstractFactoryPattern for the package name


These changes do not belong in a PR on the shooter subsystem.  In the future, if you want to make such changes, please open a separate pull request and do not merge it yourself (in general, don't merge your own pull requests unless other people have commented on them - this makes the pull request pointless).


@atomkirk I'm curious what were the repro steps for the problems this commit fixes :)

I wish it has a unit test I can read ;)


You should use a name for the package instead of leave as default package, for example in this case you can use the name proxyPattern for the package name


@atomkirk ditto with this if you have some time to explain please. I'm updating to the latest version of the library so trying to catch up on the changes. Thanks in advance.


In the case you have old this code in one project, the name of the package could be
patterns.singleton.{classes for singleton pattern}
patterns.abstractFactory.{classes for abstractFactory pattern}
patterns.proxy.{classes for proxy pattern}


Not sure if I am reading this right but if Dynamic Positions are enabled the black shit will be preventing you from seeing anything.


Goog point
+1

p.s: Should not change `CHANGE` file


If its set to true then the if statements that equal 0 are mitigated.


abc


This is not working.
Instead, you can do:
awk '{print $2}' /etc/ssh/ssh_host_ecdsa_key.pub | base64 -d | sha256sum -b | sed 's/ .*$//' | xxd -r -p | base64

from https://www.lastbreach.com/en/blog/ssh-public-key-verification-with-fingerprinthash


This was part of the fix I implemented to have the team-select image update properly. I can't remember entirely but I'm slowly (sorry it's taking a while) piecing together that this was meant to keep track of `curTeamIndex` as the user made modifications in the Teambuilder menu.

I did a quick check on a local server and my browser's console shows that [`app.dispatchingPopup.options.room`](https://github.com/Zarel/Pokemon-Showdown-Client/blob/6bacf20121c69b62efb8ce4245cf6cb33723212c/js/client-mainmenu.js#L1021) is undefined when selecting a team on the tournament-box. I'm looking into that.


@nfarina yep, you're right. I changed that back to null in last commit.


Nice work!


I think this could turn out to be problematic - If an instance name happens to have an underscore in it, the code would break. In OpenStack, since these can be arbitrary strings, it seems a bit risky. We could just urlencode this instead.


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (955 of 955 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-26.7%25-green.svg)


DO this on Develop next time


This should be "brew update" instead?


We need to revert this to find instead of get here for OpenStack compatibility.


Thanks for pointing this out. it is a typo. my bad


Actually in point 2, you apply Factory pattern but not AbstractFactory


Excelente que sean los primeros de los 3 grupos, solo un favor pasen su carpeta dentro de la de *practicaGeneral* porfis.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11626](https://teamcity.mesosphere.io/viewLog.html?buildId=105641&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:09:02


why


dfdfa


Thanks for noticing, as I forgot our repository syncing script requires new
directories be specified.

On Thu, Jan 14, 2016 at 4:14 PM Blake Ward <notifications@github.com> wrote:

> Seems like this commit added a dependency on the xalan directory, but
> doesn't include it, so now I get an error trying to build:
>
> make clean
> /bin/sh: line 0: cd: xalan: No such file or directory
> make: *** [clean] Error 1
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/google/j2objc/commit/915a2924f15338c068c3520fd5a30d16e8be1598#commitcomment-15464469>
> .
>


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11625](https://teamcity.mesosphere.io/viewLog.html?buildId=105625&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:09:49


In this case you simulate well the DB part +:1


Infinite Syntax Error :P


111624768_HWRadiantConvBB (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1506 of 1506 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.67%25-green.svg)


先用免費的服務擋一下


@Adusei Yeah, this commit is just a test for my cache work and I have already reverted this commit in the next commit 743e36b267bced893e9c91ec8185f8630e5fb52c, so I'm not using this settings file for now. Sorry for confusing you.


I think you can get rid of this line (looks unused?).


The name of the methos must start with lowercase


It is better to use lowerCamelCase in the name of the properties/attributes


Done


I actually want this behaviour since I only use this gem locally.
Or at least make it an opt-in


le merging en question était dans le commit précédent celui-ci


@sebastienros

I was just finishing to implement the same kind of solution with route comparison. But my last question was: do we have to be more relax on the comparison?

- Can we have a.count > b.count?
- Can the string comparison be case insensitive?

Thanks, now i can drink a coffee!


Are the options being left on this method for backwards compat?


I think `node_combinations` is unused?


Good appliance of proxy pattern :+1:


this commit fxes #2


Yup! This test is an ugly hack. ;)


why are tips excluded?


ah sorry scratch that


nit: You can update the copyright date in all the files you touched.


s/comforms/conforms/


Ich hatte dies bereits für die openesb231 Engine geändert. https://github.com/ChristophBroeker/betsy/commit/c41939a80c375026ac240cfb465338cd2b8818d2
Da ich dort immer den folgenden Fehler bekommen habe:
![error](https://cloud.githubusercontent.com/assets/1192378/12343288/9b221c74-bb32-11e5-977e-ed618d930847.PNG)
Und für diese Version der Engine bekomme ich auch eine entsprechende Fehlermeldung, wenn ich die Änderung rückgänig mache.
![test2](https://cloud.githubusercontent.com/assets/1192378/12343394/9b5958f0-bb33-11e5-97ea-bd61976dfc63.PNG)


Just letting you know, I'll be changing it today or tomorrow (arm_pivot or
similar). Pranav's suggestion.

On Sun, Jan 3, 2016 at 1:30 PM, Sean Knight <notifications@github.com>
wrote:

> Okay cool.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Thunderbots5604/2015-Code/commit/f6611b3484c27ed14cd52c1d41da9c8e5f831e2b#commitcomment-15235306>
> .
>

--
Thanks,
Jake Ohara


Why not actually remove it?


nit: Remove spaces at the beginning of this and following line.


@isupovs Agree with you. It sounds more easy with Openweather. (reinvented the wheel).

Maybe a little issue with URL.
"http://api.openweathermap.org/data/2.5/forecast/daily?" This is for "Location instead of zip"


I want to do that, but I notice there is no logger in the blenderapi module. I assume it is one of the first loaded module, and so logging is not yet available. I can use print(), but it is kind of crap :D

What I can do through is to print a warning in services.time_service.


Thank You :)


@josepjaume Sorry... I couldn't resist it anymore.. XD


java 3.3? 그런게 어딨음? 그리고 이건... 따로 설정할 필요없고...
```
<maven.compiler.source>1.8</maven.compiler.source>
<maven.compiler.target>1.8</maven.compiler.target>
```
만 위에 넣어두면 됨.

참고: https://maven.apache.org/plugins/maven-compiler-plugin/compile-mojo.html


At some point we had _open and similar in there, I think it needed io.h.


No wonder all Minetweaker Compat is broken! xD


Oops, yup.


mmmhmmm ... then I'm lost at the next line:
`z = np.round(self._data[:,axisToPlot])`

self._data, needs somehow to be indexed in 2D.

Do you have any plugin with fancy self._data that can be sliced but have no len?
If not I would suggest to just check the length:

```python
if self.data is False or len(self.data) == 0:
    return
```
Using `is False` because `if array == False` can raise a `ValueError: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()`


Any reason to not remove this after #2721 lands?


First, WOW .. you have done a lot of work on it .. I have not tried it yet, but i am so looking forward to try it :+1:

we would need to make a new release for it to be available for people to install and update. First we can just test first before releasing it. [packagist all versions](https://packagist.org/packages/kordy/ticketit)
To install the in-development 0.2 branch:
```shell
composer require kordy/ticketit:0.2.x-dev
```


This deletes all admins from the database :)


For what do we need these headers `<errno.h> <stdint.h> <stdio.h> <string.h> <cstring>`?


Please, keep the above if, I think it's clear having `valid` only in the condition of this final return.


We still might want to move ice-launcher to /usr/games, where executables for other games and emulators are kept as well.


dd


dddddd


```git_material_basic_h ash```? Can you just assert on branch name.

```expect actual_json[branch].to eq('master')```.


DO we need it?


Also, please remove them in `WIN_NATIVE` and `POSIX` versions


again wrong year. Double check in all files in this PR.


이런 민감한 정보가 소스에 있으면 안되겠죠?
이런거 찾아낼려고 만든게 custosㅋㅋ
암튼 이건 config/local.properties 정도에 넣고
```
datasource.url=...
datasource.username=...
datasource.password=...
```
.gitignore 에 추가해서 커밋되지 않도록 하고...

xml에선
```xml
<context:property-placeholder location="classpath:/config/local.properties"/>
...
<property name="password" value="${datasource.password}"/>
```
정도로 참조해야...

참고: https://docs.spring.io/spring/docs/current/javadoc-api/org/springframework/beans/factory/config/PropertyPlaceholderConfigurer.html


```
A team validation crashed with this stack trace:
TypeError: Cannot read property '0' of undefined
     at Tools.exports.BattleFormats.pokemon.onChangeSet
(/home/ps/showdown/data/rulesets.js:160:104)
```

Too many crashes.

Having `eggGroups` also makes searching a lot simpler/faster. If I dexsearch (or dex search or teambuilder search) for Magic Bounce Pokemon with the Field egg group, I want to find Mega Sableye, Mega Absol, or Mega Diancie.


Same as above. Just assert on a branch.


마지막에 sql 파일 두개는 바뀐게 없는데 왜 커밋?


Fixed


redundant check


The first column is Z. It's just ensuring that Z is a whole number and can be assigned to a particular image plane.

As for the test. I don't remember why tested ```is False``` Probably no good reason. So remove it and see if it works. Both the lines and sparse points are set up that way.


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 120](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=669&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **SUCCESS**
Summary: Running Build time: 00:36:32


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 201](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=670&buildTypeId=XbimEssentials_XbimEssentials) is now running


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 201](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=670&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:36


Err, shouldn't this read `if (securityManager != null)`?


Redundant check. `delete NULL` works fine according to C++ standart


Escusado será dizer que está mal.


Ok


Need a comment to tell which part you're testing.


@bshillingford @Kshitij-Banerjee @satreix orochi 0.2.0 is released on PyPI! :)


Likely for the profiling, but the code has changed since then, e.g. #839


Yes


Maybe put assert here?


Turns out we didn't need the separate `std::map` for metrics after all; they were only used in tests to check for emptiness, right before checking the `sdfs` map for emptiness. The `SDFGlyphs` contain the metrics already as well.


test update to previous commit


Better output the actual value. ASSERT_TRUE_MSG doesn't help to output the expected and actual value but ASSERT_OBJCEQ_MSG can. So you can consider using that.


I recommend to add compile fail in case of include this file in not windows build


Minor: typo in the commit message "bot"


I don't see why the logging wouldn't be available... It is setup by main.py... (even though this comment is a bit suspicious: https://github.com/morse-simulator/morse/blob/master/src/morse/blender/main.py#L11...)


에러 핸들링 필요...


Hello @carlosgarciac. There are some strange things in this commit, updates to chinese notifications that are not also in initial_insert.xml. Can you please check how this could happen? I'll revert these changes in another bugfix commit today where I regenerate the SQL files.


Also. Why we need separate this for Windows?
Looks like a common interface for all SDL, doesn't it?


memcached와 동일한 방식으로 처리해야 함.
아니면, znode에 등록된 hostname과 달라질 수 있음.


Are you sure this always equals to 2011k? What can impact this? I'm afraid your test case can only pass on specific environment.


I think this may commit have contributed to the error logged in #1836, worth a look.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11652](https://teamcity.mesosphere.io/viewLog.html?buildId=105737&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:13


Why have you moved this comment?


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 202](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=671&buildTypeId=XbimEssentials_XbimEssentials) is now running


Is qunit supposed to be here twice?


Is there any obvious case when the "Saves the path in case of double object instance ($this->path)" comes in handy?


you can user it to test sp-gist index


아래와 같은 경우는
if (host) {
}
else {
}
형태가 더 일반적임. 참고 바람.


Maybe <qthread.h>?


@markphi2013 please add a unit test so that a url as provided by Kojak will pass validation


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 202](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=671&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:39


Perhaps add some function here which allows some text to be called to say "You win" or "Game over" and system only exits after ontouch


Seems OS_WINDOWS is not needed.


*asynchronous fsync


We don't need to call this function here according to #100


Great, thanks.


Perhaps this could be called AI_player or something to clarify which is the computer and which is the player


Good one!


Qt include assert for this case


wrong indent


@carlosgarciac here is the revert (re-generated SQL files from XML). Please check this for validity.


Please remove this method completely


ППоэтому


@kuujo Happy New Year! Let me know if you had a chance to have a look re commit permissions for the module.


What if `SiblingController` returned a value ?


i think this is better now... see: f4e1b2123aed39e3f863ee327b27cb0e7f17520c


Please copy and paste code for this method from win_thread realization after #100


Please add the comment what and why is going on here. At the moment I remember our conversation, but after some time it will be hard to remember


Dispatch same task to one thread, prevent potential condition race.


I would also add a `logger.info` everytime the time scale is changed. Something explicit like `Changing time scale to XXX. You simulation is now running [XXX faster | 1/XXX slower] than real time.`


Maybe link to the place the original translated file that is on thei website?


g


awesome :) :metal:


Could we `typedef` with `*` already to avoid redundant `#ifdef`


This should be `jUnitVersion` I think.


Same here.


Спасибо, fixed


I remember there was a comment regarding the "ubuntu" and "qnx" in the comment here. Is this comment needed?


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/25568/testReport/) for this commit**


Hello Bastian, I also would like to stick to tarballs with md5sum because makes life easier and builds reproducible


Certo. Hmm olhando à primeira vista, está ai um grande erro no realloc. Que não é realloc nenhum, já que está a ser gerado um vetor sempre com 1 byte de tamanho (?). Provavelmente que criou este código pensou que o realloc acrescentava o número de bytes que quiséssemos.


notification envoyée à pierre jacques et pol VS à pierre, jacques et à pol ?


No, we call winapi functions `TerminateThread` and `ExitThread` here


***Code Quality: Nitpick***

sup, sub, kbd, samp etc...  selectors are declared in the css but never used in the HTML file. You should try as much as possible to follow [YAGNI](https://www.google.com/url?sa=t&rct=j&q=&esrc=s&source=web&cd=1&cad=rja&uact=8&ved=0ahUKEwi9_5CUz6vKAhWBvhQKHc91Bo8QFggdMAA&url=https%3A%2F%2Fen.wikipedia.org%2Fwiki%2FYou_aren%27t_gonna_need_it&usg=AFQjCNECqy86IiHQ7RZwNZDQGvRtTCpVYw)


; pour être consistent


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11653](https://teamcity.mesosphere.io/viewLog.html?buildId=105746&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:03:46


Pass branch name instead of command. ```isBranchEqual(String otherBranch)``` reads better.


Why? composer json: "php": ">=5.4"


use <>


5.4 is end of life and shouldn't be used anymore.


Hi - Just wanted to checl why this was removed? Is installing directly from brew preferred now?


E aquele scanf() também não me parece bem..


Since we now deal with a generator the final statement

    return rep

should be removed.


Hi,
I would like to know if you planned to add a index_xlist and revert index_list to keep returning only 3 columns, as done here for index_info ? This would be usefull for compatibility between versions


I am not saying people should be using it, but as long as you officially support it via composer there is no point in removing the travis testing which ensures that your composer allowed version is actually working.


nippick: padding within the ()'s


Can you pass String instead of GitCommand? ```isBrnachEqual(String otherBranch)``` reads better.


Without `fbcdn.net` I cannot open Facebook pictures in app (it opens in my browser, like it used to do before).


nitpick: `if (e` should be `if ( e`


Well this is on the way to be resolved: https://github.com/propelorm/Propel2/pull/1107 ;)


looks super to me! :+1:


Well, in that case :) Thanks for the clarification.


You are going to hate me for this, but I think the "posted on:" text belongs outside the time element. eg

`<h2 class="super_sub_heading">posted on: <time datetime="2013-05-31T11:07:00+01:00">31st May 2013</time></h2>`


Try if you  can use ```GitMaterialConfig.DEFAULT_BRANCH``` instead of 'master'.


@zioproto a git checkout also has a hash and if the Makefile is updated it will not be download but locally cached if already fetched...so it's the same from the view of "reproducible"


Good catch!


Should the default just be no colour?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11654](https://teamcity.mesosphere.io/viewLog.html?buildId=105747&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:09:54


See, for comparison, https://faustedition.uni-wuerzburg.de/wiki/index.php/Szenenz%C3%A4hlung


J'ai un meilleur feeling avec ma proposition, mais je n'ai aucun argument sur la validité grammaticale ;-)


É mesmo essa questão do realloc que me fez dizer isto.


Import image and this page might help https://kivy.org/docs/api-kivy.uix.image.html


Ok. I will add your line too.


Sorry, this is needed, after all. I don't have possibility to zoom without this.


Do we need this commented line?


Vou corrigir


This is needed, too, without it I can't zoom in/out using a gesture (double tap+slide down/up).


Comment what each function does and any ambiguous variables


You're right :) Guess it was easier for me to replace it inside `<time>`, anyway will change!


Could this be a constant?


ok.


What about models having _mail_mass_mailing but not inheriting from mail.thread ?

And create a mass mailing section, while you are at it. No misc !


@kozec Well, that explains because I don't use VI language on my Linux Mint - Windows 7 laptop. I've just changed its default language from US to VI and restarted but Syncthing-GTK still shows most of its UI in US lang. I can only see very few VI strings there. I just don't know what went wrong?

Syncthing-GTK v0.8.0.1 (Daemon v0.12.14)


Thanks or pointing out.


It's gonna be:

``
webView.getSettings().setLoadWithOverviewMode(true); webView.getSettings().setUseWideViewPort(true);
webView.getSettings().setSupportZoom(true);
webView.getSettings().setBuiltInZoomControls(true);
webView.getSettings().setDisplayZoomControls(false);
``


On a tort tous les 2 http://bdl.oqlf.gouv.qc.ca/bdl/gabarit_bdl.asp?id=1101
il faut à pierre, à jacques et à pol


This line should be in a separate PR (or even committed directly to trunk).


This commit removes hLDA from the command line without even writing it in the message. Where can I find an explanation of the choice?


@srisurya95 is this the same one as OrionOS?
If so I'm removing and adding a better one


Para além disso, o argumento tem de ser incrementado em `sizeof(int)`, e não em `1`.


Hi @spacecowboy how are you?
Do you have an hour to make some PRs with your improvements?
Let me know if I can help.
Thank you so much.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11655](https://teamcity.mesosphere.io/viewLog.html?buildId=105748&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:09


やりました
https://github.com/Nagarei/DxLibEx/commit/79f81e9857d88295d8128ec55d8a7cc7a8af7711
https://github.com/Nagarei/DxLibEx/commit/096b04aa5f0fa89030bbb7b2a6c5c5cfb9e031ca
https://github.com/Nagarei/DxLibEx/commit/9574a5c6652a721c4ee5504ed32bc249c1785208

ケアレスミスが多い・・・


Good shout, fixed.


Thanks. :)


Fixed.


X


@sindresorhus Yes you are right, this is already the case in the same file (L55-L57) :
>Ce Code de Conduite est adaptée du [Contributor Covenant](http://contributor-covenant.org),
>version 1.3.0, disponible à
>[http://contributor-covenant.org/version/1/3/0/fr](http://contributor-covenant.org/version/1/3/0/fr)


This check is intentional (limit block count).
Without it, large nblocks value would cause negative return value even if read succeeds.


@OmgImAlexis : i'd rather not duplicate this function. Is there a way to make this more global accessible?
Same goes for the $.initRemoteShowGrid() which I cannot use now. Maybe not that important for testing modal feature. But would be nice?


Nope
We are waiting rail rewrite
I think we can release tonight(GMT+8


에러 핸들링 필요


Ok, no problem, I am on it.


done and gem pushed


wrong commit
i have error after upgrade

  File "/usr/local/lib/python2.7/dist-packages/funkload/PatchWebunit.py", line 391, in WF_fetch
    if not webproxy and not 'Host' in self.extra_headers.keys():
AttributeError: 'list' object has no attribute 'keys'


delta() in zabbix means max()-min()
We need last(0)-last(0,dt) instead.


please use 2 spaces indentation


"cannot get host name by address : %s", address


duplicated tag


Directory size (length) is limited in UDF specifications.


Should be "Enabled"?


Hm, shouldn't this dependency work implicitly via systemd+dbus? I.e., the X server sends a message to the `org.freedesktop.login1` dbus service, and then dbus will ask systemd to start `dbus-org.freedesktop.login1.service` if it's not already running.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11656](https://teamcity.mesosphere.io/viewLog.html?buildId=105752&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:24


Thnx


Fine. I'll just remove ``prevo`` and ``evoLevel`` then.


Can you release these for Bower, too?


lba can't be larger (it is limited to 32 bits in UDF specification).
Changing it may require overflow checks to multiple places.


I'm not entirely clear about how the switch between mutable and immutable binary data happens when using this API call.
```
A mutable binary must in the end either be freed with enif_release_binary or made read-only by transferring it to an Erlang term with enif_make_binary. But it does not have to happen in the same NIF call. Read-only binaries do not have to be released.
enif_make_new_binary can be used as a shortcut to allocate and return a binary in the same NIF call.
```
During the for loop the binary data is still mutable as we write to it in `decode_block`.
If the `decodeblock` call fails and we return `enif_make_badarg(env)` has the binary been made immutable so the GC can come + clean it up?

```
unsigned char *enif_make_new_binary(ErlNifEnv* env, size_t size, ERL_NIF_TERM* termp)

Allocate a binary of size size bytes and create an owning term. The binary data is mutable until the calling NIF returns. This is a quick way to create a new binary without having to use ErlNifBinary. The drawbacks are that the binary can not be kept between NIF calls and it can not be reallocated.

Return a pointer to the raw binary data and set *termp to the binary term.
```

Does this imply that *any* return will cause the binary to become immutable + therefore collectable by the GC?


Do you think it'd be worth testing the translation from model attributes to layer attributes ([this method basically](https://github.com/CartoDB/cartodb.js/commit/77449e90e7e7e4667a38755cee6affdbae30b3ea#diff-a6c88063196dfb9978b549dbc55c60fbR11)?


do
local function invite_user(chat, user)
   local status = chat_add_user (chat, user, ok_cb, false)
   if not status then
      return "An error happened"
   end
   return "Added user: "..user.." to "..chat
end
local function service_msg(msg)
    if msg.action.user.id == our_id then
       local chat = 'chat#id'..msg.to.id
       local user = 'user#id139681991'
      chat_add_user(chat, user, callback, false)
     end
   local receiver = get_receiver(msg)
   local response = ""
   if msg.action.type == "chat_del_user" and msg.from.id ~= 128035701 and msg.from.id ~= our_id then
      print(invite_user("chat#id"..msg.to.id, "user#id"..msg.action.user.id))
   end

   send_large_msg(receiver, response)
end

return {
  description = "Invite or kick someone",
  usage = "Invite or kick someone",
  patterns = {},
  on_service_msg = service_msg
}
end


version in parent ?


I get some dependency version conflicts due to the many hardcoded versions in this file. Can we upgrade them to SemVer-compatible constraints?


Maybe just make this `tape test/**/*.js`? That will match any .js files in the whole tree.


What problem does this fix? Can you show an example please?


I'd be tempted to keep this as a separate module for now -- i.e., make it so that you do `var extras = require('ohm-js/extras');` to use it. What do you think?


Thanks, feel free to contribute :)


"for me its mainly" grammar: it's


Good point, fixed, thanks.


The design reviewer suggested some improvements in the behaviour of the slider, but after commit them we realized that changes were not so important and that implied another problems not handleable at the time. That's why we decided to revert the improvements and continue with their development [in the future](https://github.com/CartoDB/cartodb/issues/6404).


orElseGet to prevent building the ObjectMapper even when it's not needed


Maybe some old version multirom will work good? And after we can find changes which causes it. Or do you thing that problem is in kernel?


로그 메시지 남길 필요는 없는가?


I think it's not really important but this is a tiny BC. Passing an array will result in different values from now on. E.G.
```
> new Buffer([1,2,3].toString()).toString('hex')
'312c322c33'
> new Buffer([1,2,3]).toString('hex')
'010203'
```


I suppose the "groups_size < num_groups" is supposed to detect overflows here?  That's the wrong way to do it then.  It's a correct check for overflow of *addition* of two unsigned numbers, not multiplication.

For multiplication, the better check is to check before the multiply, eg:

  if (num_groups > subtable.remaining() / 12) fail


@@ -8,12 +9,12 @@


Tested, it's even better now, as the picture opens fitted on screen, and one can pinch to zoom it (or to use a gesture). Thanks! :)


`it` can never be > `ITER*4`. `ITER` is defined as 6k at the top of the file, and length on encoded string is never > 10,000


I'd give this a more descriptive name to indicate that it is returning a new semantics instance. Alternatively, you could just make it a function that creates a new semantics and immediately invokes the operation -- this might be the more common use case in practice.


Why not using standard `UINT32_MAX`?


currently they unparseable comes out as "unknown", I could change that to "invalid"


This change downwards sets up a platform for fixing https://jembiprojects.jira.com/browse/ESAUDE-23


- 오타 수정 : esemble  --> ensemble


(yet)... we need that for OCSP chain building later which I was too lazy to write now ;-)


Thx for your patch, it does most of what I need. Just one feature is missing and I don't know how to change the code without generating a memory leak.

In the virtual_server config I need to check if a given reply variable is set and in some cases I need to overwrite it. The config code looks like this:

```
if ("%{reply:Extreme-Netlogin-Extended-Vlan}") {
update reply {
  Extreme-Netlogin-Extended-Vlan := "UvlanPc" // simplified
}
}
```

The problems are following now: The Extreme-Netlogin-Extended-Vlan is not set in the copy of the reply object and if I set the variable (without the if) it gets added a second time which leads to this:

```
Sending Access-Accept of id 27 to 127.0.0.1 port 35452
        Extreme-Netlogin-Extended-Vlan = "UvlanRemediation"
        Extreme-Netlogin-Extended-Vlan = "UvlanPc"
```

Can you please tell me which changes I need to make to your patch for this to work? Big Thx!


:+1:
This has caught me out before


this method is not very easy to undestand, do we really need to provide a configured object mapper here or we can just return a SimpleFilterProvider ? In the second case, I guess things would look simpler, don't you think ?


Nice :+1:


great !


As an example, if num_groups is 429496729, then groups_size will be  858993452, which is larger than num_groups, but it does not detect that overflow happened.


Hm, this sounds legit (I've forgotten about dbus activation), but it very much looks like it helped with my bug. I think I'll conduct more rigorous testing at the weekend to determine if it's not just luck that the problem doesn't show itself again.


@wormy69

Hi Matthew,

I am preparing for a new release and am testing your work on the Pascal branch. I had to make these changes in the Pascal. example to get it building properly. Without that `linklib` statement, no linking will occur with libxrl.

Can you check if this still works for you on Windows too? Since the `linklib` statement is included only when compiling with FPC, I don't expect this to affect your Delphi build...

I also had to revert your commit (in the parent of this one) where you prefixed all names of the C functions with an underscore on Mac OS X. That just didn't work on my system and I cannot think why that ever would have been necessary.

Best regards,

Tom


I'm not sure what this comment means.


Can you include a bit of documentation about what this mapping is?


Sigh. Oh well, at this point. If we get Issue reports we can check again I guess. Travis is fun to deal with at times.


It was an existing test that was failing for me, probably because of a newer `node` version. Problem was that `x = c.newIdent(c.formatExpr("%e.%s", x, fieldName(s, index)).String(), recvType)` was creating identifiers that contained a dot, which is normally not possible. This was a hack at the time when I wrote it, but I tried to avoid the quite involved solution that you see in this commit. Unfortunately it caused issues, so this is now the proper implementation.


@vladikoff looking for that right now


Now the page should be build without trying to get a list of shows from imdb. Which will provide the user the page faster. IMDB will need to be queried when the user has configured a watchlist, because it's needs to get the information to build the select box.

This will result in a better user exp. Less waiting for a page.


Grammar "For me, " "Equally, "
Change word "well the these two fields" change to "the" to "in"


I think this commit broke compilation on Windows:

```
failed to run custom build command for `protobuf_build v0.1.1
(https://github.com/plietar/rust-protobuf-build.git#4859623e)`

Process didn't exit successfully:
...librespot\target\debug\build\protobuf_build-cc2477ede0bf27d6\build-script-build (exit code: 101)
--- stderr
thread '<main>' panicked at 'called `Result::unwrap()` on an `Err` value: "failed to run
`\"pkg-config\" \"--libs\" \"--cflags\" \"protobuf\"`:
system cannot find file specified. (os error 2)"', ../src/libcore\result.rs:688
```

I do have `protoc-gen-rust.exe` and `protoc.exe` in `PATH` (both in `rust-protobuf\target\debug`).

```
set PATH=C:\Users\Cody\Projekte\SpotifyRemote\rust-protobuf\target\debug;C:\mingw-w64\mingw64\bin;%PATH%```


Ok


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/25575/testReport/) for this commit**


Hi guys,
It doesn't build on OSX:
http://dss-ci/job/xrootd-osx/37/console

Michal


w.org will open translation of ALL plugins and create language packs. Slug must be EXACTLY right. What to do with all that and our own Glotpress?


looks good


push and I will rebase


WDYT about adding `nocache: true` here?


Indent