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


Yes, I was looking for backwards compatibility, maybe a parameter "mode" which extends this behaviour would also do the trick. One other glitch in the rc is that it's using the interface name of the first interface as a pid, which might interfere if you change interfaces along the way...

Any suggestions for a more permanent solution?


While this appears to fix the immediate issue with showing types properly on the package pages, I wonder whether it introduces a latent bug. Namely one analogous to: https://github.com/elm-lang/elm-compiler/issues/1076#issuecomment-145193829. If anyone ever uses the `Type` parser from here for another purpose, they may get data like `Apply name []`, which according to your comment over there breaks an invariant.


Fails to compile on Linux
(Linux Omega 4.2.5-1-ARCH #1 SMP PREEMPT Tue Oct 27 08:13:28 CET 2015 x86_64 GNU/Linux)

>/ISCSI/evemu_server/src/eve-core/utils/misc.cpp:87:14: error: ‘::_fileno’ has not been declared
     ::fstat( ::_fileno( fd ), &st );

Removing eve-core/utils/misc.cpp's changes fixes the problem


Why do this?


These code does not raise error on non-exist volume, please
use the code in `volume_delete()`:
```python
        v = _get_item(self.c.volumes(), args.vol, "Volume")
```


test


А вот это больше на строчные похоже. И кнопками вроде как логичнее было бы сделать, не?)


There is no need display a empty list. In `volume_enable()` we
just return with void.


```JavaScript
$('#wday_edit').css('display', 'block');
$('#date_edit').css('display', 'none');
```

似たような処理を何回も書いてるのは、無駄なので関数にしてしまいましょう！


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


I haven't tested this yet, but if this is PCRE regex, then `^` means the beginning of the string (likely line assuming that this scans line by line). So I think you probably want `/^$ ` (a `$` at the beginning of the line followed by a ` `).


I'm getting a 'excess elements' here. Removing "clUnknown165" helps.
Seems buildbots don't catch this. Am I the only one who sees this?


I put priority to -2 because maxAdditioanalDepth = 3 looks more promising and I want to see what is going to be with rohanryan's patch.


Well done !


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (894 of 894 tests passed)

![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-25.13%25-green.svg)


Documentation says they support `query` argument. Looks like this broke that. :/


I disagree with this. I would prefer not to install things -g. For grunt, gulp etc I deliberately use aliases to run the version inside node_modules so I don't have versioning problems when working on many projects.

In my opinion JS projects should not require any external install steps beyond Node, but if we do do it this way perhaps we should add a short note to clarify in the contributing doc.


Take a look at the pull requests. It explains the changes.

-----Original Message-----
From: gokumar [notifications@github.com]
Received: Wednesday, 18 Nov 2015, 9:16PM
To: boundary/meter-plugin-docker [meter-plugin-docker@noreply.github.com]
CC: Gwartney, David [david_gwartney@bmc.com]
Subject: Re: [meter-plugin-docker] Merge with boundary repository (08680df)

David, why are there so many changes in this commit? Is there a major change or bug fix in this?

-
Reply to this email directly or view it on GitHub<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_boundary_meter-2Dplugin-2Ddocker_commit_08680df338a0e71b32621ec96b1b80918696fa90-23commitcomment-2D14491088&d=BQMCaQ&c=UrUhmHsiTVT5qkaA4d_oSzcamb9hmamiCDMzBAEwC7E&r=VYLpxxEyFilywMjzkEFVkrSzQubw2Nlv8iKEqudOLBs&m=Mny30WUsl-CG-ncHhbEKPNJJQ03tNhhgLZZ5iHzOgwE&s=yiksTpwxcr3vwr-zNauOXV6BjYTM6afvaeUaWL-v5yI&e=>.


`checkflag`はFlagですよね？
なのであれば、1のような整数で判断するのはかっこわるいので、真偽値型(true,false)を使いましょう。


@koobs Netmap mode in suricata uses "--netmap" instead of "-i interface" or "-d port". Currently that can't be configured correctly :(

I think the pid should be /var/run/suricata_netmap.pid so that it doesn't interfere with old pid layouts, or alternatively use /var/run/suricata_inline.pid as well...


I think it should be like
```
	switch(rand(1,10)) {
	case 1: itemskill AL_TELEPORT,1; break;
	case 2: itemskill AL_TELEPORT,3; break;
	case 3: percentheal 50,0; break;
	case 4: percentheal 0,50; break;
	case 5: percentheal 50,50; break;
	case 6: getitem Apple,1; break;
	case 7: itemskill ALL_REVERSEORCISH,1; break;
	case 8: specialeffect2 EF_MAPPILLAR2; break;
	case 9: specialeffect2 EF_ANGEL2; break;
	default: specialeffect2 EF_COIN;
	}
```
the one in the commit means
10% chance condition to cast random warp
9% chance condition to cast warp to save point
8.1% chance condition to heal 50% hp ...
and the value go lower and lower until the last else statement suddenly have a bigger value

using switch rand() should have all conditions at 10% chance


А где крестик для удаления изображения? И нам в html5 не обязательно закрывать теги `<img>`, давай опять к однообразию причешем


SDF


Yeah I plan on doing that, grunt is actually made to run it's cli mode in a global install. I haven't checked yet to see if it runs fine without it. I know quite a few node modules do this and like you said it shouldn't be needed but it's how a lot of them were designed and changing it over isn't that simple for them. :disappointed:

If we can just use the standalone version I'm cool with that.


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


El delay entre ataques de una misma unidad parece que está por los 10 segundos en vez de los 2 como especifica la constante. Si alguno tiene alguna explicación para ello... es bienvenida.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.591.008ac8074659fabf26d59d329c34c01862f38661](http://build.palaso.org/viewLog.html?buildId=66364&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3323, ignored: 125 Build time: 00:07:39


Test 2


Осталось ещё выровнять всё, тк тревис будет всё проверять и будет ругаться) А я не буду принимать такие коммиты)


SDFASDF


DSAFSDF


wasn't sure whether i should make a new service, or make a fake ajax request or something like this http://stackoverflow.com/questions/23851424/how-to-detect-when-online-offline-status-changes


норм, только коммит бы и называть примерно так: «подровняли отступы»


このへんもtrue,falseをつかいましょうか


Nonono, I meant actually call `OreDictionary.doesOreNameExist(String);`. This entire method is written for you, it just didn't make the port to 1.8 so I had to use reflection to get the same effect.


про ссылку писал, оборачивающий див — зачем?


I think transform is (still) appropriate here


Мерджить в мастер — только из удалённого мастера, могут быть конфликты слияний, а это ещё тот головняк))


Please remove these escape characters "\", it will fix this error
http://i.imgur.com/qaFUlGv.png?1

' in file /tmp/vagrant-puppet/modules-76bee6c9d10deeda599d20f2249d671a/mysql/manifests/queryfile.pp at line 45
' in file /tmp/vagrant-puppet/modules-76bee6c9d10deeda599d20f2249d671a/mysql/manifests/queryfile.pp at line 46
' in file /tmp/vagrant-puppet/modules-76bee6c9d10deeda599d20f2249d671a/mysql/manifests/queryfile.pp at line 47

https://github.com/example42/puppet-mysql/blob/master/manifests/queryfile.pp#L44-L46


Also take a look at the meter-plugin-sdk-lua repos. That is where the lua framework is versioned.

-----Original Message-----
From: Gwartney, David [david_gwartney@bmc.com]
Received: Wednesday, 18 Nov 2015, 11:30PM
To: reply@reply.github.com [reply@reply.github.com]
Subject: RE: [meter-plugin-docker] Merge with boundary repository (08680df)

Take a look at the pull requests. It explains the changes.

-----Original Message-----
From: gokumar [notifications@github.com]
Received: Wednesday, 18 Nov 2015, 9:16PM
To: boundary/meter-plugin-docker [meter-plugin-docker@noreply.github.com]
CC: Gwartney, David [david_gwartney@bmc.com]
Subject: Re: [meter-plugin-docker] Merge with boundary repository (08680df)

David, why are there so many changes in this commit? Is there a major change or bug fix in this?

-
Reply to this email directly or view it on GitHub<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_boundary_meter-2Dplugin-2Ddocker_commit_08680df338a0e71b32621ec96b1b80918696fa90-23commitcomment-2D14491088&d=BQMCaQ&c=UrUhmHsiTVT5qkaA4d_oSzcamb9hmamiCDMzBAEwC7E&r=VYLpxxEyFilywMjzkEFVkrSzQubw2Nlv8iKEqudOLBs&m=Mny30WUsl-CG-ncHhbEKPNJJQ03tNhhgLZZ5iHzOgwE&s=yiksTpwxcr3vwr-zNauOXV6BjYTM6afvaeUaWL-v5yI&e=>.


スコープはfor文内なので、varではなく`let`でせいかいだと思います！


Yep, it works.  Just realized I never made an issue for the Z ordering weirdness with hardware GSDX so I'll do that when I get some good comparison screens made.


again, please rename the project :)


you don't need to commit this file


or this


Oh duh, thanks for the correction. I should have probably looked at it more than just blindly copy/pasting, huh? :P


v0.13.3 reverted above changes, but this changes still is needed for below case:
   before connection is established( when the REDIS_CONNECTED flag will be set), redisAsyncDisconnect() was called to disconnect.

In this case, onDisconnect() won't be called, so user wasn't noticed to release related data.


[Not new](https://github.com/syl20bnr/spacemacs/blob/develop/doc/LAYERS.org#cross-dependencies), isn't it?


definitely not this one


`:off`


could you fix the indentation, we use 2 spaces not 4...


Just checked and look like I was right about the globals. :disappointed:

````

➜  .build git:(develop) ✗ grunt
zsh: command not found: grunt
➜  .build git:(develop) ✗ npm install --save grunt-cli
sickrage@4.0.72 /Users/xo/code/SickRage/.build
└── grunt-cli@0.1.13

➜  .build git:(develop) ✗ grunt
zsh: command not found: grunt
➜  .build git:(develop) ✗ npm install -g grunt-cli
/Users/xo/.nvm/versions/node/v5.0.0/bin/grunt -> /Users/xo/.nvm/versions/node/v5.0.0/lib/node_modules/grunt-cli/bin/grunt
/Users/xo/.nvm/versions/node/v5.0.0/lib
└─┬ grunt-cli@0.1.13
  ├─┬ findup-sync@0.1.3
  │ ├─┬ glob@3.2.11
  │ │ ├── inherits@2.0.1
  │ │ └─┬ minimatch@0.3.0
  │ │   ├── lru-cache@2.7.0
  │ │   └── sigmund@1.0.1
  │ └── lodash@2.4.2
  ├─┬ nopt@1.0.10
  │ └── abbrev@1.0.7
  └── resolve@0.3.1

➜  .build git:(develop) ✗ grunt
Running "bower_concat:all" (bower_concat) task
File ./_bower.js created.
File ./_bower.css created.

Running "uglify:my_target" (uglify) task
>> 2 files created.

Running "cssmin:target" (cssmin) task
>> 1 file created. 10.84 kB → 8.29 kB

Done, without errors.
````


nice :)

please put it before (or rather after) the formatter block: Idea can format this correctly


Not all element types are present in all partitions. You want to check if `n` exists in `self._file` before this.


The standard XYZ format is definitely a good starting point. However, besides visualization I have seen that PIDs are needed for restarting a simulation (e.g. fastwriteXYZ). Whereas it will not follow the XYZ format. The question is if we could add a Flag to DumpXYZ so that it could work for visualization and also for saving a configuration and restarting a file.


ここfunction() {
}
だとうまくいかないかな？


What are you using a `NoopDeleter` here?!


Please, move the picture in https://github.com/netgroup-polito/un-orchestrator/tree/native_function_support/images (and then update the link the in README).


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


107310700-change-license-text (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-78.42%25-red.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-68.6%25-red.svg)


饶总测试


This is wrong. If it's a 3 argument version, there is no scalar in the macro.


Is the `const_cast` really required here?


NP don't want to break established practice here 😊 I must admit I was on a bit of a tangent last night experimenting with gulp and less because I was trying to find a nicer way to develop styles. Do you envision building that stuff here too?


Alternatively, use `window.medicmobile_android`.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.576.008ac8074659fabf26d59d329c34c01862f38661](http://build.palaso.org/viewLog.html?buildId=66365&buildTypeId=bt322) outcome was **FAILURE**
Summary: Tests failed: 4 (4 new), passed: 3298, ignored: 106 Build time: 00:20:45
### Failed tests
```
SIL.Core.Tests.dll: SIL.Tests.IO.FileLock.SimpleFileLockTests.ReleaseLock_LockedByActiveProcess_LockFileExists: <no details avaliable>

SIL.Core.Tests.dll: SIL.Tests.IO.FileLock.SimpleFileLockTests.TryAcquireLock_LockedByActiveProcess_ReturnsFalse: <no details avaliable>

SIL.Core.Tests.dll: SIL.Tests.IO.FileLock.SimpleFileLockTests.TryAcquireLock_LockedByActiveProcessNotTimedOut_ReturnsFalse: <no details avaliable>

SIL.Core.Tests.dll: SIL.Tests.IO.FileLock.SimpleFileLockTests.TryAcquireLock_LockedByActiveProcessTimedOut_ReturnsTrue: <no details avaliable>

```


Dead code ...


Change "int ny=-1" to "GO ny=-1"


I prefer B but C is also OK. It doesnt really matter what the name is ... but you should always use containers with lists of things.


@mjackson?


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


Please, write the command to be used to install the `ipsec-tools`


Update  bài 4


If you include short names, you can use SC, LO, GO, NO. So there was no point in changing LO to LocalOrdinal throughout.


Correct.


checkCaptcha函數可加到**share-function**裏面吧？


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


`test_by_committee_returns_periods_for_committee_in_all_states`


@fichtner Can you jump on IRC so I can grok the issue in greater detail and talk to you re your recent twitter DM :)


I will update the documentation and elements to the model. For now the model will only reflect the latest version. If there are major changes in the schema/model we should do revisions.


Maybe it's even better to detach the pid from the mode or interface, if it's running on em1 in pcap mode and your restarting it as netmap it will run twice :)


Same note on `const_cast`.


:heart:


For message pending in FIFO interrupts we don't need to (and we can't) clear any flag. Any other flags can be cleared with clearPendingInterruptsFlag() but I didn't tried it yet. Do we need to clear it in CanInterruptHandler.cpp or the application interrupt handler can do it?


`Enigmas/{charadeEnigmaPanel.cs => CharadeEnigmaPanel.cs} | 14 +++++++-------`
:+1:


Also will need to make this change in the sql upgrade script since it was not brought in correctly in the prior upgrade script.


Missing justification for not needing fixed typid for 3.3 tables but required for 4.0 ones ?


No result means no issuer and attachments should be related to the issuer and LOS/LOI ?


@shalabhc well you're technically correct - the best kind of correct! Done in master :)


Opened https://github.com/rackt/history/issues/144 to track this


symfony best practices recomment to not do class parameters like this. the reasoning is that if you want to replace a class, a compiler pass makes more sense - and overwriting this global parameter can easily fail as its totally implicit. typos will not be reported, the parameter just not be seen. just put that class into the serivce definition


Did you want to check this into git?


i am not sure how global parameter names are. i think i only saw them with the namespace prefix, i guess you should do that here too. oh, or actually are you overwriting the symfony class here in a configuration file? that is very shaky and not compatible with symfony 3.0 (where the parameters will be dropped, so this stops working)


What about all other factories that CAN be tested with long long int?


I would probably explain here the reasons for rhn-register and cover as options the two possible options: rhn-register and subscription_manager


I guess it should be `rhn_register`


Yes, I did some checking as well and found these as well, so perhaps a bigger limit is warranted. I did consider it based on size as well, but I believe that would be slightly counterintuitive as the max number of undos would then differ from buffer to buffer.


I'm not sure that it is a good idea to *hide* a configuration file in a compute controller plugin. Config files should be put in orchestrator/config.


HI, I am practicing fork.


We need the SwitchResX to accept the values we give. I wonder if there is any table or some sort that lists these values to get a correct 60 Hz for a LG 40 inch TV.


идентификатор!!!!


Here I would suggest to cover the two possible test sets/options: with subscription manager and with rhn register. They have a different set of env parameters


#2  ㅇ쪼긴멍리ㅏㅁ
ㅁㄴ이라ㅓㅁ닝ㄹ
ㄴㅁ이ㅏ럼ㄴㅇㄹ


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


I thought part of the reason for making it a package was to have it easily excludable?


Спасибо за внимательность) при следующем коммите поправлю


If they're only used from query.rs, they should be moved entirely. Otherwise, we can go ahead and make the methods in the impl public (ie. `pub fn`).


Hello Kirk,

I'm working on Openstack Neutron project [0]. Currently It has possibility to interact with hardware switches via ML2 drivers. Device vendors are responsible for development/support ML2 drivers and it is fairly. Unfortunately they 'vendors' do not add support of all devices. Cisco for example supports only Nexus and higher series switches, which are expensive. The idea is to create generic driver that will work with any vendor/device. I did some research and found that Netmiko is very good as background for it. It has flexible mechanism of adding new devices, lightweight comparing to trigger [1] for example. Openvswitch support is needed since there are use-cases when we need to change VLAN on NIC to  VM that is running under Qemu/KVM and we want to keep the same logic in configuration as for other Hardware switches.

[0] https://wiki.openstack.org/wiki/Neutron
[1] http://trigger.readthedocs.org/


Having the exact same problem on OS X Yosemite.


I think it is better to use `TT_FAIL()` instead


@justbur's PR introduced such guards on every application of evilification. Should we remove those as well, then?


k :(


This file contains unused source code.

*Detected by PyUnusedCodeBear with severity 1 while running section `python`.*

*The issue occurred in the lines 1 to 1.*

The issue can be fixed by applying the following patch:

```
--- a/test.py
+++ b/test.py
@@ -1,4 +1,3 @@
-import sys

 def main():
```


This change only works for GHC 7.8+, but I guess we already said we only support this branch.


TeamCity pull requests :: netty [Build 682](https://garage.netty.io/teamcity/viewLog.html?buildId=3355&buildTypeId=pulls_netty&guest=1) is now running


Does anyone know how one handles the exception that is thrown by the update function when they try to insert a user with a duplicate email? I'd like to return a JSON error message with a 409 or a 422 status, rather than "Internal Server Error" and a 500 HTTP Status.


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


Human consts instead?


If you need to get records only for one particular Id try to avoid using repository.GetAll(). Its better to use repository.Get(p => p.Id = IdUser). In this case you will get from DB only needed records. It is much better for performance


No. Read the commit message.
This fixed REGEX code init for SWASHNEW.

aux_list was fixed by deferring initialization of GVs to the right phase also.


Oops, this was unintentional.


I think this should be a dedicated commit, as this change is not related to the feature itself


:+1: Weird, I thought I had grepped for "0.1.3" but obviously not. Thanks!


测试的时候把image.zip解压缩，把解压后的文件放到沙盒 document下


What is the reason for this movement?


Cool thanks


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 7](http://build.prxm.it:8811/viewLog.html?buildId=4754&buildTypeId=Jarvis_DocumentStore_GitVersion) is now running


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 7](http://build.prxm.it:8811/viewLog.html?buildId=4754&buildTypeId=Jarvis_DocumentStore_GitVersion) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:34


107310700-change-license-text (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1968 of 1968 tests passed)

![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)


You're welcome :smile:


There is no need to have separate SaveAnnouncement() method in service class. Please call  this.unitOfWork.SaveChanges(); inside of AddAnnouncement method.


Evtl.  `_('label_current_period', default=u'Current Period')` --> `Aktuelle Periode`


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 8](http://build.prxm.it:8811/viewLog.html?buildId=4756&buildTypeId=Jarvis_DocumentStore_GitVersion) is now running


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 8](http://build.prxm.it:8811/viewLog.html?buildId=4756&buildTypeId=Jarvis_DocumentStore_GitVersion) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:10


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 9](http://build.prxm.it:8811/viewLog.html?buildId=4758&buildTypeId=Jarvis_DocumentStore_GitVersion) is now running


Tabulation issue


107310700-change-license-text (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1968 of 1968 tests passed)

![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)


Inject this dependency via constructor using Unity Container


ok. In any case this is just a sanity check, maybe I shall change in assert...


제 시간에 냈는데 실습시간에 테스트를 위해 수정했던 1000ms -> 3000ms로 바꾸었습니다.


i'd rename  `host-python-package.mk` to  `python-host.mk`


Move e-mail configuration settings (address, password, smpt config) to Web.config application settings.


Building bitcoin is surely wrong here.


Maybe just "Remove old events"?


This method name is now a lie :) How about `destroy_events_older_than(period)` and move that period into a constant or something?


Almost all the LamdaCases have redundant parens. Can you hlint it?


Why you leave `--with-gui..` here when you removed it above?


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 1.1.5](http://build.prxm.it:8811/viewLog.html?buildId=4758&buildTypeId=Jarvis_DocumentStore_GitVersion) outcome was **FAILURE**
Summary: Tests passed: 58; exit code 1 Build time: 00:05:21


Add some brackets to this if-else-if chain:

if (a)
{

}
 else if (b)
{

}


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.592.0e574038ac50ffd00178bc025d8e3455cd6ab58d](http://build.palaso.org/viewLog.html?buildId=66366&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3323, ignored: 125 Build time: 00:05:29


What is the reason for this? I could not understand it from the code:(


Oh, I just noticed that `lib/tasks/scheduler.rake` also says "six months" :) Shows how DRY also applies to naming.


This should just take a list of projects or a root directory to glob over.


Yes, you right. And `RSC14b` should be on the `RestClient.swift` file. I will fix it.


We are not using them later?


I don't like this much.
but let's put it in as is and I'll clean it up later.

I think there should be only a single `HostPython` function and should take a param like "$(1) == host" or something like that.


Это логотип, лучше соответствующий класс. Ну и это ссылка на главную, а не `<div>`


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


also month -> months


LESS, SASS, etc. would be nice to have but I can't see us moving over to it until bootstrap finishes v4 and we move over to it. Then we can move everything in a single swoop instead of slowly doing it now.


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 10](http://build.prxm.it:8811/viewLog.html?buildId=4760&buildTypeId=Jarvis_DocumentStore_GitVersion) is now running


Actually , Python don't need init value or space for variable.


Can you please add trailing `,`, this will allow to add strings to this array without changing previous line.


108453024-FixLeedNumberOfHoursNotMet (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (1967 of 1968 tests passed)

![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)


TeamCity pull requests :: netty [Build 682](https://garage.netty.io/teamcity/viewLog.html?buildId=3355&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5293, ignored: 21 Build time: 00:15:27


#5287-EnergyPlus-crashes-when-assigning-2-UnitarySystem-to-same-zone (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1965 of 1968 tests passed)

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


now that we're adding a host build, maybe move the `Build/InstallDev` to `Host/Install` and remove `Build/InstallDev` ?

I was wondering if I should add a host-build but no-one needed it until now.
So, I used the `Build/InstallDev` rule.


d'ailleurs dans mon code je fais plutôt if ($moof and is_array($moof) and ...)


When I tried to find a rationale for containers, the first thing I stumbled upon was Google's style guide saying "XML elements that merely wrap repeating child elements SHOULD NOT be used [Rationale: They are not used in Atom and add nothing.]". :)

I also tried to find any rationale behind the "has" prefixes, but couldn't find a single document about it.

It seems to me that we're discussing personal style. Which would be okay, if not for the timing - I really wouldn't like for other partners to get angry with us. :(


It will need a while loop cover input part.


Do not exclude it is a remnant of the previous logic that included RCU...
at line 310 *filling* is still true, so this was just to wait the end of the destructor.
Most probably not really needed.


This Vagrantfile isn't really used somewhere, its only for local development/testing how it looks like. Do we want to change it?


Apparently just before the 1.0.0 release in commit 3e625e2.


`.mobile-menu` — не совсем подходящий класс. Или ты хочешь для десктопа делать другое меню?


json.loginFailed? Не?)


Recommend using another meaningful variable name.


Leave commit


`<br>`'ы убираем!


add a `HOST_BUILD_DEPENDS:=python/host` here ?


и тут тоже!


null); заменить на parseBalance);


#5287-EnergyPlus-crashes-when-assigning-2-UnitarySystem-to-same-zone (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


What is the reason for this removal?


still hava no idea how it works


@MilhouseVH please test with the latest commits, the build of attr:host should be fixed, will contact you for help


asdasdasd


Calculating with float is risky, better use `x%List_prime_number[y]`.


Should this be a separate patch (i.e. line with `evhttp_is_request_connection_close()`)?


Has it been tested with a "standard" account (not admin) ?


Pourquoi ce changement d'unité ?


Не вижу этой строки в дизайне... https://yadi.sk/i/2PScIHDrkZZJX


Also Updated Administrate to 0.1.1


We mixup here ruby and python. Do we want it?

Is this file our own development or copy&paste from other project? If yes, we should check the licenses and provide a link here


Do we allow sorting only by last name? We can improve it to sort it by any column using something like this: http://stackoverflow.com/a/8660293/5308734


This change is wrong IMO.


Какой же это `<label>`? Это заголовок.


This part will waste a lot of time.
By the time the first factor is found, the loop should be break.


 期望达成
1学会能够基于设计稿来合理规划HTML结构
2理解语义化，合理地使用HTML标签来构建页面
3能够较为熟练地使用HTML、CSS创建一个静态页面
4大概了解并实践企业中对于HTML及CSS的编码规范
5编写复用性高、可读性、可维护性好的代码


I guess its should be removed like you already said:)


А вот тут почему-то нет лейблов, а должны быть!)


This look hackish, and one of the reasons is that after this `evhttp` will not own this request/connection, while user[s] can expect that after `evhttp_free()` no requests will be in progress.


No need to do more than add the description at this point. I will update the model as you have suggested earlier.


This part runs, but could be better with formatting output.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.577.0e574038ac50ffd00178bc025d8e3455cd6ab58d](http://build.palaso.org/viewLog.html?buildId=66367&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3302, ignored: 106 Build time: 00:16:54


Давай заголовки делать заголовками. Для этого у нас есть 7 элементов в html5: `h1-h6`, `header`


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1432 of 1432 tests passed)

![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.49%25-green.svg)


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


Этот инпут выключен, давай поставим `disabled`


This obviously was unintentional, but I started to add tracking for squishes ;)


Are you sure that all attachments MUST be related to the issuer?

Let's say that the student has studied on 5 different institutions within a scope of a single NCP. Wouldn't it be okay for the NCP to want to include a single PDF with the transcript of records of all of these institutions? If not, then probably I should move the attachments back under the `<report>` element (or perhaps allow the attachments to be present in two of these places).


Это у нас в другом блоке уже, давай сделаем. Серый фон же видишь в дизайне? Или как ты его хочешь реализовать?


@Thomas447 @firefly2442 I created an issue for this here: https://github.com/libgdx/libgdx/issues/3578


`--without-gui` is the correct option.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.593.e2bd433c515c13e639c4deefb7351fbbadbf39b3](http://build.palaso.org/viewLog.html?buildId=66368&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3323, ignored: 125 Build time: 00:05:15


What is the justification for using `jquery` here?


Выше было, тоже самое


Thanks Dav, will do some checks later on.


probablement pas vrai en RAD AMR avec des aexp différents par niveaux


@iconoeugen its needs some minor cleanups, but in general it looks good for me:)

@larsjuetten @simon-baatz @mwasserschaff @Frank-Burkhardt @wolfgangwirths probably you want to have a look too


@Tobion is there a more "official" way to replace the routing loader than a compiler pass to change the `routing.loader` service of the framework bundle? also, when using composition over inheritance, how would that look? move the existing routing.loader to routing.loader.original and place our service as routing.loader and pass routing.loader.original as constructor argument?


bitshift


Dr when I replace the existing code with this it tell me that variable status is already defined in the scope


это у нас `label+input[type=file]`, а не ссылка


I think you'd also want to bind `mids` to `add` at line 183 ;)


Thanks for this commit! It should definitely be configurable. Please add issue and pull request.


Line 3 should be updated too :)


Вместо просто `<div>`'а, можем сделать более подходящей конструкцией: `figure>button.close+img+figcaption`, например...


Yes, caught that just as I pushed this commit! D'oh. Unfortunately, it hasn't entirely fixed the issue. I believe there's a deeper bug in the way I'm setting up the caches. Boy, who would've thought that Cross would end up being such a complex transform!


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 1.1.5](http://build.prxm.it:8811/viewLog.html?buildId=4760&buildTypeId=Jarvis_DocumentStore_GitVersion) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 294 Build time: 00:17:00
### Failed tests
```
Jarvis.DocumentStore.Tests.dll: Jarvis.DocumentStore.Tests.SelfHostIntegratonTests.integration_out_of_process_tika_multiple_password("v2").verify_tika_job_with_password: <no details avaliable>

```


Вместо `div>img` проще сделать `a.logo`


hi  Doneill ,i  m new  for android stud,i want be develop ArcGIS attribute offline editor,could give me any code or any link thanks


Test 1


About 2 you're right, it won't be a big problem changing it if necessary. And regarding 1 if it's simple to split it for v2 and v3 then :+1:


Тут у нас ссылки


Это ссылка с классом (без картинки)


Maybe not, because the screenshots are intended for people who have never seen Vimwiki before, and I think it's not immediately clear what `:party:` would mean. Let's spare these scary features for more advanced users ;)


sortir de la boucle


4-space indentation.


It is not service's responsibility to create a provider. This will prevent you from testing the logic separately and easily


@dprince @hguemar is that really true? We need to distribute maintainership, it is not sustainable be that few folks take care of most of the packages.


Indeed, it's such a simple concept... yet causing so much trouble.


Relevant to release/4.0?


Декоративные или контентные изображения?)


Ah, true, the comment needs updating. I'll replace it with this, which explains it:

```
+                                               /*
+                                                * Some relations need to have a
 fixed relation type
+                                                * OID, because it is referenced
 in code.
+                                                *
+                                                * 90MERGE_FIXME: In PostgreSQL
9.0, there's a
+                                                * new BKI directive, BKI_ROWTYP
E_OID(<oid>), for
+                                                * doing the same. Replace this
hack with that once
+                                                * we merge with 9.0.
+                                                */
```


Align the * with the line above.


is this loading *all* routes or only from one configuration file? do breadcrumbs work when i have the parent route defined in a different file, maybe one being xml the other yml?


Строчный элемент?


Align * with the line above.


This is a bug! Pleare revert this change!(


This should be corrected to `repository`


This broke my fix. Can we use isNaN instead?


I don't see the interest of such a noop service versus a simple mock.


we dont need run anymore right?


You're welcome :smile:


Align * with line above.


Why isn't this method async?


is activation key a secret too?


This one is tested and it works. Please merge.


Now that you point it out, it's kind of obvious ;) I changed it. Many
Thanks.

2015-11-18 23:36 GMT+01:00 Tadaya Tsuyukubo <notifications@github.com>:

> Hi,
>
> just noticed this TODO while reading the code:
>
> // TODO Users of this method should use Optional directly but I couldn't
> figure out how.
>
> probably something like this:
>
> // changed to directly return optionalprivate Optional<TestDescriptor> descriptorByUniqueId(String uniqueId) {
>     return engineDescriptor.findByUniqueId(uniqueId);
> }
> // one of the users of "descriptorByUniqueId" methodprivate ContextTestDescriptor getOrCreateContextDescriptor(Class<?> clazz, String uniqueId) {
>     return (ContextTestDescriptor) descriptorByUniqueId(uniqueId)
>             .orElse(new ContextTestDescriptor(uniqueId, clazz));
> }
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/junit-team/junit-lambda/commit/bf41ffd2aa275def5a02ba58dc202c6e61b09b27#commitcomment-14485671>
> .
>


@anrishitko


I think authentication is missing for a plain integration test


I could not figure out where this name is coming from, can you explain?


What is the intention here?


what do you mean?


@Optimusprimeums are you keen to do a double release this weekend? Looking to release 5.3.1 and 6.0.0-a1.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.578.e2bd433c515c13e639c4deefb7351fbbadbf39b3](http://build.palaso.org/viewLog.html?buildId=66369&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3302, ignored: 106 Build time: 00:14:05


@emistry @MishimaHaruna @dastgir please check thanks!


I notice a trend when you change things like this that you often don't update comments or names. Could be something to pay extra attention to for a while.


From http://lammps.sandia.gov/doc/dump.html

    The xyz style writes XYZ files, which is a simple text-based coordinate
    format that many codes can read. Specifically it has a line with the number of
    atoms, then a comment line that is usually ignored followed by one line per
    atom with the atom type and the x-, y-, and z-coordinate of that atom. You can
    use the dump_modify element option to change the output from using the
    (numerical) atom type to an element name (or some other label). This will help
    many visualization programs to guess bonds and colors.

I know that it is not *the* xyz reference but given LAMMPS' wide user base I guess that they have a good basis. So, first column should be *type* and not *id*.


You should now add unitary test for method/GetMailboxesMethod


Incorrect namespace – if it was *.Categories, the Category entity would have name conflict down below in CreateCategory method.


There are 'u_forename' and 'u_uid' but 'surname' and 'fullname'? Why a different naming?


Done. Hlint ignores those!
On Nov 19, 2015 10:17 AM, "Eyal Lotem" <notifications@github.com> wrote:

> Almost all the LamdaCases have redundant parens. Can you hlint it?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sinelaw/buildsome/commit/1343bdc3974e8703d64a25f6b68bd1afd4dff8ae#commitcomment-14493002>
> .
>


Puh, das macht keinen Sinn! ;-)


Just downloaded today night build, and it is awkward, the
destination/waipoints setup is very different from what we are used to.

I usually would set my destination first, and allow Osmand to calculate the
route, then check that routing, and add waypoint to modify the route as
desired/necessary. It's excellent with this method.

I realize that this night build may be a work in progress, but as it is now
(and since day 10-11-2015), it is rather useless, and I think it distorts
what a long term osmand user is expecting.

Osmand now adds the previous desired intermediate waypoint as destinations,
so it totally messes up the intended routing.
For this method to work, we would have to first set the intermediate
waypoints, an the final destination only as the last waypoint. But if I
want to plan my intermediate points after seeing what is the planned route,
this is counter-productive.
I then have to go to the waypoints mixed listing and re-order them all
again to be able to route as expected.
Also, as it is now, the routing ignores my set waypoints?

--
Cumprimentos,
D.er Hugo Barrocas


Siehe Fehler bei Instagram:

![dateerror](https://cloud.githubusercontent.com/assets/6824488/11266168/c721ca52-8ea2-11e5-889f-27fe61a24a43.PNG)


Декоративное или контентное изображение?


Thanks for removing it


The `makeUNCHECKEDDescriptiveWrapper` function could also have its `shared_ptr` counterpart.


Hi Israa,

Make sure you delete the lines in RED and add the lines in GREEN. Don't put both in.

Your Utils class needs to look like this: https://github.com/abdulla-alali/Android_Homeworks/blob/590924b7ee8a5f3d0fc75b22e969e46c17e70221/Homework5/app/src/main/java/qa/edu/qu/cse/cmps497/homework5/utils/Utils.java

in the end.


Hm, wouldn't the root directory be a better place for the modifiers than the server module? It simply modifies the status object, that doesn't have much to do with HTTP or serving in general, right?


just wondering if the links should use https instead of http, according to https://core.trac.wordpress.org/changeset/35650 ?


Marking this line because there's no other way to know just what you changed. (radius * 10 --> radius * 6)

And that's why we clone files in one commit, *then* start changing them.


+1 bugfix release


Kann man die Monate nicht in einem assoziativen Array speichern statt ein Switch-Case zu verwenden?


@gabi2 Good point. It's because u_forename and u_uid will collide with elements defined in other modules. Maybe we should use a prefix for all the elements, but for now, I'm only fixing the schema.


감사합니다!
정상적으로 돌아가는지 테스트까지 완료하셨지요? :-)


测试2


@patriksvensson am I doing this wrong?  I am seeing the full blog post when I look here:

http://cakebuild.net/blog

But I am expecting to see a "Read more..." link where the excerpt is inserted.


All this should be moved into a separate class. Create (e.g.) MappingsConfig static class in App_Start folder, so the Application_Start method is not as long as Italian spaghetti. Do same for the binders.


Or, does it always show the full version of the latest blog post?


This really should be in FTLRevamp, seeing as that's the branch for 0.5.0. :P

I'll merge it in when I get the chance.


I mean you should really grow and stop commenting on this really old commit


usleep是个POSIX废弃的API，主要是信号打断问题和一些兼容性问题，现在都推荐用nanosleep

不过我个人感觉这个TimeProvider线程增加了复杂性，对这个改动的投入产出比表示怀疑。如果boost的universal_time函数比较慢，不如把boost函数替换成clock_gettime试试


Why is this class still public?! Should be at most internal.


Right and with that everything should work now on this new module, which I needed to take care of Tricky1975/Star-Story#284


In PHP werden alle Variablen mit einem Wert initalisiert.


It always shows the full version of the latest blog post.


@AnnieRuru How about my commit now? I refer it to aegis script.


I know that message obtain has arg1 and arg 2 arguments, but please change name arg2 into something miningfull :)


Why the result has int type? You know, MAX_INT in JAVA is 2147483647, so 13! already greater than MAX_INT. I suggest to use String type to store the factorial result.


네임스페이스 추가


Not sure. Could be useful for test purpose. But I can (should?) remove it.


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


네임스페이스로 Logger.php 추가


@lucabrivio: do you know and how do you know what is an average value of
double(Threads.main()->rootDepth - ONE_PLY) / log(double(Threads.nodes_searched())) ?

If I understand well, in ideal situation this should be a constant, and in real situation we need to estimate an average of this expression to be able to understand which is an appropriate constant in front of log.


네임스페이스로 Application 추가


Consider renaming the method to something like "Initialize".

There is still a piece of code missing here, as the seeding context will never be called (as you never use the context itself). There should be something like this in addition:

            using (var context = new Seeding.Context.SeedingContext())
            {
                context.Database.Initialize(force: true);
            }


@build000 totally unrelated, its a display thing only.


@hnyman Right, will fix it


@canapio 제가 아이폰, 아이워치가 없어서 실제 기기에서는 하지 못하고 simulator로 했습니다 ㅠ 틈틈이 보고 수정사항 있으면 pull request 하겠습니다~
맞췄으면 하는 code convention이 있으면 wiki에 있으면 좋을 것 같네요 ^^


KRE_ROOT 상수 추가


KRE_ROOT 상수 추가로 인한 수정


Przypominam, że konwencje mówią, że nazwy tabel i kolumn powinny być w języku angielskim.


오토로드 기능 추가.
namespace로 자동으로 include


namespace 추가


А что не закончили вёрстку?)


QuadrupleMapNodeData is not used by default currently. See the SingleMapNodeData implementation below that is the default implementation. Here, testing the value would require another `map.get()`; I assumed `toString()` to be more efficient.


Exception으로 변경


로그 인터페이스 추가


데이터베이스 Class 추가


Maybe it will be more convenient to operate with status is to use bunch of strings instead enum. Because it allows simple conversion to json, imho. See, http://stackoverflow.com/a/3980725


로그 Class 추가,
monolog 참고


Also https://github.com/elzbietaZ/moppa/blob/d18b300b83ba6b2d666b997eb5f4f59119c86087/MoppaCore/src/main/java/org/moppa/MoppaCore/TaskStatusValues.java


I would prefer to reword this a little bit.

```
title = NULL;
oval_def = ...
if (oval_def != NULL) {
    title = oval_definition_get_title(oval_def);
}
```

That way you avoid `return -1;` which brings us a side effect.

That is my way of thinking, let me know if you feel that differently.


Thanks Pierre, I guess the question now is to split or not to split writing
xyz conf and the restarting files? I would guess splitting them is not a
bad idea.
On Nov 19, 2015 9:46 AM, "Pierre de Buyl" <notifications@github.com> wrote:

> From http://lammps.sandia.gov/doc/dump.html
>
> The xyz style writes XYZ files, which is a simple text-based coordinate
> format that many codes can read. Specifically it has a line with the number of
> atoms, then a comment line that is usually ignored followed by one line per
> atom with the atom type and the x-, y-, and z-coordinate of that atom. You can
> use the dump_modify element option to change the output from using the
> (numerical) atom type to an element name (or some other label). This will help
> many visualization programs to guess bonds and colors.
>
> I know that it is not *the* xyz reference but given LAMMPS' wide user
> base I guess that they have a good basis. So, first column should be
> *type* and not *id*.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fedepad/espressopp/commit/b149b3b5b9f0cb6df4a0bc0ee68ed9cc754f1b15#commitcomment-14493524>
> .
>


These were in IteratorObservable.create() (which by the way isn't used anywhere else in the codebase AFAIK), I just moved them to the constructor. In my opinion we can take these checks away, I was just working on getting better test coverage.


In both these lists, strongly consider using Lazy<> concept, so all (big) static objects are not initialized immediately on application start...


Is this really useful message for a user?

I don't know just asking.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.594.dce0ad46775a6e17d8b7f7cc5d0409643c30fad3](http://build.palaso.org/viewLog.html?buildId=66371&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3323, ignored: 125 Build time: 00:14:47


The `toString()` in `Property` implementations always returns a string constant passed through its constructor. I mean further caching wouldn't improve anything. ([e.g. here](https://github.com/radkovo/jStyleParser/blob/master/src/main/java/cz/vutbr/web/css/CSSProperty.java#L1001))


画像が表示できてないっぽい(>_<)


environmENT variable


```java
Optional<TestDescriptor> optional = engineDescriptor.findByUniqueId(uniqueId);
return optional.orElse(null);
```
or even better, without temporary variable:
```java
return engineDescriptor.findByUniqueId(uniqueId).orElse(null);
```


This is the default `Property.getAsString()` implementation.


@emistry @AnnieRuru


Il debugger che ti dice?


Use a setter.


Umm... The URL contains "TouchPoint", but it's a SISIS OPAC?


sadfsadfs


В целом хорошо, но комментарии остались в предыдущем коммите, жду новых изменений и делаю слияние


I want to clobber this work entirely since IMHO it's entirely bogus. The build system can't know upfront what folders the built things will want to make, it doesn't make any sense.

https://github.com/diorcety-ctng/crosstool-ng/commit/351f395a97d807d21bbe4b8bed19492d66c79f3b


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.579.dce0ad46775a6e17d8b7f7cc5d0409643c30fad3](http://build.palaso.org/viewLog.html?buildId=66372&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3302, ignored: 106 Build time: 00:18:53


Change the spurious message about "Can't drop privilege as nonroot user" if what you meant to say is "You can't use the user= option when run as non-root"


Also, if this is a CRIT issue, perhaps execution should stop.


@briancroom I found out tonight, while trying to cut the v0.3.1 release, that this seemingly innocent change actually makes the podspec not lint anymore.

If you run `pod lib lint` locally, you'll see some bizarre messages about the Foundation target for OS X not being able to find the `UIKit.h` header, while trying to compile `NSAttributedString+PivotalCoreKit_UIKit.m`. The best I could figure out was that not setting platform to `ios` here somehow makes the Foundation target assume it can include files from the UIKit/Core/Extensions directory, which doesn't make a lick of sense to me. I think that `s.platform = :ios` causes it to assume that this spec, and all the subspecs can ONLY be built for iOS, unless otherwise specified.

It seemed to work better (albeit still fail further on) when I reverted these two lines. Is there a better way of specifying the correct platform for the project? Some of the subspecs declare their own required platform, which seemed like a nicer way of specifying what versions of iOS and tvOS we support.


#5287-EnergyPlus-crashes-when-assigning-2-UnitarySystem-to-same-zone (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (890 of 894 tests passed)

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


I am unsure about this.

Imagine I am user. Either the validation passes, and I don't care it happened. Or the validation failed and I will get the error anyway.

Why do you think this message would be valuable to user?


develop (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1432 of 1432 tests passed)

![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)


Shouldn't this be ->firstKey()?


#5287-EnergyPlus-crashes-when-assigning-2-UnitarySystem-to-same-zone (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1964 of 1968 tests passed)

![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)


@matej21 It haven't been tagged yet. How do you propose to solve this? Calling the `createInstance` is not very effective in runtime.


The comment doesn't match the function signature.


@andela-cmutembei - a better wording for the commit message is `fix failing tests`. However, you didn't have a message body for your commit, and reference to the _story ID_. Can you fix this before I merge. Do a `git commit --amend`


Your commit message states exactly why I thought you should remove it directly instead of commenting it :wink:


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


a sry, typo on my side: should read `extent` ;)


In the body, you should explain what this commit does to _fix the failing tests_ as stated.


TODO: Use a function to avoid duplication and add a comment to delete this if in the next version


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


#5287-EnergyPlus-crashes-when-assigning-2-UnitarySystem-to-same-zone (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Not directly related to this commit exactly, but now we use blue to depict download regions on the world map, which, together with the sea's blue, and regions where both download areas and the sea overlap creates 3 different shades of blue. I think we should revert to a shade of green for downloaded maps, to still make the shape of coastal areas more viewable at zoom 4 to 6.


Only option to use replace with a char is to use the fill-version, which is counterintuitive in this case I think.


It used to matter - if we were going to output to STDOUT, then we couldn't have Gin putting its own debugging messages into STDOUT as well. Now we're using sockets, it's not a big deal.


This would be a bit cleaner if it was in terms of ->first()
```PHP
$fnFilter = $fnTest ? fn\fnSwapParamsPassThrough($fnTest) : null;
return $this->first($fnFilter);
```


No biggie here, but this could be something like "older than the limit" so we're more DRY, and then the test body could get into specifics.


You can certainly get away with it here, but in general it's better to use `{!r}` in your format string rather than `"{}"`. The `!r` allows Python to choose the most appropriate quotes:
```python
>>> print '{!r}'.format('ab"c')
'ab"c'
>>> print '{!r}'.format("ab'c")
"ab'c"
```


I suggest that `failure` should be added to suppressed exceptions using `addSuppressed(failure)`. It will enable this exception not to hide stacktraces when using one of `Throwable.printStackTrace()` variants.


Is anyone with 5.0.2 or <5.1 anyway actually being able to use NetGuard?
You got stats for that in the Store I guess?


These could be "…than_limit".


and schedule ?


This change was not on purprose, it's just that pom project are unusable in m2e and I always have to make it jar to properly import it.


I see. So you don't lift it into an element. Qestion: can I group on a group, or call split, etc?


done


> another possible solution might be removing the #if ! defined altogether and just deallocate it even though we never used the tls slot.

This sounds to me like the only viable solution as well. So, what about this (untested):

````{.diff}
diff --git a/src/global-init.cxx b/src/global-init.cxx
index afbf9b4..9b347b0 100644
--- a/src/global-init.cxx
+++ b/src/global-init.cxx
@@ -529,10 +529,12 @@ struct _static_log4cplus_initializer
     {
         // Last thread cleanup.
         log4cplus::threadCleanup ();
-#if ! defined (LOG4CPLUS_THREAD_LOCAL_VAR)
-    log4cplus::thread::impl::tls_cleanup (
-        log4cplus::internal::tls_storage_key);
-#endif
+
+        if (log4cplus::internal::tls_storage_key)
+        {
+            log4cplus::thread::impl::tls_cleanup (
+                log4cplus::internal::tls_storage_key);
+        }
     }
 } static initializer;
````

If I am reading your situation correctly, the `~_static_log4cplus_initializer` dtor should be the one that is executed for you.


Reminds me of something :) ( ahem https://github.com/M66B/NetGuard/issues/145 )


maybe call it mergeBack?


Fixes `ReferenceError: repostiory is not defined`. However, tagging still does not work properly

![bad-tag](https://cloud.githubusercontent.com/assets/515617/11266726/6f86c4ec-8e5b-11e5-8292-d121795ad634.png)


might be not needed. but set still to avoid accidental error.


LGTM. I'll merge it.
Thanks.


@AnriShitko


#5287-EnergyPlus-crashes-when-assigning-2-UnitarySystem-to-same-zone (rraustad) - i386-Windows-7-VisualStudio-12: OK (1966 of 1968 tests passed)

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


Ok I will push the implmentation and try to generate the bindings..its something I wanted to learn anyhow.


Should be static


this makes the links in the rss feed to have an invalid url :)


Add `NOTE:` prefix pls


if me, I would use .@r or .@rand
but meh ... it works
so don't mind it


oh, nice generalisation :+1:


maybe name the parameter?


BEST COMMIT EVER <3


good to see this gone! Yaay! :D


thx didn't know about default `iteratee`


ok so can I change it back? because right now it produces a warning in the maven logs + a 0 size JAR


Ce commit merge bizarre c juste moi qui est pull ton code j'ai pas touché au JSON, ne prend pas peur


Roboter Rüdiger hochgeladen von Ksenia Rasin, jedoch über Laptop von Maximilian Roth. Deswegen zeigt github die Angabe an: committed by  maximilianroth


interestingly the `<:` is what makes covariance happy, but it also requires the `higherKinds` import


:+1:

I was wondering how we would do that. :)


That'd depend on whether cb5c69d0643ca0d48ebb9abf24b107a1eacec496 is reverted, since that sets z_bottom to 0 too. If it is, z_bottom will be uninitialised. I think it's better to make sure it doesn't go into this code at all when such conditions are false, like we discussed.


well probably its this, since now the links display as below:

<link>
http://iamtrask.github.iohttp://iamtrask.github.io/2015/11/15/anyone-can-code-lstm/
</link>


Hmm, after updating android plugin building fails again.

Error:(1, 0) Plugin is too old, please update to a more recent version, or set ANDROID_DAILY_OVERRIDE environment variable to "e3b3ee065e5f56094dde6a1ce2e326d9cf6ee8cc"

It looks like 1.4 never left beta:
https://bintray.com/android/android-tools/com.android.tools.build.gradle/view

How about updating to 1.5.0 instead? (dropping the + to reduce future conflicts)


Prosím ošetrite desatinnú čiarku v riadku 12 na 3.strane aj tlač pdf.


test


@Ilvec I don't trust my estimates so I run a few long matches with instrumented compiles that output rootDepth for each thread at each iteration of each search... I haven't made any real statistical analysis although I'd like to have a version whose median root depth for, say, thread #2 is as close as possible to that of the master branch...


bonus3 bAddEffWhenHit, Eff_Stun, 100000, ATF_MAGIC;


We use blue for outdated maps. However it might be not the best choice. Anyway - Victor's decision is needed.


Are no members exposed to python on purpose?


ionic 直接在controller 调用wechat  第一次会发生错误 Wechat is not defined  第二次调用就没问题，请问怎么解决？（按钮调用却没有问题）


@teanoon


Ist nur ein kleines Grundgerüst, David wird die Operationen noch mit Leben füllen. :-)


> For concatenation of the flows use a breadth of 1 but beware

Sounds like may want a dedicated method – `flattenConcat`?


I don't think it is ambiguous if you consider its type. `tables` is a list of tables.


Predplnenie dátumu vyhlásenia môžete dať aj do formulára.


Why not just use the text property? eg:

```haxe
myTextInput.text = "hello";
```


I think we should use the language of the document as `lang` and not the interface language.


It hasn't, it's still there. The code was changed in an earlier commit.


As I said there, commit https://github.com/Ultimaker/CuraEngine/commit/c8f19229661c1cb5c9b45c1cf74ec04ce950734d should prevent z_bottom from being uninitialized.

The fact that wireFrame.layer is empty doesn't mean nothing can be printed. Layer zero can still be printed!
If z_bottom is uninitialized here then that is a separate problem. maxObjectheight should be set to z_bottom in any case.


ok we don't have this ... need to code a new one ...

PS: I just compare hercules and rathena doc\item_bonus.txt
it seems they are more updated
although, it seems they don't have this in their doc too


yes, that’s a nice one


Those are installed and used or kept, right?


`public Conversation select(Conversation c){...}` braucht andere *Parameter*, und der Methodenname ist viel *zu generisch*.


Am besten noch die Sache mit dem Titel und dem Link (authorLink / link) beim nächsten Push berücksichtigen.


Yes


这里进行排名自增之后会导致rank只有奇数值


We can even set different languages on different parts of the page. E.g. set the interface language as default language on `<html>` and then the document language on the element that contains the translation.
See also: http://nimbupani.com/declaring-languages-in-html-5.html


:metal:


Hi @jviide, I don't see where `defaultNoShare` is used. Can it be removed?


模型管理对象，作为admin.site.register()的第二个参数


`rails g scaffold thesis title abstract:text defended:date`


If status code is not 200 there should be info why it failed (minimum a printed Error code or return the response), now it will just return None.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.595.ccccab2eee2729126c51be22471bd3ceb6be6983](http://build.palaso.org/viewLog.html?buildId=66379&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3324, ignored: 125 Build time: 00:18:28


(quoted from email)
> Elmo had two wrappers  - such as specifies or hasPart - and i did propose to use same practices consistently in the new version.

Note, that both of these wrappers in the original ELMO has maxOccurs="unbounded" and both of them could contain only a single child element (that's why their type was called "association" - as in 1-1 association). Your proposal changes that significantly.

BTW, I have found another document against wrapper elements: http://www.ibm.com/developerworks/library/x-contain/


I need to keep this exist, otherwise the whole loop won't work.


@capensisma `proto`


and this is fine and the common behavior in `toString` with object properties


Fixed.


Indeed a good idea, I get a bit of improvement, I've removed the `_stk` variable because it's useless and the `KUTE.tick` function execute a bit faster with the non-cached length, excellent idea.

However, this line `define([], factory);` must not change, it's needed for plugins and UMD system.


обнаружил ее выделив текст в макете


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.580.ccccab2eee2729126c51be22471bd3ceb6be6983](http://build.palaso.org/viewLog.html?buildId=66380&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3303, ignored: 106 Build time: 00:30:35


The remaining users of databuf in mxt_app.c only need a single byte (not 1024), probably should refactor this whilst we're at it.


Not seeing it here. gcc 4.8 and ocl-icd-2.2.7 (against 2.0 headers).


I am like clinic patient, still waiting for


ok, I've updated it in the later "XXX - apply PR feedback" commit


Sorry, war ein typischer Anfängerfehler. Habe wohl aus Übereifer schon einmal etwas anlegen wollen. Ich sehe ein, dass nur die Methodenrümpfe zu wenig sind. Schlimmstenfalls vergisst man noch dies fertig auszuprogrammieren, wenn ein entsprechender Kommentar fehlt.
Stimmt, loadAllConversationByUser oder andere Varianten (inkl. der jeweils erforderlichen Parameter) sind dann wirklich besser als nur SELECT.


@borshop: retry


Doesn't work for me without
```
pc->set_v_size_flags(SIZE_EXPAND_FILL);
```
Without this line nothing is displayed in help tabs.


I added a break at the begin of second loop, kind of solve this problem.


Here's the code to repeat the situation
```js
// try out commit set to true
var commitFile = new PackerFile();
commitFile.addDocker(true, null, 'python:2.7');
commitFile.shellCmd('pip install ansible');
commitFile.dockerTag('sample/python', '0.2-tag'); // errors out
commitFile.build(function (err, output) {
    if (err) {
        return callback(err);
    }
    var str = JSON.stringify(output, null, 2);
    console.log(str);
    callback(null, output);
});

// try export_path
var importFile = new PackerFile();
importFile.addDocker(true, 'image.tar', 'python:2.7');
importFile.shellCmd('pip install ansible');
importFile.dockerImport('sample/python', '0.2-import'); // errors out
importFile.build(function (err, output) {
    if (err) {
        return callback(err);
    }
    var str = JSON.stringify(output, null, 2);
    console.log(str);
    callback(null, str);
});
```

I looked at each of the temporary JSON files that were created and none of them had `post-processors`


Why? CSRF is the standard name, not XSRF.


@ownedge Sorry for the intrusion but I'd like to ask you if you're still using your username ownedge. That's the name of my company since 2011 and I would like to create a Github organisation with that name.
If you have a minute, please send me a mail to pedro [at] ownedge [dot] com :) Thanks.


I'm not sure what you want me to do here.


你别黑我


I agree with the new behaviour, but how will we handle older projects? A warning in the release notes?


You need to apply this patch to scapy to use this script
https://github.com/phaethon/scapy/pull/34


Ca a fait ça pcq t'as commit avant de pull

2015-11-19 10:23 GMT+01:00 Lopez jimmy <notifications@github.com>:

> Ce commit merge bizarre c juste moi qui est pull ton code j'ai pas touché
> au JSON, ne prend pas peur
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ShaGold/WifWaf/commit/0a36573a3f66ebcbf6503c90ee3a37d68a17cac9#commitcomment-14494165>
> .
>


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


Interesting. Was the Python install required?


indentation


Getting fancy


text-center"] => text-center" ]


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.133.7e2763519691e5c409e80d88011bf57149bdf405](http://build.palaso.org/viewLog.html?buildId=66374&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 939, ignored: 50 Build time: 00:41:21


Mal kurz zwischenrein gegrätscht. Dieses Buch ist klasse, danke dafür!


@stevelle I must admit I struggled a little with where to place this, but you're right I kind of followed the existing pattern, as I saw it.  Happy to move it if that's the consensus


@lucabrivio : anyway, I guess that true value of nodes searched is 1000.0 * Threads.nodes_searched(),
and that you should use this value, but please check this.


`rails g scaffold faculty name code:integer`


@stevelle again I followed a pre-existing pattern (seen in the ceph checks). I felt that having it at both levels wouldn't hurt, but again am happy to remove if we'd rather keep it clean


dfgas


Речь о том, что "следовать по маршруту" можно только в том случае, если маршрут проложен из текущего местоположения. Согласен, русский текст не совсем адекватен.


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


ok


ah cool, I didn't realise you could do that within the alarm language. Initially I'd calculated the percentage as a unique metric in the plugin to make alarming on it easier, but reviews on other plugins had suggested this was unneccessary


"of the state-full container of a CE report analysis task" would be more accurate


#5287-EnergyPlus-crashes-when-assigning-2-UnitarySystem-to-same-zone (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (890 of 894 tests passed)

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


no need to add devCockpitBridge to the task's container, it's already available in the parent container


Среди них есть заголовки, тексты под этими заголовками, плюс часть строк будут на разных страницах what's new.


typo "Is" => "If"


saw approval from javajolt
at https://github.com/basho/riak_kv/commit/2ea8edf1879a83e37d3c5317a0cf912f21393166


merging basho/riak_kv/fixes/am/fix_encode_timestamp_cells = 2ea8edf1 into borshop-integration-1270-fixes/am/fix_encode_timestamp_cells


basho/riak_kv/fixes/am/fix_encode_timestamp_cells = 2ea8edf1 merged ok, testing candidate = a506d098


That's great. Thank you!


Thanks, I was not sure about that.


Check please options.h


Note to self. Эту строчку пока ждем.


Stats for darkavenger.json:
BEFORE: 129983 darkavenger.json
AFTER: 54470 darkavenger.json

Under GZIP compression:
BEFORE 11949 bytes
AFTER: 9504 bytes
DIFF = 2445 bytes, or 2.445 KB.

While the actual file size was more than halved, the compressed file size isn't large enough to make a difference in my opinion. But it has one minor benefit that I would like to leverage, and it's that the skill's SkillPoint is an array and therefore can make use of array.reduce().

Regardless the benefits are fairly minor and therefore the original method should be used instead.


Where did the `@Nullable` go?


@danaspiegel This line seems to be not needed as you are referring to ugettext using django.utils import, and you are not using activate anywhere.


matching on simpleName is risquy, why not just match on class?


:ok:


Oh ok.
This is also ok.


In my eyes this modification merits a version patch number increment in order to be able to reference it as an upwards dependency.


This doesn't work actually.


我自己加了$timeout 延迟了一下就不会报错，有其他办法吗   $timeout(function() {

try {
Wechat.isInstalled(function (installed) {
alert("Wechat installed: " + (installed ? "Yes" : "No"));
});
}
catch(err)
{
alert(err);
}

      },  200);


always use ===


why do you need to check for the presence of db_info?


This is really nit-picking, but maybe split this up for readability?

```java
MutableModelNode componentBinariesNode = binaryNode.getParent();
MutableModelNode componentNode = componentBinariesNode.getParent();
```


:-D I wonder how that can just happen by accident.


Yes. This is a UPN (user principal name) whereas the one with the host is a SPN (service principal name). From the Hadoop Security book:

"Principals are classified into two categories: user principals and service principals. User principal names, or UPNs, represent regular users. This closely resembles usernames or accounts in the operating system world. Service principal names, or SPNs, represent services that a user needs to access, such as a database on a specific server."

I thought that a UPN would be more suitable for a client (Hortonworks docs do the same). What do you think?


develop (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1432 of 1432 tests passed)

![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)


Are we sure to need a full-blown component here? What was wrong with the mocking?


I changed it when I sent previous comment.


hmm I think this is going to make the coverage fail now since the surefire plugin is not going to be called!


I guess this should also use `asMutable()` or something instead of returning the private data. Not very important right now, but I think we'll eventually bump into this.


This commit is being reviewed as a part of review issue #109.


This branch isn't tested and the exception message doesn't match what is actually being checked.


This commit is being reviewed as a part of review issue #110.


Why a change here? We still have `getComponent()`, no?


big :thumbsup:


Cool. Glad I could have some positive influence. :)


Nevhodne vse najednou - dohaneni rychleho postupu.


If we had release notes - I'd say we should add it to the visual changelog (and I added a [FEATURE] to make the documentation team aware - and the lazy coder added a comment to the new issue qgis/QGIS-Documentation#679)


I think we should leave it unchanged.


I agree! Done and thanks for reporting!


Shouldn't `Integrand`, `LO`, `Indicator`, `ary` be `global` too? And I guess we're ok with `app` and `idx` being smart, even though `idx` doesn't perform any bound check?


Just stumbled upon your commit while trouble shooting my own. Is jekyll-google-auth working for you? I am having a similar issue addressed [here](https://github.com/intridea/omniauth-oauth2/issues/58), and I am curious if it's working for you. Thanks!


But the test cases (as with any input to and output from the simplifier) should say `Plate`. Any smartness should be internal to the simplifier, if only triggered with `eval(...,Plate=plate)`. Another related issue we discussed is that the smartness of `plate` should really be located in step 2 on the LO side, not in step 1 on the measure-term side.


LGTM.


SOIgnore seems a better name for me


feito


Deixado o mesmo.


Oopie. Nice catch. This is meant to prevent people from adding category:book into the title or the body.


Usei não aprimorável


I like the dryrun interface for this.  Passing as a param is still messy because every function that has the dryrun bool will need the `kubectl.Runner`

What are your thoughts on using a context struct for all the common params?
```
type ActionContext struct {
  Runner kubectl.Runner
  Home, Namespace string
  ...
}
```


aceitei a sugestão, cabe melhor na UI.


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


Will fix the spacing here.


Needs to be rc1-final not just rc1


Perhaps the difference is that I have a left hand sidenav (it's already off screen here - about half a page scrolled). With the code change, when I scroll, my tooltips are correctly positioned (although I have since modified the styling).
![image](https://cloud.githubusercontent.com/assets/9955000/11288155/50db72c4-8eef-11e5-9368-0d0df965670e.png)

When I put it back the way it was before the change, and I scroll, the same tooltip is not where it should be. It is roughly the sidenav's width left of where it should be and the vertical position gets worse as you scroll farther down.
![image](https://cloud.githubusercontent.com/assets/9955000/11288168/6b68e89c-8eef-11e5-92df-933d2e1009fd.png)

I have several config screens that are arranged this way, with a left sidenav and a vertical series of collapsing/expanding card forms. This change was the only thing that allowed the tooltips to be positioned in the correct place on the screen when scrolling since 11.0.


I disagree with this change. In this feature, the build scripts are secondary. The primary use of the feature is the plugin code.


GIT sucks!!


Mão de obra se encaixou melhor com a skill


if @event.save
redirect
else
flash?
redirect
end


The GetEncoding overload used by Windows is the one taking an int, and is documented to throw either a not supported or argument exception when the encoding isn't supported, whereas this overload taking a string is documented to throw only an argument exception. Do we have reason to believe the docs are wrong? I could leave the not supported catch in addition if so, or even change it to a catch-all, though I'd prefer not to.


This is the "only" thing I wanted to commit...


"yesno" should be a final String :innocent:. I think, defined in `BibtexFields`.


No idea onde é usado isso.


there is LITERALLY a proc `unlock_atom()` that handles this nonsense, you fucking blew it, `update_canmove()` will still break because the mob's `locked_to` is set to the kudzu.


I think we don't need try..finally here.
Successful request must fire: OnBeforeRequest |OnAfterRequest
Bad request must fire: OnBeforeRequest | OnError


same


Looks right to me


feito


How about something that kicks in only later? Say from pos.rule50_count() > 25.

so for this particular line (in pseudocode)
eval -= (eval - VALUE_DRAW) * max(pos.rule50_count() - 25,0) / 128;

and accordingly for the adjustedBestValue


Isn't the error message should be about C++01 support at least?


Thanks, that worked.


能解析，但是不能自定义NS


Euh... Ce commit ne fait pas du tout que ce qu'il dit, si ? Genre là à la lecture j'ai l'impression qu'il n'utilise plus du tout eventunit -- ce qui ne serait pas forcément gênant si du coup eventunit ne devenait pas du code mort ? (NB: je n'ai pas encore testé)


Toda a conversa é melhor


Sorry for commenting on a commit but this isn't visible in the changed files anymore. My removal comments were specifically about the last sentence. The first paragraph explains what's going on and remains correct. Please restore.


> Do we have reason to believe the docs are wrong? I could leave the not supported catch in addition if so

I was just concerned that the docs might not have been updated. Adding @tarekgh, who would know a little more if there is a doc gap or is it accurate. If its accurate, I am ok with the change :)


Blz.


Got it


this broke https://pypi.python.org/pypi/mogwai/0.7.6
had to be `'rexpro>=0.4.5,<1.0.0',`


Did you fix there that inventoru glich?

P.S. Thx for telling me that i need to compile folders in src,  i compiled all master lols =))


Prefiro deixa Jornal do aventureiro que fica mais fiel.
Sobre jornal do aventureiro atualizado resolvi aceitar.


You should say what "some more stuff" is in your commit message ;) don't worry, you'll get there some day. Keep on keepin on champ!


Can you clone this repository, upload the phar and make a pull request? Thanks. Also, we have tried to fix that bug. It should work, although there are some little glitches here and then.


Ahh. Alright will do.


missões.


We are seeing the following errors with the latest code.

Traceback (most recent call last):
  File "/usr/lib64/python2.6/threading.py", line 532, in __bootstrap_inner
    self.run()
  File "/usr/lib64/python2.6/threading.py", line 484, in run
    self.__target(*self.__args, **self.__kwargs)
  File "dynamodump.py", line 200, in do_backup
    f.write(json.dumps(scanned_table, indent=JSON_INDENT, ensure_ascii=False))
UnicodeEncodeError: 'ascii' codec can't encode character u'\u2019' in position 74773: ordinal not in range(128)

This article http://stackoverflow.com/questions/11759070/python-json-loads-dumps-break-unicode suggests encoding the dump result - error above looks like it is trying to encode unicode with an ascii codec so rejecting unicode characters.

Thanks


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 34](https://teamcity.labkey.org/viewLog.html?buildId=286379&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 17 (4 new), passed: 31 Build time: 01:16:20
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProductsSearch: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

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


I didn't make a huge change. Tis fine
On Nov 19, 2015 19:14, "Kyle Williams" <notifications@github.com> wrote:

> You should say what "some more stuff" is in your commit message ;) don't
> worry, you'll get there some day. Keep on keepin on champ!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/prithajnath/Real-Time-Whiteboard/commit/c1666ff987518ef6808943d41f867b6d2eedfa17#commitcomment-14515801>
> .
>


Sorry if i talk too much!!  You wanna me to upload a phar here in github? Im not a very active user in github so i  know nothing about it options,  all my past projects was on my storage and tested offline :D


It is really a good change


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


You probably want the check
```
if (!spank_remote (sp))
    return (0);
```

here since you only actually use `topology` when plugin is running in remote context (also it is only freed in remote context)


:heart:


This annotation needs to be on top of your function definition.

so before `def heightball(bot,trigger):`


Out of curiosity: was it accidental to nuke the roxygen style docs for `makeExampleDataset`?

Also, are you on roxygen2 v4.1.1 (instead of 5.0.1) for a particular reason?


could you change the first sentence to "Do not pass an anonymous listener to  Apptentive"?  And show the following as the wrong way, possibly in red?

//Do not pass anonymous listener to Apptentive
Apptentive.addUnreadMessageListener(new UnreadMessageListener() {
    // Your code
});


Put "shakes the magic ball" into quotes. It is a literal string you want to print out on the screen.


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


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


 @jkandasa
Howcan i translate an alert like this?
 ```
//ON/OFF Sensor
  $scope.onOff = function (sensor) {
    if(sensor.status === '0' || sensor.status === 'OFF'){
      sensor.newPayload = 1;
    }else{
      sensor.newPayload = 0;
    }
    SensorsFactory.sendPayload({nodeId: sensor.id, payload: sensor.newPayload},function(response) {
        alertService.success("Payload sent to ["+sensor.nameWithNode+"], Payload:"+sensor.newPayload);
        $scope.updateSensor(sensor);
    },function(error){
        displayRestError.display(error);
    });
  }
```


group, not goup :)


You may want to check *validty* of any options passed to `--mpibind` here in "local" context. This way user gets an error back on stderr before a job is launched. I may have missed where you do this already. In this case, you can delete this function -- it won't be called if it doesn't exist. (In spank plugins you are only required to define needed functions)


103643052-FiniteDifferenceFluxOutput (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1960 of 1969 tests passed)

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


Check out GitHub desktop - it's a GUI for GitHub. It's okay if you can't or don't know how to, I'll just ask @legoboy0215 to do it.


Copy the file into the Yumiko project. I would put it in `data/8ball.txt`


Ok,  so u wanna me make a pull request were i upload a phar?


Nice tip.  I'll change that (and a few other similar sections).


Yes, to the releases folder. Clone this repo first, then upload the phar to the releases folder. Thanks'


TeamCity pull requests :: netty [Build 693](https://garage.netty.io/teamcity/viewLog.html?buildId=3384&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4796, ignored: 21 Build time: 00:36:09


TeamCity pull requests :: netty [Build 694](https://garage.netty.io/teamcity/viewLog.html?buildId=3385&buildTypeId=pulls_netty&guest=1) is now running


This messes up the javascript in view pages.


Esse é o aviso que aparece quando você pega level 15 de classe correto?
Se eu não estiver enganado sugiro mudar para algo como: "Mudança de classe" ou "Evoluir"


develop (kbenne) - i386-Windows-7-VisualStudio-12: OK (2334 of 2372 tests passed)

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


103643052-FiniteDifferenceFluxOutput (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I wonder if it would be safer to initialize hwloc topology here in the context of the user. I can't remember when SLURM applies cpuset and other builtin containment strategies, but if they happen after `spank_init` then the hwloc topology may not match what the user's tasks actually have access to. (also just in case there is some bug in hwloc, might be more secure to run it as user instead of root)

This isn't a requirement by any means, just a thought.


nice, good work!


I'm pretty sure that returning `(pwd)` as a default would work.  But I think I'd rather call `hg` the exact same way that the user is, just in case there is some weird inconsistency in Mercurial.  Unless you strongly object.


You mean this commit right? I will do my best :D


Yes, they should.

Whether we can get idx to perform bounds checking is an interesting question.  It could at least reject things that are obviously wrong, but I am not sure how much more can reliably be done.


This URL does not work:
file: https://liferay-test-01.ci.cloudbees.com/job/blade.tools/lastSuccessfulBuild/artifact/blade.cli/generated/distributions/executable/blade.jar


huh - which manifest are you testing on?


This paragraph is still outdated though, isn't it?


I think the `is_broken_pipe_error` implementation used in https://github.com/django/django/pull/3121/files#diff-f6d1c75ec606389da5af6558bf57f171 is a bit more robust. I'm not a big fan of testing against the `repr` string.


You are right.  I will fix that (tomorrow).


Seems legit


Banner style


@jemc Are you sure this isn't a typo? Shouldn't it be this?
```
for callback_type.argument where defined (argument.ruby_coerce_arg)
    $(argument.ruby_name) = $(argument.ruby_coerce_arg:)
endfor
```
Just wondering. I'm not sure.


Insira o nome do seu novo mascote


I just looked through the code, and this appears to call into arbitrary methods on arbitrary encoding providers.  Since they could theoretically throw anything, I guess I'll change it to a catch-all, as we don't want some poorly written encoding provider preventing usage of console...?


@sindresorhus SMIL animations are deprecated in a number of browsers. Any chance you could re-PR the header minification?


I strongly agree!  It's switched and not having own color is really confusing still...


Here are the assets for this project.

![enemy](https://cloud.githubusercontent.com/assets/13373270/11288653/d011a22a-8f71-11e5-9eeb-ab6d74adaea9.png)
![mc](https://cloud.githubusercontent.com/assets/13373270/11288654/d058c63c-8f71-11e5-8e61-79ab77c15f68.png)
![bush](https://cloud.githubusercontent.com/assets/13373270/11288656/d05bde12-8f71-11e5-85de-e696d2744212.png)
![bush2](https://cloud.githubusercontent.com/assets/13373270/11288655/d05b1c84-8f71-11e5-9874-fdd5fccd22ca.png)


This looks like a lot of changes for a broken link ;-). Shouldn't we only be changing line 7?


Fechar janela de adoção de mascote


typo: encapsulation


104921238_EnergyPlus_Performance_Framework (lefticus) - x86_64-MacOS-10.9-clang: OK (1962 of 1962 tests passed)

![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)


>  just looked through the code, and this appears to call into arbitrary methods on arbitrary encoding providers. Since they could theoretically throw anything, I guess I'll change it to a catch-all, as we don't want some poorly written encoding provider preventing usage of console...?

Hmm.. in that case, should we also change the windows code to do a catch-all? There is a potential for a behavioral difference between Desktop and NetCore.


Guerreiro


Not an error, the makefile expects to pass the ld options through the compiler.


FYI per VEP devs, not putting a CSQ value at all is expected behavior if VEP cannot annotate the variant.


Do we want to even allow this though? If we try to drive trust, etc. this is not a good suite. WDYT?


That's weird. Yes, it should only be line 7. Don't know why line 233 was included in this. Let me try again.


It looks like Intyre is creating a 13 branch and putting it on the official jenkins.pocketmine.net, so I think we're going to go that route.


:smile:


No, the repo. Not the commit... Clone the repo. Put the phar you compiled into the releases folder. Submit s pull request.


{memo X}


And it appears that 233 has no actual differences...


Really? That's nice


He hasn't done it yet. But he will soon.


This is the fix for customer reported issue from Blackstone - DE12312


TeamCity pull requests :: netty [Build 694](https://garage.netty.io/teamcity/viewLog.html?buildId=3385&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5298, ignored: 21 Build time: 00:15:52


You're definitely getting the IP and port correctly via the helper. The same for getting the username and password from the winrm config. :+1: I think we can refactor the annoying duplication of getting the entire winrm information set in the future and is out of scope here.

My only question is around the optional user name. Username and password is always required for winrm connections. It also defaults to `vagrant` so it its probably just an unneeded check.


Hi @FuadEfendi , your URL uses blade.cli instead of com.liferay.blade.cli. I use the following URL in the commit and when I click it here in GitHub, it takes be to the JAR. Can you try this URL?
<https://liferay-test-01.ci.cloudbees.com/job/blade.tools/lastSuccessfulBuild/artifact/com.liferay.blade.cli/generated/distributions/executable/blade.jar>


Nope, `create_adjustment` returns `true`. Even if it didn't, I'd argue it was returning a value indicating truthiness, as that is how the other Promotion Actions behave.


TeamCity pull requests :: netty [Build 692](https://garage.netty.io/teamcity/viewLog.html?buildId=3383&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4796, ignored: 21 Build time: 01:07:58


Cancelar troca


Minha lista de trocas


@gildegoma if we can get it into 1.8, let's do it!


Ai ai ai ...... Pauraaaaaaaa .......kkkkkk

Enviado do meu iPad

> Em 19 de nov de 2015, às 22:07, philippedevloo <notifications@github.com> escreveu:
>
> This is the "only" thing I wanted to commit...
>
> —
> Reply to this email directly or view it on GitHub.
>


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


It's the last paragraph of the file. Maybe it's an invisible line break change or something.


ah, fair game. I assumed create_adjustment would return an adjustment. My mistake. Clearly.


My idea is like:
ot=1   OCG
ot=2   TCG
ot=3   both in OCG/TCG
ot=4   all custom cards not in OCG and TCG

Custom cards including but not limited to Anime cards, and what ot=4 should mean can be discussed by the players before starting a duel.


This method does not need this argument.


I'm assuming you mean: `Recommend to use this with a filter...` ... or maybe I don't know what wtih means :-)


Interesting... I'd thought the `MultipleObjectsReturned` errors were kind of randomly happening in all of our tests, but looking through the build logs, all the recent failures with that error do seem related to this test. Nice detective work.

Instead of sleeping, do you think we could use the same approach edX uses in their bok choy tests?

```python
def is_ajax_finished(self):
    return self.browser.execute_script("return jQuery.active") == 0
```

(And then wait until `is_ajax_finished()` is true before proceeding)


Choices:
URL.createObjectURL
1. [Using URL API (createObjectURL)](http://stackoverflow.com/questions/6775767/how-can-i-draw-an-image-from-the-html5-file-api-on-canvas)
1. [Using File API (FileReader)](http://stackoverflow.com/questions/13938686/can-i-load-a-local-file-into-an-html-canvas-element)


new line


Ok will keep that in mind in the future.


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


@paddor I had to take a moment to refamiliarize myself, but yes, I believe should use `ruby_coerce_ret` here.  The `ruby_coerce_arg` expression is for converting Ruby object to FFI pointer and `ruby_coerce_ret` is for coercing FFI pointer to Ruby object.

For a method wrapper (where we use Ruby to call C), we convert the arguments from Ruby to FFI and the return value from FFI to Ruby.

For callback functions, this is a little different because the call direction is reversed (C is calling Ruby), so we convert the arguments from FFI to Ruby and the return value from Ruby to FFI.  However, as I explain this, I actually notice that I forgot to add the code that converts the result of the `yield` from Ruby to FFI using `ruby_coerce_arg`.

This is because the "arguments" of a callback type


@kenmazaika @Matt-Lepley I like it! Maybe add a page that displays an individual user's posts? It's probably a bit much to start off, but it could be a cool feature to consider down the line.


I'll look into alternatives...  if worse comes to worse, we can provide
guidance to upgrade older systems.

On Thu, Nov 19, 2015 at 5:36 PM, Derek Collison <notifications@github.com>
wrote:

> Do we want to even allow this though? If we try to drive trust, etc. this
> is not a good suite. WDYT?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nats-io/gnatsd/commit/0675a8fd1fa274c52cc05cc0cc0bf4f1feaa5dd4#commitcomment-14516067>
> .
>

--
*Colin Sullivan* | Principal Engineer
colin.sullivan@apcera.com <brian@apcera.com>  | @ColinSullivan01


????? je comprend pas ce que ca veut dire...


+ETC_20150317_001252	{memo X}$Grande Planalto Isolado


I will probably revert this change back to using simplejson.loads() due to issues w/ the stdlib's way of parsing JSON. I remember having issues w/ Python parsing malformed unicode (https://bugs.python.org/issue11489) and simplejson has proven to be more robust.


This should solve one of the issues mentioned in Issue #6


catching all exception will be good here because some other exceptions can occur when we have registered provider. there is no real difference with the desktop except that in the desktop we have all encodings supported by default even without providers. but if you have provider, same thing can happen there too


@cbas That worked!!! Thanks! :rocket:


@IIvec By the way I am still not sure that my formula is correct in a logical fashion (let alone precision!!), I'm not even sure anymore how exactly I got it first, so tell me if you find any formula backed by a thorough reasoning to use here! :-)


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


This is the wrong place to store this state. We should never assign CurrentBreakpoint if the condition is not true.


```python
@commands("8ball")
def heightball(bot,trigger):
```

the `@commands` is a function decorator. It decorates the function `heightball` if you put it on top.

But you can't put the `with open() as f:` code right after the decorator. Or else the function doesn't get decorated and the command doesn't get defined.


Let's rename this to GoroutineID, since I feel like this is going to be a "special case" condition, even once we implement real user provided conditional breakpoints.


Might be good to get @philpennock input here, but RC4 is scary, and people moving away quickly from SHA1 as well.


Make sure the 8ball file is somewhere in the Yumiko project, and make sure the path specified in your code (`modules/data/8ball_responses.txt`) is the same as the path to the file.


Breakpoint conditions should be evaluated as soon as we realized we've stopped at a breakpoint. Maybe in `handleBreakpointOnThread` or `trapWait`.


I would make a directory called `data` inside the Yumiko project, and put 8ball inside it.