TeamCity MonoGame :: Develop (Win) [Build 3.5.0.944](http://teamcity.monogame.net/viewLog.html?buildId=12241&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 553, ignored: 6 Build time: 00:33:14


TeamCity TripleA :: UnitTest [Build 487](http://home.k-moeller.dk/viewLog.html?buildId=1174&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:48


Add `:` after the closing bracket of `bump()` in commit's title. You can omit the `github.com` prefix.


@suunrider I know abigails works, I was talking about yours


this answers your question?


See explanation in transport.hh


TeamCity TripleA :: UnitTest [Build 488](http://home.k-moeller.dk/viewLog.html?buildId=1176&buildTypeId=TripleA_UnitTest&guest=1) is now running


Here's what it would have looked like if @Divran had done it right the first time :P
https://github.com/wiremod/wire/compare/70939209568b8606a3b9cc03776e84b7b78d7196...7f0a7694a196c8027421855a03c957a48c21cd6d


I don't get it. If you add a `stp_value`, the simulation will just stop after `stp_value` iterations, won't it? How is related with it being killed?


TeamCity TripleA :: UnitTest [Build 488](http://home.k-moeller.dk/viewLog.html?buildId=1176&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:48


`RUN_STANDALONE` won't be set for RPC calls, but it shouldn't matter in this codepath anyways, since the server won't be initialized and thus we won't listen on any ports. This is a bit of ugliness really: we're using `setup_ServerHandler` to parse the ports configuration and don't use the result to actually setup the server handler.


I made this mistake too. These have to be `<meta property`, not name. See https://www.optimizesmart.com/how-to-use-open-graph-protocol/ @herb-t


TeamCity TripleA :: UnitTest [Build 489](http://home.k-moeller.dk/viewLog.html?buildId=1180&buildTypeId=TripleA_UnitTest&guest=1) is now running


ah OK.  was just curious


TeamCity TripleA :: UnitTest [Build 489](http://home.k-moeller.dk/viewLog.html?buildId=1180&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:46


I haven't seen a citation for the AoA for Gemini, but Braunan reports a Gemini L/D of 0.16, which to be honest is new to me. I recall having trouble altering the Gemini L/D formulaic-ly in the past, but that might work better with modern FAR?

http://www.braeunig.us/space/specs/gemini.htm


There's quite a bit of overhead for a function call. I would rewrite this to make a single call to a `loadImages` function and pass an array. @herb-t


Only thing you changed that matters in any way is underscore to camel case, and the code guide currently says camel case is ok. The space between brackets thing doesn't matter at all in my opinion. And the `return` instead of `portname =` makes no real difference to readability or code efficiency.


its added to fix min freq locked at lock set by touch boots driver, if you set min freq holding touch.
its not about thermal.


This introduces a regression bug: support for obfuscated fonts is broken. I am working on a fix.


Do you recall what was the critical factor in your Gemini near-failure? I use the Gemini roll position to control my L/D, which might be harder to do if we are able to achieve the Braeunig reported L/D? If that ends up being the case, then I think the current thermals are probably quite good.


Why you remove test for PHP 5.3, but you merge 6748e16204d812745d758c6514328c548334d3dd? This is nonsense.


Thanks! See: <https://gerrit.gromacs.org/#/c/5455/>


TeamCity TripleA :: UnitTest [Build 490](http://home.k-moeller.dk/viewLog.html?buildId=1186&buildTypeId=TripleA_UnitTest&guest=1) is now running


@MMajko How this function should behave? Is it intentional to accept even properties with non-Boolean value that is not false?


* Não consegui modificar a parte do cabeçalho do arquivo de entrada, eu não sabia se devia mudar aquele que estava no programa ou criar outro, na dúvida não modifiquei. Porém, coloquei lá nos issues como deve ser o formato dele.


TeamCity TripleA :: UnitTest [Build 490](http://home.k-moeller.dk/viewLog.html?buildId=1186&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:45


let me know if you need help, an invite might be nice too. Go UC!
![try](https://cloud.githubusercontent.com/assets/11181778/11792994/1f430bc8-a277-11e5-8736-84c9dafda038.gif)


replace "%d" by "% 6.1f" for example


Options should be defined using the `Option` class or one of its subclasses, which in this case is `BoolOption`. For example:

    validate = BoolOption('tracsubtickets', 'validate', True,
                          doc="Some text to describe the option.")

Example: http://trac.edgewall.org/browser/tags/trac-1.0.9/trac/env.py?marks=176-178#L176

The 4 lines of code can then simply be replaced with `validate = self.validate`, or just replace `validate` with `self.validate` in the lines that follow.

It might be more useful to provide the option in the `[ticket-workflow]` section. Example:

    [ticket-workflow]
    resolve.validate = False
    reopen.validate = True


Yes but you control the kill and espatially the quality of the restart file.  At the end of the output file I d'ont want an half state. A manually kill do it


OK, I will add the handle of EPS file to generate the pwg_representative in function add_uploaded_file, with the same code as for PDF. But I don't understand why we need to modify admin/include/image.class.php and include/derivative.inc.php


Addressed here: https://github.com/readium/readium-sdk/issues/221


Trochę bez sensu był ten commit, ale pomieszały się pliki po  'merge'u


ewedsds


Then sounds like the offset should get rather lower than it even is now.


Ops


Plik 10 najlepszy do edytowania bo ma najwięcej linii


Higher speed and lower perigee = higher q" and higher skin temp, I overtemped.


Nie wiem skąd tutaj znalazła się pi


I woulld like to enter it in parameter exe
But exe take only characters in fortran apparently


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 71](https://teamcity.labkey.org/viewLog.html?buildId=291417&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 22 (1 new), passed: 268, muted: 2 Build time: 01:35:10
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSHelpCenterTest: <no details avaliable>

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

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

org.labkey.test.tests.CDSTest.verifyGroups: <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.verifyFilterPane: <no details avaliable>

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

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.verifyDensePlotBrushing: <no details avaliable>

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


Dlaczego po PUSHu tak brzydko pozmieniało mi kolejność linijek :(


@swiftix This commit added a bunch of tabs for indentation. Please fix.


Why not use this.parentParentPanel. getTableOfIdeasCollapsedState() again? I feel the object creation here breaks abstraction as well as DRY.


Proszę o nie usuwanie moich komentarzy


On Mon, Dec 14, 2015 at 12:13:07PM -0800, MercenaryJosh wrote:
> 1) Using https://api.azurestandard.com/products?category=null
> returns many items with empty packaging arrays. Since categories are
> attached to the packaged-products that should be in the packaging
> array, these are results are useless to the categories app.

That may be #956.

> 2) Since categories are attached to packaged-products and not
> products it in more intuitive that the categories app should use the
> packaged-product api and not the product api.

That cuts both ways, since names are attached to products and not
packaged-products.  Maybe it would help the intuitiveness if we just
drop /packaged-products and /packaged-product/{code} from the API?  I
can't think of a situation off the top of my head where they'd be a
better choice than their product analogs.

> 3) When the /product?category=xxxx was used in retrieving the
> packaged-products for existing categories, it was returning all of
> the packaged-products for that product and not just the
> packaged-products that belonged to that category. This caused
> packaged-products to be displayed in the wrong categories.

That sounds like some additional filtering we can add/fix on the
Beehive side of things.

> 4) Algolia is useless for the categories app since it is not updated
> in real time and therefore will not reflect any of the changes made
> with the categories app.

Ah, right.  Thanks for keeping all of this straight, there's a lot
going on ;).

Anyhow, I'll work on
https://api.azurestandard.com/products?category=null to see if I can
address your (1) and (3).  In the meantime, [1] has my September work
on converting the category manager back to /products endpoints
(especially in the WIP azurestandard/internal-website@d513b89), if you
want to look that over [2].

[1]: https://github.com/azurestandard/internal-website/tree/tk/category-tags-and-other-fixes
[2]: https://azurestandard.slack.com/archives/general/p1443130706002399


Wszędzie tylko objetosc walca


Yeah, sorry for not responding earlier. This was the hackiest hack I wrote to get compilation running. There were no conditionals I could use at the time to make this work.

We might need to introduce a DebugMac configuration for this on the next merge from master.


Może printf a nie ntf?


Pracujemy w domu!


Kolejny niepotrzebny enter


Jeszcze jeden komentarz


ok, thanks @aedmonds.

Remember if you are using https://github.com/dotmailer/dotmailer-magento-extension it's already integrated into so don't have to use this as separate module anymore.


I na sam koniec jeden!


```lua
local function mapOutputAlias(ent, portname)
	if ent.OutputAliases and ent.OutputAliases[portname] then
		return ent.OutputAliases[portname]
	else
        return portname
    end
end
```
would be even more readable


Komentarz pod commitem! Niestety nie ma mi kto odpowiedzieć...


Odpowiem sam sobie a na zajęciach poproszę Irminę


Nope, still not ^^
I don't get how is is better than `n_iterations`, except that they are not part of the same file… You can do exactly the same job by replacing your `stp_value` by `n_iterations`. It'll stop the simulation at `n_iterations` and do a clean output for that.


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


This could be more direct, really.
```
if (! isset)  { $_SESSION[csrf] = generate_token(); }
return $_SESSION[csrf];
```

but we'll fix that later, it's late. :/


komentarz


Yeah, my approach requires a decently high L/D (skim in a sweet spot somewhere in 60-70km so you don't overtemp, use L/D to hold you down in that range rather than escaping) which will enforce a lower window for success once we adjust the L/D to match Braeunig. So, the thermals seem in order,we just need to fix the L/D.


Sounds great. I've also raised a PR with a very minor change to help do key derivation from a password, let me know what you think


komenatrz do calego komitu


Shouldn't it be "revision 1"? At least that's what [Formula-Cookbok section on revisions](https://github.com/Homebrew/homebrew/blob/master/share/doc/homebrew/Formula-Cookbook.md#formulae-revisions) suggests me. [Its example](https://github.com/Homebrew/homebrew/pull/31207/files) is a case where the formula is changed to add "recipe 1". Same with the [other example](https://github.com/Homebrew/homebrew/commit/6b9d60d474d72b1848304297d91adc6120ea6f96).


Looks fine to me.


Anything under pkg/devmapper/ is now a separate package. I am thinking will it make sense to prefix `devicemapper` for these and all the code which is under daemon/graphdriver/devmapper/ will prefix `devmapper`.

Subtle but will allow us to differentiate between messages originating from package vs originating from graph driver.


Only way portals can be destroyed is by an explosion. If a client tries to break a portal, we should tell them to go jump in a lake.


komenatrza do calego komitu2


saw approval from macintux
at https://github.com/basho/riak_kv/commit/0c715c94b1be8f8741380d1309bb2e1504116728


merging basho/riak_kv/perf/test_release_candidate = 0c715c94 into borshop-integration-1306-perf/test_release_candidate


basho/riak_kv/perf/test_release_candidate = 0c715c94 merged ok, testing candidate = 8e56658c


Right, and this was just addressed as you suggest with commit 245b7ab3d1c06efe5b76a9412996fac50aa77f72.  I'm surprised I didn't write it that way initially, it must have been an oversight while I was reviewing it.


Man, it almost seems like we could use a meta language like Clojure to generate these cases :)


Members with *NO* address see an "Add Address" button instead of the
checkout button


Nowe commity :rocket:


I think this is another case of _"no, it's not quite as simple as initially thought"_. This solution doesn't work well with Java 8 fanciness or Scala code, see: https://gist.github.com/jotomo/41a5bf31796fc0fbdb4f
The _WikiParser_ example is from the original issue on this, the _Batch_ example is from my Scala code.
Right now I'm thinking that if an uppercase letter follows a single dollar sign, then it's the name of an inner class. But only if it comes after the first dollar. (e.g. the second _Batch_ in the _Batch_ example). I'm not sure what double dollar sign means, but it seems whatever comes after it, is generated, so maybe a double dollar can be used to shortcut the stripping.
I generally just want to see the class name and then use the line number to lock into code. It could be argued that in inner class is an implementation detail of the outer class. That viewpoint is probably why I didn't consider inner classes different from anonymous classes.


actually I believe having
mode get_mode(mavlink)

 instead of

bool check_mode (mavlink, mode) would be a bit more compatible with our API and would be more general purpose.

if you're worried about returning error, it could be
bool get_mode(mavlink, &mode)


gmt prefers to keep his method, I trust his judgement as he has done a lot of the core work for the bot.


Google voice login information and text message information.


Le 14/12/2015 20:14, Paul Franklin a écrit :
> If you mean "Until" in line 1871, I suggest "Till" be changed.
>
Yes. It's a mistake. I modify this.
> —
> Reply to this email directly or view it on GitHub <https://github.com/gramps-project/gramps/commit/8bf8e92e3c293cf34d2cc51db45864e5599b2c65#commitcomment-14967368>.
>


Making IStoreGroup extend IStoreItem and Collection seemed natural to me, since right now the UI refers directly to StoreGroup (the implementation). With this change only in the interface declaration, the implementation does not need to be touched, and yet the client (UI) code can use IStoreGroup instead of the concrete class.


![screen shot 2015-12-14 at 12 35 18 pm](https://cloud.githubusercontent.com/assets/360927/11793314/b6599152-a25f-11e5-939d-b973aa5bfc8c.png)


you absolutly right


thank u sir


are


make a separate file that is playerState.js. All player functions and variable should live here.


Doesn't the document element listen for 'click'?


Use require


about the error stuff @glima was mentioning, let's use this one as example (sol_mavlink_land)


You're welcome.


Indeed, binding shouldn't be done here, instead the function should come already bound.


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


ANTs actually pretty closely tracks ITK's git repo, rather than building against release, see
https://github.com/stnava/ANTs/blob/master/SuperBuild/External_ITKv4.cmake#L163

Maybe it's a safer idea to do the same?


This is me from my Mac. Forgot to run git --config


http://www.2ality.com/2014/09/es6-modules-final.html for require modules


for this two checks you could return -EINVAL indicating an expected parameter wasn't valid


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


here you could check if write == -1 and return -errno in such case

otherwise you should return 0


FINALLY EVERYTHING WORKS


:+1:

Not sure if we want to explicitly say, "update your bookmarks accordingly" or not ?


Not quite in line with the style guide for objects. Try doing something like this to get the lines shorter
```javascript
const numEnv = repo.environments.length;
const firstEnv = repo.environments[0];

dc.deployedRepos.push({
   environment: numEnv > 0 ? firstEnv.name : '',
   // ...
})
```


Agora tem que concordar o gênero de "múltiplos" com "setas".


m_Transport can never be null since m_Open = true.

m_Transport.close() handles cases when port is null


If we would assume Redux is always as a part of a plugin, we could use the built-in function https://codex.wordpress.org/Function_Reference/plugin_dir_url

If Redux was part of theme we could use function https://codex.wordpress.org/Function_Reference/get_template_directory_uri

But since we cannot assume either, we use this function to get the url to wp-content, as both themes and plugins always reside under it. The location of wp-content might change, but that does not matter.

I don't think anybody would ever place Redux outside wp-content in a WordPress context. If you want to create a generic PHP library that is also WordPress compatible, then make it into a Composer package.

Anyway, this piece of code is an improvement over the old one, please accept it.


Faltou acentuar o "tóxica".


with this approach, a dev would have a better clue if something goes wrong, don't you think?


No, it is no longer used. Can be deleted.


Acho que seria melhor "e atinge mais dois inimigos próximos", pra ressaltar a ideia do efeito adicional.


esse predicado tem apenas a utilidade de alterar a condição do wumpus na memória do agente, deve falhar por não conter instruções de ação passadas ao mundo.


Falta acentuar o "são".


Имя класса принято записывать с заглавной буквы


Issue5276 (wfbuhl) - x86_64-MacOS-10.9-clang: OK (1935 of 1969 tests passed)

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


I agree on removing "_".
@otaviobp 's suggestion is good and should be done as well.


Any reason you removed the public visibility in this checkin?


final? private constructor?


Hmm...then it may be correct to add a super call to TrackSuspended.


Isn't there a bit more of this? Aren't skins made up of the arm and sleeve?


para o modelo apenas a ordem é importante, no caso ele o cabeçalho vai definir a ordem de leitura...

Porem os arquivos ja devem usar um cabeçalho, o modelo ja pula a primeira linha do arquivo, e como o cabeçalho em si é importante para a interface é bom definir não so a ordem como o nome dos itens no cabeçalho.


Option Constructor deprecated


Thanks for noticing - my mistake. Fixed now in a new commit.


I'm fine with changing it, though, I think it would be better to gradually change it as we improve the different methods.


I think parts.get(3) should be parts.get(2) (though I have never seen an exception in tempto because of this).


@jnv I’ve fixed it in bd18e2cacce4c65573aef71c656392f39bb1307c. Does it behave the same as previous? I’ve also fixed test, so it should be okay now. However, I don’t know if it’s intentional to let it pass for truthy non-Boolean values.


rule the world


Ten unit to teraz niczym się nie różni od zniżki procentowej, myślę, że tego rodzaju zniżka będzie mieć inne obliczenia.   To przekazanie value i potem dzielenie przez 100 itp. tak mi jakoś dziwnie to wygląda.


The REPL is using `InputT`, of which this is running inside, and yet here I'm calling `getLine :: IO Sttring`. Hope that won't cause any complications.
This call to `getLine` on a separate thread is just to provide the user an opportunity to break out of the watch file loop by pressing enter.


Эти два поля лучше отнести не к классу «список персон», а к новому классу «персона». В текущем классе должна содержаться коллекция объектов типа Person.


My bad, this was use by tests:

```
 File "/home/travis/miniconda2/envs/pyenv/lib/python3.3/site-packages/theano/d3viz/tests/test_d3viz.py", line 3, in <module>
    from theano.d3viz import has_requirements
ImportError: cannot import name has_requirements

  File "/home/travis/miniconda2/envs/pyenv/lib/python3.3/site-packages/theano/d3viz/tests/test_formatting.py", line 3, in <module>
    from theano.d3viz import has_requirements
ImportError: cannot import name has_requirements
```

Can you update the 2 tests to import from the file theano.d3viz.formatting ?Otherwise just remove this commit.

thanks


Should this be options.animationTime !== 0 instead of options.springStiffness !== 0?


Not that it matters, but the indentation in this file is a little janky


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


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


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2949


Welcome! :+1:


I am using ITK to build bunch of other tools , apart from ANTs


Конструктор тоже должен относится к классу «Персона»


Needs javadocs.


We should trust the Templates.Grains package to pull these other packages in - is that broken?


OK!


@pmwmedia P.S. forget to ping you in the above comment.


Ja uwazam, że w API tego SDK na zewnatrz wartosci unit i percentage powinny byc wystawione jako double.


Ahh..someone used TABS before...let me stab him in the eye...


teste comentário


Use `asset-path`, not `asset_path`.
https://github.com/rails/sass-rails#asset-pathrelative-asset-path


teste comentario


That is not broken. That is a not implemented feature by the nuget wizard extension ;-) (yes...)


This appears to introduce an undeclared dependency on tilt.


If you want to use Github for Chullo, then you should replace this with the actual repository address.


Thanks for reporting. It's a trivial change, I can fix it directly in SVN.


Issue5276 (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (901 of 901 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.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.59%25-green.svg)


Здесь получается, что список персон содержит список списков персон) Вместо этого лучше использовать ArrayList<Person>.


Bardzo fajny


Where are these duplicated?


Jeszcze lepszy


Fixed in rev 24816


`global: isort configuration fix`


Yeah, it probably should be "add".  The only difference with `set_callback` is that the latter will create a callback category if it doesn't exist before registering to it.  If there is code that is simply using a callback created by some other entity it should use `add_callback`, which will raise an exception if the callback doesn't exist.


Potwierdzam


@KostyaSha


cool


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 109](https://teamcity.labkey.org/viewLog.html?buildId=291451&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:02:47


Для начала в этом классе нужен  метод printPersons(), в котором будет производится вывод всех персон из коллекции поштучно на консоль.


The problem is that this function is very badly named. We should figure out better name.


Don't you want to add that the order in which types are defined will also define the order of parsing?


Thanks :+1:


Makes sense - I almost pushed as two separate commits when I first opened the PR, for similar reasoning, but refrained. Will fix, split, and force push again momentarily.


Yes, for example, for local environment (no CDN):

<img width="1111" alt="screen shot 2015-12-14 at 21 57 15" src="https://cloud.githubusercontent.com/assets/132146/11793802/04a349ae-a2ae-11e5-94c3-2fe4506846c8.png">


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.403.0](http://build01:8888/viewLog.html?buildId=13589&buildTypeId=ConfigInjector_CI) is now running


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.401.0](http://build01:8888/viewLog.html?buildId=13587&buildTypeId=ConfigInjector_CI) is now running


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.402.0](http://build01:8888/viewLog.html?buildId=13588&buildTypeId=ConfigInjector_CI) is now running


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.402.0](http://build01:8888/viewLog.html?buildId=13588&buildTypeId=ConfigInjector_CI) outcome was **FAILURE**
Summary: Cannot start build runner; exit code -42 (new) Build time: 00:00:07


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.403.0](http://build01:8888/viewLog.html?buildId=13589&buildTypeId=ConfigInjector_CI) outcome was **FAILURE**
Summary: Compilation error: src\Sample.IntegrationTestsWithSettingsThatDoNotHaveConfiguration\Sample.IntegrationTestsWithSettingsThatDoNotHaveConfiguration.csproj Build time: 00:00:10


I thought we could use 'till' as it is the older form of 'until'.
I didn't take attention to this as richard propose 'till'.
The exact usage of this word in the generated page if for a marriage. We know the duration of one mariage is for example 30 years from 1900. between parenthesis, I wrote 'till' 1930 because the husband or the spouse deceased in 1930.
In this case : till or until ? I know this question is a FAQ.


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.404.0](http://build01:8888/viewLog.html?buildId=13590&buildTypeId=ConfigInjector_CI) is now running


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.405.0](http://build01:8888/viewLog.html?buildId=13591&buildTypeId=ConfigInjector_CI) is now running


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.404.0](http://build01:8888/viewLog.html?buildId=13590&buildTypeId=ConfigInjector_CI) outcome was **FAILURE**
Summary: Cannot start build runner; exit code -42 (new) Build time: 00:00:04


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.405.0](http://build01:8888/viewLog.html?buildId=13591&buildTypeId=ConfigInjector_CI) outcome was **FAILURE**
Summary: Cannot start build runner; exit code -42 (new) Build time: 00:00:05


That's what I tried originally. However since the follow lines are dependent on that line, they raise AttributeErrors.  I would need to add try blocks for each of the lines or I could skip them altogether if the model_url isn't present. Which way do you think is best?


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.401.0](http://build01:8888/viewLog.html?buildId=13587&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:31


Этот класс и всё его члены лучше пока сделать статичными, чтобы не решать вопрос, в каком месте его инстанцировать и как передавать ссылку на экземпляр.


You have a typo here when you converted it; there should not be any `()`.


You should add a comment that explains what happens here, because this seems super arbitrary.

Also, did you try with unit tests, or only with apps?


cc @chrisseaton


Dlaczego tak a nie inaczej


To jest komentarz taki a nie inny


Komentarz 2


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


Komentarz 3


ack


Komentarz 4


Komentarz 6


video added


Gdzie się podział komentarz 5?


Nie ma i już


Nie drąż tematu


В данном месте можно поместить вызов статичного метода printPersons() класса PersonsList.


Więcej komentarzy!!!


Ale owner zaspamował komentarzami


Pewnie robi zadanie na NPG


[nit] I would use (0, 0) for non-hierarchical tuples.


Love this solution.


should 'bank' parens in the expansion ? I'm not sure


Yeah, no caller, but we'll use this later. (If I had a caller where I need one, I get weird behavior).


Agreed.

Also, this page will likely be removed, and the new REST service should be used instead.


Why doesn't this work on msvc? Looks sane.


I'm a little confused. The execution will stop when you reraise the ParseError, thus no AttributeError will occur. I just wanted to catch it and reraise it with a more descriptive message.


approvo


Great use of `each_with_index`. What's a different (but similar) `Array` method you could use to do it in one line?

http://docs.ruby-lang.org/en/2.2.0/Array.html


Looks A-Ok!


Is this asymptotically optimal?


YES! And this is why I asked for you to review the docs before I released :-)


Thanks!  Fixed by 50ce76bfa73ca369c276739c2bece0616e53dfba


I don't have a good suggestion here right now, so I can understand why these `$max_running` constants are hard coded here.  However, I'd like to start a conversation about whether there is a better place to put these constants.  Perhaps in our config or in the genome database.


Don't do that, if you do it, then the output of the simulation will depend on the frequency of the output!


It seems suspicious that these intrinsics don't work correctly *on this one platform*. Is there a technical reason for that? I might prefer to leave this as is and consider it a bug - intrinsics should work the same everywhere generally I hope.


Look like you forgot to add processing for bobsYerUncleBerry to emerald


develop (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2337 of 2373 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)


Ok, my bad about the inheritance hierarchy; still getting used to your terminology. The fact that CubeSensor should be just a listener you register somewhere is still valid.


`memo` is a String => Int, as is `get` - imho better for memo to call get than vice versa.


хороший комментарий


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


Do this for me. Test this code with a symbolic link and see if it works.  :)


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


Interesting thought. Another option is .withDefault {} and just putting the function logic there.


Issue5276 (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1941 of 1975 tests passed)

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


Not sure whether this mapping is actually needed. Don't think so. IMO you simply can hand the numbers over to Kodi as they were delivered by tvheadend.


Done!


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.947](http://teamcity.monogame.net/viewLog.html?buildId=12283&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Mac) Build time: 00:02:31


Yeah, I changed it in 817757c36ad94e5ad53b1e1178a863f47994ef0b so it's a little easier to configure.  I still don't think it's ideal.


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


Use this as a javascript guide
https://github.com/airbnb/javascript

express on top of node.
look for a mvc framework that interacts spring

use mustache if we want more templating

this is called a route. it will end up in javascript file
rolepanels/chooseRolePanel

rolepanels/werewolfpanel.html
rolepanels/seerpanel.html
rolepanels/robber.html


wow


much fun


so lol


OK, sure. ;)


Use lodash


```java
    CAPE(0, "cape"),
    JACKET(1, "jacket"),
    LEFT_SLEEVE(2, "left_sleeve"),
    RIGHT_SLEEVE(3, "right_sleeve"),
    LEFT_PANTS_LEG(4, "left_pants_leg"),
    RIGHT_PANTS_LEG(5, "right_pants_leg"),
    HAT(6, "hat");
```


I personally think leaning on the regex detracts from the beauty of your solution (which is one of the things I really appreciate about your code).


Why 100 and not 80 characters:
* it's what [PEP8](https://www.python.org/dev/peps/pep-0008/) defines, and flake8 validates.
* it's what I can fit on half of my screen? ;)
* Most python IDEs and editors will display a warning for lines > 80chars.


If all you are doing is comparing to 0, you should create and use an [`equals` method](http://docs.oracle.com/javase/7/docs/api/java/lang/Object.html#equals(java.lang.Object)) - please see the linked web page.


Probably a good idea to match (and right)? `LEFT_PANTS_LEG`


Best commit iv ever seen 420 NOSCOPE!!!!!


Advice on what?  I'm asking if you meant that those two #defines should share the same value (0x8).  I would have thought `GRID_FLAG_FGRGB` should be `0x10`?


This step isn't O(N) in the size of the list is it?  I think the overall time is more like O(N)*O(D) where D is the size of the document?


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


+153957 Thanks!


Ref on CPP vs. CXX: https://www.gnu.org/software/make/manual/html_node/Implicit-Variables.html


Deleting fields generally leads to poor performance.


Indeed, I was not sure what to best do here. This regex basically takes all the concecutive slashes and then the code counts whether there is even of odd of them. If even, then the following is not escaped, therefore a penalty of 0, otherwise the whole escape represents one char, therefore penalty = size - 1.


nah
the "else" part there is the general case. let's have extra syntax for stuff that deviates from the general case, not for all cases.


I still insist in keeping all the sensor stuff out of the GVRSceneObject itself. For now the InputManager (or the SensorManager) could be the entity end-users talk to.

This is something Danke brought up; we definitely need something along the entity-component-system. If we had that, you could have a Sensor component that is added to the sceneObject - we would not add any sensor-specific knowledge to the sceneObject. When we have that we will expose better apis.

On unrelated side-note - should this be called setSensor? Can we have more than one sensor attached to a sceneObject?


This should be taken into account: https://bugs.launchpad.net/puppet-keystone/+bug/1524927 that's currently causing us problem using puppet-keystone in tripleo


For blocksize 6, there are 20 possibilities, not 18.
Probably on purpose, but you did not use 101010 and 010101 ?


I was a bit surprised to see this. Are you familiar with Stream?


@Magmakom Well done, mate. Thanks for your fix.


The fact that mocha didn't detect the missing `var nid` here seems to indicate that we don't actually have a test case exercising this bit of code.  It would be good to add tests.


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


if the ssl_cert is enabled, is there, we should probably set the barbican's endpoint properly, as it's hardcoded as http is no host_href is given.


Noted, but I just hate double brackets, like ((0, 0))


super


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 847](https://teamcity.labkey.org/viewLog.html?buildId=291453&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 30 (8 new), passed: 217 Build time: 00:25:24
### Failed tests
```
org.labkey.test.tests.DataFinderTest.testImmuneSpaceStudySubset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudySummaryWindow: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyCards: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testNavigationDoesNotRemoveFinderFilter: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testBackDoesNotRemoveFinderFilter: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyCardStudyLinks: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testFinderWebPartAndActionShareFilter: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testRefreshDoesNotRemoveFinderFilter: <no details avaliable>

org.labkey.test.tests.RedCapTest.testMergeDuplicateNames: <no details avaliable>

org.labkey.test.tests.RedCapTest.testFailDuplicateNames: <no details avaliable>

org.labkey.test.tests.RedCapTest.testConfiguration: <no details avaliable>

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


Eek, good catch, @sit (my IDE defaults with `throws Exception` for my "new test" macro). Looks like the PR got merged. I'll submit another to remove.


can we make the host to bind to configurable? this would be problematic if HAProxy is in the same host, such as in some HA setups


Huh, this is a surprising patch. Were these four blocks of code in these two lookup_*_uncached methods not actually doing anything useful? Where is this work done now?


I think you misspelled "olive"!


Neckbeard! How fitting. :-P


Super


Tak


taaak


asdaw


awdawda


dawda


Thank you for reporting this redundancy. We will change this in the next release!


Why and when did Ridley get there?


dawdaw


Good catch. It's n². It could be done faster with a dictionary. like a HashMap.


wadw


Overall it makes sense. Just left some comments


Dope. Updated in dc3c279


awdaw


seeing unapply / unapplySeq always looks a bit odd to me (same as seeing .apply).


Not going to lie, I just don't like solutions of Day 12 that just lex the input - I think Eric was very kind in not putting unbalanced {}s in the strings.


closes #51


oops, this reversed one that didn't need to be reversed. may i recommend: `s/(\d+)\$/$\1/`?


I didn't want stream as it keeps all the previous values. Maybe Iterator, but it is mutable.


So this commit changes the handle_command in the case of async_put=true to be a synchronous put - do we then not really need async_put=true in the pattern match? And why is this different than the async_put=false path, as you do to very different things in those two cases, which seems odd.


@gabrielfalcao you probably want to revert this change to `https://secure.gravatar.com/avatar/3fa0df5c54f5ac0f8652d992d7d24039?s=64`

--
see https://github.com/gabrielfalcao/HTTPretty/commit/889408040334ad85c764d4eabcf7e27c21d5bd51


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


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


Watch your copy-pasting, this is an automated line!


I'm evaluating whether this could be done with just a plugin rather than having to modify `connectToStores` and `FluxibleMixin`. I was thinking that it may be possible to use a plugin that wraps `getStore` with a method that returns the stores with `addChangeListener` and `removeChangeListener` overridden to use the batch updater instead of registering directly on the store. It's a little hacky, but covers use cases where a component may be manually registering to the store rather than using the HoC or mixin.


That's me.
On Nov 15, 2015 11:44 AM, "Kanstantsin Shautsou" <notifications@github.com>
wrote:

> @n054 <https://github.com/n054> ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/KostyaSha/jenkins/commit/c7fe94deeb56c088dd24d9a799c877d23ea3ec9b#commitcomment-14403239>
> .
>


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


@ah3548 Please advise.


Using a Set as a key saves you a bit of twiddling below.


Oh, and also your solution is slow. Like extremely slow. [97% slower kind of slow](https://jsperf.com/fittable-mappropstoclass). But, oh well, that's FP, right?


becomes `distances.flatten`


Why does this need eval? Can you not just use JSON.parse()?


becomes `case pair => distances(pair.toSet)`


Aㅏ... 그렇군요. 나중엔되기를기원합니다 ~_~;
아참고로 저도이제 불머리로갑니다. 다시만나게되서반갑습니다 ㅎㅎ


Love that you did this. :)


Thanks.  I'll go over this.


이것도문제가아니라서 덧글을지웠습니다 ~_~;.
gpro,gk도똑같습니다 현재 다른개발자분께양도한상태입니다


@casualjim, I apologize but, what was the meaning of changing style in docs?

I've noticed the only change of all of my `models/*` was:

```diff
-/*
-Representation of Date interval
+/*Representation of Date interval
```

I consider there is a best practice for it, can you point me to it?


Wrong issue. Ignore last comment.  Sorry.


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


Basically I want to get rid of dynamic require's. Thanks for the reply, I will try and get back at this soon. I know a lot of people are affected by it, and unfortunately I'm at an incredibly busy time.


Have seen this


I'm finding similar issue just using ffmpeg from command line. Last night I saw your commit adding `Connection: keep-alive` -- I thought that fixed the issue because it streamed for hours....but today its crashing every few minutes again on bad key retrieval.


38 fail tests ;-( https://github.com/broerse/ember-cli-pagination/commits/cli2.2.0-beta.1


@hythloday: Thanks to you, I have found out there exists Json parser in the Scala stdlib. This is amazing. If I were doing this again, I would just parse and recursively traverse the tree and avoid those regexes.

But if {} were unbalanced, then it would not be a valid JSON, if that is what you mean. I claim this solution properly works on any valid JSON.


@EthanHeilman You can just use `./contrib/devtools/clang-format.py` to fix all of this.


@AmbientMalice Thanks, I Googled it but nothing came up except for YouTube rips of the soundtracks.

> GoodN64 is outdated

Aw, crap. What else don't I have, then?! :disappointed:


Want to take a stab at writing unit tests for this?

Also, the exception being caught should be type checked before doing anything fancy with it. If this is supposed to work with 3rd party parsers, it needs to be able to support that parser's exception format.

I think one way to do this would be some code like:

    if (this.parser.getExceptionMessage) {
      nodeEl.title = this.parser.getExceptionMessage(e);
    }

Then it would be up to the parser (which throws the exceptions anyway) to explain to codemirror-blocks how it ought to be rendered.

What do you think?


Arrgh - sorry, was completely my fault - I didn't push all the changes. Thanks for fixing that.


YES! The teapot lives!!


test


To make your examples working is trivial by checking for ````firstLetter = '$'````, too. In general, I don't want to strip named inner classes. Other developers use other logging format patterns and want to output the full class name of named inner classes. For tinylog 2.0, I plan to introduce an optional parameter for stripping named inner classes. I have already developed a prototype.

If you have other examples of anonymous class names, please let me know. tinylog 1.1 is still in beta stage and I'm open to extend or exchange implementations.


Oh, huh. Good to know! I will update my template with the info. :smile:


LGTM


nie zgadzam się


I bet we could use **.delete_if** so something like:

```ruby
@entries.delete_if {|entry| entry.name == name && entry.phone_number == phone_number && entry.email == email}
```

Wow, that's a LOT cleaner and easier to read.


tutaj racja, rejczel joo


why only us-west-2?

also doesn't this need to be written in a way to be mocked out for tests?


dzień dobry


cref


Derp, this doesn't exist anymore. Ignore me


As of this branch, returning to Euler method. See #3


It has to do with the linter for go, golint is unhappy with another form


Weird. This is showing up correctly on my local machine in Vim, Sublime, Git, and DiffMerge.


Would you change this "IMBU" and all the rest to "Imbu"?


I don't know who did it but I'm proud of them.

Not because I like Ridley, just because he's Jigglypuff's best matchup.

It's a puff buff without actually being a puff buff.


You can take a look here:

https://github.com/golang/lint
https://golang.org/doc/effective_go.html#commentary
https://github.com/golang/go/wiki/CodeReviewComments


This is the recommended way to name fields to avoid conflicts with getter names (see: http://docs.scala-lang.org/style/naming-conventions.html). With the emphasis on immutability in scala, this comes up fairly infrequently though.


@bboreham, wrong?


I have a better fix for this issue.  It uses the resource pack for sounds, rather than hardcoding different creatures to use the same sounds.  See:

https://github.com/CplPibald/spider-queen/commit/5fef19bf0b66bd747fb28d4315948874e3311ef6


Furthermore, you have a duplicated name here. I assume this happened just by addition of `curry`.


@janlav
Can you confirm the following:
I have the impression you also made it compliant with the instructions in #52


Fair!


101010 and 010101 are merely repetitions of the 2 bit-length patterns  already used for the threads with idx 1 and 2 therefore I did not consider them again on purpose


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


I guess you are talking about this line, sorry it was for debugging.


These things should not be happening on the gl thread. What is the reason why they are now?


@JackpotSHUnited Rest.


Yes, all of these are immutable sets.


What happened here?


That's what I mean. Ridley dies to Rest at like 20%, and he's a huge target. EASILY Jigg's best matchup.


Cool, can you just change the empty block with a  `nil`? (The completion handler is optional as per https://developers.facebook.com/docs/reference/ios/current/class/FBSDKAccessToken/)


To jest przykładowy komentarz do danego comittu.


To jest przykładowy komentarz do linijki


To jest drugi komentarz do tego comittu


I'll be sure to test.


Komentarz do innej linijki


Inny komentarz do innej linijki


The commit is mentioned in my deletion commit: 8ffa1ee


Is this the right information?


Jeszcze jeden komentarz


Will remove all of these.


Please take another look!


Shouldn't it be like `git+ssh://github.com/DaQuirm/nexus#master` ?


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


I wasn't suggesting narrowing the code to my view, but rather extend my view ;-) Thinking about it, I think I'll prefer having inner classes names too. Your approach to use a parameter in 2.0 seems rather sensible.


Yikes... I still new to testing on this, so I'm not sure how to run them and look through them.  Is that through ember-cli?


Can you change this to `$('.active:not('li')').removeClass('active');` to it doesn't un-highlight the navbar?


VerifyTriangleInequality


Jeszcze kilka komentarzy do linijki


I jeszcze jeden


I know. @JackpotSHUnited


I jeszcze jeden kolejny


SInce this commit, uchiwa-web does not work well on my Galaxy S4.


I forgot to add to the commit message that the commit would have broke incoming call with old blobs, remember on Slack? And please do not add responseOperatorInfos, it's not needed at all here!! >.<


Can you call this "TestSpecificMetric"


Why device-specific? it's the same for both


Well, either me or @antoinemarchal didn’t understand what you were suggesting at all… I don’t understand commits starting from 0bbb4aa7c1971d8ad2f6cce7cd5fef59626d5cf0. Why start became load, why stop at n_iterations rather than the desired number (for instance, to increase output frequency easily starting from there), etc…

I’m rather going to sleep and look at this tomorrow.


I don't know, could be :crying_cat_face:


saw approval from javajolt
at https://github.com/basho/riak-python-client/commit/7885c90715a2e7ae95999db51b1d6102b8bd8467


merging basho/riak-python-client/features/lrb/riak_pb-submodule = 7885c907 into borshop-integration-418-features/lrb/riak_pb-submodule


basho/riak-python-client/features/lrb/riak_pb-submodule = 7885c907 merged ok, testing candidate = eef40ee0


Can we just make an array of paths, e.g. [nested, flat] and return early when first match is found? Styling tip, but should look cooler :dancers:


With DescentModeCoM = 0.0, 0.0, -0.06

I had an L/D of 0.185 at beginning of re-entry (~0.05G of decel)
0.17 @ 5 km/s
0.15 @ 2.5 km/s

This appears to be a reasonable bracketing of the cited value


I am almost new to testing `ember test` will run the tests.


Na wszelki wypadek wiecej komentarzy


And can you rename Top1000Test to TestTwoElementMetricPropertiesTop1000? :smiley_cat:


Bulky, but necessary to preserve italics. Without tags would be just the last few lines


OK, found this right after.


https://github.com/AOSP-JF-MM/platform_device_samsung_jf-common/blob/aosp-6.0/loki/Android.mk#L31 probably needs more work


Jakiś komentarz


Pińćsetny komentarz


I pińćset pierwszy


<b>100% completely random names!!!</b>


Pińćset drugi komentarz


I am totally the bestest at guessing! \o/


I'm with you. You can get an ephemeral stream (without pulling in Scalaz) by calling .view on a Traversable. But I can see that's not appropriate here.


I didn't know you could do this! That said, I think it's slightly better and more concise just to do `rng=np.random.RandomState(seed)` when you call `dist.sample`. Not really a big deal, though.


fixes #5


can this be combined with the if condition above? the predicate is the same


I don’t agree with you @cphyc, but agree with former points of @antoinemarchal. The n_iterations limit is here because I don’t want the code to run indefinitely if it does ever success. And I don’t want to kill it unproperly if it does.

However, I want to use `stp_value` to control where it should stop when I want to look at what happens there.


`params` will always be truthy here because of line 534.


this is pretty big; can we move it to a helper method?


A proposed solution to #420, to make turning logs public or private trivial even when Lychee fails to work.


https://github.com/AOSP-JF-MM/platform_frameworks_base/blob/aosp-6.0-test-caf/telephony/java/android/telephony/RadioAccessFamily.java

There are groupings, so this commit it's not necessary. You're basically duplicating them


All trait impls have annotations now (this work was done in https://github.com/rust-lang/rust/pull/29152 and https://github.com/rust-lang/rust/pull/29083), so there's no need to infer them from annotations on traits.
But this is not important anyway, because annotations on impls are not currently used during checking phase and serve only as documentation.


Maybe I wasn't clear, I think it would be better if we had a `start`, a `restart` and a `load` actions, the two first being what you implemented and the latest skipping the part that's just reloading from the file `tmp.dat` (not overwriting it). It would be easier to debug (you create a `tmp.dat` and then you just do `./simul load` as many times as you want).

As for `n_iterations`, it *is* the number of iterations we want to do…


In `./tests/dummy` you will find the app where test run against. If you like we can perhaps help @mharris717 to fix this.


can hasVisitedTrail return false if can't use the feature?


`{ "foo}bar": 3 }` is valid JSON, but doesn't terminate.


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


Это мрак!

```Criteria``` всегда была независима от представления данных и роутинга к ним. Какие запросы перенаправлять на слейв должна решать имлементация ```QueryInterface```. Что бы передавать параметры, специфичные для реализации, был сделан параметр ```$meta```.  Can you refactor that?


"'Twas brillig, and the slithy toves did gyre and gimble" :-)


:+1: :sweat_smile:


Bulky, but necessary to preserve italics. Without tags would be just the last few lines. Taken from here: http://superuser.com/a/685243/221939


So you've got two variables that does the same thing (stop the simulation). The only thing being that until now, `n_iterations` doesn't stop it correctly…
It's confusive and error-prone (according to me). But let not waste time on this, do as you want, I'll adapt.


Please keep the same coding style, i.e. also use opening and closing curly braces for one-liners, as in lines 1370 and 1379.


looks good, some nits. I'll merge it in tomorrow.


:birthday: :balloon: :smile:


This is just to tell the user he failed at giving the arguments required :)


Why so/in which way will it depends on that? `compute_variables` is already called before each integration of T and after each cycle of integration…

In fact I think we should just move some `compute_variables` calls in order to do less of them with the same result, but that what @FranckObspm said to be wrong… But since we almost do the same thing…


Assignment Branch Condition size for create is too high. [15.3/15]


Good catch.


Well, why not.
I don't mind.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.948](http://teamcity.monogame.net/viewLog.html?buildId=12317&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 558, ignored: 6 Build time: 00:29:44


Oh, you're absolutely right.


.war. I like it.


good catch


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


This array key should read `conflict`, right?


Fixed here a3ba02a809a7a00f227601ca426392b0b6097cef


It seems like on the chrome browser i am using, the textarea wasn't being displayed as expected. (I'm using Ubuntu 12.04 LTS and the latest Chrome v47) .
![copy_clipboard](https://cloud.githubusercontent.com/assets/7693347/11794982/0b165046-a269-11e5-962c-f9491ea547ad.png)

Could you try using 'form-control' class instead of 'well well-sm' for the textarea, wrapping it in a 'form-group' div ?
```
<div class='form-group'>
  <textarea id='textAreaCode' class='form-control' readonly></textarea>
</div>
```

With it perhaps you could also remove the width:100% in style.css (since it is default for form-control) along with the copyText margin attribute.
Though I am not sure how it will appear on your browser and if everything would work fine. Do let me know in case you have any concerns.

Thanks!


Actually we don't (we don't update variables between an S and a T integration).
I've checked and we don't get the same output. The quickfix is to recompute the variables between S and T integration.


Why are these added as deprecated?


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


Well, the apache config was working fine before letsencrypt modified it. Haven't had time to figure out exactly what's wrong yet.


Consider moving `destination-expression` to a constant in `IpAdapterParserUtils`.


Now that' a big patch.
Having this in at least two pieces would be nice: one for the port to Qt5, one for the session support.


38 failing tests https://github.com/broerse/ember-cli-pagination/tree/cli2.2.0-beta.1


Go ahead and add method of obtaining redshift and source of measurement?  We could split this into 2 attributes:  'zmethod', spectro or photo; 'zsource', SDSSDR9 or 2dFGRS.


Issue5276 (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1439 of 1439 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.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.51%25-green.svg)


@avimehta,

How do you run changes like this in your dev environment? I've hacked the validator to allow amp-analytics but when I pull down this commit I get validation errors, specifically on the new <script> tag.


makes it run in background


oh god...i guess shipit....


I think method name `addSocketExpressionToConstructor` should be changed to `addSocketDestinationToConstructor`.


you can run anything from console ^_^


Why?
#25
#32
#34
#35
#72


Is 'main table' referring to the targets table?


Issue5276 (wfbuhl) - i386-Windows-7-VisualStudio-12: OK (1941 of 1975 tests passed)

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


Maybe I am oversimplifying but I imagine the end user getting the event whenever the collision is detected by the framework. Now everybody needs to go through these hoops to get the result and it doesn't seem like a stretch having all this in the fw. I mean how any random gvrf user know how to write a proper onSensorEvent? Other than copying and pasting from this sample?

isColliding() use should also be hidden in the framework as it is likely this api will change or completely disappear in the future.


Ah, right. I will make this change if/when there's something else to go with it.


Symfony 3.0 back to ArrayChoiceList


Le concept me plait bien, et je me permets une revue de code:

~~~python
class Interface:

    def __init__(self):
        self.remember = None

    def overwrite(self):
        if self.remember is not None:
            return self.remember
        choice = None
        while choice is None:
            answer = input("Overwrite ?").lower()
            # Look at the first char for the answer
            if answer[:1] == "y":
                choice = True
            elif answer[:1] == "n":
                choice = True
        # Look at the alst char to remember
        if answer[-1:] == "*":
            self.remember = choice
        return choice
~~~


Yes - I was curious why `BASE_PACKAGE` was used here...


Well, it's a matter of opinion.
I'd prefer do not extract non-public constants. It's faster to type directly without using of variables :smile:


should we have a constant instead of hard coded values?


Looking forward to this!


Why so? When iterations reaches `n_iterations`, the while condition became false and the program stop normally (except that I see a case where it wont stop, i.e. when trapped in the `T_converged` loop).

Also, what happens currently if you do a restart without changing `n_iterations`? It stops right away I suppose? I had in mind we should write `n_iterations` in `stp_value` on restore, so that it works OOTB.

And the (arg == 'load' .and. iteration > n_iterations) could be reduce to the second term, even when not in load mode.

Anyway, I’m going to sleep and will look at this tomorrow.


Good catch!
:+1:


There was another comment in the FirstUpload.txt file "De ce mi-ai sters fisierul ca te tai :))"


If there's a reason the prioritization order of AES128 vs. AES256 has changed, it should probably be reflected in https://wiki.mozilla.org/Security/Server_Side_TLS#Prioritization_logic with a relevant citation.


That's because an old Spring version wasn't able to work with `Class<?>` directly.
Some legacy code. We prefer do not change it until the side-affect like this our work.


"Added a cosine similarity distribution, which is the distribution of the norm of the dot product (cosine similarity) between two random vectors. It is useful for setting ..."


Please see related comment below


You use `listenBefore` instead.


Please see related comment below.


The commit message says it adds rebar3, but this line is for the next commit in the pull request. I'm not sure you want the history to be clean, just making sure you are aware of this.


Ok, got it now! Sry :)

On 12/14/2015 11:05 PM, Bruno Pagani wrote:
> Why so? When iterations reaches |n_iterations|, the while condition
> became false and the program stop normally (except that I see a case
> where it wont stop, i.e. when trapped in the |T_converged| loop).
>
> Also, what happens currently if you do a restart without changing
> |n_iterations|? It stops right away I suppose? I had in mind we should
> write |n_iterations| in |stp_value| on restore, so that it works OOTB.
>
> And the (arg == 'load' .and. iteration > n_iterations) could be reduce
> to the second term, even when not in load mode.
>
> Anyway, I’m going to sleep and will look at this tomorrow.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/M2-AAIS/BAD/commit/3a2bb3c8829fe8135a5710f5b38df634e6aa33b5#commitcomment-14971558>.
>


We do it in do_timestep_T. So we do.

The output changes because currently we output before this update, but the simulation state itself isn’t affected.


Damy radę.


Kto jak nie my.


Sa nu se mai repete, ca pun anaf-ul pe tine!


its actually cm's Cclock :-p

but its cool


It is, I can assure you (it doesn't crash at the same step…)

On 12/14/2015 11:08 PM, Bruno Pagani wrote:
> We do it in do_timestep_T. So we do.
>
> The output changes because currently we output before this update, but
> the simulation state itself isn’t affected.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/M2-AAIS/BAD/commit/54011a484a3be728f18dedc4c54b90ac03fb2f0d#commitcomment-14971629>.
>


You'll probably want parentheses around `new_root` and `put_old`.


It does - the event will bubble up. The problem manifests when that element is clicked though in Firefox.


Now I see your point.
Maybe some TODO here would be helpful?
Otherwise I don't see any concerns.
Feel free to merge.
Thank you for patience to review it and believing in me that there was an issue anyway :smile:


Just curious - why new variable here?


`experiment` is a getter that checks that the values is not null, then returns it. We call that once above on line 870, then refer to the local variable `expt` thereafter. This was the way it worked before too, except there were a few repeated calls to the getter instead of using the local variable. Of course, the overhead of a null check is negligible, so we could do away with the local variable entirely if we really wanted to.


This causes this assertion failure
```
>> make object! none
src/core/n-data.c:1268: VAL_SERIES_Ptr_Debug: Assertion `(VAL_TYPE_Debug(v) >= REB_BINARY && VAL_TYPE_Debug(v) <= REB_LIT_PATH) || (VAL_TYPE_Debug(v)==REB_MAP) || (VAL_TYPE_Debug(v)==REB_VECTOR) || (VAL_TYPE_Debug(v)==REB_IMAGE)' failed.
```

I don't think `none` was allowed as a spec to make object!, is this intentional?


(Pewnie wszyscy inni)


It seems there is a CI build failure with this commit:
https://travis-ci.org/wxWidgets/wxWidgets/jobs/96610761


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


Ale to nic.


LGTM apart from the typos in the Jabberwocky quote.


usethis


Niedługo święta.


A screenshot of search results for the following URL:

http://localhost:8080/exist/apps/hsg-shell/search?q=nixon+AND+china

> <img width="1149" alt="screen shot 2015-12-14 at 5 11 32 pm" src="https://cloud.githubusercontent.com/assets/59118/11795654/d4c7791c-a285-11e5-9cd0-14cd44de7a74.png">


Choinka.


Prezenty.


I prefer to put member variables behind a getter/setter even if, as here, we effectively are exposing the variable. This way we _could_ add checks that when you set a new value it is non-null, or when you get the value and it is still null we throw an exception with a helpful error message.


Awesome!


Kolędy


Guess this is because sf2.8+ errors?
Would you mind doing a pr?


I projekt na WDI :)


TY BRO YOY

2015-12-14 23:22 GMT+02:00 jhnsmth <notifications@github.com>:

> @Magmakom <https://github.com/Magmakom> Well done, mate. Thanks for your
> fix.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jhnsmth/restplay/commit/a533a92e08fc25046ae2383c92bb3a176a14c36c#commitcomment-14970469>
> .
>


OK, found it. That’s because timestep are also computed before the calls to `compute_variables`. Actually, that’s an issue, because timesteps are supposed to be adapted to the current state. Not sure of the proper fix though.


We should reference an existing/established standard rather than try to write our own for this. If we think


Ja tylko tak commituje.


Znaczy kreatywnie.


So why allocating a huge buffer on the stack then?


`tcp://` -> `udp://` ?


ack


@jleonardw9  I have our dev server configured with production-like settings. After installing the latest from master which upgrades to Rails 4 and Ruby 2.1.5, I needed to keep config.assets.compile = true but it seems like the correct answer is false. I tried a few things but was unable to get them working. For my understanding, why did you disable the asset pipeline with config.assets.enabled=false?


It's not **new**, it's **local**.
The `this.waitForAck` isn't `final` and can be changed in between :
```
countdownLatch = new CountDownLatch(ackCounter);
```
and
```
if (!countdownLatch.await(this.ackTimeout, TimeUnit.MILLISECONDS)) {
```
So, we would end up with NPE, if `this.waitForAck` was `false` before, but becomes `true` before that second `if`.


Or better "Added a cosine similarity distribution, which is the distribution of the cosine of the angle between two random vectors"


ack, who did the `__str__` before anyway?


#5252_TinDiscrepancyNewVRFModel (rongpengzhang) - x86_64-MacOS-10.9-clang: OK (1977 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.95%25-yellow.svg)


I made a commit to enable this in KernelAdiutor and made a pull request: https://github.com/Grarak/KernelAdiutor/pull/311

I also built with the commit: http://files.oceighty.co/other/KernelAdiutor.apk

I'm not sure if you have your own kernel app, but I just thought I'd let you know.


Symbolic link from where to what? It depends so much and I know at least 10 ways to break both the old and the new version of the code. Please be more specific, and present a case where the old buggy code works better than the new version and I'll engineer a new version in that case.

I am happy to help you improve your code to make Redux framework better for everybody, but I'd rather not put time on something that is off the point.


ack, that's a good one too


Had to resubmit because I didn't realise I need to commit the gemfile too.. and that caused the tests to fail >,<


Let's do that tomorrow morning.

On 12/14/2015 11:13 PM, Bruno Pagani wrote:
> OK, found it. That’s because timestep are also computed before the calls
> to |compute_variables|. Actually, that’s an issue, because timesteps are
> supposed to be adapted to the current state. Not sure of the proper fix
> though.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/M2-AAIS/BAD/commit/54011a484a3be728f18dedc4c54b90ac03fb2f0d#commitcomment-14971737>.
>


You're right - I missed the second parameter on the two-parameter macros such as mbox, fifo, and bank.

I can do another rebase to fix that later today.


:smile: It doesn't matter! Can even be `foo://`: http://stackoverflow.com/questions/2345063/java-common-way-to-validate-and-convert-hostport-to-inetsocketaddress.
Although I agreed with you: the context-specific name would be better.


I thought Scala tries to make this nicer by allowing argument-less methods to be called without `()`.


RedispatchOnCondition is a lowest-level method that will be tried if no other method applies. That is, if there is a group that has no method for `Socle' this method fill test finiteness and IsNilpotentGroup and if both are true go through the method dispatch again.

I presume you want to test this before trying other methods. For this you would need to rank your method  higher and bail out with `NoMethodFound' if the condition does not apply.


this adds the requested feature and fixes #240


Since this was introduced in the previous commit, you might just want to squash them.


This is actually a pretty common pattern in Go for one method interfaces (see [http.HandlerFunc](https://golang.org/src/net/http/server.go?s=41267:41314#L1408)). It just means you can provide a simple function that implements the interface, which comes in really handy when you want to wrap something with middleware in a closure.


private static final?


Then we lose the height here?


Why do people keep doing this? this completely breaks arc for people. It should likely be removed completely if this keeps being allowed into the repo @bfontaine.


final?


Removed an unnecessary "pwd" command in the script


I forgot war archives existed lol


What about initializing this in the controller?


#5252_TinDiscrepancyNewVRFModel (rongpengzhang) - 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)


Good job!


I think there should also be an if statement for gcc version >= 5.0
It won't compile with boost 1.55


By the way David Nolen said that patches for this are welcome :D !


Awesome work @LostKobrakai  !


Just for the reference.
This is no new regression. The missing `UA_ChannelSecurityToken_copy(&response.securityToken, &client->securityToken);` for `renew` was in the first commit of this function: https://github.com/open62541/open62541/commit/624f7089fa0e8093142787f510b516e74abefd76


the potential for infinite recursion leading to a panic is still here. this should be rewritten to simply use a loop instead of recursion, which btw also uses less resources.


It was just icons left over from my personal build.


Just curious, is there a particular reason why this is before `process-goog-import`? Just because in the file is before `process-reload` and that triggered my (maybe dumb) question


To quantify just how awesome this is: Before this commit there were 783 compiler crashers that were failing. Now there are 74. So this commit fixed 91% of the outstanding compiler crashers.


This code change captures the case where the DocumentClickCapture Click Listeners are attempting to listen for the click event.  However, when we call to see if the Global Variable for that click event is true or false the javascript call never returns because the webpage in this case has been blown away--and, presumably, so has the Selenium JavaScriptExecutor.

The call to getGlobalVariable in theory should be a quick call.  There is no significant code to execute other than to check the value of a window global variable.  Thus, I felt safe to put this call into a thread with a timeout of 1000 milliseconds.  However, the 1000 milliseconds is rather arbitrary.  It may be safe to make this significantly shorter.  Alternatively, calls to Selenium on Remote Servers may require a higher timeout value.  This needs to be evaluated more closely.

There may be other locations to 'fix' with the new js_executeWithTimeout function.  However, we should only 'fix' calls that we know are supposed to be fairly quick and immediate.  Some examples to be evaluated for fixing:

    js_getErrorCode()
    js_getErrorObject()
    js_getJSDebugArray()

Ideally, anywhere we call getJS().executeScript(code) directly should be evaluated for whether it is known to be quick, or needs to be allowed to run indefinitely.


I wasn't able to get them running completely today, but I did open up an issue on emberjs, because I think this is actually an Ember bug at this point. https://github.com/emberjs/ember.js/issues/12713  If they respond that it's not, I think I'll work on a fix for here.


this comment doesn't say anything that the line of code doesn't already say, it can be removed cause the code is clear enough.


If you want/have time, you could add `defrecord` to `alterable` tests with `:reload` and `:reload-all`... this is already pretty solid testing :D Great job!


I thought only newer browsers shipped with this. That won't work in IE9, will it?


This should be:
``` C
snprintf(sys_path, _MAX_SYSFS_BLK_PATH_STR_LEN, _SYS_BLOCK_PATH"%s", sd_name);
```


Some history -

The reason why we have an ExtractionResult and SimpleExtractionResult is because the users of KFileMetaData extractors typically needed to save the data instantly or had to slowly stream out the data. Metadata can generally be saved in memory, but text cannot. Specially with large PDF/ebook files.

We then landed up adding a SimpleExtractionResult since it was convenient to have.

In the case of writeback, since we will never be writing the text content of a file back, and only the metadata. We can just have 1 class - WriterData.


Why move this to the presenter? Maybe double check with someone else, but I would only expect to see this here if there were some string logic "if this, then that", otherwise it's just moving language from the view to the code.

Maybe a translation instead if you don't want that string in a template?


`class TagLibWriter: public Writer`?


This magic number probably caused this test failure: https://s3.amazonaws.com/archive.travis-ci.org/jobs/96846583/log.txt


LGTM so far.


GitHub will actually send back the same error regardless (presence of the `X-GitHub-OTP`).


Okay, I changed it as part of #223.


You probably don't want to class this result.

Also, the reason Extractor is given a pointer is because it modifying the result, and I wanted the semantics to be quite clear. Though I could have just passed a reference. In your case, a const reference should be fine.


This won't render correctly, unfortunately. Need:
```
.. note::
    It is a very bad idea to use the same ``session_id`` for
    every page load; you are likely to create scalability and
    security problems. So ``autoload_server()`` should be called
    again on each page load.
```


The output in the test runner when this executes is...

```
Test Case '-[iOS_OperationsTests.CancellationOperationTests test__operation_with_dependency_cancelled_before_adding_still_executes]' started.
Delay for 2.0 seconds: Did cancel.
Delay for 2.0 seconds: Initialized -> Pending
Test Operation: Initialized -> Pending
Delay for 2.0 seconds: Pending -> Finishing
Delay for 2.0 seconds: Did finish with no errors.
Delay for 2.0 seconds: Finishing -> Finished
Test Operation: Pending -> Ready
Test Operation: Ready -> Executing
Test Operation: Will Execute
Test Operation: Executing -> Finishing
Test Operation: Did finish with no errors.
Test Operation: Finishing -> Finished
Test Case '-[iOS_OperationsTests.CancellationOperationTests test__operation_with_dependency_cancelled_before_adding_still_executes]' passed (0.051 seconds).
```

Which is what we expect - the cancelled operation goes `Initialized -> Pending -> Finishing -> Finished`.


oddly short line


There was some discussion on the matter on mozilla.dev.tech.crypto:
https://groups.google.com/forum/#!topic/mozilla.dev.tech.crypto/fohi1x40UKo

Even now I don't think there is extremely strong consensus either way, but I would probably sum it up as:
- There are some minor concerns over it, see: http://blog.cr.yp.to/20151120-batchattacks.html
- The NSA recommends AES-256 over AES-128 in Suite B, due to concerns about quantum crypto
- AES-NI instructions fix most of the problems with AES-256 and make it stupid fast. This is only in the Modern recommendations, which is intended for systems that have AES-NI instructions. AES-256 is only 19% slower on my Macbook, and a single thread is able to push ~2.8 Gbps/sec of AES-256.
- Intermediate still recommends AES-128 over AES-256, and any site that is interacting with systems that are hurt by the ~20-30% speed penalty will almost certainly be using Intermediate/Old.
- I personally get hassled over this recommendation a couple times a month, so clearly perceptions matter

I'd also add that Chrome and Firefox would still get AES-128, since they don't request AES-256.  For those manufacturers that prefer AES-256 over AES-128 (Safari for sure, I think Edge as well), they can get the cipher suite they have chosen as best.


Hmm. Why is this needed?


Looks good to me. Alex, ok to merge?


@raindev somehow I changed the order and didn't break anything


Let's remember to reenable coverage once we get it to work.


I don't think this needs to be virtual. No one is going to inherit from it, right?


may need to change this to another gettext, need to reread the docs


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


I don't think we can write the type back.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.949](http://teamcity.monogame.net/viewLog.html?buildId=12321&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 558, ignored: 6 Build time: 00:32:06


Interestingly, when I put the getGlobalVariable call into a separate spawned Thread and ran it DID seem to fix the problem.  However, I expected to see certain debug messages indicating the DOM may have been gone and the event is assumed to have been fired.  Instead, the debug log suggests everything went just fine without any issue and possibly without hitting the new thread timeout.  More investigation of the log and flow is required to confirm.


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


This is also probably not required.

In the end all you need is a `QMap<Property, QVariant>`


When is 1.3.8 going to be released?  that bower.json addition will help a lot


OK. Thanks for this code. It's more clear for me now.
So what is the next step? Are you going to incorporate these changes into my original PR?


this removes the default `all` groups. not sure what the repercussions are of that. i was thinking i could add that into the new `MultiSelectGroupField`, but wasn't sure if that was left out by design


I opted not to since it wouldn't give you much more information than you already know. The message we return from the API is a [little nicer](https://github.com/remind101/empire/blob/auth/server/heroku/errors.go#L24-L28).


Organized Parameter.h and Parameter.cpp. I also moved all R only functions into a block where the C++ compilation will ignore them. Because of this several other spots in the other parameter objects have similar blocks at the moment. The code compiles and runs in C++ and R. Some C++ functions were calling R functions, leading to misleading results definitely in C++ and possibly in R.


#5252_TinDiscrepancyNewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1983 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.95%25-yellow.svg)


Use `properties.value` instead of `properties[]`. The latter modifies the host.


Any idea why ``heap-size`` is negative?

```
IN: scratchpad heap-size .
-760
```


this could probably be written to better utilize the ES apis


This seems like a good start. Perhaps you can implement one for pdfs and ebooks as well?


LGTM


How about checking that the missing party isn't redundant here? (That may catch bugs in the combinators.)


this should be `mesos_version` not `cmd`


Hi @rpocklin !

I found this very useful, I was about to work on this feature as well by my own, but did a little research on the forks and found your change. Have you already did a pull request? there are more people waiting for this, maybe it could be included on next official v0.3.x.


So if there are existing users of devicemapper package and if they are using loop devices, they will be broken and they will have to change their code to move to loopback package?


finished


This is work in progress, so please go ahead and do it.


Why host and port doesn't have to be optional?


By popular demand `installPackage()` is here!


I know @Rage17


Happy to see 4.0 closing in :dog2:


Could this maybe come out to a named function? These types of click handlers tend to grow.


Misspelling: ChatVisibilties -> ChatVisibil**i**ties


You can have a method named `currentBlock_=` and then call it like `inst.currentBlock = whatever` which looks just like assigning to a member. We could certainly make that change, but it's still a setter method, just with a different name and some syntactic sugar.


Maybe a rebase is in order?


Usually either taken all the way:

```
fn(
  arg,
  arg
);
```

…or in the case that an object literal is the last arg:

```
fn(arg, {
  prop: 'value',
});
```


See the `use` attribute definition:
```
    <xs:attribute name="use" use="optional" default="optional">
     <xs:simpleType>
      <xs:restriction base="xs:NMTOKEN">
       <xs:enumeration value="prohibited"/>
       <xs:enumeration value="optional"/>
       <xs:enumeration value="required"/>
      </xs:restriction>
     </xs:simpleType>
    </xs:attribute>
```
So, it is `optional` by default!
Plus see here: http://www.w3schools.com/xml/schema_simple_attributes.asp


No las reemplaza, en el bueno es el útimo que subí de los tres que se llaman igual, pero cuando subí el commit que te envié se me olvidó hacer el commit antes subirlo.


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


Oh, isnt Cmake awesome


worldrenderer.func_181668_a(7, DefaultVertexFormats.field_181706_f); //might be 707 instead of 706


I didn't even know they were archives! You have one up on me there.


Now we see how having this in a separate function would help. We have an input `previous_poll_id` that can be passed in.


Yes it is


renderer.func_181662_b(x, y, z).func_181673_a(u, v).func_181675_d();


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


jak to dziala


I've been following through the tutorials - the code is great and clear.  Can I recommend one thing?  It would be great if each of these Layer implementations could have a link to a URL where there is a paper or other docs corresponding to the notation that's being used here.  I looked up LSTM on wikipedia and learned a lot, and the notation generally matches, but it seems likely the code is following some specific paper or other implementation notes.  So a link for each kind of layer would be really helpful to follow it more closely


Well.
Originally this is @ garyrussell 's code, so I'd like to see his opinion.
Although you are free to incorporate this my commit to your PR to play locally.
I might not be correct everywhere...


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


As we have talked in person, having the part of the MIME message that happens this problem in the commit body message would be great as well.


Not sure if I understand you here.
"m_iDvrLifetime" is a settings value and it's used as an enum, so values from 0 to 14 will be assigned for an enum from 15 values long {1day, 2day, ... ,forever}
This needs to be translated to the real lifetime in days, that's what I do here.
example when not doing this:
 1) user selected "2 weeks" as default lifetime in settings
 2) the enum "m_iDvrLifetime" will hold value "4"
 3) user adds a new timer with default settings (simple record button for example)
 4) a timer with a lifetime of 4 days will be added....


This is not entirely correct, it seems that willy do have the latest debian package 4.0.0 rc1. I think we should maybe name the version of the old stable, it is somwhere around bzr rev 4022 (posisbly plus a tiny bit) and still mention that it is not recommended for new projects.


Sure @blmstrm, I'd be happy to open a PR, thanks!


duplicated code, removed line


@jdalton Out of curiosity: Why did you remove the trailing `newlines` from the `HTML` files?


Sure. Will add.


Well this project.json is intended to be the one that restores buildtools itself, so my expectation is that this file should ONLY contain buildtools and its version, and then all of the rest of the dependencies for building should be inside buildtools itself. One thing we could potentially do is to create this file on the fly when you run init-tools.cmd instead of checking it in, so that when we upgrade buidltools we only have to upgrade the version in one place.


Incorrect. Just double-quote around the variables in the Array assignment.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.950](http://teamcity.monogame.net/viewLog.html?buildId=12325&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:12:14


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


If you include a ":" in the string given to any of Bitcoin's addr constructors, such as CNetAddr or CService, the constructor will build an IPv6 address instead of IPv4 address. For example consider the following constructor:

1. CService("250.1.1.1:8334") looks like an IPv4 address but is fact treated as IPv6 in Bitcoin.
2. CService("250.1.1.1:8334") has default port of 8333 not 8334.

Problem 1 is a readability concern, problem 2 means that tests based on port number do not actually test the port number.


Seriously, move the dlashes down to where you use this. Reduces mess.


The program should stop as it didn't find what to run, but I agree a Runtime exception might be better as well as changing print for log or error message in the exception. I'll change it


Good call. I need to tinker with the test cases before I really know what's going on, but I'll figure this out tonight


C'est pas supposé être ignoré par git les copains ? @davidlglt @cyrilchehovah ?


Good catch by whoever pointed this out to @aschleife.


You committed a conflict.


You can show off your results by doing some du here. Optional though.


Thanks, but I already fixed it is this commit: https://github.com/iLexiconn/LLibrary/commit/9e8e7fd8f3f3528768f5d78df4f4a8d1fcbffca1#diff-707448b7735f0e4b4f6f3dd863ca5a7cR97 ;)


genre tout le fichier application.yml ?


For consistency with other projects I manage. Nothing critical.


> Replacing g++ with CXX - I am not sure all those flags work for clang. Have you tested?

It's not necessarily for Clang. On many systems, the right g++ binary isn't named g++. For example, I use OpenBSD, on which the older system compiler binary is named `g++` and the newer version is named `eg++`. On other platforms, the version is appended to the name, like `g++4.9.2`.

The nice effect of CXX is that people can expect to specify the compiler used in all build systems. Not a huge deal, though.

> In any event, the CC commands are explicitly for the Solaris C++ compiler - those command lines would not work with a generic C++ compiler, and definitely not for straight-up C.

Makes sense.


I received 8 notifications from GitHub in your name, all saying "Git commits -s"


@xianhuawei for the extension or the class generator code?


Yes. addrs are assigned to buckets based on their source IP and IP. Changing these values so they are actually IPv4 addresses also changes the buckets they are assigned to and thus the expected output of the test.


Just did, and it breaks some tests.
Is it possible to have both complete and incomplete blame objects flowing to the same checking location (i.e., instrumentation point)? In which case we may need to be redundant in some cases.


:confetti_ball: Nicely done.


Shouldn't this be using `next_id`?


aaa


You left the out/kernel/zImage in the commit.


一个循环：variance = E(X^2) - (E(X))^2?


Because I missed it, nothing more.


`Poll.first.id` is always the "next poll"?


Oops. I always thought M_PI was a standard define in math.h. I guess not. Should be fixed now.


yeah


I think it should be possible to fix that so it doesn't happen, but I'm not 100% sure.


TeamCity DNN JavaScript Libraries :: CI [Build 123](http://teamcity.engagesoftware.com/viewLog.html?buildId=401&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


Jumping in kind of late but... good to see this implemented :) My only suggestion is that the styling be tidied up some - right now it is sort of hard to discern titles from text, and things look inconsistent on mobile. Maybe bold the title and indent the answer? Something like that.

Also for the multi-paragraph answers, there are <p> tags inside <p> tags, and the faq answer <li>'s aren't sitting inside a <ul> (I think these errors contribute to the inconsistent look on mobile).


TeamCity DNN JavaScript Libraries :: CI [Build 123](http://teamcity.engagesoftware.com/viewLog.html?buildId=401&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:18


@oritwas missing encode / decode for realm id?


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


Yup, just re-pushed. Should be fixed now.


jajajajajaja


Fixed, waiting to merge.


Doesn't matter, this branch isn't used


No, currently not.


最好就用heap的标准操作 push？
append是最后一个吧？这儿为什么是fix 0呢？


Nice find!


Fixed an issue with accessing the Broker via its defined path.
Corregí un problema de encontrar el Broker según su ruta de instalación


Start Of The Chooser Function


Fixed in 48d377c.


I don't claim to be an "English major" in university but as a person
with English as my mother-tongue I can say that I've never seen "till"
written in a phrase meaning a duration.  I have seen "til" written, but
it is a contraction and correctly should be written "'til" (with an apostrophe
to indicate the missing letters, a shortened form of "until").

I'd use "until" but you can certainly ask on gramps-devel
(or gramps-users) as there are certainly other English
speakers on both.  I don't claim to be an expert.


to use the chooser start with how many choices then all the choices


Just change these as required and write 'setlist' in game chat, it will generate a console output to copy into the units


Fixed in 3753273.


Ah, didn't know that.


Yep. Fixed this (I think). See commit message.


yeah, sorry forgot to change it back. Will do before I submit my PR.


i am beginning to hate GitHub. I haven't wrote these lines. I only fixed Pressure Plates and rails

By the way: $down is needed.. ehoever messed that up -.-"


Great idea, putting it on my list of things to do.


Yikes, done.


This was removed on purpose https://github.com/TryGhost/Ghost/issues/6106#issuecomment-159773139:
> make localhost / 127.0.0.1 always allowed, not just in dev mode

I think the reason was to improve the first run experience for users trying out Ghost from a ZIP download?


You are going to get AddinRegistryUpdated shortly after client init -- is it enough to rely on that?


Fixed in 5a161ce.


#5252_TinDiscrepancyNewVRFModel (rongpengzhang) - 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.81%25-green.svg)


TODO? What is to do?

Do you mean add a comment to say why we're skipping this adapter?


can you give me an example of how to use this.  lets say i have a directory called submodules within the middleman app that I want to ignore recursively


I think we need the `SessionInitEvent` to handle browser reloads as well (otherwise, in testing, the keyboard shortcuts aren't refreshed after reloading the RStudio page)


I am just working on this now. I have more conditions I am checking afterwards for tie breakers.
I'll probably clean it up once I finish writing the logic.


This should be CountOf for consistency.


I see the problem. Scala doesn't have something like python's properties where the setter and getter have the same name. In python we can do

```python
class Foo(object):
    # I have attribute bar
````
and do assignment, retrieval like
```python
foo = Foo()
foo.bar = 42
print("The answer is", foo.bar)
```

We can then later add stuff under the hood

```python
class Foo(object):
    def get_bar(self):
        <do stuff>
        return self._bar  # or whatever

    def set_bar(self, val):
        <do stuff>
        self._bar = val

    bar = property(get_bar, set_bar)
```
and consuming code doesn't need to change. I think Scala gets us part of the way there in the sense that one can write

```scala
class Foo {
  def bar = {
    <do stuff>
    _bar}
}
foo = Foo()
println(s"foobar is: ${foo.bar}")
```
The apparent "access" of the `bar` field actually invokes the `.bar` method. The `()` can be omitted in the method call because there are no arguments. This means that we can upgrade a field to a method without changing the code accessing that field.

I can't think of how to make that work for a setter without using macros (which now that I'm writing this sounds like it would be pretty useful).

@maffoo have I missed anything obvious?


Seems for me you said before that we should consider to add more exclusions here. Similar to this `vboxnet`...
If not never mind.
Although you can merge as is anyway.
A new failure will say us that we should dig again.


Okay, agreed!

On Mon, Dec 14, 2015 at 6:02 PM Kevin Ushey <notifications@github.com>
wrote:

> I think we need the SessionInitEvent to handle browser reloads as well
> (otherwise, in testing, the keyboard shortcuts aren't refreshed after
> reloading the RStudio page)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rstudio/rstudio/commit/85b2b098f8f4809bde296584fe52c3a5f0dd8c27#commitcomment-14972664>
> .
>


I don't understand how ext_method_cmp() is supposed to set ext_method.type without being able to parse "method"
Shouln't it be int (*ext_method_cmp)(struct evhttp_extended_method *, const char *);  ?

To be honest, I'm not following you... I think your version needs a pretty good documentation to be understandable.


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


why not multiline broadcasts: https://github.com/ddnet/ddnet/commit/4b70e195c12be3b3c95e5914159ab48e835761cb


Psst @ldenevi it looks like there were some uncommitted changes after the html files were compiled.


If we abandon ship on this we should make sure to delete this commit too.


#4994-ControllerWaterCoil-TemperatureAndHumidityRatio-control-active-when-it-shouldn't-be (mjwitte) - x86_64-MacOS-10.9-clang: OK (1959 of 1979 tests passed)

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


Thanks. Errors are mostly long lines/white space/unused vars - easy to fix. This does not check naming convention though. I guess we should stick to the standard CapWords for classes and lower_case for functions etc? These aren't respected right now, so it would require some changes but it's probably better to fix the style now rather than later.


Fixed.


@DanielSank, I was trying to explain previously how you do this for a setter, but maybe I wasn't clear. You just have to make a method with a special name with suffix `_=`. Fore example, if I define a method `def bar_=(newBar: String): Unit = { ... }` then this method gets invoked when you do `inst.bar = whatever`.


obviously, I'm missing something...


this proves to be problematic when using an ordered broadcast in conjunction with getResultExtras as getPendingResult() becomes true even thou finish throws IllegaStateException


you are.