Naming things is hard :wink:

Yeah, that describes the process exactly. Thanks!


Regarding trying (maximumTime - optimumTime) / 5.
Well, maybe it is worth trying on fishtest. Locally I tried dividing by 2, 3, 4, and 8.
And they all we controversial, i.e on some time controls (like >10 s) the change was an improvement, but on some others (like 5 s) a regression. So I wanted to be conservative and chose division by 10 :).

Let's see how this test ends. Should do next tries with sprt[-3,1]?


I can't say I fully understood it either, but the light reading is:

- kicks off a replication
- spins out a goroutine to watch for errors
- waits for the first node to be hit by the replication
- destroys the lock without the replication's permission
- goroutine expects to see the lock renewal error, and it expects the error to be fatal (ie would stop the replication)

I may well be introducing another flake by removing this call... not sure to be honest. I don't know if pkg/replication is important enough to justify a serious cleanup so that tests like this make sense. I'm just sick of travis turning red on failures that weren't mine.


Note: still errors when I tried to combine both .js files.
New error:
TypeError: DashboardYelp is undefined. But at least it's in the list of assets now.


The order on features is here to ensure an offset on their depth and let the depth buffer discard elements in the order that we want them to. On ES, icons are projected onto the screen (in order to have arbitrary view label placement) those are drawn as 2d elements with a depth inconsistent with other features of the world (polygons/lines); as a result they would always be drawn on top of other elements since they are really close to the camera in terms of depth.


yes, that is equivalent and better


Hm, oder eher Erfrierung heilen?

Restore Frostbite


False, actually, we are thread safe there.
Once created, it is safe to call AddRef / Release, and it will release on the last usage.


# Testes que passaram após a correção:

* testCanMakeRecipeWithExactlyEnoughAmountOfMoney


a


"official"


>If people are having problem with ghc-mod please direct them to #ghc-mod

Will do! I know having people in a more focused channel can make it easier to get things collected properly.

>I've met people using Atom that don't even know they're using ghc-mod with everything working like a charm ;)

That's very, very good then. Did you run into cases like that before Stack was released?


That is very inspiring too ^^ :+1:

 void PluginFinder::addPathToScan(QString path){
    if(!scanPaths.contains(path)){
        scanPaths.append(path);
    }
}


DANG YO


mum help someone is harassing me on github


- whish -> wish
- brivety -> brevity


fix for:
↘ python3 define -t penis
PENIS:
The male organ of copulation in higher vertebrates, homologous with the clitoris. In mammals, it also serves as the male organ of urinary excretion.
Traceback (most recent call last):
  File "define", line 304, in <module>
    print_each_definition(required_args, client, optional_args)
  File "define", line 294, in print_each_definition
    print_thesaurus_response(word, client)
  File "define", line 255, in print_thesaurus_response
    response = client.getThesaurus(word)
  File "define", line 71, in getThesaurus
    return response[0]
IndexError: list index out of range


Ezee, maybe may names are not clear. addPathToScan is not adding a DLL absolute path, is adding a directory to scan. So a better name is PluginFinder::addFolderToScan or PluginFinder::addDirectoryToScan.


Sorry, I just pastes with any concern. I wiil fix it tonight

On Fri, Oct 30, 2015, 12:02 AM Jaewe Heo <notifications@github.com> wrote:

> probably need --init option too.
> it will initialize any uninitialized submodules.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/react-photonkit/react-photonkit/commit/47cff74b8b41bc774c41842593bf7dfd0a89d147#commitcomment-14069485>
> .
>


И всё-таки
```
<a href="#/">Go Back!</a>
```
это правильный вариант


fix for:
↘ python3 define -w funkthat
FUNKTHAT:
Not found
Traceback (most recent call last):
  File "define", line 192, in print_wordnik_definition
    print(client.getDefinition(word, source))
  File "define", line 35, in getDefinition
    raise
RuntimeError: No active exception to reraise

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "define", line 301, in <module>
    print_each_definition(required_args, client, optional_args)
  File "define", line 280, in print_each_definition
    print_wordnik_definition(word, client, "wordnet")
  File "define", line 206, in print_wordnik_definition
    print("Did you mean: " + ",".join(parse_hunspell(word)["suggestions"]))
  File "define", line 97, in parse_hunspell
    output = hunspell.communicate(input=word)[0]
  File "/usr/lib/python3.5/subprocess.py", line 1068, in communicate
Can't open affix or dictionary files for dictionary named "en_US".
    stdout, stderr = self._communicate(input, endtime, timeout)
  File "/usr/lib/python3.5/subprocess.py", line 1683, in _communicate
    input_view = memoryview(self._input)
TypeError: memoryview: a bytes-like object is required, not 'str'


importing MediaType requires adding some extra library to the project. I thought this was too  much for importing a constant. I have declared a class field instead.


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1424 of 1426 tests passed)

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


![cry baby](https://dl.dropboxusercontent.com/u/7351705/gifs/upset/XnLNHaV.jpg)


java doc was added in a later commit.


Even after I fix the 401'g of the first PUT request, I think it makes sense to keep this if-statement.

If we keep the if-statement, we'll be able to keep from adding contests to our list of tracked entries when there's a problem pushing the actual contest data...

Perhaps I'll do the same thing for entries...


7d1dc6b5977433a98046e15767364f461e75f163
f4c0f2f4dc0deb15fedf9485b271befd4d59ef02
5b7f450c50b2a3992eb09256bd9c7e5cbc70a8e6


Change the name of the text.


> That's very, very good then. Did you run into cases like that before Stack was released?

This was at a Haskell meetup in Tokyo while I was working on ghc-mod there. I don't think the Stack support for ghc-mod was even released at that point since this was like half way through but we only released that at the very end.


# Testes que passaram após a correção:

* testInventoryIsConsumed


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


all css were moved in a another commit.


Fixed misspelling + naming consistency fix.


@groldan: Cheers mate. 0b4c859a3c7ab493e024458cf1b204e61dd8f214


yeah yeah , i wasn't talking to use it as it ,
just the filter that it is in 1 line of code with QStrings retained my attention
^^
For example , clear the cached , then use that filter to fill it again with the allowed vst dll paths
but do your thing man , i'll learn from it .


it should be possible to push a new context that binds `:trusted_information` to `nil` around those tests - say using an `around` and a `Puppet.override(...)  do <run_test_here> end`


Unfortuantely, no


done


# Testes que passaram após a correção:

* testCanMakeRecipeWithExactlyEnoughInventory


/cc @alanz @DanielG @mgsloan @gracjan please critique/edit.


1) поведение как у нативного [].map()
в функцию передается, как минимум, текущий элемент массива, 2,3 аргументы - индекс, и сам массив
2) оба правильные, т.е. если параметр 1, то можно опустить скобки


@lenguyenthedat As you can not decrypt the secret you can not access the bucket.


Typo in the comment.


I made the edit and pushed another commit to the branch for this.  Go ahead and pull it.


Why replace toxdns with curl? This is a harbinger of toxdns abandon it?


@jmcarp LGTM


Cool


Ok, obrigado.


use xcode 7.1

DVTAssertions: ASSERTION FAILURE in /Library/Caches/com.apple.xbs/Sources/IDEFrameworks/IDEFrameworks-9079/IDEFoundation/Initialization/IDEInitialization.m:590
Details:  Assertion failed: _initializationCompletedSuccessfully
Function: BOOL IDEIsInitializedForUserInteraction()
Thread:   <NSThread: 0x7f9f66779e50>{number = 1, name = main}
Hints: None
Backtrace:
  0  0x000000010328af96 -[DVTAssertionHandler handleFailureInFunction:fileName:lineNumber:assertionSignature:messageFormat:arguments:] (in DVTFoundation)
  1  0x000000010328a723 _DVTAssertionHandler (in DVTFoundation)
  2  0x000000010328a98f _DVTAssertionFailureHandler (in DVTFoundation)
  3  0x000000010328a8f1 _DVTAssertionFailureHandler (in DVTFoundation)
  4  0x00000001048456fd IDEIsInitializedForUserInteraction (in IDEFoundation)
  5  0x0000000107078081 +[PBXProject projectWithFile:errorHandler:readOnly:] (in DevToolsCore)
  6  0x0000000107079c06 +[PBXProject projectWithFile:errorHandler:] (in DevToolsCore)
  7  0x00007fff8645af44 ffi_call_unix64 (in libffi.dylib)


being pushed from out side folder structure


seems good.


This comment is pretty useless, actually. It's the distance allowed for a target before the creature moves. There's really no reason to comment about why it's there, what it does, and especially not what changing it will do (which is not anything good). I'll just remove it.


You're more than welcome to add, the onActivateEvent simply won't fire. Once you add a container to the game world, its onactivate event will fire, calling it's children's onactivateevent


This should be str.toInt <= Int.MaxValue instead of <. The same is true for other conversion methods.


--no-clone-bundle


@johanneskoester and should the char be u8?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2791](https://teamcity.mesosphere.io/viewLog.html?buildId=65611&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:15


dirty=delete the zip file too
orion-*.zip


There are numerous small advantages to using HTTP over DNS, but the main reason is because a lot of us devs want to eventually phase out official support for name lookup services (eg. purge the [disgusting](https://github.com/irungentoo/toxcore/blob/master/toxdns/toxdns.c#L105) toxdns API from toxcore).


Completely... and absolutely agree with you. You've done a fantastic job, and it sucks that people take advantage!

Perhaps you should shame them ;-)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2792](https://teamcity.mesosphere.io/viewLog.html?buildId=65612&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:11


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2793](https://teamcity.mesosphere.io/viewLog.html?buildId=65613&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:19


Make sure Vim is built with clipboard flag (`+clipboard`).

# OS X Example
`brew install vim --with-clipboard`


copied from caf

for intial boot up


What does schema_generation=4 mean compared to 3? Just wondering if that should be commented since it's not clear at a glance, and digging around for the ShotgunOverlayModel's docs or code would require me to not be lazy. ;)


101162510-SubsurfaceOutwardAngleWarning-5016 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1902 of 1912 tests passed)

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


What does returning None mean in this context? Seems weird to have a getter for a "version" and return None.


@avartanii
29/30 for not throwing IllegalArgumentExceptions


Looks good.


Typo "returned"


This change makes the next function not atomic and makes it buggy. Right now if i start 2 consumers, both pick up the same job.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2794](https://teamcity.mesosphere.io/viewLog.html?buildId=65614&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:02


That's a good idea, thanks!

On Thu, Oct 29, 2015, 01:28 Stefan Sauer <notifications@github.com> wrote:

> Can you call them "__Led_mix_colors" to make them 'private' ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ensonic/ev3dev-lang-python/commit/c868289a19e85e969ed100073dcf33fe368767aa#commitcomment-14053913>
> .
>
--

Cheers,
Denis


101162510-SubsurfaceOutwardAngleWarning-5016 (rglazer2) - 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)


It was fine already. The changes I made were barely significant.


License header. Run `gradle licenseFormat` to auto-fix


License header.


春春给力啊！


@nverke
10/30 for your BaseConverter. Take a look at the [test cases](http://my.cs.lmu.edu/~pdorin/281.stuff/BaseConverter/test.cases.txt) and please let me know if you have any questions.


Not finished with this class! That's why "EnumClefType t" is not being used.


YOYYOOOOO


go tii


Updated, thanks.


Should we not try this before we even execute Corral? Shouldn't this be the first thing we try?


Is there a reason why we use this special golang container ?


Multi-munged


Why Goodbye? Hello is so much better!


:+1:

On Wednesday, October 28, 2015, Justin Sainton <notifications@github.com>
wrote:

> We have advised people, for millenia gone by, to use Say What to adjust
> strings. With much thanks to @leewillis77 and @pippinsplugins, this filter
> will maintain back compat for those changes.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/wp-e-commerce/WP-e-Commerce/commit/f24b2c5c75eee39e878a0bf7a382335755079f23
>


I aolso have problems with ss since 2.83


I support the elimination toxdns, but do not like HTTP.
I hope this is only an interim solution.


oh la la la la ... qu'est-ce qu'ils sont beaux tes messages de commit ...  dommage que ce ne soit [pas le cas pour tout le monde](https://github.com/mackoj/ConfTV/commit/c6acf397603404b558dae15848f8449e25a57afa#commitcomment-14008484) :trollface:


101163020-SetpointAirTempWorksOneWay-4627 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1425 of 1425 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.4%25-green.svg)


Blah blah


101163020-SetpointAirTempWorksOneWay-4627 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1961 of 1961 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)


More explanation:
NCF api `shortDisplayName` shows mascot "name" which is the same as `["team"]["name"]`, e.g. "Blue Devils"
NCB api `shortDisplayName` shows college "name" which is the same as `["team"]["location"]`, e.g. "Duke"

Made the change to achieve consistency.


Some of the tests under this describe group configure keyspaces with RF of 3, this causes queries to fail at LOCAL_QUORUM.  To keep this consistent with prepare:1 tests changed to use a 3 node cluster.


@GianlucaGuarini , I think the directories `test/compiler` and `test/specs/compiler` can be merged to be a duplicate of the `test` directory of the riot-compiler.


In other contexts we are now using the form

     ti is not None

instead of

   ti != None


Close #14


Please clean up these assembly references to not use fully versioned assembly names, per proj file conventions.

Compare with PR #936


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


This should almost certainly not be here. Right?
Why would you return unknown when running in debug mode?


Also closes #5 which was duplicate of #14


Alternatively, you could also create two variables `lowNumber` and `highNumber` to only have one loop:

    NSInteger lowNumber = MIN(number, otherNumber);
    NSInteger highNumber = MAX(number, otherNumber);

    while (lowNumber <= highNumber ) {
        ...
    }


Hahaha, now that I've reread this, I understand what it is trying to say. When I first read this, I thought it was talking about the conceptual difference between these two distances. Instead, it is talking about the mathematical difference (as in subtraction) between these two distance.

Regardless, I don't think it's necessary to say this here since the equation on L1372 makes it obvious.


@OceanHelloWorld
5/30
This does not compile. I removed the first line to make it compile but the majority of [test cases](http://my.cs.lmu.edu/~pdorin/281.stuff/BaseConverter/test.cases.txt) fail.
Please let me know if you have any questions.


Are mungers run more than once? I don't see this in the master branch.


Thanks!
You save my time

recently changed :
import org.xwalk.core.internal.XWalkCookieManager;
=>
import org.xwalk.core.XWalkCookieManager;


Hope this helps!


dope


Same as a previous comment re: "Info Panel" vs. "Panel"


good job! masterful spelling.


couldn't believe I made that mistake :)


Any reason not to "except Exception:" instead of the bare except? Bare excepts catch much more than is typically appropriate to catch and ignore.


Associate id of SSH key saved in SoftLayer to the provisioned instance so that we can authenticate with the key in ssh_private_key_file later.


@syohex Thanks for fixing this.


Ah, this is not in the 1.1 branch


Maybe we should delete the v1.1 docs and re-run


Same as previous comment related to bare except calls.


For consistency, should we add `NSURLIsSymbolicLinkKey` to the `infoRec` and create an `isSymbolicLink` method for this type of test?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2796](https://teamcity.mesosphere.io/viewLog.html?buildId=65619&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:26


I'm overseas and don't have access to my licensed 010 so this is from
memory: I was getting errors complaining about lack of argument passed to
ReadInt, and LC_CODE_SIGNATURE structure was not being parsed as result.
Sample is one of apple provided binary in /usr/bin, such as lsof or zprint.
I can get the exact binary when I get back. I was using 010 4.x version so
maybe that has something todo with ReadInt expecting an argument.
On 24 Oct 2015 3:41 am, "Tim Strazzere" <notifications@github.com> wrote:

>
>    1. This is interesting and I guess I never noticed that you can pass
>    in an argument to ReadInt for a location.
>    2. I'm unsure why this would change anything as there is an
>    FSeek(blob_begin) immediately before it?
>
> Can you provide a binary where this was erroring?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/odein/010Editor-stuff/commit/87ec04d10fc741a1df39b8a77ce9d298ab9be3cd#commitcomment-13961071>
> .
>


Это форк, форк инициируется как есть. Правки в следующих коммитах.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2797](https://teamcity.mesosphere.io/viewLog.html?buildId=65620&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:02


radical


You can remove this one


Make this better >:(


add another const, maybe `const targetCache = new Map()`


"tailored" ;)


This change originally prevented the ability of the constitution to effectively define non-voting officers of the committee - I believe this modification retains the intent of the change.


tango_with_django_project/templates/registration/profile_form.html

Above file accidentally added, should be in next commit.


Yes,sir. I'll recommit later

Thanks & Regards

2015-10-28 14:33 GMT+08:00 Jim Huang <notifications@github.com>:

> Don't mix the independent changes into single GIT commit.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JoeJJT/jit-construct/commit/1517e6ad6bef1a171ce8ef31235aa0a0243d7b73#commitcomment-14030986>
> .
>


Then you can replace this with `cachedTarget = cachedTarget || new Map()`, or just init the map where you define *cachedTarget*


Good point. https://github.com/kubernetes/kubernetes/pull/16486


I've strengthened your wording to clarify the timings. I believe we discussed time periods of the order of 3 college days in the meeting, so I've updated this to reflect this.


No, I didn't notice that. Thanks for mentioning! That side-affect should be documented. I'll make sure it does.


10 college days is two weeks - after they've been officially approved? Was there a reason for this? Have reduced to 3 days in line 136 of the new version


Required by higher-up policy, I believe


Cool!
I did write some stuff for quad mesh XDMF in this pull request: https://bitbucket.org/underworldproject/underworld2/pull-requests/2/added-2-3d-xdmf-support-for-quadratic/diff
Might be helpful.


"A time prior to the next meeting" could be 1 whole year, which seems excessive?! Hopefully this new wording is a nice middle ground?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2798](https://teamcity.mesosphere.io/viewLog.html?buildId=65621&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:17


Cheers Luke! I mentioned your bitbucket pull request to Julian a few days ago, so we will prob incorporate your work shortly.


yes.

 I think `2.1.*  2.2.*` is better.

What do you think?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2799](https://teamcity.mesosphere.io/viewLog.html?buildId=65622&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:02


It seems a bit *ott* to involve the committee in this decision? It's just a case of nominating *someone* to have the role, just so someone is actually responsible for it. This would, I imagine for most small events, be the TD


There is a typo here. It should return `unknown` outside of the if block.


d


Ask KG


101162510-SubsurfaceOutwardAngleWarning-5016 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (838 of 838 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-24.31%25-green.svg)


This is the wording from the Constitution, whence I have removed this line


@gregcwinn can we make sure to get these environment variables in rainman tomorrow so that they aren't nil in acceptance/production?


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - 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 don't think `2.1.* 2.2.*` is better.
Because these notations are workaround for rvm bug. see: https://github.com/travis-ci/travis-ci/issues/2220


@mikewalch  do you remember why this if is here?


@BillyBarbaro is there anyway to do something like: Buttons.size - 1?


I think this should be `instanceof Sumbject`?


Nice! I'll merge and tweak a couple things - the config will be just node not nodeless


@vsavkin having `setTimeout` does not seem right here. Maybe run on next microtask? `setTimeout` seems to heavy weight.


More modern syntax is "except Exception as error"...unless that came about in 2.6...which it did because I just looked. Blargh.


The motivation behind this change was to simplify the implementation of a RequestHandler (from a user point of view). I was trying simplify the non-routing logic.
I didn't have the context behind the decision of introducing the `initialRequest`, it all makes sense now.
I'll refactor.


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - i386-Windows-7-VisualStudio-12: Tests Failed

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


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


You are welcome :) Thank you for you are in touch.


Commented code should perhaps be removed?


你改进的很不错，赞。


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


I just needed iptables package to be installed in the container before dnet starts. Dnet needs it way early during init so I cannot side load iptables after container starts. Unfortunately no official base image satisfies all my deps. Ubuntu had iptables but had a very old unshare program. My custom golang is a good solution for now


BasementDomainNumCounterBug (mmatts) - Win64-Windows-7-VisualStudio-12: OK (1960 of 1960 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)


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


Wow! thanks!


I see, probably better to have it in a zalora-deployment branch, what do you think? :smiley:


Strange things happen when you write a 200+ page working draft.


Because of the way the GDAL trampoline stuff works, this will throw before it hits this else statement. That's why there was a try/catch there in the previous version.


Test Readme file.


Merged to master branch


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (769 of 769 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-18.84%25-green.svg)


I think we need deal with ideal and max separately when scaling (effectively scale twice), because their requirements are different.

E.g. consider a 4000x4000 *max* constraint vs. a 4000x4000 *ideal* constraint when applied to a 8000x4000 original.

    .------------.    .------------------------.
    | constraint |    |        original        |
    |            |    |                        |
    |            |    |                        |
    |            |    |                        |
    |            |    |                        |
    '------------'    '------------------------'

The *max* constraint requires scaling original by the smallest factor to fit entirely inside. `scale = min(1, 0.5) = 0.5`:

    .------------.
    |            |
    .------------.
    |    0.5     |
    '------------'
    |            |
    '------------'

Whereas an *ideal* constraint means "as close as possible" (above or below). e.g. average `scale = (1+0.5)/2 = 0.75`:

        .------------.
     .------------------.
     |        0.75      |
     |                  |
     |                  |
     '------------------'
        '------------'

Since we can have both ideal and max constraints at the same time, and they can have wildly different size and aspect, I propose we scale by ideal first (if present), and then scale the result of that again to fit inside max.


ReadMe File


@flavour I solved the issue by making person_filter as a separate filter_form in the main form. When the form is accepted, the ```id__belongs``` filter is saved in the ```comments``` of ```pr_subscription``` table. While the filter except person is saved in the ```pr_filter``` table.

Everything seems fine. The value of subscription is of form:
```{'get_vars': {u'location_id$L0__belongs': u'Afghanistan', u'language__belongs': u'en-US,tl', u'event_type_id__belongs': u'24'}, 'method': ['EMAIL', 'Sync', 'SMS']}```
and the value of recipient_subscription is of form:
```{'get_vars': {u'id__belongs': u'10,5,1'}}```

Yet the subscription is loaded but not the recipient_subscription. What am I missing here?

The snapshot is as:
![capture](https://cloud.githubusercontent.com/assets/4207677/10810175/96a83686-7e30-11e5-9dd0-921ff1350fe0.PNG)


We were only catching `pdal_error` there before, would that have caught it?  What kind of error do we need to catch - or should it be a catch-all?


This will...? Incomplete thought?


Yeah, that's what the trampoline throws. It used to throw a specific `pdal::gdal_error` but that seems to be gone now.


For `$href="#foo"`, this will return `()`


Returning two arrays feels a bit unreliable for callers - there isn't anything in the response that guarantees that casOut[2] corresponds to keys[2]. There's no problem in the actual implementation - it should be doing the right thing - but from an API perspective it feels a bit flimsy.

What do you think about either
(a) following the model used by GetBulk (returns a map indexed by key), or
(b) accept an array of struct as input, and use it to return the response as well. e.g. accept []*BulkSetEntry, where BulkSetEntry is a struct with key, value, cas, err.


Also - do you think it should be SetBulk, to better match GetBulk?


No probs - I imagine it'll just be be the XDMF wrangling you'd need to steal. Might save some time to have a rough reference point!


101162510-SubsurfaceOutwardAngleWarning-5016 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1902 of 1912 tests passed)

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


This is another reason it would be nice to use []*BulkSetEntry - could then have BulkSet itself return an error in this scenario, instead of having to overload the array.


Awesome.


This is not a nice thing to say, I removed it


This is just a wrapper around a bucket (which could be a gocb bucket, I think?).  In any case could just call bucket.BulkSet and let it panic if it's a go-couchbase bucket.


test


From what I can see, this doesn't handle the case where the same element ID is used to multiple topics inside the same target file.


++Another Test


Could you cherry-pick this change on to release and merge it back to dev? We'd like to re-enable NuGetPackageVerify in Coherence-Signed release


You should use loop to TestDriver.java.


Do you working very good


use loop hai mun tam ngan na ja


We only want to scale down, so this test seems wrong. For all frameInfo.widths and heights that naturally fit inside the max, no scaling should occur.


Good catch, it should actually be `instanceOf RxObservable` as Subject and EventEmitter both inherit from it. Fixed and added a few tests to check.


Same change required here too?

https://github.com/JuliaLang/julia/blob/8cb9e6f5b6f57455eb911dbb5ac3905d540378f8/base/multi.jl#L1308-L1310


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


Any reason for moving `Regex.match?` into another function?


If author feels ready to submit a pull request he can do, and it will be
evaluated by maintainers.

On Wed, Oct 28, 2015 at 10:10 PM, Atahan-Turkoglu <notifications@github.com>
wrote:

> dear @mcostalba <https://github.com/mcostalba>
> please merge this patch into master. time is running out, superfinal is
> coming.
>
> no regression at STC th 3:
> http://tests.stockfishchess.org/tests/view/562de2110ebc5964d344609a
>
> +3.44 elo gain at STC th 24:
> http://tests.stockfishchess.org/tests/view/5629dc9e0ebc5964d3445e9a
>
> no regression at LTC th 3:
> http://tests.stockfishchess.org/tests/view/562ed6320ebc5964d344613e
>
> very promising test at LTC th 24:
> http://tests.stockfishchess.org/tests/view/562e2dbd0ebc5964d34460c7
> JojoM-24cores: 114 wins, 108 losses, 778 draws.
> fastgm-31cores: 111 wins, 82 losses, 585 draws.
> vdv-31cores: 120 wins, 95 losses, 651 draws.
> JojoM-24cores: 74 wins, 66 losses, 383 draws.
> Last situation: LLR: 1.26 (-2.94,2.94) [0.00,4.00]
>
> this is a good patch. sf binary for superfinal will need this. sf fans
> want this. every sf fan attach great importance to tcec. sf fans don't care
> ccrl as much as tcec. tcec is the most prestigious tournament.
>
> please merge this. future days, if low thread tests go bad, you can
> revert. but now we need this patch.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ajithcj/Stockfish/commit/e8b409d740e3170aaa52d0304bab330a845e6ffe#commitcomment-14052207>
> .
>


[fisman.xlsx](https://github.com/guerramarcelino/regressoesespurias/files/23069/fisman.xlsx)

Dados utilizados no post "A cultura da corrupção". Fonte: Fisman e Miguel (2007).


Did you test this? It didn't compile :(


Woah, nice one. I've been meaning to do this for a while. That NBNotifier was always the plan, too.


Hrm, seems like we should (reverse) translate the errno here too for completeness. Or, assert(0) if we never take this path.


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


Call this ceph_to_host_errno(x) to make it clear it's a one-way translation?


@shrayasr its something which does not belong to the abstraction of do_match. It belongs to a lower abstraction. Hence with it being in a diff function the logic of how the match is happening can be changed without touching other parts of the code.

In future, I think this function will be moved into its own module. Because I know matching involves
1. Load up rules
2. Do the match (might not be regex)


interactive python moved to free books


Does this have anything to do with the `ResourceType._get_or_create_cache = {}` that was added to ProgramFrameworkTest.setup()?


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


(commit 1880a7c2)


It's supposed to be infinite.


Happy to assist :)
It looks like you also no longer have a use for the "target" and "name" arguments ;)


Indeed it does...result of a 2am code refactor :P Fixed now.


![icon png](https://cloud.githubusercontent.com/assets/14905987/10810631/2eb8d8ca-7dd9-11e5-901f-43b2bfac322d.jpg)


could we maybe split this up into two steps to make it more clear what's going on? e.g.
```
- cat requirements.txt | perl -p -i -e 's/>=/==/g' > exact_min_versions.txt
- pip install -r exact_min_versions.txt
...
```


it's _pointfree_ you cretan


Ah makes sense :+1: :smile:


I've run into issues with this again, for some reason `pod install` kept failing and only `pod update` was able to rectify the issue.
How about Runes, gets subspecs, say `Runes/Core` (`Runes.swift`), `Runes/Optional` (`Optional.swift`), `Runes/Array` (`Array.swift`), and `Argo` would depend on `Runes/Core`?


Hi Zalak, it's generally not good practice to comment out code when it should be removed.  Please remove and reissue to the future branch, thanks,  Brian


I hope it works. *fingers crosses*


@gabizou - agree with this change or should I scratch it?

I think it doesn't make sense for all the implementations to make their own value - boolean only has two values, and only of one type, we may as well simplify.


I Wish I Was dead....


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


This is makes it very similar to the catalog data (which also seems to need a final value...)


2am is not that bad, I did worst things later on :ghost:


I was having trouble with git, and was unable to sync my files up with the servers. So I downloaded a zip copy and replaced all my files and reuploaded. Ignore this commit


Should it not just be one test here?  Write the failing test for the issue, e.g. name would be ShouldReturnSubsciptionsWhenPagingNotPassed(), then make the fix, then the test passes.  Thanks for this Zalak, please just update and reissue to future branch.


Just a couple of small changes required, please reissue to Future branch


The above is executed in the SSHManager case where we initially launch only one worker on the remote host and then use that to launch additional workers.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.722](http://teamcity.monogame.net/viewLog.html?buildId=10965&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 551, ignored: 6 Build time: 00:20:47


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


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Where these two files meant to be checked in?


howdy @RickCarlino !

I'm making a (very mundane) MUD about sitting on a train.

The interesting part is that I'll deploy it not on the public internet, but on a ESP8266-powered mesh network *on the Chicago subway*. I can leverage the lack of WiFI in train tunnels, and name the access point `xfinitywifi` so many phones will autoconnect, and then display a js telnet client rather than the expected 'accept the terms and log into the network' captive portal popup.

That way I hope to create a social (if potentially a bit intrusive) experience from the antisocial reality of the commute. It's a lot easier to type `say hello` than to *actually say hello* to a stranger on a train.

Basically a chatroom, but I think styling it as a MUD gives the whole project a kind of friendly-nostalgic vibe.


I thought htis was a better lookup that flipping through all the times :)


... :| yeah. way better.

movepopup still snaps. :+1:


Where you got it?


So many cleanings.
Sorry about the global variables, only just getting my head around scope in JS :)


@angerman If you're getting unusual results it sounds like more of a CocoaPods issue than an Argo issue. You might open an issue with them to see if there's something they can improve. Breaking Runes into subspecs would require source changes and wouldn't work well with other distribution methods.


btw, any reason `equipped` is an array?


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


Assalomu alaykum.

color: #fff;
     display: none;
     box-shadow: 0 0 12px rgba(0, 0, 0, 0.1);
+    z-index: 3;
 }

ok ishlar joyigami


Should we default to AUFS and allow it to be configurable? It requires new kernel versions, but aside from that… ?


color: #ffffff;


Yes, but not really usable image - for developers only. Check this thread: http://www.orangepi.org/orangepibbsen/forum.php?mod=viewthread&tid=119


Hmmmm... the other option is to not specify a driver by default and let the auto-detection stuff do it's work. That will result in aufs or devicemapper as a fallback.


I have always wanted to save the planet


Thanks Dan!


Consistency?


Be careful comparing checking against NaN this way. Try this out in your console. Is it giving the expected behavior? Check into using the isNaN() function.


@pranavkm There is no release in DnxTools. The same feed is used in both dev and release.


Thanks @trauber!


Why hide that the file does not exist?


There was a small hiccup on my part and I had to commit super quick so on the commit: Add jason to other-responses folder
I didn't write your github username. Not a big issue but wanted to let you know just in case you were wondering lol.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.723](http://teamcity.monogame.net/viewLog.html?buildId=10969&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 551, ignored: 6 Build time: 00:20:15


The time taken by the kernel layer to complete the PDO buffer setup was found to be greater that 4 secs in worst case. So a higher wait time is required for Windows kernel interface design.

Please let me know if I should be prepare a fix commit to extend the comment. Thanks


Oh yeah... RxJava...


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1301 of 1301 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.13%25-green.svg)


Please change this to "Automatic Restart on System Freeze".


Najnowszy commit usuwa ta rzecz.


Yeah that sounds more sensible…


@gfontenot thanks.


please remove this patch!!!


We could just make the whole `DOCKER_DAEMON_ARGS` configurable, forget about making individual env args…


You should add a `bark` property


What happens if the user passes an empty string to the prompt (by just pressing `enter`)? What is value of `years` after `parseInt`, and how does that affect the control flow in your following while loop?


then you can configure the socket location, or do whatever you want really…


There's only one backtick for inline code. Please fix.


It is already! If you override DOCKER_DAEMON_ARGS it will be used. The value in the script is a default.


Thank loads to @DimalChandrasiri  and @dakshika for the concept :)


I reckon you are right though, keeping it as the docker defaults is probably sensible, provided the image has the correct modules to pick aufs over devicemapper.


rollback


I think we need to fix this string interpolation since ecmascript does it differently from coffeescript. Eg. `floorplan.${loc}.x`


@RobLoach how about:

In this section, we show how organisms manifest content and interactivity to demonstrate the functionality of key site components.


Yep, coming up quickly. Currently stressing out over preparing the presentation.
So you be there, too? Please add a gravatar picture to your github account so I have a fighting chance to recognize you :)


@jsha I just committed 30c949085cad82d07562ca3403a22513b8fcd440 which is kind of an initial attempt at checking the OCSP status every hour and, if the status changes at all, the server is restarted, which updates the OCSP stapling.

I tried updating the listener while the server was running, but that turned out to be racy. You were right, though, it didn't panic!


This needs to return false.


This needs to return e.handle, I think.


It needs to be "Empty Trash on **A**ll Mounted Volumes".


I se. I think `Kernel` is the linking to the 2/3D concept.


The word *hash* should be translated into Spanish: *suma de comprobación* → *suma*


ah. fair enough


Minor point, but the correct spelling is holiday.


Same here. Holiday.


6번까지


@Atahan-Turkoglu , @mcostalba, I've raised a pull request.
LTC has consistently performed above 7 ELO so far. Its clearly a matter of time before passing LTC. But no point in delaying the pull request as it is gating the creation of SF-binary for superfinals.


This is fine, it's actually in line with what needed to happen.


Sounds great, I will test it than. +1


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


good


@donghaox hey, i dont' have number parsing working yet. But I managed to shorten your logic of block comment.


Is this definitely the only case? What about `UnknownSagaType` or just `Unknown`?


1911-8-4-Update (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


u forgot to update that super important field !
item_bonus.txt#L6


弹你小鸡鸡……


why do you need to search namespaces to find supported drivers? Isn't this info static?


I think this can be written as:
`employee() { ... }`

See https://github.com/lukehoban/es6features#enhanced-object-literals


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


I really don't think it matters since we don't audit those messages and SI doe not display error messages


I think the const should be declared before the try since we use it later.


aaa


bbb


Any reason we set the socket location? i.e. is it cool just to nuke the defaults we have here?


I can see 'ed' being invoked in the ED_PG_CONF macro. Are there more calls?

For ED_PG_CONF, I think we should replace it with a small python script. Editing a postgresql.conf file is a common task, so it would be handy to have such a script. Something like:

pg_conf_edit <filename> port=1234

which would find the possibly commented-out line for "port", and replace it with the given setting.

initdb also edits the config file like that, so we could also lift the function from initdb into a separate tool.


Oh okay, @Kimmax  I didnt manage to get the repro working i was rather confused what you were saying. how do i do it.


I guess we don't need to declare evt if we don't use it in the method body.


But until we get around to do that, you're right that this should be documented, so I committed this. Thanks for the patch!


Yeah, I reckon.


This "interface" does not abstract well over JDBC—the industry standard Driver interface for Jvm languages, and doesn't provide clarity on the boundaries of Client -> Metabase -> Jdbc/SQL. There is no contract, just arbitrary keys I have no reason to provide. For instance `(date-interval [ ...])` is just one of many possible coercions possible when dealing with arbitrary data types.

Does that mean, I need to read the source code, before writing a driver?


you can add some more pics... and some other flower's information in this.. but good one


Yes. Fille have been updated.


typing smth


@thebritican


thanks to suggestion. I will try on later


@joaquinloustau Why do you need to specify that the day changes in this case?


3 is a reasonable depth. Why did you need to switch to 5?


Cool, fixed in c6b0b7a


Fix error in repeating sections for melee and ranged, add counterattack button to melee


Does the client get to handle the errors if syncing fails? What are the semantics wrt transaction boundaries?


Nope, that is intended


just use argv[1] here and remove line 37


Looks good.


Could you rename `dir1` to something like `subdir_name`, to make it less similar to `dir`?


ceylondoc support it since version 1.0, but it seems that nobody knows it, so it good to mention it in release notes ;-)


эти зависимости наследуются от модуля "module"
убери их, чтобы задваивания не было


it seems we'd keep the 2 spaces indent.


Minor typo in commit message: "maintanint".


I'm very aware this still isn't too pretty, but I'm intending on overhauling this particular section anyways in the next branch. So there's no point in putting a lot of effort into it now.


Docs are good.


Good catch, thanks!


Noted!


same indent issue here


ditto


@juansc which line exactly?


It is well spelled


OK, if the user really wants he can do:

```avocado run --dry-run --force-job-id <CUSTOM_ID>```

I guess we should not enforce any policy here. I'm OK with it.


always use { } on if else blocks even if it is just one statement in the block


mf2util! woo! oh man, long time coming. glad you added it! cc snarfed/bridgy#178.

feel free to identify other code that we should drop in favor of mf2util too.


OK.


Would it be possible to expose the syncing process in a Protocol, with well-defined steps and state-transitions? I would be interested, once I understand its behavior, to extend the behavior of sync by hooking in a pre, or post-process step. Imagine a DynamoDb driver, where I could write my own sync to S3 bucket (using my own encryption protocol). Something fundamental as _sync_ warrants attention to details.


Do we need to sort the import in alphabetical order?


Nice! Great use of strtok!


Cool, this is one of the really useful bits.


No reason, I just made a copy of someone else's code when googling for whether or not Markdown is supported. I'll remove that when I bump the version number.


This is an outstanding issue: might be able to send the refund reason id along with the initial requested in the `merchantData` field


same as above


permissions are borked for these new pics


Just for consistency, most of our unittests are using the `self.assertSomething()` idiom. IMHO we should try to stick to it.


why is this required?


change '4' to sizeof(int*)


this should be added to ignore list


Yeah, I'm all too familiar with that issue, so no worries.


Agreed with the `\n`.


change '4' to sizeof(int)


Rewriting branch history is a fine thing in my book while it's still work in progress. Sensible history über alles.


產生完記得隨手把型別調好唷。


dat I18n


Wow, foi mal... acho que enviei modificações pro seu arquivo ao invés do meu(se é que tem um no meu usuário mesmo, se entendi)... não entendo muito bem o GitHub ainda.


this is unnecessary, i already added the lines:

 # expose globals, for debugging
   window.view = view
   window.key_handler = key_handler
   window.key_emitter = key_emitter
   window.key_bindings = key_bindings


顶一个


case 2,3,4,5 都是需求的測試案例還沒出來，就偷跑了。

這是大家在練習TDD過程中，很容易犯的錯誤。一次只針對眼前的 scenario 調整 production code。其他的 production code, 等有對應的紅燈再來撰寫即可。

另外，不建議使用 -=0.25，直接改採用 0.75, 0.8, 0.9, 都會顯得直覺許多。


why is this reverting https://github.com/artus-analysis/Excalibur/commit/fad8fe4d9?


could also put
```C++
ret = -EPERM;
```

to be consistent with other branches in this switch-case block.


git likes new lines at the end of the file


這個測試案例壓上去時，結果是綠燈。在TDD的開發過程中，通常就是不太正常的情況。除非這個 scenario 被定位乘跟之前的 scenario 屬於同類似的商業邏輯。也就是同一類的 scenario。


OK.

BTW, this just strengthens the point that we need to review the individual exit codes. Even though this is inside `run_test` it's causing a job abortion AFAICT. But this will be tackled somewhere else.


please remove unused code


you should delete the definition of the error itself


Aha no problem. I was just a bit confused.


"Foresight Advantage Script by Oper8or version 1.0"


constructor 的內容建議不要太複雜。

因為對外部使用者來說，new 起來用之後，不一定會用這個功能。但那些成本就都是多的囉。或許可以寫在某一個 property 的 getter 中。這樣就只有實際取用時，才會跑那一段 GroupBooks() 的效能消耗。


嗯,原因應該是上個commit提到的,production code偷跑了,直接把相關的邏輯寫完,導致後面連續幾個test case都直接pass


You copy config.weio on bckpdir. Where and when do you restore it ?
`config.weio` contains the version, so it will change with each release. Do you handle this case ?


Have you actually been able to reproduce this?

I mean, it looks legit, and even the second check (`and queue.empty()`) makes sense, but I'm weary of adding another special handling without being certain we're actually fixing a bug and not introducing another one.


Simple. The reason to change it in the first place was because shit was
inconsistent. Some files used spaces, others used tabs.

Why spaces? Tabs are funny little characters and there is no real standard
on how long tabs should be. Sure, most people have 4 as default, but some
editors and services (git, github, etc.) use 8 by default.

If you align something with tabs in sublime, it will look fucked in git
diff and on github. That is solved by simply using spaces.


ha csináltál neki service -t, akkor miért nem onnan kéred ki? Így nem kerül a konténerbe


Hi,
Any one help to integrate geofence Enter/Exit alert to SMS


謝謝91,對不常寫code的QA來說,這的確是比較不會去思考到的地方


Fixed it in 2ce532ef5458b5701bb9048d06cd2f39fc418870


Wow, this is an extreme scenario. I'd recommend the user to replace his machine completely! :)


yeah. how do you commit perm changes?


I suppose it would be nice if we could use final static strings to replace the hard-coded command, message and event names, which would be much cleaner an easier to maintain.

For your reference:
https://github.com/crosswalk-project/crosswalk/blob/master/runtime/android/core_internal/src/org/xwalk/core/internal/extension/api/presentation/PresentationExtension.java


It was supposed to fix #8479 instead.


Lets not get ethics involve with code for wallhacks... ;p also i forgot to update this with my code thaf rewrites a majority of it.


also wow, github shows photo diffs!


Ez a megoldás sem rossz, de mit fogsz csinálni, ha több helyről is kell majd jelszót módosítani? Mondjuk egy elfelejtett jelszó, admin felület vagy személyes oldal. Mindenhol megvalósítod ezt? Érdemes lenne az entitásban megoldani ezt, mondjuk ez jóval bonyolultabb, de akkor megteheted azt, hogy plaintext jelszót adod be, és majd a setPassword metódus az entitásban kódolja, ahogy kell.


Hmm, why is this needed on OS X? I would think that it depends on the CPU you're targeting, not the OS.

PostgreSQL actually just got similar hardware-accelerated CRC code, in version 9.5. There we have quite a bit more complicated configure logic to detect, whether we're targeting a system that:

a) always supports SSE 4.2, a system
b) doesn't support SSE 4.2, or
c) SSE 4.2 support might or might not be available at runtime, and so we need to choose at runtime

If you just build everything with -msse4.2, the compiler might emit SSE 4.2 instructions anywhere in the code, so the binary won't work on a system that does not have SSE 4.2.

Perhaps we should backpatch logic that from PostgreSQL. Want to take a shot at that?


One link too much?


`numLinesOfCode() { ... }` here too?


I understand, if we're not able to import basic Python libs, we should not try to import `avocado.core.exit_codes`. But, let's at the very least put a reminder here (and there) about these, so they do not get out of sync.

Even better would be if we could somehow have a functional test that would trigger this situation (maybe a fake `logging.py` module) and check the expected exit code.


Ha ráraktad a userinterface -t, akkor itt érdemes lenne vmi salt generáló műveletet alkotni (amely saltot persze mentesz is), mert így mindenkinek azonos (null) saltja lesz, ami nem túl biztonságos.


@thebritican I want to make it do something like this


Same here about the exit code.


Érdemes kiszervezni külön metódusokba az egyes mezőket, úgy jobban átlátható.


Could this use `$URL:contentdeliverycontentcategory-list` instead? With changed base names in the router, both the urls could be autogenerated. Maybe this could even use the `$LINK$` macro to simplify it even more.


ohhhh


@jakubpawlowicz: `4.0` means `4.0.x`. I'd suggest you use either `4` or `stable`.


Haha I just found this bug myself and solved it in exactly the same way. This caused my chrome to ignore all keyboard shortcuts including the shortcut for writing the @ signs (altgr+2 on a Swedish keyboard). Could you please publish another release where this is included so that my team and others can easily fetch it. It seems to just have missed the previous release.

Thanks in advance


The naming convention of event handlers seems always use lowercase, you may refer to: http://www.w3.org/TR/presentation-api/#event-handlers. I suppose it would be better to keep align with the W3C spec style.


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


I think these need to be template strings with backticks.


path is not always same, on my box anyway its different


And it would be much better to use a variable like "var WIFI_STATE_CHANGED_EVENT = 'wifistatechanged';" to define the event name at the top of the js file, which could be much easier to maintain in the future I guess.


That is great that you brought Wifi Direct to the Crosswalk Android, I'm ok with the functionalities changes, just some nits on coding styles. Thanks!


Image source :- http://gethue.com/how-to-use-the-livy-spark-rest-job-server-for-interactive-spark/


這個不叫重構，叫調整唷。

重構指的是原本執行結果不變，只調整寫法。


I think this comment is a remain of a copy&paste ;-)


I think it's good to go. Let's run the final experiments!
What do you think first starting with a small network (CNN S) and fine-tuning it and then moving to the VGG 16 (or 19) for the final runs?


This should include Diedra!


And this should exclude Diedra!


There was a reason i didn't do it this way was that that it makes them publicly accessibly functions, i do not see any reason where the SetAlgorithm would take a different input.

We should make them un-exported functions and private.  Only few things are exposed as in Key, Bucket and Expiration etc.


Does this work in Ecmascript? Can you use `in` as a way to check whether an element is in an array?


@mcostalba If I read the code correctly then with 20 threads you will call the very expensive now() system call with a 20 time higher frequency than with 1 thread.

Why not add something like

    static int calls;
    ... && ++calls % nr_of_threads == 0 && now() - .....

to the condition at line 578?


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


hash is actually "picadillo" but it doesn't really make sense.
it's not as simple as a hash+sum, it's really a Secure Hash Algorithm -> SHA Hash.
So it's just simpler to keep it "hash" in spanish, like many other words that are better left in english.


right, let me make it private


In-case of Golang its simple enough to not do the same approach of other libraries since formData really is not private at all from the point of presingedPostPolicy().

TO fully behave the same as other libraries - formDatas scope shouldn't be accessible outside. The functions look dummy in the sense that not service much purpose other than just putting an entry in a map.


Why are you asking it in a commit?

I think the easiest way will be to integrate notification sending through the [twilio API](https://www.twilio.com/docs/api/rest). It should be quite easy to implement in a similar way that it is done for the [pushbullet](https://pushbullet.com). Look here https://github.com/vitalidze/traccar-web/blob/dev/src/main/java/org/traccar/web/server/model/NotificationServiceImpl.java#L255 for a current pushbullet notifications implementation.


有點奇特的寫法。弄得有點不必要的複雜了。


U


It's fairly obvious that hard-coding libdir is bad, but perhaps not all devs are aware of multilib or the lib/lib64 difference on some distros, so a few words explaining the negative effect of hard-coding "lib" would be good.

More importantly, "Upstream-Status: Submitted [<url>]" should be in the patch itself, not in the commit message.

Please include a pointer to where it was submitted upstream.


Dah kan mashroo3 we fashal fashal zare3
On Oct 29, 2015 3:36 AM, "Ahmed ElSayed" <notifications@github.com> wrote:

> eh el 3ak da?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/shobak101/carpool/commit/02dbb084714ae48936cbfdf6f8facd1a388286d4#commitcomment-14056341>
> .
>


functions are not dummy here ... they help in readability/maintainability
you can see how presignedPostPolicy() looks before and after.


@EugeneButusov that's true in most cases, but when we start the app after the user has logged in in the previous session, no logout is made, which leaves the following state:
1. User is not logged in with social provider (assuming no autologin)
2. User has profile in stogare

By calling logout we would ensure that No.2 is false.


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


一般的行為來說，比較像是給N本書，做一次結帳時，才算折扣。
當然，這邊的 order() 有點像「加入購物車」的味道。

這會影響到的是，API的設計是給集合，還是給單筆要呼叫多次，才能達成 scenario。

取決於想要讓呼叫端怎麼使用比較方便或彈性。


0.95 跟 95，這個測試案例應該是會失敗的吧？ XD


May be merged.


I would put Janus server details to config file too


@cvrebert: so apparently this fails.

```
C:\Users\xmr\Desktop>jekyll s
Configuration file: none
            Source: C:/Users/xmr/Desktop
       Destination: C:/Users/xmr/Desktop/_site
 Incremental build: disabled. Enable with --incremental
      Generating...
jekyll 3.0.0 | Error:  No such file or directory @ rb_sysopen - /Users/xmr/Desktop/bootstrap/node_modules/babel-eslint/node_modules/babel-core/node_modules/regenerator/node_modules/defs/node_modules/yargs/node_modules/cliui/node_modules/center-align/node_modules/align-text/node_modules/kind-of/node_modules/is-buffer/index.js
```

Any idea what might be wrong in this branch?


這一步需不需要做，我覺得都可以。

要講正規作法，當然做了會有做的好處。但要講實務，我傾向的作法是，測試資料先複製貼上也沒關係，當真的發生需求異動，導致我們需要一次改好多份測試資料的情況，那時再來重構測試程式，還來得及。

原因是，測試案例沒有邏輯，bad smell 最多就只是 duplication 而已。而這不太會影響到擴充、彈性跟可維護性。


@borisspektor, got it, thanks.


Can you add a comment noting that this test is currently disabled because it is not yet complete?


I guess it does, I only kept it for future reference if anyone wanted to look at the original.


Has been fixed with the latest version of memutils.


okay, 我搞懂你的  DISCOUNT_THREE_BOOKS 指的不是「折扣」，而是「該折扣每本書的單價」。
恩....這樣雖然可以滿足需求，但跟需求的描述上，直覺並不相對唷。

另外，如果未來每一本書的單價不同時，這邊會有問題。因為需求論的是折扣比，只是目前剛好每一本書都是100。

不需要針對目前不存在的需求額外設計，這是沒問題的。但要思考一下，改不改得動，改起來輕鬆不輕鬆。


@BecCowley I think what you describe is only true when you interactively choose the offset value via the GUI and that there's no offset value read from the deployment database. When the offset is read from the deployment database, the second time the timePP routine runs then it is not reset to zero but to the actual value found in the deployment database... So if we're using offsets from the deployment database, your fix would apply the offset twice on the FV01 dataset. Hope this makes sense.

There is definitely a bug there but it's not the one you think. I will create a new issue for it. The problem is that the first time preprocessManager runs we are in interactive mode with a GUI where the offset can be set to whatever value. On the second time preprocessManager runs, it is in auto mode with no GUI and instead of using the value set the first time it uses the value found in the deployment database or zero if none.


Can you add a comment somewhere that explains why we need both normalize and normalize.old?  Looks to me like they do essentially the same thing.


there is useful alternative to pmenu - https://github.com/junegunn/fzf


You have already cast `q5.submittedAnswer` to lower case on line 146. Do you need to do it again on this line?


Yeah, I considered that. Perhaps laziness was slightly involved to save me another `import` in `Prompt.py`.


I'm looking at the quality values of characters but having trouble tracing how this calculation works?


Yes you are correct on the time frequency, regarding the "very expensive
now() system call" I am not so sure, I have done some googling and it seems
we are below 1 microsecond, without considering some Linux trick  called
vdso that allows to read the time from
userspace, without the context switch.

Anyhow if we see scalability is poor I have already in mind the next tweak:

now() - lastTick.load(std::memory_order_relaxed) > 5 * (Threads.size() / 4)

But first I would like to _verify_ a poor scalability before adding
complexity.

P.S: Currently nodes are checked only in search(), not in qsearch() so that
when we reach the 'modulo 4096' condition in qsearch we skip the
check altogether. I am not sure this is a _real_ issue, so for now I keep
like this.

On Thu, Oct 29, 2015 at 8:02 AM, Tom Vijlbrief <notifications@github.com>
wrote:

> @mcostalba <https://github.com/mcostalba> If I read the code correctly
> then with 20 threads you will call the very expensive now() system call
> with a 20 time higher frequency than with 1 thread.
>
> Why not add something like
>
> static int calls;
> ... && ++calls % nr_of_threads == 0 && now() - .....
>
> to the condition at line 578?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mcostalba/Stockfish/commit/0d79b149986aaebd803fa58cacf5925eb1273db7#commitcomment-14059320>
> .
>


I would prefer to load it via `require('riot-compiler')` instead, trying to find a solution to the parsers `require` calls. @cognitom any idea?


每呼叫一次 getDiscountByBucketCount() 都要重 new 一次一樣的 array，再透過 bucketCount 取得折扣，這個 function 還有優化的空間。

因為可以直接定義成 fixed 的 hashTable （以.NET來說，是Dictionary<int, double>的結構，這樣取得折扣的速度是 接近於O(1)）


Can you find another place increment `totalCorrectAnswers` without needing the if-block here?


I don't think so, those were changes related to streams and components. in janus.c you access sessions. I also noticed some notifications apparently sent after the time (mostly from a few leaks that appear in AddressSanitizer) but I haven't had the time to figure those yet.


Need a space after ```if``` and before ```{```


Try clobbering your `node_modules` ? Makes no sense to have `babel-eslint` in v3.


Wow, das ging aber schnell. Sieht schon echt cool aus. :smiley:
Hab mal den rawgit-Link dazu in die Readme geschrieben


this will break the coverage reports please switch back to the previous solution. In the riot main repo we will just check the coverage of the `riot.js` file. The compiler is already tested in it's own repo


I don't think this is correct. Should you use the same docs own ID to get the lock?


Hi xiakeliu,

I'm not strong in Chinese. Could you comment in english?


Sorry about that. It was my first attempt of a very quick (and probably dirty) fix online on github without ability to test it... probably a bad idea :-)
Apparently all tests now pass again. I hope you'll consider the change!

Thanks


Oh, I see this is the same as it was before. Just seems odd.


@lucabrivio : maybe it's time for pull request.


@dannyroberts is this intentional?


在目前只有三個簡單的測試案例底下，是否需要把程式碼調整到這麼複雜，這是值得思考的。

如果現在的調整，已經是為了後續的 test cases，那就是錯誤的步驟了。因為這些新增調整的 production code，不知道為什麼要增加，因為還沒有對應失敗的測試案例，來定義現在調整的動作是為了什麼。


Sorry, but I don’t see how things are being made more complicated by providing a proper Spanish translation, instead of lazily copying words.
You don’t need to be overly precise in this context (so it being a SHA isn’t an actual problem), besides, *suma de comprobación* is a standard term for this (*cf.* Microsoft Terminology).


See issue https://github.com/aodn/imos-toolbox/issues/269


一次只新增一個失敗的測試案例，不要偷跑。一次新增兩個測試案例，一次調整 production code 通過兩個測試案例。在這例子當然是因為夠單純，使用同一種作法。但建議要培養好習慣，否則會很不自覺地偷跑。


SUCCESS: Offer approved, accepting.
Offer accepted from STEAM_0:1:61266233
ERROR: Steam API error encountered. Trying again. Tries: 1
ERROR: Tried total number of tries, giving up.


Thanks for the hint!

There is even more, the pmenu project page lists a lot of alternatives: https://gitlab.com/sgtpep/pmenu#alternatives :)


does this mean that "scheduled" jobs have no link on the logfile anymore?
Might make sense, since the old logfile would appear, on the other hand it is a behaviour change which some users might confuse... hmm ....


:+1:

faster way:
```
    if (m_spellInfo->HasAttribute(SPELL_ATTR_EX_CANT_TARGET_SELF))
    {
        switch (targetMode)
        {
            case TARGET_SELF:
            case TARGET_SELF2:
                break;
            default:
                targetUnitMap.remove(m_caster);
                break;
        }
    }
```


BookFactory() 會不會顯得有點太 over design 了一點，值得考量。


all of ```NULL``` should be replaced with ```nullptr```.


Sure, but Jekyll should even care...


這邊測試程式除了使用到 target 以外，還相依了 BookFactory(), 代表就算 target 的程式碼是對的，但 BookFactory() 錯了，那測試案例也可能會 failed。

而且說真的，測試案例透過 BookFactory ，並不會比較直覺好讀。這一份工省下來，可以做更多的事。


tab?


1. Снипет сортировки изменен на фильтр
2. Изменено меню деталей. Добавлены пункты: Заморозить, Разморозить, Остановить
3. В списке отображаются системные(цвет) замороженные(цвет) и запущенные приложения(текст+цвет)


over design 了....

需求跟這越離越遠了...


@mcostalba You are right that Linux has the trick for gettimeofday() without context switch, but Windows or other OS probably not.


I added this since this number gets incremented whenever add a database item or a new global (this lets users know that are using the development codebase that they need to run the sql_upgrade.php script).


so the deprecation workflow is confusing. the just-submitted form gets the old form's `_id`, and the previous form gets a new id, and has the `_id` saved as `orig_id`.

so `new_doc` should be locked with the `old_id`, and `existing_doc` should be locked with `new_id` (which is actually kind of silly because I think that's randomly assigned).

It's hard for me to completely reason with this whole change but does that clarify anything?


indent?


ditto


Building from scratch is awesome!


@aMarCruz great job! I will review this branch hoping to find a better solution to the copy of the compiler.js file in the `lib/server/` folder. I will merge this in the `dev` branch ASAP making a new riot `2.3.0-beta` release.

p.s.
@cognitom @aMarCruz  once the riot beta release will be rolled out without problems before releasing the final riot 2.3.0 we need to be sure all the submodule are published on npm  as `2.3.0` release and they must load the right dependencies `2.3.*` for example: `riot-route` will load `riot-observable: "2.3.*"`


move to `delete`.


Somethings are just better in their own language. Not everything needs to be translated, we live in a globalized world, people know what a "click", "mouse", "flash drive", etc. Software becomes uncomfortable with these unnatural translations, now that more and more people are becoming aware of things like cryptocurrencies, public private key encryption, the term "hash" is becoming universal, no need to force it. "suma de comprobacion" is simply too long, if we start putting it here, then we also need to put it on android and that would make the visual experience very bad, you'd instead end up reading "suma de..." not cool. Keep it simple, just call it what it is, a "hash"


On Thu, Oct 29, 2015 at 8:02 AM, Tom Vijlbrief <notifications@github.com>
wrote:

> @mcostalba <https://github.com/mcostalba> If I read the code correctly
> then with 20 threads you will call the very expensive now() system call
> with a 20 time higher frequency than with 1 thread.
>
> Why not add something like
>
> static int calls;
> ... && ++calls % nr_of_threads == 0 && now() - .....
>
> to the condition at line 578?
>
>
>
Also your condition is possible, but you introduce a shared variable
'calls', talking of hacks :-)  it is better to use directly position
searched nodes:

!((pos.nodes_searched() >> 12) % Threads.size())


Si je ne l'ai pas, je suis sûr que vous avez l'empathie nécessaire pour faire les choses correctement vis-à-vis: des auteurs orignaux (je pense aux images également), de vos collègues (qui ne partent pas forcément avec un projet faisant exactement ce qu'ils veulent), etc.

Si aucune licence n'est donnée, le droit d'auteur s'applique. C'est pourquoi il est conseillé de toujours en mettre une sur son contenu public (les Apache, BSD et consœurs sont à la mode).

Ensuite ma question ne vous menaçait pas de vous administrer cette note. Elle voulait dire: “cette manière de faire est considérée comme étant du plagiat” et dans le monde professionnel comme scolaire c'est un problème sérieux.

De manière plus scolaire et formelle, si nous vous demandions un rapport, cet autre projet irait dans une section [état de l'art](https://fr.wikipedia.org/wiki/%C3%89tat_de_l'art). La description de l'état de l'art se conclut par quelles sont les améliorations que vous allez apporter dans votre travail. Donc sans rapport, je vais considérer _votre contribution_ à la problématique du _jeu Dixit en ligne_ comme ce qui a été fait moins ce qui est déjà présent dans le projet de base.

Ensuite, même si la forme n'est pas toujours au rendez-vous, vous donner des retours tout au long du projet va vous permettre de progresser bien plus qu'une unique évaluation finale.

Fini de tourner en rond. virez-moi ce schéma horrible (entre autres `latin-1` et `auto_increment`) et dont vous n'avez pas la parternité et qui est inutile. et partez d'un meilleur pied:
* http://laravel.com/docs/5.1/migrations
* http://laravel.com/docs/5.1/seeding


directly use `$stateParams.id`


no error case still.


What's the default for here? IMHO it should just raise if API user gives unknown value.


```wich ``` -> ```which```


I'm using，thank you!
Tips：At the prompt you can display parameters of the method？


Also average_angle_matrix added.


no error case.


Move User Id: Outside of input, ng-bind `record.userId`


:ok_hand:
> riot-route will load riot-observable: "2.3.*"


indent?


... *wasntme :smiley:

Habs für den Artikel so übernommen, danke @JohannesHoppe .
Ist so auch viel einfacher, als Typescript zu transpilieren. :clap: :clap:


`ng-messages`


풀리퀘스트 올려주세요. 다만, 풀리퀘스트하기 전에 최신 upstream의 index.md 파일을 반영한 다음 '가나다' 순서에 따라 '리노트' 아래로 옮겨주세요.


I just realized that it makes more send to revert this here to:
``````
$search_type = $default_search_type;
``````
And then to place the above code block more upstream here(and use $default_search_type rather than $search_type):
https://github.com/openemr/openemr/blob/master/custom/code_types.inc.php#L86
(and also remove line 63 and 84 of above file)

This will then in effect all other downstream places that use this, which appears to be used in:
library/coding.inc.php
(you don't need to do anything in coding.inc.php ; by doing above changes will also support the default code type in this script)


Move Required outside span, use `ng-bind`.


Ezzel itt több probléma is van:
1. így minden entitás mentésnél új create date állítódik be, pedig azt nem kellene, ha módosítom akkor csak a modified date -nek kellene módosulnia
2. ma még persze tudod, hogy mit is csinál a setCreateDate metódus, de 1-2 hónap múlva lehet valami komolyabb kódot raksz bele, és így ha külön dátumokat adsz át a két metódusnak, akkor lehet 1-2 másodpercet is eltérhetnek (hiszen futhat annyi ideig a setCreateDate)
3. fusizható a rendszer. a célja az lenne, hogy a create date és a modified date ne legyen módosítható, az a tényleges regisztráció és módosítási idő legyen, de ha paraméterként te oda bármit átadhatsz, akkor én bárhol módosíthatom, visszaállíthatom a regisztráció dátumát, stb. Jó lenne, ha nem kapna argumentumot a setter (egyik sem), egyszerűen elvégezné a melóját
4. Ez nem probléma, csak megjegyzés, de a DateTime konstruktora nem várja el a "now" stringet, ez a default, felesleges átadni.


Agreed...


@mcostalba
yes

      !((pos.nodes_searched() >> 12) % Threads.size())
is indeed a very elegant approach :-)


Should ```browser_context``` be checked with ```nullptr```?


ha ez a file nem kell, töröld kérlek


I feel like this is slowing down the loading sometimes. Can this just be R.drawable.placeholder_rectangle_scene instead? Especially because we get in here only if currentBackgroundImage exists.


Még annyi az összes filehoz, hogy érdemes lenne egy kicsit jobban kommentezni a metódusok fejét, hogy jobban működjön a típuskiegészítés, autocomplete, könnyebb legyen olvasni a kódot.


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


What does your commit have to do with the bet sequence?


1911-8-4-Update (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

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


indent.


ditto.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.724](http://teamcity.monogame.net/viewLog.html?buildId=10993&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependencies failed: 2 Build time: 00:00:00


Hi Kyle,

I rebased your commits into one commit and placed a review here:
https://github.com/bradymiller/openemr/commit/5ccca663f147e5f134b0886df2255d1eaf1d34af

Note I just realized it makes more sense to set $default_search_type more upstream with your block of code. See the commit comments for explanation.

-brady


Actually, I just realized this was for the background image, not the verse image. The question still remains as to whether this is needed because the XML also default to this. I'm curious as to whether there would be any sort of performance improvement if we didn't a placeholder here at all.


This is not an error. It's a warning - it will NOT break your local builds.
Only travis is set up to fail on warnings.


Probably I will change this to a symlink https://github.com/substack/browserify-handbook#symlink


of course, I don't even know how it landed here.


Ide minek a ternary operátor?
 a
($bigLetters === true && $smallLetters === true && $numbers === true && $specials === true)
kifejezés is boolean értékket ad vissza, nem kell itt még eldönteni, hogy ha true akkor true, ha false akkor false.


It means I will ask you "a lot of" questions.
Orz


@Zedron much better code produced by gcc


of course


align with above parameter?


I don’t think *suma* is unnatural (and it’s certainly not long), I can tell it’s been attested at least since Windows XP (translations made in 2000). It’s also used by other software projects such as GNOME, KDE and LibreOffice.

I know languages borrow words from others all the time, and I know it’s a natural process – especially in the information technology field. But in this case, a standardized piece of terminology native to Spanish exists, so it’s an unnecessary borrowing, which you won’t find in technical textbooks written in Spanish. And I’m of the opinion that *localized* software should set an example of proper use of a language, any language.

Whether US citizens like to use Spanglish is completely irrelevant — most of the Spanish-speaking world does not live there. Also, if you thing we’re “teaching a little English” to people here by shoving Anglicisms everywhere, well, it’s a very bad “teaching” practice.


這個就是偷跑，你怎麼會知道3,4,5的折扣是多少？

只完成通過眼前 scenairo 的程式碼，一行不多，一行不少。完成之後要確保所有測試案例都是綠燈。再進行重構。


This is just for reference in future. When ask community to review, this will be a good thread to announce it in:
https://sourceforge.net/p/openemr/discussion/202504/thread/af95d99a/


Yeah, we could remove the line above and could use `var compiler = require('riot-compiler')` for here:
https://github.com/riot/riot/blob/d75f3204630083956c3aee7953be9a67f56d87a6/lib/server/index.js#L5


> p.SetPolicy(policyBase64)
> p.SetSignature(signature)
> return p.formData

Looking from right here - p.setPolicy, p.setSignature and then returning p.formData.  One cannot really tell you are modifying p.formData or even that you did successfully modified it underneath.

Unless you are doing p.formData.Set("x-amz-signature", signature") - that would make more sense, but then it wouldn't serve any purpose since map already does get/set by default.

Adding these methods serves good purpose is other languages where formData is kept private and not modifiable outside other than the PostPolicy class itself.  Here we cannot enforce that, so won't serve any good purpose.


@Kimmax I figured out how to do it but im running vs 2015 packages will not restore do you maybe know how to do so?


You're welcome!
Not exactly understand, what you mean.
Could you make an example?


有點過於複雜了。還不如直接用 for loop 累加 price 會不會比較直覺？


Thanks, well I wouldn't call it elegant but thanks anyway :-)

Because doing so we could remove now() call, it turns out it is even a
simplification and hence I have submitted a new test:

http://tests.stockfishchess.org/tests/view/5631cdd70ebc59227df16aec

On Thu, Oct 29, 2015 at 8:34 AM, Tom Vijlbrief <notifications@github.com>
wrote:

> @mcostalba <https://github.com/mcostalba>
> yes
>
>   !((pos.nodes_searched() >> 12) % Threads.size())
>
> is indeed a very elegant approach :-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mcostalba/Stockfish/commit/0d79b149986aaebd803fa58cacf5925eb1273db7#commitcomment-14059661>
> .
>


Might be good to pair that with a test for aufs. Might need kernel extras
installed.

On Thursday, October 29, 2015, Tim Lucas <notifications@github.com> wrote:

> Cool, fixed in c6b0b7a
> <https://github.com/buildkite/docker-buildkite-agent/commit/c6b0b7a0510bb13537c4e1e29f7478c7c6246415>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/buildkite/docker-buildkite-agent/commit/3e2ffd4091fe99028a5b256ab2b9ca30738546b3#commitcomment-14058883>
> .
>


THE AWESOME KIND.


突然發現，算完 price, 好像沒拿來用？XD 這就是想太多啦！


Why do these traits extend `FromIterator`? I might be missing something, but it seems that this is only useful with an `IterableFactory`. I would suggest removing the `FromIterator` trait and move the method inside `IterableFactory`.

It feels like this:

```
class Person {
  def marry(person1: Person, person2: Person) = Person.marry
}

object Person {
   def marry(person1: Person, person2: Person) = ...
}
```

If the `marry` method was like this: `def marry(other: Person)` it would use information from the first instance and it would make sense to have it.


Nice!


You should use underscores instead, dashes are invalid in python module name ;)


這就是上一個 commit, production code 偷跑的問題。

在寫 production code 的時候，根本不知道需求期望的是什麼。只是自己推估折扣，所以先寫對應的折扣上去。這個時候才寫 test case 來驗證 production code 寫的，跑起來跟想的有沒一樣。這不是TDD。


@cognitom the problem seems to be that the `require` calls in the compiler  (to load jade, coffeescript...) will be resolved in the `node_modules/riot-compiler` folder and this is a problem, but I will investigate the issue asap


Ezt a metódust gondold át, túl van bonyolítva és az, hogy for ciklussal mész végig rajta nem túl optimális.


剛寫好一個測試案例發現是綠燈，在TDD中不是件好事。除非與其他的 scenario 是代表同一種 scenario, 只是 input/output 不同而已的情況。
（在這邊不是，在這邊是因為 production code 忍不住偷跑了，所以導致後面幾個測試案例加上去就是綠燈了，而不是紅燈）


Unnecessary fat arrow


Hmm...


It's strange, because in the `cli` I also load the compiler normally and the parser require calls get correctly resolved
https://github.com/riot/cli/blob/master/lib/tasks/Make.js#L7
https://github.com/riot/cli/blob/master/Makefile#L27
https://github.com/riot/cli/blob/master/.travis.yml#L13


请把 ActivityGroup 去掉一下，引擎后面会去掉ActivityGroup


dd


example:
now:
<?= Html::checkbox [function];?>
but,I think the best is like this：
<?= Html::checkbox [$name, $checked = false, $options = []];?>
After entering the ':: checkbox'.
The expression may not be cleare...


I like your insistence :)

Could you please provide with screenshots of the use of "suma" in GNOME, KDE and LibreOffice.
If you have other BitTorrent software running in spanish, and you can provide us with screenshots of them using "suma", then you'll finish convincing me.

No te gusta el "sha" :)


please use np.full instead


این roll چیه تهش؟


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


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


Yea, that does clarify, thanks. Seems like this is correct then. Perhaps a small comment would help:
`lock docs with their original ID's (before they got switched during deprecation)`


Haja, unser Schnellstart-Beispiel soll ja auch toller sein, als das von Google! 😎

Von meinem Windows Phone gesendet

----- Ursprüngliche Nachricht -----
Von: "Gregor Woiwode" <notifications@github.com>
Gesendet: 29.10.2015 08:33
An: "Angular2Buch/code" <code@noreply.github.com>
Cc: "Johannes Hoppe" <johannes.hoppe@haushoppe-its.de>
Betreff: Re: [code] HEUREKA! :-) (526420a)

... *wasntme
Habs für den Artikel so übernommen, danke @JohannesHoppe .
Ist so auch viel einfacher, als Typescript zu transpilieren.
—
Reply to this email directly or view it on GitHub.


np.full


It is about abstracting PostPolicy structure and it's internals so that its users have a clean interface.
You can add p.getFormData() too if it helps it.
Anyway if you think it's not adding value you can drop the patch.


Конструкция выглядит не очень естественно. Если нужно преобразовать какой-то произвольный тип к строковому, то лучше делать как-то так:
```javascript
td.innerHTML = that.numberToLetter(j).toString();
```
А вообще здесь преобразование делать не обязательно, тк оно делается автоматически при присвоении.


Yeah I wonder :sweat:


toUTC shoud force minus


Seven Stars!!!


Непонятно назначение данной строки.


LGTM


ok


Здесь тоже непонятно зачем Вы создаете локальные переменные. Можно ведь использовать аргументы напрямую.


ок


В целом все хорошо, молодец.


Thank you, thank you.


Useless assignment to variable - `members_to_remove`.


你好，我有一个疑问
bool TNonblockingIOThread::notify(TNonblockingServer::TConnection* conn) {
    THRIFT_SOCKET fd = getNotificationSendFD();
如果fd > 1024下面的
ret = select(fd + 1, NULL, &wfds, &efds, NULL);
调用行为是怎样的？
是否是个bug？
...
}


> (может даже и не стоит делать глобальный параметр text):

ну `text` глобальный только если он передан, можно ж его и не передавать.

на счет остального -- ок, сделаю как в других методах -- `suggest(*args, **kw)`


Because the 3 classes ProductValueNormalizer, PriceNormalizer and MetricNormalizer are very similar, is it not a good idea to create a parent class for DRY?


نمیدونم والا از چیزای آماده استفاده کردم ولی فکر کنم پاکش کنم اتفاقی نیوفته


Fixed here: https://github.com/netguru/access/commit/cbc5bbae456e0982453b17112f35cc85dfc4af25


When I navigate to [sprsquish/vertica](https://github.com/sprsquish/vertica), it redirects to [wvanbergen/vertica](https://github.com/wvanbergen/vertica).


@Kimmax  only way i know to fix it is to uninstall the packages and reinstall or upgrade them


تو این خط
  </ScrollView>roll

اسکرول ویو ایراد نداره
ولی roll خالی نمیتونه باشه
باید ارور بگیره


* godfery


timeComponents = @[ [string substringToIndex:r.location], [string substringFromIndex:r.location] ];

Cette ligne est fausse: le time zone component va être utilisé en micro-secondes dans cette ligne:

if ([timeComponents count] > 1)
				useconds = [[timeComponents objectAtIndex:1] intValue] * pow(10, 6 - [(NSString *)[timeComponents objectAtIndex:1] length]);


wtf


As far as I could tell you just need to call the `getUpdates` endpoint. https://core.telegram.org/bots/api#getupdates


@Zedron : ... or rather: "How DARE THEY treat them as trivial simple integers!" ;-)


1921-WaterToWaterHP (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed

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


See https://github.com/tomchristie/django-rest-framework/issues/3565


If you do not have a large amount of cases a switch usually compiles into the same assembly. Any compiler worth of being mentioned does this optimisation for us already.
Line length isn't too long either and you could nest the second and third if inside the first one if that became a problem, but in my opinion a switch of that form (default case being the actual doer) looks weird to me and less readable. /2c

As a sidenote: the last break is not needed as you break out of the switch anyways.


As far as I understand, in this case, the 'pinkie' will execute the code in sync.


No descarto utilizar «SHA» como alternativa ;) Actually, on a second thought, I think I prefer SHA… You made a good point on Android UI, which is really space-constrained.

But for the record, here are examples of the term used (in its full form) in [Launchpad (Ubuntu)](https://www.google.com.mx/search?client=ubuntu&channel=fs&q=%22suma+de+comprobaci%C3%B3n%22+site%3Alaunchpad.net&ie=utf-8&oe=utf-8&gfe_rd=cr&ei=WtIxVqfaGomAqQXahZ-oBA), and
[a screenshot of LibreOffice](https://cloud.githubusercontent.com/assets/554953/10813090/efbd9a1c-7de1-11e5-9b79-5fe405d6c33f.png), don’t have KDE at hand.


Reading this source-code line I wonder if you have any ideas how to treat allocation in VFS in the future. If the VFS code is part of the VFS server, then allocations must be accounted to the session initiating the allocating operation. Otherwise, the VFS server may exhaust its own quota with misbehaving clients.


A similar function is already defined in ProductExportController and ProductCommentController. Is there a way to merge them ?


I will run final experiments today, inshallah.

> On Oct 29, 2015, at 10:29 AM, Erfan Noury <notifications@github.com> wrote:
>
> I think it's good to go. Let's run the final experiments!
> What do you think first starting with a small network (CNN S) and fine-tuning it and then moving to the VGG 16 (or 19) for the final runs?
>
> —
> Reply to this email directly or view it on GitHub.
>


>Since this is empty you should delete it.

Just forgot, fixed now.

>Does it auto-center to parent?

Yes, as long as `AddNewTorrentDialog` have a valid parent, if not, will center to primary screen.


sdf


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


Le franglais est un peu surprenant, e.g. `timeEntre` est difficile compréhensible sans connaître le contexte. Sinon, ça m'a l'air bien. :+1:


Oh good idea!


cc @DeepDiver1975


si `nbEquipe` est sensé être un nombre, mettre un petit casting est généralement prudent. `$nbEquipe = (int) $_POST["nbEquipe'];` Le contenu de `$_POST` étant toujours du texte.


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 40](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=385&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) is now running


in PC mmex we have Bitcoin
    r.push_back(std::make_tuple("BTC", "Bitcoin", L"Ƀ", L"", L"", L"", 100, 1, " ", "."));


infinispan80 plugin should contain latest micro release of Infinispan 8.0.x, for 8.1 please create another plugin.


Great! Looking forward to it.


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 40](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=385&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:09


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


These aren't the same.
This is about app.app.config (the Flask application config)

The YAML thing is about config.config (a dictionary loaded from yaml of medea-specific config)


closes #4


Feels like it shouldn't be in parseConfig, because it says "parse config", it does not check if some values are supported. I think parseConfig should validate if there's `language` entry, and versions entry, etc.


Erf, thank for the advices. Done !


Hi Viktor,
It's possible to add support for extended key codes to GTWIN and I plan to do that though it will be nice if later it can be tested in different MS-Windows versions. Current keyboard code in GTWIN needs some cleanups so probably I'll start updating it to the level which allow to easy add support for ExtKeys.

In most of cases GTTRM replaced GTCRS and GTSLN anyhow it only support few most popular terminals. If someone uses some more exotic terminals (i.e. some old hardware) or terminal emulators
then he has to use termcap/terminfo based GTs like GTCRS or GTSLN because GTTRM will not work correctly. They also have feature which sometimes is important. GTSLN and GTCRS by default use virtual terminal (vt) functionality and switch to new VT at init and restore original one at exit. It allows users to create environment based on this functionality, i.e. to catch output from few applications. So far GTTRM does not support this.

best regards,
Przemek


Fair point :).


map String path to Path objects to avoid repated String->Path->String conversions


```python
dic["sortByTime"] = "t" in option
```

and so on


Extract helper method.
deleteChildDirectoriesUntil(path, parent)


<!-- Lab -->


a


looks good - though it would be nice to split commitInsert into smaller chunks in separate commit as part of this work.


I guess you can remove the `std::unique_ptr`, now that `clone() already returns a `std::unique_ptr`.


Congrats!


Why this?


Most commonly used version pattern is 0.9.4, 0.9.5....0.9.9,  0.9.10, .... 0.9.39, 0.9.40, and then 0.9.41.

The next version should then be 0.9.5.


Hi Franco,
thank you very much for your quick answer!
In fact I have a lot of other questions.
Some are more general, so I'll write them to issues here in the github project, hope it is OK.
Or do you have a better way?

Best regards
Bettina


I'll have to try this on my Linux box when I get home -- [I figured `File.separator` would resolve to whatever platform-dependent separator it needs](http://stackoverflow.com/a/27752648). I think this is a Windows 10/Java bug. This fix seems a bit hacky and backwards though, so while we're refactoring we need to come up with a better solution I think


A class Point with the coordinates (x,y) as variables and the possible movements on the coordinate system as methods


`if ['developer','teamleader','freelancer'].include?(current_user.role)` или типа того


this was only accidentally committed ... will revert the change


This change is not backward compatible. To a change in behavior, cannot be part of devel-3.


@Bui-Hung-Mulodo
Vao xem choi


@nguyen-ha-mulodo
alo. vào xem start


I think we can do without this option. First of all, it's a Linux issue, because Debian for some reason decided not to have a standard clang-format version in testing. There is probably a reason behind this, but most likely they will have a standard clang-format version in the next Debian stable, or else we should issue a bug report on this. OS X and MSYS2 both have clang-format-without-version-binary.

In other words, I think the solution creating a symbolic link as described in https://github.com/cppit/jucipp/blob/master/docs/install.md is a good temporary solution until Debian fixes this.


I believe there is a mistake on the Mac/Linux install command, it should be ./ before the install.sh


toto vrat naspat, neprechadza build


The end


TeamCity pull requests :: netty [Build 487](https://garage.netty.io/teamcity/viewLog.html?buildId=2733&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5640, ignored: 21 Build time: 00:43:54


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


@BrianErikson libgdx is meant to be cross platform, this should work just fine on every platform. This is handled behind the scenes.


Quelques commentaires/questions :
- Pourquoi ne pas intégrer strings.exe et md5.exe dans collecte.tar.gz ?
- Comment passes-tu du nom de fichier à la liste des _strings_ et au _MD5SUM_ ?
````sql
UPDATE filesenhanced SET StringList = LTRIM(RTRIM(REVERSE(SUBSTR(REVERSE(FilePath),0,CHARINDEX('\v\v\v\v\v\v', REVERSE(FilePath),0)))));
UPDATE filesenhanced SET Md5sum = LTRIM(RTRIM(SUBSTR(FilePath, 0, CHARINDEX('\v\v\v\v\v\v', FilePath, 0))))
```

- Je pense qu'il manque un `;` dans ta dernière requête


My fork uptime2. https://github.com/gunmetal313-nodejs/uptime2


Rebased and made a fix for urls like https://www.youtube.com/watch?v=-abcdefghij


\o/ great fix btw, idk how this got removed.


Hi,
slightly simpler names could be `findAllFromDestination` / `findAllToDestination`. :)


Could you move lines 171-173 to Window::configure? Or else the actions after saving config.json will be spread among too many functions. MainConfig should only load the configs I think.


@alanfx My expectation for default is that each group is clustered independently, e.g if you have 2 sites (each being one group) or servers and clients. The `expectedSlaves` value is checked against the Clustered.getMembers().size(), which is exactly the number I am providing here as default.
What's the use case for having all executing slaves (the previous behaviour) as the expected number of cluster members?
Maybe I don't fully understand the comment. You're talking about `getExecutingSlaves()` - I am not changing behaviour of *that* method.


@Mamouti70 این اسکرول ویو الان توش خالیه و هیچی نیست
پس میتونی پاکش کنی


This commit is nice. In future we need a test for `remove` to catch this type of incompatibilities.


Code around this part is about removing members from parentGroups of removed Group. But in the new behavior, we don't take members from subGroups.


این برای linearlayout بود
و میتونی الان این خط رو پاک کنی
@Mamouti70


> `cb` should be called `callback`

If your prefer, I saw the twos, but no problem :)

> Function name is not clear. What is "a"?

`a` like in `btoa`, standard conversion to a _base64_ string. I kept the convention name.