develop (lefticus) - Win64-Windows-10-VisualStudio-12: OK (387 of 389 tests passed, 0 test warnings)

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


Thanks, I've just pushed a fix to handle that particular situation.

On Fri, Mar 4, 2016 at 7:55 AM, Iskender TOTOĞLU <notifications@github.com>
wrote:

> I have syntax error for usage :
>
> <?=wireRenderFile('./views/additionals')?>
>
> *error output :* <?=
> \ProcessWire\wirerenderfile(\ProcessWire\wire('files')->compile(\ProcessWire\wire("config")->paths->root
> .
> 'site/templates/views/additionals')?>,array('includes'=>true,'namespace'=>true,'modules'=>true))
>
> If i put a ; to end there is no error
>
> <?=wireRenderFile('./views/additionals');?>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ryancramerdesign/ProcessWire/commit/2093e8d4e5a746db13d6ff682eb54e0226335698#commitcomment-16498972>
> .
>


Storing an outer scope variable like this seems very strange to me -- I think I would actually prefer a nested `.then`.


@flefevre there is an issue in your code on line #68. The get() method returns an Option[T] which is **None ** when the element is not found. Thus your check is a tautology, i.e. its always true.  Additionally, you'd want to only start a microsvc if the UUID is not already present, thus only in the cases when the get returns None.

There are a few ways to fix it, here a few examples:

`if(runningMicrosvcHashMap.get(microsvcId) == None) { ... } `

another a bit more cryptic but in the Haskell way is:

`runningMicrosvcHashMap.get(microsvcId).orElse { () => { ... }}`

Where in both cases the "..." should be replaced by the code you have in your `if-then` block.

There is another way of dealing with this using pattern matching:

`runningMicrosvcHashMap.get(microsvcId) match {
   case None =>
      ...
   case Some(id) =>  // log error
}`

One final comment, as your function can fail to do what requested ideally you'd want to return a Try[Boolean] type (or raise an exception or return a boolean if you are going to call this from Java) otherwise the caller has no way of knowing if the microsvc was started or not.


Tu l'as pas mis à la bonne place ce `break`, il faut mettre après le `end` suivant sinon ça peut envoyer plusieurs objets `player`, et pas juste un seul.


Perhaps some branching would be in order here. ISO-8859-1 for subtitles  UTF8 for captions obtained from Amara.org. I would really like to see this problem addressed. I do lots of subtitles in Amara using extended characters which almost never display correctly using this extension.


IMO best practive would be to use composition over inheritance


handleRequest.


develop (energy-plus) - Win64-Windows-7-VisualStudio-12: OK (1576 of 1577 tests passed, 0 test warnings)

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


So you have a bidirectional relationship here -- let's talk about that.


that behavior seems randomly inconsistent to me, which I hate, but I'll do it if you want.


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


Remove, unused


promise chains rule, nested thens drool


335 has db2 ?


When I remove everything but `main { content: template('calendar.xml'); bind: data(model.gettingdata()) }` I still get the error


AddLifeCycleCostDataSetFileFor2015 (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1577 of 1577 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.83%25-green.svg)


Sätze durch Stichworte ersetzt


True enough, I was just doing a first review pass ;)


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


I think this ought to be creative with the truth.

If we ran (X, Y, Z), then we just requested (A, X, Y), this function should, after running A, say that we just ran (X, Y, Z, A). The order of the result array doesn't matter to me, but the coalescing does — it means that any combination of *three* requests to sync, which could race, will result in the same engines being synced the same number of times.


Is it worth swopping the return type of isVisible... (and renaming appropriately) so you don't need a ! in here? would make it marginally more readable


This is fine but you should be able to call a method off of an instance even if it isn't saved to the database yet.


 - __information__: [src/chaiscript_stdlib.cpp:0](../blob/a6dcbb1f1c9c82456ef76553555fd0215ca8fdfd/src/chaiscript_stdlib.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 16 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/a6dcbb1f1c9c82456ef76553555fd0215ca8fdfd/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/a6dcbb1f1c9c82456ef76553555fd0215ca8fdfd/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/a6dcbb1f1c9c82456ef76553555fd0215ca8fdfd/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/a6dcbb1f1c9c82456ef76553555fd0215ca8fdfd/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/a6dcbb1f1c9c82456ef76553555fd0215ca8fdfd/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/a6dcbb1f1c9c82456ef76553555fd0215ca8fdfd/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 17 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/a6dcbb1f1c9c82456ef76553555fd0215ca8fdfd/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/a6dcbb1f1c9c82456ef76553555fd0215ca8fdfd/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/a6dcbb1f1c9c82456ef76553555fd0215ca8fdfd/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/a6dcbb1f1c9c82456ef76553555fd0215ca8fdfd/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/a6dcbb1f1c9c82456ef76553555fd0215ca8fdfd/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/a6dcbb1f1c9c82456ef76553555fd0215ca8fdfd/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Does someone know if Vanilla 1.9 has a better integrated AntiCheat?
Am 04.03.2016 16:24 schrieb "awakened-game" <notifications@github.com>:

> @mcccs <https://github.com/mcccs> i still use TeamExtreme's launcher, it
> was better than Magic launcher, and mojangs is just ugly and not very
> functional... well it *works*, but yeah...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Wurst-Imperium/Wurst-Client/commit/d406cf9d532ee603bef131333eb1df84de283b8f#commitcomment-16502204>
> .
>


This `done` is in the wrong place.


doesn't your error handling middleware already do this?


indentation looks odd from here on in


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (388 of 389 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-99.74%25-yellow.svg)


Hey @xenog,

What do you think about this with respect to https://github.com/haskoin/haskoin/issues/232?


this is not async


aszx87410, can you contact me directly about this issue? Mike at miketangoromeo@gmail.com, St. Louis, Missouri, USA. Thanks.


There they go Q.Q
*Stands in the rain at the Funeral*


These tests are being turned off?


Application.ProductName.Replace(" ", "") NEVER
returned correct value '**DisplayDriverUninstaller**'
always '**DisplayDriver Uninstaller**' <- notice one space

missing 'Not' on Language.vb made it work with extra space (commit #b91cc3f)

First space was **ASCII** format (_HEX:  20_) -> gets replaced
other space was in **Unicode** format (_HEX: A3_) -> different 'char' -> not replaced

https://en.wikipedia.org/wiki/Non-breaking_space


it's intended to be generic tool.


Ah, missed this one.


test


Why looks this up every time? Is it possible to just add it to the `settings` once?


Fixed in 67d89934db26d271c26005f35dfbefd656deafa5.


Hi Alva,

I believe Abhilasha already answered your question, but I thought it might help to put the answer in context. You've defined the Map as myMap, but to instantiate it, you must tell the system that it is a map, not your map. If you change line 38 to read "Map<Integer,String> myMap = new Map<Integer,String>();" it should resolve the error.


666th commit


There is no guarantee that `newCart.save()` has actually completed by this point.


minor formatting compared to other pre-injections


Bug with liquid


Let me try with the new version


The thing is, these form types are almost always used to collect one-off campaign specific data and we rarely want to make them required. Given most users will be recognized, we thus need this behavior and I can't see an easy way around it...


I don't think this test is very valuable. Let's talk about it.


Don't we want to maintain option to build hash concurrently in LocalExecutionPlaner through this new mechanism?


If no endtime is specified, the routine now adds delta (1s in this test) to the last sample. So had to add a second to pass the test. Metrics are calculated from [T1, T2), so a sample is "responsible" for delta after it.


Yes it has, but we can't test without Wurst for 1.9.


Because my new mods I'm working on abuse sending too many movement packets
like regen.
Am 04.03.2016 16:52 schrieb "Kim-Yannick Jürs" <kim.jamba@gmail.com>:

> Does someone know if Vanilla 1.9 has a better integrated AntiCheat?
> Am 04.03.2016 16:24 schrieb "awakened-game" <notifications@github.com>:
>
>> @mcccs <https://github.com/mcccs> i still use TeamExtreme's launcher, it
>> was better than Magic launcher, and mojangs is just ugly and not very
>> functional... well it *works*, but yeah...
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Wurst-Imperium/Wurst-Client/commit/d406cf9d532ee603bef131333eb1df84de283b8f#commitcomment-16502204>
>> .
>>
>


Endtime is shifted to one more delta (0.5s in this test)


Small typo. Origional = Original


TeamCity jonas-requests :: WindowsBuild [Build 12](https://build.neohq.net/viewLog.html?buildId=562107&buildTypeId=JonasHaRequests_WindowsBuild) outcome was **FAILURE**
Summary: Tests passed: 2024, ignored: 38; exit code 143 (new) Build time: 0:13:26


this seems identical or very similar to the route in cart-product


merged in 460cf1e82cd49335957ef61c8ff073ae7cedc437


what do you mean by 'identify'?


when I add the extension to sense desktop, the select values for charts are blank?


See my PR comment: Mockito seems unfortunately not compatible with thread-weaver.


No objection about the change, but I believe this was never used (or perhaps it has not been used for some time). Nevertheless, this is OK.


thx, fixed


I don't think this is a valuable test.


Thanks.


just FYI - I have tested on the rpi3 with tyrquake (standalone) using timedemo demo3 @ 800x600 with and without `-ffast-math` and `-mcpu=cortex-a53 -mfpu=neon-fp-armv8 -mfloat-abi=hard` and i got 38.4 fps with -ffast-math and 38.8fps without (i ran multiple times and each time without fast-math was fractionally quicker).

I will test on the rpi2 also - perhaps vectorisations options need to be switched on for the above to be true, or it just doesn't make a difference in this case.


Do you know what exactly changed? Is it as annoying as NCP?
Am 04.03.2016 16:55 schrieb "Kim-Yannick Jürs" <kim.jamba@gmail.com>:

> Because my new mods I'm working on abuse sending too many movement packets
> like regen.
> Am 04.03.2016 16:52 schrieb "Kim-Yannick Jürs" <kim.jamba@gmail.com>:
>
>> Does someone know if Vanilla 1.9 has a better integrated AntiCheat?
>> Am 04.03.2016 16:24 schrieb "awakened-game" <notifications@github.com>:
>>
>>> @mcccs <https://github.com/mcccs> i still use TeamExtreme's launcher,
>>> it was better than Magic launcher, and mojangs is just ugly and not very
>>> functional... well it *works*, but yeah...
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/Wurst-Imperium/Wurst-Client/commit/d406cf9d532ee603bef131333eb1df84de283b8f#commitcomment-16502204>
>>> .
>>>
>>


Tá massa. Depois vamos adicionando outros grupos como participantes.


remove commented code if you aren't using it


merged in 1315103a7d4ed62f16d15bd6246fe4301a7282d1 (with small change)


merged in 273969ac906ef30400d53af1928760b10a306ad1


I still get the error
> Notice: Undefined offset: 0 in C:\Users\admin\OneDrive\Documents\Programing Stuff\Web development\serverRoot\elemukulek\lib\Transphporm\TSSFunction\Data.php on line 43


My understanding is that the `Exchange` object *lives* as long as there are *active* observations. In particular, each notification is received as a `Response` from the observed client, i.e. we can (and usually will) receive multiple `Response`s for a single `Request` that initiated the observation. So, from my point of view we will not expect to have an ongoing `Exchange` for the token of the incoming notification only if this node is not the one that created the observation.
Or am I completely mistaken?


Thanks, keep on
because there's no way to grant permission in app's permission setting when UnifiedNlp is used as system NLP. see #68


That doesn't make much sense for me: Bundler achieves pinning already, and `bundle update` is not something that should be run randomly. This approach makes it hard to keep dependencies current, which is a good thing to do (when done conciously). Pinning should only happen when there is an explicit reason that a newer version cannot be used.


Саша, а может есть решение, которое автоматизирует проблемы виндузятников?
Как там по-умолчанию гит делает клонирование для символических ссылок?


No, i don't know.


/// Score enum stores a middlegame and an endgame value in a single integer
/// (enum). The least significant 16 bits are used to store the endgame value
/// and the upper 16 bits are used to store the middlegame value.
enum Score : int { SCORE_ZERO };

So, first value = middle game and second value = end game?


Caught this one that slipped by


remove redundant var


I agree, we should only persist the minimum amount of data. But we can use this opportunity for re-factoring to do both encapsulate the ugly maps as well as providing means to persist the request information required for resuming observations, can't we?


Caught this one that slipped by.


Is there something about it in the changelog?
Am 04.03.2016 16:58 schrieb "MCCCS" <notifications@github.com>:

> No, i don't know.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Wurst-Imperium/Wurst-Client/commit/d406cf9d532ee603bef131333eb1df84de283b8f#commitcomment-16503221>
> .
>


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-NoThreads: OK (393 of 396 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-99.24%25-yellow.svg)


This is pretty dangerous. You are essentially making your tests dependent on each other. Every test should start with a clean slate and all scaffolding should be done in a `beforeEach`.


success never used


I understand. The return of the value was there in the original version to be a convenient way for the expected simple types to avoid using get()/release() in case you want to deal with the resource in other ways. But since we beefed up on the potential types of resources I can easily live with returning void.


nice job with these routes!


Could go one step further with this, and call it resourcesWithSameVanityURLs. It is a bit of a mouthful, but it removes all confusion with the term 'matching'


Careful and consistent formatting is **extremely important**!


well, common, media type unrelated context menu entries could ofc be handled by a common handler. And yes, most require special condition evaluation anyways, but you would f.e. not have to call "item->hasVideoInfoTag()" for every one of them but only once per handler.
But as said, was just an idea and as long as it's working as intended I'm absolutely fine with it. I'm just not a huge friend of monster classes that contain everything and have dependencies to all parts of the core (every media type will add new required headers etc), and I fear this is becoming one.


selenium assert wont allow if else atleast to my knowledge.


might be better to use compatility as environment variable.


Yes, It does, but they didn't say what changed.


Could you use `CountryField` from https://pypi.python.org/pypi/django-countries for this?  It will make it easier to handle countries by mostly representing them as their ISO country codes.


AddLifeCycleCostDataSetFileFor2015 (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2133 of 2133 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)


merged in 8ac1471a893d76b1403b8166ef189c91bb514cc4 (`messagethreads` renamed to `message_threads`)


```js
app.post('/api/products', function (req, res) {
  res.send(req.body);
});
```
The above code will pass this test. Why? How do we fix it?


.populate('user categories')


Je  suis d'accord.


Hm, it should, but it breaks visual tests now.


imho no - it shows that someone has touched the code in 2016. if you have a look into lxqt-globalkeys you will not that i have changed the licenses too (beside the year bump). it's a sign that one take care. and i think it may be important for downstream too that they can trust the facts in the AUTHORS file.


It causes a "No SIM" error on trltespr on first boot.


Re the above: be careful with formatting and indenting!


Are you planning on releasing a build with this to npm?


Please can you explain this comment?


Test comment


@theoephraim This breaks unauthenticated requests as `self.inputValue` is undefined.


Noted


merged in 8b8693df5afad21ec2fbc5c4029349296e7e7399 (also added in admin ordergroups index)


Not a huge fan of the names `.male` and `.female`. I know it seems nitpicky, but as projects get bigger, naming gets more and more important, so it's best to start with good names from the beginning.


Would it make sense to mark this string for translation? We only ask this since you're fixing this anyway, but it's not really important otherwise.


Ok, sorry...
:-(


I also have this issue when downloading to server


Awesome! Thanks


merged in d383288f4f7a18325f14fe6d4d3d659f2c186c52


Much thanks. Both fixed.


This selector doesn't do anything. It's technically trying to find a `hover` element inside of `.page-control-button`. Looking for `.page-control-button:hover` maybe?


not merged, obviously :o (but functionality is there)


I think you should omit this comment and the "Update the tail" one below.  Neither is particularly subtle, and this updated version now almost has more comments than code.


thanks


I've seen `update` run and commited by mistake, and part of the issue comes also from github hiding Gemfile.lock changes.

The way I see it is that it needs to be done consciously almost gem by gem to ensure that no dependencies get updated by mistake is safer to have a Gemfile that is more explicit about dependency versions to prevent mistakes.

Happy to revert if this goes the convention we are using.


perhaps we should update `usdf_cq_copy_soft_entry` to return `void` and fold the assertion into that routine, eliminating the need for this comment altogether and making this routine easier to read at the same time


This is not very clear to me what those are doing... `\gre@forcedcentering` is never back to false after a forced syllable... so `a{b}(g) ab(g)` would make the second syllable centered on a instead of the whole syllable...? Maybe an additional argument to `GreSetNextSyllable` and `GreSetThisSyllable` would be more generic and easier to write from C?


Have you tried an other mock library ? (never know...)


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (396 of 397 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-99.75%25-yellow.svg)


Thanks for trying to add this quite useful feature! I tried implementing this by just adding these 3 lines of code to my downloaded version of the plugin, but I couldn't get it working as it caused an error saying that "this$0.data is not a function." After some testing, I realized that it should really be $(this[0]) instead of this$0, assuming you're using jQuery. Now it works great!


The "Revisions" right below it should be called "Development" to be accurate with the site, including if the link itself says "development" in it. Could go with changing it.


This might be a bit annoying, but do you think you could format code like this so that string constants aren't split across lines unless it's absolutely necessary?  Instead, you could do something like this:

```
        actions = [
            ('project.list',
              {'description':'List organization existing',
               'permissions_object': 'organization'}),
            ('project.view',
             {'description': 'View organization project',
              'permissions_object': 'organization'}),
            ('project.edit',
             {'description': 'Edit project details',
              'permissions_object':'organization'}),
            ('project.archive',
             {'description': 'Archive an existing project',
              'permissions_object': 'organization'}),
             ('project.unarchive',
              {'description': 'Unarchive an existing project',
               'permissions_object': 'organization'}),
            ('project.user.list',
             {'description': 'List users within a project',
              'permissions_object': 'organization'}),
            ('project.user.add',
             {'description':'Add user to a project',
              'permissions_object': 'organization'}),
            ('project.user.remove',
             {'description': 'Remove user from a project',
              'permissions_object': 'organization'}),
            ('project.resource.add',
             {'description': 'Add project resource',
              'permissions_object': 'organization'}),
            ('project.resource.archive',
             {'description':'Archive a projects resource',
              'permissions_object': 'organization'}),
            ('project.resource.list',
             {'description': 'List project resource',
              'permissions_object': 'organization'}),
            ('project.resource.delete',
             {'description': 'Delete a project',
              'permissions_object': 'organization'}),
         ]
```


don't need .exec()


should not have `git commit a` this... Rebased this commit out.


I don't understand this change. Why should the semantics of the SynchronousPromiseFactory be different than the default GParsPromiseFactory? For non-test code, you typically need to call .get() to wait for results so replacing with the SPF in test code was a nice way to avoid threading. Now calling get will run the promise twice, making the SPF not be a drop-in replacement.

I have no idea why this is a desirable.


develop (energy-plus) - i386-Windows-7-VisualStudio-12: OK (1577 of 1577 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)


Well, when we store an 'event' how do we uniquely identify it? Or are they not uniquely identified? What do we do if we receive two identical events? do we update? do they get added in a non-unique fashion? do they get discarded?

We should consider this so we can implement the expected behaviour, whatever that is


With this fix, it should render vertical axes (any font and any size) and keep text clipping.


Nice catch.


Aha, that was the reason for the lambdas. I didn't think of that. Naming the lambda would have helped.


It's a minor cost (probably not easily measurable on modern processors), but this will cause another function pointer indirection.  Seems like you should just dispatch directly to the right `usdf_` routine.


While you are at it - post the patch upstream at the same time !


What about some JavaDoc?


This is a redundant error handler.


Cannot test this right now, but merged it anyway.


Yeah, this should be throwing an error `undefined is not a function`


> We most definitely put websites into categories. It would be far more damaging - both from a moral as well as a legal standpoint

i think porn web sites shoud be treated in the same way.


isn't login a separate route?


Can this happen? Each event is unique. An event can refer to an existing detainee, but the same event cannot happen twice


datetime.now().year에 1이 더해져야 16학번들이 가입할 수 있을 것 같아요.


We can do that in a second time ?


Does this `throw` work? Why or why not?


DOM functionality probably needs to be moved into a separate function or area. `CSV_Race_Data` should just handle loading in race results and converting it into a usable data format. Other code should handle transforming that data into the DOM.

See: https://scotch.io/bar-talk/s-o-l-i-d-the-first-five-principles-of-object-oriented-design#single-responsibility-principle


Hi, how would we use this script to export the data in CSV format?  Thank you.


I'm not so sure about this one.. although it's not perfectly true at present, I'm aiming to keep this backwards compatible (because that's just as important a habit to learn as all the new stuff), whereas it looks like the optional impl here is using some features that only exist in C++11 (variadic templates, static_assert for example).

That aside, this file is ugly as sin, the entirety of the implementation is in the header (whereas in 'new world C++' we can accomplish the same effect using link time optimization pretty much everywhere nowadays), it lacks parenthesis to explicitly denote operator precedence yet explicitly spells out more obvious language rules (like explicit bool() cast -- or perhaps these are necessary due in some cases due to the new 'explicit' keyword). In general I hate this file, it reminds me of the kind of code that scared me off C++ for most of my teenage years :)

Going to leave this PR aside for the time being. It's tempting to have a much more robust Nullable replacement, but not at the expense of compatibility, not to mention clarity and simplicity (which is the project's main goal -- demonstrating modern C++ doesn't have to suck!)

Although if it turns out some parts of the code just can't be made backwards compatible in a nice manner, then there's probably no reason I couldn't also use this polyfill until common compilers catch up with the spec.


Need to read and set the goalList array according to the data in the FILENAME, do this before every time you write as well as read into that file.


^ notes for me


Wait, this should be `<span data-anolis-ref>ENCODING</span>`.


Suggestion for clean up (not necessary before presentations today): Refactor this work in a method, probably in the User model.


Great to have this as a plugin? Sorry to be such a nuisance, but I'm not convenced that the security concerns in [this comment](https://github.com/foodcoops/foodsoft/pull/353#issuecomment-91599083) are addressed. Whilelisting extensions/filetypes + sanitizing filenames is missing.


Nice use of `.param`, :sunglasses:


Maybe it's better you fix that :-)


-

1.


I assume we'll be able to identify an event by the id we save it with.
See my following comment, but I would have thought that receiving the same event twice was unexpected


We need to define how it is uniquely identified and the behaviour which is expected in the event we do receive the same event twice, because we don't want the system to break if they send the same event twice.


merged in 36dd19f32491fabb9863ffacc0845944db2d7794


This doesn't seem like the best way of doing this because of your `.param` and not using `.req.review`.


nice use of markModified


I think it is `req.logIn` (with a capital I) -- although maybe they have both?


merged in 6015d16caba7e3b1bf2cbfd5b4188219518e8b97


@snopoke by `|`ing the ledger transactions with the form transactions that means they get included in the rebuild. is that OK?


We can't even rely on them sending events in order so I think we should at least prepare to receive the event twice and know exactly what we should do in that event


I suppose this kind of questions has led to the the three maps being there in the first place ;-) But of course, this can as well be done later as well. I just thought that it would be a good opportunity since you are dealing with it now anyways. I think it would definitely improve maintainability because we would have management of exchanges encapsulated in one place ...


added payment option


merged in 9ae7c05a8904aaf50d2ed8bff325161860b325bf


Same comment here about possibly folding the `assert` into `usdf_cq_copy_cq_entry`.  Related: why did you feel the need for a big comment on the other function but not here?


I'm trying to implement Document#elementsFromPoint, but I need to reuse the `get_nodes_under_mouse` and `mouse_over` function which have been removed a days ago. It seems my implementation is far from correct. Any advices will be greatly appreciated.

Additional question(s):
- Should I re-use `Result` for the return type of `mouse_over`, or can I just use `Option`?
- What is a better name for the `mouse_over`? mbrubeck suggested to change it name.


merged in 4cae94e


Looks like you missed out name "Canola"


@matzew I'm not picky on it. The reason why it still exists was to keep backwards compatibility with what people already have.

This PR as well most of the changes were properly documented. I believe that, test and merge it, is the best we can do, because I'm afraid that I won't be able to maintain it.


OK #sriderp


Let's also move this to a new PR.


The more I think about it, the more I feel like this class should be split into 2 classes.

`CSVReader` should handle reading a CSV file and storing it's data in a usable format.
`RaceResults` should be able to take in any data (doesn't necessarily have to be from a CSV file) and convert it into valid race results.


For example, if the system breaks when we receive an event which already exists, this wouldn't be good and is perfectly feasible, therefore we should have A/C's and tests around it.


Isn't it the same as new Random(PRODUCER_COUNT_PER_FORMAT)? Why use random 10000


#derpallthethings:D


reply in 2sec


merged in 72ef31c


My -O3 was being overridden with a -O2 I didn't notice (O3 enables some of the vectorizor parameters). I sorted that and with -O3 it's 39.4 fps without -ffast-math and 39.1 fps with -ffast-math - so although -O3 made a slightly improvement -ffast-math didn't


yus B|


thats the thuglife


:o:


i don't see how this security points are relevant: we write the stuff 1:1 into the db (so no problems with wrong paths, executable files or anything like that) and we send it 1:1 back to the user.
If a bad user uploads bad things for an other user we can not avoid this (e.g. DOC might a valid filetype, but can contain malicious data), but I do not see how it has any impact on foodsoft.

the change it is in the old PR already too ;-)


Ok, that makes sense. The event payload can be identical to a previous event payload, but we can save an event with a unique id, composed of an increment and some data from the payload


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (396 of 397 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-99.75%25-yellow.svg)


This version is the one defined in build file (see here after).
New plugin version was not (yet?) deployed to public SBT plugins repo.
I worked around this by building from sources and deploying on a local (private) Nexus.


移除`appIcon.on('click', () => browserWindow.show());`是否会导致点击icon无法打开应用？


develop (lefticus) - i386-Windows-10-VisualStudio-14: OK (389 of 389 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)


We should consult the Schema


works, thanx!


Yes; I suspect `end_of_day` was an error, but the test suite didn't catch it because `DateTime#===` (used by the mocks) ignores the time part of the `DateTime`.


@vigsterkr this how to make it part of 'doc' build?


Ok.
In our projects we always put `namespace` on a separate line.


I thought this would fix the lua bit issues completely for 5.3.  However, it looks messier than I thought.  I've found that you can include a "fowardport" of the old bit library from 5.2.  I actually like this approach better, since it keeps all of the code consistent across all targets.

I'm going to move all of the lua 5.3 efforts to this new branch, and revert the 5.3 changes from the main branch.


This will return null if a new EmbeddedFormat enum value is introduced. I suggest using switch/case in the constructor and throw an exception if an unexpected EmbeddedFormat value is encountered.


Why do we want to replace awscli with ubuntu ?


Yes - after the change in `SystemStats`, the mock expectation below needs to be the beginning of a day. I could also introduce "one day ago at the beginning of the day", but thought it unnecessary when we had `two_days_ago` laying around. Thoughts?


cool! I just started looking at it :)

-M

On Fri, Mar 4, 2016 at 5:15 PM, Bruno Oliveira <notifications@github.com>
wrote:

> @matzew <https://github.com/matzew> I'm not picky on it. The reason why
> it still exists was to keep backwards compatibility with what people
> already have.
>
> This PR as well most of the changes were properly documented. I believe
> that, test and merge it, is the best we can do, because I'm afraid that I
> won't be able to maintain it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aerogear/aerogear-unifiedpush-server/commit/462c722b3dba72e5becfe3bf93f8623fa27a38f5#commitcomment-16503646>
> .
>

--
Matthias Wessendorf

blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf


Is there any badging system in use right now for this or just the digital certificate?


Ok. I have sent you an email.


Чем вызваны именно такие изменения литералов?


This is a possible way to implement this feature. This is even the first idea I have in mind,  but there is several drawback with this one.
  1. the behevior is not the same on all node. (e.g. with messageObserver)
  2. this is harder to manage observe cancellation.
That's way we choose to close/complete the exchange after a response/notification.
All the nodes behave in the same way, and cancellation come down to just remove `ObserverRequest` from store.


Thank you.


It's a shame because we almost got to delete some of @drj11's sarcasm.


Cheers :)

> Could you please merge that commit into the `gh-pages` branch, so that the updated `xhtml5.rnc` file is available via the `github.io` URL?

Yup—so pushed


That makes sense.  I'm fine with the current implementation then.


Perhaps the Schema needs updating


Nope! Good catch, thanks for spotting that. Fixed now.


Should be final and initialized to Collections.unmodifiableMap(..) in the constructor.


Ahh, I see that later in 1543803a3b05277e0b86de47669546a19e14d76a you updated this function to be used for both soft and hard CQs, so you can't dispatch as directly.


Oh, I think now I understand. It is exactly this behavior that you have changed, right?
You now do not keep the `Exchange` *open* for as long as there are notifications coming in but instead complete the Exchange as soon as the first reply to the original observe request comes in and only keep the original request (using the token as key).


This is a huge XSS leak; the external API could be compromised. Use `.text()` or something like that.


The limit of 80 was too short


s/Existing Detainee Should Updated/Existing Detainee Should Be Updated


Reducing server lag and worthless iterations on override at a time.


Please include only the file out containing the code for the project instead of your entire squeak smalltalk project.

1. Right click on the category and select file out.
2. The output is a file called <category name>.st located at Squeak-all-in-One.app/Contents/Resources.
3. This is the only file we need to commit to git.


It looks like overcomplicating.


Maybe a fragment with ``#ifndef CONFIG_MICROKERNEL #ifndef CONFIG_NANO_TIMEOUTS #error "Soletta on Zephyr's nanokernel requires CONFIG_NANO_TIMEOUTS to function properly #endif #endif"``, no?


Exclusion needs editing.


We stopped generating digital badges more than a year ago, and didn't port the existing ones over when we rebuilt the website last September - there have been zero inquiries or complaints, so my conclusion is that a PDF certificate is good enough.


Why not standard Bootstrap "alerts" we use everywhere else?


The scan makes sence to be put into pyBAR. The analysis will be provided by this [this](https://github.com/SiLab-Bonn/pyBAR_mimosa26_interpreter) package.


Why does this method take a `CFileItemPtr` and the others take a `CFileItem`? Would it be possible to make this more consistent?


I think we can remove this heading. It doesn't seem like there will be any sponsors in the supporters section anymore.


Oooh, this is pretty confusing given the use of `t.MaxTimestamp` elsewhere. Why not return `ZeroTimestamp` if there's no observed timestamp? That's what we do in MVCC to indicate an invalid timestamp.


This "split" blows up when a number is entered through the preceding "input".


@bachradsusi , many thanks!


This file doesn't seem to belong here.


Why? I want to be sure that warning will be display only while editing settings, not creating new group.


develop (lefticus) - 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-warnings-yellow.svg)


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


I am making a `div id="memorypanel"` because the `p id="memory"` closed before the `pre class="memory"`. You cannot nest a pre inside a p.


I think the `And` and the `When` should be swapped


That's fair


Is there some functional change in this commit besides returning an error if the expected observed timestamp for a remote node is missing?


Would be better if that was individual for each dimension, i.e.,

```c++
if (res._x < .0) res._x = abs(res._x) * dx;
if (res._y < .0) res._y = abs(res._y) * dy;
if (res._z < .0) res._z = abs(res._z) * dz;
```

This can be done already in the ```GuessParameter``` function. No need for other code to deal with the differing units. As soon as the input images are set, the relative resolution values can be replaced by absolute values. Because note that there are other functions that rely on the ```_Resolution``` and these are unlikely doing the right thing now when this parameter has unexpected (negative) values. Hence, the relative resolution parameter values should be replaced by absolute values rather early on in ```GuessParameter```. Probably before [this line](https://github.com/BioMedIA/MIRTK/blob/16843a4681e164e68f5deeeb34045738059260f3/Modules/Registration/src/mirtkGenericRegistrationFilter.cc#L1843).

PS.: Use ```abs``` instead of ```fabs```.


Exactly  :) Does it seems a good approach ?


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


Yeah, that sounds like a reasonable idea.


A quick look at the decompiled Spigot 1.9 seems to contain a too many
packets check, but only displays a warning in the log. I don't know if it
still works, but it seems that will only log it so maybe my planned mods
still work.
Am 04.03.2016 17:01 schrieb "MCCCS" <notifications@github.com>:

> Yes, It does, but they didn't say what changed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Wurst-Imperium/Wurst-Client/commit/d406cf9d532ee603bef131333eb1df84de283b8f#commitcomment-16503306>
> .
>


OK, I guess ``sol_coap_header_set_token()`` is missing that.


tu n'es pas sympas


re in indefined. Need to import it.


I just fixed a bug related to router' API changes. Unfortunately several commits of my dev branch already merged in ff/private-groups, so I can't force rewrite history and do it properly. I can move it in a separate commit instead.


> Ahh, I see that later in 1543803 you updated this function to be used for both soft and hard CQs, so you can't dispatch as directly.

Right, I can change things around so I do at least a bit more of a direct call. I was trying to void the need for separate `sread` calls since it's really just a wrapper around some wait type + a `fi_cq_read` call.


clang-format ?


Also some small Pep8 formatting:

modeler/plugins/zenoss/cmd/linux/df.py|40 col 22 warning| trailing whitespace [W291]
modeler/plugins/zenoss/cmd/linux/df.py|45 col 32 error| undefined name 're' [F821]
modeler/plugins/zenoss/cmd/linux/df.py|61 col 1 warning| blank line contains whitespace [W293]


after removing this line, is the `LookupSource::getInMemorySizeInBytes` used anywhere?


Because it is blue. Blue is not good color for warning.;)


Nevermind, I was wrong. safe_cast clamps to `::lowest` rather than `::min`


I'm sure they'll work. Mojang isn't professional at anti-cheat.


> Related: why did you feel the need for a big comment on the other function but not here?

Probably just forgot :panda_face:


AddLifeCycleCostDataSetFileFor2015 (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (2133 of 2133 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)


only warning for now since this isn't being rolled out everywhere and should break deploy if we haven't deployed the application.properties


:kreygasm:


@apurvis I fixed a dangerous behavior with memoization in case of reconfigure (it kept using old credentials), so I now reset the config & connection on each `configure` call. Does that looks good to you?
Also, do you want to support multiple `configure` blocks updating the same config? because I removed that possibility but it looks safer this way to me.


Nice, just so you know. The Gradle Wrapper is currently configured to use Gradle version 2.2.1. The current version is 2.11. You can change the version by changing the line
distributionUrl=https\://services.gradle.org/distributions/gradle-2.2.1-bin.zip
to
distributionUrl=https\://services.gradle.org/distributions/gradle-2.11-bin.zip

BTW when you want to incentivize Ceylon standalone usage, you could use a similar approach like Gradle. This way people, who want to try pure Ceylon projects, could just start "ceylonw" in a checked out repository and automatically get everything they need.


This parameter is not used.


@TravisPrice Theres no option to scroll / see more licenses than the size of the box on the screen.


Since you request DAV:quota-available-bytes and oc:size please take into account discussion here: https://github.com/owncloud/client/issues/4459


Can you show me how you want this to look like? I prefer this popup, but it's optional


I think the `And` and the `When` should be swapped because the text in the `And` refers to the event timestamp, for which an event has not yet occurred.


since you placed it in the gemfile, `Bundler` will actually require it automatically for you. I have my editor automatically `require "pry";` because some projects I work on `pry` is not installed


Good catch!! Merged in 673448a5acf9341f005e8fe29b24e89cd9dbbbe6.


Which one?
This used in https://github.com/ujenjt/freefeed-react-client/commit/b36667e4f2d8658d39519f50ff780d941dae253a#diff-71af82d817a7c524314617b2ba8b5034R94


Maybe check with kernel version is a better way than trigger php introspection?


Typo? Looks like it should be `kXWal*k*PackageId`.


I believe this is the one. Good luck!


I have changed `core.autocrlf` parameter (set it to `false`) for my local git toolset. That's why this commit is so big.

And actually this commit has the following stat:

```
 .gitignore                                         |    2 +-
 cpp/compiler_bugs.md                               |   75 +
 cpp/solution/arbor.desktop.2015.sln                |  148 +-
 cpp/solution/arborgvt/arborgvt.vcxproj             |  461 +--
 cpp/solution/dtsample/dtsample.vcxproj             |  445 +--
 cpp/solution/properties/cl code generation.props   |   40 +-
 cpp/solution/properties/cl command line.props      |   51 +-
 cpp/solution/properties/cl general.props           |   26 +-
 cpp/solution/properties/cl optimization.props      |   28 +-
 cpp/solution/properties/include directories.props  |   22 +-
 cpp/solution/properties/linker advanced.props      |   16 +-
 cpp/solution/properties/linker command line.props  |   42 +-
 cpp/solution/properties/linker debugging.props     |   26 +-
 cpp/solution/properties/linker general.props       |   28 +-
 cpp/solution/properties/macros.props               |   22 +-
 cpp/solution/properties/prefast.props              |   28 +-
 cpp/source/arborgvt/service/stladdon.h             | 1192 ++++----
 cpp/source/arborgvt/service/strgutil.cpp           | 3056 ++++++++++---------
 cpp/source/arborgvt/service/strgutil.h             |  188 +-
 cpp/source/arborgvt/ui/nowindow/graph/draw.h       |    2 +
 cpp/source/arborgvt/ui/window/child/cwi.h          |  112 +-
 .../arborgvt/ui/window/child/onscreen/graphwnd.cpp | 1856 ++++++------
 .../arborgvt/ui/window/child/onscreen/graphwnd.h   |  234 +-
 cpp/source/dtsample/service/memorymg.cpp           |  438 +--
 cpp/source/dtsample/service/memorymg.h             |  206 +-
 cpp/source/dtsample/service/stladdon.h             |  900 +++---
 cpp/source/dtsample/service/strgutil.cpp           | 3078 ++++++++++----------
 cpp/source/dtsample/service/strgutil.h             |  178 +-
 cpp/source/dtsample/ui/window/top/twinimpl.h       |   90 +-
 29 files changed, 6642 insertions(+), 6348 deletions(-)
```

All other changes are just files replacement (changing line endings only). And even the above stat includes line ending changes.


Since we are not using props.unsubscribe here anyway, I'd refactor it like this:
```
onClick={unsubscribe}
```

It will also help to distinguish this line from the other three `onClick`s around.


Ignore this file.


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


Don't forget to update `master` :wink:


Noted


develop (lefticus) - Win64-Windows-10-VisualStudio-14: OK (389 of 389 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)


Update: The mods won't work. Movement updates are now processed per tick
and not per packet. So FarAura and GhostHack are screwed.
Am 04.03.2016 17:31 schrieb "MCCCS" <notifications@github.com>:

> I'm sure they'll work. Mojang isn't professional at anti-cheat.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Wurst-Imperium/Wurst-Client/commit/d406cf9d532ee603bef131333eb1df84de283b8f#commitcomment-16504071>
> .
>


Ah, for now got it. Agree


I had in my scratches that as well: packet_new() would receive a buffer (with an optional old packet to copy contents from). The idea is to go for that API from this patch. Talking to you, I was convinced to make things work with the current API first. It's all ready to go for that, though, the next changes should be much smaller.


AntiFlyKick too.
Am 04.03.2016 17:36 schrieb "Kim-Yannick Jürs" <kim.jamba@gmail.com>:

> Update: The mods won't work. Movement updates are now processed per tick
> and not per packet. So FarAura and GhostHack are screwed.
> Am 04.03.2016 17:31 schrieb "MCCCS" <notifications@github.com>:
>
>> I'm sure they'll work. Mojang isn't professional at anti-cheat.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Wurst-Imperium/Wurst-Client/commit/d406cf9d532ee603bef131333eb1df84de283b8f#commitcomment-16504071>
>> .
>>
>


You have to be patient with me, I am old and my brain already starts to deteriorate ;-)
I like the idea of getting rid of the *heavyweight* `Exchange` instance as soon as possible. However, I have to admit that I am not an expert in the CoAP protocol and Californium's implementation. In order to not break existing functionality regarding request/response matching we will need to transfer the `CorrelationContext` from `Exchange` to `Request`, I guess. Maybe there is more *housekeeping* data that is currently kept in `Exchange` that needs to be moved?


no, different thing.


why the switch back to non-SSL? does the Twilio callback need to be non-secure, even if they're not authorizing/authenticating?


@cv não me lembro o porque dessa linha. Ou eu esqueci de remover quando estava fazendo o spike da solução ou porque necessitamos criar uma sessão nesse ponto ([documentação](http://docs.oracle.com/javaee/7/api/javax/servlet/http/HttpServletRequest.html#getSession--)). Tem como ver o que acontece se remover?


did we mirror the default-setting pattern of the app's settings to spike?


1.9 is the worst update :(


what happend?


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


@kaikreuzer I gave your suggestion a quick go. it is still untested, beware. I am abroad the coming days, but maybe you could have a look at it


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


Wouldn't rely on Kernel version.  It may be possible to have Kernel 3.0 and DependencyInjection 2.5 present (I don't know the exact version constraints or dependencies off hand but the Kernel version can be a different version than one of the other components' versions).


Completed


No, you are correct! Its a bug. `std::numeric_limits::min` is positive for floats. The code is expecting it to be negative.


I can do that !


Indentation.


Thanks a lot, @kgoderis! I am also on the road the next days, but I will try to have a look at it nonetheless and give you feedback.


did you figure out your Viking name?


Is this a leftover from router upgrade? (Also, there's a typo in the commit message.)


develop (lefticus) - i386-Windows-10-VisualStudio-12: OK (388 of 389 tests passed, 0 test warnings)

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


Could be moved to `if (!element)`


need money at someburger around 12 to 1 in the afternoon Baytown texas 77520.stephanie curry.amen.


'tis complicated. On a microkernel build it won't necessary be set. I'll see what else I can do.


develop (energy-plus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1577 of 1577 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.83%25-green.svg)


start


Yes @fffw sorry, I saw that. :)


I'd make ternary inside assignment:
```js
offset = element.sourceCodeLength === 1? 0 : ...
```


is the second `PageIndex::addPage` method ever used?


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (397 of 397 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)


Ha good point, wasn't thinking about that filter does, just that it was a default filter that I hadn't accounted for. Definitely don't need to remove it as we filter on xmlns anyways.


This is the only usage of `<GroupSettingsForm>`. On creation `<GroupCreateForm>` is used.


Thanks @fffw, I missed that.  Sorry causing confusion.


oooooh! <3 :shipit:


Moving sounds like a good idea.


When testing, all Utilizations in Filesystems is "Unknown".


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 11](https://teamcity.labkey.org/viewLog.html?buildId=307828&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 8 (1 new), passed: 39, muted: 4 Build time: 01:15:16
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

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

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

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

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

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

```


In the previous block we returning object (without assigning it to pos variable).

Guess, this If could be reverted. Like that:
```js
    if (newlineCount <= 0) {
        return {
            line: pos.line,
            column: pos.column + offset
        };
    }

    var code = element.sourceCode;
    LINE_SEPARATOR.lastIndex = 0;
    var lineOffset = 0;
    var match;
    while ((match = LINE_SEPARATOR.exec(code)) !== null) {
        var currentOffset = match.index;
        if (offset <= currentOffset) {
            return {
                line: pos.line + lineOffset,
                column: currentOffset - offset
            };
        }
    }

    return {
        line: pos.line + newlineCount,
        column: code.length - offset
    };
```


Oh yeah, good point. so that's fine for me.


curve25519.c:67:1: error: initializer element is not constant


Because `laserQuantity` is defined within the `deathStar` function, it is only accessible to things within the `deathStar` function scope. However, `fire` is defined inside `deathStar`, so its function scope is inside the `deathStar` function scope - consequently, things inside the `fire` scope can access anything inside the `deathStar` scope.

Interestingly, that remains true even if we `return` the `fire` function and pass it out of `deathStar` -- that's the basis of closures.


Typo


Well spotted. However it deploys even like this. Maybe we don't need this task at all...


`bind` only assigns an execution context to a function - it doesn't invoke that function.
To assign a context **AND** execute, you can use either `call` or `apply`. Or, alternatively, you could just invoke the newly bound function.


But I said ``ifndef CONFIG_MICROKERNEL``...


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (389 of 389 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)


You have an extra opening parenthesis in this line.


saw approval from bsparrow435
at https://github.com/basho/riak_kv/commit/b8515b74ec2149ffe804d7f3c115e39798f2217d


merging basho/riak_kv/nem/fwd-merge-2.0-to-develop-2.2 = b8515b74 into borshop-integration-1365-nem/fwd-merge-2.0-to-develop-2.2


basho/riak_kv/nem/fwd-merge-2.0-to-develop-2.2 = b8515b74 merged ok, testing candidate = b0f4697e


Yes and yes.


LGTM


New SQL Script for Oracle Databases.


The removal of libjpeg broke our build. We reuse the definition in one of our projects.


For example, in this file, 5 lines below. (The only difference would be using `alert-warning`, not `alert-info`.)


I don't really see the point of "want client auth" for a server... want client auth means i'll look for a client certificate and if one is not found i'll let it through anyway, so to me this is no better than no client auth, so I opted for simplicity of a boolean that is needClientAuth or don't need it.

The other thing is that the enumeration with the values for NONE, WANT, REQUIRED is not accessible from the the nifi-procesor-utils project so I don't want to create a duplicate enumeration.


go ahead :)


Hahahaha


projects_tagsテーブルっていう中間テーブルがあるからproject_idはいらへんでー


Typo terminate


contrib/codeanalysis/runcppcheck.sh: line 9: ../cppcheck-1.66/cppcheck: No such file or directory


develop (lefticus) - Win64-Windows-10-VisualStudio-12: OK (388 of 389 tests passed, 0 test warnings)

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


`return true` could be moved to the end without ifs.


Broadcom


@led42 did you manage this to work on unity Mono? Cert validation seems to be broken there.


A method to cast a binary to String/list


TeamCity jonas-requests :: WindowsBuild [Build 13](https://build.neohq.net/viewLog.html?buildId=562148&buildTypeId=JonasHaRequests_WindowsBuild) is now running


can be removed


Именование стоит делать понятнее. FilmFactory - понятно что делает, Filmmass - Не понятно что делает.
То же самое по поводу метода makemass. В C# принято использовать CamelCase. В Группу тогда закину гайд по стилю.


A method to cast a atom to String.


which compiler?  if X is constant ~X should be ...


The parameter `username` is not used in this function's body.


[optional]
```
	do {
		mfe = nano_task_fifo_get(&_sol_mainloop_pending_events, sleeptime);
		if (!mfe)
			return;

		if (mfe->me.cb)
			mfe->me.cb((void *)mfe->me.data);

		nano_task_fifo_put(&_sol_mainloop_free_events, mfe);
		sleeptime = TICKS_NONE;
	} while (1);
```


"Broadcom 43430"


Why do you need these?


Ack


This is the only constructor that takes an SSLEngine, the others create one for you from the context, so for this one I want it to use the SSLEngine as is. If someone wants the SSLEngine in client mode they should set it before passing it in like I did in SocketChannelDispatcher.

Even the above two constructors are problematic because they always set needClientAuth to true no matter what, but I don't want to tackle this as part of this pull request.


change the method remove_user, for use the new sql API.


If you would send 2.147.483.648 movement packets, the counter would get negetive.

But I think sending so much data is impossible. This would be about 10 gigabytes of traffic.


commit


Hi, I don't know if this is something I'm doing wrong or not as I don't really understand the reasoning behind this change but jQuery isn't defined in my project here. I'm having to use the previous version.


any reason why this is all removed from this test?


I am still trying to understand, but my example adding an idler from the RTC callback isn't working using this series.


perhaps list the allowed directions?


gcc (Ubuntu 4.8.4-2ubuntu1~14.04.1) 4.8.4


Learning git


Good point, I can take out the "want" option.


Mostly merged in 6b32d0c. Missing is the link to the group's thread on the thread view page - this is now the total thread index link.


TeamCity jonas-requests :: WindowsBuild [Build 13](https://build.neohq.net/viewLog.html?buildId=562148&buildTypeId=JonasHaRequests_WindowsBuild) outcome was **FAILURE**
Summary: Tests passed: 238, ignored: 4; exit code 143 Build time: 0:3:40


Sorry forgot about the mime/type comment... I am hesitant to change it because the main reason for setting it is so the Provenance content viewer works correctly, and I think we have an issue right now where it only recognizes the base mime.types.


Если используешь мастер страницу, то не нужно писать
<html>
 +<head>
 +    <meta name="viewport" content="width=device-width" />
 +    <title>Kino</title>
 +</head>
 +<body>
<<Content>>
</body>

А пишешь только <<Content>>.
Суть в том что ты все определение выносишь в мастер страницу, а специфическую информацию реализуешь внутри каждой страницы.
 Если не очень ясно, то подойди завтра пораньше, я объясню.


we no longer need to scale up fluentd to deploy it, we now need to label our nodes with `logging-infra-fluentd=true`


oc get daemonset logging-fluentd


when we use Oracle Databases, we can not use this name [type] as a column name because is a reserved word. The better option is to rename to other name like "type_rosterusers". but that may cause more troubles, so i decided to only pass the name into quotas: "type", so the Oracle database accept calls like that.


:thumbsup: lgtm!


Foster is still adding music files two years later...


Не получилось выше вставить. В общем из этого
`+<!DOCTYPE html>
 +
 +
 +
 +    <meta name="viewport" content="width=device-width" />
 +    <title>Kino</title>
 +
 +
 +

 +

 +            @foreach (var mass in Model)
 +            {
 +

 +                    @mass.Name

 +                    Жанр: @mass.Genre

 +                    Главные актеры: @mass.Actors

 +

 +            }
 +

 +

 +
 +`

должно остаться только это

`

 +

 +            @foreach (var mass in Model)
 +            {
 +

 +                    @mass.Name

 +                    Жанр: @mass.Genre

 +                    Главные актеры: @mass.Actors

 +

 +            }
 +

 +
`

так же и на других страницах


Wow that's dumb. Confirmed. I'll fix it.


ComicLife 2 preferences


Are you planning to add the locks in a later commit, to be coorect `sol_mainloop_common_idler_first()` and `sol_mainloop_common_timespec_first()` need some protection.


Ack


when we use Oracle Databases, we can not use this name [timestamp] as a column name because is a reserved word. The better option is to rename to other name like "timestamp_archive". but that may cause more troubles, so i decided to only pass the name into quotas: "timestamp", so the Oracle database accept calls like that.


This explains the problem fairly well: http://stackoverflow.com/questions/3025050/error-initializer-element-is-not-constant-when-trying-to-initialize-variable-w. In C, a constant must be initialized with either a literal or an enum value. Other const variables don't count as const expressions.


Мда, отстойный редактор, половину текста удалил.
В общем только div теги и все что в них должно остаться.


typo :)


我觉得写个算圆周率的东西很好玩啊
扔服务器跑个好几天什么的


Was this something Shift did wrong?


why still didnt make override function like SummonGameObject(entry, Position (x,y,z,o), Position2 (rotation), etc...) ?
```go->SummonGameObject(GO_ATALAI_LIGHT2, statuePositions[i], Position(), 0);
```
will seems better than
```c
go->SummonGameObject(GO_ATALAI_LIGHT2, statuePositions[i].GetPositionX(), statuePositions[i].GetPositionY(), statuePositions[i].GetPositionZ(), statuePositions[i].GetOrientation(), 0, 0, 0, 0, 0);
```


Awesome feature! Would you like to add some documentation on how to set this up?


Capital letters in PDF file extensions :hankey:

https://travis-ci.org/newtheatre/history-project/builds/113688066


change to use the new SQL API.


updated to Streams functions


This seems to be roughly what we do elsewhere -- but do we need to do anything to make hygiene work out for us? Can we make a test where the context defines `const val: u32 = 0` or something, just to make sure that the name `val` here is always a binding and does not conflict with this in-scope constant?


And well done with the email-address hash :+1:


Oh, great point.  It looks like we did!  :)


Should probably be a thing, yeah. I'll look into it.


yes for sure, but only after the code gets accepted. I do not want to adopt the documentation ;-)


# Gagarin


`\ifgre@forcedcentering` is forced to false at the end of every call to `\GreSetThisSyllable` (and likewise for the next variants).  See line 541 below.


So, when we use Oracle databases, we have a little problem:
"count(*)" returns a numeric value, not a integer, and the ejabberd need to receive a integer value. So the most simple change is to cast this value to integer directly in SQL sentence.


cleaner


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (3047 of 3047 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)


Can't make it work with Easymock neither, while it is a dependency of thread-weaver.


How does it fail?


You are right, i was working fast and looked for an existing @OnStopped method and thought there wasn't one, will combine


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


TeamCity Glyssen :: Glyssen-master Continuous [Build 0.2.2.cd5bafa52fe699205ce958c37a7495c590aef891](http://build.palaso.org/viewLog.html?buildId=69909&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1029, ignored: 12 Build time: 00:06:02


linux-firmware-nonfree not linux-firmware


Can you just clone, run this specific test, and see?


> it doesnt seem very portable towards compilers that don't use -g got debug build

As far as I understand they are checking that compiler is gcc or clang first.


:fappa:


Brilliant.  Thanks for this.


This is an initial test of this respository.


Test Comment


1.9 sucks


wrong commit message :(


You were completely correct.  Can't believe I forgot how this worked.  The next release (which should be today) will revert this change and probably update the docs to make it clear how project specific settings work.


@pkan I wouldn't say it "traditionally contains a space" unless I'm confusing paths - I think we're talking about (default) "c:\Users\USERNAME" - but yes, USERNAME can contain spaces afaik - it's just that probably neither the patch committer nor the reviewer (me) have that :)

I'm trying to get #2104 tested and merged ASAP.


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


We definitely cannot do a "accept by default" model, as our backlog would grow a lot and it would put people in the uncomfortable position of needing to say no (and therefore becoming the bad guy).

Right now I'm focusing on documenting what we are currently doing. We can follow up with proposals about how to change things separately so we can more easily see the differences.


I suggest to use either "tecla", or "auto-completion" USE flag to be consistent with other packages, from use.local.desc:

dev-php/PEAR-PHP_Shell:auto-completion - Enable tab-completion
net-wireless/bladerf:tecla - use libtecla for command line editing, tab completion, and history


My plans are ruined


Задача решена неверно, если я правильно понял что это задача с камнями. В данном случае на тестовом наборе
5
5 8 13 27 14

Выводит -6. Задумку алгоритма не особо понял, но вообще эту задачу можно решить в один цикл.
Можешь еще подумать, если не придумаешь, я завтра расскажу верное решение.


Missing 95% of the parameterization, but committing before I get on a plane.


Hmm, I sort of expect this to go in the same place where we process fields/methods


Needs to be in the gen_require block above.  Move one line up.


Salam khubi? heif emruz ke nashod hamo bebinim. Man ba in maghalehaye ke khundam relation ha ro dorost kardam albate hanuz kamel nashode. age vaght kardy bebin


Question, do you need the function name twice ??var setupDBConnection = function () {   .....} isn't this the standard??or am I wrong?


@JPercivall sorry for not providing my reasoning on this first time through... the annoying thing about separating into two lines is that flowFile is a final member variable so i can't just do flowFile = session.penalize(flowFile), I would have to create another variable name like FlowFile penalizedFlowFile = session.penalize(flowFile) and then transfer penalizedFlowFIle.

If you prefer that I can do it, but I thought it just added more confusion.


specifically, `parse_dot_or_call_expr_with_` (which might want to be renamed)


I'm not sure what is the content of this page. For simplification lets assume we are talking about broadcast join. Does every instance of every operator get's all the pages from build table? Or this local exchanges/local parallelism that you are introducing are spitting Pages before they get here?

As I understand it is the latter one. In that case, there is probably no this multiple memory reservation issue here that I have fixed https://github.com/facebook/presto/pull/4710 . Regardless if this issue is still here or not, there definitely should be test for that (hopefully mine from this mentioned pull request will work)

However I have couple of concerns. If the Pages are being rewritten for parallel hash building, won't that have an impact on the performance? On the one hand rewriting blocks is costly (previously PageIndexes were sharing blocks), on the other it could be faster because we don't need to iterate over every row in every partition (only to rule out the row). This should be tested.

Secondly, it probably means, that InMemoryJoinHash and PageIndex could be cleaned out. For example `LongArrayList addresses` could be dropped, since it should contain all positions from all pages right? That should/could further simplify/speed up InMemoryJoinHash building.

And just to make sure, hopefully this code doesn't work in such way, that every PageIndex has to process whole build table?


> you could use a similar approach like Gradle.

There is already an issue for that ;-)


that would avoid the need for this ad-hoc match, which kind of gives me the willies, since it seems like it's begging to get out of date, and also sidesteps the grammar structure


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (389 of 389 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)


10 gigabyte, isn't this DDoS and illegal? if its not, it would work.


I think that this code should go in `feature_gate::PostExpansionVisitor` instead


@ricepeterm Where is this URL suppose to lead ? If I'm not mistaken this is a 404 :
https://github.com/transmart/folder-management-plugin/blob/master/grails-app/controllers/fm/FmFolderController.groovy


So thanks! I added commit using `mapnik::safe_cast`. Visual tests changes are in https://github.com/mapnik/test-data-visual/pull/33.


If the server goes down, it's a DoS attack. (DDoS is if multiple users do
that to get the server down).
Am 04.03.2016 18:10 schrieb "MCCCS" <notifications@github.com>:

> 10 gigabyte, isn't this DDoS and illegal? if its not, it would work.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Wurst-Imperium/Wurst-Client/commit/d406cf9d532ee603bef131333eb1df84de283b8f#commitcomment-16505105>
> .
>


merged in 4e35e2d58e176c44e2dd4512c91f489230468022 (plus small tweaks)


What you think?  Too much ego?


Thanks!


How many times is this line called for each lookupSource instance? Only once?


修正する前みたいにモデルごとにアソシエーションまとめて書いたほうがわかりやすいな！
以下のアソシエーションも同様


what should i do ?


The following doc(s) have been deployed to the [1390codea](http://1390codea-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/backups/backup-creation](http://1390codea-static-docs.pantheon.io/docs/articles/sites/backups/backup-creation)
- [docs/articles/sites/backups/cloning-an-existing-site-from-a-dashboard-backup](http://1390codea-static-docs.pantheon.io/docs/articles/sites/backups/cloning-an-existing-site-from-a-dashboard-backup)
- [docs/articles/sites/backups/restoring-an-environment-from-a-backup](http://1390codea-static-docs.pantheon.io/docs/articles/sites/backups/restoring-an-environment-from-a-backup)
- [docs/articles/sites/code/hot-fixes](http://1390codea-static-docs.pantheon.io/docs/articles/sites/code/hot-fixes)
- [source/docs/assets/images/desk_images/169628.png](https://github.com/pantheon-systems/documentation/commit/2dc11cf5f7508911fe3934355455463d25ea97dc/source/docs/assets/images/desk_images/169628.png)


@cthbleachbit Okay, I'll remove the .pem keys. Thanks for your advice.


done


I'll upload it now.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-NoThreads: OK (393 of 396 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-99.24%25-yellow.svg)


Thanks. I'm not sure why no email address was shown with the git shortlog command.


I think we can skip running Docker once.


ok sorry for my impatience and THANKS YOU !!


thanks


Да, хорошо, если не опоздаю - подойду перед занятием и все подробно спрошу


Предлагаю ID именовать как Id, сам вдохновляюсь этим объяснением http://stackoverflow.com/questions/8277355/naming-convention-for-upper-case-abbreviations


Задача решена верно, но опять таки вопрос с именованием и насколько нужно объявлять столько переменных здесь. Советую прочитать http://professorweb.ru/my/csharp/charp_theory/level3/3_5.php
Думаю ошибка(ну не то чтоб прям фатально, но все таки не очень красиво и правильно) будет понятна)


Done.


Could be a way to decompose this?


On full-width pages, this max-width is set by the outer-container, you do not want that default.
I included the clearfix when it is false.


Yeah.. It was an old commits when I was trying to bring up s4.. Don't need it anymore..


contrib/codeanalysis/runcppcheck.sh: line 9: ../cppcheck-1.66/cppcheck: No such file or directory


This will break my installation... My Transmission daemon is running in a different Jail from SickRage, and as such the mount points for the (common) storage are different. Validating the path in SickRage will fail even though it is valid on Transmission's side.


creates several


update_travis_toolchain (lefticus) - i386-Windows-10-VisualStudio-14: OK (389 of 389 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)


Err.. yes, that works.


What is `model.gettingdata()` returning here?


This one was causing the actual lockup. cast does not round!


I'm a zealot about comments that restate the variable name. If it were public I'd probably keep the javadoc in order to keep the never null but otherwise those three lines just get in the way.


Could this be moved to `token-assert` so other rules could use it?


... and this one was causing the actual lockup as well. cast does not round!


done


Yep. Will amend.


```
array(2) { ["calendar"]=> array(5) { [0]=> array(7) { [0]=> array(0) { } [1]=> array(0) { } [2]=> array(2) { ["day_number"]=> int(1) ["events"]=> array(1) { [0]=> array(0) { } } } [3]=> array(2) { ["day_number"]=> int(2) ["events"]=> array(1) { [0]=> array(0) { } } } [4]=> array(2) { ["day_number"]=> int(3) ["events"]=> array(1) { [0]=> array(0) { } } } [5]=> array(2) { ["day_number"]=> int(4) ["events"]=> array(1) { [0]=> array(0) { } } } [6]=> array(2) { ["day_number"]=> int(5) ["events"]=> array(1) { [0]=> array(0) { } } } } [1]=> array(7) { [0]=> array(2) { ["day_number"]=> int(6) ["events"]=> array(1) { [0]=> array(0) { } } } [1]=> array(2) { ["day_number"]=> int(7) ["events"]=> array(1) { [0]=> array(0) { } } } [2]=> array(2) { ["day_number"]=> int(8) ["events"]=> array(1) { [0]=> array(0) { } } } [3]=> array(2) { ["day_number"]=> int(9) ["events"]=> array(1) { [0]=> array(0) { } } } [4]=> array(2) { ["day_number"]=> int(10) ["events"]=> array(1) { [0]=> array(0) { } } } [5]=> array(2) { ["day_number"]=> int(11) ["events"]=> array(1) { [0]=> array(0) { } } } [6]=> array(2) { ["day_number"]=> int(12) ["events"]=> array(1) { [0]=> array(0) { } } } } [2]=> array(7) { [0]=> array(2) { ["day_number"]=> int(13) ["events"]=> array(1) { [0]=> array(0) { } } } [1]=> array(2) { ["day_number"]=> int(14) ["events"]=> array(1) { [0]=> array(0) { } } } [2]=> array(2) { ["day_number"]=> int(15) ["events"]=> array(1) { [0]=> array(0) { } } } [3]=> array(2) { ["day_number"]=> int(16) ["events"]=> array(1) { [0]=> array(0) { } } } [4]=> array(2) { ["day_number"]=> int(17) ["events"]=> array(1) { [0]=> array(0) { } } } [5]=> array(2) { ["day_number"]=> int(18) ["events"]=> array(1) { [0]=> array(0) { } } } [6]=> array(2) { ["day_number"]=> int(19) ["events"]=> array(1) { [0]=> array(0) { } } } } [3]=> array(7) { [0]=> array(2) { ["day_number"]=> int(20) ["events"]=> array(1) { [0]=> array(0) { } } } [1]=> array(2) { ["day_number"]=> int(21) ["events"]=> array(1) { [0]=> array(0) { } } } [2]=> array(2) { ["day_number"]=> int(22) ["events"]=> array(1) { [0]=> array(0) { } } } [3]=> array(2) { ["day_number"]=> int(23) ["events"]=> array(1) { [0]=> array(0) { } } } [4]=> array(2) { ["day_number"]=> int(24) ["events"]=> array(1) { [0]=> array(0) { } } } [5]=> array(2) { ["day_number"]=> int(25) ["events"]=> array(1) { [0]=> array(0) { } } } [6]=> array(2) { ["day_number"]=> int(26) ["events"]=> array(1) { [0]=> array(0) { } } } } [4]=> array(7) { [0]=> array(2) { ["day_number"]=> int(27) ["events"]=> array(1) { [0]=> array(0) { } } } [1]=> array(2) { ["day_number"]=> int(28) ["events"]=> array(1) { [0]=> array(0) { } } } [2]=> array(2) { ["day_number"]=> int(29) ["events"]=> array(1) { [0]=> array(0) { } } } [3]=> array(2) { ["day_number"]=> int(30) ["events"]=> array(1) { [0]=> array(0) { } } } [4]=> array(2) { ["day_number"]=> int(31) ["events"]=> array(1) { [0]=> array(0) { } } } [5]=> array(0) { } [6]=> array(0) { } } } ["cal_links"]=> array(6) { ["previous"]=> string(6) "2016/2" ["next"]=> string(6) "2016/4" ["year_select"]=> array(7) { [0]=> array(1) { ["year"]=> float(2013) } [1]=> array(1) { ["year"]=> float(2014) } [2]=> array(1) { ["year"]=> float(2015) } [3]=> array(1) { ["year"]=> float(2016) } [4]=> array(1) { ["year"]=> float(2017) } [5]=> array(1) { ["year"]=> float(2018) } [6]=> array(1) { ["year"]=> float(2019) } } ["year"]=> string(4) "2016" ["month"]=> string(1) "3" ["datetime"]=> string(6) "2016-3" } }
```


Also, It's almost certainly another rule that's breaking it, it's just those rules are not being applied because the data isn't bound and the sub-template isn't loaded.


Поле нужно сделать not_analyzed или так
```
@Field(index = FieldIndex.not_analyzed, type = FieldType.String)
```
или явно прописать мэппинг для этого поля в src\main\resources\index\question.json

Первый вариант предпочтительней, т.к. не требует явного запуска апдейта мэппинга, spring-data сам все сделает. Иначе мы не сможем делать по нему фильтрацию по term или она иногда будет отваливаться, будет зависеть от того насколько нам повезет с идентификатором. Причины такого поведения можно почитать здесь https://www.elastic.co/guide/en/elasticsearch/reference/1.4/mapping-core-types.html


# Coś, coś się popsuło ...


I think it has to do with repeat


I think a few comments on the function would be neat to explain how it works


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (397 of 397 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)


The following doc(s) have been deployed to the [1390codea](http://1390codea-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/backups/backup-creation](http://1390codea-static-docs.pantheon.io/docs/articles/sites/backups/backup-creation)
- [docs/articles/sites/backups/cloning-an-existing-site-from-a-dashboard-backup](http://1390codea-static-docs.pantheon.io/docs/articles/sites/backups/cloning-an-existing-site-from-a-dashboard-backup)
- [docs/articles/sites/backups/restoring-an-environment-from-a-backup](http://1390codea-static-docs.pantheon.io/docs/articles/sites/backups/restoring-an-environment-from-a-backup)
- [docs/articles/sites/code/hot-fixes](http://1390codea-static-docs.pantheon.io/docs/articles/sites/code/hot-fixes)
- [source/docs/assets/images/desk_images/169628.png](https://github.com/pantheon-systems/documentation/commit/6fc667bb6f423e0d38fecb27b197454134e64494/source/docs/assets/images/desk_images/169628.png)
- [source/docs/assets/images/direct-download-archive.png](https://github.com/pantheon-systems/documentation/commit/6fc667bb6f423e0d38fecb27b197454134e64494/source/docs/assets/images/direct-download-archive.png)


missing change to navio2


Yep:

![image](https://cloud.githubusercontent.com/assets/151829/13534524/a02ae3c0-e203-11e5-80f7-4c8ee2ca9022.png)

This is what it used to look like:

![image](https://cloud.githubusercontent.com/assets/151829/13534552/b772eabe-e203-11e5-8bd7-a7b71d3e737e.png)

I think Shift might assume the namespace is inline with the `<?php` opening block?


why not simply strip both sides btw.? Removing trailing whitespace on the fly


@laravel-shift ^^


Только поля authorId недостаточно, велика вероятно, что в конце концов нам захочется рядом с вопросом выводить и имя пользователя. Поэтому предлагаю сразу добавить и поле authorName.


Ugh, this is terrible, since a number of these is optional. Perhaps this is better after all:

```yaml
description1_order: "%{state} order from %{supplier} created by %{who}, "
description1_period:
  starts: "open from %{starts}"
  starts_ends: "open from %{starts} until %{ends}"
  pickup: "and can be picked up on %{pickup}"
```


This was perhaps already in a somewhat messy state, let me explain: So far we didn't use the `django.contrib.staticfiles` collector, apparently, as all static files are shipped in `static/` in the first place, and obviously the management command will refuse to run with the same source and destination. So presently I think it is not necessary to enable `STATIC_ROOT`, until such time that (for example) we move `static -> djnro/static` (and amend instructions that static files need to be collected). But I may be missing something. What do you think?


can you accept the `use_spaces` setting and the `tab_width` setting and use them correctly?


Yes, I think that's what we want since we need to rebuild the ledgers as
well.


if there is no support for that yet, add that info to the doc comment


LaTeXDummy (Myoldmopar) - 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)


LGTM


it'd be good to have documentation comments for functions with this complexity


Добавлять сюда поля концептуально неправильно на мой взгляд О_о. По сути DslQuery является объектным отображением того, что пользователь ввел в строке поиска.


LaTeXDummy (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Build Failed

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


yeah dcument everything please


2097-EP-8-5 (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


:bow:


My mistake for bleeding one commit into the next.


LaTeXDummy (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Build Failed

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


done


moved to #403


Commit message should explain why it's being removed.


Этот метод можно автогенерить с помощью IDEA https://www.jetbrains.com/idea/help/generating-tostring.html


looks nicer! i'll update my patch with it


LaTeXDummy (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


the extension of this file is *.mdb for mono thus config should be dynamic, use XmlTask for update depending on OS


I've just added a commit which will stop the notice. Is the correct information being displayed on the page, regardless of the notice?


the problem is not with the null configuration since but with null bootstrap config. In context ITs MockCoreConfiguration service is used, if you look it up you can see that the method for loadBoostrapConfig always returns null - update it to return a boostrap config that will work with ITs


There's a lot of space between when this variable is created and when it is initialized.  Consider `JobType type = jtINVALID;`.


+        classpath 'com.android.tools.build:gradle:1.5.0'


I don't understand why we needed to do this more than once, or why we don't now - can you comment?


I expect this only works if config.frameBufferEmulation.enable=1? Have not got it running. Would it be possible to add this feature for low power arm devices like a raspberry pi which cannot use frameBufferEmulation?


@aphearin I don't want to fork and put in a pull request.

Add the following to line 178 in nearest_neighbor.py

if len(i)==0: #no neighbors
        result = np.zeros(N0, dtype=int)-1

The problem involved a list being empty, i.e. no neighbors were found.


2097-EP-8-5 (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

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


No, nothing shows up


blarg, the second line of the code snippet should be indented.


По Code Convention здесь желателен пробел между аргументами ^)


Commit message should describe why this change is being made.


I split up the problem line and this part issues the error `main { bind: data(model.getData()); }`


Instead of assuming, could we ASSERT_THAT the other alarm is in a cleared state at the start of the test? I think this will give u a more obvious failure if it isn't.


Can i translete full imagicalmine?


А здесь перед запятой не очень желателен :)


Why is name commented out?


Typo: Secerity


@yehudasa amended the commit (https://github.com/rzarzynski/ceph/commit/01f848e7eaa5a78603a64f78e55178beed665d88) with ```X-Versions-Location``` printing.


merged in b30b4e46d84a66a2f8022159f5533607a0ae7d01


저희 jpg jpeg png 이렇게 3개만 받죠?

백그라운드에 gif도 받나요? @juice500ml  zzzz


Congrats for identifying and removing dead code.


Can you paste the return value of model.getData() as json so I can easily load it into a test?


@winstonf88 quando o rebase foi feito, ficou faltando pinar o collective.portlet.calendar para 1.0b3.


a lot cleaner and also `@tailrec`, thanks!


The current Master branch is going away in favor of what's on the admin-change-breakout branch


I'm wondering if this check for OptionId == null is needed.

I haven't found anything in the MSDN stating that OptionId == null explicitly means "the printer has changed".

Would it actually be correct to just always call InvalidatePreview here, regardless of the value of OptionId?


@abdollheid  which gapps u have used is it opengapps ? if not can u try with opengapps and also if it 64bit device try 64bit gapps


Ah, I think you might mean "by design, the alarm code assumes this is cleared" not "when writing this test, I've assumed it's cleared because I haven't touched it". Best to be clear.


We want to avoid this `enforcer.skip` setting. Does your build still pass if you remove this line?

If so, please purge it from this commit.

Or if the build fails, let's troubleshoot why!


pkg_conmtrol? typo?


Shouldn't you be the author? :smile:


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


`buildPipeline` will not be called if it is empty.

```scala
if(pipelineFlowNames.isEmpty) { None }
else { buildPipeline(pipelineFlowNames) }
```


I checked it with Toby quickly, and I tested it with my project/experiment and it works well. Important use case: when ignore == 0, the testTrafficAllocation will always return true.

I also talked about Toby how the Math.random() function doesn't seem to be working well in Safari (that's why we're not using it ourselves for our own bucketing), but I hope that's not a big blocker here.


00 is an outlier in that it has no rendering, just text. As such I felt it wasn't necessary, since I'm trying to only include things that are actually needed for the given sample to work.


Вот поэтому стрелки и не работают. Потому что поиск по одному выражению, и позиция вопроса определяется в рамках того поиска, который делал юзверь. Например, он искал вопросы с тэгом core и этот вопрос был в его поиске седьмым по счету, он переходит в карточку вопроса, и тут ты принудительно меняешь его DSL, и получается что с учетом всех его удаленных вопросов этот вопрос уже получается не седьмым, а восьмым например, а логика стрелок такова, что если есть разночтения, то они просто выключаются, потому что логика становится нечеткой.


I would change the commit message to "Implemented triple integral test" instead of just "Modified" because that is more specific: you did actually write the test and put it here!


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (397 of 397 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)


This is actually week 7


I think this comment can be removed.


When does the counter reset? When player quits?


Fix to #1.


is this still the case after the last update?


@winstonf88  ver: https://github.com/plonegovbr/brasil.gov.agenda/commit/bef3505b4b697fb0c819151c60d469c048399e2f


@paulbarber What do you think about this? Does it look OK to you? Would you mind quickly testing with some of your local use cases?


```json
{"calendar":[[[],[],{"day_number":1,"events":[[]]},{"day_number":2,"events":[[]]},{"day_number":3,"events":[[]]},{"day_number":4,"events":[[]]},{"day_number":5,"events":[[]]}],[{"day_number":6,"events":[[]]},{"day_number":7,"events":[[]]},{"day_number":8,"events":[[]]},{"day_number":9,"events":[[]]},{"day_number":10,"events":[[]]},{"day_number":11,"events":[[]]},{"day_number":12,"events":[[]]}],[{"day_number":13,"events":[[]]},{"day_number":14,"events":[[]]},{"day_number":15,"events":[[]]},{"day_number":16,"events":[[]]},{"day_number":17,"events":[[]]},{"day_number":18,"events":[[]]},{"day_number":19,"events":[[]]}],[{"day_number":20,"events":[[]]},{"day_number":21,"events":[[]]},{"day_number":22,"events":[[]]},{"day_number":23,"events":[[]]},{"day_number":24,"events":[[]]},{"day_number":25,"events":[[]]},{"day_number":26,"events":[[]]}],[{"day_number":27,"events":[[]]},{"day_number":28,"events":[[]]},{"day_number":29,"events":[[]]},{"day_number":30,"events":[[]]},{"day_number":31,"events":[[]]},[],[]]],"cal_links":{"previous":"2016\/2","next":"2016\/4","year_select":[{"year":2013},{"year":2014},{"year":2015},{"year":2016},{"year":2017},{"year":2018},{"year":2019}],"year":"2016","month":"3","datetime":"2016-3"}}
```
I fixed a small thing with my template and stuff shows up now and it repeats the node the correct number of times but doesn't change the content


humn... but we will always use the cmake build system. Why do we need this comment?


pro multithreading skills


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


It gets converted to iurl when it's not an image because the HTML is a standard anchor (a) tag with no target, which SMF automatically converts to iurl. The extra code here looks for the "data-attachment" attribute, which tells us that it's an attachment link rather than a regular link. We need to fix the code so that we don't run into problems when attrs isn't defined.


에 이렇게 쓰는거 맞건가요 ㅋㅋㅋㅋㅋㅋㅋ


I don't know, but I think, when the players quits, like you said. Or at
most if the server restarts.
Am 04.03.2016 18:32 schrieb "MCCCS" <notifications@github.com>:

> When does the counter reset? When player quits?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Wurst-Imperium/Wurst-Client/commit/d406cf9d532ee603bef131333eb1df84de283b8f#commitcomment-16505608>
> .
>


What you have here is "if the Event Listener is *NOT* an instance of GeListener, try to cast it to GeListener", which will throw an exception.


No idea what this is


Unfortunately, this commit conflates a reversion of the previous commit with the intended addition of a new test. Please rebase to address the issue. I'm happy to assist on Monday if you need help (or @hinerm or @gab1one would be equally capable of helping you).


네. 왜요 너무 쉬워서요?ㅇㅅㅇ