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


addresses issue #4


Good question, I asked myself for many times...How to make the number more like calculated with logic not guessed...But in fact, I just guess a number to make sure the cache is large enough...


@fbertux this commit needs to be reworked so the changes are done in the respective commits (linux and u-boot) leaving just the machine addition here.


Minor: not sure exactly what this comment is. if there are few requests (less than 10) don't bother?


syntax error here. look at where you are closing the `)`


thanks, fixed.


could put printing commands in the TextIO class instead


at this state, computing aclosure20() of `scale_free_rcc.qcsp` took **11342.433340549469** seconds


This (and the commit that originally disabled this functionality) should be merged with the varchar(n) commit before we merge


should be `})` here


Leave a line note


Also added support for CANTalons.


Minor: two spaces.


Right. Let webpack users minified their own bundle.


@Piedone more generally see #6294, i can do some updates but according to your choices.


Right now this doesn't allow configurations whose upper and lower bound are the same, e.g. min_rtt = max_rtt = 150. I'm going to modify the check so that this is possible.


@cadu é bem provável que o cenário do docker descrito, "Invalid remote: origin", tenha relação com as variáveis EDS_CARTAS_REPOSITORIO e PDS_CARTAS_REPOSITORIO. Essas variáveis necessitam existir no ambiente quando as ferramentas inicializam. Não entendi a relação com a versão do jgit.


we should rename `pathToPloltySrc` -> `pathToPlotlyIndex` to avoid confusion later on.


:tada:


Why cant' you generate the pod IP here?


Is there a functional advantage to this, or does it just look prettier? Why is this prefered for set while dict() construction is prefered for dicts?


Wouldn't it be better to report something like "no data present" instead of just returning no meals at all?


fwiw: you can also pass in nil instead of a new set for the unknown metrics


Nice, thanks for adding this!


Is this used anywhere


This is because that each time we `runContainer`, we'll create a new container and start it. No matter the container failed to be created or started, `runContainer` will return an error. For now we just catch the error from `runContianer` in `SyncPod()` and put it into `PodSyncResult` with action `StartContainer`.

I'll change the comment above, and we may want to distinguish create and start in the future. :)


Where is this function called and shouldn't this function subsume generatePodStatus?


ACK


Shit, shouldn't have a `!`


Looks good, thanks


TeamCity LabKey :: Sprint :: Premium :: Git sqlserver 2014 [Build 15](https://teamcity.labkey.org/viewLog.html?buildId=298356&buildTypeId=LabKey_Sprint_2_LabkeyPremiumTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:02:09


This is a stupid error, I've removed it in this PR: https://github.com/kubernetes/kubernetes/pull/19784


What I was suggesting is that the Configuration class would contain everything pertaining to running a command. The command themselves would be argumentless, and thus no argument exception would be thrown. We would only validate that when you give command line arguments

What would end up happening is that you'd load the config file, then override any values through the values given to the command line.

What you gain from your method is that symfony takes care of validating arguments.

What my method means is that you need to handle arguments validation inside of the commands, but that you don't need an InputMerger.

I'll have to think about it a little and weigh the pros/cons of each.


Remove "docker" references in this function now that it should be generic.


It's not part of the image. Replace the file on your musicbox with this changed one and it should work.


reduce_redimensions (mbadams5) - x86_64-MacOS-10.9-clang: Tests Failed (1625 of 2050 tests passed, 54 test warnings)

 * 510 tests had: AUD diffs.
 * 212 tests had: MTD diffs.
 * 368 tests had: ESO big diffs.
 * 400 tests had: EIO diffs.
 * 195 tests had: MTR big diffs.
 * 181 tests had: SSZ big diffs.
 * 225 tests had: ZSZ big diffs.
 * 321 tests had: Table big diffs.
 * 45 tests had: ERR diffs.
 * 11 tests had: SSZ small diffs.
 * 28 tests had: ESO small diffs.
 * 15 tests had: MTR small diffs.
 * 46 tests had: Table small diffs.
 * 7 tests had: ZSZ small diffs.

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


Just as a thought, in the near future, community modeling will need more than 26 compartments.

Best,
Nick

Sent from my iPhone

> On Jan 20, 2016, at 5:12 PM, cshenry <notifications@github.com> wrote:
>
> Multi-letter compartments are extremely problematic across the board. Assumptions of single-letter compartments are everywhere in the code and in our web visualizations.
>
> Plus, I really just can’t believe we need more than 26 compartments to capture a euk… nor do I think we really have the ability to adequately and accurately assign reactions to 26 compartments.
>
> On January 20, 2016 at 4:20:02 PM, samseaver (notifications@github.com) wrote:
>
> @ctseto I copied and expanded your compartment list for the human template. You'll see there's a whole bunch of sub-compartments, I will not be using many of these, for now, but they arose out of conversations with others who are focused on very specific parts of cellular metabolism.
>
> This list of compartments pretty much covers cellular biology, so I don't see any need to add more with the exception of abstract compartments for the sake of modeling itself.
>
> Couple of points, the plastidial stroma and the mitochondrial matrix is where the majority of the reactions take place, and so, at least in my plant models, these "spaces" default to the general plastid and mitochondria spaces.
>
> @cshenry Can't remember if you said otherwise, but as far as I understand it, we're using single letters to capture compartment identity. We may want to consider moving away from this to a 2 or 3 character system that covers a wide range, in a future iteration of ModelSEED, but in the meantime, yea, anyone can declare their own list of compartments, so its not a big deal.
>
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub.
>


could rename as there are two methods called play


I think we've covered this before, but I can't find the comment. Can you explain why we need this (and add some details to the commit message)? Intuitively, `varchar(n)` should be able to coerce to `varchar` (which is syntactic sugar for `varchar(max)`)


take that


1968-new-oa-methods (kbenne) - i386-Windows-7-VisualStudio-12: OK (2354 of 2391 tests passed, 0 test warnings)

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


lgtm


This should be merged with the varchar(n) commit before we merge


Getting from status manager is fine, but if status manager hasn't been populated yet, you may still want to check the status from the apiserver. (for some reason, I thought we already used status manager...)


Good Point. Thanks, :)


87382760-VAVTerminalUnitSchedule-Issue4285 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2080 of 2088 tests passed, 3 test warnings)

 * 6 tests had: AUD diffs.
 * 10 tests had: EIO diffs.
 * 5 tests had: ERR diffs.
 * 8 tests had: ESO big diffs.
 * 3 tests had: MTR small diffs.
 * 3 tests had: Table small diffs.
 * 5 tests had: Table big diffs.
 * 3 tests had: ESO small diffs.
 * 4 tests had: MTR big diffs.

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


ex:2 + drill


This revision is clean.


So could you explain what './entry.js' is and what goes in it?


Fixed in 142c9ab5b5da82ca3d7cc5d5bd42e3f919d2d6f6


dude is stalking my pull requests


I prefer comprehension construction universally. For dicts it is:
```
    {
        key: value
        for key, value in ...
    }
```

It's 90% a style preference, though there is the advantage of being able to use `if` clauses in the comprehension syntax for filtering.


Is that better?


Should I really still have access to any maluuba repos?


@RazzYoshi This broke the build.


Good catch. Fixed.


LGTM.


Yes, the function seems unnecessarily long (not your fault) and hard to parse.


That's appcache, autocorrect.


We should just return `_emptyArray` here.


Hi Love, it all looks good!
One suggestion, place the description comment of the allButOne function INSIDE the function, as the first line. Also, there is a type in "comparition"


[![Stem table](http://api.artsmia.org/images/122509/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=122509)


[![Altar table](http://api.artsmia.org/images/122515/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=122515)


[![Incense burner with Chinese lions](http://api.artsmia.org/images/122528/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=122528)


[![Portrait Of Beauford Delaney](http://api.artsmia.org/images/121645/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=121645)


[![Portrait of Moses Kimball, Jr.](http://api.artsmia.org/images/5423/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5423)


[![Portrait of Dorothy Beridge, nÃ¨e Gladwin](http://api.artsmia.org/images/546/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=546)


[![Portrait of Sarah Allen, nÃ©e Sargent](http://api.artsmia.org/images/593/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=593)


[![Christ and the Woman Taken in Adultery](http://api.artsmia.org/images/1672/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1672)


[![The Prodigal Son](http://api.artsmia.org/images/1849/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1849)


[![Portrait of Catherine Coustard, Marquise of Castelnau, Wife of Charles-LÃ©onor Aubry with Her Son LÃ©onor](http://api.artsmia.org/images/2487/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2487)


[![The Asparagus Vendor](http://api.artsmia.org/images/3076/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3076)


[![Portrait of Lucia Wijbrants](http://api.artsmia.org/images/4418/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=4418)


[![Portrait of Antoine Coysevox](http://api.artsmia.org/images/1859/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1859)


[![Portrait of a Burgomaster](http://api.artsmia.org/images/537/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=537)


[![Penitent Magdelene](http://api.artsmia.org/images/3059/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3059)


[![Portrait of Lucas van Voorst](http://api.artsmia.org/images/3740/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3740)


[![Portrait of Catharina van Voorst](http://api.artsmia.org/images/3741/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3741)


[![Portrait of Commander Lewis Warrington](http://api.artsmia.org/images/473/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=473)


[![Portrait of Robert Devereux, Second Earl of Essex](http://api.artsmia.org/images/281/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=281)


[![Portrait of Abigail Gowen](http://api.artsmia.org/images/1794/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1794)


[![Portrait of Mary Croswell](http://api.artsmia.org/images/10537/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=10537)


[![Portrait of George Washington](http://api.artsmia.org/images/486/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=486)


[![Portrait of Captain Samuel Chester Reid](http://api.artsmia.org/images/727/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=727)


[![Portrait of Judge Daniel Cony of Maine](http://api.artsmia.org/images/2204/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2204)


[![Portrait of Diana Mary Barker](http://api.artsmia.org/images/448/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=448)


[![Portrait of Captain Enos Reeves](http://api.artsmia.org/images/1267/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1267)


[![Portrait of Alexander Townsend](http://api.artsmia.org/images/1183/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1183)


[![Coastal Landscape with Granite Cliffs](http://api.artsmia.org/images/1645/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1645)


[![Portrait of Ruggiero Leoncavallo](http://api.artsmia.org/images/2040/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2040)


[![Portrait of Paul ValpinÃ§on](http://api.artsmia.org/images/2038/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2038)


[![The Boat](http://api.artsmia.org/images/120584/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120584)


[![Portrait of Paris von GÃ¼tersloh](http://api.artsmia.org/images/10219/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=10219)


Have you changed anything in the code below?


[![Portrait of a Burgomaster](http://api.artsmia.org/images/537/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=537)


[![Portrait of Dorothy Beridge, nÃ¨e Gladwin](http://api.artsmia.org/images/546/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=546)


[![Portrait of Sarah Allen, nÃ©e Sargent](http://api.artsmia.org/images/593/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=593)


[![Christ and the Woman Taken in Adultery](http://api.artsmia.org/images/1672/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1672)


[![Portrait of Abigail Gowen](http://api.artsmia.org/images/1794/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1794)


[![The Prodigal Son](http://api.artsmia.org/images/1849/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1849)


[![Portrait of Antoine Coysevox](http://api.artsmia.org/images/1859/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1859)


[![Portrait of Judge Daniel Cony of Maine](http://api.artsmia.org/images/2204/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2204)


[![Portrait of Catherine Coustard, Marquise of Castelnau, Wife of Charles-LÃ©onor Aubry with Her Son LÃ©onor](http://api.artsmia.org/images/2487/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2487)


[![Penitent Magdelene](http://api.artsmia.org/images/3059/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3059)


[![The Asparagus Vendor](http://api.artsmia.org/images/3076/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3076)


[![Portrait of Lucas van Voorst](http://api.artsmia.org/images/3740/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3740)


[![Portrait of Catharina van Voorst](http://api.artsmia.org/images/3741/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3741)


[![Portrait of Lucia Wijbrants](http://api.artsmia.org/images/4418/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=4418)


[![Portrait of Moses Kimball, Jr.](http://api.artsmia.org/images/5423/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5423)


[![Portrait of Paris von GÃ¼tersloh](http://api.artsmia.org/images/10219/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=10219)


[![Portrait of Mary Croswell](http://api.artsmia.org/images/10537/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=10537)


[![The Boat](http://api.artsmia.org/images/120584/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120584)


[![Portrait Of Beauford Delaney](http://api.artsmia.org/images/121645/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=121645)


[![Stem table](http://api.artsmia.org/images/122509/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=122509)


[![Altar table](http://api.artsmia.org/images/122515/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=122515)


[![Incense burner with Chinese lions](http://api.artsmia.org/images/122528/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=122528)


Doh. Of course, I forgot ```List<T>``` had its own field for this already.


I wonder if it even makes sense to have a process with no breakpoint manager. So this is good.


Still has problem with executable script being moved from root folder from previous two commits


Why?


This was intentional, since all it does is call `.collect()` on the stream. I think the alternative is worse:
```java
Map<String, NullableValue> filters = bindings.entrySet().stream()
        .collect(toMap(
                entry -> checkType(entry.getKey(), InformationSchemaColumnHandle.class, "column").getColumnName(),
                Entry::getValue));


```
$ npm uninstall tsd
$ rm -rf typings
$ npm install typings
$ typings init --upgrade
$ rm tsd.json
$ typings install
```


Seems consistent now with https://github.com/scala/scala-asm/blob/master/src/main/java/scala/tools/asm/tree/analysis/Analyzer.java#L258-L275.


ごめんね


文字化けた


Should be right here.


might be detritus from another approach i was playing with, i'll double check in the morning and remove it if so.


Instead of changing these to `ObservableInput`, I'd suggest changing the definition of `ObservableInput` [here](https://github.com/ReactiveX/RxJS/compare/master...david-driscoll:typings-option-a#diff-589f3f86d1945dea4eead11921732c34R44) to remove arraylikes and promises (possibly adding a TODO to add it back once the implementations are updated). Right now you're relying on the lack of type safety in the implementation to make this work.


was just retaining the logging that already existed.  if we trust whoever catches this error is going to log it, then no...personally i don't like to make that assumption so i'd rather risk logging it twice than not have it at all when we're trying to debug something.  I can move the log level up to 5 if that would make you happier.


@davidfestal I can't build without rolling back to the commit before this one, and in ceylon-ide-eclipse, d97dcb889.


Well, that was one big mess by me. Thankfully i don't have to Revert"Revert"Revert"Revert" now


we have to remove Angular2's typings so it won't have to ignore it with `ts-loader`


I hope I did this pull request correctly, I don't know why it shows two files...


For the differences between `tsd` and `typings` see https://github.com/typings/typings/issues/72


please update the version code to 3.0.5 and build code to 5.


Better to include numeric and string literals only, rather than excluding class constants. (I assume we also need to exclude method handles).


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


Specifically, once the extra type parameters in `lift<T, R>` are removed, this will be broken.


Stopped working completely as well

Executing ./sickrage/SiCKRAGE.py results in:

Traceback (most recent call last):
  File "SiCKRAGE.py", line 23, in <module>
    from __init__ import main
  File "/opt/Sickbeard/sickrage/__init__.py", line 23, in <module>
    from sickrage.requirements import install_pip, install_reqs
ImportError: No module named sickrage.requirements


Oh, that's done in a subsequent commit, great!


Maybe just

``` cpp
if (_type == TypeInt)
    return std::to_string(u.asInt);
```
?


Same result with me as well


Why remove this operator?


Neat. What made you change your mind on this? :-)

Also; you don't need moment anymore. Moment-timezone works like Moment.


Sorry I really can't understand the O(NlogN) method. Let me be lazy this time and forget it this time.


Be good to document this given the number of inputs.

Also, is there a test_clusterer that needs updating or making?


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


be good to test for the new slashending instance variable too.


Oooh, @lucaswerkmeister, how did I miss that you added this? Very nice!


^^^ /cc @modocache in case you missed it.


109490574b-UnbalancedFlowWarning-4293-PlanB (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1508 of 1508 tests passed, 0 test warnings)

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


@hitherejoe
It should `com.hitherejoe.mvvm_hackernews.view.activity.CommentsActivity.EXTRA_POST`, shouldn't it?


This is the 3rd file added in github directly


Actually, it has a limit of 60 cards to your deck! (There's another change in the PR for this.) Most deck-builders only want to add a few more than 30, so 60 seemed like a good limit, and even if that didn't exist, surpassing that wouldn't too big of an issue (because they're just doing it for fun at that point.)

Also, if you're not reading the pop-up, and you have too many cards in your deck, then I think that's kinda user-error. Sure, it's not following Blizzard's implementation, but I find it rather useful for building a deck.

There's benefits to both, but maybe there's a solid middle ground, like making the number big and bold and red or something to that effect? An alert message at the top? Maybe a checkmark similar to the proposed arbitrary deck deal? It's workable.


@mehrpadin were these CSS values supposed to be empty? When diffing (https://github.com/mehrpadin/Superfish-for-Drupal/commit/544b5e3a2e3726524a3f83cf8bffeffe19a304de?diff=split), I noticed that these values had changed. It was also causing an issue in one of the projects I'm working on, so I created a pull request for the change I made to make it work: https://github.com/mehrpadin/Superfish-for-Drupal/pull/11


test 12


3rd file added in github directly


Thanks for the input Alyssa!

I should have been more clear. By movement, I meant to also include the, for lack of a better term at the moment, "non-physical" movement as well.  More generally I meant dynamism of the novel, or the opposition of forces that construct the theoretical and physically landscape of the novel's space.

Does that make more sense? or confuse things more?


The same problem, cannot build on Windows

```
oaidl.acf
        cl -c /W3 /nologo  -I. -Iproto -DHAVE_PATHDEF -DWIN32  -DFEAT_SNIFF -DFEAT_CSCOPE -DFEAT_NETBEANS_INTG   -DFEAT_XPM_W32   -D
WINVER=0x0500 -D_WIN32_WINNT=0x0500  /Fo.\ObjGXAMD64/ /Ox /GL -DNDEBUG  /Zl /MT -DFEAT_MBYTE_IME -DDYNAMIC_IME -DGLOBAL_IME -DFEAT_M
BYTE -DFEAT_GUI_W32 -DFEAT_DIRECTX -DDYNAMIC_DIRECTX -DDYNAMIC_ICONV -DDYNAMIC_GETTEXT -DFEAT_HUGE /Fd.\ObjGXAMD64/ /Zi ex_cmds.c di
mm_i.c
ex_cmds.c
ex_cmds.c(560) : error C2065: 'DBL_MAX' : undeclared identifier
dimm_i.c
NMAKE : fatal error U1077: '"C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\BIN\amd64\cl.EXE"' : return code '0x2'
Stop.
```


Looks good.


generally don't touch gradlew files (they're auto generated). someone (clayton) might want to run on windows someday anyhow


BTW. Im decent with php but my styling ability is lacking. I just am not artsy. So I may have you look over this and see what you can do with the css side of things. I have a coder type css and its usually not pretty lol.


Sort them, but the order of declaration.


wow, I did not know there was another translation file here


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 134](https://teamcity.labkey.org/viewLog.html?buildId=298030&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 11 (2 new), passed: 38 Build time: 01:30:12
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

org.labkey.test.tests.CDSSecurityTest.verifyFirstTimeUse: <no details avaliable>

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

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

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

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

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

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

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

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

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

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


This example is not working with `v0.9` anymore as `branch`, `revision`, etc parameters are removed. I'm trying to figure out what is the correct way of customizing Force Scheduler form in Buildbot 0.9 and how to specify branch to be built.

If someone could provide a hint for the correct syntax, I'd be happy to contribute the update to documentation. Thanks!


Ah, thank you; I'll patch that up soon. :)


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


could do `yes = !isinteractive()) # don't prompt for unattended installs` if you want to get more specific, don't think it matters much though


Seems not working.  I will add the .torrent file and try that.  That should work.  Let me fix the patch.


(DHT seems to be down for some reason)


zackharley.ma to zackharley.me


Yes. That makes sense. I agree that the maps seem to show forces in an individual novel in a way that other visualizations of data may not.


Great job refactoring your code up top! I wonder if you can refactor your getWinner function as well by "switch"ing the if statements for something different... I'd love to see you write a playToX function as well. Great job! :100:


Este es la  primera del FRAMEWORK izel


@XECDesign Thanks. I got graphics working by changing Xorg driver to modesetting. I use Ubuntu MATE so I still have to recompile them or ask Ubuntu people to update them.

An issue: I tried adjusting the display resolution and the display reports unsupported signal if the resolution is not the default one.


요새 자꾸 바벨의 도움을 받다보니 하위호환성을 까먹고 있었네요ㅜㅜ 감사합니다:+1:


See #864: boolean values were throwing an error, so that doesn't seem to be true.

> ```
Uncaught exception:
[string "bundle:main.lua"]:53: attempt to concatenate local 'v' (a boolean value)
```


File was redundant


Why not just have the line below wait until it succeeds?


这是啥啊?


Also, not sure how you committed this, but you seem to have lost the author metadata for @zagy


109490574b-UnbalancedFlowWarning-4293-PlanB (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (2056 of 2057 tests passed, 0 test warnings)

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


Here is a torrent file: https://rudd-o.com/downloads/qubes/Qubes-R3.1-rc2-x86_64.iso.torrent  please try it.


High five for trying the extra credit! I love how you broke all your logic down to three states for your big functions BUT it can get hard to read in the middle of your getWinner function. Try not to chain more than two conditionals ( .. || .. )) || (( .. || .. )) etc. I'd like to see you refactor that by "switch"ing it into a different format... Stellar work! :star2:


links to guides on the index.adoc return 404, but guides themselves look good.


this breaks the build :( as it ignores the entire main package


Nice, but there is a problem. Players can use multiple accounts to circumvent max plots per player, also there is no guarantee that the new owner exists.


I think it *can* be used for multiple texts, just as the others *can* be used for single texts.  However, I agree, that maps, as presented by Moretti in this essay, work extremely well in terms of a single text, where as graphs and trees seem to do better on a larger volume.


There are a couple of things that I think we could do better with this patch. Let's talk in person tomorrow.


@mattpker Yeah, thanks for your remind, because I can't use pm2 to set my slack url,
So I tried to add the url directly into the package, it's pretty stupid :P.


LGTM.


Mm... The second problem is easier to fix, but I might need to do a little bit of work to fix the first.


@happyfredo Uuups, thanks! :-)


The white paper that you link to here says "Developers can write data access applications that enable idle connection resiliency with .NET Framework 4.5.1". Is the white paper wrong?


class Animal{
   public static void main(String[]args){
       System.out.println("Hello")
   }
}


Really great work! Very clean, very clear. I would clean up lines 114/115 and refactor some of your code even further:

```
// you can simplify statements like this with a ternary operator
if(computerMove === 'rock') {
                winner = 'Player';
            } else {
                winner = 'Computer';
            }

// like so:
winner = (computerMove === 'rock') ? 'Player' : 'Computer';
```

Loved the attention to detail in line 107 with your quit functionality!

I would have loved to see you try the extra credit as well.
Overall Great work! :100:


This commit breaks the build on my machine.

```
libtool: compile:  g++ -DHAVE_CONFIG_H -I. -D_REENTRANT -D_THREAD_SAFE -I/opt/java/jdk/include -I/opt/java/jdk/include/linux -Wall -g -O2 -MT libjzmq_la-Context.lo -MD -MP -MF .deps/libjzmq_la-Context.Tpo -c Context.cpp  -fPIC -DPIC -o .libs/libjzmq_la-Context.o
Socket.cpp: In function '_jbyteArray* Java_org_zeromq_ZMQ_00024Socket_getBytesSockopt(JNIEnv*, jobject, jint)':
Socket.cpp:246:10: error: 'ZMQ_LAST_ENDPOINT' was not declared in this scope
     case ZMQ_LAST_ENDPOINT:
          ^
Socket.cpp: At global scope:
Socket.cpp:602:6: warning: 'void s_delete_ref(void*, void*)' defined but not used [-Wunused-function]
 void s_delete_ref (void *ptr, void *hint)
      ^
make[2]: *** [libjzmq_la-Socket.lo] Error 1
make[2]: *** Waiting for unfinished jobs....
ZMQ.cpp:25:14: warning: 'void* get_socket(JNIEnv*, jobject)' defined but not used [-Wunused-function]
 static void *get_socket (JNIEnv *env, jobject obj)
              ^
```


nick, why


Sorry, this was a mistake when I thought it merged more than just this commit. I didn't actually click merge, but I didn't realize it left the branch behind.


Is there a way to contact u.. sorry abt posting in wrong place.
there is one issue in beanstalk
navigation bar keeps turning up randomly in capacitive key devices.


Would you please show a reference about the recent value of 'Standard-Version'? I cannot find it.


这里是干啥


如果是 --harmony 避免使用 var


(branch now deleted)


yeah, old code behavior
fixed


无意义 div 嵌套太多，尽量用 class 或者 id 区别。这种情况基本 H1 就能 select 吧？


because teehee. that's why.


Nice attention to detail with the `.toLowerCase()`! I wonder if you can refactor the code for your getWinner function by "switch"ing it for a dry-er pattern... and I would have loved to see you go for the extra credit. Overall great job! :100:


不要用 style 做 select ，样式是很容易变化的， html 结构是不容易变化的


+    if (poemAuthor === '' || !poemAuthor) {
这个没必要， !poemAuthor 就已经判断了。不信试试 !'' 看看是不是 true


避免这种方式，用 trim 函数


@galatrash-at-dnn Looks like this isn't an unambiguous date format (which was news to me, I just had seen mention in the forums that this was causing an error for folks).  See http://sqlblog.com/blogs/jamie_thomson/archive/2009/12/08/unambiguous-date-formats-t-sql-tuesday-001.aspx.  You can apparently remove the dashes (i.e. `'99991231'`) or use ISO 8601 format (i.e. `'9999-12-31T00:00:00'`).  Should I submit a pull request?  Does anyone know if there's already a JIRA for the errors that've been reported through the forums?


I'm not agree, c.Type is used during client connection to register them as AlertEventListener
Obviously only Alert client are registered to the listener.


用 const


do Normal = 0, Fighting =1, ....


Not sure if I need to add all the parameters for technique0. Should I condense it?


Thought we didn't need to because of the enum order?
On Tue, Jan 19, 2016 at 7:47 PM Ethan Celletti <notifications@github.com>
wrote:

> do Normal = 0, Fighting =1, ....
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/slabserif/Pokemanz/commit/de7cde6225eeb7ec8e75ba3e04824648aa12fd42#commitcomment-15547856>
> .
>


can combine cases.
case StatusType.Paralyze:
case StatusType.Burn:
    return 5;


SF 648


52줄과 236줄에 MakeEmpty() 함수의 반환값을 변수 L이 받아주어야하는것같읍니다


Technically no but it prevents screw ups/changes in the future


ASIHPdev (shenb) - 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-56.45%25-red.svg)


Should this have been v1.11.0? Personally, I have no idea. Just wondering.


Nevermind, just saw this ( https://github.com/numpy/numpy/pull/7065 ).


Nice! :smile:


Oh, I see, here is the url:

https://www.debian.org/doc/debian-policy/upgrading-checklist


and debuild reports it.

```
W: libhinawa source: ancient-standards-version 3.9.4 (current is 3.9.6)
N:
N:    The source package refers to a Standards-Version that has been obsolete
N:    for more than two years. Please update your package to latest Policy and
N:    set this control field appropriately.
N:
N:    If the package is already compliant with the current standards, you
N:    don't have to re-upload the package just to adjust the Standards-Version
N:    control field. However, please remember to update this field next time
N:    you upload the package.
N:
N:    See /usr/share/doc/debian-policy/upgrading-checklist.txt.gz in the
N:    debian-policy package for a summary of changes in newer versions of
N:    Policy.
N:
N:    Refer to https://www.debian.org/doc/debian-policy/upgrading-checklist
N:    for details.
N:

```


I loved your use of the ternary operator in your getWinner function! And using a testing suite? Impressive!  Getting very detailed, in lines 65-69, do you need to declare them as `null` or could you simply...declare them? Just a thought. Stellar work! :star2:


一些想说的


Oops looks like ill be generating a new token..


TESS YOU ARE SUCH A JERK


develop (DavidGoldwasser) - i386-Windows-7-VisualStudio-12: OK (2352 of 2389 tests passed, 0 test warnings)

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


reverted


Rather than making this test an "expected failure", does it make sense to just relax the tolerances to match the (hardcoded) tolerances used by the state setting algorithm in class Substance?


_"[Project64] Remove windows.h from CodeSection.cpp"_

Every change below this single line looks like it was not dependent on `<windows.h>`, so this single deletion looks like it could have been the entire commit and still work.  None of the below changes past this point look required for the purpose of removing windows.h.


This how a comment is made


Ah! I was looking for a document by a different name. Thanks!


I want to see you clean up this code a little bit. Spacing and indentation helps you keep track of where you are and that will be very important when your logic gets more difficult.

I saw your question in line 72 but you debugged it yourself in line 80! Great job! :100:


This change will break the build server.


Gah, weird. `gq` wrap fail.


As the commit message says, it allows `commit-dwim` etc. to work with the official smali versions, as it inserts the first item in this list. Before, activating `comment-dwim` would insert `//`, now it inserts `#`.


interesting idea. So like you 'fall asleep' on one server, join another and 'wake up'?


No need to have trailing space in `cd`. `:execute` will handle that.

```vim
execute cd fugitive_repo.tree()
```


```vim
execute cd '-'
```
is simpler, right?


Leave a comment


... R u sayin Thers a storyline in darkrp?

Sent using CloudMagic [https://cloudmagic.com/k/d/mailapp?ct=pi&cv=7.0.46&pv=7.1.2]
On Tue, Jan 19, 2016 at 6:04 PM, 808dkaneohe/billard-darkrp
<reply@reply.github.com>
wrote:
interesting idea. So like you 'fall asleep' on one server, join another and
'wake up'?

—
Reply to this email directly or view it on GitHub
[https://github.com/808dkaneohe/billard-darkrp/commit/483f0d9546a9b0ca3779bce63817048d1ca497b0#commitcomment-15547953] .[https://github.com/notifications/beacon/ANkX1STG2fgXOSPCIoXzJeU7_lU0Vyqzks5pbv7lgaJpZM4HIT0T.gif]


That last edit though. Most useful deletion I have ever made.


Great github handle! :joy:

This is beautiful work. I love that you incorporated all of the logic from the previous chapters of pre-work! Switch cases, ternary operators (nested!? Nice!)... I so wish you would have finished the extra credit.

Still, stellar job! :star2:


0 and 1 should be added so users know the available options?


I just inverted the motors instead
On Jan 19, 2016 7:46 PM, "Jaime Yu" <notifications@github.com> wrote:

> Hmm, I don't see it...
>
> I was thinking ... drive( Robot.oi.getJoystick().getX,
> Robot.oi.getJoystick().getY * -1);
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FRC2706/2016-2706-Rubiks/commit/1857e8f5ae7cb8777462ada930f37856345c133e#commitcomment-15545500>
> .
>


No, oh god no. Not at all.


Correct.  Much has changed since this patch was written.

```
$ git diff --shortstat 2a428d88e49e452dfa3b22cda7568de580e93b19^
 303 files changed, 7306 insertions(+), 5865 deletions(-)
```

You will need to update the patch or check out an older commit where it still applied.


yeah, I was thinking about that too
but was worrying that the storage engine may not be only string storage
perhaps the StorageEngine interface should should specify the key and value type ?

I saw you have another PR and issue regarding the JSON
let's discuss there


You should use a `std::vector<Vector>` here so that we have one large allocation instead of lots of small ones.


Wanna go fuck around in gmod?

Sent using CloudMagic [https://cloudmagic.com/k/d/mailapp?ct=pi&cv=7.0.46&pv=7.1.2]
On Tue, Jan 19, 2016 at 6:10 PM, Billard <notifications@github.com> wrote:
No, oh god no. Not at all.

—
Reply to this email directly or view it on GitHub
[https://github.com/808dkaneohe/billard-darkrp/commit/483f0d9546a9b0ca3779bce63817048d1ca497b0#commitcomment-15547988] .[https://github.com/notifications/beacon/ANkX1bmwyCFz_RkUvlTH8nkT7vflauHMks5pbwA0gaJpZM4HIT0T.gif]


This probably goes into `solvespace.h`, until we split everything properly into libraries


sure! download my hack


54, 55줄이 MakeEmpty() 함수와 똑같으니까 MakeEmpty()함수에 초기화 기능까지 넣어서 구현하면 코드가 더 깔끔해지지 않을까요?

그리고 저는 상수를 쓸 때 직접 숫자를 쓰는것보단 #define이든 const int든 사용해서 심볼릭 상수를 쓰는 걸 선호합니다. 코딩할 때 실수할 가능성도 적어지고 나중에 변경할 때도 한방에 바꿀 수 있으니 유지보수에 좋다고 배웠기때문에... 52,53,58,59,155,156에 심볼릭 상수를 쓰면 좋을것같아요.


:+1:

On Tue, Jan 19, 2016 at 10:20 PM, wpride <notifications@github.com> wrote:

> generally don't touch gradlew files (they're auto generated). someone
> (clayton) might want to run on windows someday anyhow
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dimagi/formsplayer/commit/55fb485eaec59cf062ba2dc57ee8a5631019c8d8#commitcomment-15547390>
> .
>


Looks alright to me


Needs work


This line is the issue.  The problem is the previous `plt.loglog` command creates axes in linear values but with a log scaling between the ticks (i.e., 1e1, 1e2, etc).  So, you need to make a plot in the linear values.  Thus I think you want something like
```
plt.loglog(mytable['MASS_EXTRAP'],1e1**(m*(np.log(mytable['VIRMASS_EXTRAP_DECONV']))+b),"-r")
```
But even this in non-ideal because you have a line that is just over the data.  Another approach would be to create a test set of values using [np.linspace](http://docs.scipy.org/doc/numpy-1.10.0/reference/generated/numpy.linspace.html) and then plotting that one.

```
x=np.linspace(1,1000,50)
plt.loglog(x,1e1**(m*x+b))
```
Or the even fancier `x = np.logspace(0,3,100)`


is T<extruderNum>  not clear , it is used for others commands

Or may be T<extruder> ? no need to put all parameter as
- T0 - select extruder 1
- T1 - select extruder 2

are already described


I wish you would have tried for the extra credit since you finished 7 days ago, but nice work here! Great use of switch case for getWinner. Just nitpicking here... Can you refactor code like this:

```
    if (move) {
        return move;
    }
    else {
        return getInput();
    }
```

using a different operator? True : false ....

Overall great job! :100:


Nice, thanks David!

-Chris


Just start a server that I can join

Sent using CloudMagic [https://cloudmagic.com/k/d/mailapp?ct=pi&cv=7.0.46&pv=7.1.2]
On Tue, Jan 19, 2016 at 6:11 PM, 808dkaneohe/billard-darkrp
<reply@reply.github.com>
wrote:
sure! download my hack

—
Reply to this email directly or view it on GitHub
[https://github.com/808dkaneohe/billard-darkrp/commit/483f0d9546a9b0ca3779bce63817048d1ca497b0#commitcomment-15548001] .[https://github.com/notifications/beacon/ANkX1ZHAkywuDPHejET9OyKy9vqjHqkRks5pbwB9gaJpZM4HIT0T.gif]


# Assignment 1 Grading Rubric (15 points total)

## Screenshots

- [ ] Assignment 1 contains directory of files for user (0.5 pt)
- [ ] Repository contains screenshot of [github][] account (0.5 pt)
- [ ] Repository contains screenshot of [Slack][] session (0.5 pt)
- [ ] Repository contains screenshot of [Atom][] installed on users computer (0.5 pt)
- [ ] Repository contains screenshot of screenshot showing that user installed [linter-flake8][] (0.5 pt)
- [ ] Repository contains screenshot of the Anaconda [REPL][] on user's computer (0.5 pt)
- [ ] Repository contains screenshot, in the repository, displayed by Github Desktop (0.5 pt)

## jupyter notebook

- [ ] Repository contains ipython/jupyter mini-tutorial as ipynb file (0.5 pt)
- [ ] ipynb Task 1 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 2 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 3 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 4 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 5 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 6 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 7 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 8 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 1 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 2 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 3 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 4 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 5 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 6 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 7 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 8 appears to be answered correctly (0.5 pt)
- [ ] You can successfully re-run every step of the ipynb notebook (3.0 pt)

[screenshot]: https://en.wikipedia.org/wiki/Screenshot
[git]: https://git-scm.com/
[github]: https://github.com
[Slack]: https://biolprogramming.slack.com
[Atom]: https://atom.io/
[linter-flake8]: https://atom.io/packages/linter-flake8
[syllabus]: https://github.com/biolprogramming/syllabus
[ipython]: http://ipython.org/notebook.html
[jupyter]: http://jupyter.org/
[REPL]: https://en.wikipedia.org/wiki/Read%E2%80%93eval%E2%80%93print_loop


It still didn't work when I changed it to *(INT*)(*(DWORD*)0x7A97F0 + 0x55A8) = 3084069;
Should I try using your memory code?


shouldn't this be `results.counts[`${folder}_count`] === 0`?


abc


Maybe put at least T0 = extruder 1 so new users know the starting point.


What exactly does this fix?


but it is already :
https://github.com/luc-github/Repetier-Firmware-0.92/blob/master/src/ArduinoDUE/Repetier/Repetier.ino#L56-L57


Ah, apparently I stop reading (or comprehending) things if I don't have coffee first thing. Thanks for the commits!


This was in an effort to help Ashok with one of his refactoring to remove all of the duplicate code between the two repositories.


true, I made lots of changes then looked at checking them in most I broke up to smaller parts, this one I just when what as the most important change in this file, call the commit that .. but yes it could have been broken down even more, guess I was just being lazy


没有


Lots of new tests! These would fall under Integration/Acceptance Tests. They test the whole workflow of adding users and events to the database which passes through several layers of classes, API's, and a database.


Continuation of new tests! These would fall under Integration/Acceptance Tests. They test the whole workflow of adding users and events to the database which passes through several layers of classes, API's, and a database.


Nit: Make this private static class


thanks! I also changed the values so the find should find more than 0 values


PrecedenceCompare()함수는 정말 잘 만드신 것 같아요 ㅜ.ㅜ

176줄에 input < '10'을 보면, 작은 따옴표 안에 문자열이 들어가 있습니다!


Nit: Declare this as final. Doesn't look like you allow it to be changed anyways.


gfdfgdsgf


a[j-1]>a[j]の>を<に変えると降順のバブルソートになる


I had a look at the ZMQ changelog and it looks like you're compiling against a version of ZMQ prior to 3.2.x based on when `ZMQ_LAST_ENDPOINT` was committed. Can you please confirm that? I'll amend my change to include `ZMQ_VERSION` `#ifdefs` appropriately for 2.x compilation and make another pull request.


Good job. Do more coding!


On second thought, do you really need to have this as a member variable? It looks like you only use it to instantiate a deviceLogger and don't use it for anything else. Maybe we can just leave it as a parameter to the constructor and not declare it as a member variable.


Nit: See above. If you get rid of this member variable, you can just check if the deviceLogger is not null.


Um das Liniensuchverfahren zu implementieren hat man sich entschlossen, es eher direkt in die Newton-Methode einzubauen, weil es im Allgemeinen eine Verzögerung der Lösung verursachen wird, unangesehen davon, welche Methode es implementiert. Und da die Newton-Methode immer schneller die Antwort trifft, macht es besseren Sinn, das Verfahren steilsten Abstiegs zu überspringen.


『%s』前后要加上引号，以防止路径中有空格。另外就是要用unicode字符串 & 输出前需要手工encode一下，以避免中文bug

demo(via [之前的修改记录](https://github.com/YaoZeyuan/ZhihuHelp/pull/54#commitcomment-15520838))
```
os.system(u'  "{}"  '.format(captcha_path).encode(sys.stdout.encoding))
```


I have added them on purpose because these lines are not in original doc


Nit: Why not just leave it commented out instead of using an if (false), since that will always skip it anyways.


I have reverted my installation to [4.2.5](https://github.com/SiCKRAGETV/SickRage/releases/tag/4.2.5) and will not be updating until these issues are fixed. Unless you were having problems with that version I suggest you all do the same until @echel0n sorts out what happend


Added some unit tests! Some of these are commented out. They may be implemented at a later date. These test single functions within the code so they fall under Unit Tests. Woohoo!


It is valid to start an expression with a closureExpr, e.g.:
let x = { 4}()

besides that, I'm not sure what is wrong with the current approach, it works great.


因为只有Mac一个特例，所以这里应该直接用else，不用考取其他情况。强行考虑反而容易引入bug（Linux用户哭晕在厕所←_←）


I don't see how the two are related.  This is checking to see if the diagnostic will get emitted to a valid source loc, e.g. that the location isn't implicitly synthesized or something.


Octree (DeadParrot) - Win64-Windows-7-VisualStudio-12: OK (2065 of 2068 tests passed, 0 test warnings)

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


I don't know, but it definitely does return invalid candidates sometimes, see higher in this function.  I didn't dig into why, but feel free if you're curious.


I don't understand what you're saying here.


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


@flavour I cannot get you...Do you mean tables event_checklist_template and event_checklist_template_entry with location_id? Why do we need template here? Can you share what structure are you thinking?


Could you test building the rom with disabled WITH_DEXPREOPT?
I heard that odexing CM can't install Xposed (i got that)


Stock rom can install it!


We can now remove trailing spaces. `:execute` joins arguments with spaces, so it's no longer required.


Yes, i know but in CM the setup/install button is grayed out


I think we don't have to enclose `log_opts` and `s:setup` in try block. We can do them before try and cd.


将println改正为print


Why the need to write to a new log file?  Why not just add a `LogggingUtil.LogDebug(this, <data node info>)`?


这一行写在captcha_path = Path.base_path+u'/我是登陆知乎时的验证码.gif‘下面？我加了这一行，输出时有提示，Permission denied


The comments saying stuff was added by Linuxgurugamer really aren't necessary.


This line effectively writes everything that was logged for this contract type to the debug log again a second time (minus the stuff that you've regex'd out).


Log a warning here if the delete fails.


Hi,
latlong_to_chartpix is called in KAP::Init before m_LonMin and m_LonMax are set in SetMinMax and AdjustLongitude uses uninitialized
values.

Regards
Didier


Initial version.


Thanks


`s:log_opts` definitely needs to be called from top-level. Otherwise, `git ls-tree` will fail. Would you like me to move `s:setup` outside the try block? It doesn't appear to have any code-level dependencies.


This will only work if VERBOSE logging is on for the contract type.  I'd rather something that just loops through the items in the data node (similar to the debug window's logic) and does a debug log.


You need a better description


learn to spell noob


Ah, sorry I overlooked. Yeah we can move `s:setup` outside of the block.


Add comment here about the inversion.

Note that the kThrottle doesn't invert iirc, it ranges from 0-1 and is that throttle thingy on the sidewinder. (Oh, maybe we should make a decision sooner than later on what's more comfortable as controls. Last team I was with did not enjoy the joystick).


Found it. Thx


tes' come"mnt:


I'm unsure on the naming/use of this method by the notifications. The reason I am reusing this code is to allow for clearing recent documents to correctly switch to Spotlight's tab.

`CPDocumentController.RecentDocumentUpdateNotification` would need to actually call this same method, but could not call `recentButtonTapped:` directly.


connect to


Escaping the quotes within the default filter shouldn't be needed. A lack of entry should trigger the default of "" when read by the master process.


Learning Angular. I wasn't able to connect the app to Firebase, therefore is not visible in  "https://shining-inferno-4012.firebaseapp.com/", the project only runs in localhost.


et elle


Instead of `osm_` it should probably be `openshift_master_` since that is the ansible convention.

We should probably update the style guide accordingly.


wrong indent, see https://github.com/ceph/ceph/blob/master/CodingStyle#L76


Outside of the minor nits I mentioned, this looks great.


Is this format correct?


for some reason I am getting "CallExpression" nodes passing the isClassMethod test. And they don't have an id so it causes an error


Yes it is.


and i am not sure if we are aligning the member variables in columns in Ceph.


make some line breaks here as well as the following test cases, easier for reading.


one more test case here is the invalid toleration operator


coud just put
```c++
static pidfh pfh;
```


SCRATCH THAT THIS WAS A 100 RED


There was extra credit? Was it the playToX() function? I actually did do
it, but commented it out in the submission because I thought I was supposed
to show the playToFive() function only!

On Tuesday, January 19, 2016, Rachel Smyth <notifications@github.com> wrote:

> Great github handle! [image: :joy:]
>
> This is beautiful work. I love that you incorporated all of the logic from
> the previous chapters of pre-work! Switch cases, ternary operators
> (nested!? Nice!)... I so wish you would have finished the extra credit.
>
> Still, stellar job! [image: :star2:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/penguyen1/wdi-fundamentals-rps/commit/c4f1f672adab3f155a9fdcbaa5fd24f9b4984e60#commitcomment-15547974>
> .
>


In don't get this why are we using an older Tika? @danlamanna @jeffbaumes


`pidfile_write(void)` is identical to `pidfile_write()` in C++. could we adhere to the C++ way instead of sticking to the C idioms?


Hey Alex, can we adjust this to also support thumbnail images, I suggest using `originalAlt`, and `thumbnailAlt`.


add spaces around `=`.


a the validation about taint key, please refer to label key, should be in the same format requirements.


Changing this line to the following seems like a reasonable solution

```js
 value.id && (value.id[t.NOT_LOCAL_BINDING] = true);
```


@AlexMeah Also, can you combine the commits into one pull request.


How about
```POST /druid/coordinator/v1/namespaces/SOME_NAMESPACE_ID```
for adding one namespace and
```POST /druid/coordinator/v1/namespaces```
for replacing current set of namespaces with new set?


NAN might not be defined in windows.  I think this is breaking the build.

I found this on the web:

```
#ifdef _MSC_VER
#define INFINITY (DBL_MAX+DBL_MAX)
#define NAN (INFINITY-INFINITY)
#endif
```

I don't have a windows machine to test, but you might want to try throwing this in `interpreter.cpp` and see if that fixes the appveyor issue.


에러 화면이 예뻐요


le message de validation du _commit_ finalement pointé par HEAD ("eventually" est un faux ami, ça ne veut pas dire éventuellement mais finalement)
une sorte de `git log -1` OU `git log -1` en quelque sorte


remove the space between '('s. and add spaces around '='. or you can just move `int ret` down to where `ret` is assigned.


It turns out there is a dependency. `s:setup` issues `tab new` and s:scratch(), so when `s:log_opts` tries to `expand('%')` it is given an empty string and GV! fails to give a file-only log.


Much appreciated! I saw it the first time but haven't had a chance to look into it quite yet :sweat_smile:


I think `value` ends up being a "callExpression" when one of its argument names collides with the name of the method


Descrição do Projeto, ferramentas utilizadas, comando para executar projeto e porta.


Thanks :D


https://github.com/libgit2/libgit2sharp[]
Pourquoi l'avoir mis sur 2 lignes ?


关联错误，应该是 #893


Won't we end up sending multiple `CONTINUE`s for `POST`s with large bodies (i.e. ones that require multiple HTTP frames)?


don't we have this method somewhere already?


sdfsd


line too long, it's probably fine to just say:

```
fail("IOException was expected");
```


No, this is only called once when the read is initiated.


Damnit, in the habit of typing before fully reading. Ignore this statement. This is only called once.


would probably be nice to put in a util lib somewhere along these tests to avoid rewriting it


The fix of gyp should be submitted to the upstream at first and then we will consider to apply this as a floating patch. It should be a separated commit so as not to be lost in the future updates of gyp.


Octree (DeadParrot) - Win64-Windows-7-VisualStudio-12: OK (2064 of 2068 tests passed, 0 test warnings)

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


The series of fixes in node-gyp should be submitted to the node-gyp repo.


also goodguy IDE seems to have tidied up unnecessary whitespaces in the new files


Revert this please. Its handled in the viewer....


Only true if the override has a matching type signature -- if parameters are more general or if the return value is more specific, this could potentially break callers.


inline comment on line no. 11


Whoops pushed to group not fork! Doesn't seem to have destroyed anything


#21 #109


Lol. What IDE you use? IntelliJ, Eclipse?

I'll update the jar in the release with these changes and any others you make.


This can be removed if we change https://github.com/nodejs/node/blob/master/deps/openssl/openssl.gyp#L70. since  only Linux is supposed to  run on arm.


添加
```ruby
include AsJson
define_as_json :simple_json, :only => [:_id]
```

这样两行，相比于我自己去定义一个 simple_json 方法，有何优点


Ha, you're right. Thanks for looking into it.


the error for this was downgraded to a warning for this release


当需要对 controler 返回的 json 数据进行精确控制时，可以使用 as_json 方法传参数解决，具体用法可以参考 http://api.rubyonrails.org/classes/ActiveModel/Serializers/JSON.html


can you just remove the status field here to make the test case more simple? Status is omitempty


Comment on whole commit


I am not a big fan of keeping 'legacy' stuff around to keep older projects working.
I believe most plugins are open source and you could send in a PR to fix the naming issues.

We have been in beta a while and have some breaking changes, the maintainers of the plugins will have enough time to update their plugins.


test


Is `<input type="number">` enough for this?


Le constructeur contient une API souple qui fournit tout ce dont il a besoin pour trouver un dépôt Git, que votre programme sache ou ne sache pas exactement où il est situé.


在模型上声明方法对 as_json 进行二次封装可以解决问题，但是参数比较长，这些只是演示一下可以通过这种形式简化 as_json 二次封装，比如简化参数，抽取公用参数等


:beers:


This `ret ;` looks like he is lost.  :-)  Is this just not supposed to be here?


当需要对 controler 返回的 json 数据进行精确控制时，可以使用 as_json 方法传参数解决，但是在 controller 直接调用 as_json 不能达到有效的复用，所以还是需要抽取模型方法


WHole commit


这样输出._id
就不会是返回{_id =>{$oid => 'xxxx'}}的形式了
而直接返回{_id => 'xxxx'}的形式了


20jan 2016 !!!


whoops, my copypasta is showing.


This is a logical AND... it will only pick 0 and 1.  You should not have to mask VAL_TYPE(val), it is a number that is set at 0 - 63...


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


@fushang318 这里同意，需要封装方法，而不是在 controller 里考虑取什么属性等问题


I noticed that this line appears to cause the cop to run whenever RuboCop is running under Ruby 2.3, even if the `AllCops.TargetRubyVersion` config option is set to 2.2 or older. Should this instead be:

```ruby
return if target_ruby_version < 2.3 && style != :always
```

I'm asking as the specs explicitly check `RUBY_VERSION` when defining examples, but I would think only the `target_ruby_version` matters here.


I use Eclipse.

I'm currently testing the new methods, not sure yet if further changes have to be done.


This is breaking for me on master.

![errorimg]
(http://chris.si/img/RCTNetworking.png)


应该是这样
```
import sys
if platform.system() == "Darwin":
     os.system(u'open "{}"  &'.format(captcha_path).encode(sys.stdout.encoding))
```


This patch fixes vulnerability CVE-2016-0728 as reported here : https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=23567fd052a9abb6d67fe8e7a9ccdd9800a540f2


define_as_json  第二个参数目前是直接传递给 as_json 作为参数的，我设想是这个 参数可以进一步简化和抽取，比如
```ruby
define_as_json :simple_json, :except_timestamps => true
```


`<input type="number">` is good idea.
However, `<input type="number">` can prevent to input such as alphabets?
I'm afraid to show 500 error on admin page.


good find


In addition, we should consider to add a new config of WIN_CE in `deps/openssl/config/archs`. I'm not sure that the openssl built with VC-CE still works fine even on Window10 IOT.


This was a good call, it was unnecessary.


You could also have used the global join method: join(strings, ";")


latin2_czech_cs problém vyřešilo? Normálně se tohle řeší jen jedním SQL řádkem navíc, který se o překlad postará před každou komunikací... musel bych chvíli hledat.
A v 99.9% případů se používá utf8_foo. UTF8 je jasný a to, co je za podtržítkem, určuje, jak budou fungovat funkce na abecední řazení, protože např. znaky s diakritikou v anglický abecedě nejsou, takže třeba s utf8_general_ci by český znaky s diakritikou byly až za "z", pokud si to dobře pamatuju...

Dej vědět, jestli jsi tímhle latin2_czech_cs problém vyřešil, pokud jo, utf8 v DB nebudeme řešit, pokud ne, můžu to dohledat :)


Preview


Sb


What do you mean "500 error" ?


rest


write


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12031](https://teamcity.mesosphere.io/viewLog.html?buildId=108449&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:00:45


la syntaxe « rev-parse » (pour rester cohérent avec la même référence dans Libgit2)


@Kangz: https://github.com/Unvanquished/Unvanquished/blob/1e207d36b109e64e35bdaecd29e05038a26fbc8b/daemon/src/engine/qcommon/msg.cpp#L473


我知道不影响在，只是和原来对应下比较好，还是加上空格比较美观


这样改了，，，没变化，图片还是可以打开，还是会有提示信息
<img width="850" alt="2016-01-20 13 36 28" src="https://cloud.githubusercontent.com/assets/6964284/12440825/e15dce76-bf7a-11e5-9c09-f2dcbe756bb1.png">


I agree with @sds here.


不要不要的


`Permission denied`是因为文件还没创建呢←_←


I looks like it should be 0xC0, but I'm not sure what that block of code does. Commit a5f15c62853080dbb29f44bc7483c14b86936fcc is what I'm using in production and it works pretty well.

To me it looks like line 139 writes the contrast byte at 0x081, then line 140 immediately writes the contrast byte to whatever was passed to `u8g_dev_st7567_pi13264_fn()` as an argument. So I guess 0x081 is the default value in the event that `u8g_dev_st7567_pi13264_fn()` without an argument. Is that right?

If it helps, in my main code I set the contrast level with `u8g.setContrast(0x018*7)`.


ewewe


acknowledgement


Oh, sorry. At the moment, `PG::NotNullViolation: ERROR` is occurred when non-number is input to order.

![screenshot from 2016-01-20 14-38-49](https://cloud.githubusercontent.com/assets/386687/12440841/92382158-bf83-11e5-9a43-65e381370bda.png)


chk


ok


@craig-warren  Why was this removed? Any particular reason?


创建了啊，代码下一行就是打开这个文件


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

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


disponible à


de recettes JGit à https...


énumérées à


Passer les valeurs dans status.json à 100.


No. Was a corrupted cache of the dependencies. I've cleared it out.


Yeah, it seems like the wrong thing to do, especially from the split manager. Also, `Exception` is way too broad, this code doesn't throw any checked. Seems better to remove it for now and add in more precise error handling later if people complain


[Diagrama de Gantt.pdf](https://github.com/CeleneCamargo/IngenieriaDeSoftware2016/files/96997/Diagrama.de.Gantt.pdf)


I was thinking that but couldn't come up with a good name/signature. Will do that later after everything is merged


Heyo, before I convert the site, it would be nice to start using <br>

Thanks
The Freshie of High School Programming


I added this plugin in my project in cordova visual studio but it is not working.

Error - Plugin (or a specific plugin version) not found in external repo: cordova-plugin-splashscreen@3.1.1-dev": Fetching plugin "cordova-plugin-splashscreen@3.1.1-dev" via npm Failed to fetch plugin cordova-plugin-splashscreen

when i make apk it is giving error


paged=False for gnip client, so we don't do full download of tweets


https://github.com/kidaa/followers You appear to be following this person. Just look at its activity log. You must have thought something when following this "Kidaa", right?


109490574b-UnbalancedFlowWarning-4293-PlanB (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (2056 of 2057 tests passed, 0 test warnings)

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


This should go into a #ifdef USE_MUMBLE section... and you'll want to update CMakeLists.txt


Do I need to test this route? I feel like this is a place for synod


I dont understand. Using what?


Sorry for the delayed response. What does this screen look like on your
skill?

https://raw.githubusercontent.com/parisbutterfield/alexa-stocks/master/screenshots/schema.png

On Mon, Jan 18, 2016 at 2:16 AM, wins12ton <notifications@github.com> wrote:

> please help
>
> Error: There was a problem with your request: The intent 'StockTickerName'
> was not found in the domain definition. Occurred in sample 'StockTickerName
> What's the stock price for symbol {A | Symbol} {A | Symbol} {P| Symbol} {L
> | Symbol}' on line 7.
>
> how can i fix
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/parisbutterfield/alexa-stocks/commit/db76b99411b77794a9eb9352a2a4c87a2dce6412#commitcomment-15500319>
> .
>

--
Regards,
Paris A. Butterfield
Mobile (347) - 405 - 4241
Email: parisbutterfield@gmail.com
Email: paris.butterfield@lc.cuny.edu
Are we LinkedIn? <http://www.linkedin.com/pub/paris-butterfield/23/433/a0b>


I originally was doing the list(set(tags)) thing, but it doesn't seem to preserve the order in which tags appear in the email. It can end up returning a random subset instead of the first three. Which I guess is fine, just less predictable for the user


thanks, got it.


Actually, this one allows for 0 to 1 in 0.1 increments. It was straight up 0 or 1 before.


How are you going to change the date and not change the answer???


https://wpilib.screenstepslive.com/s/3120/m/7912/l/79828-operating-the-robot-with-feedback-from-sensors-pid-control

screensteps about how to use the pidcontroller in 2016 (different api than last years)


Woah it didn't show up, lemme try that again use

<br>


这一行的作用是防止中文路径的编码问题？我重命名了一个中文的路径，注释掉了那一行，没有出现问题，可能把 OS X 设置成英文才会有编码问题吧，我猜的，，，我也才刚用。我觉得删掉这一行问题不大，如果有 Mac 用户提了 意见再修改吧
<img width="694" alt="2016-01-20 13 50 46" src="https://cloud.githubusercontent.com/assets/6964284/12441003/6b986072-bf7d-11e5-8462-9d9ae544ddd2.png">


This break looks like it went missing!  I don't know if it is good or bad to just use the low 32-bits or whatever vs. the XOR with the high bits, I do not have data to back it up... did you have a reason to want to change it?  If there is a good reason then it should probably say `cast(REBCNT, VAL_INT64(val))` so it can be clear that it's throwing away bits...


WOAHHHH

//<br>


AFAWARGHARE

< br > without spaces


It seems you're using spaces instead of tabs, maybe we should both use the same style :P


Thanks I will fix this.


Nice commit


And what are you converting to what?


oh interesting. Hm I guess I haven't encountered that in the past, but theoretically it can certainly happen. I will change it to something that is order-preserving. Thanks!


and whats a br


Yes, I am allowing use of `internal` for `InterceptorPool`, but `ApplyChanges` should not be marked as `internal`.  In general `internal` is no longer considered good practice (see the MS engineering guidelines for OSS).  So let's limit our use of it.


So this is interesting, is there any type it should not be legal to put in a MAP!?  Just because the hash is poor it seems that people might not care that it's bad...and be willing to pay for the slowness.

I have thought there might be a "performance hinting" mode where it tells you when you are doing something it knows to be slow, like writing to a log some kinds of warnings.  Basically that being the answer in cases like this vs saying *"you can't do that"*...there are some cases in PARSE where Carl didn't want to allow the use of a construct that would be knowingly inefficient, but I think erring on the side of letting people do their work lets the people who care about performance do that.

Basically: when you try and tell people "no you can't do that" they get madder than if you tell them they can do it if they want, but it will be slow.  :-)


npm install does not work anymore

you can check it directly: `npm run postinstall`

-> the second command begins with a dot and this does not work

I have solved it and will committing it soon.

```
C:\Users\moz\Workspace\angular2-webpack-starter (master) (angular2-webpack-starter@0.0.0)
λ npm run postinstall

> angular2-webpack-starter@0.0.0 postinstall C:\Users\moz\Workspace\angular2-webpack-starter
> npm run clean:ng2 && ./node_modules/.bin/typings install

> angular2-webpack-starter@0.0.0 clean:ng2 C:\Users\moz\Workspace\angular2-webpack-starter
> rimraf ./node_modules/angular2/manual_typings

Der Befehl "." ist entweder falsch geschrieben oder
konnte nicht gefunden werden.

npm ERR! Windows_NT 6.3.9600
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Users\\moz\\AppData\\Roaming\\npm\\node_modules\\npm\\bin\\npm-cli.js" "run" "postinstall"
npm ERR! node v4.2.4
npm ERR! npm  v3.5.3
npm ERR! code ELIFECYCLE
npm ERR! angular2-webpack-starter@0.0.0 postinstall: `npm run clean:ng2 && ./node_modules/.bin/typings install`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the angular2-webpack-starter@0.0.0 postinstall script 'npm run clean:ng2 && ./node_modules/.bin/typings install'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the angular2-webpack-starter package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run clean:ng2 && ./node_modules/.bin/typings install
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs angular2-webpack-starter
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls angular2-webpack-starter
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\moz\Workspace\angular2-webpack-starter\npm-debug.log
```


If this is aesthetics/performance related, then it's most likely unnecessary. We should focus on uploading outreach and photos to the website, rather than focusing on design and things like that (site already looks good).


Yes agreed.


cari cara lain jangan rubah langsung di ad_order_preparation


Why `require false`?


Jake u are literally the best frosh ever
yeah sean, these coming weeks need to be focused on robot stuff.


Dongjie, thanks for your effecient & hardwork


Is there a reason to use NONE! instead of an UNSET! here?

I could see myself perhaps wanting to map from NONE! to something.  For instance if I were doing some kind of search and replace, maybe I want to make a MAP! of patterns.  The keys are the things I find in a block and then the value is something I want to replace it with.  I might have something I want to say is used to replace a NONE!

This is why I am trying to push that UNSET! (or VOID!, I want to call it) cannot be put in a block or as a value in an object.  I want to have something you can really say "isn't data".  But NONE! is data...

Does this seem sensible to change these to IS_UNSET()?


                                                                                Figured she would just pick up on it...                                                                                Sent from my BlackBerry 10 smartphone.                                                                                From: Alex NavarreteSent: Tuesday, January 19, 2016 10:55 PMTo: turingschool/black_thursday_spec_harnessReply To: turingschool/black_thursday_spec_harnessCc: Allan Maxwell BlairSubject: Re: [black_thursday_spec_harness] Changing the date in revenue by date (a45abea)How are you going to change the date and not change the answer???

—Reply to this email directly or view it on GitHub.


也就是说现在的情况是『可以打开图片，但会提示`Permission denied`错误信息』？
那就先这样吧，不影响正常使用就行。


Might be worth describing in your commit message what the bug is that you are fixing... just my 2c worth :-)


Done


你用chmod 755 给图片加上可执行权限试试，看还有提示不


https://github.com/franciscofranco/bullhead/commit/1e044e73b39f41add215cbc1472e579c6751a78f
방안을찾은듯합니다.
이커밋이 써멀기준점을 높여주는패치입니다.
아마도이커밋을적용시킨뒤 템프부분을높여주면 아마도해결될듯합니다.


代码里加？我试试


@kcnm please pass this test.


This change disables x11 support if platform has x11 flag.


approved BranchC merge req from jeff


So is it the case that this checks for zombie keys but it does not set any yet, because that would be done by removal?

*(It's okay if the removal is not written... I'm just asking to know, because I see where to change IS_NONE() to IS_UNSET() but not where the NONEs were set!)*


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


Questions to ask in class:

1) What is .DS_Store?
2)Why did I have to force the push to github with git push -f origin master?
3)Once I was able to push it, why did it add numerous times? (Did git push -f origin master push once and then I did again with 'git push origin master'?)
4)Why didn't git push origin master work in the first place?

**(Updates were rejected because the remote contains work that you do
hint: not have locally. This is usually caused by another repository pushing
hint: to the same ref. You may want to first integrate the remote changes
hint: (e.g., 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.)**

5)Had to do 'git pull' in terminal - what is this exactly?


It would be nice to use a less generic name, for example "GitFreenet".

One usecase is readable high-level unit tests:
Compare:
```assertEquals(testData, new Plugin(...));```
vs:
```assertEquals(testData, new GitFreenet(...));```

The latter is more self-explanatory.


could we use some more descriptive name? such as V4l2_TIME_TO_YAMI_TIME YAMI_TIME_TO_V4L2_TIME...
double make me confused with C native type double.


At the same time it's also tough because things like Curse ride off `ModifyMove` to change its targetting and causes bad interactions in its old spot...

Can you open an issue with all current wrong interactions? I'll think about this later


It didn't work son


Sure about that? Copy was probably there for a reason...


This code block is a duplicate of the one at line 57.
I would remove this one here, the line 57 placement is more where one would expect such code.

Also notice: IIRC you only need to do the Logger.registerClass() thing if you want the Logger to deal with the "logDEBUG" / "logMINOR" special field initialization. You don't seem to have such fields yet.
(Those fields are a performance optimization: They allow stuff such as "if(logDEBUG) Logger.debug(very_expensive_to_compute_string());" to only cause computation of the expensive string if logging is enabled. I.e. without the if() guard, the string would have to be passed to the Logger.debug() function in all cases, even when the function decides that it doesn't want to log it anyway)


Or, I guess you already fixed it


I committed a "fix" to it. It'll do for now.


windows doesn't like `&&`


If there's no Class, `device['Class'] == None`, right? Or may `device['Class']` lead to a `KeyError` or similar?


你的


手工在终端里敲就行。
你昨天测试的时候有这个问题吗？是不是『我是知乎的验证码.gif』本身就有问题，直接删掉试试？


fml...


can you remove `./node_modules/.bin/`


Hey @tpudlik and @nmayorov , here's something to try out:

https://gist.github.com/person142/40be5bf7503cc7895bfe

It explores applying Shanks transformation to the Taylor series. Based on a not statistically significant sample it seems to speed up convergence, but similarly to the asymptotic series it reaches a minimum error and then crashes badly, so if you don't truncate at just the right spot then life will be bad. Unfortunately its minimum errors aren't any better than straightforward use of the Taylor series, so it doesn't gain us any precision, just (maybe) some speed.


I don't think it does - x11 only added the --enable-sdl2 flags, but since they all had that I added it to the initial params


Remove catch block


Nice work, good job. :+1:


NumPy doc follow the pep and it tell it does shape "[1, a.shape[0]]". When we want to add an extra dim, we are better to use dimshuffle. This shoud be?

a = a.dimshuffle('x', 0)


b = b.dimshuffle(0, 'x')


aah sorry. I misread the += vs ==


I'll sort it.


CentOS 7 build is not affected as it links to OpenSSL dynamically -- you just have to upgrade the system package. Only Windows is affected as it does not come with a system-provided OpenSSL.


SS use a modified version of tinyalsa which has pcm_set_sample function, cm does not. Goolgling around, found nothing.


Very nice!


删掉了重新运行还是一样。昨天用的 Win 7，没发现这个问题，然后我删掉了那一行 push 了。。


This introduces a crash  when creating a new player.

diff --git a/source/PlayerInfo.cpp b/source/PlayerInfo.cpp
index ca3321f..eb83c89 100644
--- a/source/PlayerInfo.cpp
+++ b/source/PlayerInfo.cpp
@@ -571,6 +571,9 @@ const shared_ptr<Ship> &PlayerInfo::FlagshipPtr()
        }

        static const shared_ptr<Ship> empty;
+       if (!flagship)
+               return empty;
+
        return flagship->IsYours() ? flagship : empty;
 }

or something like that


Using scan will work. But a more optimal version can be done with that condition:
`a.broadcastable[:-2] == b.broadcastable[:-2]`

In that case, we can reshape a and be and call batched_dot. Then we need to reshape again. This is the case where there is no broadcasting.


Why indeed! I don't have a recollection of removing it, my guess is that it's some merge error between different cutorch versions. I can send out a PR with it added back. Thanks @dominikgrewe


fixed. I changed the logic around again as I think the way it was before was confusing with += and then resetting params for x11.


aaaah this explains this https://github.com/karpathy/neuraltalk2/issues/52 as well, send a PR to add it back.


I'm thinking it more, I think the logic when the number of dimensions do not match should be to explicitly add dimshuffle to a or b to have both of them have the same number of dimensions.

Then if the condition above is true, use the reshap trick you have at the end. If there is broadcasting use the scan. In resume the order is changed to this pseudo-code:

if nb1!=nb2:
   # make both tensor the same number of dimensions
if a.broadcastable[:-2] == b.broadcastable[:-2]:
   # use the reshape trick.
else:
   # use the scan version.


... Please please pass ... it would be so good to have some simplification of time management...


I would be happy to accommodate a PR


Interesting. Opening the image URL directly works as does 'view image in new tab' right-click on the CfH homepage. Not sure why it's not working in a standard embed.


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


hallo


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


Same problem,  on both OS X 10.11 and Linux.


I'm sure we will be switching the logic around more here if/when we add more targets ;-) thanks for the heads up. Please let me know if you spot anything else.


if (zk_rm_init_group_matched_node(mapping_info, &ops[op_count++]) < 0)
  break;

이렇게 해도 될 듯...
아래 부분에 있는 코드도 마찬가지...


lChnage this


Che schifo di nome xD


109490574b-UnbalancedFlowWarning-4293-PlanB (mjwitte) - i386-Windows-7-VisualStudio-12: OK (2056 of 2057 tests passed, 0 test warnings)

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


Which change would you prefer?

**A**
```json
"postinstall": "npm run clean:ng2 && npm run install:typings",
"install:typings": "./node_modules/.bin/typings install"
```
Link: https://github.com/martinoppitz/angular2-webpack-starter/blob/c4a7beac598c10cacd4add06eec65cd845f72ee7/package.json

**B**
```json
"postinstall": "npm run clean:ng2 && typings install"
```

*I thinks there was a reason for `./node_modules/.bin/`.*


indentation 오류


用ROOT_PATH进行切分比较奇怪，增加一个PATH_SPLITTER吧，虽然都是“/"，但是意义不一样。


struct里直接写个type指针是什么用法？


este formato aun lo voy a trabajar mas


这个圆点是怎么打出来的？


ahora tiene como parametro una funcion que sera la que compare


will it now take faction taken form dbc files? i think just like TC?


Seems to 'dset[10]' is 10, not 9.


I have commited this change. For some reason it is registered under @rbruhn


Would be cooler yet if you check the arena files for the mapname if available like votes.


loading特效


There should be a space between *components* and *to*.


EOODF would suggest to do it the other way round: Switching off Gradle junit4 support, which can be done with the Gradle plugin of JUnit 5.


Awesome! But you will need to set the [testTarget](https://maven.apache.org/plugins/maven-compiler-plugin/testCompile-mojo.html#testTarget) and [testSource](https://maven.apache.org/plugins/maven-compiler-plugin/testCompile-mojo.html#testSource) to 1.8 in the maven-compiler-plugin configuration so that this works from the CLI. (Did you test from the CLI? :wink:)


-Learning Guthub


Ini kayaknya ga dikondisikan,,


Typo: *Fileds* -> *Fields*.


Also, that will have implications for downstream projects that inherit the SJC tests JAR (yes, some actually do that!). Those projects will also need to set their testTarget/testSource to 1.8 once they update.

I don't foresee any serious issues with this, though.


NuGet is parallel by default. They were dropped here to simplify transitioning to `dotnet restore`. There was no technical limitation that caused them to be disabled.


.ap s tsos
'try2hack'


增加FindServlet的单元测试，顺便增加WebService的单元测试


iya itu masih langsung.
dikondisikan spt gimana mas?


It doesn't drop EXP when kill.


many thanks!


very helpful


yikes! sorry soumith, it looks like you were chasing that for a while.

Fixed in https://github.com/torch/cutorch/pull/305


Hi,

I think this might be a workaround but I'm not sure it's "the right way" to fix this issue.

Please note that I have not tested this extensively.


done


空格太多，造成一行太长


Ok, now its make sense why i had warning on boot about no work list from this driver probe after tasklets commit.


please remove these comments


I can't believe I accidentally deleted my entire review for Everything Bad Is Good For You.


To documenters, it would be nice to speak of the [0.01, 4] range for the zoom in the description of this function :)


Yes, the `break` goes after the `ret =...`, thank for noticing!
The reason is that XOR with high bits collapses -1 with 0 etc...
If your key k is |k| < 2^32 high bits are 0-informative.


@lubomir, thank you~!


Can the publisher here be something like "https://github.com/Microsoft/bond" instead of you?


Could you please briefly describe what the criterion does? Thanks.


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


改为server_ip和server_port，增加client_ip和client_port，方便用户使用，譬如用户可能需要打日志、统计client来源等


Nice find Fernando. Would there be a way to make this conditional to CloudFlare being used? Is there a constant, setting or function that CF sets that we could query to achieve that?


Minor: can you alpha sort the list to make is easier to add/remove in the future?


Just for when we will use VAL_TYPE > 63 for user types... ;-)


Hehe, was pointing the "break" statement not the ctype line :)


that didn't help with clean.

@Bogdan-G  I'm not totally sure what you mean - should i reinstall gradle, java, delete cache and change user?
that seems like a lot of work for it to compile each time?


啊😭 怎么了，chmod 755那一行要删掉么、


Absolutely, I'm doing that!


I don't think we should trim. The spaces were there to find only package lines and to exclude package dependency lines.


Shouldn't this take a number parameter?


Same here.


So I think the axcrypt-sha1 format stays and we need not bother with dynamic. The beauty of dynamic is for end users to whip up new formats that aren't present in their JtR binary.


The initialization changes depending on the constructor so in-class initialization doesn't really fit the need. I will sort those.


Yes, that IS_NONE -> IS_UNSET


应该是只留这一句`os.system(u'open "{}" &'.format(captcha_path).encode(sys.stdout.encoding))` :joy:


生成后的效果
https://github.com/GameXG/gopl-zh/releases


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


I may be wrong, but looking at the source, it looks like the IdleService also has running(), idling(), and watch() functions.


kudos to @ser-kletrov


I'm not sure if @drtimcooper wants C++11 constructs in his code. The `std::move` was only used for VS2013+ so I kept it generic.


Alright I see you've corrected this.


应该是把这三行都删掉，换成`os.system(u'open "{}" &'.format(captcha_path).encode(sys.stdout.encoding))`


How about to count each instructor only once, like it is done in the InstructorLuchBreak.getValue(assignment, variables)? Would summing up Instructor.value work?


To localize select2 and jQueryUI I introduced [`AssetLocalization->getSelect2Javascript()`](https://github.com/concrete5/concrete5/blob/60b8ae436c5ea65e3c66b694256bcae2be2f9121/web/concrete/controllers/frontend/assets_localization.php#L309-L339) and [`AssetLocalization->getJQueryUIJavascript()`](https://github.com/concrete5/concrete5/blob/60b8ae436c5ea65e3c66b694256bcae2be2f9121/web/concrete/controllers/frontend/assets_localization.php#L482-L506).
Maybe for CKEditor we could implement a similar assets-based approach...


Please Brian, wait to modify and merge, other changes are needed. Coming soon...


It seems that the ng.idle module prefixes should be changed to angular.idle. This was never updated when the module name was changed previously.


done


不自动打开验证码？


亂入... 這個 ip 應該不是用這個吧？


用`os.system(u'open "{}" &'.format(captcha_path).encode(sys.stdout.encoding))`打开验证码


@justinjdickow , are you sure that we need to remove this one. We can hide some problems if remove this `DCHECK`. I mean if `DCHECK` means that something wrong happened and I think we need to investigate the root cause of this dcheck instead of removing one


这里没加open


Hi Vanessa,
Thanks for the reminder.
v1.8.1 has published to npm.

Bruno Correia

> On Jan 19, 2016, at 5:21 PM, Vanessa <notifications@github.com> wrote:
>
> Are you going to push this latest version to npm?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/bcorreia/panels.js/commit/1b6f33e5d0983aaac72234bbf0b35d6c46d0051b#commitcomment-15546096>.
>


Remove <V extends Variable<V, T>, T extends Value<V, T>>, use <Lecture, Placement> instead of <V,T> where needed (the criterion is not general enough to use V, T anyway)


I didn't notice that it also happens even on OS X, as I compiled vim with this patch successfully.

But having read @chdiza 's note, I tried to compiled vim again without GTK GUI and then got the same problem.

So, those who set `--gui-enable=gtk2` haven't seen the problem...


It does not work as expected. `std::string(xmlValue)` causes a compilation error. I don't really see why though but it's consistent with the comment `This is an alternative to operator std::string() that Aigner claims is needed for VS13`...


I think you could (should?) have used `CultureInfo.Parent.Name` instead of splitting `CultureInfo.Name` by dashes. You could probably even recurse this to check for the parent culture file as long as parent culture is specified. Check out https://msdn.microsoft.com/en-us/library/system.globalization.cultureinfo.parent%28v=vs.110%29.aspx .


Should not this be List<InstructorConstraint> instructors2 = conflict.variable().getInstructorConstraints();? The getDiffInstrValue takes InstructorConstraints now.


噢。。尴尬了。。。


:joy:


@kevingo

我在後面的 commit 有修正了唷 在這邊 https://github.com/Mediatek-Cloud/mcs-resources/commit/dfc35871e46c475992f12d29c69825ac0b770a34


It looks like KeepaliveService also has a setInterval() function.


The single quotes looks weird?


John *


Please add a dot.


https://github.com/offensive-security/nethunter-utils/blob/master/usr/bin/dumpmifare.sh (this is the file)

Solves this issue: https://github.com/offensive-security/kali-nethunter/issues/347


Le code retour 0 c'est généralement pour dire que tout s'est bien passé. C'est 1 pour dire qu'il y a eu une erreur


Should not this be called addInstructors instead of getInstructors? It does not return any value.


done


@JustAkan 지금 gcc6 관련 하여 수정 중이라 수정 완료 후 확인해 보도록 하겠습니다.


I debugged and found no use of msm_rpm_data.work. Later found this commit was present in CM gerrit. :)


done -but it did effect the height of the header


That `destroyMethod` is superfluous (Spring always calls methods called "close" on shutdown).


Yer a wizard harry
![image](https://cloud.githubusercontent.com/assets/1292406/12442288/72dfb13a-bf57-11e5-9ea8-db022ae5b66c.png)


Also, fixed #57


use fairnessDouble (so there is only one place to change / override if different criteria are to be used in the fairness)


@nyalldawson Is this considered a feature?
It works fine, I think it is safe to do and it allows to define expressions which adapt to categorized and graduated renderer symbologies.


You can have a position that is "left:0px;right:10px;top:15px;bottom:1px" and it is a valid position. In that case it creates a slot with those dimensions.

So, instead of using if-else, could you change it to just using if-statements for each value?


asdsd


i just like coding !


Look at this to see how the build system find ril class https://github.com/ngoquang2708/android_frameworks_opt_telephony/blob/cm-11.0/Android.mk


okay! done


I plan enable all types, then remove that default.


No stress!  Just stay not bored... :-)

The following changes are good enough for me for now, but review if you want to work some more:

https://github.com/hostilefork/rebol/commit/509bc5e6ff7a5370bc52a8ed3965cde176289c1d

You can just ignore it and do other things, I'll merge the above and we can just deal with it on an as-needed basis... that is fine by me!


The value this function takes now appears more complex than a simple boolean. It now appears to have 3 valid values according to the documentation: "Possible values are off/false, idle/true, or notIdle". A boolean | string enum (https://basarat.gitbooks.io/typescript/content/docs/tips/stringEnums.html) parameter type may be in order.


Great. Btw that'd require change in `HumanOutput` as it only shows `9` chars of the status, while `INTERRUPTED` has 10. I think it should keep using the 9 chars as it's sufficient.


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


Why did you removed the :image from here?, that the problem image is not getting saved with messages and its not going in emails.


I had forgotten to push the related changes in `ceylon/model` and `ceylon/typechecker`. With ceylon/ceylon@db6c275 pushed this should be OK now.


How about to reuse sDoubleFormat (from AbstractCriterion) instead?


As I remember, something is called this in your commits.


I did this refactoring while looking for a solution to a problem I had. Probably the old code works, but it was creating the jobsIterator, checking for hasNext(), but never calling the next() method, instead using the
```
for (Job job: new ArrayList<>(workflowInstance.jobs))
```
statement to iterate through the jobs.
I think this code now works as originally intended, but pls check because I might overlook something.


This was actually an accident, BTW, will put it back before merge.


I don't understand why this line was modified. It seems like it will cause a CONST_INITIALIZED_WITH_NON_CONSTANT_VALUE error to be reported for final variables, which don't have to be const. See:
https://github.com/dart-lang/sdk/issues/25526


:+1: workses


Plugins will not work without upgrading to beta8 anyway. Maybe we should write down this requirement somewhere.


copypasta


@kkateq Do you prefer this method of passing variables to JS or with a script tag and putting the variable in the global space?


That's almost certainly it. I'm running the libzmq versions available from Ubuntu's repos, which is probably somewhat old.


@unfulvio why was this method removed? I am pretty sure we still need to filter `woocommerce_order_number` even if we don't have to add the `#`. I will add this back unless there are any objections


Ok.


I meant to put in the cast(REBCNT, ...) here.  Will do before merge.

(just wanted to push it so you would know you didn't have to work on it if you didn't feel like it!)


Hi,

really need your help,

these class help to make connection and send messages to PMTA?

Or there are other things as well which required attention?

Looking forward to your prompt reply.

Kind Regards,
Kamran


thanks, fixed


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.640.e85806f49f02a615edbd030952f5b1f3c93bfba2](http://build.palaso.org/viewLog.html?buildId=68105&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3337, ignored: 130 Build time: 00:05:39


I think I removed it after noticing there was no other class method calling this method

if as a filter we return the number as it is... what are we filtering, exactly?


distinct.

Otherwise, LGTM.


fgffdsdf


develop (rpg777) - i386-Windows-7-VisualStudio-12: OK (2352 of 2389 tests passed, 0 test warnings)

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


Oops, initialization too early.  See this is what happens when I push things too fast to show people.  :-)

I've git push --force'd a few changes, BTW.


ok i see the original problem now.
rather then instantiating an array, another solution is to use the iterator for the loop like this

```
Iterator<Job> jobsIterator = workflowInstance.jobs.iterator();		 +
while (jobIterator.hasNext()) {
  Job job = jobIterator.next();
  ...
  if (jobsIterator.hasNext()) {
    ...
  }
}
```


By saying _they are all merged together_ and _So be careful not to define a custom validation that collides with any of the base types or validations in Waterline_ I don't think (in my mind) that we get that, if we name two custom validation rules with the same in different model, the two rules will be one finally. IMHO, I think it would need to explicitely say it (even if you warned before that they will not be used anymore with waterline 1.0, if I understood well).
Anyway, great work on this docs additions.


@unfulvio we are filtering the [order number](https://github.com/woothemes/woocommerce/blob/7ece70a6ae89207f6fa406ab164de567b0145908/includes/abstracts/abstract-wc-order.php#L997-L999) which is the order ID (post ID) by default. Removing this removes all of the functionality of this plugin


I was wondering why there is a style attribute in package.json, then I found the commit message realy helpful.

:+1:


the commit command saved my day


ok but if the filter passes `$order_number` and all we do is `return $order number;` what functionality is added here? sorry I might be missing something


@unfulvio it doesn't look like we use these two `is_wc_version_gte_2_*()` methods. Do you think it's safe to remove them?


ah I get it... `$order->order_number` is the raw one?


the filter passes the order ID (`$this->id`). We return the `_order_number` meta we set [here](https://github.com/skyverge/woocommerce-sequential-order-numbers/blob/master/woocommerce-sequential-order-numbers.php#L168-L193). It works exactly the same way as the Pro version


No, if ResultSet variable declared inside the `try (...) { }`


Sounds legit.


Only security issues are backported to Django 1.8 since Django 1.9 was released.


This stuff all got refactored in 0.12. Not an objection, just an FYI.


done, Thanks! I remember us going over that.


what the fuck are you doing here???????? >.<


fdgdfg


Yes, the first `AddInterceptor` on line [26](https://github.com/luboshl/trackable-entities/commit/a3c80766bd755d208c41b3ed32ba474bcd41babc#diff-3649edc0bdc8c231bc1a934890587674R26) calls the second `AddInterceptor` on line [37](https://github.com/luboshl/trackable-entities/commit/a3c80766bd755d208c41b3ed32ba474bcd41babc#diff-3649edc0bdc8c231bc1a934890587674R37).  That's a good thing. :smile:

This matches the behavior of `AddStateChangeInterceptor` on line [48](https://github.com/luboshl/trackable-entities/commit/a3c80766bd755d208c41b3ed32ba474bcd41babc#diff-3649edc0bdc8c231bc1a934890587674R48) which calls `AddStateChangeInterceptor` on line [61](https://github.com/luboshl/trackable-entities/commit/a3c80766bd755d208c41b3ed32ba474bcd41babc#diff-3649edc0bdc8c231bc1a934890587674R61).