`@health_points` and `@attack_power` are also set in `Unit`, which `Footman` is inheriting from. It works to reinitialize them here as well, but it would be cleaner* to set these values only in the `Unit` class - can you think of a way to do that here? Hint: you'll have to use `super`.

* - inheritance lets programmers reuse code without having to copy-and-paste, by sharing code between classes that inherit from other classes. This saves time in the long run when fixing bugs or adding new features, since there's only a single point (in the `Unit` class, for example) where the most general attributes are set. This reduce the chances of forgetting to modify an attribute in all the places we've initialized it as well as the chances of introducing bugs due to that.


That's what it is now on the master branch... what do you mean by having a duplicate log for the test? Sorry don't understand whats the advantages of this check. I thought here we just needed to check the `error` event shouldn't emit?


String.format or a single line with two + top


You might want to deploy this change to your Heroku server... :)


tried to port this to the upstream xpad driver to be able to have this in the kernel one day - however I messed up the handshake somehow and the pad does not connect at all most of the time. Maybe you can help out a bit if you have some time: https://github.com/paroj/xpad/issues/1


make a static and a constant with uppercased name.


You've also defined `attack!` in `Unit`. Since `Footman` inherits from `Unit` and since `Unit`'s `attack` method does the same thing as the one you've defined here, you can actually eliminate this one (same goes for `damage`). That's because all the attributes and functionality (methods) of `Unit` are available to instances of `Footman`.


Please, check what is the normal return parameters for similar functions. In this case, it is a tuple.


aad


@cnathe @labkey-matthewb @labkey-danield This is associated with a new set of working data files as well (sorry about that).


Hey Steffi,

You've written some really great looking code! It's clean, well-indented and simple! There are a few things that could be cleaned up, and I left comments on the relevant lines in your code above.


Make this field private and get the foreign referencing code to use a getter or setter.


oops.  sorry about that paolo.  :)

forgot about priming the pump on that script.

stand by.  I'll try to fix it tonight if I don't pass out from the beer
first :)

On Tue, Jun 30, 2015 at 5:28 PM, Paolo Oliveira <notifications@github.com>
wrote:

> Yes. The error is on first line of rebuild.sh:
> cd libuv; sh autogen.sh;
> there is no libuv/autogen.sh... libuv directory is empty.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jvburnes/node9/commit/0ca97e87e38515b6a2317d5a0548b8f28891cee4#commitcomment-11943530>
> .
>


this does not make sense. recvfrom() can't take an address as an input. recvfrom() is "receive from anyone and tell me where I received the packet from"


This should probably updated to reflect the new API.


This should be updated too


Nice, excellent use of `super` here!


Hah.  Not only that I seem to have succeeded in deleting the entire libuv
directory and link structure.  Never work on stuff late at night or for
more than 24 hrs at a time. :)

On Tue, Jun 30, 2015 at 5:43 PM, Jim Burnes <jvburnes@gmail.com> wrote:

> oops.  sorry about that paolo.  :)
>
> forgot about priming the pump on that script.
>
> stand by.  I'll try to fix it tonight if I don't pass out from the beer
> first :)
>
>
> On Tue, Jun 30, 2015 at 5:28 PM, Paolo Oliveira <notifications@github.com>
> wrote:
>
>> Yes. The error is on first line of rebuild.sh:
>> cd libuv; sh autogen.sh;
>> there is no libuv/autogen.sh... libuv directory is empty.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/jvburnes/node9/commit/0ca97e87e38515b6a2317d5a0548b8f28891cee4#commitcomment-11943530>
>> .
>>
>
>


Add `assert.strictEqual(error.message, "Invalid string")`


1679-DualDuct (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2183 of 2216 tests passed)

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


Keep this and replace `"Invalid ip address"` with `"Invalid string"`


This `unless` statement can be shortened somewhat. It's equivalent to:

`if can_train_peasant?`

This is because `can_train_peasant?` already returns `true` or `false` so we don't need to check it against `== false`, we can work directly with that value. And in our case, we only want to create a new `Peasant` instance when `can_train_peasant?` is true.


93469234-DemandResponse2 (vidanovic) - x86_64-MacOS-10.9-clang: OK (1679 of 1682 tests passed)

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


if you initialize an new JarEntry with name, it would complain "Error opening zip file or JAR manifest missing". So, it requires more process to copy other meta data.


Check parameters?


gj


Maybe all caps on N1QL keywords?


It's only for caching textures, to while it would take a while, it's not rendering in real-time unlike retroarch


Explicitly typing `return` is usually frowned upon by Rubyists since its unconventional. I personally think it's fine, but my opinion doesn't count for much - so if you'd like to write conventional code, it's recommended to omit the `return` and simply say `true unless gold < 90 || food < 5`.


Please don't [commit to the master branch](https://github.com/RedstonerServer/redstoner-utils/wiki/2.-About-branches) only.
Can you fix this?


I suspect what was intended here was to get the address and port of the first packet and then call connect() on that address. That would mean we only accept packets from one source


I think this is the wrong order.


:bow:


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 96](https://teamcity.labkey.org/viewLog.html?buildId=258684&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabkeyTrunk :: build Build time: 00:00:00


Both `Data.Map` and `Data.Set` should be `qualified` here.


It is not. I tried it. It looks like context is the last argument passed


Since it should return a number wouldn't `===` be considered idiomatic then?


I think this is about as good as you do with creating new compiler keywords for NEST and UNNEST.


Edit2 really necessary???


Not sure about the '_'
Find out if that is standard


No (((


Not sure about the '_'
Reconsider.


Delete THIS!!!


@applepicke this is why we can't have nice things


develop (axelstudios) - i386-Windows-7-VisualStudio-12: Tests Failed

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


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (351 of 351 tests passed)

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


```clojure
clojure.test/run-tests [(U nil Namespace) -> (Map Any Any)]
```


這裡又是一個案例，昨天討論到，我這邊用了 `getSitePath()` 來幫助測試常數，但又變的不 Unit 了

可是如果我用 Reflection 或是直接模擬內部邏輯做一次，不只程式碼太長，又侵入了內部實作

所以結論是很多東西要看 case 選擇解法，而且還會衍生出很多其他問題

例如我這邊的 `getSitePath()` 運作正常，那就等同測試過了，那還有沒有必要再寫一次 `testGetSitePath()`? 會不會常數錯了，`getSitePath()` 跟著錯，測試卻愉快通過？

這些都是可以思考或選擇的問題


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/72aedca39ce00e8da8693a3ec0c019d1c9a8ac45/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/72aedca39ce00e8da8693a3ec0c019d1c9a8ac45/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/72aedca39ce00e8da8693a3ec0c019d1c9a8ac45/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/72aedca39ce00e8da8693a3ec0c019d1c9a8ac45/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/72aedca39ce00e8da8693a3ec0c019d1c9a8ac45/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


develop (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (350 of 351 tests passed)

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


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (351 of 351 tests passed)

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


r+


develop (axelstudios) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (351 of 351 tests passed)

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


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (351 of 351 tests passed)

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


I'm not sure what the intent is here. By not defining this attribute, we're going to throw an exception later when we test for this attribute. Storing None would give us the expected behavior though.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


That's a workaround and you know it.


develop (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (350 of 351 tests passed)

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


Is that all it takes? A pull request would be most welcome.


one-shot-publish can be used as "rostopic pub -n 1" from roseus
this is not equivalent to latching option.


Ugh, ugly. What with `example.Svg` file?


This seemed inaccurate, from metaslab_group_activate():
```
mg->mg_aliquot = metaslab_aliquot * MAX(1, mg->mg_vd->vdev_children);
```
So it's multiplied by the number of children of a top-level vdev.


Middle button click please?


TeamCity NEventStore :: MongoDB :: Build [Build 5.4.0-unstable.1+1](http://build.prxm.it:8811/viewLog.html?buildId=3259&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Tests passed: 75; exit code 1 (new) Build time: 00:01:33


Thanks for the PR @iurimenin I have picked up only required strings from here and merged them myself as the directory structure has changed in one branch of PolyPicker. I will bring these translations to life soon. All the best.


https://tools.ietf.org/html/rfc5280#section-4.2.1.1 says that the ```authorityCertIssuer``` element of the sequence is ```OPTIONAL```.

```
type authority_key_id = Cstruct.t option * general_name list * Z.t option
```

in x509.mli:179 is in conflict. why?


rebase...


@tdhock I was missing apply so I made a couple of apply-macros that may or may not make the code less repetitive and more readable. Please tell me if you think I am going over board with stuff.


This change was reverted.


There is no MD5 change.


To use `no-unused-vars` with React's semantics, look at [yannickcr/eslint-plugin-react](https://github.com/yannickcr/eslint-plugin-react).


 @tdhock I have now tree struct that appear in the main loop, one for them model, one for the datasets and one for the trainer. This way we can just add loss function in glmmodels.c and all the rest remains the same. The relevant files are in glmmodels.[ch], trainers.[ch], and dataset.[ch]. I only applied this modular structure to sag_constant.c as I wanted to have your comments before proceeding further.

The nice thing with this code structure is that we get encapsulation but also inheritance and overloading for free.


yep, sure. you're right.


Camera control.py is now deprecated, just using it as a reference now.


TeamCity NEventStore :: MongoDB :: Build [Build 31](http://build.prxm.it:8811/viewLog.html?buildId=3261&buildTypeId=nes_mongo_build&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 1 (new)) Build time: 00:00:11


thx


TeamCity NEventStore :: MongoDB :: Build [Build 5.4.0-unstable.2+2](http://build.prxm.it:8811/viewLog.html?buildId=3262&buildTypeId=nes_mongo_build&guest=1) outcome was **FAILURE**
Summary: Tests passed: 75; exit code 1 Build time: 00:00:48


@tdhock I though you might like this. This is really starting to look like an object, just mentally replace &trainer with "self" or "this". Beyond aesthetics and having fun, I think this interface will make thing easy when we'll move to passing arbitrary C or R function to SAG. It would be as simple as defining a new model containing pointers to the relevant function while keeping all the remaining code  unchanged.


HBIRE_vector_take2 (amirroth) - i386-Windows-7-VisualStudio-12: OK (1680 of 1682 tests passed)

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


@tdhock This is the struct containing pointers to loss and gradient function. The parameters are y (targets) and w'x (innerProd in the code)


TeamCity NEventStore :: MongoDB :: Build [Build 5.4.0-unstable.3+3](http://build.prxm.it:8811/viewLog.html?buildId=3263&buildTypeId=nes_mongo_build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 75 Build time: 00:00:50


Ah, fair enough.  You're exactly right about why I changed it, I'm a sucker for consistency.

-[Unknown]


That is correct

I'll push the merge and deal with the fallout. KallistoExperiment has methods getTpm(), etc to deal w/this and ultimately will have getNormalized() to use RUVg vectors adjusted against ERCCs to do it properly. Plus you can extend the class as you see fit...

--t

> On Jun 29, 2015, at 7:14 PM, Anthony R. Colombo <notifications@github.com> wrote:
>
> From the meeting with Harold, the eff_length is a weighted measure of the est_count. is this tpm here correct? thank you very much
> tpm <- assays(merged)$est_count / assays(merged)$eff_length
>
> —
> Reply to this email directly or view it on GitHub.
>


Code ruby here


@LaikuenLau @esoliatakumi this last line had the domain spelled wrong so I fixed and will merge. However, how is that supposed to work? I have heard of putting an address in your address book to avoid it getting sent to spam, but, I have not heard of doing it for a domain, unless it's on the system admin side. Can users easily do this?


我爱她！


Could this be changed to use portable POSIX sh rather than bash? In addition to the shebang, I think just changing "[[" to "[" or "test" should be sufficient.


Deben indicar la ruta completa al modelo (incluido el namespace)


おみくじゲームこれでも動きますよ！


All looks good to me, please merge to master.


90946736_SHRgt1_4567 (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1157 of 1164 tests passed)

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


On 30.06.2015 02:15, Henrik S. Gaßmann wrote:
> The |BOOST_USE_WINAPI_VERSION >= BOOST_WINAPI_VERSION_WIN6| check is
> unnecessary, because |BOOST_PLAT_WINDOWS_RUNTIME| means we are compiling
> a windows (phone) store app which means in turn that the WinSDK version
> is at least WIN8.

BOOST_USE_WINAPI_VERSION is not Windows SDK version, it's the target
Windows version. As far as I can tell, it is controlled separately from
the application type. I found no relation between these two settings
neither in Windows SDK nor in Boost.Predef.


Nope, I meant the Azkaban node should set the version as epoch, so that other nodes will have the same epoch version. This way all nodes will have the same version number and no computation required. The nodes should have the same version number for tracking, restore and debuggability reasons.

If you observe our RO clusters closely the version numbers are different across different clusters. Which makes it harder to reason which version is pushed where.


Totally agree :beers:


Thanks Mark; I'll check it out tomorrow.

Fred

On Mon, Jun 29, 2015 at 7:02 PM, Mark Adams <notifications@github.com>
wrote:

> @wfbuhl <https://github.com/wfbuhl> You can change the name of the column
> if something else fits the data better. But other than that, this commit
> should fix the failing SQLite unit test. It does for me locally.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NREL/EnergyPlus/commit/815f5e26f923cfedeeb9abb28a8960501f70d845#commitcomment-11922898>
> .
>


2 things-

1) I believe rects are stored as ```model_x```, ```model_y``` etc. after a detection. Will this check recognize those?
2) Can you also add this change to classifiers/cascade.cpp? The same logic should apply there as well


one commit to rule 'em all


HBIRE_vector_take2 (amirroth) - Win64-Windows-7-VisualStudio-12: OK (1680 of 1682 tests passed)

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


Did the references not work for your local folder? Hmmm. I apologize for that.


Sorry Win, this change was just to make testing easier. It's been removed.


Another part of #184.


```Utils.nonNull``` on each parameter.


Fixed and code is available in following pull request
https://github.com/wso2/carbon-analytics-common/pull/64


Well because it's not really required in Swift, and it is not consistent
within the project anyway...

  And it is easier to remove all of them than look for the places where
they are missing :)


Technically speaking the name field is currently size 200 :D


@cdeil, @adonath

`_bound` property is now defined for most models where appropriate, will set the `extent` attribute to some default defined in terms of other parameters.


- _txtInExit->setText(tr("STR_UNITS_IN_EXIT_AREA", _inExitArea));
+ _txtInExit->setText(tr("STR_UNITS_IN_CRAFT", _inExitArea));
Seriously? o_O Aborting base assault better to be have "in craft" string?


Thanks for catching this guys. @danielpquinn , you are correct, I didn't mean to delete that line.

I reverted that and published a version update to npm. If someone could test it for me that would be great as I can't atm.


줄에도 코멘트를 달 수 있구나


줄줄이?


Yeah, I knew it had been doing the double mana cost/power/toughness/abilities thing. I've just been typing them as soon as they're spoiled instead of waiting for them to be added to mtgsalvation. I'll try to tighten up my typing. :)


Reviewed.


여기에도


Michal, the merge caused  a conflict here.  I can fix it, but it will show my name as the author.  Can you rebase agains the master, resolve the conflict in your branch, and submit another request?


추가로 댓글


Why doesn't the import of `SupportsBytes` work for me?


Updated.


thanks, I will fix


not necessary, i will remove


回退回第一个commit


Is the type will be set for "PackageResourceReference" only?


that would be right :-)


I agree that throwing non-errors has been an unfortunate issue for a long time now:
http://www.nczonline.net/blog/2009/03/10/the-art-of-throwing-javascript-errors-part-2/
http://www.devthought.com/2011/12/22/a-string-is-not-an-error/

Doesn't help that so many resources show it:
* http://eloquentjavascript.net/1st_edition/chapter5.html (recommends using objects only at the very end)
* https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/throw
* https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/try...catch

And it probably doesn't help that some of the most popular JS libraries used to do it:
* https://github.com/jquery/jquery/blob/9dc7a3a7d20c0e75b927c956cba8de4dc30db077/jquery.js#L987 (jquery 1.3.0)

I wonder if it was just to avoid adding an edge case for the handling of existing internal completion types like break, continue and return. That and limitations to the types thrown aren't usually specified in a language's grammar.

My best guess though, is that JS' major influences, as well as many other languages at the time, didn't limit the type thrown. For example, python could raise strings and others in 2.4 and earlier: http://legacy.python.org/dev/peps/pep-0352/  Scheme can also raise anything. Same with ruby, lua and C++. Just doesn't seem common to limit the type thrown. My first time encountering that limitation was with Java and PHP.


这是标题


这是主题内容


91958608DOASEffectOnZoneSizing (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


TeamCity NEventStore :: SQL :: Build [Build 6.1.0-unstable.1+1](http://build.prxm.it:8811/viewLog.html?buildId=3264&buildTypeId=NEventStore_Sql_Build&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:01:02


Well, `renderDepth` was not renamed to `renderOrder`. It was removed, and `renderOrder` was added. They have different units.

    console.warn( 'THREE.Object3D: .renderDepth has been removed. Use .renderOrder, instead.' );

I would not return or set `this.renderOrder`, as the units are different.


91958608DOASEffectOnZoneSizing (wfbuhl) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Implementation of functions are still not showing in the view layer. We need to get the game running to be able to show everything.


Individual assignment for lesson 9


貌似不错哦， solarized


HI, yes it does.

Best regards


test


@igas Thanks for adding this. I was abt to send a PR :)


I agree with all the fixes in this commit except this one.


This is an error, No changes were made to this file...It was simply created in Git Bash.


@ariccio Nope, never done a line of Python in my life. It was added by @milipili


Shrink it one more:

```html
<body onload="s=0,d=Date.now,t=d()"><p onclick="(e=d()-t)<15e3?style.margin=++s+e%300:alert(s)">X
```


Hi

This doesnt catch the Statement:

    class { 'base::root'
         stage => init,
    }

Note the missing ':' after the Ressource Name

The Log-Message is:
`Suspicious state from syntax checker puppet-parser: Checker puppet-parser returned non-zero exit code 1, but no errors from output: Error: Could not parse for environment jahe_dev: Syntax error at 'stage' at /tmp/flycheck19432d0B/init.pp:16:9`

Jan

Should i File a Bug for this?


missed some `~`: `~cherry`, `cherry-pick~`, `format-patch~, `~rebase`, `reflog~`, `~stash`, `submodule~`.


K go for it.

Sent from my iPhone

> On Jun 29, 2015, at 8:33 PM, Tim Triche, Jr. <notifications@github.com> wrote:
>
> That is correct
>
> I'll push the merge and deal with the fallout. KallistoExperiment has methods getTpm(), etc to deal w/this and ultimately will have getNormalized() to use RUVg vectors adjusted against ERCCs to do it properly. Plus you can extend the class as you see fit...
>
> --t
>
> > On Jun 29, 2015, at 7:14 PM, Anthony R. Colombo <notifications@github.com> wrote:
> >
> > From the meeting with Harold, the eff_length is a weighted measure of the est_count. is this tpm here correct? thank you very much
> > tpm <- assays(merged)$est_count / assays(merged)$eff_length
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
> —
> Reply to this email directly or view it on GitHub.
>


The fifth parameter is `description`, which is obtained via `description = get_text_tag(item, 'description', '')`. The function `get_text_tag` should have never returned `None`, but the default text `''`. Therefore, when fixing `get_text_tag`, this unit test must be fixed as well.


Whoops, thanks!


Don't remember exactly, this is an old commit. Basically it removes /proc/M4U_device entry and the user use the /dev entry instead.


Why do we need to collapse the states so much, should we not return a more 1-1 mapping.

Also I feel that these should be bound to an enum.


Some DSMX Remote Receiver can't enter BIND mode with the duration about 25us but 120us.


Hi Jeromen,
Thanks for your reply. But am just wondering whether there is away to configure the module and set it  to give certain amount of money to those who would have referred a person to sign up. And when a referred person  buy products from a specific category, the referrer gets paid some amount of money. Lastly, can it also make it that the referrer gets paid every time the referred person buy a product from a certain category?

Your help is always appreciated

Thank you
Ronnie


@helixbass nice!


Awesome


In scala its normally more idiomatic to use Option

```scala
val lastSuccess  = Option(job.lastSuccess)
val lastError = Option(job.lastError)
```

this will make your string matching go away and make your code more robust


Currently yes, because to resolve the type other ResourceReferences may resolve it somehow else. But you can setDisplayType to true and setType to do it the way you want it to be.


OK, thanks!


FYI, this fixed a bug in #select_all, etc in 4.2.2 that I just ran into whereby the arel binds would be left out, leading to invalid queries and mis-matched binds. Thanks for the fix!


Thanks.
I missed to test the simple case and I should have checked travis, bad.


Working on it right now.  I didn't realize how much of a disconnect there
was between the methods that expect a KallistoExperiment vs. those that
don't (yet).  It's taking a few more minutes; I'll push it as soon as it
builds cleanly.

Statistics is the grammar of science.
Karl Pearson <http://en.wikipedia.org/wiki/The_Grammar_of_Science>

On Mon, Jun 29, 2015 at 9:34 PM, Anthony R. Colombo <
notifications@github.com> wrote:

> K go for it.
>
> Sent from my iPhone
>
>
> > On Jun 29, 2015, at 8:33 PM, Tim Triche, Jr. <notifications@github.com>
> wrote:
> >
> > That is correct
> >
> > I'll push the merge and deal with the fallout. KallistoExperiment has
> methods getTpm(), etc to deal w/this and ultimately will have
> getNormalized() to use RUVg vectors adjusted against ERCCs to do it
> properly. Plus you can extend the class as you see fit...
> >
> > --t
> >
> > > On Jun 29, 2015, at 7:14 PM, Anthony R. Colombo <
> notifications@github.com> wrote:
> > >
> > > From the meeting with Harold, the eff_length is a weighted measure of
> the est_count. is this tpm here correct? thank you very much
> > > tpm <- assays(merged)$est_count / assays(merged)$eff_length
> > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RamsinghLab/artemis/commit/bc204808813903af6a27cfc8b1d7f4b2e969c899#commitcomment-11924045>
> .
>


@HashNuke You are welcome :)


This is awesome.  I was just thinking about doing this and I know that it will make things so much easier.  Thank you @mostaphaRoudsari !


or add cm sepolicy to pa.dependencies


I'm sorry you merged so soon, I was still working on the build errors :) Specifically, this particular change is definitely wrong and should be reverted (the extra parentheses removed).


TeamCity NEventStore :: Core :: Build [Build 87](http://build.prxm.it:8811/viewLog.html?buildId=3266&buildTypeId=NEventStore_CI&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:00:17


Do we want to delete SearchingClassLoader?


@moollaza For some reason this selector doesn't get called if i make it just `.zci--snow`


Same thing as the above one


Maybe add the period too.


FrameSheet(AnimationSheet에서 변경)로 합쳐서 사용하게 수정했습니다.


test coverage seems to be missing for this


This is really nice model!


done,..


Maybe note that all titles should be capitalized and spelling should be checked. Also this says nothing about the order of the list.


TeamCity NEventStore :: Core :: Build [Build 5.4.0-unstable.1+1](http://build.prxm.it:8811/viewLog.html?buildId=3267&buildTypeId=NEventStore_CI&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 279 Build time: 00:02:16


Is this a real test that needs to be committed? Looks just like some temp code to generate the TOML.


bind*


what is the expected output of this method if the type is mockable?


This is the only essential change in this commit.


added


> Use the following format: `[List Name](link)`.

I meant a period inside of the backticks (as your actual list items end with periods too)


why method is exported?


The indentation here could be improved.


JsonParser의 설계가 <String, T>로 설계되어서 그대로 따라했습니다.

Enum으로 선언된 데이터를 가져오기 때문에 예외가 일어나지 않을 확률이 높습니다. 그래도 예외처리를 한다면 어떤 식으로 해야할까요? 아래처럼 하는거밖에 생각이 안나는데, 의미가 없다고 생각해서요.
try {
			animationSheetMap = JsonParser.parseMap(
					FrameSheet.class,
					filePathMap.get(
							JsonEnum.ANIMATION_SHEET_FILE_PATH.toString())
							.getFile());

			for (Entry<String, FrameSheet> entry : animationSheetMap.entrySet()) {
				entry.getValue().loadTexture();
			}
		} catch (NullPointerException e) {
			Gdx.app.log("StaticAssets", "AnimationSheet 로딩 오류");
		}


changed...


This is no good—now you can click anywhere in the body, not just the X! :)


Please following exclusion option instead
`[credit_tracker_table exclude="ident_nr, source, author, publisher,license, link]`


I have fixed by logging error instead throwing and changes are available in following pull request
https://github.com/wso2/carbon-analytics-common/pull/64/files


Unfortunately, the precursor commit https://github.com/rails/rails/commit/16ce2eecd3eb23034555bb37b29c12985243d908 was released in 4.2.3, but not this one. Could we get this fix in 4.2.4?


Please use indentation (code formatting) here; see next line inside `if`


so if it's 101 nsec that's fine but if it's 100 nsec it gets bumped to 1000?  i'm not sure i understand the logic in line 135 vs line 138.    additionally, 1000 nsec is one microsecond, not one millisec.   mmm secants.  :)


when using numbers is recommended to use variables instead of literal numbers, variables are auto-documented once your provide them a name, also numbers tend to change often.
They are called magic number ..read why you should avoid them:
http://stackoverflow.com/questions/47882/what-is-a-magic-number-and-why-is-it-bad

```
var lat = 33.0487641;
var lng = -96.8524734
new google.maps.LatLng(lat,lng ...
.....


bájate `vertexCount` porque ya esta en `vertices.size()`


Queue Interface에 생성할 수 있는 것들 중 가장 익숙한 것이 LinkedList라서요. 데이터를 Sequential하게 저장하는 ArrayDeque로 수정하겠습니다.


Description should show all exclusion options, e.g. all column names (sample or either description text; see also `style` or `size` for example)


위와 같은 이유(Parser가 <String, T>)로 이렇게 했습니다.


At this point we need the touch-debug feature in production/release builds also. Merging the fix of for the variable scope.


@multidis, should I remove it? Have linked `TradingLogic.jl` too. Thanks.


Good jobe


Nit: I prefer tripple-slashes since it shows up in the option-click quick help. Not a blocker though.


91958608DOASEffectOnZoneSizing (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (647 of 649 tests passed)

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


This isn't a thread-safe call. Use `@syncronized` (easy), NSLock (easy), or a spin-lock (for speed since contention is unlikely).


@nbjorken Formerly you could click anywhere in the rest of the line. My intent was to keep that behaviour, because the body is of minimal height, but apparently Chrome doesn’t treat it that way. I guess I’ll revert it, even though it worked fine in Firefox. I think I’ll change it to `<a>` so it’s inline and you can’t click on the rest of the line either, even though that behaviour existed in the original.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.701.a46c0a67c3afa99748ae76925488c03179c31086](http://build.palaso.org/viewLog.html?buildId=59077&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 355, ignored: 3 Build time: 00:05:34


non-blocking style nit: `self.weakDefaultLogStore != nil`


I'd probably code this as:

```rb
NotificationMailer.send(name.to_sym, args.merge(subscriber: subscriber)).deliver
```
From browsing this particular commit `merge!` is safe. But if you happen to pass in a hash along the way that you expect to re-use after calling `NotificationChannel.send!`, there could be confusing results.

My default is to avoid mutation until there's some sort of reason not to (e.g., performance, memory pressure).


Hi I am not able to create an issue in this JiraTestResultReporter repo...
can you please solve it?


검색 결과, 현재는 여기서만 사용되는 루틴입니다. 자주 사용될 가능성이 있어 보이니, 추후 같은 루틴이 한 번이라도 반복되면 같이 분리하겠습니다.


А зачем удалять значение? Может это временно отключили


ack


Сортируются данные всей таблицы, поэтому при большом количестве записей возможно зависание. Нужно оптимизировать код для ускорения отрисовки.
Не очень понятно про "страница не сортируется естественно". Какое поведение ожидается?


I think '-bind' cli flag is not applied,
because of the call of bind.Default() before flag.Parse()


Seems clear enough to me as a ruby dev. There's also `.push` but I've seen `<<` commonly in ruby code.


Не понятно почему 12 именно должно быть?
Давай 7 или 10. Можно вообще ничего не писать и пользователь сам сделает


unneeded newline


null check는 iterator.hasNext()가 하는게 아닌가요?


91  +  ABCD -->  This line apply style css file in same orgin.
 92  +  Tis line apply style css file not in same orgin


why not `for failed, skipped, message in pool_outputs:`?


hey, you seem to change those back and forth in several commit, I'd suggest you avoid that ;)


Test


lgtm


lgtm

I mess this up all the time. =[


Teeeest


The admin page of the website


Hi! I apologize for not responding earlier, I completely missed your comment!
Are you still interested? If you are I will try to hurry up and do it.


Maybe change the original script with one more command line option.

./xx.rb display_list_index to_space_id


Closes #1. I didn't know I could close issues through commit messages.


yep


Seems there are other issues apart from this one with the use-case I was working on in 4.2.3. I'll see about checking master / making up a PR.


Yes, I think so, there is probably no need to keep an auxiliary package in this public list.


The persistent timeout, while a (happy) side effect of how I implemented the bug fix, isn't a necessary part of the bug fix. It's also not related to the bug that's being fixed, functionality wise. It's really just a new feature that I ended up implementing along with the bug fix because I personally prefer it that way. However, I expect that there are people that when they close the chat window they want all of the previous chat to go away. Hence I felt it best to leave it as an option.

The actual bug I'm fixing is that as lines in the chat feed time out those lines are never shown again, no matter how many new messages are added, or even if you open and close the chat window. So shortly into a game the feed is reduced to only two functional lines of text. But for the persistence part, I don't think it's a bug that closing the chat window would clear the timer on all current lines in chat. Like I said above, having them instead resume their timer after closing the chat window was just something that ended up being an easy win with how I implemented the fix and I liked the functionality better that way.

If you guys feel differently about including the option I don't have a problem with changing that.


It allow to load style in same orgin and deny cross orgin's, so the color of two lines below are different. -->
The following 2 lines load style src file in in same orgin and cross orgin,
The "style-src" restriction allow to load style in same orgin and deny cross orgin's, so the color(Green) style of first line takes effect while secondline does not.


I was trying to make every commit work ... In this commit I change it because this commit wont pass circleci with the old timeouts as they are too small.
Later with the `-j` option, these timeouts can be decreased again.

Was trying to make every commit pass circleci


@chris-morgan I think that's a good call. I'd consider it a bug in the original implementation that you could click outside of the X.


Why not declare this variable `mutable` instead of using a `const_cast`?


Нужно проверять будет ли вообще задача перемещаться. Если нет, то ставить значение 0 или -1


`reinterpret_cast`? Why?


ah ok, then maybe mention in commit messsage

2015-06-30 7:40 GMT+02:00 AbdealiJK <notifications@github.com>:

> I was trying to make every commit work ... In this commit I change it
> because this commit wont pass circleci with the old timeouts as they are
> too small.
> Later with the -j option, these timeouts can be decreased again.
>
> Was trying to make every commit pass circleci
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/216c701457ba4b735c9f00cdc66d85c308d81636#commitcomment-11924532>
> .
>


Awesome! But, a suggestion?

Since we're removing these kind of stuff that aren't supposed to be there, why don't we bring Spell1 - Spell8 to a new table as well? character_vehicle_spells? Hmmmmmm?


@formanek I probably misunderstood your comment. Did you implement a new dataflow mechanism? (I am very open to integrate new ideas)

Here is a quick overview of the proposed new implementation.

Once a method contains a suspicious api (see *position 31* in the following sample). The [`ByteCodeReplay`](https://github.com/h3xstream/find-sec-bugs/blob/43e579a20c5750f0fd2552e18529805e6a8af874/plugin/src/main/java/com/h3xstream/findsecbugs/injection/util/ByteCodeReplay.java) class simulate all the operation up to the targeted position. The simulation reproduce the impact of the instructions to the local variables and the stack.
Every source is consider tainted except constant loaded (`LDC`). I track the state of StringBuilder instance. Once an instance is affected by a dynamic value, it become tainted.
Once the position is reach, the stack state is use to detect if the parameter being passed is safe or suspicious.

**Byte code :**
```java
[-1] NOP
[00] NEW java.lang.StringBuilder
[03] DUP
[04] LDC "select * from Users where name = usernameParam"
[06] INVOKESPECIAL java/lang/StringBuilder.<init> ((Ljava/lang/String;)V)
[09] ASTORE (?prev?) => 3
[10] ILOAD 2 => [stack]
[11] IFEQ target =>   21: aload_0[42](1)
[14] ALOAD 3 => [stack]
[15] LDC " and active = true"
[17] INVOKEVIRTUAL java/lang/StringBuilder.append ((Ljava/lang/String;)Ljava/lang/StringBuilder;)
[20] POP
[21] ALOAD 0 => [stack]
[22] GETFIELD em
[25] ALOAD 3 => [stack]
[26] INVOKEVIRTUAL java/lang/StringBuilder.toString (()Ljava/lang/String;)
[29] LDC "testcode.sqli.UserEntity"
[31] INVOKEINTERFACE javax/persistence/EntityManager.createQuery ((Ljava/lang/String;Ljava/lang/Class;)Ljavax/persistence/TypedQuery;)
[...]
```

**Original code :**
```java
public void queryConditionalConcat(String username,boolean onlyActive) {
    StringBuilder sql = new StringBuilder("select * from Users where name = usernameParam");
    if(onlyActive) {
        sql.append(" and active = true");
    }

    em.createQuery(sql.toString(), UserEntity.class)
            .setParameter("usernameParam", username)
            .getSingleResult();

}
```
As an example, here is the **stack** and **local variables** at the *position 31*

```java
==================================================================================
Stack :
<00>  OK       java.lang.StringBuilder
<01>  TAINTED  java.lang.Object
<02>  TAINTED  javax.persistence.EntityManager
<03>  OK       java.lang.String
<04>  OK       java.lang.Class (LDC testcode.sqli.UserEntity)
==================================================================================
==================================================================================
Local variables :
<03>  OK       java.lang.StringBuilder
==================================================================================
```
Since the value at stack index `<03>` become the second parameter (second `pop`) passed to `createQuery()`, it is consider a safe call.

-------------

I hope it give you a high level picture.
In the case of hard-coded value, I'm pretty sure tainted value would means safe value. For the detector [`ConstantPasswordDetector`](https://github.com/h3xstream/find-sec-bugs/blob/a6ea3e832b6a98ed5fb6c428e1c16cabdd7a74c2/plugin/src/main/java/com/h3xstream/findsecbugs/password/ConstantPasswordDetector.java), many things would need to be added.
 - Consideration for byte array, char array and BigInteger
 - Intuitive guess for


when_date_automove не нужно показывать. Достаточно сколько дней осталось. Это ведь не жесткая привязка. Незачем пользователя датами перегружать.


This PR has nothing to do with 4.2.x, and wouldn't be possible there. There's lots of larger issues that these refactorings have attempted to address, that are far out of scope for a 4.2.x fix.


Could you add a test, matching the reported bug, to ensure this stays fixed while we refactor etc?


> ignore errors when calling middleware to set debug mode as debug-mode
  has function now in clj only too.

What does this mean?

I think it's a good idea to give the user immediate feedback when the function he called fails for some reason.


Resolves #2.


indentation


indentation, below too


ack


ack, any progress on getting it installed btw.?


this crosswalk core lib beta version need to updated during each build ?


They are more than vehicle spells.


also this version


Spelling Error


add basedir support


No ... I haven't checked on it yet. I'll figure that out and squash it with commit.


This line would display fatal errors on PDO including exceptions and disables prepared statement emulations


i think this is causing the following failure for me using vagrant on OS X.

```
PLAY [Set scheduleability] ****************************************************

GATHERING FACTS ***************************************************************
ok: [master]

TASK: [set_fact ] *************************************************************
fatal: [master] => Traceback (most recent call last):
  File "/usr/local/Cellar/ansible/1.9.1/libexec/lib/python2.7/site-packages/ansible/runner/__init__.py", line 582, in _executor
    exec_rc = self._executor_internal(host, new_stdin)
  File "/usr/local/Cellar/ansible/1.9.1/libexec/lib/python2.7/site-packages/ansible/runner/__init__.py", line 785, in _executor_internal
    return self._executor_internal_inner(host, self.module_name, self.module_args, inject, port, complex_args=complex_args)
  File "/usr/local/Cellar/ansible/1.9.1/libexec/lib/python2.7/site-packages/ansible/runner/__init__.py", line 1009, in _executor_internal_inner
    complex_args = template.template(self.basedir, complex_args, inject, fail_on_undefined=self.error_on_undefined_vars)
  File "/usr/local/Cellar/ansible/1.9.1/libexec/lib/python2.7/site-packages/ansible/utils/template.py", line 138, in template
    d[k] = template(basedir, v, templatevars, lookup_fatal, depth, expand_lists, convert_bare, fail_on_undefined, filter_fatal)
  File "/usr/local/Cellar/ansible/1.9.1/libexec/lib/python2.7/site-packages/ansible/utils/template.py", line 122, in template
    varname = template_from_string(basedir, varname, templatevars, fail_on_undefined)
  File "/usr/local/Cellar/ansible/1.9.1/libexec/lib/python2.7/site-packages/ansible/utils/template.py", line 371, in template_from_string
    res = jinja2.utils.concat(rf)
  File "<template>", line 13, in root
  File "/Users/dlbewley/src/openshift-ansible/filter_plugins/oo_filters.py", line 80, in oo_collect
    retval = [FilterModule.get_attr(d, attribute) for d in data]
  File "/Users/dlbewley/src/openshift-ansible/filter_plugins/oo_filters.py", line 38, in get_attr
    ptr = ptr[attr]
KeyError: 'openshift_hostname'
```


Имелось в виду что возможен только один вариант: или сортировка всей таблицы, что очень медленно, или без сортировки вообще (только средствами самого запроса). Я видимо не понял в чем был баг. (Cлово "естественно" это не к поведению, а констатация факта, что без сортировки скорость работы сразу заметна). Для больших таблиц идеально было бы сортировать только страницу результата (при помощи опции типа make-sortable-scope или make-sortable-result).


See https://springrts.com/mantis/view.php?id=4861


This will cause AlloyEditor exclusive widget (probably?) to hide all toolbars


@zhuyongyong for the version, I assume it need to be identified when build,
It is better to auto update when build rather than hardcode update.


Что-то я тут не понял сначала. У нас allow_automove говорит будет или не будет перемещаться. Зачем еще 0 или -1? Это куда?


image 1, load from resource in same orgin. should load scuessfully.


It is mentioned in the commit message
`Finally, increase some of the timeouts used in
various places - as some tests which create subprocesses within them
take more time to complete now - as it needs to wait for other processes
to free up before executing.`


Ээээ, а как потом этот гем для других магазинов использовать?)


cc @jlipps


True that! 😅


@jakennedy Is this change really needed?


Потому что track_visibility='always' у нас же


@svenmueller sorry, now directly in master.


Hmm, why? What's funny?


What are the coordinates doing in package.json?


This check is not required here.


it would be safer to return 0.1 if we don't recognise the vehicle class


g


gfg


Why added curlies?


I think we should call `target` "project" since targets mean something else. See https://msdn.microsoft.com/en-us/library/ms171462.aspx


We can call `build` `target` if you wish


Delete :
    $lang->load('ckeditor');
of this file because it show an error when you want to delete this plugin
thanks


LTR: This is content 9
RTL: 9 This is content


I suggest to use standard `Arrays.stream(T[] array)` here. Not necessary to create ArrayList for streaming.


1. Subframes 1,2, and 3 with hex-codes are not included. Clarify this on the final OGRP.
2. Time_stamp: It is better to separate in week number and seconds a week.
3. GPS week number was set at 1.1.1980 to match up with UTC.
4. Does 'timestamp_xxx' mean signal transmission time corresponding to each satellite? It is confused because time in channel measurement section, should be the receiver time (signal receiving time). If it is right, better to move all into ephemeris section.
5. Carrier ranges are not needed in channel measurement section. Because a receiver itself can not estimate the integer ambiguities and it depends on user operation.
6. Code-phase is a similar case with carrier-range as long as not providing at every code epoch.
7. In signal type, "E5a" is "E5b", right? because "E5aI" exists.


Can we get the pom changes and docs in two separate PRs? Those look like two separate issues to me.


Send pull request so I can merge this.


It's probably not working because you already committed the compiled code before adding the Gitignore.


yes you are right, but I will wait until there is something substantial to commit for a 1.0.6 release and update then.


My bad, I did it just to be able to set a breakpoint there. No curlies and the breakpoint wouldn't trigger for some reason


Исправление: 3 этап.


is there a bug report about this? if yes please paste the link here


destroying the dialog should not quit the full application


91958608DOASEffectOnZoneSizing (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1159 of 1166 tests passed)

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


these params are not properly aligned


@inakianduaga Globs are not supported by TypeScript, see Microsoft/TypeScript#1927, Microsoft/TypeScript#3043


You could add a `-worker` prefix to this function as it's used throughout the code to indicate internal functions.


use 4 space indentation


call it create_cert


add the modeline at the end of the file


check the exit status for each command you call


bit of a typo here, sry. it has functionality in clj-refactor.el only. that means if we can't call the middleware we should not throw an error as the var is set in emacs -- best effort. debug mode will be set in the middleware anyway if/when it starts up


I don't think you need the arrow here. Also make sure the indent below is correct


test


>  it has functionality in clj-refactor.el only. that means if we can't call the middleware we should not throw an error

Ah, if that's your intent then I think a better approach is to check if the middleware is present or not (we have code for this already).  That way the user still gets feedback if the middleware is connected but the interactive call fails to turn on debug mode for some reason.


ni 号


nice


can do that too. no worries


845185892865c47c6efb15832b79f36966e7f769
the comment is updated
EFFECT_NO_EFFECT_DAMAGE  means "the player is already affected by changing effect damage to 0"


okay, @jakecraige sorry for my style, i am new on github, and sorry for my
languge mistakes, i come from non-english part of world
and about the code. i see what you mean - i think, but... the condition
above this line doesn't guarantee the err is not an error (with possible
stacktrace)
i used mocha, and sandboxed-model to test my nodejs code, and i make a
mistake: the injected dependency doesn't have an important property. this
couse the original code couldn't run, because throw an exception:
'TypeError: Cannot read property 'xyz' of undefined'. the (TypeError
instanceof Error) condition is false, so the condition is wrong in my case.
and i am angry because waste my time, i cannot found what does i wrong,
what is my mistake, and the funny errormessage (which drop the original
stacktrace) bad for me.
i propose a little code, but i see, that code is buggy. so, i think the
condition above this code is wrong, because typeerror doesn't instance of
error. i don't know how many type of errors now, and the future, so i
propese fix the condition: if(typeof err.stacktrace=='undefined') or
similar.
and againn sorry :)

2015-06-30 4:47 GMT+02:00 Christopher Hiller <notifications@github.com>:

> I think @sarkiroka <https://github.com/sarkiroka> probably wound up here
> searching for why s/he received the exception. Hopefully your explanation
> will suffice.
>
> I'm honestly not sure why JS even allows throwing a non-error. I think I
> saw this done somewhere in angular-mocks, but maybe not. Anyone know a
> valid use case? Maybe I'll ask SO.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mochajs/mocha/commit/02d421b1b548c55ce8c0bd12aae3e94f635aa4ba#commitcomment-11923239>
> .
>


嘛 尽量不要写这一个org吧 可以换成 `http://telegram-china.org`
另外对于汉化资源来说我们推荐还是去找机器人 `@tgcnlang_bot`


Yes as a big noob i had missed it.
It's now fixed.
Thank you


SDK is available in jCenter()

compile 'com.github.alorma:github-sdk:1.0.1'


Hey man, these are localhost links ;)


Or it meant to be localhost? Then my bad.


@KingFelix rules, closes #7


Maybe an alternative to the last sentence:

"The Working Group solicits feedback on whether this mechanism is useful and whether it represents major implementation difficulties."


s/debuged/debugged/


closes #1


I think this section is the issue behind #1131. Is it just me, or are the `X.{pe,se}` files are created in the current directory, regardless of `args.output_{single,paired}`?


HBIRE_vector_take2 (amirroth) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1680 of 1682 tests passed)

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


i love html


The build is broken due to this commit. https://api.travis-ci.org/jobs/68925989/log.txt?deansi=true
This is a good reason to always make a PR, let travis check if everything works before apply changes into master. :smile:


Please use try-except-finally and move `do_cleanup` into the finally block.


English?


closes #5


Likewise this bit is also a probable cause of #1131


where do I put this file to download 1.6?


HBIRE_vector_take2 (amirroth) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Peter there is also need for updating the [AndroidDownloader](https://github.com/graphhopper/graphhopper/blob/master/android/app/src/main/java/com/graphhopper/android/AndroidDownloader.java#L63) extending class.


fffff


@19peaches Please run `brew update && brew upgrade elasticsearch`. If you have any trouble, please submit an issue. Git log comment isn't the right place.


Thanks!

Date: Mon, 29 Jun 2015 06:21:29 -0700
From: notifications@github.com
To: PrestaShop@noreply.github.com
CC: twc02@hotmail.com
Subject: Re: [PrestaShop] [-] FO : Fix bug #PSCSX-4424, could not index products in multishop (6ee1c4b)

Hi,

4c87e57

Regards

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


> It was added by @milipili

Huh?

![authored_5_days_ago](https://cloud.githubusercontent.com/assets/2142308/8425250/f81c6f5e-1ed1-11e5-8fac-f9eae95e2b3d.PNG)

...ok, I guess GitHub can be funny.

Sidenote: does `Notepad++` have any sort of logging infrastructure? This should really be logged, not `assert`ed. Maybe even a plain old `OutputDebugStringA("Notepad++: failed to retrieve system info 'SPI_GETFONTSMOOTHINGTYPE'\r\n");` would be better?


Why is this not:

Package ('yasm', '1.3.0', sources = ['http://www.tortall.net/projects/yasm/releases/yasm-%{version}.tar.gz'])


The dialog title has the wrong capitalization and should be marked for translation


Can you rename the result to `stdout, stderr`? After all that's what it is. -- `err` may suggest it's an error code.


I already fixed this, I think it will work with new vala: https://bugzilla.gnome.org/show_bug.cgi?id=751142


Thanks worked perfect. Sorry for wrong placement of comment.


spelling is hard


@firejun Thank you for your report and contribution. We've finally closed [the #365 issue](https://github.com/leo-project/leofs/issues/365#issuecomment-117022436).


In the spirit of proper error propagation, the ideal would be to display the error "No appropriate VO was found", and return the original message, so "return res", but I suppose here it is acceptable


Done : commit c1ac171


What does "dto" stand for? Maybe add a class comment to explain what this class is used for. Thanks!


please add a log statement (at debug or trace level) for such an action.


@danielsdeleo @lamont-granquist @double-z @justinredd

There is a problem with having fully qualified names and version constraints on `automatic_attrs[:recipes]`. The cookbook list on `automatic_attrs[:recipes]` resolves to contain a list of cookbooks, recipes, and their version constraints.

I would expect the list to contain only the cookbook and the recipe, since the version numbers break code elsewhere (see notes below).

Consider,
- A user calling `include_recipe` in a recipe cannot pin a version constraint.
- The synchronized cookbooks always download the pinned version of the cookbook, so the run list does not require the pinned version in the list—it's already downloaded.

An alternative is to alter the client to accept a fully-qualified cookbook, recipe, and version constraint, and disregard the version constraint.

[@justinredd pointed out that PR 2312 is related](https://github.com/chef/chef/pull/2312/files#r29922276).

**Problems caused by passing fully-qualified cookbooks, recipes, and versions**

[`#with_fully_qualified_names_and_version_constraints` is now producing a hash of recipe names with full version pins](https://github.com/chef/chef/blob/d8172e646d9fbf43e57bca5e20d0ac352ba9a66a/lib/chef/run_list/versioned_recipe_list.rb#L71-L83), which causes exceptions elsewhere when checking to see if a recipe is found.

For example, loading with the fully-pinned cookbook string parameter

    cookbook_version#load_recipe('myrecipe@1.2.3', run_context)

will fail because `cookbook_version#recipe_filenames_by_name` [will produce](https://github.com/chef/chef/blob/master/lib/chef/cookbook_version.rb#L611-L613) something like

    {"mycookbook"=>"mycookbook.rb", "bbb"=>"bbb.rb"}

and [elsewhere at `cookbook_Version#load_recipe](https://github.com/chef/chef/blob/master/lib/chef/cookbook_version.rb#L216-L218):

    unless recipe_filenames_by_name.has_key?(recipe_name)
      raise Chef::Exceptions::RecipeNotFound, "could not find recipe #{recipe_name} for cookbook #{name}"
    end

will raise a `Chef::Exceptions::RecipeNotFound` error, since `recipe@1.2.3` is not `recipe`.


Append a comma after this line


should this not be something like "52n-sos-dashboard" ?

Also, have you considered multiple config files? Or can Kibana just take multiple dashboards/configurations in one JSON file?


hello
there is a time we know about recent technologies and try to put them in our daily lives


@rosenhouse Does this project migrated to a new one or just abandoned?


This currently prevents us from running ilastik successfully, as we do not fix a version of h5py - and the newest h5py doesn't work with the old hdf5 version any more. (libs have names `libhdf*9*` vs. `libhdf*10*`)

Is there a solid reason for not using the newer HDF version in vigra/opengm/itk?


There shouldn't be, it just fixes the case for when games provide oddly sized textures that arent in pow2 and ensures Direct X will create a properly sized texture that wont get it confused. DirectX 9 from memory can support non power of 2 textures but only under very limited circumstances, and also requires one of the device caps to actually be set to say that the app will be using them.


also retrieve the inserted configuration and read one specific value, e.g. the dashboard id, or the index-pattern with the name ``ogc*`` exists.


yes. and also I didn't take care of single quotation mark.
But no worries. The BUG has been killed.


已经更换链接，并且加上机器人的AT了https://github.com/jqs7/telegram-chinese-groups/commit/fdf1110763df666219069478db962bbd2a383146
------------------------------------------------------------------发件人：jsw <notifications@github.com>发送时间：2015年6月30日(星期二) 14:30收件人：jqs7/telegram-chinese-groups <telegram-chinese-groups@noreply.github.com>抄　送：Jqs7 <7@jqs7.com>主　题：Re: [telegram-chinese-groups] 新增非官方汉化资源链接 (f547c7a)嘛 尽量不要写这一个org吧 可以换成 http://telegram-china.org
另外对于汉化资源来说我们推荐还是去找机器人 @tgcnlang_bot—Reply to this email directly or view it on GitHub.


Oh I guess that dependency was reintroduced after my recent changes? Somehow a merge inside pgmlink must have done something wrong to the CMakeLists.txt or sources. I'll remove that again soon.


Isn't this still one step too much? I mean, first setting the alpha value of every pixel to 255 and then discarding any alpha value of each pixel by the conversion to RGB (without alpha)?

Wouldn't it suffice to use `image.toNewBufferedImage(BufferedImage.TYPE_INT_RGB)`?


I think it is fine to have lots of in/out arguments, as long as you add
comments in the C code so you know what each of them are. You can use a
struct if you think it will simplify things. I would recommend making
design choices based on speed if you can (less reads/writes/function calls
is usually faster). Also I talked to Mark yesterday and he said that out of
the 3 versions of the SAG code the most complicated one is the fastest
(*SAG_LipschitzLS)
so I would recommend using that as the basis of your L2-regularized GLM
implementation.*

On Mon, Jun 29, 2015 at 8:23 PM, Ishmael Belghazi <notifications@github.com>
wrote:

> @tdhock <https://github.com/tdhock> So I started modularizing the code. I
> broke-up the main SAG loop in an outer loop responsible that will call
> _sag_constant_iteration at each visited example. I am a bit bothered by the
> function above, it just seems really messy. It has in arguments, out
> argument and it returns the nCovered variable that SAG updates after each
> iteration.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IshmaelBelghazi/bigoptim/commit/f5d2d3827e6b93182281dce1ebbcdbf0d682c466#commitcomment-11916057>
> .
>


@iyee It has not been migrated.   It is still used by some CF components, but no new feature development will occur, and we strongly discourage its use in new projects.

cc: @ryanmoran @pivotal-cf/cf-la


@andyjmaclean This is test content and does not belong in the view template.


Ah yeah, sorry for that. My IDE has auto-ident on


@andyjmaclean if we are switching to selenium, please remove poltegeist.


k, will look at them when have time (hopefully over the weekend). Have another massive review that am planning to start soon for MU2 stuff:
https://github.com/zhhealthcare/openemr/commits/CareCoordinationV3


@andyjmaclean `if (true)` does not belong in the repo


@andyjmaclean this debugging stuff does not belong in the repo


I don't have time to look in detail right now, but at a first glance I
don't really understand what those macros do and how they help clarify your
code. Anyways it is your coding project so if you think it makes the code
more readable without sacrificing performance than that should be fine.
Let's talk about that in our next face to face meeting.

On Tue, Jun 30, 2015 at 5:20 AM, Ishmael Belghazi <notifications@github.com>
wrote:

> @tdhock <https://github.com/tdhock> I was missing apply so I made a
> couple of apply-macros that may or may not make the code less repetitive
> and more readable. Please tell me if you think I am going over board with
> stuff.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IshmaelBelghazi/bigoptim/commit/151eee55e1330b544d738e036512c066d0aebed9#commitcomment-11923627>
> .
>


HBIRE_vector_take2 (amirroth) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (650 of 650 tests passed)

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


Despite updating to the latest version of steroids I am still facing the same issue. How do I get this running also for cloud deployments?


This increases the code confusion even more. See http://foswiki.org/Tasks/Item13484#Analysis


Also simplify this method a bit.


If you want to use C++ instead of C that is fine too.

On Tue, Jun 30, 2015 at 5:30 AM, Ishmael Belghazi <notifications@github.com>
wrote:

> @tdhock <https://github.com/tdhock> I though you might like this. This is
> really starting to look like an object, just mentally replace &trainer with
> "self" or "this". Beyond aesthetics and having fun, I think this interface
> will make thing easy when we'll move to passing arbitrary C or R function
> to SAG. It would be as simple as defining a new model containing pointers
> to the relevant function while keeping all the remaining code unchanged.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IshmaelBelghazi/bigoptim/commit/80cff82a26c65d073eb474c54ff2be0cd7943661#commitcomment-11923704>
> .
>


WTF is bdogg?!?! @bdogg718k hahaha


If the source is not enabled in ES, if you query with _source_include = ['include'] part, the response never gives you the hits as it checks the source to contain these fields. Could you please have a look at the same?


nice, I got this and just do a work around to avoid dropping message with EnforceSecureReception to false.


there's a missing space in the string, resulting *„preventingthe“*.


@vpisarev, I'm shifting loaded vectors to reduce the loading of values for the middle column values when applying a Sobel filter. I tried to find a way to do it with the current HAL API but could not, so used `_mm_slli_si128` and `_mm_srli_si128` to do so.

Did I miss it? If so, what should a relevant method be named? I could add it to hal.

In practise this change does not seem to improve the performance of the code.


Should not `https://subscribe.adblockplus.org/` always be first part of the subscribe URL to allow including special _subscribing_ script for this domain only? Or even intercepting filter list url inside _filtering engine_? (I know ABP scans DOM for all links)


``` python
for mail in ProgressLogger('Migrate mail message filename', objects):
    mail.update_filename()
```


It's not as simple as placing the two lines in every script, otherwise we would just make changes globally.
The issue with the above flags is you only place them when you convert the entire script/module into the new security module. This requires conversion and testing of everything. When you are focusing on just patching up isolated security holes (such as you are doing here), then shouldn't use the flags (unless the script is super simple to completely convert, which is not the case here; to convert this properly, would need to convert the entire CAMOS module which is a huge undertaking).


Alright, thanks for the explanation!


Yeah but here it uses the same build number and same artefact which is not good. I wanted the same and in appvoyer docs it was exactly the same like my last edit. I just wanted it to be 4.0 beta and different artefact so it doesn't overwrite the master


Reviewed!


@ivogabe Yeah, thanks for the response, found that out a bit after I posted the comment (sorry for not updating it). Funny thing is your gulp-typescript was actually working just fine with globs before this update (the check can be commented out and the compilation works). Seems atom's tsconfig implementation has a "filesGlob" field where globs are supported

https://github.com/TypeStrong/atom-typescript/blob/master/docs/tsconfig.md

At any rate, I just switched to gulp.src that obviously accepts globs, so all is fine.

Thanks for the help!


typo now -> nor


For a straightforward script conversion, here's one your already did :)
https://github.com/openemr/openemr/commit/a5437757fa25dd7c0cd96f9c9edb8b33a4bc591c

For some example of more complicated conversion, check out the following commits here:
http://www.open-emr.org/wiki/index.php/Codebase_Security#Implementation

Regarding your work, I'd rec continuing on fixing the issues in Brian's above vulnerability report and working with Kevin, if he's available(I and the community are also happy to review your stuff if needed since getting OpenEMR security up to par is important). If I see a script your working on that would be worth converting (ie. minimal resources), I'll let you know. You'll learn a lot about security by doing this after just several more fixes.

-brady


Oh, now I see... it's an un-escaped minus sign... I thought you meant that we were using a character other than a minus sign (as has happened before as a result of doing copy&paste from my slideware (where autocorrection changes the "minus" character to a different one))


I wonder, why you removed these files, how can I save config in flash ?   If I try to keep this module with my 4MB module I can't manage to build my project.
Is the 0x3C block not available anymore with OTA enabled ?


+1, some pre-existing tests fail because of that.


I don't know if this line should stay, but the error it generates shows my current problem rather nicely - v is serialized as "<django_hstore_tests.models.CustomType object at 0x...>" instead of "[some value]".


Does not seem to be as good as the version cyanogenmod has pushed online.


Not sure, but I think maybe this will put `<commcare_phone_number>None</commcare_phone_number>` if the phone number doesn't exist. It should likely default to an empty string. This would be a super easy thing to add a test for


Why was this method added?


@baccia: there was a typo here... It is a bit tricky, but a typo in the yaml file breakes the entire page. Next time save it as a branch and notify me so we can check it off-line first.


stywertyrety er


etry werter


etyertyertyery


why I don't want update core of rule: channel, time slot, search type


See #188 for explanation.


i am going to bpl


this argument is in minutes, not seconds


 http://rockitbootcamp.com/

Just here for storage!


yeah this isnt needed anymore, as the same happens in the callee...i squashed the commit


Maybe that's why nothing happened when I tried it on avalanche.


No need to check for the nullity of `obj` again


This is actually not required and left-over from my initial spike, it is however not unhelpful to have available options in the example configs, so perhaps leave?


Should I leave these as functions or switch to an if-elif set of statements?


IQR is replaced with the MAD if it's 0, which means I can't shrink these statements more.
This was meant to avoid a nested if-elif instead.


Capturing this *shouldn't* actually be necessary now as the work to lock the channel around each request should mean the .NET Rabbit API does get it's knickers in a knot any more. However as the .NET Rabbit API is the work of a masochist, and given use of Polly, this perhaps is worth keeping in but perhaps with WarnException instead?


Am concerned this is too verbose for the parameter set.


I use the same approach now. I'm going to think about it some more, maybe a nice solution is possible.


Commenting this out in the manner seems like a smell, but it's important it doesn't end up in production, perhaps a discussion around release/debug builds being published to NuGet is worth having?


does `self.assertEqual(d.custom, v)` yeld a different result?


Thanks. Will update.


`libc++-dev`


This change breaks standard usage pattern, using the config file bundled in jar.
This means that the very first example on how to use CheckStyle (http://checkstyle.sourceforge.net/cmdline.html#Download_and_Run) is broken.


You don't need this. These are set by CMake


"ASC" büyük değilde "asc" şeklinde olabilir.


kugiklkkhkjhkh


0x3C block is located at 0x3f000 in these code, which is not suitable
for OTA of 4MB (1024+1024) since the config location is overlapped with
image1.

If you really need this "config saving", write into a separated module.
Ah, don't forget to write API to enable the ability to re-config via
rest/mqtt or this kind of config make no sense overall.

On 30/06/2015 14:23, Jérémie Foucault wrote:
>
> I wonder, why you removed these files, how can I save config in flash
> ? If I try to keep this module with my 4MB module I can't manage to
> build my project.
> Is the 0x3C block not available anymore with OTA enabled ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nqd/esp8266-dev/commit/5bdd5f24fcf61a83768209fcb65891baf59fb669#commitcomment-11925567>.
>

--
Nguyen Quoc Dinh,

CTO, Ubisen Co.


right, I'll fixed that, wrong file


hi


Working hard (or hardly working, hoho) I see McCoy (@surrealmccoy). Trying to make fancy smancy exclamation marks are ye? Walking round thinking you're deadly at the same time no doubt.

Actually, "fonts" could be a cool feature down the line.

By the way, you did this on the addMaps branch of your fork. You can work away there and send me a pull request (probably not as dirty as it sounds) and I can still incorporate it into the real master. But if you want the latest app, you'll need to rebase, which might be difficult (I say might because I have no idea, I think you have to specify my repository as upstream and the rebase against it)

Git is quite interesting when you think about it. Must be very powerful if you know what you're at. Linux is opensource, they use Git for version control.


UPLOAD_UUID doesn't exist in CallHomeSettings. Is it missed?


Javascript'de keyler için ' (tırnak) işareti kullanılmayabilir. Genelde kullanılmamakta.


@UtahDave so sorry, all my fault and confusion.
I had only glanced at your commit commnent and missed the fact that you only removed the "extraneous msiexec=True ", with english not being my first lanaguage I read it as you removed the "superfluous misexec=True", in other words I thought your comment said that 'msiexec=True' was not needed, when in fact you only removed the duplicate entry for it. If I had looked at the file for a couple seconds longer I should have noticed that it is still needed.

So based on that mis-reading and mis-understanding of your comment I questioned why the docs still say that msiexec=True is to be added for msi installers.


Hi Tom,

I spend a lot of time finding this error and solving it, only to find you had already solved it. how can i get your version of the file in the adapt authoring tool? this version is not automatically installed, because it is still in the development branch.


Question: why did you use type=6 for all the texts?
Unless there is a strong reason to use 6, you should use 1.


lol


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


These are really messy. Have you cleaned them with ``msgattrib --translated --no-fuzzy --no-obsolete --no-location`` ?


В последнем коммите я уже исправил свою же ошибку )


Yes, it crashes on the `assert raw.startswith` in `CustomType.deserialize` which shows a rather unhelpful message from the test runner. By looking at the actual raw value you can see why it fails.

But it's the same if you only look at the fail/pass status.

Federico Capoano <notifications@github.com> napisał:
>does `self.assertEqual(d.custom, v)` yeld a different result?
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/djangonauts/django-hstore/commit/3216fdc9615a8b090ee947f9fccb4fe1dbd423fa#commitcomment-11925816

--
Wysłane za pomocą K-9 Mail.


78行“CPU熄屏”去掉"CPU"比较好


Thanks, @Morfent! The `packedTeamIcons` code you wrote ended up in PS after all!


I wonder if "web interface language", "browser interface language" or simply "interface language" is clearer than "WWW language"?


This gives issue if the timestamp is unix_ms. "object of type int doesnot have any len()".


Ah, ugly and travis-ci is not reporting because it cannot build Android ... will fix!


So is there any replacement recommended?


My bad, I must have staged my changes on an old version.


I'm on a phone now, but I think I forgot to implement __eq__ on CustomType, so that might not work even after #123 is fixed >_>

Federico Capoano <notifications@github.com> napisał:
>does `self.assertEqual(d.custom, v)` yeld a different result?
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/djangonauts/django-hstore/commit/3216fdc9615a8b090ee947f9fccb4fe1dbd423fa#commitcomment-11925816

--
Wysłane za pomocą K-9 Mail.


You have to remove this line.


Since composing and submitting the requested answers from Github turns out to be quite convenient for all applicants, *private persons* would feel definitely happier if you guys kindly setup the repo as a private one the right way. There're some pros on going down this path for you guys as well: The appraisal test remains private and nobody can assess the appraisal test before actually applying for the role.

For doing so just click on the *Settings* link at the menu on the right in the repository main page and then scroll down and set the repo as private. Applicants will have to be invited to join the repo before forking it and nobody will see such fork in the applicant's public repository list. E.g.:

![captura de pantalla 2015-06-30 a las 9 56 50](https://cloud.githubusercontent.com/assets/1104146/8426223/6cd2e044-1f0e-11e5-9704-87feb72f9c64.png)

Thanks!


Why did you add these two lines ? I added their error explanations a few days ago. They have nothing to do here.


el alışkanlığı :)


Yes, just needs a `tryCatch...` to reset the flag when it fails


don't think this is needed?


HBIRE_vector_take2 (amirroth) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1167 of 1167 tests passed)

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


Onu kullanmaktaki nedenim, post edilirken kullanılan keyi ayrıştırmaktır. Bir sıkıntısını görmedim açıkçası...


You are correct, there is no _direct_ relation between those two settings, but there is an _indirect_ one: If you target WinRT which is indicated by `BOOST_PLAT_WINDOWS_RUNTIME_AVAILABLE` the APIs `CreateEvent`, `CreateMutex`, `CreateSemaphore` and many other older APIs won't be available and things have to be done in a different way (in this case we use the Create*Ex APIs). So if `BOOST_PLAT_WINDOWS_RUNTIME_AVAILABLE && BOOST_USE_WINAPI_VERSION < BOOST_WINAPI_VERSION_WIN8` evaluates to true it will be a configuration mistake.


This looks great. @cldougl 's adding subplot titles to make_subplots in the next few days, might want to wait for that.


格式化一下... 乱了... 哈哈


@Ladeburger Obviously, yes…


It also removes root access


![capture du 2015-06-30 10 07 34](https://cloud.githubusercontent.com/assets/151042/8426458/167740c6-1f10-11e5-898a-e877f7c34913.png)


Removes a compiler warning. It's ambiguous. Is the else I added part of the first if, or second if? Of course we know it's second and I'm sure the compiler got it right. But, it complained in Travis, so I changed it to be unambiguous.


Please remove this line, this is why the test break I think.


I think it could be better to leave the "For the manager itself, [...]" in the first paragraph? I don't see why it is on the same paragraph as the properties. Giving a new paragraph just for the properties will help to distinguish the what "systemctl show -p=XXXX" do and what can be the XXXXX.


@BGS deci esti viu.


We can merge it list this, but just for the record it seems like `self.get_dim(name)` would do the job  for all names except for `feedback`, for which indeed `readout` should be asked.


TeamCity Ybackend :: synteny/ybackend [Build 56](http://192.168.30.225:8111/viewLog.html?buildId=1477&buildTypeId=Ybackend_SyntenyYbackend&guest=1) is now running


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1167 of 1167 tests passed)

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


My mistake : this line must be deleted, ioinfo was made to help me make benchmarks and tests on MPIIO hints at file opening. Ir is not uploaded.


*imagine a boot stamping on a human face — forever*


Do you want to create a PR with that change?


NOthing huhu~~~


Actually it doesn't :(


:+1: plus it broke everything.


put here for debugging reasons, not needed anymore


It is the revert of the pinning for the release. Also removing postgresql and monitoring here was not a smart move.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


`.axis.get()` выглядит странно.
Почему просто не завести метод `getAxis()` в классе `ChartSlider`? То же касается `.position.get()`


Мне кажется, более принятым стилем в джаве будет использовать `horizontalSlider` в качестве имени параметра, а чтобы не спутать его с полем, писать у поля `this.horizontalSlider`, т.е. это будет выглядеть вот так:
```java
public final void setSliders(Slider horizontalSlider,
		Slider verticalSlider) {
	this.horizontalSlider = new ChartSlider(horizontalSlider, new Axis() {
		@Override
		public ValueAxis get() {
			return getChart().getXYPlot().getDomainAxis();
		} ...
```
На мой взгляд это также лучше читается.


(Sorry, didn't see this reply before.)

The most likely reason for singing a test assembly is so that you can trust it with InternalsVisibleTo in the production code. I've lost where we've got to with friend assemblies, but it can certainly be very helpful. It means you may need conditional compilation, however - see https://github.com/nodatime/nodatime/blob/master/src/NodaTime/Properties/AssemblyInfo.cs for an example of this.


Yes. It might be worth doing that in a different PR though - maybe get this one in, and then have a "tidy up" PR.


@perry HI PERRY :) Could I ask why `overflow: hidden` was added on the body and footer please?


Okay, I think I'm happy with this if there's a follow-up PR to sanitize the project configurations overall - in my experience it can be somewhat painful to do, but the benefits are worthwhile in the longer term.


"dst" should be named "src"


I realize that, but I still prefer an explicit check for Windows version. The Create*Ex API is conditioned on Windows version, so it is straightforward to check it on the API use as well.


hm okay, shall I revert it?


done


I've added some inline docs to this in a new commit (ad6149c).


This change does not belong to this commit; but to the one that adds "iio_buffer_set_blocking_mode".


Do we also need to [reload exim](http://alaveteli.org/docs/installing/email/#making-the-changes-live-in-exim)? If so will also need to update https://github.com/mysociety/alaveteli/wiki/Migrating-an-existing-Alaveteli-site-from-ruby-1.8.7#exim


As you can see in https://github.com/xwiki/xwiki-platform/commit/dc5fb3c230c68394efcbe3a6968a272db618600d#diff-9c8228109f081a40ad73bfd95c485902R6174 it was not removed.


Sorry, do you still need this testing? Was a little busy yesterday


Evet sıkıntı olmaz, çalışır her türlü.


Think-idea พัฒนาโดย Django Framework python language


Вот и мне не нравится. Если удалить метод ```update()```, то можно обойтись без ```axis().get()```


This will crash by trying to free dev->pdata->blocks and dev->pdata->addrs in local_free_pdata( ).
Either revert to free(dev->pdata), or add null-pointer checks in local_free_pdata( ).


The example is a bit hard to follow. Could it be possible to have a bit of the html on which the jQuery selection operates? Also probably better to put the actual output of the `console.log` statement instead. Unless it is the `contentTypes` object below. In which case the console statement could probably be removed.


Could you fix the indentation?


А внутри ```new SelectionListener() {``` что писать ?


Please do not rip off authorship:
https://github.com/neobuddy89/hammerhead/commit/99a71a15abf8745aaa6f83580a1aeb33f84b49ce


Look more closely and you'll see the doc was jumping from step2 to step4... and step3 doc was removed in your commit...


This line should be removed, I think.


duplicate code with upper line


:heart:


kernel size -> kernel side


Moved functionality to ContainerManager. Now I'm taking into account both the nodeNameList and the rackNameList.
The relaxed locality can be achieve by adding /* at the end of the rack list. If you add it before, then it will take precedence, and will try to allocate the containers anywhere (like ACLs).
I'm also normalizing the rack names (similar to what hadoop does). A rack name should start with "/", do not end with "/" neither start symbol.
When the user specifies the rack name list to run evaluators on, he can add a star at the end, e.g. /dc2/star (don't know the markup for the star symbol), then we will look for available containers in datacenter2.
I've also added and improved the UTs, I'm now capturing the argument.

I don't know why the build failed, I've it working on my local environment. Can you take a look @markusweimer ? If there is no problem, then the code is ready to review.


i have add this because of the like/dislike dropdown menu where the long channel name messed things up... allthough not badly... i did not see any regressions with default font size...


another


Awesome good start!


another on the right


valid fd -> a valid file descriptor


There is no fun here. I prefer to manually merge PR (and I want to preserve the author of the PR). I would like more cleaner commits than "merge pull request on some branch which does no longer exist anyway".
In the new code will have something like (same behavior than std::cout):

    logs.info() << "some message";
    logs.error() << "failed to retrieve some value";


Maybe name the private backend function just "get_fd", I will re-use it later for other purposes (see my recently pushed zero-copy branch).


I'm not sure that separating cmake files in `toolchain` and `target` is useful in you case (in the `modules-cmake` branch, the CMakeLists.txt had to include both files).
I would let `atmega2560.cmake` in `toolchain`. Or at least, if you want to separate it (since ther will be a lot of files), you can put it in a directory `toolchain/avr`. The same could be done for sasiae and stm32.


So on my code, for this problem, shouldn't be taken into account changes on lines : 264 to 617, and on the line 2198 (equivalent to 1974 on old code).


Yes it can have as many dashboard it wants only with different names (which is the id field).


In the end, this has been fixed by 130b22ff74d33c94c5055031ffbe34401fea2ac1


Why inline every function? Especially virtual ones? Let the compiler handle that for you, it is smart-enough for this.


I think you can just return iio_device_set_blocking_mode() here.
The high-speed devices can also be used in non-blocking mode.


Good point. Let's change this to ```or```


We can create an issue "[roadmap] logging facility". And maybe a dedicated label for this would be good.


Again look at the link I gave you, it's an exact copy/paste of the step 3 doc with its related code.


Maybe add a note to say that it's blocking by default.


apply--> load


You are missing the copyrights and license here!

Add them to your name.


Didn't check for a plugin (in fact, I think it's preferable to use a plugin so that we don't need to touch core in case a API changes).
Should'nt there be a wiki place where we can promote our plugins so one can find them easely?


요거 차이가 뭐죠?


This comment should be before the function, not after.


TeamCity Ybackend :: synteny/ybackend [Build 56](http://192.168.30.225:8111/viewLog.html?buildId=1477&buildTypeId=Ybackend_SyntenyYbackend&guest=1) outcome was **FAILURE**
Summary: Tests passed: 16; exit code 1 Build time: 00:27:22


I'm learning hyper implementation and see your code, I never saw code write like this

```rust
pub struct StrictTransportSecurity(pub bool, pub u64);
```
can you point me the meaning of `pub` in `pub bool` and `pub u64`? thanks?


You've broken identifier completion man.

e.g after your commit

```
uint azert;
void main()
{
    az // no completion.
}
```

previously this was working fine, got `azert` when invoking completion after `az`.


Как насчет абстрактного класса с двумя абстрактными методами `getAxis()` и `getPosition()`?


are you missing an `end`?


Thomas you just broke the flow of the doc (missing "step3"), that's all, no need to become upset about it, it's just a fact and I don't think you're a bad developer because of it! :) It's a no brainer, just took me 10 seconds to fix and 10 minutes to reply to your comments :)... Peace!


@adamgoose Thank you for informing this. I forgot to delete this exit calls.


Should use whitespace instead of tabs! :( (Even tho tabs are the best)


@frankheinen you should have provided this as a proposal in your personal repository. I don't see a necessity to change our current implementation including what we proposed for RTKLIB as your changes don't seem to provide any benefit.

The changes you did are rather hard to follow. You should have kept the diff easy to read. The whitespace changed over the whole file, e.g. you removed spaces after punctuation.

 * With your reasoning regarding the subframe I don't agree. As you mentioned at first, both should be allowed
 * For now we should rather allow more than less, e.g. keep the time status options
 * pseudorange is definitely a single word, e.g. see https://en.wikipedia.org/wiki/Pseudorange
 * "L2CA" does not exist. Do you mean L2C? see https://en.wikipedia.org/wiki/GPS_signals#L2C Also the other signal definitions don't seem to follow a clear scheme. Also see comment by @sjko on this


This frame load image file in same origin.
This frame load image file in different origin.


Can't believe I'm still here... But—one more byte off:

```html
<body onload=d=Date.now,t=d(s=0)><p onclick=(e=d(++s)-t)<15e3?style.margin=e%300:alert(s)>X
```


Да, это проблема, не заметил.
Хотя вроде можно `ChartSlider.this.slider` (http://stackoverflow.com/a/2808514/4764971), но проверить сейчас не могу.


Before this can be merged I think we are going to have to be sure we have met this requirement.


Add the comments inside of iframe ?


a


b


Is case sensitivity of the keys important here? Probably best to have them all lowercased if not so that we can allow people to be lazy:

```javascript
headers.contentType('followrequest');
headers.contentType('FollowRequest');
```


So for this problem, shouldn't be taken into account changes on the lines 264 to 617, and 2 other single lines in the code:
the line 1836 (equivalent to 1656 on the original version)
the line  2198 (equivalent to line 1974 on original version)


apply media file --> load video file


Тоже думал. Может быть попробую.


Ah, I see you added this ;-)  Next need to ensure this is also added to WildFly Core on the next Elytron update.


Houkou passed away? :(


Sehr gut, das wollte ich auch schon machen. Diese Dateien müssen raus aus der Versionierung


@andresgutierrez var_dump ? :-)


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 331](http://192.168.30.225:8111/viewLog.html?buildId=1479&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


I think drag and drop fails the 'could my mom do it?' test, there should definitely be some buttons.


Ну т.е.
- или использовать имя атрибута
- или очень длинная запись (как ты предложил)
- или использовать horizontalSlider_ (т.е. имён параметров) - а вот этого не хочется. Хочется их запомнить и забыть про них.

Сейчас выбрано первое, под него переименованы параметры метода.


Could you handle the case where (ret == 0) (end of file)? And return -EIO. Same in write_all( ).


> There is no fun here.

Haha nope, didn't mean that in the *laughing* sense. [English is a funny language](https://en.m.wiktionary.org/wiki/funny). *Pun not intended*.

> In the new code we can have something like (same behavior than std::cout)

Maybe. You might want to look at [Chromium's logging infrastructure](https://code.google.com/p/chromium/codesearch#chromium/src/base/logging.h&sq=package:chromium&type=cs&l=49), which I think is really neat.

There are a few downsides (vs `OutputDebugStringA`), related to `iostream` and such, but this isn't the place for a full discussion :)

> We can create an issue "[roadmap] logging facility". And maybe a dedicated label for this would be good.

Absolutely. I like the idea.


Issue created with a note to do this https://issues.jboss.org/browse/WFCORE-782


in der Tat. Aber es ist momentan noch nicht eingestellt, dass „Beta-Version“ der default-Zweig ist. Ich glaube, das kannst nur du unter den Einstellungen, rechts unterhalb des „Graphs“  machen.

Am 30.06.2015 um 10:48 schrieb Johannes <notifications@github.com>:

Sehr gut, das wollte ich auch schon machen. Diese Dateien müssen raus aus der Versionierung

—
Reply to this email directly or view it on GitHub <https://github.com/giovannotti/biblatex-archaeologie/commit/c57aea383cbad4bc597e4208817be40ed877c08d#commitcomment-11926797>.


Have you considered chaining the calls?

```javascript
function headers() {
    if ( !(this instanceof headers) ) {
        return new headers();
    }
    this._headers = {};
}

headers.prototype.accept = function (contenttype) {
    this._headers['Accept'] = contentTypes[contenttype] || contenttype || '*/*';
    return this;
};

headers().accept('file').contentType('document');
```

Only if that is useful of course.


You can probably just do `var url = estURL + '?api_key=' + apiKey + (search ? '' : '&q=' + search);`


It's not the end of the world that this is in core, although I do think we need to tidy up the javascript libraries.

Re wiki: probably. I think @benwerd was talking about a community site some time ago, but I think like all things, there are only so many hours in the day :)


Я вообще за "длинную" запись (хотя  `ChartSlider.this` vs `horizontalSlider` столько же символов), но мне не принципиально. Оставьте что вам кажется более читаемым.


I'm going to revert this so trunk starts as it also means that people
creating pull requests will get failed builds as they will be based on a
build that didn't work to start with.

On 30 June 2015 at 07:38, Juanjo Meroño <notifications@github.com> wrote:

> The build is broken due to this commit.
> https://api.travis-ci.org/jobs/68925989/log.txt?deansi=true
> This is a good reason to always make a PR, let travis check if everything
> works before apply changes into master. [image: :smile:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sakaiproject/sakai/commit/210b286306cff0952f485b741a7335cec5758842#commitcomment-11925030>
> .
>

--
  Matthew Buckett


Hi, @iemejia.

On Jun 29 2015, Ismael Mejia wrote:
> Ok, please do the fixes you can from your comments (including if you can
> the renaming of video_youtube_url to youtube_url).

OK, I will commit some changes directly to the master branch while you are
away.

> and I will attack the missing bugs (e.g. repeated downloads, and the two
> new ones who are reported).

Thanks. I will clearly forget to fix some of the things that I put in my
comments and I'm sure you will catch them.

Thanks once again,

--
Rogério Brito : rbrito@{ime.usp.br,gmail.com} : GPG key 4096R/BCFCAAAA
http://cynic.cc/blog/ : github.com/rbrito : profiles.google.com/rbrito
DebianQA: http://qa.debian.org/developer.php?login=rbrito%40ime.usp.br


Always take your whitespace serious, man ;)


Le 29/06/2015 17:33, jsanda a écrit :
> I am still trying to sort this out. Currently, there is no requirement
> to explicitly create the metric or the tenant. If we keep it this way, I
> see a couple of options.
>
> First, we could provide an endpoint to update the metric, which would
> allow you to inform the server that you want rates to be computed. This
> of course is not all that different than explicitly creating the metric
> up front; however, it does provide some additional flexibility.

We need an update metric endpoint if we want to, for example, let users
change retention policy for a specific metric. Or does such thing
already exist?

>
> Another option is to handle it on the server side. We have a job that
> runs periodically runs to determine what metrics are in the system. The
> job would see that your counter does not have rates generated for it
> since the metric was not explicitly created. The job at that point could
> schedule the task to generate the rates.

+1

That would be very convenient. Let's say I'm deploying a new application
on a new machine. I'll setup collectd and JMXTrans. It's painful if I
need to write an update request for every single counter collected.


Тогда уж надо сравнивать
```ChartFrame.this.horizontalSlider.slider``` vs ```horizontalSlider```


Hi, I am sure kerberos works well and I can use command line access hdfs. At the same time I ensure krbv lib works well. Then I set a break point in Class SaslRpcClient with file 'rpc_sasl.py' , and when code "ret, chosen_mech, initial_response = self.sasl.start(s_mechs)" is executed(line 100),  I will get "chosen_mech" with empty string(''), is my sasl python lib have something wrong ?
PS: I have set chosen_mech="GSSAPI" with hard code but to get another error:
javax.security.sasl.SaslException: GSS initiate failed.
Thanks for your attention.

"


@Maximus5 В английском, слово "[hooker](https://slovari.yandex.ru/hookers/%D0%BF%D0%B5%D1%80%D0%B5%D0%B2%D0%BE%D0%B4)" имеет жаргонное значение.


Test comment


ㅋㅋㅋㅋㅋ


De ce este nevoie de metode?


Yes, in general that's good practice, but in this case I would prefer to not check against the target windows version, because you get a much more detailed compile time error instead of a link time error when the linker fails to lookup `CreateEventA`.
But how about adding the following to `boost/detail/winapi/config.hpp`:
```
#include <boost/predef/plat.h>
#if BOOST_PLAT_WINDOWS_RUNTIME_AVAILABLE && BOOST_USE_WINAPI_VERSION < BOOST_WINAPI_VERSION_WIN8
#ifdef _MSC_VER
#pragma message("warning: targetting WinRT and a WINAPI version less than WIN8, is this intended?")
#else
#warning "warning: targetting WinRT and a WINAPI version less than WIN8, is this intended?"
#endif
#endif
```


Am inteles. Ca sa poti edita userul din contestant. Nu?


Messages


E nevoie in rails admin. Ca sa pot afisa un contestant cu tot cu nume si prenume


If you have a problem with this id, update your device plugin :

```
cordova plugin rm org.apache.cordova.device
cordova plugin add cordova-plugin-device
```

See this article for more details : http://cordova.apache.org/announcements/2015/04/21/plugins-release-and-move-to-npm.html


Just no


This won't work with cyclic mode on low-speed devices, as the samples must be written *before* the buffer is enabled. I would suggest to just make local_set_blocking_mode( ) return -EPERM if pdata->cyclic is true, thus forbidding non-block and cyclic modes to be used at the same time (which wouldn't make much sense anyway).


typo it should be updated to version 0.2.8.


You should configure your editor to automatically add new lines at the end of files, that's a convention most people use (or at least, that's what we do in the team).


typo it should be release to version 0.2.8.


Почему `ChartFrame`? `SelectionListener` это же класс, вложенный в `ChartSlider`. Или оказывается, что он вложен в `ChartFrame`? Если так, то действительно проще оставить подчеркивания.


这个改动看不懂


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


Agree. Will refactor.


![screen shot 2015-06-30 at 11 04 19](https://cloud.githubusercontent.com/assets/138906/8427454/d05ba43a-1f17-11e5-9ab9-12e762c08167.png)

Much better!


Take a look at `t.tokens`. That's the intended way of doing this.


A comment here and on the subsequent line would be useful


First Clone