Au passage je me demande si le test "if ($errorInfo[0] != 0000)" est correct syntaxiquement : il ne serait pas préférable de tester plutôt la chaîne '0000' ?
Car le problème que j'ai rencontré ne semblait pas lever l'Exception (ou alors cette dernière n'est pas remontée jusqu'aux logs).


sdf


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#TDKaob_7QOGsHhzoY-1fKA


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#e8UDqpVISRO6kogpD7-sig


agree, trying to merge and resolve the conflicts


Hello @rafaelfranca -- I was curious about this commit. Why do we use a file called `railtie.rb` in every module but a file called `engine.rb` in Action Cable?

Based on https://github.com/rails/rails/commit/346a752 I thought we would simply rename `action_cable/engine.rb` to `action_cable/railtie.rb`.

Thanks!


@jniles The handler function for the route POST /debtor_groups/


We should do a pass and remove commented out code like this, if it relates to prior bugfixes.


Ca ne gène pas PHP


Prueba


Error


jävligt bra kapitel ! :+1:


@jniles The handler function for the route POST /debtor_groups


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#E1dv0Xv4S0StPOYZNESHIw


I have split this commit


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#fKGCUZ-DRIOdH9Hbw3pSYQ


If MEM_read32(match) == MEM_read32(ip) is not fullfilled currentMl = 0;
And while still works:

            while ((match-back > dictStart) && (ip-back > iLowLimit) && (ip[-back-1] == match[-back-1])) back++;   /* backward match extension */
            currentMl += back;


Would you be able to add support of -v to the tcp transport, right now only udp takes the port specified


Either should work. I still work with minc-toolkit, not minc-toolkit-v2, so I'm more likely to notice if something is broken there.


@gabriele-v would you mind to sync the database folder to reflect your latest schema change?
@moneymanagerex/desktop-mmex-team  as fyi.


`nrecords` is exactly the number of records they are exporting... why the "more than"?


This segfault is only superficially due to lack of contigs in the header object.  It is because records contain many key and index values into data structures within the header associated with the VariantFile object with which they were read.  Writing currently only works when the input header is specified, though it can be altered from that point.  The missing component is header translation -- where a record is translated from one header to another so that all keys/indices are valid or reset to missing.  This next task is on my TODO list.


This change is not in master, only for illustration. All existing methods seem to reexport all of DBI, so this is probably something we want (and something that should be tested).


Thanks


This may be a matter of style, but I really don't like this extracting most of the pipeline into methods, and then calling those methods.  I think it's slightly less bad when the methods just return some tasks, but when they also take tasks in and set dependencies using those tasks... Since a pipeline models a dag, I think it's more readable to have all the dag construction in one place and to make it as clear as possible.


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


Much better.  This makes much more sense.


@jniles query string for getting locked debtor groups, unlocked debtor_groups or all debtor_groups


Yeah, but it's not part of my commit. I opened issue #6 a long time ago for this.


Yay!


@jniles test requirements for debtor groups


You're missing xdebug.show_error_trace here: https://github.com/xdebug/xdebug/blob/master/xdebug.c#L277

But, I don't see why you want to do all of this. If people have Xdebug going, let them have it!


As this is almost the same as the lines at the bottom of [`app/view/root/local_transaction.html.erb`](https://github.com/alphagov/frontend/blob/master/app/views/root/local_transaction.html.erb#L56-L66), could we make it into a shared partial?

Two differences are the value of `userAlerts` and the value of `postcodeErrorShown`.  Setting `userAlerts` could be an argument to the partial, but we might need to tweak the implementation for `postcodeErrorShown`.  For local transactions we just use the value of `@location_error.postcode_error` whereas here we don't.  `postcode_error` is 'validPostcodeNoLocation`, but we're setting `postcodeErrorShown` to `fullPostcodeMapitMatchNoLocationData` - I don't suppose we can change it to match what's in `postcode_error` could we?  We'd then get parity between this use of `@location_error` and the way it's used for local transactions.


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


It's better to name it
processMerlinNative.setName(processMerlinNative.getName() + "-native");
thus it will be removed by @AfterMethod (removeTestClassEntities();) as part of this test class.


Why are you returning 0 when a connection times out, and especially why are you returning 0 when the buffer is empty, while you return -1 in other cases? It seems to me that 0 is not discriminable from actual data. In the documentation it only says that -1 means error.


develop (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1529 of 1529 tests passed, 0 test warnings)

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


Добрый день, скажите пожалуйста, на каком основании вы предъявляете нам такие претензии и кем вы нам приходитесь? Можно контактные данные для связи?


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1529 of 1529 tests passed, 0 test warnings)

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


Because baseReference already fallback on current reference so it contain everything we need already and default is a lower dependency than current (which is implemented in oldcore).


Bom dia! Você colou o nome do "database" casadocodigo_javaee?
no projeto o nome do banco esta casadocodigo!
no livro não fala sobre a criação do banco no mysql até no capitulo que me encontro!
Acredito que devo criar, a base de dados com as tabelas, qual seria o nome do banco?
Sou iniciante em programação.
Agradeço qualquer ajuda!


Same as above. But here you don't seem to return -1 at all except if `recv` returns -1.


Correct Przemyslaw, my mistake ! :+1:

What I initially wanted to do was :
```
            if (MEM_read32(match) == MEM_read32(ip))  /* assumption : matchIndex <= dictLimit-4 (by table construction) */ {
                currentMl = ZSTD_count_2segments(ip+MINMATCH, match+MINMATCH, iHighLimit, dictEnd, prefixStart) + MINMATCH;
                while ((match-back > dictStart) && (ip-back > iLowLimit) && (ip[-back-1] == match[-back-1])) back++;   /* backward match extension */
                currentMl += back;
            }
```

In which case, the condition is guaranteed.
Somehow, brackets got lost in the copy.

I also wanted to do the same for first place :

```
            if (match[minml] == ip[minml]) {  /* potentially better */
                currentMl = ZSTD_count(ip, match, iHighLimit);
                while ((match-back > base) && (ip-back > iLowLimit) && (ip[-back-1] == match[-back-1])) back++;   /* backward match extension */
                currentMl += back;
            }
```

but here, the test is only a "hint", not a guarantee, so currentMl can indeed be 0 after count.
So let's keep the test.


No I did mean relativeReference.getParent(). This method expect to get the relative document reference. If it's null then it mean documentReference is equal to baseReference so it won't go that far.


Students/30421/Filip O. Sebastian Mihai/Digital School take II/Digital School/src/GUI/login.java

super sugestiv numele


This is a merge commit, so it has two parents - one (efbf792) is SR's branch, where SVG clipping works, and the other (6f19816) is ROLI's official branch, where SVG clipping is not supported.
If you're comparing this to 6f19816 then it is clear why you are noticing a difference.


Right was too quick and totally forgot to rename it. Thanks.


Oi Junior,

Você precisa criar o banco sim. Aproveitando, se você realmente está
começando, acho melhor investir um pouco mais de tempo no básico do Java e
Web. Para que aí sim, o livro te retorne tudo que você pretende.

Em qua, 10 de fev de 2016 às 12:41, Juniorcorpse <notifications@github.com>
escreveu:

> Bom dia! Você colou o nome do "database" casadocodigo_javaee?
> no projeto o nome do banco esta casadocodigo!
> no livro não fala sobre a criação do banco no mysql até no capitulo que me
> encontro!
> Acredito que devo criar, a base de dados com as tabelas, qual seria o nome
> do banco?
> Sou iniciante em programação.
> Agradeço qualquer ajuda!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/asouza/casadocodigojavaee/commit/e5198d60e604a1106c1f9508ccf5537ad66588d5#commitcomment-16006830>
> .
>


Added extra check here, to prevent adding execution to built-in flow


And still works on 2016, as said.
Thanks!


I'm probably not going to merge this.

Ticket link with valgrind logs: https://bugs.bitlbee.org/ticket/1248

Other than the design issue stated in the commit message, this doesn't fix http or socks proxies. And I wouldn't want to copypaste these nontrivial-yet-unsafe checks in every instance of a `phb->func()` followed by `phb_free()`.

Maybe what I want here is some sort of reference counting? So that the inner `proxy_disconnect()` doesn't free it because `proxy_connected()` added 1 to the reference count.

Or maybe get rid of the PHB struct before calling the callback, and hold func and data elsewhere.

[whoops i wrote this hours ago and didn't post it]


I thought so, but this will change the compression ratio (I'm not sure for better or for worse).


No, the initial probes will not cause a change in state, so they won't trigger the callback at all.

A recurring monitor failure (e.g. pacemaker_remote crashes on the remote node) will cause a recovery of the connection resource, and the stop from that will change the state. So the callback is where we fail incompletable actions in that case. We'll also get here for normal stops, but there won't be any pending actions to fail by that point.


I don't object to this, but I tend to think it's unnecessary when overriding a method (since return type must be a subtype of super's return type) and the method body is short.  Hence why I've been leaving them out.


Should this be private?


@duramato in the json the ' doesnt need escaped =P


Added extra check here to prevent setting provider to invalid value.
Is there a use-case where it's required to allow setting provider to yet unregister authenticator provider id?


not sure to follow how it could change the compression ratio in the second place,
since if `(MEM_read32(match) != MEM_read32(ip))`, then, in both writings, there is no check for `back` and currentMl remains at 0.


This seems bad to me.  This will presumably cause everything to run fine and all the outputs to get lost?


Added NPE prevention check.


Completely forgot about baseReference. Cool.


Note : restored `currentMl>0` test in 62ae262771c2ad1e0a8f993f9d4d44d19e64c463


второй комент


I think this should just use this._prefix from PicardMetricsTask, though we should fix how `prefix` and `_prefix` are named there.


Sorry, I'll keep that in mind next time.


Looks good to me, committing.


I wonder if `getExtent` itself should be responsible for retrieving the layerDetails (and having an async return type), instead of making the main.js function handle it.  If we're potentially grabbing properties from layers that aren't loaded (extent, metadata, description, etc), this file might accumulate a lot of this fetching boilerplate that seems more suited to the layer object.


оригинальный файл


Good question @claudiob - what are the differences between engines and railties? It looks like `Rails::Engine` just inherits from `Rails::Railtie`, and adds some extra functionality.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1283.b3809a79fb22ae0da2e2a8d85d38a7e51b353d57](http://build.palaso.org/viewLog.html?buildId=69057&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 960, ignored: 12 Build time: 00:08:13


Perhaps we have `private val prefix : Option[Path]` in the constructor and replace `_prefix` with:
`def prefix = this.prefix.getOrElse(PathUtil.removeExtension(in))`


`fr:`


You have removed possibility of currentMl=1 or currentMl=2 which will be backward extended to currentMl >= 3.


With lodash 4+, `return _.find(_countryList, ['ISO.alpha3', country])` would work too.


I think it should be `luaotfload-tool -u`


`perTargetMetricsPath`?


:sigh: misread that. I thought it was printing export_warn


According to https://docs.google.com/document/d/1eNpI_02NYb8K-x014Jo8iO6wjlpQQttDO3mVYZUXVtQ/edit#,
Array implements IArrayLike


ok


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1284.11a764bc561e73b8f4cb45963b0ab3915aba6fb2](http://build.palaso.org/viewLog.html?buildId=69058&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 960, ignored: 12 Build time: 00:09:07


I'm talking about the first place:  if (match[minml] == ip[minml])


Irado o logo em!


They were not tested either in the initial formulation :

```
            match = dictBase + matchIndex;
              if (MEM_read32(match) == MEM_read32(ip))   /* assumption : matchIndex <= dictLimit-4 (by table construction) */
                  currentMl = ZSTD_count_2segments(ip+MINMATCH, match+MINMATCH, iHighLimit, dictEnd, prefixStart) + MINMATCH;
             if (currentMl > 0) {
                 while ((match-back > dictStart) && (ip-back > iLowLimit) && (ip[-back-1] == match[-back-1])) back++; /* backward match extension */
                 currentMl += back;
             }
```


Instead of throwing exception at the client side how about exposing a configuration at the server saying "list.children.batchsize" or feel free to use a better name.  If some users don't want to worry about the batch size value then they can configure at the server and they can pass a value <=0 as an argument to this API.

Defaulting to 1000 children count. I consider 255 chars for each child znode name and the total in one request would be (255 * 2 bytes * 1000) = 500KB approax.


bien vu! C'est corrigé.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 256](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=777&buildTypeId=XbimEssentials_XbimEssentials) is now running


ah ok,
in the first place, I've restored the test as it was.


Can we make this an `Option` and use the auto-detect feature as standard?


@MagicDroidX I beleive this is causing the invisible tile bug.. ?


Might be also removed as it's not necessary anymore after adding createInstance method


Well, up until now a common practice to ignore SSL certificates has been to return null in getAcceptedIssuers(). How can I achieve this instead? Returning an empty array means accepting **no** certificates, right?


Yeah, replacing it would be another big project. We could use cluster messages to keep all nodes' crmds in sync, but the problem is when a new cluster node joins, or a network partition happens and is healed, or network connectivity issues interfere with the cluster messages. The simplest approach would be for the DC to send its entire remote peer cache to any newly joined cluster node, but corner cases would need to be handled.


Nice that this still solves the issue for people. Sad that this was never fixed in master. I even went through the process of submitting this patch through other methods and signing the proper releases when asked.

CC @sparsuram


I believe they are runtime dependencies brought in through reflection. maven-bundle-plugin would not pick these up on its own.


Irado mesmo


The need for this commit nicely summarizes what’s wrong with ES6 classes.


Also, if we're going to rely on the enum, why not just take the enum value instead of a string?


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 256](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=777&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:41


É o que eu quero fazer, Rafa! Mas eu não sei nada de jQuery

On Wed, Feb 10, 2016 at 12:53 PM, Will <notifications@github.com> wrote:

> Irado o logo em!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tudoNoob/cronus/commit/de2a7232fcbaa806bf3d6840b226d5ba7e43d4e6#commitcomment-16007091>
> .
>


Actually the conditions should be the same. I will do some experiments with if (match[minml] == ip[minml]) and (MEM_read32(match) == MEM_read32(ip)).


# hello.

 Hey all the **Gits** (git users).
 #Well im Kacper and i've started my way to a .NET developer.
 Just started to learn c# from a nice book, after that i'm gonna jump into ASP.NET MVC 4.0 book, and hopefully find my first job in IT afterwards.


This works great on my Mac, but since strncpy doesn't exist in the GNU stdlib it causes the compiler to fail on Linux.

Turns out I think some of my other attempts to fix this _would_ have worked if I remembered that the MAX_DATA definition's set to 512, not 256. I should refrain from programming late at night.


Is a Chest a Tile? If yes, then my chests are invisible.


I see that you do explicitly handle the 0-case in https://github.com/Ultimaker/libArcus/commit/9304fe205f13bf0a6a78f7b4cd1ebc991106f6f7, so it's probably not a coding error but a documentation problem.


This is new change


Ditto - let's use the enum if we're going to compile-time depend on it.


@MrPowerGamerBR Yep! Signs too. Well, I suppose there called BlockEntities now (which makes more sense).


The comment below describes current behaviour which could be constrained on the server to prevent buggy usage, but the check for it requires some processor cycles. For now I left it as is, but I think it would make sense to add the check.


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


It might be best to also pass $c["id"] to the user function as well.


@gridchin can you revert this please? It's breaking the main tes.com site styles


I think the prior version was clearer and more concise, but it's up to you.


Also addded new icons for the Android app.
Please integrate them into the Android app ASAP.


@xxFlare strange that I only seen it affecting chests, not signs (well, I found out that there was a invisible chest because there was a floating sign (that was attached to the chest))...


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#NlsiRuhiRluuGAl2I0t17A


Does anyone know how I can solve the Problem?
I know it's this commit


While I agree with this 100% I'm not sure it'd go a long way in convincing people. The notes in the paragraph right below this one are good better motivation than pattern matching being blocked IMO although I definitely agree pattern matching being blocked is problematic.


The issue with that is that `Layer` doesn't know anything about the state or `State` module, which handles all of the fetching. As it is now, `Layer` is just a wrapper around the config to make things easier to access and interactions with the state are handled here. But I see what you mean.


Fuuu , meh the excpetions.txt is going to be depreciated soon :P
Ill change it


should this have `override`?


113360729_HumidifierOASys (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


What was the rationale for removing this, out of curiosity?  (I had started using it, but not in so many places that it's any trouble to change.)


Was the map pre-made or did you place the chest/sign?


Well, did you check this on Firefox? I had made those files today only for Chrome and not tested in FF...


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1285.52ead9ee86a6e8a86f5c0508585c8f004c8790ee](http://build.palaso.org/viewLog.html?buildId=69059&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 947, ignored: 12 Build time: 00:06:24


@claudijd we needed to re-enable api access for https://ci.us-west.moz.works/view/Bedrock%20Pipeline/ -- let's discuss alternative mitigations for this specific url pattern.

cc @glogiotatidis, @ameihm0912


If a parent model has taper and Lmax as variables, then I don't want an instance of WingBox to have taper and Lmax fixed. On the other hand if there is no parent model or it doesn't say anything about taper or Lmax then I want them to be fixed variables. That is my attempt at making WingBox a standalone model.  That said, I'm leaning towards using only:

`taper = Variable('taper', '-', 'Taper ratio')
Lmax = Variable('L_[max}', 'N', 'Maximum wing load')

because it isn't really a standalone model anyway.


@xxFlare the map is a Anvil world, imported from a Minecraft 1.7.10 Server, however the chest and the sign were placed in Nukkit.


No I'm afraid I don't have anything to suggest. It just caught me off guard because an [issue](https://github.com/torch/optim/issues/44) in torch optim suggested that qlua was supported.

Regardless qlua is causing me far more issues that it's worth, would be much nicer if it wasn't need at all for generating images.


Please keep whitespace changes to a separate commit.


This is the answers of theory questions.


Testing the commit


It's important (perhaps at a later point) to motivate why rejection reactions are no longer valid. It seems obviosu now but I remember it being confusing when I first learned about these semantics.


How about:

```patch
diff --git a/contrib/avocado-find-unittests b/contrib/avocado-find-unittests
index 92fbfa1..610a14f 100755
--- a/contrib/avocado-find-unittests
+++ b/contrib/avocado-find-unittests
@@ -37,8 +37,10 @@ if __name__ == '__main__':
         except:
             continue
         for klass, methods in test_class_methods.items():
-            test_module_name = os.path.basename(test_module_path)
-            test_module_name = test_module_name.rstrip('.py')
+            if test_module_path.endswith(".py"):
+                test_module_path = test_module_path[:-3]
+            test_module_name = os.path.relpath(test_module_path)
+            test_module_name = test_module_name.replace(os.path.sep, ".")
             result += ["%s.%s.%s" % (test_module_name, klass, method)
                        for method in methods]
     if result:
```

Then you can provide complete paths.... Also the `xargs` is not necessary, so the command to run unit and functional tests is:

```
avocado run `contrib/avocado-find-unittests selftests/[uf]*/*.py` --external-runner '/usr/bin/python -m unittest'
```


_Test_  **test**


Unnecessary whitespace alignment.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1286.d497a513ff51260fb8f8ce9a1398a6561e6059fa](http://build.palaso.org/viewLog.html?buildId=69060&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 947, ignored: 12 Build time: 00:06:36


That's why I include both notes :)


In that case we lost category labels translation.


The existing equivalent is done when there is an unexpected disconnect on the remote connection. However, if the remote node is fenced, the connection doesn't drop on the cluster side, it just hangs. Without this new code, the failure isn't detected until the next poke. This basically simulates a poke failure immediately.


@MrPowerGamerBR @MagicDroidX Signs show only when placed by the user in-game,not premade and chests don't show at all after placing & relogging (or premade).

Hmm.


Great edit! Please finish the commit


Looks good.


Btw this is very unsafe, as it removes all matching chars from the set `.` `p` and `y`. See the patch below for safer implementation.

PS: Try `"pypypypy.py".rstrip(".py")`


@powtac Could you change this line at your server please?
see #71


Re: `postcodeErrorShown`, should be possible. Will check with Tim.


No, the DC already knows the node is up, and we don't want to clear the LRM history etc.


Indeed :)


You are right, it looks the code was taken from ```RGWValidateSwiftToken::receive_header```. In the near future I want to make some broader cleanup of Keystone-related things.


splashDiv - not splach :)


Does it need to be explicitly tested? i.e. if they're missing won't it fail very obviously?


From

> …how rarely it is appropriate to intercept it…

to

> …common thing to do is use `finally` blocks to react…

sounds contradictory.


this reminds me the case where programmer tries to write a sourcecode which should produce the sourcecode. Yes, it's possible in python :smile:


It's worth to explain (perhaps elsewhere, not sure this is the place) why this is desired behavior and why this is not "action at a distance" which is an often cited claim against promises with cancellation. Mainly the fact it means the _consumer_ side gets consistent behavior.


Of course, it should be ```keystone_version == 2```. Thanks for pointing this. :-)


one line per arg

missing array typehint for $domains


More eye-pleasing?
1. Università della Svizzera italiana (Lugano, Switzerland) in 2011
2. Manchester University (Manchester, UK) in 2013
3. Forschungszentrum Jülich (Jülich, Germany) in 2014
4. Technische Universität Dresden (Dresden, Germany) in 2015


I suggest using === whenever possible


Nit - Regular assert may be more appropriate here.


Insert a nice picture of BIRS here (so much text...)?


Can clarify. I meant "intercept" as in "intercept cancelation specifically"


Just want to clarify, based on my understanding. This is a temporary measure we fix issues with our growth and current hydra setup.

Issues:

- 8 TB is not enough anymore to process all the things we have
- nixpkgs evaluation takes more than 6 hours since it has to write thousands of drv files

The plan is, @edolstra and @rbvermaa will look into this and propose/implement the necessary changes. ETA is unknown, hopefully soon as we're all interested to get this going.


/cc @mairatma Could you review? Thx.


see above


This code is really comprehensive, and as you put it, really complex.

It also and changes behavior in so many places, that even unrelated bugs such as:

https://trello.com/c/wrDuEnYI/578-bug-avocado-can-t-mix-remote-vm-runners-and-additional-results-html-xunit-etc

Are not reproducible anymore. But, it's impossible for a small-sized-brain human being such as myself to ACK it. It just doesn't fit my buffer. It's easier said than done, but IMHO you should really try to break those logical changes into its own commits. Examples of that would look better on its own commits:

1. log and output were merged into output
2. avocado.core.result.Result accepts "Job" rather than "args" and "stream" (to get data regarding the job like "uid" and "logfile")

And even the various small fixes (say the json result smal fix) could happily live on its own commit. As I said it before, it's easier said than done, and how much you can break it down is also an open question. But, I'll leave this as an exercise to you, unless someone else has the brain and ACKs this big one.


Sorry to dig this out after a year, but shouldn't the message id be shifted to the right by 8 rather than to the left?
(this notation is consistent through the file, but seems weird nevertheless!)


I don't quite understand the concern.


This can also be
```php
if (isset($string[1]))
  return $this->connection->real_escape_string($string[1]);
```


A simple `in_array($parsed['host'], $this->domains, true)` would work as well ;-)


organized/organised?


Oh! Have you changed already?
It works still as it ist! :D
But please tell me to fix it.


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


I have got difficulties to understand this wording.
How do you think about to clarify affected details a bit more?


I wonder what a use case for that would look like. Honestly I've never needed to react in the intermediate steps to cancellation - only in the source node level.


New batch of security updates: https://nodejs.org/en/blog/vulnerability/february-2016-security-releases/

recommending at least 4.3.0 or 5.6.0


Entry (2) can be removed in our case. Canonical absolute URL-s have the protocol built-in


I agree. @bterlson had some use cases for reacting specifically to cancelation, if I recall, but I am not sure what they are.


s/actobatics/acrobatics/


Forgot to note this change.  We've updated our prim8 app, so these defaults were changed to include the new version number and setup file.


I don't understand the case you're trying to describe.  I'm even more perplexed by the location where you decided to describe it.

A merged pull request (like this one) is less likely to be read than a Jira bug report.  A bug report is less likely to be read than a posting to the Jenkins users mailing list.

I'm also not clear on your claim that "it still gets executed".  I'm not sure of the context of this pull request compared to the current code base.  What is "it" in the "it still gets executed"?


This one condition changes a lot:

if (match[minml] == ip[minml])
zstd v0.5 level 23       40584143  38.70 // win81
zstd v0.5 level 23       29346784  29.35 // ENWIK8

MEM_read32(match) == MEM_read32(ip)
zstd v0.5 level 23       40505250  38.63 // win81
zstd v0.5 level 23       28977991  28.98 // ENWIK8

So my recommendation is to use MEM_read32(match) == MEM_read32(ip) also without extDict.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#TSyhn8xxR7mVjSGO5HmTXw


Reiterating my comment from above, I'd love to see a use case for this.


It's not too clear that the following example is showing undesirable behaviour


Yeah, that's one possibility. But then it breaks if the user passes a path that is not completely made of Python modules. What I mean is, doing something like:

```
avocado run `contrib/avocado-find-unittests /home/foo/avocado/selftests/[uf]*/*.py` --external-runner '/usr/bin/python -m unittest'
```

Would fail, because it would translate into `home.foo.avocado.selftests...` which is not a valid Python module namespace. Tough call.


Well, and the other commit needs more discussion, hence not sure about the whole PR...


> header("Access-Control-Allow-Headers: Origin, X-Requested-With, Content-Type, Accept");

is on the server.


On Wed, 10 Feb 2016, Markus Elfring wrote:

>
> I have got difficulties to understand this wording.
> How do you think about to clarify affected details a bit more?

What part is not clear?

julia


well that's why I used `relpath`. I mean yes, you can break it, but it should work most of the time...


How about adding

```js
// HYPOTHETICAL: showing how a "returned promise" has non-useful behavior
```

to the top? Or do you have another idea?


why not  :
} else if (side->edge.link != NULL)  {
        count++;
}


<img width="598" alt="header" src="https://cloud.githubusercontent.com/assets/36167/12951174/593826c0-d011-11e5-8eef-2edcf8870498.PNG">


How about?

rlAssertGrep "Successfully created .* $Upload_URL" out

The msg looks like 'Successfully created file:///tmp/tmp.nuP8SWYi9V/out.tar.gz'


Extraordinaire :) I can't wait for the release


Agreed. The ideal way would be to get the module name from `inspect` or something like that. But the whole point of the script is to be simple and not try to re-implement what the unittest code will do. Anyway, maybe using an `endswith` is good enough.


Uhhh you might wanna recheck your positions on these... Youre currently asking for the coords (2x, 2y+1, and 2z). Just use, pos.up()/pos.west()/etc. No need for the intermediate xyz variables


@GaurangTandon Yes, so far I've only found one thing wrong (the exercise link button is green, should be black), and I'm about to fix that. Is there anything I've missed?


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/output/ConfWriter.py`, line 35.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/output/ConfWriter.py
+++ b/coalib/output/ConfWriter.py
@@ -32,7 +32,6 @@
         (self.__section_name_surrounding_beg,
          self.__section_name_surrounding_end) = (
             tuple(self.__section_name_surroundings.items())[0])
-

     def _close(self):
         self.__file.close()
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/output/ConfWriter.py`, line 30.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/output/ConfWriter.py
+++ b/coalib/output/ConfWriter.py
@@ -27,7 +27,6 @@
         (self.__section_name_surrounding_beg,
          self.__section_name_surrounding_end) = (
             tuple(self.__section_name_surroundings.items())[0])
-

     def _close(self):
         self.__file.close()
```


Yep, I guess for the majority of use cases `relpath` will give us better results indeed!


This sounds like very much something that model composition should handle gracefully -- this is going to be a very common situation (variables that are fixed in a Model but should be freed when that model is combined with other models).

@bqpd please weigh in


... you can add `if test_module_name.startswith(".."): raise ValueError(...)`


Fixes warnings but breaks display :(

![screenshot-1](https://cloud.githubusercontent.com/assets/6065069/12951238/c71edca6-d02a-11e5-8b2a-40b96f361301.png)


ok


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15247](https://teamcity.mesosphere.io/viewLog.html?buildId=131276&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:22:45


This logic is a unclear to me... so, if we want to copy a slice then the set `a` and `b` to the sliced version of the signal, yes? Does that mean that `a_slice` and `b_slice` are always `Ellipsis`? If so, then can we remove `a_slice` and `b_slice` from this object and just always do `b[...] = a[...]` ?


Great, thanks!
closes #71


s/creates/Creates


Looks like good work


"If you have problems about black screen with return to Emulationstation, please read wiki below"

Betterer English:

"If you encounter black screen problems when returning to Emulationstation, please read the wiki explanation below"


Looks good.


LGTM


113360729_HumidifierOASys (Nigusse) - x86_64-MacOS-10.9-clang: OK (2076 of 2076 tests passed, 0 test warnings)

 * 3 tests had: AUD diffs.

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


As far as I know, the ```<tag>``` part can be omitted nowadays from yarpmanager application XML files, am I right?


`GiveFollowRewardMessage: Given Reward`


Don't forget to also bump the podspec version :+1:


OK, but note that `match[minml] == ip[minml]` is also supposed to improve speed, by a fair margin, simply because it quickly prunes uninteresting cases.
( _note_ : "uninteresting", without taking in consideration `back`, which is why some cases could in fact prove interesting after `back`).

So a question is : is the compression gain worth the speed loss ?


Me neither honestly, but I'm still thinking about how to reply in a constructive way to the last comment on "Aborting a fetch" which is concerned about it.


Show us your support: Star the project on GitHub! :)


@cesclaveria - We somehow missed your comment here.  I've opened [an issue](https://github.com/Automattic/WordPress-tvOS/issues/165) to get this fixed.


create ok


Suggestion, maybe for future versions: use udp protocol for high-traffic types like images, tcp protocol otherwise (i.e., vectors of numbers).


BLAME CANADA!  (We typically prefer British spelling in the Great White North.)


ack


Consider using smaller section titles (3rd/4th level).


113360729_HumidifierOASys (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (978 of 978 tests passed, 0 test warnings)

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


Yea the new template variables are used in base.html near the bottom inside the new javascript I added.

https://help.segment.com/hc/en-us/articles/204913199-Using-multiple-Segment-projects

This might explain what I did better perhaps. I actually used both methods described in this article and mashed them together. The config file and Detecting by domain.

When I tested this method it was correctly passing events to segment debugger


@phausler I still see lots of warnings for use of `CFStringRef` in Foundation.  I wondered why only some of those were fixed?  Is it just a search/replace job?  I'm happy to do a PR for the rest if it would be helpful?


Yep, broke our CI build as well and we had no idea why. Please don't remove 0.18.2 when #296 gets fixed.


@matuella relaxa meu shushush a gente tenta te ajudar. Entre nós, ninguém manja dos js


would be nice to use the ternary operator in getChildPropertyDescription


Hi, are you still working on the assignment? Then I won't continue reviewing it, it looks to be sort of incomplete, so I'd give you more time.


I want to create my this website a better place for online shopping users, [http://dilokideal.com/](url)


o Logo ficou muito foda


Does `onCanceled` behave the same as the others in terms of handling returned values?


`Uncaught ReferenceError: x is not defined`

This is in the ES6 spec presumably?


we have a function StringProcessing.escape that can escape all given characters for you on its own


GameMode(Version 1.0, 10.02.2016)


Hi @ChristianKohler ,

if I remember correctly, it was not by accident that the dependencies are listed as dev-dependencies.
Users of twigs have to include angular and bootstrap as a direct dependency in their project.
This did suit our use case nicely.

what do you think...?


Ignore me.


Can this be used from an ember-cli-based app?

I tried:

    import {StateManager, State} from 'ember-states';

But it didn't work.


I don't think so. Just need to make sure __CC_ARM is checked before the versions. Will update the patch


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


> Assigment in logical statenents shiuld generally be avoided though.

Indeed. My point was just the "shorter" part, which is not necessarily "better."


Does:

```js
p.finally(func);
```

…translate to:

```js
p.then(() => {
  func();
}, () => {
  func();
}, () => {
  func();
})
```


Let me say this, no. Implementing and rebasing this for more than a month took way too much energy from me. I split it to make it possible to follow the story of the change and splitting it even more is beyond my nerves. I'd rather erase everything and live with the current way.


@lsmith77 Does it need to be updated to mark the 1.4 version as stable? I don't know how Composer works with this configuration. Thanks.


Yep. This was hopefully addressed in the follow-up https://github.com/domenic/cancelable-promise/commit/1da06cd55fb1527077cda63ea84d817f4d0460e3


One looks at this _conceptually_ as if the PWP was unpacked


Est-ce que c'est pertinent d'en mettre autant ?


what's instance?


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


cool


please push this to a branch. it is not a fast forward relative to master to needs a PR to merge it


A good question :)
I'm not sure about speed loss because it concerns only opt_parser which is slower than 5 MB/s.
BTW for the fastest opt_parser you can use ZSTD_opt with sufficientLen = 12 (works well in LZ5) and check if it's still better than ZSTD_btlazy2:
{ 0, 12, 23, 21, 22,  5,  4, ZSTD_opt     },


that's exactly why I didn't wanted to include this commit, but Ademar tricked me into keeping it. So let's just leave this commit out for now and I'll resend it with test later...


I don't think it's an issue here. That criticism is aimed at promises that are externally cancelable.


OK


Nope. It's something closer to https://github.com/domenic/promises-unwrapping/issues/18


Replaced favicon is incomplete, should also include replace brand image and adjusted nav_brand to match.


I get it now I've read the `catchCancel` vs `finally` stuff later


Github is driving me crazy!


@andreasabel  this list of issues fixed is too error-prone. In addition, some issues are explicitly mentioned in the CHANGELOG. I propose to replace this list by a generic comment like

```
There was XYZ issues fixed in this release (see https://github.com/agda/agda/issues).
```


FIXED an issue with the syntax. This could have caused issues when calculating who was friend of foe with West.


You can also do

```
$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=12.04
DISTRIB_CODENAME=precise
DISTRIB_DESCRIPTION="Ubuntu 12.04.5 LTS"
```


?
so you sugest to have this:
```if (side->edge.link == NULL) {
} else if (side->edge.link != NULL) {
count++;
}```

?


Sorry about the short-lived (develop branch). My smart git client got all weird for a second. It created and pushed to a new branch. Should be fine now.


I think we can keep it this way, it doesn't change much


thx you are the best :D


That works


Maybe add a unit test to make sure the version returned is >0.


This seems to have fixe my problem although it was a bit harder to update the minimized version of the code.  Will this be integrated anytime soon?


113360729_HumidifierOASys (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2114 of 2114 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.

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


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


Ok fair enough :)


Note that the optimal case for DST-I is the opposite of that for DCT-I.   If N is highly composite, DCT-I is best for N+1, but DST-I is best for N-1.

(This is not just a quirk of FFTW, it is inherent in the definitions of the transforms.  The key fact is that a DCT-I of length M is equivalent to a DFT of length 2(M-1), but a DST-I of length M is equivalent to a DFT of length 2(M+1).)


just a rebase


I realize it's all ridiculous and awful, but I'd rather if we could keep the language G in the comments when possible :)


`host` can also do this and has a slightly nicer output:
```
$ host www.google.com
www.google.com has address 62.119.230.54
www.google.com has address 62.119.230.30
www.google.com has address 62.119.230.45
www.google.com has address 62.119.230.29
www.google.com has address 62.119.230.35
www.google.com has address 62.119.230.49
www.google.com has address 62.119.230.39
www.google.com has address 62.119.230.44
www.google.com has address 62.119.230.34
www.google.com has address 62.119.230.59
www.google.com has address 62.119.230.55
www.google.com has address 62.119.230.50
www.google.com has address 62.119.230.20
www.google.com has address 62.119.230.24
www.google.com has address 62.119.230.40
www.google.com has address 62.119.230.25
www.google.com has IPv6 address 2a00:1450:400f:804::2004
```


Oh wow. I totally missed that. Makes sense now - and this is totally an acceptable solution, but because part of this is explicitly for learning, I want you to make a small change to this.

The conditional should really be as close to the beginning of the code as possible and we shouldn't pass multiple variables like this.

So either in the config.py or the main.py should be where you detect if it's localhost or the live server. I'm not exactly sure how to do that but I'm 99% sure you can - although it may require that you include a new library for detecting that.


@giogonzo @FrancescoCioria Checking for duplicates into an Array of Obj seems not an easy task, but i'm guessing there are at least a tons of other ways to do it. Any help is appriciate


they do not exists, but if I rebase it again, I can add them.


Hi Michael-

Can we move the autonomous code to a separate class?

Thanks-
Tom


Also I run my localhost as 127.0.0.1 so it would run as production on my computer, so account for that as well.


I'm not sure that's what @werbeckes meant


Minor: There are (unfortunately!) cases when a restart is needed. See http://stackoverflow.com/a/20215497/260805.


Nice. what do you think about this? Does it seem cleaner to you? Both the regex rule refactor and the boolean simplification?


not a problem but i think just a clear cache and rebuild should fix this


that should stay html...


`<link rel="stylesheet" href="//code.jquery.com/ui/1.11.4/themes/smoothness/jquery-ui.css" />`


Refactor to use boolean simplification.

return valid_month(month) and valid_day(day, month) and valid_year(year)


Dear Misan, what does this line do?
Regards and respect


I wasn't sure about swf merging, but I since found that the tag is skipped for that anyway.

I believe this also fixes that Flow accessibility problem because they use Haxe to link swfs via -swf-lib.


@NiiRoZz


ok I've understand, you're comment is about the code below.
Ok I guess you're right, good point.


great use of assert!

*above* your validation on line 31, this assertion would be out of place. but putting the assertion here makes it clear to the maintainer that valid day should only be run on valid months!


Also please  explain the purpose of the above line?


well I still prefer messages in commits as I can go through them and make sure they all get fixed. With patches I still need to go through them and make sure you did not left anything out, maybe even extend them and then I easily forget about few parts you included...


what does this function do? It builds a target object but doesn't return it or store it.


what's this? Surely that's worth a comment


nice tests


Oh right, thanks. Wasn't sure which flags will work so left for the time being.


Затеняет имя встроенной функции `eval`


Yes, I'd imagine so.


A thing of beauty. :princess:


You added this to the 228 section of the NEWS file. It should really be in the 229 section...


If changing to this style of writing, probably worth changing in `CREDITS` too for consistency.


As part of #70 and #69.


@matteocrippa Thanks a lot!


Fixed in the next commit.


@Snake1999 Why to Skulls count as a BlockEntity? I thought they were blocks.


I think you worked on this more during class? Be sure to PUSH it here so I can see it before todays lesson.


just wanna say that it is magic


AddNetAreaColumnToEnvelopeSummary (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (2052 of 2052 tests passed, 11 test warnings)

 * 546 tests had: AUD diffs.
 * 11 tests had: BND diffs.
 * 8 tests had: MDD diffs.
 * 8 tests had: MTD diffs.
 * 8 tests had: RDD diffs.
 * 2 tests had: EIO diffs.
 * 2 tests had: ESO small diffs.

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


This will always be false AFAIU. Unit tests create a new instance per test


I know you finished this assignment last week. Be sure to push what you had + this weeks homework before class!


I'm not saying this error is not valid.  I'm saying that you cannot do a pre without a post.

But that said, I was completely mistaken ;-)  If `!hwc`, then the pre didn't actually happen.

So I don't think this needs to change.  Sorry for the confusion.


Эта функция должна была называться по-другому. Кроме того, у вас уже была функция `calculate`, которая занимается вычислениями, не надо дублировать её функционал в этом задании. Вы должны были воспользоваться `calculate` в этой функции, а не переписать её.


Be sure to push what you had from last week and this week's homework!!!


メソッド名をgetRoundとしていますが、get***は今後クラスなどを学習するときに出てくる
getterで使用することが通常です。単にroundで良いかと思います。


I believe this was just an issue with the matching, which needed to be anchored to the root.

I think I fixed this in #8 without having to remove the api reference.


On modern Linuxes you would rather check `/etc/os-release`, which is a cross-distribution standard.


@xxFlare well, they have NBT data, like signs.


If you think it could grow later, then we can leave it as 32.  It doesn't hurt, and I'd rather not change it later.  That said, I don't know if it will actually grow later.


hiii...


learning


Want to take a stab at a trans-js.lm in the test directory? That will get us some existing (and future) test cases for free.

Which javascript engine is preferred on ubuntu?


That's a good idea (kind of like Backbone models). However, layers should be considered immutable. That has been working great so far so I would be reluctant to change that. Also, I like that `LayerNode` is very plain and completely synchronous. I'll reflect on this.


Unneeded overriding


This crashes if I don't have CEPH_LIB environment variable defined.  Also, would it be possible to default it to "lib" since I would expect in the future we would run the unit tests from the CMake build root directory?


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


done :)


これもgetは要らないんじゃないかなと


I already reverted.
Take a look at what happend; why the push is incomplete.
About the property, I am unsure whether this planes has both an elevator and an elevator trim. Take a look at both the source code and the property tree to evaluate that -- because if there is not property for elevator, then you are assigning an orphan output to your autopilot, basically causing misbehavior. Let me know whether this is clear.

Thanks for revising this.
IH-COL


oh, rad. If you want to pr that, feel free :+1: I agree it looks cleaner than a function declaration, probably more performant, too.


It definitely could have existed for some time. I had test failures based on this change from the Scala classes to the java ones that lead me to this. Its possible, that though insertion order is not guaranteed, the usage in Scala happened to maintain order, while the Java usage does not.

There are still a few places in the Scala code where set is used too, perhaps I should change those as well.


*osutil


ready!


Consider removing this conditional. `this.state.fetchLayerDetails` should work for both cases.


The worst commit message ever :P

Must have accidentally removed the first line, but it said:

> #14, which added two more occurrences of leading underscores, was merged to master.


1. Зачем лишние скобки вокруг `int`?
2. `evaluate` - строка, соответственно, вы итерируетесь по символам в строке, т.е. `eval` по определению не может быть ничем, кроме строки (`str`)


nice - thanks for making these changes. I was looking at this myself but couldn't get it to work without the client secret!


This test guarantees `ttl`, `issued` and `expires` are all consistent (same unit). Please confirm if you think this new test is needed.


@colmnet Well, this just generates pure JavaScript, so you can use output wherever you want - include as script in any modern browser, run as script in Node.js, etc.


this is hardcoded to the dev env - this won't work for (some) of the other envs. Populate from configs?


Unneeded overriding, you can add this callback, when an actual implementation is ready


As for test cases - sure, will take a look.


It might be as simple as
```
handler_hw_##ID:		\
	cli; 			        \
	pushl $(ID);		\
        jmp irqproc

irqproc:
	IRQPROC(hw_handler)
```


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


あと、valueとmultiPlierはStringだと、このメソッドの役割としてキャストする作業が増えてしまうので、キャストはメソッドを呼ぶ前に済ませておくのが良いでしょう。
`計算をするメソッド`
という風に割り切るべきです。


Done


How about this week's homework?


LGTM


Where's last week and this week's homework?


sdafasd


Where's this week's homework?


For parity with `getServiceData` consider renaming to `getLayerDetails`.


(picky style) JSON and XML are initialisms and thus should probably be written in caps.


update ok


Can we create a local variable for this NSRange value? We use it throughout the loop and I should have done it the first time around


s/renegages/renegades/


Um nope it doesnt
Actually there is always a switch to Override this . i havent tested this with any theme yet but resetting colors back to white might work with themese again
incase you get the chance to test , let me know, if the theme issue exists
i can make a master switch


А с вами как связаться?


You could remove a variable above by changing this to `self.rebuildLayers()`.


Extra C in "CConverters".


and Assignment 4 Task 2. Added notificaion and toast while adding to favorite from dish view.


This will break in Safari <8 and any other browser that needs prefixes for flexbox. Would've been broken from the start.

https://github.com/bendrucker/preflex


It samples the current location every five 5 milliseconds (only once every five loops each loop taking 1 millisecond)


I'll leave this class as it was generaed by android studio. Anyway it is just a stub.


Same as before, misspelled "renegades".


git think lines that start with `#` are comments ;-)


This could really do with a test to support it, and maybe be split out to a helper method for readability?


This is to provide the time evolution of the target signal responding to the last X command (motion caused by a serial command like for example X100 )


変数名resultは曖昧な表現かと思います。
個人的にメソッドの返り値を格納するくらいしか使えないと思っています。
`jissu * getPow("10", args[1]) + 0.5;`
の結果につけるラベルという意識で考えてみてください。


Задача звучит как: "Добавьте, оптимизированную с помощью таймаута, который срабатывает раз в 100 миллисекунд (throttle), проверку, виден ли блок с облаками и если нет — отмените смещение облаков при прокрутке." То есть у тебя отрисовка движения облаков должна происходить не в scrollTimeout, понимаешь? У тебя в таймауте только проверка, а виден ли блок с облаками. А из-за того, что у тебя и отрисовка движения тут, облака выходят немного "дергаными" (https://monosnap.com/file/fAnPtJg0Rce3pyMnp6aMJRWlPnuEdN). Надо сделать движение облаков отдельно, проверку отдельно.


Is this information really relevant? I can't see a use case in which it can be useful…


This would fail quite loudly at the integration level if it were wrong – what risk would the test mitigate?


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.api/38//violations
```
src/plone/api/content.py:268:1: C901 'delete' is too complex (15)
src/plone/api/content.py:360:1: C901 '_wf_transitions_for' is too complex (13)
src/plone/api/tests/test_doctests.py:121:13: T001 print statement found.
src/plone/api/tests/test_env.py:31:14: D001 found declarePublic replace it with AccessControl.ClassSecurityInfo.public
src/plone/api/tests/test_env.py:32:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:33:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:34:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:35:14: D001 found declarePrivate replace it with AccessControl.ClassSecurityInfo.private
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


see `--cleanup` on https://git-scm.com/docs/git-commit


Same as below for `plyalist`


Consider returning the new layer from `fetchMapService` for parity with this function `fetchLayerDetails`.


OK, than
try to understand what the variable or register in your code meaning.


I suggest you won't hardcode metas, but instead use a `metas` properies in which you'll store an object containing your informations (`title`, `artist`, etc). I'll be more flexible in future.


Reviewer: Britni alter ego. I'm reviewing the PR and making a comment.


I suggest you won't hardcode metas, but instead use a metas properies in which you'll store an object containing your informations (title, artist, etc). I'll be more flexible in future.


I have a  stupid question. It appears that TermSocket is only inherited from `tornado.websocket.WebSocketHandler`.   Calling `super(TermSocket, self).open(url_component)` would be calling the next function in `mro`, which would be the `open` function defined by `tornado.websocket.WebSocketHandler`. It is empty.  I am sure I missed something.


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


You should return an empty array. If you grab OkHttp 3.1.2, that’ll work.


This one should work just fine with Go, no?


Not sure about this one: playlists already knows their tracks, why storing them in return here?


sure


Creating a blog. This is work in progress.


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


Tukaj najbrž hočeš `muzeji` namesto `muzejiinrazstavisca`.


ff


Thanks @Wilfred  and @proofit404


Good call.


Thanks.


Doh! Just bumped it and we'll make sure to keep the two in sync moving forward. Thanks for the reminder :+1:


Will do.


if defines in hosts file is :
-------------------------------
2.2.2.2 ReD.test.com
1.1.1.1 1.TeSt.CoM

it just return NULL record , so add **strings.ToLower** at all "domain" fields .


This introduce an issue on openshift-entreprise with atomic-host, playbook fail with Access denied:

    -bash-4.2# systemctl mask atomic-openshift-master
    Failed to execute operation: Access denied


やっぱり</h3>がきになる。


> git think lines that start with # are comments ;-)

Of course :)

![facepalm](https://cloud.githubusercontent.com/assets/2803721/12952534/0f62ac22-d017-11e5-8e2a-0f0f8284200e.png)


インデントがおかしいようです。


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.api/39//violations
```
src/plone/api/content.py:268:1: C901 'delete' is too complex (15)
src/plone/api/content.py:360:1: C901 '_wf_transitions_for' is too complex (12)
src/plone/api/content.py:421:1: C901 'transition' is too complex (13)
src/plone/api/tests/test_doctests.py:121:13: T001 print statement found.
src/plone/api/tests/test_env.py:31:14: D001 found declarePublic replace it with AccessControl.ClassSecurityInfo.public
src/plone/api/tests/test_env.py:32:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:33:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:34:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:35:14: D001 found declarePrivate replace it with AccessControl.ClassSecurityInfo.private
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Layout has no children, it can be closed with />


oh NICE, I just noticed this issue myself :)
excellent work on this layout system!


Interesting hack, I might be a bit worried that BlockRedstoneEvent gets called too much, I mean just think of redstone clocks, and as I understand it, it gets called for every block of redstone dust affected.

I'm gonna test it out first, and if it's not causing too much lag, I'll accept it to the server, but if we're doing it like this anyway what do you say to just teleport all (non-player) entities within 1 or 2 blocks of the player when they teleport by crouching?


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


oops


Seems that this has not been merged and it causes problems on my side. Could we perhaps get the cffi and cryptography changes into master at least?


ugh. silly muscle memory. that silly ' gets me every time.


gj


It seems websocket-1.3.jar is now missing in git :(
red5 server fail to start :(


So this breaks any mailto: or data: links on the page.  Is that good?


Should this be `bestMatching`


Actually, you don't need to specify the table name, Doctrine will use the default according to the [naming strategy](http://doctrine-orm.readthedocs.org/projects/doctrine-orm/en/latest/reference/namingstrategy.html). And if I remember correctly it's `version` instead of `Version`.


[質問]squってどういう意味？


This is probably negligible, but I think that although ajax requests are cached, `rebuildLayers` gets called every time this runs. The same thing happens in my original code and I'm planning to fix this later.


Thank you!


四捨五入で作ったべき乗のメソッドが使えそうですね。
とりあえず、一通り他のレビュー対応が終わったらやりますかね。


\n


I think using `.utcnow()` instead of `.now()` can make the output "timezone-indipendet".
What do you think about it?


Yeah, that sounds likely. Particularly for small immutable sets (due to Set{1-4} in Scala). Sounds like this is something we'd want to fix for 0.9.0. Any chance you could provide a PR for that?


I haven't tested it, but I assume it's been written such that contiuous railways are possible through portals? If that's so, how about using a VehicleBlockCollisionEvent? I would think it would get called far less frequently.


Discussed in person, there's still a bug, I will fix it.


It is done


If you're busy, let me know and I'll do it (it would have to be quick for it to be considered).


Perhaps an initial answer :

with `minml` :
`bench/silesia.tar : 211948032 ->  57153170 (3.708),   2.9 MB/s , 557.8 MB/s`

with `MEM_read32` :
`bench/silesia.tar : 211948032 ->  56772198 (3.733),   2.5 MB/s , 560.7 MB/s`

so better compression, slower speed.

Why not, it's all a matter of trade off.


great catch and i am working on this


Yes, that should definitely happen. I was going to do it here, but it looks like I forgot. I'm working on another TFS-side change to add the `<clear/>` back in (to avoid leaking environment into the build in Visual Studio) and I'll remove this source in that change. I can put up a quick pull request if you want the removal in sooner, though.


This may not work as you expect it - see the "Let's copy some stuff" section of the [plugin writer's guide](https://github.com/sosreport/sos/wiki/How-to-Write-a-Plugin). Size limiting is currently only supported for single files, lists of files, and glob patterns that expand to a list of one or more files.

Normally when `all_logs` is in effect either all size limiting is disabled (not preferred) or plugins do things like:

```
            self.add_copy_spec_limit('/var/log/kimchi/*.log',
                                     sizelimit=log_limit)
```

To ensure the size limit is applied properly.


is there a reason to use vendor Promise instead of native one?


> "a cache without an eviction policy is a resource leak".

I absolutely agree with this.


I prefixed them only on direectives


ローレルに指数を代入するとかよくわからないので
```java
double rohrer  = calculate_rohrer
```
がいいかな。
メソッド名の頭に動詞をつけるのはよくあるかなという気がします。


copy


There's multiple inheritance - Jupyter's handler class inherits from TermSocket and some other classes, so it the next class in the mro is not necessarily tornado's WebSocketHandler. Now I look at it, though, I think I misidentified the problem affecting you - this change still makes sense, but I don't think it should be necessary for what you need in Jupyter.

Multiple inheritance is confusing and we generally try to avoid it where possible, but it's occasionally convenient.


@parente sorry for the delay, I'm certainly appreciative of the work! I'll give [the new README section](https://github.com/jupyter/docker-stacks/wiki/Docker-Recipes#using-pip-install-in-a-child-docker-image) a shot later today, but it seems pretty clear to me. Thanks!


It is fixed


これも引数は文字列じゃないほうががいいでしょう


Works for me


とりあえずdoubleでよいかと


Would love to see Auth0 implementation of this!


done


Are the redis logs rotated with a new suffix? E.g. `redis.log.1`, `redis.log.2` etc.?

Just wondering as this looks at a glance like it's collecting all logs (more commonly the non-all_logs case has `/var/log/package/package.log` here).

If the log structure of redis is more complex (e.g. there are separate `daemon.log`, `client.log`, or whatever) then that should be noted in the comments.


これもインデント


In sync with [this comment](https://github.com/codebendercc/eratosthenes/commit/04008171417d1d09837d374bea2a32a5f48c5d20#commitcomment-16009072) maybe removing it altogether would be better.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1287.330cd949d0faa4476b7dac359b277875c58b05b1](http://build.palaso.org/viewLog.html?buildId=69061&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 947, ignored: 12 Build time: 00:05:56


Doesn't the AGPL 3 disallow re-liecensing the code under another licence without the explicit consent of the original author?


FYI Neo4j switched to Scala 2.11 with their 2.3 release.


This was an example of using @IBAction to wire a event handler to a function. I find this to be very clean and clear. You can easily at a glance see what functions are tied to events.


We got to all the trouble of keeping bitfields here and then blow a whole 8 bits on a flag?


Note that commit message is wrong, there are no conflicts remaining


We can probably make this one a `static` as I don't think it can be changed after the database is created.


Consistency.


Yeah. I tend to agree.
This is ported from one of our internal branch.  So I don't want to change it at this time.
And it is lower priority to changing something just because of style.
Hopefully we can catch this next time when we code review such code.

Hopeful


Move the diff calculation to here


Why does the rppa benchmark require curl ? or RSQLite?


Move setMotors to here


Not to sure what changed but having the the media plugin installed crashes ripple emulator.

You can test this out by creating a test project, adding an ios platform, then running ripple emulator from the command line, and it should run as normal. Next close the emulator, add the cordova-plugin-media, and run ripple. You will see that it crashes on startup, stating "Something went wrong while displaying this webpage". Remove the media plugin, run ripple and it will open up fine again.

I believe the problem is caused by an update that happened either on Thursday the 4th or Friday 5th, since that was the last day the plugin was working for me. Although I know that google also updated chrome around the same time period, so the bug might be coming from their side.


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


ok It is done just on controller alias


Move setpoint setting to init(before enable)


comments go here>


Have you tested this?  It would seem that the backslashes in the programdata string would need escaped.

-Brad


Can this be packaged into a release?


インデントおかしいです。


I think a better name would be ``bool case_sensitive() const``. "sensitive" carries the additional connotation that this could have privacy implications or something.


Change this to currentDist and the setpoint to the desired distance


Maybe also set (my tests failed because I have a different name here)

```
\Config::set('security.csrf_token_key', 'fuel_csrf_token');
```


I've spun up the Vagrant instance, copied in the changes to the module, and added the lines
```
   use_embedded_ruby => true,
   rabbitmq_hosts    => ['localhost','puppet'],
```
to the sensu-server.pp file in tests.

Doing a `puppet apply sensu-server.pp` generates this error:
```
Error: Evaluation Error: Error while evaluating a Function Call, Failed to parse template sensu/rabbitmq.conf.erb:
  Filepath: /etc/puppet/modules/sensu/templates/rabbitmq.conf.erb
  Line: 5
  Detail: undefined method `has_key?' for "localhost":String
 at /etc/puppet/modules/sensu/manifests/rabbitmq/config.pp:125:18 on node sensu-server.example.com
Error: Evaluation Error: Error while evaluating a Function Call, Failed to parse template sensu/rabbitmq.conf.erb:
  Filepath: /etc/puppet/modules/sensu/templates/rabbitmq.conf.erb
  Line: 5
  Detail: undefined method `has_key?' for "localhost":String
 at /etc/puppet/modules/sensu/manifests/rabbitmq/config.pp:125:18 on node sensu-server.example.com
```

Any thoughts on where the error might be?


Thank you. I was wondering there was no way to avoid creating as many IRQ handlers. But I think I now understand your suggestion about `jmp`, we can avoid inlining rest of IRQPROC() for reducing instruction cache footprint.

I'll need to ask you later about `inline` in C and Instruction cache footprint.


It would be more consistent with the ``MIMEField`` style to not have a setter. Also, setters are ugly :)


Adding interface elements was fairly strait forward. Xcode uses a wysiwyg editor like more IDE's. I really liked how the elements can be found easily with a search bar and have strait forward names. Also they have really good descriptions on the elements.


Assuming these are the only secrets to scrub this looks good. Since you're using a capture group in the regex you could simplify/shorten these calls by excluding the secret and substituting it into the result:

```python
    def postproc(self):
        self.do_file_sub(
            "/etc/redis.conf",
            r"(masterauth\s)*.*",
            r"\1********"
        )
        self.do_file_sub(
            "/etc/redis.conf",
            r"(requirepass\s)*.*",
            r"\1********"
        )
```

It's not a huge saving but it's a common idiom in sos plugins and it has the benefit of preserving the whitespace structure of the file.


No worries about the `<clear/>`, will wait for the changes to propagate through TFS.
I did however already take out the `dotnet-coreclr` reference as part of 77fd82c (merged)

Thanks!


用加上註解的方式來說明嗎?


Just:

```python
        self.add_cmd_output("redis-cli info")
```
is fine :-)


I think we can have a static array something like:
```c
static const char *fake_bdw_guids[] = {
"asd",
"sdf",
"sdf",
};
```


remove this line (we add the feedforward manually)


instead of the constant i limit, something like this would be good: `(sizeof(array) / sizeof(array[0]))`


[Arkady Apelchuk's Solution] (https://github.com/apelchuk/project-euler)


Initial commit to index.html. Created the overall structures of the page.


ack


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 15258](https://teamcity.mesosphere.io/viewLog.html?buildId=131293&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:11


I have even bigger difference (no extDict used):
zstd v0.5 level 23       2.15 MB/s    325 MB/s     40505250  38.63
zstd v0.5 level 23       2.81 MB/s    440 MB/s     40584143  38.70

So actually you should use minml also for extDict.


Thanks :blush:


Here we have another example of inline event handler. This one is wired to the slider on the app. I also got my first look at a math function in Apple's Swift. This example also shows me getting the value from the slider. really easy.


I don't see anything, just by looking at the code, where it's obvious. When you uncomment this line of code, look at the output in your browser console and server in the terminal. Webpack's reaction to broken code is to just show nothing, so it may be that. If you don't see any errors, let me know and we can walk through it after class!


Quid de postcss?
On pourrait aussi rajouter une ligne pour gérer les fichiers `scss`. Par exemple
`{ test: /\.scss$/, loaders: ['style', 'css', 'postcss-loader?sourceMap', 'sass?sourceMap'] },`


Thank you :blush:


Thanks for the analysis. Yes, we realized that it was parsing too eagerly in this commit and will maintain the same level of lazy parsing when we roll it forward.


Is this commit related to some issue?


This seems to work with both python2 and python3, but is it worth settling on one and specifying it explicitly in the shebang?

For me, this always seems to be the bane of using python in build systems -- it somehow always assumes a version which happens to not be the default for me.


Aaaand you would be correct.  Will fix.


``allow``.... would prefere more positive wording, ``add``, ``enable``, ``make possible`` or somesuch


Composer isn't needed atm


why not add the backslash to the chain?


phpunit  => PHPUnit


113360729_HumidifierOASys (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (2082 of 2082 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.

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


This will never actually get called for a MaplyViewController.

`MaplyViewController dealloc` calls `[self clear]` which calls `[super clear]`(MaplyBaseViewController clear), which sets `scene = Null`, so by time it gets to this dealloc, scene is always null.


oh and also can you make the bin width equal (as opposed to equal #s of bins) - makes the comparisons of each distribution easier


it shoudn' t be ifneq ?


Does the parameter need to be in plural? Language should be consistent with the code. Saying "AddVersion($versions)" is confusing, gets you to think if you're actually adding multiple versions. Same for remove below.


Ca coûte pas grand chose et c'est une bonne pratique ;-)

Après, je pense qu'il ne sera pas nécéssaire de rajouter des tests pour les autres langues (on teste déjà les 2 cas : langue par défaut et langue "autre")


Honestly, I could not find any better solution than a rough error suppression in this case: two PHP notices are raised within the vendor library.


Review my assignment


It'll complain about assert returning `Unit`.


I don't think we need to change this hear but I think we should use the rspec convention of `subject { described_class.new "actor_id". 123, Object }` instead of `let(:subject)`. This helps identify the SUT.

Thanks for fixing this :+1: I remember seeing some sporadic issues with this. :ok_hand:


i think it can be useful to sort by date


I'm hitting the following issue after update http://projects.theforeman.org/issues/13650, was it intentional to not have a migration as part for the original PR?


i copied that from the old model


`<script src="//up.htmlacademy.ru/assets/js_intensive/jsonp/reviews.js"></script>`
Вот так не пробовал?


ah ok :)


Issues don't have a slug, actually it's just the number of the issue. On https://github.com/GlotPress/GlotPress-WP/wiki/5.-Git-workflow we mention "keywords".


Why do we need to require all the things here? It may signify that we have too many dependencies in this class. Can we pair this down to the support files we need?


113360729_HumidifierOASys (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1531 of 1531 tests passed, 0 test warnings)

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


Good question. Unfortunately the host executing the RA command doesn't know why the monitor is being called, and the correct response would depend on that. If it's a routine recurring monitor, or a monitor on a node where the connection isn't supposed to be (which will lead to a stop), we'll do the right thing (i.e. nothing here). However if the initial probe finds the connection already active on the node where we want it, we probably should do remote_node_up(). Not sure what to do about that; I'll just add a comment about it for now. Connections have to be initiated by the cluster, so the chance of that happening should be minimal.


As I use Swift more it really feels at home. The syntax is easy to follow because it is very similar to java, javascript and even C#. On a side note, reference to ui elements is also easy to see because they are declared with @IBOutlet. Again, this allows at a glance, you we see what is tied to a interface element.


We won't use binaries to store object, but rather a pointer to a resource (URL or Document ID that references a Doctype File)


You're right. Earlier i was using dict which didnt preserve order. But
chain does preserve order. Will change

On Wed, Feb 10, 2016 at 9:43 PM, Lasse Schuirmann <notifications@github.com>
wrote:

> why not add the backslash to the chain?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/b580baacabcc479a7f20f4256676fa9091dc8a15#commitcomment-16009487>
> .
>


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.5 [Build 94](https://teamcity.labkey.org/viewLog.html?buildId=302129&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 14, passed: 34 Build time: 01:08:04
### Failed tests
```
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.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

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

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

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

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

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

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


@thet Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.discussion/30//violations
```
plone/app/discussion/contentrules.py:20:5: E731 do not assign a lambda expression, use a def
plone/app/discussion/browser/comments.py:132:1: C901 'CommentForm.handleComment' is too complex (17)
plone/app/discussion/browser/conversation.py:29:1: C901 'ConversationView._enabled_for_archetypes' is too complex (14)
plone/app/discussion/browser/conversation.py:135:12: P002 found "hasattr", consider replacing it
plone/app/discussion/browser/migration.py:34:1: C901 'View.__call__' is too complex (32)
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Thanks for adding the example project.

Did you do something to remove disqus from pages? It shows up automatically.  I would like Butterfly to be "batteries included" (everything works out of the box) as opposed to minimal and opt-in. Google Analytics is an exception, because I need the user's google analytics ID to activate it.

Sorry, it's not clear to me yet: do you need the ability to disable disqus (site-wide)?


This was a mistake. I think I undid it though...


url paths and ctrls on app.js


Conditionals are exactly like I expected. Nothing new or surprising here at this point.


Can you merge the code of the above methods where you create actual clients from prepared config? It'll save us a few lines duplicated of code.


That's lame.


Hi @mcrmfc : this change is breaking our tests


@DarkSilencerZA Your HUD is fucking nice :+1:


Clairement. Je n'ai pas encore exploré postcss, mais il le faudrait.


You're right, the dealloc calls for the MaplyBaseViewController subclasses aren't doing anything.  It's all in the base class dealloc now.


Whoops! I've left `$field` in here. I will push a fix for this tonight.


[no problem](https://github.com/SIGSEV/Starveller/blob/f8b20e8226b4381368d3d0f42686b9ad677b6766/src/api/Shot.service.js#L12)!


lol... thanks for finding my mistakes @tunnckoCore
It actually is helpful, if you'd like to do a PR, that'd be cool, otherwise, one of us will get it fixed when we work on it again.


This commit also has one bad side effect (slow block placing/digging rendering), I already reported it to @RealBadAngel.


Est-ce qu'eslint le reconnaît sans le point de `.eslintrc` ?
En tout cas, ça peut être sympa d'utiliser le même format que dans blease v1 (j'ai fait un lien symbolique avec mon éditeur par exemple)


J'utiliserai directement le eslintrc de systematic


This is only true for branches which are in the GlotPress repo, for external branches we could mention this link:

<img width="714" alt="test-pr" src="https://cloud.githubusercontent.com/assets/617637/12953405/9df99808-d01a-11e5-88eb-958d23871f87.png">


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


Safezones?


Вроде работает, но я бы не закладывался на то, что один раз созданный `fragment` можно будет использовать много раз. Я бы, пожалуй, создавал его в renderReviews


So one discussed practice here is to not have kwargs in the setup function, but to do them all as values and have the user specify changes via a substitutions dict. That cleanly fixes this problem.


Before throwing the exception, you might want to set the interruption flag by `Thread.currentThread.interrupt()`.


Doesn't this need to return ``HPACK_ERROR_COMPRESSION_ERROR`` on error?


For posterity, I could change `map flatten` to `flapMap` (as long its iterable).


C'est toujours utile ce fichier ?


So for each job, you are adding a map (key value pairs). First, it will describe the session type: process or yarn, then for yarn, we are adding an application ID, correct? And you are putting a variable applicationId in the Job itself. I thought we said we want an object which can be easily replaced through inheritance and polymorphism so we don't need to add session specific variables to the job itself?


patch about checking for libEGL.so which we talked about...

```patch
diff --git a/gputop/gputop-main.c b/gputop/gputop-main.c
index cc72252..7f8b0f9 100644
--- a/gputop/gputop-main.c
+++ b/gputop/gputop-main.c
@@ -95,7 +95,7 @@ usage(void)
 }

 #ifdef SUPPORT_GL
-static void
+static bool
 resolve_lib_path_for_env(const char *lib, const char *sym_name, const char *env)
 {
     void *lib_handle;
@@ -106,7 +106,7 @@ resolve_lib_path_for_env(const char *lib, const char *sym_name, const char *env)
     if (!lib_handle) {
         fprintf(stderr, "gputop: Failed to dlopen \"%s\" while trying to resolve a default library path: %s\n",
                 lib, dlerror());
-        exit(1);
+        return false;
     }

     sym = dlsym(lib_handle, sym_name);
@@ -129,6 +129,8 @@ resolve_lib_path_for_env(const char *lib, const char *sym_name, const char *env)
                 lib, dlerror());
         exit(1);
     }
+
+    return true;
 }
 #endif

@@ -273,8 +275,11 @@ main (int argc, char **argv)
 #ifdef SUPPORT_GL
     env_append_path("LD_PRELOAD", GPUTOP_LIB_DIR "/wrappers/libfakeGL.so");

-    if (!getenv("GPUTOP_GL_LIBRARY"))
-       resolve_lib_path_for_env("libGL.so.1", "glClear", "GPUTOP_GL_LIBRARY");
+    if (!getenv("GPUTOP_GL_LIBRARY")) {
+       bool found = resolve_lib_path_for_env("libGL.so.1", "glClear", "GPUTOP_GL_LIBRARY");
+        if (!found)
+            exit(1);
+    }

     if (!getenv("GPUTOP_EGL_LIBRARY"))
        resolve_lib_path_for_env("libEGL.so.1", "eglGetDisplay", "GPUTOP_EGL_LIBRARY");

```


Код почему-то не записывает в numbers последнюю цифру и всегда в начале numbers ставит 0, хотя должен делать это, только если перед первой цифрой стоит "-"


Ugh... Again like this?


Mm, this is also gone by the end.


effectivement. Après, eslint reconnaît le fichier qu'on lui passe en paramètre de conf, donc pas de souci.
Je ne sais pas comment passer le mien en paramètre de l'éditeur , comment ça se passe chez toi ? C'est bizarre de mettre avec ou sans point ?
Je vais remettre le point dan tous les cas, par clarté et rétrocompatibilité.


introduced with 7067c10 due to signature change. (first template parameter is now the index type, not Coord anymore)


bien vu, je le dégage.


This is the correct fix:
class product_product(osv.osv):
    _inherit = "product.product"
    def _bom_orders_count(self, cr, uid, ids, field_name, arg, context=None):
        Production = self.pool('mrp.production')
        res = {}
        for product_id in ids:
            res[product_id] = Production.search_count(cr,uid, [('product_id', '=', product_id)], context=context)
        return res

    _columns = {
        'mo_count': fields.function(_bom_orders_count, string='# Manufacturing Orders', type='integer'),
    }

    def action_view_bom(self, cr, uid, ids, context=None):
        result = self.pool.get("product.template")._get_act_window_dict(cr, uid, 'mrp.product_open_bom', context=context)
        templates = [product.product_tmpl_id.id for product in self.browse(cr, uid, ids, context=context)]
        # bom specific to this variant or global to template
        context = {
            'search_default_product_tmpl_id': templates[0],
            'default_product_tmpl_id': templates[0],
            'default_product_id': ids[0],
        }
        result['context'] = str(context)
        return result


113008903Provide-a-better-warning-message-for-a-triangular-window (lgu1234) - x86_64-MacOS-10.9-clang: OK (2074 of 2074 tests passed, 0 test warnings)

 * 3 tests had: AUD diffs.

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


It has no sence. Should be wxGetTranslation("somthing").
wxTRANSLATE("Other Income") do nothing. It just marker for gettext (poedit).


<img src="imeges/91.jpg" class="photo" alt="Это я" title="Мамина радость">


non !


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


Сколько раз повторять, что не надо использовать множественное число там, где должно быть единственное. Ты по одной букве итерируешься. Дальше буду не засчитывать задания с такой смысловой ошибкой в имени переменной. Без права на исправление.


Why move clearly needed depends to run this app fully functioning to makedepends and leave users with a crippled down app, not intended at all that way upstream.
Again, don't blindly copy some other distro, depends for this app where set as upstream intended, make sure to revert this


Added a commit for this (and more extensive comments)


ack


other patch...
```patch
diff --git a/gputop/gputop-ui.c b/gputop/gputop-ui.c
index 594eea4..db703f5 100644
--- a/gputop/gputop-ui.c
+++ b/gputop/gputop-ui.c
@@ -1414,6 +1414,11 @@ gputop_ui_run(void *arg)
         free (tab);
     }

+    if (fake_mode && gputop_get_bool_env("GPUTOP_TRAVIS")) {
+        uv_timer_init(gputop_ui_loop, &fake_timer);
+        uv_timer_start(&fake_timer, quite_fake_mode_cb, 1000, 1000);
+    }
+
     return 0;
 }

```


Nice test. I like that you verify that it will stay on the second page.


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


Почему сверху ты используешься для сравнения строк оператор равенства, а тут оператор идентичности?


So if no user is specified then ALL messages can be deleted?


I might have expected that with no user NO messages could be deleted, but I can see going either way on this.


``size()`` would be a more conventional name for this.


Школьный физрук


Most of this part is done in the wysiwyg editor. Most of the code above is from that action. However the process was extremely simple. In Xcode its like making a storyboard diagram. You drag out your new View onto the story board. Then you ctrl+drag the element you want to link to opening that new View. Its that simple!


Я говорил не наращивать так строки никогда.


113360729_HumidifierOASys (Nigusse) - i386-Windows-7-VisualStudio-12: OK (2082 of 2082 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.

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


we don't need wxTRANSLATE here.
Espetially for
wxTRANSLATE("."), wxTRANSLATE(",") and wxTRANSLATE("VEF")
May be wxTRANSLATE("Venezuelan Bolívar") has sence. But I'll prefer just English names for currencies.


Typo: vsibility -> visibility


I can submit a PR request today. To be clear, do you think this patch include all changes from Set -> List bot in Scala and Java? And do I need to maintain compatibility and deprecate or are these considered internal messages?


Consider ``capacity()`` as a more conventional name for this if it is setting the number of items that might be stored. If it is setting the number of items currently stored, then ``resize()``.


Tyop: ptoperties -> properties


Thanks for considering it!

However, I mainly use R4rb (the most stable) and jl4rb (less often) in my
projects since I prefer to embed R and julia in my ruby projects. On day, I
would tp move from ruby to red (Rebol successor) ...

Cheers

Remy

2016-02-09 3:23 GMT+01:00 SVAKSHA <notifications@github.com>:

> @rcqls : jl4R
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/svaksha/Julia.jl/commit/84e3fe2211787b48c526587dd251bb0365c07a7b
>


remove this to get 15% more also it confct with this memcpy tag
TARGET_USE_QCOM_BIONIC_OPTIMIZATION := true that bionic_optimization is for cortex-a15


Suggest you ask Jim about wording and capitalization.  Also, in the docs, is thorslave one word or two?


> With 2 GB, services on the compute node are running from time to time
OOM.

This sounds like postponing the solution for an unknown problem with the cost of reducing the capacity of our mkcloud workers (we either need to reduce the number of parallel clouds or the maximum size of a cloud).

This could be a short term workaround, sure. Ayhow, I would prefer a real solution, or at least some evidence how or why 2.5 GB really solves the issue rather than making it less likely.


If no user is specified but `canDelete` is set to true, then yes, all messages can be deleted. Note that `canDelete` defaults to false, so by default no messages can ever be deleted


Because this one seemed more like a Github label, whereas the `x-hub-signature` is just a side effect.


</li>


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


`soft` is misleading here, use status or anything other than soft, which seems like you're talking about soft resource quotas, if such will ever exist.


Это `things` вполне определенная вещь - символ, по-английски `character`. Так и пиши.


Typo: lasntName -> lastName


It's in the object, so no.  Perhaps you mean should the method be named differently.


It looks like you're confusing wxTRANSLATE and wxGetTranslation.
Please be careful.


i like this


Where did you find the color codes?


OK. You're right. I learned something today. I change that.


This for loop is repeated twice, previously in `Put`, how about creating a function that will do it?


it really speaks to my soul


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 106](https://teamcity.labkey.org/viewLog.html?buildId=302125&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 19 (5 new), passed: 275, muted: 1 Build time: 01:45:57
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

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

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

org.labkey.test.Runner: org.labkey.test.util.JUnitHeader: <no details avaliable>

org.labkey.test.util.JUnitFooter.afterJUnit: <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.verifyLogGutterPlot: <no details avaliable>

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

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

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

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


isEmpty() is faster than length()


Rules :D


Твой алгоритм делает много лишних проверок. Логика вывернута наизнанку. Ты создаешь специальные случаи на ровном месте.


Here I have added visuals to the app. Only code in the ViewController was added by me. All of the other visual images and styles where added with the wysiwyg editor. This editor again is really nice once your used to it. Here you can set different styles depending on the state of the element your want to visually change. There is some limitations to this process however. An example of this was the slider. To change all the elements of the slider that I wanted to I had to do this with code as shown above in the ViewController.swift


Ich habe mich eventuell etwas unklar ausgedrückt, ich meinte natürlich, dass alle Versionen unter 4.x, das heißt alle, die mit "0." beginnen ES6 nicht wirklich unterstützen(ausgenommen mit der harmony flag, was aber nicht das wahre war). In den Debian Repos sind zum Beispiel derzeit nur alte versionen (zur zeit 0.10.29) Eine Übersicht habe ich auch nicht gefunden


I fixed it.


Yeah, I like this a lot more. The only thing that bothers me is that the REGEX_RULE has to float outside of everything in order to be accessible to the unit tests.


Найдешь в этой строке ошибку сама?


The Javadoc should probably be expanded to describe what this key is for.


We have a test like this at the end of configure.in Just leave out this commented-out copy


Why this special case for forbidden error? Aren't the errors we return in the next if sufficient?


I meant that the method name should just be `taskToInfoBiMap` not `forTaskToInfoBiMap`, so the invocation would be `taskManager.taskToInfoBiMap`


Mmm.. What a good-looking interface!


"this" is not neede here, see coding style :)


minor point, but length() returns an unsigned, so I believe  "== 0" is more efficient at runtime


Might be worth mentioning why the strategy would be null, here.  It seems to be the crux of how this maps "defaultKey" to "when running on passive".


Thanks!


Let's leave this out, and fix the #ifdef instead, to use the preprocessor symbols that are already available without this. There are plenty of places where we check for Windows, so just mimic any of the existing ones.


I don't think opentkproject(.net)  would work b/c google will confuse it with opentk.com the current site, which would remain a "fossil site" if @thefiddler  remains MIA forever.

I was thinking maybe NewTK.org/com/net & keeping the name OpenTK but @cra0zy has a point, OpenTK's name has baggage - some good, but maybe even more bad :(

BTW, there is a project called "MathParserTK" (totally unrelated) - it's also abandonware, but it works good & it's C# so it should be compatible.   That could certainly be merged in.  Maybe "TotalTK" b/c it will have so many features or "NewTK" or.....???

I'm still optimistic we wil hear from @thefiddler, just when.  We could strategize for 2 months or so, but if this is a 2 year hiatus, that would kill the project unless others intervene.

What spambots? http://www.opentk.com/forum is as quite as a ghost


Why not just: `group.permissions = new_permissions`?


About using hashtags:

Why **#STIWNUrequest** and **#STIWNU** announcement instead of **#request** and **#announcement**?

Indi tanan nga request kag announcement, kinanglanon natun. Budlay sa aton kay gintagaan na duwa ka hashtag, gusto pa dugangan. Ang para lang sa STI WNU ang kinanglanon natun. Binlan ta man ang iban. Indi ta dapat magdinalok kay indi ta na tanan maubos.