I would prefer that we keep the versions in the folder name actually..

* it makes it easier to track changes made to an individual release. If you're looking at git log on the folder you won't see anything other than changes made to that particular release.
* it makes us less likely to pollute a new version with files from the previous version
* it allows two different versions to live alongside each other


https://github.com/hmy2001/PocketMine-0.13.x <-- CREDIT GOES TO THEM


thanks


Please don't add Redstone stuff :) It would mess up the project.


threat < 10 that's always isn't it >.<


Yes, because `findObjects` [removes the `'error'` event listener before emitting `'end'`](https://github.com/joyent/node-moray/blob/master/lib/objects.js#L199), and [vice-versa](https://github.com/joyent/node-moray/blob/master/lib/objects.js#L206). It also seems like this is the idiom used in tests and code samples for node-moray.


This is already handled by the method call below (`with(Iterable)`).


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 42](https://teamcity.labkey.org/viewLog.html?buildId=286653&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 7, passed: 40 Build time: 01:15:18
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

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

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.tests.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

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

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

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

```


did you mean from the beta here?


Sounds good, will fix, thank you!


Don't print things in the normal flow of the application.


Also, a meta comment -- ideally content like this could be written in Markdown (so they render well on GitHub), a text file or CSV.

ODS requires you to install a specific app.


What are you doing here exactly?


Not sure about type of crtc_gamma, I was getting uncignedshort and ushort cast errors, so I made it void.

On November 20, 2015 5:12:39 PM CST, Tristan Rice <notifications@github.com> wrote:
>I take it under BSD crtc_gamma.{red,green,blue} is a `void*`?
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/howeyc/go-sct/commit/72cfa6324a46cebbd001c952b035c125bb5d3781#commitcomment-14539013


@legoboy0215 How? Its in 0.13.0


Absolutely, good catch. I'm too used to name by async callbacks `'next'` :)


Done, sorry @alloy, and thanks for letting me know


d


The return value of this is constant, so there's no need to regenerate this array over and over again. You could even just hardcode a big array literal for this in a static class property to avoid having to do any looping whatsoever.


Absolutely, will fix.


You perhaps need to protect `languages` to make sure it isn't updated/accessed concurrently without race protection? (it is always used in a separate goroutine).


Why did you move this outside of development?


thanks


I don't think it is very efficient to copy the whole vector as return in this and other functions. There are several alternatives with pros and cons that we can discuss


@bloodmc What about players stepping on pressure plates and tripwires? Or is it not affected by this?


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


Looks good. Maybe now is a good time to actually use cpp files and to not implement everything in the headers...


Also pushed a new release with your changes :rocket:


Nice catch, merged - thanks for PR! :+1:


Since the jucer files are in the repo, maybe just remove the XCode project files altogether.


kk, I will use this in the (Selection, Selectable)` case also


Sweet, thanks!


What happened here?


Revised Vegetable oil to 3/4 cup


Habits are difficult to break ;-) I know that switching between C and GO is challenging for that reason.

Remember to run these command from your root directory before doing a commit:
go fmt ./...
go vet ./...


Found this while tweaking gyp settings when I removed `-Wno-error=unused-parameter`.

@zugaldia @tobrun @bleege


Ok, just a reminder that the clean mode be documented and the automatic option removed.


Why are all these overrides needed?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.806](http://teamcity.monogame.net/viewLog.html?buildId=11519&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 552, ignored: 6 Build time: 00:21:07


` RealTensor?=nil` -> ` RealTensor? = nil`


why are these necessary?


Yep, I'm discovering they are.  ;)  Thanks.

On Fri, Nov 20, 2015 at 4:25 PM, Ivan Kozlovic <notifications@github.com>
wrote:

> Habits are difficult to break ;-) I know that switching between C and GO
> is challenging for that reason.
>
> Remember to run these command from your root directory before doing a
> commit:
> go fmt ./...
> go vet ./...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nats-io/gnatsd/commit/c56b3677da1c94ff7c49c05ec7168338b1409fa6#commitcomment-14539202>
> .
>

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


Is this right? Shouldn't this be `('peers',)`? Does the interpreter allow both forms?


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


Just a couple of comments:

The RTG defaults for memory (90%) and cores (all available) assume fairly exclusive access to the node (and I would think they are fine if that is true). For vcfeval you might find that 2g is not enough on large datasets or on nodes with many cores (since memory use will be proportional to number of threads), where you may need to either give it a bit more memory or use --threads to reduce parallelism.

For the ROC score field, it does seem a shame not to use the best score available that different callers provide as there really can be a huge difference in ROC quality between the best recommended score and a lowest-common-denominator attribute. E.g. I would default to the best available score (RTG=AVR/GATK=VQSLOD/MuTect=t_lod_fstar, otherwise GQ, otherwise QUAL, otherwise DP.)


Index update?


There are some backends that depended on the old behavior of the predicate not being an input and I was not sure how to properly implement the fix for the backend. I tried with all of the backends I know about. I sort of see this as a shim until we can properly fix this in the backends that are using this. I think this is used by spark, mongo, and bcolz. For bcolz, the real fix imo is to use dask.array


@btm I created an issue thanks.

https://github.com/chef/chef/issues/4194


That "vertical datum" part implies that no 2D SRS can appear. For example, EPSG:4326 is not a valid spatial reference system for an elevation data set.

The implication of that is difficulty in automatically converting existing data (e.g. if I give you EPSG:27200 LIDAR, how do you get it into GeoPackage?).


Basically I am drawing the piano keys first and then based on the detected quantised pitch deciding which key to highlight. For doing that I am trying to find the note octave and the pitchclass of the detected pitch and accordingly selecting which key to highlight. This is definitely a messy may to do this, but I thought let me first get it working before trying out better ways to do this. Having separate rectangle objects for the different keys might be a better option.


I'm not sure when I can solve it. It's on my stack of stuff.


Your right 2D top view for now sorry, you can kill foes as well knocking them out bounds might take more HP? Then have them pushed back in at less HP before they die?


This whole logic block is redundant, already implemented in disarm_if_autoland_complete(). What is the purpose of it here?


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.553.31bbb247821df5a39af22f229e4ccb9590130581](http://build.palaso.org/viewLog.html?buildId=66490&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 931, ignored: 56 Build time: 00:22:40


#391 Check inputstream EOF


good point, fix pushed


This commit reduces the amount of node_modules dependencies from >180MB to ~50MB. It removes karma, jasmine, eslint; to leave only the required code to have a watch and transpilation of JavaScript.


don't forget to remove all of these new k_param_ values, they're handled in /AP_Land now. Also, make sure to make a comment of "depreciated" on the ones being moved to AP_Land


I think to test what you were testing here before, you'd have to do something like:

    for i in sorted(values):
        assert heap.pop(int_heap) == i
        verify(int_heap, 0)


@KillAshley you realize this is all your fault, right?


Are you sure ? that doesn't seem right at all, it should probably return the empty list.


This needs to be changed


Not sure why you're duplicating these tests, but sure. I'm not gonna complain about more tests. :)


I don't know how this happend. But the number reported by `duplicate-remover` is different from `wc -l` by 1.


dfg sdfg sdfgs sdfg sdfg sdfgsdfgsdfgsdgsd gsdfg sdgsdfgsdfgdfsg


Test


Thanks for fixing the code, that Joan DID NOT break


this failsafe behavior change from using RTL instead of CIRCLE doesn't belong in this PR.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.137.31bbb247821df5a39af22f229e4ccb9590130581](http://build.palaso.org/viewLog.html?buildId=66489&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 943, ignored: 50 Build time: 00:34:52


same here


nice! :1:


:sweat:


5258-Plant-HX-Dual-DeadBand-Control (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1434 of 1434 tests passed)

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


I see your form muahaha >:)!


Fix that right now.


Is this a correction on a *very new* change ?
I see why the old (without correction) wouldn't work, but  I fail to see how the corrected version could work, because the above doesn't touch the Ispresent variable.
It could work without the Ispresent variable, by  adding return 1 at the end.
The advantage is one local less and a nice speed up because as soon as the first  option is found, the function doesn't have continue : return 0 stops the function.

In my download from today (21-11-2015), this function is a bit different, but is working correct:
local isPresent=1
for option in "$@"; do
      for option2 in $option; do
            if echo "$FFMPEG_OPTS" | grep -q -E -e "$option2"; then
                isPresent=0
            fi
     done
done
return $isPresent


yeah ulitimately i think this backend has low value


The problem is that I'm currently using paste, both as a simple 1:1 putting one pixel on top of another with the consideration of alpha values, and I'm using paste with the capability of resizing the incoming image (for mountains). We could possibly get rid of the resizing, but that would require all mountains to be the same size (although we could connect it to some of the work being done on mountains right now so that we have multiple size mountain symbols for tall mountains, regular mountains, and hills).

I'm also making use of the blur function to give us the blue coastal outlines and giving some consideration towards using a much lighter version to help remove aliasing issues with the coastline itself. Finally, I may end up using it to make rivers look a little nicer by having rivers occupy multiple pixels. The edges of the river would be black and then the interior pixels would blend from an edge color to a different color for the core of the river.

There's also a very real possibility in the future of taking advantage of the path feature (which may be a pillow feature instead of straight PIL). This would enable rivers and coastlines to be drawn a bit cleaner at higher magnifications.

So I'm not entirely sure what the path is here. I suppose we could just write our own versions of all of those functions. The two other options would seem to be to re-incorporate PIL as a requirement or to make the new map routines a plugin. As a plugin it would still require PIL to be installed but Worldengine itself would be free from the requirement of PIL.


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


note, on your branch you removed dist and added C where on my rebase merge I added your C value. You may want to check that this fits in the buffer and the text doesn't get clipped. However, the correction may change right here because we're a bit point-blank to the ground. Printing the correction just before may be a more reliable place.


whitespace


it is not required.


There's something off/sloppy about the candidates function and variable naming.


sorry to hear that, bro.


Yay!


TeamCity pull requests :: netty [Build 709](https://garage.netty.io/teamcity/viewLog.html?buildId=3461&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 5708, ignored: 21; compilation error: Compiler Build time: 00:42:20


remove commented code, that's what version control is for! :)


make multiple import


added comments


Adds an element (of this class) to the stack trace which is used to locate the caller inside of the stack trace


What?


Done


do_checkForOptions looks if any option exists, not all of them, so it's useless to check for any further.
I was planning on adding syntax to `return 1` if an option was found (like a negative) but it wouldn't be needed for much so `! do_checkForOptions` is good enough.
The additional variable shouldn't matter that much since it's a local that should be gone after the function ends.


WTF glitchhub


is there a reason to have this `req` shadow the `prototype.req` that this is defining? .. makes it harder to read


Done.


Oh yeah, I think currently I has some good speed improvements over the original routines because I made the symbol drawing routines occur in a single pass (the routine would look at a square and decide what symbol to put there rather than iterating through the entire map once for coniferous forests, then again for mixed forests, etc.).

I lost some of that speed advantage when I made the rivers carve out parts of the masks so I'm not sure where I currently sit. However all of these numbers are extremely 'soft' because before release I will probably undergo a couple of more big organizational changes. This will allow me to check the symbols and textures through iteration (does this spot allow a symbol from the first set? Does it allow symbols from the second set? Etc.) instead of explicitly checking the different groups (Coniferous forest? Mixed Forest? Deciduous Forest? Etc.).


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


Actually `ImmutableList.copyOf` handles that already because they don't allow null elements. :P


So some issues: Can't actually add a song from the modal yet.

There's still some weird stylistic issues when the queue is short - whatever container the app is in doesn't want to color the whole page (there's a section at the bottom of the page that remains white).

The add song itself isn't actually sticky to the bottom of the viewport like originally requested.

The spacing is slightly weird due to the fact we're using the history sidebar as a pusher instead of designing content around it.

There's probably some excess CSS somewhere.


or maybe no HP, just negative effects (slowing down, chaotic movement) that increase chances for failure.
Then, the only way to win and be defeated is pushing off.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6157](https://teamcity.mesosphere.io/viewLog.html?buildId=78923&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:09


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6158](https://teamcity.mesosphere.io/viewLog.html?buildId=78924&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:08


NOTE: Also needs changes to alexa-utterances to change the grouping characters from {} to (). The curly braces are used to specify the custom slot name as in the following:

    "(change|switch) (level to|to level|to) {Level} (|problems|questions|challenges)"


非常感谢。有兴趣聊聊吗？记得给我回复邮件。


I think there is no need to iterate through all squares. If checking piece is sliding piece (you may need to store checking piece square in position state) you need to iterate through the absolutepin's bitboard and look what sliding piece can move to that squares using attackTo functions.


This is really great! Do you need me to do anything here?


How I feel about OTs.
1 = OCG
2 = TCG
3 = ALL (OCG+TCG+Korean+XCG)
4 = Anime
5 = Korean
6 = Beta (Experimental cards, not printed IRL, prize cards.helps keep track of outdated card IDs)
7 = CCG(Custom cards broken, overpowered
8 = XCG(Custom Cards in line with the meta aka cards Konami would make.)


@spencergibb This was dumb on my part. CONNECTION_MANAGER_TIMER must be an instance variable for DirtiesContext to cleanup properly.


I've been waiting this for 6 years


You forgot to include <ctime> :(


@freedomofpress/employees Apologies for the direct push.


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 43](https://teamcity.labkey.org/viewLog.html?buildId=286660&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 17, passed: 31 Build time: 01:18:55
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.tests.CDSLoginTest.testSessionTimeoutIntercept: <no details avaliable>

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

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

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

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

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

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

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

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

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

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


:+1: lgtm


thanks


OK, I await your changes.


You're right because it was a written interview. What's the preferred way I fix this?


I did not realize that alexa-utterances was by a different author. I'll create a pull request there. If necessary, I can get you the diff to apply to your code.


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


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


[![Celebration of Spring](http://api.artsmia.org/images/123287/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123287)


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


[![View of Lake LÃ©man at Nyon](http://api.artsmia.org/images/1545/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1545)


The "circle" snap indicator seems deformed.

A small square has been requested as a snap indicator shape type,
and I imagine an alternative in "diamond" orientation.


@D3thw0lf Thanks


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


The thing is you will use `Void` when you explicitly `return undefined` and also when you don't return anything. Your description only covers the second case.


For right now if we can make a shape for the Board, and get two circles to fight with their attributes i.e skills mp , and hp that would be a good start.


I mean comma, not common


Preferi deixar como estava uma vez que não tenho certeza.


This test does not work ...


Why is git MAKING you delete stuff? (no pun intended)


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


This is wonderful. How about the captilization on the OBO homepage? Maybe it will take a longer time for the change to be displayed?


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


Now to piss you off even more. In retail party members of invisible players can see them :D


Very good :D


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


fix the limb ribbon flip issue when ik handle move


Here's some profiler output, for reference:

```
Timer unit: 1e-06 s

Total time: 0.860113 s
File: /Users/roger/Development/sgdf/sgdf/fusion/alternatives/reference/__init__.py
Function: poisson_blend at line 151

Line #      Hits         Time  Per Hit   % Time  Line Contents
==============================================================
   151                                               def poisson_blend(self, source, mask, tinyt, t, tinyt_topleft, maximum=False):
   152                                                   """
   153                                                   Modifies the target image to blend the source image at the designated mask pixels using a
   154                                                   Poisson blend.
   155
   156                                                   Args:
   157                                                       source (2D float np.array): Values of source image at relevant blending pixels;
   158                                                               same size as mask array (may be smaller than source image)
   159                                                       mask (2D bool np.array): Mask with 1 (True) values for source pixels
   160                                                       tinyt (2D float np.array): Values of target image at relevant blending pixels;
   161                                                               same size as mask array (may be smaller than target image)
   162                                                       t (2D float np.array): Full size target image
   163                                                       tinyt_topleft ((int, int)): (row, col) designating top-left coordinate of tinyt
   164                                                               with respect to image t
   165
   166                                                   Returns:
   167                                                       (2D float np.array): Channel of modified target image with section of source image
   168
   169                                                   """
   170         3         1439    479.7      0.2          mask_inside = self.inside(mask)
   171         3           15      5.0      0.0          mask_border = mask & ~mask_inside
   172
   173         3        31900  10633.3      3.7          A4 = self.construct_A4(source)
   174         3          908    302.7      0.1          t_prime = A4.dot(tinyt.ravel())
   175         3          972    324.0      0.1          s_prime = A4.dot(source.ravel())
   176
   177         3          195     65.0      0.0          b = t_prime.copy()
   178         3            3      1.0      0.0          if maximum:
   179                                                       max_prime = np.maximum(s_prime, t_prime)
   180                                                       b = self.set_b(b, mask_inside.ravel(), max_prime)
   181                                                   else:
   182         3          242     80.7      0.0              b = self.set_b(b, mask_inside.ravel(), s_prime)
   183         3          276     92.0      0.0          tinyt_values = np.concatenate([tinyt.ravel(), tinyt.ravel(), tinyt.ravel(), tinyt.ravel()])
   184         3          178     59.3      0.0          b = self.set_b(b, mask_border.ravel(), tinyt_values)
   185
   186         3        39328  13109.3      4.6          A4 = self.construct_A4(source, mask_border=mask_border)
   187         3            6      2.0      0.0          imh, imw = source.shape
   188         3       784566 261522.0     91.2          v = lsqr(A4, b)[0]
   189         3           12      4.0      0.0          out = v.reshape((imh, imw))
   190
   191         3            2      0.7      0.0          tttly, tttlx = tinyt_topleft
   192         3            3      1.0      0.0          tty, ttx = tinyt.shape
   193
   194         3           65     21.7      0.0          t[tttly:tttly + tty, tttlx:tttlx + ttx] = out
   195
   196         3            3      1.0      0.0          return t
```

@sarahjkim @gilbertghang @jasonzhang4628 @quinnromanek


done


MArray_Elim (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (910 of 910 tests passed)

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


Thanks


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


Na verdade esse fica melhor:
Sou muito preguiçoso para pensar.
Vou colocar assim.


Once on the Web, always on the Web.............


Mantendo level como comentei anteriormente.


Done


conf没有实现序列化，当初把conf设成全局变量，导致网络传输的字节流无法识别，正确的写法是 定义局部变量，用单例模式访问


Como falei, do primeiro arquivo antes do icbt 1.
Não imaginei que tivesse mudado tanto.


markdown语法还有些问题，我再修改一下


closes ArthurPBressan/TCCCidadeIluminada#15


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


Pretty sure this bug was my magnum opus.


fixes CidadeIluminada/TCCCidadeIluminada#15


Move atfer line 31.


建議補上 example 檔案


5321-GroundDomain_RDD (mmatts) - Win64-Windows-7-VisualStudio-12: OK (1963 of 1968 tests passed)

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


Fixes #169.


Are you using the most recent version of the JS compiler? For me the above does not interpret the null as a string.

Sent from my iPhone

> On Nov 19, 2015, at 4:56 PM, timwu <notifications@github.com> wrote:
>
> Seems like this might be revealing another bit of weirdness. So if you do a simple:
>
> a = null;
> In planout DSL, the compiler turns it into:
>
> {
>   "op": "seq",
>   "seq": [
>     {
>       "op": "set",
>       "var": "a",
>       "value": {
>         "op": "get",
>         "var": "null"
>       }
>     }
>   ]
> }
> Which seems to imply that null (and similarly true and false) are just pseudo-variables. When evaluating the interpreted code with planout-golang, I get an exception about not having an input variable named null.
>
> I could be missing something, but it looks like we'll need some builtin variables to make this work.
>
> —
> Reply to this email directly or view it on GitHub.
>


done


@sbeaver-netgate  "Sysytem" is a typo. But in any case I think it should be "Status" anyway?


s/return RUNNING/return  vm->state/


後來不是改用 0.3.x 的 mysql2 了？這邊看起來還是 0.4.x


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


JavaPairRDD有bug，不懂怎么使用lookup函数


s/return HALTED/return vm->state/


s/return UNDEFINED/return vm->state/


idk just wanted to compile the build


Este deve ficar como itens mesmo pois isso é o que aparece na hora de configurar o grupo.
Quando vc for escolher a opção de divisão de itens fica melhor Distribuição de itens do que espólio.


Deixe como Livre para todos.


Yup


Ok


And I don't see why that comma after summary has been there. It is not needed in either the old or new version of the sentence.


Merely unaware of it. I'll do that.


Fewer characters? Not a great reason…


不使用 V8 應該也可以正常執行，有什麼原因導致需要安裝這個 gem 嗎？


Um membro por vez fica bom


Hm, from 0.0.1? It makes sense, but it seems like we can't have been doing that before?


done


I see it in https://github.com/coreos/baselayout/pull/41/files ?


On Fri, 20 Nov 2015, mli2830 wrote:

> Why is git MAKING you delete stuff? (no pun intended)

Hee hee. Not sure. I guess I deleted stuff on my directory that was still in the repo. It won't sync until I decide what to do about that. But I'm not sure why the problem comes up now, nor what I really want to do with those files.


Ugh good point.


git mv the 'coreos-0.0.1-r263.ebuild' symlink


```javascript
callback(null, pgdata.rowCount === 0 ? null : pgdata.rows[0])
```


Hmm, I'm using the one found here: http://facebook.github.io/planout/demo/planout-compiler.html and our in-house compiler.

Could be out of date?


my bad, carry on.


I just found an alternative without manually installing:
https://github.com/winkelsdorf/homebrew-custom/blob/master/Casks/jdownloader2.rb


You could do separate commits, you know...  Anyway, I saw several kangs, and only with my commits (you cherry-pick from others without problems), that's why I asked.

https://github.com/AdrianoMartins/device_lge_v500/commit/6c551c4cb8694d91920850fbeae886c92bbdda37
https://github.com/AdrianoMartins/device_lge_v500/commit/e9035c4acb83d3dda34e9a1f3e39c56056cd6ec4


good job


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


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


Remove this line.


if (vm == NULL) {
    return UNDEFINED;
}


I could not see a preview of a white image with background transparency in the preview area. Adding a background to this area helps make it more visible.


https://facebook.github.io/react/docs/events.html
需要说明一下调用 `persist` 的用意是什么?


done


Note, to use the new commits, simulations will need to be restarted


Deixei apenas deslogar.


Add "#define VM_CREATE_FAILED      255"  // We allow the number of VM is to be 254.


if (vm == NULL) {
    return VM_CREATE_FAILED;
}


状态会触发界面渲染, 这里为了保证组件的性能, 特别是这个数据不会渲染到 DOM 当中属于私有的状态数据, 建议用 `@_delayY` 的私有变量形式记录.


103643052-FiniteDifferenceFluxOutput (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1960 of 1969 tests passed)

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


@jef-n , that was quick, thanks. BTW, the strpos function help text needs updating too.


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

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

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

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

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

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

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

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

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

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

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

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


LGTM！


We can have it at 100 for first int and 102 for second int and place the settings offset at 105? Or what's a better way to do it? If you guys give me a lil heads up, I'll get it done!


hi,

System.exec() / fork()
+
Process.killProcess()

this causes zombie process. (You should call wait() in parent process.)

I suggest to use Runtime.getRuntime().exec() to replace System.exec().


s/struct vmcb *vm/struct vmcb *vm = NULL/


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


s/return NULL/return vm/


Ok, I have local setup again and everything should be smooth sailing now.

Please let me know if anyone has trouble with the new install script. As far as I can see it is working perfect on my end.


need to check for allocation failure and set enable to 0 if alloc fails (plus print a warning)


Done


the thread radius and list length should be parameters (that can be a future PR)


done


this should take a const ref and be a const fn


@flavour The inline component with the image did not worked here (does it work?), so I created a field "image" here but the image is not uploaded into the images folder?
Also is it possible to get the mime_type and size of the file during upload?


Trocado para jornal do aventureiro.


*Sigh*. This isn't a permanent solution.


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


Mas tinha ficado tão engraçado... Brinks
Erro meu tbm, corrigido.


Nice, I had the precedence backwards and was wondering about this line. Thanks for the extra clarity.


These aren't really necessary since they're mapped by default anyway.


104921238_EnergyPlus_Performance_Framework (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (894 of 894 tests passed)

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


dude clearly you're wrong how can you not see it


Thanks. Unfortunately i don't think isatty is enough because there are lots of other cases where stdin could be unable to seek (e.g. a pipe). I wrote a patch that catches the exceptions.


实现了断点续传&&下载过程中的进度展示


*IMarkdownOptions* is the interface that was included in the Nuget build of MarkdownSharp. Here's my not so good attempt of doing it (based off the metadata definitions in the Nuget version)


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


Those from devices did not come from your repository, there are different codes:

https://github.com/awifi-dev/android_device_lge_awifi/commit/220f4f71a934bc4d6b2b5d5454ddbe86232a468b
https://github.com/AdrianoMartins/device_lge_v500/commit/e9035c4acb83d3dda34e9a1f3e39c56056cd6ec4

Coincidentally have the same name. But a lot of people is applying these settings, as they are common to marshmallow.

Anyway, I will attend to leave as cherry-pick if I use something your in the future.


@hzhr How to reproduce this? I see no zombie process.


This really confuses me actually, as the metadata definitions for the Nuget build say that it's only ``get;`` in there only. I know there's more than that, but since the only lead is the metadata, we don't know exactly what's going on here.


should this be user = nil ?


you don't need user = true


@kevinschaul Are you able to remove my email address and ideally purge it from the commit history with a rebase & force push?


Wow, I didn't realize you were still working on this project. If I gave you my email would you mind chatting over there?


+ First goals
- One geometric level, one character, one foe
- Character and foe can just be circles from 1D top view that fight
- objective stay in bounds


John Bickar likes this


I meant Gradle 2.8, not 1.8


misc_standard_report_measure_issues (DavidGoldwasser) - i386-Windows-7-VisualStudio-12: OK (2335 of 2371 tests passed)

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


merged as 3e0fa493697b29a51ec3d4a5757667ac6177aab8


Shoot. The precise behavior of these `platform` keys, especially in the presence of subspecs, doesn't seem terribly well documented. I had a difficult time even getting it this far.

I'll poke around a bit more and see if I can come up with right incantation to make everybody happy.


I thought we should first decide on how we want to implement the program. This file contains a getRank function based on what I think might be efficient for our program. Check it out and include your suggestions.


Oh really? Oops...


Oops...


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


NO


Committed to my branch code for geomag70.c which deals with different line endings, both in the input file and the coefficient file

9e2e9ff024c8f7c5c2fb009773226259c6229a3f


Thank you for advice!
I've integrated Parallax with Travic CI, just tests are needed :)


Try recompiling...


ok


Yes.
With use the NSIS Unicode version, it work fine on ascii or unicode path.


MArray_Elim (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1448 of 1448 tests passed)

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


Nit, this could move to the `if` block.


I did. Fixed :)


@ml019 could you give me some data to replicate this issue as I didn't encounter any issues while testing the previous change, thanks


Yeah I will do that now


MArray_Elim (DeadParrot) - x86_64-MacOS-10.9-clang: OK (1969 of 1978 tests passed)

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


I think you mispel `proto` to `porto`, cause `porto` is not using anywhere in this file :dancer:


Blue Alliance Only


hi.R under development meaning still on your computer and not ready to push to the repo? Or you forgot to push to repo?


Hi @derekcollison , Thanks a lot. does the present nats supports two-way authentication?

Thanks & Regards,
SriniCH


saw approval from ksauzz
at https://github.com/basho/riak_cs/commit/5f4e402d57e864d8ab76118901645d50b9ba3539


merging basho/riak_cs/support/inspector-for-mp = 5f4e402d into borshop-integration-1272-support/inspector-for-mp


basho/riak_cs/support/inspector-for-mp = 5f4e402d merged ok, testing candidate = fc382c9c


yes I agree :dancer: It was 3 AM so I can't really make a difference though I was going to name it p :D and changed it to proto lool


103643052-FiniteDifferenceFluxOutput (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1960 of 1969 tests passed)

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


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


ここからかな


Hello. Right now, I really don't know what the hell I'm doing. It's time to experiment...trial by fire.


@nicogago en  vez de andar duplicando esos for que haces para mandarle la novedad a todas las entidades seleccionadas defini un método!!!


특수문자


Make current figure the axis if one is not passed in.


oooo noice :+1:


This is an ugly way of doing it. If there's a way to do it without cell arrays, it would be nice to refactor this.


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


Should this be assigned to anything?


Commenting is a little sparse. If there are parts that are hard to understand, let me know.


See 2371877 for an example of how to link to another route. We may as well stub out the actual links, to save time later.


=>使わない書き方に記法整えましょう！


I upgraded today and our custom installer stopped working.

I custom installer just changes the package base path so we install outside of `vendor`:

```
    public function getPackageBasePath(PackageInterface $package)
    {
        $packageName = $package->getPrettyName();

        $prefix = substr($packageName, 0, strlen(self::DEFAULT_PREFIX));

        if (self::DEFAULT_PREFIX !== $prefix) {
            throw new \InvalidArgumentException(
                'Unable to install our package, packages '
                . 'should always start their package name with '
                . self::DEFAULT_PREFIX
            );
        }

        // Strip default prefix
        return self::DEFAULT_INSTALL_PATH . '/' . substr($packageName, strlen(self::DEFAULT_PREFIX));
    }
```

My last working version is 2015-11-18_18-53-33-f5e0fc1

Thanks.


ここも同様


OK. Thank you for your review.


Add 'pip install psycogreen' for long polling correction.


@taylorotwell I was looking into this too after upgrading to MySQL 5.7 on Laravel 4.

Would you consider cherry picking the commit to the older branches?


シンボルを使う記法で


Eu coloquei duas ações no mesmo botão na classe FrmObraArtistaSelec somente pra teste. Ele realiza a ação do controller e da própria tela do Form. Outro problema a ser corrigido é quando for adicionar mais obras, utilizando a tela de seleção de obras, na JTable da tela exposição já com contéudo.


Yeah. I will consider this in the next week.


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


This is where I want to reduce the query...... 1 blog driver x 1 link - 122 loops. 2 links - 244 loops. 3 links - Server cry.


Oh bother, sorry, yes


Make `/* global -Promise */` the first line of the file so the linter passes.

https://travis-ci.org/coursereviews/schedule/builds/92190596


you can just omit the `href` to keep the help field from being a focusable element


Please remove this line and extra empty lines.


Same issue as above. Unused variable fails the build.

https://travis-ci.org/coursereviews/schedule/builds/92190596


What is the point of this? Options:

- if `request.args.getlist` returns `None` when there is no argument, then this is redundant and can be removed.
- if `request.args.getlist` return something else (e.g. empty list, 0, empty dict), then explicitly test against that, rather than implicit conversion to bool.


Question: When the input is larger than 16, Why does the output comes out as a negative number? (-n).
Integer overflow, maybe
1. https://msdn.microsoft.com/en-us/library/5kzh1b5w.aspx
2. https://en.wikipedia.org/wiki/Integer_overflow


what's the reason for making the footer buttons non-navigable via the tab key?


Setup Project


How about making `sample_id` an argument of the images function (and thus a required element of the URL)? Slightly different routing schema but I think it makes sense when

- there is a single argument
- it is always required


k


Make sure your IDE is respecting our indentation conventions. Should only be 2 spaces here.


You rock man!


The unreso9lved symbols you are seeing here are from libstdc++ and almost certainly from your version of `arm-none-eabi-g++` which is version `4.7.4` (from Ubuntu).

My main dev machine runs Debian and my `arm-none-eabi` is :

```
$ arm-none-eabi-g++ --version
arm-none-eabi-g++ (15:4.9.3+svn227297-1) 4.9.3 20150529 (prerelease)
```

which doesn't have this problem.


Also, see https://github.com/airbnb/javascript#7.1 — if you're using an unnamed function, use an arrow function instead.


You can give me apk file? Please


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1703


Can you benchmark this vs:

- `json` (part of Python stdlib)
- `ujson` (A much faster library)

We should pick the fastest, *perhaps* making `ujson` an optional dependency (its interface is identical to `json`'s).


I was wondering why my server memory goes up after few refresh..


:blush:


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


yessssss.   4ee0ee7673a8661204da60631b108c6ade5ad903


Had put the listening function in the wrong place. Sorry! it works now though


Made the database namespace more accurate so this still works with other drivers and namespaces in use.


@steveluscher Since this was warning previously, could it be upgraded into a `__DEV__` only check instead of being run on each `buildQuery`?


抱大腿！！！！


I would probably break this out into a function and just do a oneline register and context push, e.g.

```javascript
context.subscriptions.push(vscode.commands.registerTextEditorCommand("extension.rustfmt", formatCommand))
```

Related, I think the pattern would be `nameOfExtension.nameOfCommand` rather than `extension.nameOfExtension` based on https://github.com/Microsoft/vscode-go/blob/master/src/goMain.ts#L41 and similar MS repos


I'm not sure it can be done, in one case you are using AccessCoins and in another GetCoins.
Also most client behaves differently according to the fact they have or not all the heights. I'm not sure any function would save us lots of code.


You can probably leave this out, it would just get annoying imo


I also notice that in https://github.com/PX4/DriverFramework/pull/10 you have removed all use of the STL so that should fix this issue anyway.


If a child process exit normally, we definitely get zombie process. However, if we simply (not gracefully) kill the child process, we won't have zombie process.

But it's still a good point to use Java's API to execute binary.


if s.GlobalScope && v.Scope == ScopeSession, continue, else do following things may be simpler?


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


MArray_Elim (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1975 of 1984 tests passed)

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


Looks good! We may be able to simplify a bit by dropping the `inherited_association` check and extending over an empty object on L171:

    proto._associations = _.extend({}, proto._associations, associations);


Plz Review it @Ashish-Developer


Nit: s/useful/which is useful


This "if" seems entirely redundant given the other "if" above.


could you please test the newer version of test file . We had already moved them to new folder and all the tests are passing .


* Must make directory 'users' and 'users/images' in /tmp/.
* Create 'user_details.txt' file in 'users' with csv headings.
* 'recorder.sh' is stored in /cgi-bin, others in /htdocs.
* Set appropriate privileges so files work properly.


MArray_Elim (DeadParrot) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Added this line to URLs in order to get site_media/media/ to work as well as css.


Technically, an `attr_accessor` on an ActiveRecord was never supported, since it turns out that wasn't tested. However, see #822 where this is being added.


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


ARGH SORRY!


Looks good.


But the value of `XWALK_VERSION`, `XWALK_COMMANDLINE`, `XWALK_MODE` will   must be appended to `Cordova plugin add cordova-plugin-crosswalk-webview` if install the plugin on the older Cordova version that don't support the feature of `Default Variable`


Why was this change necessary?


<!--hello-->


OK James, here comes your code review. I'm going to leave a bunch of comments commit-by-commit, so here's hoping you don't have email notifications enabled!


I tend to think this patch should be dropped in favor of PR #32.


fix #20


why this file is not available ? Where to find the file ?


As before, I think I would favour making required arguments parameters of the `samples` function.


Though maybe you can talk me out of it. =)


I mean  modify the hooks as below
```
projectConfigurationFile = path.join(context.opts.projectRoot,
            'config.xml'),
```
```
projectConfigurationFile = path.join(androidPlatformDir,
            'res/xml/config.xml'),
```


Another thing I'm noticing in this schema is a lot of repetitive code, `find_query = {} ...`. It might be worth making a helper function that sanitises all queries.


It appears that you can't have colored phantom flashes. Could we have it so the Phantom Flash lens would accept colors, but still only render the stuff to monocled players?


Nice merge!


Thanks!


I'm helping!


This seems like a heavyweight solution... You can probably just create a `ValueError`, add in the status code, and then raise it...


Interesting...did you plan out this design before implementing `Board`, `View`, `Player`, and `Computer`?


Remember also Korean format has a different card pool our users have asked about it, also a pool for 'unreleased' which is OCG+TCG+unreleased, OCG experimental where its OCG+unreleased, TCG experimental where it is TCG+unreleased, Speed Duels an anime format which hampers card pool substantially; this requires a few more permutations than we are allotted.

For the __majority of players__ using ygopro they are doing so via a server service where the DB is locked to that specific server and the client only has one DB. On some servers the server can switch DB's but effectively the user can not, or not efficiently so they only get 1 compiled database.

@Steeldarkeagel @Percival18 @ChibiChanNya @Stormwulf @Moonlightsorrow

Summer 2014 @matteoserva measured the user base of YGOPro at around ~40,000 users as a conservative estimate. Asking them all to agree on one custom definition is asking a lot. Players usually can not or do not communicate before a duel in a server environment.

There is use case for having this number be higher and more along the lines of a setcode in its implementation.


Thanks! Not sure if there is some syntax to follow here, but could include a link to my GtiHub profile or website (http://andrz.me), or at least my middle initial so people can find me?


Yikes, yes, that demo is way out of date.  Do you know if this is something
that is still linked to in any of the documentation?    See
http://planout-editor.herokuapp.com/ instead.

I would suggest using the JS-based compiler if possible, as we have made a
number of very important fixes to the compiler over the past year, see:
  https://github.com/facebook/planout/commits/master/compiler/planout.jison

e

On Thu, Nov 19, 2015 at 7:17 PM, timwu <notifications@github.com> wrote:

> Hmm, I'm using the one found here:
> http://facebook.github.io/planout/demo/planout-compiler.html and our
> in-house compiler.
>
> Could be out of date?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/URXtech/planout-golang/commit/749972de713f435b2c64112653db66c51d12351e#commitcomment-14517841>
> .
>


Why is it the `View`s responsibility to set up players?


If you do end up keeping the custom exception, I think this would be more Pythonic:

```
def __init__(self, message):
    super(self).__init__(message)
    self.status_code = 400
```
The nice thing about this is that it respects single point of control. If you decide to change what `InvalidParams` inherits from, you will only have to change it in one place. (Also, it's only Python 3 compatible. =P)


That is, is there another concept that you could extract?


This probably isn't a good name for the exploration YAML file. The name doesn't seem to match the other explorations in this list. Those are named after what they teach or demonstrate.


h哈哈哈哈


What are the benefits and drawbacks of storing this as a mutable instance variable? Are there any alternatives?


bootstrap のdropdownを使いました。


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


Git-huh? :D This fixes it, thanks.


104921238_EnergyPlus_Performance_Framework (lefticus) - Win64-Windows-7-VisualStudio-12: OK (1968 of 1968 tests passed)

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


The Low ID DBScripts are Generic Scripts and will be used eventually. They are FALSE errors and these scripts belong here.

Might convince Core team to make exception for ID range for these.


Не, не нужно так использовать switch, эта конструкция для упрощения, а не для усложнения :-)
Я не хочу домысливать какой из case-ов сработает, если case 1 и 2 возвращают true. Switch для перебора однотипных значений. Представь, что тебе придется этот код поддерживать и добавлять еще условия в case :-)

Несколько последовательных if тут — ОК.


ok


あ


Please add more tests thoroughly testing all aspects of reader.classify (including hard and soft rules). It would be nice to test the edge cases of classify itself and make sure it properly uses hard/soft/classifier rules. For instance, the reader.classify contract guarantees that in the event of a tie, a hard rule will win over the other two. A fuzzy rule will win over the classifier. We should defend this in tests.

Secondly, I don't think this name is quite right. It's actually testing classification in general (since this is from the perspective of the reader). We have an entire test suite for the string classifier. Perhaps 'test_soft_classification' or something of that nature.

Finally, all the new tests I'm recommending you add should probably go in their own test class which will contain all the tests for reader.classify.

Please let me know if any of the above is not clear. Thanks!


Errr ..... The `.travis.yml` I added should work and should also support codecov once it's enabled........


104921238_EnergyPlus_Performance_Framework (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1432 of 1432 tests passed)

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


Perhaps soft rules should be tested in another test. I'm not very picky about this though and will leave the decision  up to you (beyond what I mentioned above).


:flushed: Oops, forgot to run the linter before committing.


I don't think it's ever about the number of lines of code. It's all about organization and clarity especially in Python.


Can you explain all this config-foo in a comment or, better, in a module docstring?


El simulador en el que podemos probar estos ejemplos que acabo de subir, esta en: http://www.futureworkss.com/tecnologicos/electronica/softwaresimuladordigital.htm
En la siguiente reunión igual explicare este commit.


This commit also allows many accounts to link to the same oauth accounts


I like the "design by wishful thinking" thing going on here, but this seems too complex. Read this back to yourself: does it read like English? Code that is as simple as it can be usually does.

I see three concepts intertwined here: 1) prompting for a player, 2) human and computer players, 3) "X" and "O" player markers. Can you extract and hide some of these details? I'd suggest that only players need to know about markers, and that it might be better if the view were the only thing that knew about players. In that case, this might look more like:

```rb
@player1 = View.get_player_type
@player2 = View.get_player_type
```

or something.


switch -> if


This stretches my Flask knowledge. What's happening here?


x


2015/11/20 create a branch.


Also use round shape icon for the google plus icon. It looks better that way.


this is typically one word.


Here's another detail that can be hidden behind an abstraction. Players that do the same thing `choose_move` but accomplish it differently (`View.get_move` vs. `Computer.get_move`) are an opportunity for polymorphism.


You're right, I had to read the thread till the end ;)


Should be convert?


Incorporates
Added a option to show GUID #332


El documento de requisitos de usuario y de software actualizado hasta el día de hoy, 19/11/2015 y revisado por el cliente.


there is no need.


> `contentView.removeConstraints(contentView.constraints)` ではダメなんでしょうか？

何度か上記の方法や他の方法も試してみたんですが、対象となる制約の削除は出来ませんでした。


OK,　add　the author


OK


l


Something to think about: are `game_over?` and `valid_move?` concepts that belong on the board, or should they go somewhere else?


MArray_Elim (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (910 of 910 tests passed)

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


Might be more readable to use enums/constants for the `algoType`


Having only 4 possible OTs is quite limiting. But I don't think that many more are needed, or combinations.

 5 for customs and 6 for unreleased cards or other testing purposes would be a huge help. Also I believe Anime should allow TCG and OCG. At the moment, it only allows OTs 3 and 4, so no exclusives whatsoever. These proposed Custom and Unreleased modes should allow all TCG, OCG and TCG/OCG cards since they are clearly not meant for competitive duels; no harm in having the widest card pool availible.


Great work!


I'm comfortable with dropping 2.7 support. This comes with many advantages, such as not having to specify the class name when calling `super()`.


And I'm *very* comfortable dropping 2.4 support! =P


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6020](https://teamcity.mesosphere.io/viewLog.html?buildId=78502&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:13:20
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


@albertshau Thanks for the response. Yes, I do see the addFlowet() calls in the configure() method. Please note that I have not modified any script, but just trying to reproduce the netlens app as downloaded from github here. Any advise on this error is helpful. Thanks.


Done.


@sneal @sethvargo Thanks for your positive feedbacks, I'll try to add a unit test today, and then open the PR :-)

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

@sneal I guess you refer to the special handling of `config.force_remote_user` setting. This comes from a new behaviour that will be introduced in 1.8 as well (see #6348). I still have to check if Ansible behaves the same when running with WinRM connection mode. If not, we could (at the moment?) get rid of this. I'll give an update in the upcoming PR.

@sneal Actually, it leads me to another question: Is there any plan to introduce a shared `username` setting in Vagrant, which could affect both: `ssh` username and `winrm` username ? (Vagrant might have similar backwards compatible issues as Ansible regarding to add `WinRM` in a `SSH` kingdom ;-)


You have yet to start your magnum opus ;)


We definitely need an OT dedicated to Korean exclusives


This is clever, but it's a good practice to keep dependency loading isolated, even in tests. All those `require` statements at the top of your tests will give you feedback about how many collaborators each object has, and when they start to get unwieldy, it's a good time to think about your dependencies.


By name I assume you mean the title - I think "Testing String Classifier" better fits the other ones, but let me know if you have something else in mind.


Yes


I was going to do that but instead of taking up flash allocation indexes which would likely change functionality very soon, I opted to leave this as a define so we can discuss what we would like all these knobs to be. The fact that it ignores altitude is an example of levels of complexity that I opted to not parameterize to leave open for debate.


I don't want to use Spring Boot, I just want to practice the xml configuration, but good to know with it doesn't need a lot of config.


@rich-harris @victorystick I'd appreciate a quick look at this plugin to see if the API and use case makes sense to you. Any suggestions are appreciated!


I forgot to fix "visível" and "difícil".
I will do it later.


It should be $response->getPaymentProfiles()


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


Finally able to push the file to github


misc_standard_report_measure_issues (DavidGoldwasser) - Win64-Windows-7-VisualStudio-12: OK (2336 of 2371 tests passed)

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


Wasn't my intention to limit the flexibility of your set handler..  I'll lay out my rationale -

:/   prior to adding /set sell and /set buy; "/set 100 item" and "/set blacklist item"  worked with the regex as it was; didnt think adding buy and sell to the regex would have been much more of a stretch to the logic since it was already checking for a number on the last parameter :/

Could almost have gotten away with a regex that only did a check "if (number or string), and (string), (and/or number)"  then just validated the string in the /set logic to match one of our commands - but the advanced way the regex formatted also allowed me to skip the need for a try parse on numerical input, since the regex would have stopped it trying to validate a string where a number was expected on the set on hand.. to begin with.. i thought it was cpu wise more efficient ..  i might have saved an entire millisecond of processing time *chuckle*   but then after i pushed it to master i had broken the regex.. so i guess not working at all is a lot slower than saving 1 ms :(

As for the structure of the way I format the commands...   i am trying to make the commands sound conversationally as natural to use as possible..   its almost like we are inventing our own scripting language for talking to our shop bot..

i would like to "set 100 explosives" available in my market mr set command.

price "set buy explosives 100"  mr set command

price "set sell explosives 200" mr set command

please "set blacklist organics" (or toggle its status) mr set command

etc..

If i wanted it less fluid, /set stock explosives 100  would also sound right, but take more typing.  The most common use of set would likely be restocking markets so I was saving the admin some typing :)
Had I not had a regex (they are pretty handy) I probably would have used the longer formatting in this way.  with some extra try parses to catch invalid input.  Regex seems to allow a lot crazier syntaxes if used right.

:)

Admittedly our buy/sell/value/pay  structure looks somewhat inconsistent looking too

/value item     most likely scenario is people wanting unit price so we cant really put number first; but sometimes they do so there is an option for a number at the end

/sell 100 item   - sell is a bit fiddly,   conversationally sell 100 items, or  sell all items sounds more fluid to write.  It could just as easily be /sell item 100|all   to bring it in line with value command layout but would be less conversationally smooth sounding.  Using numbers would be a more likely event too, so they went first

I dunno maybe I am clutching at straws here.   No real good reason why value cant accept a number in front too, but the logic would be a lot more fiddly for not much functional gain.

It is up to you if you want to bring all command order into a consistent format; just seems like extra work that would  be better used adding other cool stuff :)


Looks like you haven't removed the files. Do some research on why that happened.


At 8th Light, we typically start writing tests before thinking much about the details of our production code. As we test-drive a problem, the tests give us feedback about the design of the production code: something that is hard to test is usually a poor design. You're doing the opposite here. What gives you confidence that these five methods are the right ones?


combine to one foreach


HI,U would Have Made It more better....Mr.santhosh


You can use method GetBundleHashStart here


確認しました！


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6021](https://teamcity.mesosphere.io/viewLog.html?buildId=78503&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:13:10
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


It's hard to tell what this array represents. Which `true` is which player? What does `true` mean in this context? Primitive values without context are sometimes called "magic numbers," although these are magic booleans, I guess. Is there another data structure that could make your intent more explicit?


x


MFW I add a word that I shouldn't have.

> rare pepe


This is pretty dense. Can you simplify and name this to make it more clear?


:+1: for removing duplication by extracting a method.


why do you need this check. I didn't understand


At this point, you're starting to accumulate a few methods like `welcome` and `try_again` that have similar responsibilities (displaying a string). What if they were all grouped together?


是类似于万年历然后计算返回明天的日期的东西吗。


do this better


I had some problems with the react-element-to-jsx-string implementation to
the point that the last set of the expect-jsx testcases (the .toIncludeJSX
ones) would fail with an exception inside the react-element-to-jsx-string
functionality when run under a karma test runner with webpack. That was
easily resolved by switching it to react-to-jsx. But since I can't
reproduce the problem right now I'm more than likely going to completely
switch back to https://github.com/algolia/expect-jsx and wrap the rest of
my needed testing functionality around it.

On Thu, Nov 19, 2015 at 10:48 PM, Vincent Voyer <notifications@github.com>
wrote:

> Hey I am VERY curious, why did you switch to react-to-jsx?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EliasRut/expect-jsx/commit/68412811a1a614ce16ad5dcba95c5dd86d4bf69c#commitcomment-14512811>
> .
>


Are these all the possible things a board can do? It's hard to tell, but even if so, it would be easy to miss one. One benefit of writing tests before production code is knowing that every new bit of behavior has a corresponding test. in this case, you have to work back from existing behavior to a set of tests that may or may not be complete.


failed to unload driver, %s" % output


Hey look, a token!


same as above line


you're welcome :smile:


Ruby methods can simply return a value, e.g.:

```rb
def always_return_three
  3
end
```

What is the benefit of storing this in an instance variable?


```
post := fmt.Sprintf("POST /%s/%s/%s/proxy/namespaces/default/foo/id/some/dir HTTP/1.1\r\n", prefix, newGroupVersion.Group, newGroupVersion.Version)
```


wow!


1.why not support TEncoding of the utf8?
2.why not open the class of TRedisClient?if open,then connection can keepalived


it's easier to see the structure of the line with the vars pulled out, and you need `\r\n` to match the HTTP spec (not `\n`, which Sprintln gives you)


This has to be 'g', no?


might be better to leverage `path.extname`? otherwise this could lint files like `something.js.map`

maybe something like this:

```js
var extname = require('path').extname;

_.contains(cli.options.extensions, extname(path))
```

https://nodejs.org/api/path.html#path_path_extname_p


Looked at it for a while and couldn't figure it out. I'd suggest deleting the line.


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


Fixed in 5b535d96f18cef83c605a755420858aa358d0b34


Cool, I like the idea of passing a proc to this method.


:bird: :arrow_up_down:


kia dot god hyun uk


greate you are greate team member


준철아 일 안하니


I'm not sure you get a lot out of injecting `choose_move` here, since the return value of this method encodes details about `move` and `marker`. That means this method knows about how `choose_move` works, and relies implicitly on the values it returns in the order it returns them.


注册bug


Here is an example of the badges that might be found. These are for my weave repo, but you get the idea
  Notice_exc
BADGE YOUR REPO: WEAVE   Badge
We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.	BADGE URLS ▾
README BADGES X
If you need to use a raster PNG badge, change the '.svg' to '.png' in the link
MARKDOWN
[![Coverage Status](https://coveralls.io/repos/n054/weave/badge.svg?branch=master&service=github)](https://coveralls.io/github/n054/weave?branch=master)
TEXTILE
!https://coveralls.io/repos/n054/weave/badge.svg?branch=master&service=github(Coverage Status)!:https://coveralls.io/github/n054/weave?branch=master
RDOC
{}[https://coveralls.io/github/n054/weave?branch=master]
HTML

RST
.. image:: https://coveralls.io/repos/n054/weave/badge.svg?branch=master&service=github :target: https://coveralls.io/github/n054/weave?branch=master


hi


The way Travis was found was by going to settings, then webhooks. I did it wrong here I believe and so errored out.


Go 1.5 doesn't like `main` as the package name.


Here's another magic boolean. Is there another more explicit way to describe what kind of Player this is?


the `nginx` binary has been moved to `cedar` and `cedar-14` subfolders...

```
remote: -----> Fetching set buildpack https://github.com/leafo/heroku-buildpack-lua.git... done
remote: -----> Lua app detected
remote: -----> Copying lua to bin
remote: -----> Installing packages
remote:        Cloning into 'heroku-openresty'...
remote:        cp: cannot stat ‘nginx/sbin/nginx’: No such file or directory
```


Ruby provides a lot of rope to do things like this, but it's generally a bad idea to reach for `instance_variable_get` and `send` in tests. Try to test things through their public interface, and when you do reach for `instance_variable_get`, think about whether it's OK to just expose the thing you're trying to test.


The same comment goes for `instance_variable_set` (try to avoid it). This probably indicates that the thing you're trying to test should be more open. Why can't I just pass in a marker when I instantiate a `Player`?


Generate viewer id if viewer id doen't exist.


Si, Septiembre de 2013


using print in a function is not a good idea, it's might result as lots of output line when program run, but you can't even know where they from.


@eugene-bulkin Take a look, Rithvik said we were allowed to do this telepathically at the beginning of the network starting up, but I think it works fine without it.


Hint: return


@eugene-bulkin This is the alternative way of handling the case when routing tables aren't created.


This is a bit problematic. Added a request here for changing the color from dark gray to purple: https://github.com/ansible/ansible/issues/13224


@eugene-bulkin


add a test for

```
assertTrue(UNSUPPORTED_FUTURE_VERSION.isUnsupportedVersion());
```
?


parse_extensions takes a string representing extensions. Not an object. So, Extension -> extension. Since it's possible that it's thrown when parsing "," between extensions, "an extension" is also not appropriate if strictly speaking.

how about "errors in extension parsing"?


:+1: makes sense.


If it works fine without it, then we can leave it for now and handle it later.


No I didn't, but I guess this is due to different Encoding. Should I revert and try again?


I like to think of class methods as things common to a, well...*class* of objects, but not to any particular instance. Getting a move and choosing a marker seem more like things that an individual `Computer` instance would do. Yes, class methods are easier to call, but the structure of these abstractions matters.


If you mean do we support client certs, yes.

https://github.com/nats-io/gnatsd/blob/master/test/tls_test.go#L78


s/completed/completely/


yes, 1 should be a special case, but shouldn't it be place on the head of the function?


The which-clause was modifying "an extension", not ExtensionParameter. As you had to split this into two lines to conform to PEP8, it's ok to just throw away this sentence.


Can you give this ternary a name to describe what it does? One drawback of this nested-proc design is that it seems to be harder to extract and use meaningfully named methods, since they need to be wrapped in lambdas or otherwise converted to procs.


白框


yes, 1,2,3 should be a special case, but shouldn't it be place on the head of the function?


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


You didn't add deceit here.


Nice! :+1:

I'd like to see a test where the block is part of a subexpression, e.g. of a closure passed to a predicate function.


Here's a helper method just begging to be extracted.


if 0,1 are needless, why not calling `range(2, limit+1)`


this is indeed strange. i used just zest.releaser straight away. i'll look
later when at office what's the cause of this.
Am 19.11.2015 23:08 schrieb "Maurits van Rees" <notifications@github.com>:

> @jensens <https://github.com/jensens>: You have released 1.4.9 today.
> With that version, Jenkins fails to run some tests:
> http://jenkins.plone.org/job/plone-5.0-python-2.7/3326/
>
> [Errno 2] No such file or directory:
> '/home/jenkins/.buildout/eggs/plone.api-1.4.9-py2.7.egg/plone/api/tests/doctests'
>
> Same problem locally. I have released 1.4.10, which seems to fix it. I did
> not do anything special.
>
> Do you have an idea why the doctests directory was missing in your
> release? The MANIFEST.in was already fine. Can you try a python2.7
> setup.py sdist and see if the doctests directory is again missing?
>
> I can't really think of a reason why it failed...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/plone/plone.api/commit/8bdd2bd8564820b83ea04079bf29c808ffab1db1#commitcomment-14513236>
> .
>


Why is this side effect necessary? Why not just return `true` or `false`? (I think the answer is that `@win_status` stores some information that's not really related to the responsibilities of this class).


s/shama/shaman/


That is, how to display the concept of a tie in words.


Are you sure this should be `!=`?


It seems neither PEP257 nor PEP8 prohibit a blank line, here. Please point me to the section if either of them has. Otherwise, I'd like you to keep this.


Fixed this (both the upgrade script and the database.sql script).
DONE with this item.


Should we report this to Vim?


e


What gave you confidence that you didn't break anything with this refactor?


option value


Released lita-salt v1.1.0


Looking better after extracting some helpers here!


I meant the line above: `sed -i  "s/CC = g++/CXX = $CC/" Makefile`.
It's the second time you set `CXX =` ...


MArray_Elim (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1448 of 1448 tests passed)

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


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


Yeah!


@llogiq, thanks, your blog post on lints was quite helpful to me.

For the test, I'm not sure I follow, do you mean something like this?
```rust
    let opt = Some(3);
    if { opt.map_or(false, (|x| x == 3)) } {}
```


The parentheses around the first argument aren't needed. Please break in front of the '('. Update other lines in this test to match.


"""Return the attribute value

Returns the value overridden by this class for some attributes, and forwards the call to _file for the other attributes.
"""


I test it, no http error appear,but it seems that there is still some logs miss in elasticsearch.I print indexs of ESJsonEncoder generate in a file,the number of index is more than docs in Elasticsearch and is positively related to flush_count


So? It won't go away yet. This header is temporary anyways.


... stahp with your sound logic.


Reverted the revert. Yay for git!


嘛意思？


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


please undelete gb-1.aux


We have one input file, through which we have calculated address of corresponding statement .
We have also maintained the symbol table in symtab file. we have defined our own mnemonics in optab with file code.
Finally we have calculated the program length for the given assembler code.
This is my Pass 1.


Nice eh!


como puedo descargar el archivo?


solo puedo ver las lineas de codigo


is this for pydoc?


Including this module means this class now has some new responsibilities. If you don't want all the new enumerable methods to explode, you need to define `each` and `<=>`, the comparison operator. Including a mixin like this can be pretty dangerous because of these kind of implicit dependencies.


`$ man git-commit` and look for `--amend` :smile:


I saw unpredictable failures with this patch. It reported AnsibleError("the loop variable '%s' is already in use. You should set the `loop_var` value for the task to something else to avoid variable collisions" % loop_var).

I fixed it in  lib/ansible/executor/task_executor.py (line 191 - the first one in above diff) from:
items = self._squash_items(items, task_vars)
to:
items = self._squash_items(items, task_vars.copy())


Nie przetestowałam. Jakbym to zrobiła, to bym dała komentarz. Póki co tylko wrzuciłam pierwszego commita z danymi używanym w tutkach do share'owania wiadomości na twitterze.


FYI, there's a literal syntax for hashes. This would look like:

```rb
{[2, 1, 0] => -1000,
 [1, 2, 0] => -1,
 [1, 1, 1] => 0}
```


@Daniel15 is there some miss communication somewhere? with all due respect, a person tell you he/she needs A and you are saying no just use B instead...

I'm not even asking to add the `browser.js` file to this package. You got rid of it for some reason and maybe that's okay. Maybe my case is an edge case. Okay. ;-) All I'm asking is **what is the fastest way to get the `browser.js` file with minimum steps and have it updated to the latest version of babel periodically**?


@loganfsmyth what is the best way to build that `browser.js` file with minimal steps and modules involved?


Maybe it's better like this
```javascript
let item = this._appendMenuItem("Dash to Dock " + _("Settings"));
```


Are these messages the responsibility of the `Board`?


Love this change :)


blank line here plz


We have react-lazy-load with version 2.0.0, which supports 0.14


**Please IGNORE .odg file**. Replaced with remade one in PDF
[sf-wdi-25-tree.pdf](https://github.com/anonym0us3/homework/files/39800/sf-wdi-25-tree.pdf)
from google drive


Don't leave code in comments


Can you tighten up this assertion?


Looks like this commit broke build.
Please see http://ci.cgeo.org/job/c-geo/4147/console to remedy the problem.


Hi, add the closing list: "trigger.trigger('close.fs');"

options.on('click.fs', 'li', function(e) {
        var clicked;
        clicked = $(this);
        sel.val(clicked.data('raw-value'));
        if (!isiOS) {
          sel.trigger('blur.fs').trigger('focus.fs');
        }
        options.find('.selected').removeClass('selected');
        clicked.addClass('selected');
        trigger.addClass('selected');
	trigger.trigger('close.fs');
        return sel.val(clicked.data('raw-value')).trigger('change.fs').trigger('blur.fs').trigger('focus.fs');
      });


this is a beginning!


And this is a comment on the brief bio @hubyeti


Introduced namespace Ann to isolate from global namespace.


>  You got rid of it for some reason and maybe that's okay.

That wasn't me, that was @sebmck :stuck_out_tongue: I'm actually interested in this too, as I need it for [ReactJS.NET](http://ReactJS.NET).

> what is the fastest way to get the browser.js file with minimum steps and have it updated to the latest version of babel periodically

I'm actually wondering the same thing - I made a task to follow up on this with documentation on how to do it (https://phabricator.babeljs.io/T6651). I think the solution is to use Browserify or Webpack and `require("babel-core")` as Sebastian mentioned earlier, probably exporting it as a global (`global.Babel = require('babel-core')`). I'm not sure how to bundle plugins into that though!


hello, this is my contribution for pass 2 assembler.
here i have used that intermediate file which was created by Maninder in pass 1 as a input.
This code will handle each statement(of intermediate file) and calculate the corresponding object code.
after completion of object code creation , you will see that the symbol table ,and object program is printed at the console.
OBJECT PROGRAM
we have header record which will have a starting address and the program length (calculated in pas 1)
we have text record  which will have the starting address and the object code .
at last we have end record which will only contain the starting address.


yeah.... the overflow detection is required by oj...


memeda


FIXED.
DONE with this item.


Я вижу два варианта: 1) если new возвращает 0 при слишком большом размере, то проверять на 0 и бросать исключение, 2) иначе ввести максимальный размер очереди (константу в common.h) - 2^27-1 и проверять MaxSize на нее.


And add your favorite color


Also code in comments.


Add your favorite color


Indentation


Code in comments


[mukul1.txt](https://github.com/Mukulsaklani94/subSeaShepherd/files/39805/mukul1.txt)
[saklani.txt](https://github.com/Mukulsaklani94/subSeaShepherd/files/39806/saklani.txt)


104921238_EnergyPlus_Performance_Framework (lefticus) - i386-Windows-7-VisualStudio-12: OK (1968 of 1968 tests passed)

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


got assertion failed on windows:

    Assertion failed!
    Program: C:\Python27\python.exe
    File:..\..\source\fitz\error.c
    Line:117

    Expression: ex->top == nelem(ex->stack)

not sure it related to http://git.ghostscript.com/?p=mupdf.git;a=commit;h=2f0b62b1c02cb6826168ad73023697980b251fe6
did these libraries built before this commit?


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


ArtistB ? typo


added ability to review a workflow and its history #4


I don't agree that the default configuration should be the minimal one. I think that giving the full configuration as an example forces the user to try and change it.


hello, this is my contribution for the loader(mukul.txt) of SIC assembler.
basically loader function is to load the object program into the memory.
i have used the saklani.txt file which contains the object program(header,text record).
this loader will load the  object program with their addresses and corresponding opcode.


hey hi


Hi Feng
And other please add auto options for numqueues options
by default this options set to 1  but need to change this if you set 0 by default to use all install cpu core
if you not need to use all cores set a number to use.

m.


Sorry, my bad.

I introduced quotes in order to avoid issues with pathes containing spaces when used as args to commands (not here though), it's common practice to quote path parts.

one of the next step we should do is to introduce some simple integration tests to avoid this in the future.

sorry again ....


'Paused' State Machine can't Stop. because Stop internally call `acpipowerbutton`.
So I change from `stop` to `kill`(power off). I think anyway machine will be deleted, even it's in the `Running` state. it doesn't matter


не нашел ничего лучше чем объявить переменные


1) if prefixing with with VSSIM, it worth to change everywhere.
2) VSSIM => eVSSIM


my OMR has four circle in all corners in stead of rectangle. Would you please help me how may i find corners.


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


@brusenskiy, так ведь после return управление функцией прекратится, а это произойдет после первого case, разве не так?


that's the key change here, right?


sorry for nit picking, but SUCCESS is ok
FAIL is better to change to FAILURE


I am not an expert with this but as far as I can understand it it looks good. good idea.


s/Warningf/Warnf/g


MArray_Elim (DeadParrot) - i386-Windows-7-VisualStudio-12: OK (1975 of 1984 tests passed)

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


Humm, I probably need to update the python compiler that we use internally (@eytan the one I shared with you a long time ago) as well


I've left a lot of specific feedback, but I'll leave you some general comments here.

Some positive things:
- This fixes most of the problems we pointed out in the problem.
- You're moving towards a cleaner separation of concerns, with separate classes for each component.
- I can tell that you've put some thought into making the design flexible, by passing around procs and callbacks.

Some negatives:
- I was able to beat your computer player with a fork!  Try `a, g, c, e`.
- Classes are still more complex than they need to be. Ask yourself: does each one just do one thing? If not, there's probably more to be extracted.
- Inside classes, there are still a few gnarly, complex methods. You can probably simplify these too.

In addition to the small stuff I commented on, I'd like you to try one big change that will test your design: Make this work for a 4x4 board. Take as much time as you need for some revisions and contact me when you feel ready for another round. :smile:
-


And may be check code for more cpu optimizaandtion and reduce load


@deepgully, could you please provide some more details, like versions? Note that currently the windows libs are in a separate repo here: https://github.com/JorjMcKie/PyMuPDF-optional-material. And I think that those libs are up to date.

@JorjMcKie


thanks!


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


Extract sleep time calculation to a function.


this 'for' loop seems usesless.....


LGTM


+1. Actually they are not needed for Maven as they are Ant specific.


Thank you! This commit fixed the build. It was broken since http://ci.cgeo.org/job/c-geo/4147/


FIXED.
DONE with this item.


Got ya! Thanks for trying  to help again.

Cheers!


maybe some write,create and unlink access for a manager?


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


header is not required into __init__ files


Should be 8.0.1.0.0


What exactly was the issue here? I'm curious to know just in case it's something I should also fix in my build.


to be removed


merge upstream


you can remove 3 '=' of the title


indentation


header is not required into init files


@deepgully

We've decided to remove prebuilt libs from the PyMuPDF repo, which can be seen here: https://github.com/rk700/PyMuPDF/issues/29

However it was done on the 1.8 branch and not merged into master until just now. So I guess maybe you were building from an outdated PyMuPDF source? If so, can you please try the latest commit again(use `git submodule` to pull the windows libs)? It's not in the README, which may be a little confusing. But we will update it soon.


set loss_volume = added_loss_volume first.


One would tend to say non-nil rather than true.


The best commit message :+1:


👍

On 19 November 2015 at 22:47, Ben <notifications@github.com> wrote:

> i like that. i think our "hack" for the dbaccessor is the property
> accessing xform_model. i think if we do this right we could remove all
> those references.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dimagi/commcare-hq/commit/ecf78dc311b35f2e040197d646bd3f02dd091f0b#commitcomment-14511557>
> .
>

--
Simon Kelly
Senior Engineer | Dimagi South Africa


Как лучше сделать?


onWorldTick and getTileEntity ... wouldn't that casue immense ghostloading? or is the reutrned list of ImmersiveNetHandler.INSTANCE.getAllConnections(world) only containing the loaded nodes ?


I think these 3 fields could be written on one line ..but more important: should have a string attribute


LGTM


What about removing the bitcoin here completely? Ie. Only Building?


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


You shouldn't make replace, but change readonly attribute.


Please use Bitcoin Core or nothing here. bitcoin-core is an additional term not used anywhere.


FIXED(changed object counter to "patients").
DONE with this item.


FIXED.
DONE with this item.


combine this the following line


:heart: :heart: :heart:


@tacovandenbroek but this is only pushed to 7.0+, it should not affect the 5.6 branch


"size_t size" should be changed to
storage_strategy_metadata_ptr metadata

typedef struct storage_strategy_metadata {
      uint64_t id;
      uint64_t partition;
     ...

} storage_startegy_t, *storage_strategy_ptr;


"uint64_t id" should be changed to
storage_strategy_metadata_ptr metadata

typedef struct storage_strategy_metadata {
      uint64_t id;
      uint64_t partition;
     ...

} storage_startegy_t, *storage_strategy_ptr;


@Dietmar-Franken This commit https://github.com/mycontroller-org/mycontroller/commit/1c7c49aff3f8e700f77d320067c0023a07c3799f will explain how to change `notify` messages. By mistakes I removed your latest changes from `de-de` language file. Again merged your changes on this commit https://github.com/mycontroller-org/mycontroller/commit/a15a5bdde87b5d4297797cfab229426dbd5f1b72

For notify error messages we have to change in Java files. we yet to create language support feature in core `Java` of `MyController`


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


color


Yeah, realised that just as I pushed. Sorry! :/


Si, no hay problema, però este código està en el fork de @blackleg

Deberias hacer un PR al repositorio de Som Energia mejor.


Also worth mentioning, you can just check `if(!messages)` instead of `if(messages == null)`.

Additionally, you probably want to be using the triple equals to compare if you stay with `if(messages === null)`.


1. Good idea, it will fit better here, as you are finding User. It is worth to mention that you can also change it to findUserById($id) if you want to find users using only their id. In most cases you will use BySomething(...).
2. Sure, it is always better if phpdoc block is in the code


What about CreateThread vs _beginthreadex?
https://support.microsoft.com/en-us/kb/104641
http://stackoverflow.com/questions/2100589/beginthread-vs-createthread


1. Yeah, good idea.
2. Sure, let's add them.


Did you try increasing galaxy spacing? I play around 20k most of the time, that could be a problem.


pass


One line comments are ok if you really need it for some reason.

If you have line:
$storage->add($user);
there is no need to add comment like "Adding user to a storage" as the code is self exexplanatory.
If you want to comment some bigger block of code, then it is fine to add it.


Eventually this is going to run the expression evaluator code, I see no practical benefit to running it concurrently to target threads. Consider that even when the condition isn't met we still need to stop all threads anyway.


**Code Quality - Nitpick

You have two methods namely `getSlang` and `getSlangs`. It's recommended that your methods are named in a very legible and less confusing way like so:

`getSlang` could be renamed to `getASlang` or `getOneSlang`. This easily differentiates itself from `getSlangs`