__[API Description file `ertewrtert`](https://app.apiary.dev:9000/ertewrtert/editor) has not been updated in Apiary.io__ due to server timeout.
Please try it later by trying to redeliver the associated [web hook in settings](https://github.com/tully-test6/testXXXxxx/settings/hooks) of this repository. (Autogenerated message from apiary.io)


__[API Description file `aaaaa123`](https://app.apiary.dev:9000/aaaaa123/editor) has not been updated in Apiary.io__ due to server timeout.
Please try it later by trying to redeliver the associated [web hook in settings](https://github.com/tully-test6/testXXXxxx/settings/hooks) of this repository. (Autogenerated message from apiary.io)


__[API Description file `biiiiiig`](https://app.apiary.dev:9000/biiiiiig/editor) has not been updated in Apiary.io__ due to server timeout.
Please try it later by trying to redeliver the associated [web hook in settings](https://github.com/tully-test6/testXXXxxx/settings/hooks) of this repository. (Autogenerated message from apiary.io)


According to the codegen rules this won't work you already have a `session` named method.


dadweasdad


i've upgrade to 0.29, then how can i get those third-party packages to work witch need to pass the `Activity` to the constructor such as `react-native-code-push`


Oops! :)


dasdadada


I think this was the reverse of what the original implementation was trying to achive, no dependency on handlers.


Are you planning to submit this to CM?


Correction: Fixes #1980


haha perfect noticer 😃  👍


code style: no spaces before ";"


Not sure about this, the error is silenced/ignored and a new session is created


@piratadelfuturo https://github.com/sulu/sulu/pull/2505 i have added a PR for that.


Fixes #23


Before only if a new session was created it was stored, now it is always?


Corrected the names of the packages as they were referred to in the staged files.


I don't know of any changes. Either way, you are looking at 2 different things, subtitle_list and subliminal.


cool


Alright. In the newest push there are some strings that are in need of translation also.


Once of your concerns was GC preasure, so before if nothing was added no garbage was being produces but now there is always a map instantiated, is this deliberated?


`Map.new` or `Enum.into(%{})`?


**GREEB**: do you have web site? I can add for your servers description in readme


Fixes #17 and #22


ELC_Storage_Simple (jmarrec) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2368 of 2436 tests passed, 0 test warnings)

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


Black is a find default color.  At some point I would like a user-functor to override these, or perhaps one could:

```
atom.SetProp<DrawColour>("color"/"colour", DrawColour(0.5,0.5,0.5)):
```


ok


You're commenting on an ancient commit here.  As far as I'm aware pretty much all intel graphics chips available within the last few years support 3.2.  The only ones that don't tend to be the really dated ones.

3.2 is 7 years old now.  3 is required due to due to FBOs especially, VAOs and other things are also nice to have.  More texture format support is great.  Shader version 1.5 especially is really nice to have, as the others below are severely limtied and cumbersome to use.  If I could I'd prefer to be on 3.3 though at least due to sampler objects, but they're not really necessary, so 3.2 was the bare minimum to do everything I wanted.

Anyway, again, if your computer is running a GPU that's not capable of using 3.2, doing live capturing/encoding probably isn't going to be something the system will be ideal for.  That almost always tends to be the case from my experience, it's just not worth it.  It's only the third core profile release of GL.  Having to strip myself of available graphics features just because of a few horrible GPUs out there just feels unfair.  I'm not going to remove features from the program or redesign the program for older chips when technology is constantly moving forward rather than backward.


r


this is code from 2014... for 5.0...


This is a broken copy of utils since the pid file is wiped on a time out.


Hmm, to canonicalize a smarts you need to canonicalize the logic as well.  never really thought about that...


Well, you do understand that with this code in place it's just impossible to change the speed limit from outside while the GUI is shown, right?


Or you could have kept it as a comprehension:

    for directive <- state.directives,
        match?(...),
        do: ...,
        into: %{},


I have a doubt. Wasn't there language-specific code there?


Barbosik: We use a custom version of Agarplus i made. We don't have a web client. Our website is [Agarlist.com](http://agarlist.com) where you can install the [extension](http://agarlist.com/agarlist.user.js).

If you only want to add direct links to webclients feel free to use a client.

-Thanks


ELC_Storage_Simple (jmarrec) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Why do you need backlog = 10?
If you avoid reformatting the code, the diff will be a lot smaller.


And we must change the string name settings_confidence_text "Min. Konfidenzintervall für den Alarm" // This was a interpretation error from my side.

When I better understand the software, coming soon polish. Thanks


I'd explicitly _avoided_ the use of `os.path.join()` here, since MSYS2 can handle Unix path separators just fine. In fact, I think allowing backslashes in there is probably going to cause trouble in some cases as they might get interpreted funny by the shell...


The above fix replaces two consecutive double quotes, which is not the issue in this case, but the issue is any single occurrence of a double quote. Here is the message that gets packed into the reason object:

     out loop, peer reset, message [notification sent (1,0)] error[Message header error / Unspecific / can not decode update message of type "2"]

Try this instead:
 +		def escape_quote (reason):
+			return reason.replace('"','\\"')

To avoid escape characters entirely, this also works, but is not as proper:
 +		def escape_quote (reason):
+			return reason.replace('"',"'")
it just replaces and double quote instance with a single quote.


@m-kuhn
It looks good to me. I tested it in postgis and it works well.
WFS-T is completely broken anyway (except for INSERT), working on that with @rouault

I wonder why the original implementation was iterating backward on remote features ids ...


Hello, there's a `:h manpager` built into vim. It's a simple plugin by me that uses the `manpage.vim` plugin.


Fair enough. (Though cases exist where the chipset is borderline "just close enough" and can run the encoder at decent speed, and has FBO support via extensions - but I see your point on that the device will probably be too slow to encode, say, 3D games. Under such a particular case it's probably worth using LIBGL_ALWAYS_SOFTWARE anyway.)


Magic numbers? Why is it 100


Ok thanks, later I change it.

BR


В каждом тесте у тебя создаются эти переменные. Нельзя в статику вынести?


nice :)


The resulting value needs to be stored in the **correct position of a result array**.
Make sure you take the situation where the input array **is empty** (and thus no promise will ever resolve) into account. (_all([])_)


we should use `Object.keys(jsonObject)` to extract keys.
`var $ref=Object.keys(jsonObject);//returns all the keys of jsonObject in an array`


is this is working yes, i never saw this working on old OC servers, but maybe i even didn't noticed it.:D


Just tested, and was as I suspected: `os.path.join()` on MSYS2 yields Unix separators (and : rather than ; for `os.pathsep`).


+ AtmosphereAutopilot.Instance.mainMenuGUIUpdate();

Every time a control state is changed a GUI state should be updated accordingly. At least for hotkeys.


666 这里我没去抓log 直接一个else哈哈


А зачем тебе эти переменные? Не вижу использования


This one is not finished yet, I will talk to Jaime and Ruben in a minute

On 24 June 2016 at 13:33, Tim Kuipers <notifications@github.com> wrote:

> code style: no spaces before ";"
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/Ultimaker/CuraEngine/commit/4e246b8b29424cdcae6ef634a556a2a3fe97f6e8#commitcomment-18002197>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/ATBjmaV2-skPkI2uUS80kJEVDNV-cJFQks5qO8CfgaJpZM4I9sG4>
> .
>


TeamCity OTA+ :: rvi_sota_server :: migrations container [Build 5](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5758&buildTypeId=ota_plus_RviSotaServer_MigrationsContainer) is now running


А wrapper где-нибудь используется?


ok


TeamCity OTA+ :: rvi_sota_server :: migrations container [Build 5](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5758&buildTypeId=ota_plus_RviSotaServer_MigrationsContainer) outcome was **SUCCESS**
Summary: Running Build time: 00:01:34


А нельзя решарперу сказать что вот так я в тестах пишу? Не гуглила?


I guess it will become more obvious once the translations are visible in the app


Remove unneeded variable assignment so it passes static code analysis...should have put a commit message ;(


Не понимаю логику. Dinner.Count == breakfast.Count? Как так? Пояснение нужно


Не вижу полностью файл. Убедись что в комментарии у тебя есть упоминание что вернуть может null


@qus1225 http://owlcarousel2.github.io/OwlCarousel2/docs/api-options.html 1개나 2개일 때는 그냥 loop 옵션을 false로 두면 어떨까요?


Integration test for XMLParser.
Build success on cluster(104.155.156.158) and standalone:
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 4:08.545s
[INFO] Finished at: Fri Jun 24 17:29:37 IST 2016
[INFO] Final Memory: 79M/333M
[INFO] ------------------------------------------------------------------------


Fix_ELCD_idd (jmarrec) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2259 of 2259 tests passed, 0 test warnings)

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


hid is the handler id of 'row-activated'. [[1]](https://goo.gl/Ov7H8k)
Hence when we click on a row 'row-activated' is emitted and the handler in list_view is called first launching the activity!
So now I disconnected the signal and reconnected it to different handler here.
I would be better you test the latest code.


see https://github.com/sapegin/react-styleguidist/pull/152


Yes, that's true. I test then.


Fix_ELCD_idd (jmarrec) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


This made me laugh and cry at the same time.


Pourquoi ne pas faire
```php
$billet['titre'] = htmlspecialchars($billet['titre']);
```
?


Faudrait l'indenter correctement :-)


Uporabi jQuery selector $ (dollar) [1].

[1] http://www.jquery.com


This commit contains some awkward code, especially:

- why was `library/Erfurt/Sparql/Query2/TriplesNode.php` remove
- what is happening in `library/Erfurt/Sparql/Parser.php` around line 227
- what is happening in `tests/unit/Erfurt/Sparql/ParserTest.php` from line 32 on


you can already do roughly that with the highlighting code


Можно. Сделаю


being able to select different color schemes at the atomic number level (i.e. "all carbons are hot pink") by changing the default colour dictionary would be a nice option to add at some point though.


yes, it's just a note for others who also might go down that road. utf8 names are supported since 5.8.4, with negative HEK_LEN. but not properly for this module.
that's why hugmair had to fix it in 5.16
(and then went overboard and added binary names)


Пропустила, спасибо


ELC_Storage_Simple (jmarrec) - x86_64-MacOS-10.10-clang: OK (2388 of 2436 tests passed, 0 test warnings)

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


@leletec @dieechtenilente I did really manage to call it "Meeting on Montag". This depresses me.


В строке №100 передается в параметре конструктора


Not yet)


Следы копипаста T_T спасибо


First release


Мне вообще надо определиться с правилом, что тут везде возвращать.
Задачку поставила себе


ew JsTasks.task1.UI.PlainJsToDoList(parentElement, elementFactory, ['todoList']);


meninos, o screencast ficou mto bom, mas uma coisa q me intrigou. os dados do prof vinicius, assim cmo os de vcs (email e siape) nao deveria estar oculto?
^^


Remove the 'afterImport' also... right?


This won't likely give any performance boost, since TeaVM supports only green threads (which run on one physical core).


I'd call it `available` (no "is" prefix)


@modus75 you're perfectly right when trying to have less "update" SQL queries. The problem with the action of "log in", is that you may not do it very often, if you have the "remember me" option. If we can trigger the "session" creation, it would be perfect (I'm sure we can, certainly with function auto_login and trigger login_success :-)


Where did you get this and why is it needed?


`supported`


![draw](https://cloud.githubusercontent.com/assets/5659459/16337747/054db522-3a18-11e6-95b9-c1fa65bab691.jpg)

:D


why you need load_to() if there is no to zone?


Thanks! Looks really good :+1:


May i know the reason behind using rating id 1 over here?


120694307_Fault_ChillerSWTSensorOffset (rongpengzhang) - Win64-Windows-7-VisualStudio-12: OK (2210 of 2213 tests passed, 554 test warnings)

 * 557 tests had: AUD diffs.
 * 4 tests had: EIO diffs.
 * 3 tests had: ESO big diffs.
 * 1 test had: MTR small diffs.

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


oh


i tried to modify Block.get() method and if some block doesn't exist it replaces it with an update block. but still crashing...


If you do something like that you would also make sure that we don't have a hidden fifth housenumber:
```
assert postcode.housenumbers == [hn1, hn2, hn2bis, hn2ter]
```


Collaborator:
A review for this commit was created. [Link to review](https://localhost:8443/ui#review:id=452)


Fix_ELCD_idd (jmarrec) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1109 of 1109 tests passed, 0 test warnings)

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


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

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


That'd be great!


this file defines a zero rotation of the joint "tool_tip_joint_kuka", which would align it with the flange.
The iiwa_tool.URDF however has a rotation of pi around 'y' defined for this final joint.
I assumed the urdf was generated by the former, so how this discrepancy?
I assume the zero rotation is the correct one and that the tool_link_ee is the one that is offset by -90 degrees to accommodate moveIt


I always hate it when the variables are present at the right side of the values. ``0 == (_Imm_ & 1)``

something like  ``(_Imm_ & 1) == 0`` (or) ``!(_Imm_ & 1)`` would have been much better (or) maybe it's just my personal preference. dunno.


Ho, you find a way to send message. Great ! I'm on a pymumble bot, and this is the nextfeature I want to implement, but if you have already done it, that's good. One question, your fork is too away from my fork to implement with a pull your changes. Are you agree to add it on my fork ? (or on the Xefir one, we work together).


Can you please provide the post parameters to submit a review. I tried but 'Customer id' and 'Rating votes' are not getting saved :(


Hi @yurikuzn , First I would like to thank you for this.
You forgot the Drop to attach on the Documents attachment.


decided set it like in vanilla?))


Nice!


TeamCity OTA+ :: rvi_sota_server :: ci [Build 245](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5782&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


I agree, that does look a bit horrid.


@stephanesan @rabardini @OICAdmin is the change of dmv property in oic.wk.d valid?
It's supprising change for me. What is the reason and where can I found information about this change? From lastest TestSpec (v0.15) and other sources I did not receive any information about such change.


@lge You may want to do the same in Heartbeat as indicated in the commit message.


Just wondering, does Google Maps allow SVG markers as well? Vector-based images would be a huge improvement on HiDPI screens instead of a blurry bitmap ;-)


@mkendall07 @benjaminclot I see mention of adding the reserve parameter, but it didn't make it into this PR. Is this something we could get added?


change url


nit: Rename to `protocolVersion` in order to avoid ambiguity.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 245](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5782&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 202, ignored: 3 Build time: 00:05:51


any update on this? debugger is still present :(


Вот это
```
Окружение.Получить("USERPROFILE")
``` надо вообще убрать, такой переменной нет. А путь "~" как раз и указывает на домашний каталог.


Closes #3


wo kommen die immer her? oO


Just to clarify (because the diff shows a lot of changes when actually the change was very simple): the change I did was inside the try:except block. I've just moved this lines:

```
if f is None:
    # delete and never look back
    self.r_server.delete(newKey)
```

Those lines were already in the code. I moved them to the top of the try:except block, so the first thing the code does is to check if f is None it deletes the key; else, the code does what it was already doing, nothing changed there.


typo: should be “a hyphen”


After the unmarshalling of p, its field `protoVer` should be filled (correct me if I am wrong). It makes more sense to use the information from there.


Hi Igor. Is it possible to initialize funcaddrs and last_elem inside the constructor?


tolia


//hello


last_elem is uint32_t. Return uint64_t is what you wish here?


nit: for me a better way to do this would be
``` go
    if data[0] == ProtocolVersion1 || data[0] == ProtocolVersion2 {
      p.ProtoVer = data[0]
    } else {
      return ErrInvalidProtocolVersion
    }
```


wait even long ones?

On Fri, Jun 24, 2016 at 3:43 AM, Alexandru Simandi <notifications@github.com
> wrote:

> i just fixed it now, it can copy any phrase you say, hold on commiting and
> pushing now
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/AdminOdd/POISamaritan/commit/2ce5e0b2b89c053b362e62eb7124312bbca0d52e#commitcomment-18000076>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/ASemlRBzq-KhsG2fQozAuYC7NJ2b15vsks5qO4qogaJpZM4I9iPP>
> .
>


Wouldn't it be the easiest to use https://sapui5.hana.ondemand.com/explored.html#/sample/sap.ui.vbm.sample.GeoMapSpots/preview ?


It already works if you add it to the params:
```{
    bidder: 'appnexus',
    params: {
        placementId: '123',
        reserve : '2.00'
    }
}```


Yep, it seems that's the approach taken by https://github.com/mattn/go-pointer as well.


A new test will get SIL generator to hit emitWildcardDispatch() on a catch statement


Why do you give it as a parameter? You could have put it in the `PushACKPacket` which seems as a good place to store it (at least to me). Now the type PushACKPacket no longer implements [BinaryMarshaler](https://golang.org/pkg/encoding/#BinaryMarshaler). If there is no specific reason to do so, I would rather have the parameter inside `PushACKPacket`.


啧啧


Gah. Thanks for spotting that. This file wasn't supposed to get checked in. The version in RaptureCore is correct; trying to call this from ReflexTest did not work due to the build dependencies.


nit: Rename to `ProtocolVersion` (see above why).


just try it, i just said "_i like potatoes in the backyard_" and it worked


TeamCity Xbim :: Xbim Web UI :: Xbim Web UI [Build 53](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1087&buildTypeId=Xbim_XbimWebUi_XbimWebUi) is now running


nit: reformat as suggested above.


TeamCity Xbim :: Xbim Web UI :: Xbim Web UI [Build 53](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1087&buildTypeId=Xbim_XbimWebUi_XbimWebUi) outcome was **SUCCESS**
Summary: Running Build time: 00:00:28


TeamCity Xbim :: Xbim Web UI :: Xbim Web UI [Build 54](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1088&buildTypeId=Xbim_XbimWebUi_XbimWebUi) is now running


<< 2 is * 4.


TeamCity Xbim :: Xbim Web UI :: Xbim Web UI [Build 54](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1088&buildTypeId=Xbim_XbimWebUi_XbimWebUi) outcome was **SUCCESS**
Summary: Running Build time: 00:00:17


TeamCity Xbim :: Xbim Web UI :: Xbim Web UI [Build 55](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1089&buildTypeId=Xbim_XbimWebUi_XbimWebUi) is now running


TeamCity Xbim :: Xbim Web UI :: Xbim Web UI [Build 55](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=1089&buildTypeId=Xbim_XbimWebUi_XbimWebUi) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:09


It is not a constructor.


Consider moving `protoVer` inside `PullACKPacket` (see above why).


JSONValue.parse is called with `new FileReader(...)`. This leads to problems with non-closed files after API is done installing the version. The FileReader needs to be closed after JSONValue.parse(...) finishes. This issue appears on more spots, so I'll just refer to it as the issue with JSONValue.parse.
For more info, see #27.


This is most likely wrong, see #60 . A factor `(2 q_0^2 - 1)` in front of`\identity_{3\times 3}` should fix it.


JSONValue.parse(...) needs to be fixed


I totally missed the point that PBf has no TO at all .... I am going to fix
that and a few other things then

On Fri, Jun 24, 2016 at 2:29 PM, Sven Waschkut <notifications@github.com>
wrote:

> why you need load_to() if there is no to zone?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/cpainchaud/pan-configurator/commit/e391c99ba03c13aa87131d6a2413bd0d63c51fc9#commitcomment-18002794>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AGYuvl7XyVaahviFkPo3s9AAOSM_LQulks5qO82lgaJpZM4I9uQf>
> .
>


I'll care about this in feature/cleanCommit3cbedbf


You should better use === check and 'if' with block.


Ah, wait. You have that already after the next equality. So you've addressed  #60 already here?


Can be simplified using *=


unused arg e


Please take a look at this commit to make sure everything was brought over from the main.css that should have been. I maybe have accidentally left some stuff out, because it overlapped. Like the nav and body styling as well as the html styling. We can copy from here and add back in if necessary. #19


i'd put this one in the bottom with other 'private' functions


I just tried to follow these instructions and cloning the repository and deployment failed with this error:

Binding service text-to-speech-service to app image-analysis-acflee-142 in org adrian_lee@uk.ibm.com / space dev as adrian_lee@uk.ibm.com...
OK
FAILED
Could not find service visual-recognition-service to bind to image-analysis-acflee-142

Finished: FAILED


Fix_ELCD_idd (jmarrec) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1668 of 1668 tests passed, 0 test warnings)

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


There is no NAND flash on DART-MX6.


WTf it only revert 1/3 of it?????


update readme.md


Arg, this has been solved already. Only somebody forgot to close the issue. Sorry, never mind.


Fix_ELCD_idd (jmarrec) - i386-Windows-7-VisualStudio-12: OK (2225 of 2225 tests passed, 0 test warnings)

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


attach: function($anchor) ?


Perfect. Thanks @mkendall07


Does it really return promise?


Hmm I think this was missing a <stdint.h>


Do we need to set both `permissions` and `rights`?


remove


It's bad to OR when you should have ANDed like hitting the gas instead of the brakes!


No need to run the apt-get update 2x in the Dockerfile ( also in line 8 )
Every line is cached from top to bottom, so it makes sense to add new stuff at the bottom to save time on rebuilds :dancers:


`Map.new` isn't available here, I'd prefer current code to comprehension. :bowtie:


var o;


Apart from the small code changes that I have suggested, I think it will be nice to have tests that check

1. When packets of different type and protocol version 1 are received, that the responses have the same version.
2. That even if a packet v1 was received and afterwards a packet v2 is received, the response will still have a correct protocol version (i.e. version2). I guess this might not be very important, but it is something that I expect to fail with the current implementation.


your indentation is wrong


**FIRMWARE_DIRS** -- This parameter defines the list of directories with firmware and microcode.


I think this code was ok before. Line can be:
`https://example.com/aaa/bbb/ccc`
and whitelist
```
https://example.com
//other.domain.com
```
It should search for whitelisted line in url not opposite otherwise we have to put all urls into whitelist instead on domain only.

The problem is that tests are not correct. See my other comments.


```
    it('should find the line and return true', function() {

        var list = "//example.com\n//test.com";

        expect(contains('http://example.com/path/to/page', list)).to.equal(true);
    });

    it('should not find the line and it should return false', function() {

        var list = "//example.com\n//test.com";

        expect(contains('http://example2.com/path/to/page'', list)).to.equal(false);
    });
```
If '.app2' could be URL still it should match with .app whitelist entry as it contains whitelist term.


Meshes can be weighted or not weighted. Should use VertexAttachment#ComputeWorldVertices.


I can always add more documentation for a thorough understanding, but when first encountering something new, it's best to keep things as closely as possible to a simple "On" switch.


corrijo: la actualizacion es  Configuration.LazyLoadingEnabled = false; Disculpen jeje


May as well add this, it's pretty conventional, and this way outside users of the code have a sense of how they can use the code.


@nihgwu Those packages need to be updated so that they don't need the activity to be passed in the constructor. The correct way is to extend `ReactContextBaseJavaModule` and use the `getCurrentActivity()` method.


Don't worry about doing things right, first thing.  Instead, just get some work out there to play with.  Important is the key word "play".  This ought to be fun.

Throw some concepts out there and see what they get you.


@ephemer: Sorry I missed the last comment. This should be fixed in https://github.com/meteor/meteor/commit/238623ca74873a4882d332d9bbafb9d539a64754.


Sorted is only needed internally, can hide it.


Should that say "TCP stack"?


Might have to update the script that the cron is running. Does anything depend on this, or is it minor issue?


Мы в коде обычно юзаем след.код
`ЭтоWindows = Найти(НРег(СистемнаяИнформация.ВерсияОС), "windows") > 0;`
и сверяем на Истину (Вин), Ложь (Linux)

Предлагаю тебе аналогично поправить.


Oh, nice to see `scss` 💃


Hey @lefarmer, as far as I see we're using this branch in veeqo for awhile now, can we merge or is there anything missed, problems, etc.?


Field for internal sorting.


You can find my PR [here](https://github.com/Tivix/django-rest-auth/pull/182) and there is the corresponding [source code line](https://github.com/Tivix/django-rest-auth/blob/334a29c4d976d826d8d19123611e6184656016e9/rest_auth/registration/app_settings.py#L8).


You could use a module pattern and export just it, instead of multiple exports


Space goes before and after colon (:)! :D


\cc @lexmag


Well, list won't get updated anyways, especially when players are leaving


nice  first good job


new file


TeamCity OTA+ :: rvi_sota_server :: ci [Build 246](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5809&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


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


'const' in ES5? As well as template literals...


Also write a negative test


OnePlus 3!


Was this fixed by https://github.com/i7/kerkerkruip/commit/439faaa866be3f1fe9714b70ca1ff104e3900b49 ?


changes done by shabbir


@martijnwalraven all good, thank you for fixing this!


TeamCity OTA+ :: rvi_sota_server :: ci [Build 247](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5810&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


I like all fields and properties at the top of the class, makes it nice to get an overview of the class and API.


And what if the number will be not natural? Where is a message/exception, or default value?


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


Shouldn't this be "Center" during test time?


database updated too, id fields in tables : friends and messages needed auto_increment;


This repo is for test purposes. I am using it to push graphics and frontend information about my cms.


Oops. I will get it fixed and re-released shortly.


Booo get in property name! :p


@kohsuke it would be seem simpler to just create a lexical variable at top level:

```groovy
String GHE_TOKEN = 'YOUR_TOKEN'
```

or

```groovy
def GHE_TOKEN = 'YOUR_TOKEN'
```

Unfortunately that does not work in Groovy if you want to refer to the variable from within a function definition; you must use a global variable (in `Binding`):

```groovy
GHE_TOKEN = 'YOUR_TOKEN'
```

That is why [this example](https://github.com/jenkinsci/workflow-aggregator-plugin/blob/590d107fd0ef4ddfa59deb3e45685d5aac3831fb/demo/repo/Jenkinsfile#L44) needs to pass `servers` as a function argument, awkwardly (and cf. CJP-3851).


Maybe you're already looking at this Peter, but why did you make the change to the package name? I think that's what's causing Parthi's error. I think it should probably still be called "block_module_add" and that that should be in the $plugin->component entry?

Jock


You're very welcome. I've been able to spare a lot of time thanks to you. :D

On 24 June 2016 at 01:36, Dmitry Marakasov <notifications@github.com> wrote:

> Thanks for credits 👍 :)
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/yoanlcq/FATE/commit/b7dad701ad83f5339f9b8fa271a4981e6529c74d#commitcomment-17996963>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AJiVjO5bnqUO2myAev41fLqa90PzczlAks5qOxh-gaJpZM4I9VtB>
> .
>


Done.


init commit


Anyway this would be better handled by using `withCredentials` (from the Credentials Binding plugin) so that the token can be kept secure.


@julianpost Hmm it looks like perhaps we didn't upload the right code because I dont see any of the controllers or files we were working on last night here....

By the way I found these articles which it looks like might explain how to implement what we are talking about... a method called prepareForSegue can be added to the controller than is being left (the one with the form) which would delegate the work to the other controller (the one that lists the records). I wonder if that player game tutorial you have uses delegate/prepareForSegue in this way...

https://makeapppie.com/2014/07/01/swift-swift-using-segues-and-delegates-in-navigation-controllers-part-1-the-template/
https://makeapppie.com/2014/08/04/the-swift-swift-tutorial-why-do-we-need-delegates/


TeamCity OTA+ :: rvi_sota_server :: ci [Build 248](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5811&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:02:40


TeamCity OTA+ :: rvi_sota_server :: ci [Build 249](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5812&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


@tkelman thanks for the hint. We will look into it.


`$needle` should be escaped using http://php.net/manual/en/function.preg-quote.php.


Not a constructor


What is this?


А если мак?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 247](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5810&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 202, ignored: 3 Build time: 00:08:16


Abigail I wish I understood all this cool javascript. Great job!


Declaring variables should be on the top of the scope


Omggg it's way better then I imaged 😂 but do you have a kik or Skype I can
contact you by?

On Friday, June 24, 2016, Alexandru Simandi <notifications@github.com>
wrote:

> just try it, i just said "*i like potatoes in the backyard*" and it worked
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/AdminOdd/POISamaritan/commit/2ce5e0b2b89c053b362e62eb7124312bbca0d52e#commitcomment-18003080>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/ASemlUOxCtAu0_qyivTXljpQQxNNYxFvks5qO9TqgaJpZM4I9iPP>
> .
>


A space! I found a space!


you see i just modified readme


I have made my config like this.

download-subtitles:
    priority: 31
    template:
      - disable-seen-retry
    subtitle_list:
      list: subtitles
    list_queue:
      - subtitle_list:
          list: subtitles
    subliminal:
      exact_match: yes
      languages:
        - pt
        - en
        - fr
        - es
      providers:
        - opensubtitles
        - addic7ed
        - podnapisi
        - thesubdb
        - tvsubtitles
      single: no


The branch is complete, it fixed the SSL issue we had, and there are no problems that I know of. I believe it can be merged.


Is it possible to check here (or somewhere) if the ImageReader cropped the image and hence the ROIs are not valid anymore? I guess the answer is 'no'


Hello mister
LockScreenItemsSettings.java:44: Settings cannot be resolved
Fix Settings cannot be resolved
src/com/pure/settings/fragments/LockScreenItemsSettings.java
 import android.preference.ListPreference;
 import android.preference.Preference;
 import android.preference.PreferenceScreen;
+import android.provider.Settings;

a greeting


@VladimirMangos I get what you're saying but unless we know just how to fix it then not doing this would leave at least 1 player-producable crash open..


Fixed.


I can't imagine this will decrease efficiency, but just want to point out that this tergmLite approach is motivated by our typical use cases first. And we never use directed, looped, or inverted networks. Bipartite may be good to keep, although when we do use functionally bipartite networks (e.g., males and females) my approach has been to use a one mode network with a nodematch term with a target stat of 0.


write some explanations here?


Fixed by https://github.com/ldaptools/ldaptools/commit/4d04ffcf0eddfd2a513b2297f6886b81b6aa013e. I still have to go through and fix other potential occurrences (hopefully over the weekend). This changed the behavior of an expected query in the specs so it was a little messier to resolve on the spec side. I really need to clean those up.


nit: shouldn't the years be numbers instead of strings?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 249](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5812&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Exit code 2 Build time: 00:06:45


try to fix conflict


const


move this into the if statement?


FWIW, this changed the compiled signature of the `Parse` member; from VB/C#, the old method was seen as a function with the signature `Paket.SemVerInfo Parse(string version)`, and now it's a property that returns a instance of type `Microsoft.FSharp.Core.FSharpFunc<String, Paket.SemVerInfo>`. Doesn't affect F# code at all, but from VB/C# it means a call to `Paket.SemVer.Parse("1.2.3")` now becomes `Paket.SemVer.Parse.Invoke("1.2.3")`.


this.$el.css(position)?


There is a typo in the et8500 plugin line ... double ipk


Ah, may be a case of git spaghetti, gotta love that sort of thing. Really not confusing at all.


Not a constuctor


Acho que vocês substituíram essa maneira de criação por arquivos de seed. É uma boa maneira de ter dados para testar.
Talvez vocês tenham se perguntado se existe também uma boa gem para criação de objetos em testes. Existe a [FactoryGirl](https://github.com/thoughtbot/factory_girl). Dêem uma olhada se puderem.


Already in progress.


Better naming for this class.


修改了yue.jsp


А OneScript поддерживает Мак ? :)


I don't know then. Seems like it works both ways.


Ok that's not optimal, but I don't consider this a public API...


A bit weird that this is inside of setupGeolocation :)


Юмор понял, перепишу ;)
НО! Mono работает под маком?


Ter lógica na view não é uma boa pedida. Numa próxima vez, uma possível boa solução talvez fosse que vocês extraíssem para um método em outra camada que pudesse ser responsável por isso e na view vocês só expusessem o resultado desse método, invocando-o.


刚刚我修改了yue.jsp


code


ELC_Storage_Simple (jmarrec) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2372 of 2436 tests passed, 0 test warnings)

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


Eu não sei exatamente se existe uma maneira de esconder a key do maps para que ela não fique exposta dessa maneira. Pra falar a verdade, eu nem sei se é importante que essa chave seja escondida. Podemos pesquisar melhor.


Переписал, Commit сделан.


If av_buffer_unref in tcp_getaddrinfo_async,then av_freep(&req)@tcp_getaddrinfo_request_free() process before freeaddrinfo(ai)@tcp_open(),this will cause crash.


Boa melhoria, visto que vocês conseguiram uma maneira de criar objetos no banco e buscá-los diretamente usando o ActiveRecord 😀


Oops. My bad.


This is a very old change and we don't use github code review anyhow.  Please take any discussion to the golang-dev mailing list.  Thanks.  (For the purpose of `ByteSliceFromString`, see the doc: https://golang.org/pkg/syscall/#ByteSliceFromString.)


why?


If you're not considering it to be public, is it possible to mark it as such? It is part of Paket.Core's publicly-exposed API at the moment, and I'm using it, so if I shouldn't be using it... :)


Hi @gromero. funcaddrs and last_elem are static variables. I tried to initialize them in constructor, but this would be possible only if they were not static.


Not. I will change this.


ELC_Storage_Simple (jmarrec) - x86_64-MacOS-10.10-clang: Build Failed

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


Usar uma gem para criação de múltiplos objetos é uma estratégia interessante. Faker é uma boa gem para usar num arquivo de seeds. Lembrem de dar uma olhada na FactoryGirl para ver se faz sentido para vocês.


@PortAIOMuse  Menu = MainMenu.AddMenu("Ekko God", "EkkoGod"); Ha ha...  ("Elise God", "EliseGod")


ELC_Storage_Simple (jmarrec) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Mind adding that sound to the authors.txt @ https://github.com/TheThow/OpenClonk-Stuff/blob/master/Knueppeln.c4d.ocd/Sound.ocg/authors.txt ? :)


Only the file PublicAPI.fs is considered as user API ;-)


The SetupParser() function is missing from this change. I don´t see it defined in any of the lua files and my local builds fail because of it.


Oh geez... How do you revert commits again? :P


TeamCity OTA+ :: rvi_sota_server :: ci [Build 250](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5821&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


xDDDDDDDDDDDDD


啊


Sure. Thanks.


OK


Lembram que comentamos sobre nomes claros de variáveis e usar mesmo idioma ao escrever código num projeto? Esse **x** é um pouco confuso de entender. **todasNotas** também.


傻逼


@keithkfng so Imagical dead?


我将echarts.js中18956行中的  if (categoryDimIndex === dimIndex) { 替换为 if (dimensions[dimIndex].type === 'ordinal') {  是可以出现折线，但是只是一条直线，没有显示出数据的变化。我用索引的时候画出的是一条斜线。您有时间可以再测试一下，谢谢！


a lot of it is frankenstein from the nav bar Tom used in class! I am going
to re-work it further to make it our own look and feel. Plus I want to add
a drop down for the authors pages.

On Fri, Jun 24, 2016 at 9:41 AM, Sara <notifications@github.com> wrote:

> Abigail I wish I understood all this cool javascript. Great job!
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/AsaSmith/bsomethingmore/commit/b8c4d350634ea6296a7582a956c189bf9e60e4b1#commitcomment-18003642>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AQWPAPEFdjNaEGaqXBc1oCrMvqAaQ0bAks5qO96BgaJpZM4I9xmf>
> .
>

--
*Abigail Pendergrass*


@qualiture: I think it does support SVG files. Not sure if I have the Mentors logo in that format though. Do you want to give it a try?


A gente chegou a comentar sobre esses sinais de erros de merge, lembram? Acontecem por vários motivos. Então, uma boa maneira de evitar é criar um ritual de conferir os arquivos que foram alterados para ver se eles não possuem esses sinais antes de dar push no código.


Ok, I guess I need to write up my own SemVer parser then? I was using it so that even if Paket was wrong, my code would at least match Paket's behavior.

Since `PublicAPI.fs` is the only part you consider to be public, would it be acceptable for some future version to mark other things as `internal` so that they aren't part of the binary's public API?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 250](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5821&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Exit code 64 (new) Build time: 00:05:12


Lógica de Javascript em código na view não é uma boa ideia. Já que vocês usam JQuery, uma boa ideia seria organizar essas definições e chamadas de funções nos arquivos Javascript.


@nihgwu We're making the necessary changes to the CodePush plugin right now in order to support RN 0.29. We should have a release out by early next week.


@gregorwolf: I usually try to avoid Geomap like plague. It's slow, ugly and buggy. The Google maps control by @jasper07 works just fine. Just needed to carry out a tiny tweak to get its height on 100%


"lease"? I don't think it will be lastest commit xD


You can still go to the unofficial repo ;) .


Use json string here.


We don't want to make stuff internal because it can still be useful for hackers. Like in your case.

That said. Please send a PR which adds public API to PublicAPI.fs and we keep that stable.


As mensagens de commit precisam ser claras. Qualquer pessoa que for ler a mensagem precisa entender o que foi feito. Uma boa dica é tentar usar o valor que foi agregado para o negócio.
Por exemplo, refletir algo como _"Por que nós adicionamos /5? O que isso significa para o negócio?"_ pode ajudar.


Is `declare` really necessary?


Для кода в Линуксе переменные окружения не нужны, а для Винды нужны.


alex_simi_jr on skype, don't laugh i have that account for over 15  years


@jpenninkhof Will look into it tonight. Creating SVG should not be hard (firing up Adobe Illustrator, tracing official logo, export to SVG, get rid of irrelevant SVG paths, commit to GH)


![screen shot 2016-06-24 at 16 10 38](https://cloud.githubusercontent.com/assets/7998310/16340046/411711ee-3a26-11e6-8289-5bf917c3df91.png)


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


@safallia-zyxware This is deprecated now.
Prefer using \Drupal::entityManager()->getStorage($entity_type)->load($id).


I'm not sure I agree with that reasoning, but I'll try to open a PR this afternoon for it.


@safallia-zyxware Same as above.


You can remove `:thread` plugin helper.
`:event_loop` plugin helper use `:thread` plugin helper internaly.


@Qualiture while you're at it, do you think it's possible to draw the lemon on a pin? I think the bottom of the rendered image is pointing at the geo location. The non-pinny image implies that it's center indicates the location. It may also make the pins clearer. Notice that tiny yellow logos don't stand out on the map so well.


Last comma not needed while declaring JSON Elements.
 "data": [
 +       { "manifestUri": manifestUri, "location": "National Library of Wales"}**,**
],


List of changes :
- Moved all sequence functions to separate files
- Concatenate reused code into functions
- Readme clarification
- Tested the interface on STM32F4 and PC again
- Makefile and Makefile.pc update
- Added drone test footage


Не, я имел в виду, что ты можешь изменить задание — так, чтобы фильтр по году надо было вот так определять: `netflix.define_filter(:new_sci_fi) { |year, movie| movie.year > year && ... }` (требовать, чтобы movie всегда был последним аргументом).
Это вполне нормальный подход: поправить требования, если функциональность не теряется и результат тебе кажется разумнее.


Please add phpdoc for all of the functions.


Обрати внимание, что в этом методе два раза повторяется `@filter_store.merge!` и прикинь как лучше.


PHPdoc missing here.


@lostintangent that's great, then i'll disabled it temporarily


PHPdoc missing for the class.


_IOR is defined as this in `sys/ioccom.h`:

`#define _IOR(g,n,t) _IOC(IOC_OUT, (g), (n), sizeof(t))`

IOC_OUT: `#define IOC_OUT (__uint32_t)0x40000000`

_IOR:

`#define _IOC(inout,group,num,len) (inout | ((len & IOCPARM_MASK) << 16) | ((group) << 8) | (num))`

Given that all this does is manipulate an unsigned long (same size as UInt in this case?), what makes this require wrapping? Could it be made that these macros get evaluated at the same time the Swift standard library is compiled? Would that make it easier? This would be a cross platform solution because the standard library would be compiled for each architecture. While this has the chance that the macro _could_ be different on one version of the OS, I think that will not be an issue because the header was last modified on `3/28/94`.

I used a C program that printed out the value of IOCGWINCZ, and it was "1074295912" at compile time after those macros had been run on it. This number worked for ioctl, and gave the desired results.

Correct me if I am wrong, for I have limited knowledge of C.

Would running the macro at the time the standard library is compiled and just exporting the IOC* macros to Swift be an insufficient or not working solution?


@safallia-zyxware This variable is not in list of arguments. Same for the other variable.


Nope. I needed it at the time so I figured I'd make a PR for it. If you don't want/need it you can reject it. I don't mind.


You should use HTTPS because [protocol-relative URL is now an anti-pattern](http://www.paulirish.com/2010/the-protocol-relative-url/).


please add a space before `=>`


ELC_Storage_Simple (jmarrec) - x86_64-MacOS-10.10-clang: Build Failed

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


Yes, there is something wrong.
Empty  NOT NULL MySQL columns is not nonsense.

Consider this:
1. column `x` can contain NULL.
2. after a few inserts you have a few records in the table, and column `x`
may contain NULL in each record.
3. now you decided to modify column `x`, to make it NOT NULL
4. guess what, MySQL will really allow such modification, though column `x`
indeed contains NULLs.

2016-06-23 5:53 GMT+03:00 adanteng <notifications@github.com>:

> When $strValue is empty string and the column in table is set to NOT NULL,
> this change will cause column value in postgresql is null, final can not
> alert column to not null.
>
> Though empty in NOT NULL mysql column is nosence, but migration need deal
> with this problem.
>
> Is there something wrong like said. Or I am miss unstanding.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/AnatolyUss/FromMySqlToPostgreSql/commit/d20f483c57a3c5e0348e8fe5dcac730f3f243ef1#commitcomment-17980866>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AM8ft2W_DcupuaGsGoMaoLoIds_w0tDhks5qOfVHgaJpZM4I8aI4>
> .
>


Did you say you'll be at RH Summit next week?  Hope to see you there.


Maybe you should add `css-loader` as peerDependency.


Fix_ELCD_idd (jmarrec) - Win64-Windows-7-VisualStudio-12: OK (2225 of 2225 tests passed, 0 test warnings)

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


Currently the ko files are still required. Plan is to create a new recipe that builds the drivers during bitbake but not done yet.


Why don't you run this as a tweak ([0.00,4.00]) ?


Don't see these imports being used in the file. Lets remove if we not using them.


ELC_Storage_Simple (jmarrec) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Hi Dimple,

Thanks for using our review plugin. Following are the parameters :

{
    "productId":1,
    "nickname":"Ajay Bhardwaj",
    "title" : "Ajay testing",
    "detail" : "Submitted by Rest api .....",
    "storeId":1,
    "ratingValue": 1,
    "customerId":1
}

you can use our documentation for this :

http://magento22.ipragmatech.com/dist/index.html#!/Review/rest_V1_review_mine_post__post

If you find that this is working for you could you please  review  us on
google
https://www.google.co.in/?q=iPragmatech+Solutions+Pvt.+Ltd.&hl=en&gws_rd=cr&ei=TkNtV9HTM4H1vgSB_Y6gCw#hl=en&q=iPragmatech+Solutions+Pvt.+Ltd.&ludocid=1997149593055534822
?

Feel Free for further queries :)

--

Thanks & Regards

Ajay Kumar | Sr.Software Engineer
iPragmatech Solutions - www.ipragmatech.com
*Email Id *ajay.kumar@ipragmatech.com <noopur.virmani@ipragmatech.com>
Skype ajay.bhardwaj87
Tweet iPragmatech <https://twitter.com/iPragmatech>  | Facebook iPragmatech
<http://www.facebook.com/ipragmatech>
socialfoxhosting.com - Optmized hosting for Socialengine

On Fri, Jun 24, 2016 at 6:07 PM, Dimple Patel <notifications@github.com>
wrote:

> Can you please provide the post parameters to submit a review. I tried but
> 'Customer id' and 'Rating votes' are not getting saved :(
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/ajay-ipragmatech/magento2/commit/5f4e5d6a1e2cd8177a54880e992150e0d6b7ce60#commitcomment-18002877>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AK7BsU73-3VBrIVLs95Gex76nElJqW-3ks5qO8-CgaJpZM4I9uVC>
> .
>


Sorry "" should have been " :facepalm:


Your solution also works and is more elegant.


Can't use this as it needs the internet (e.g. Debian run the tests in a network-less environment)


We not using all of these in the controller. Lets remove the ones not being used. Looking at the current version, i see only the following as needed items:

use Drupal\Core\Controller\ControllerBase;
use Drupal\mirador\SharedCanvasManifest;
use Drupal\mirador\Canvas;
use Symfony\Component\HttpFoundation\JsonResponse;


Why didn't you run this as a simplification [-3.00,1.00]? It would have passed with positive score, wouldn't it?


Following example is not self contained, could use ``%cont-doctest`` but the previous steps would fail anyway as need the internet.


$author is not being used anywhere in the file. Can you double check & remove it if not needed?


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


Because.


will do #7


We leave 127.0.0.1 intact for localhost as well, so it's not an issue. The VM will resolve on the hostname, fqdn, and localhost.


https://github.com/yiisoft/yii2/commit/c370c17e93f364a843ed7c31e1e1f7fc8caef0a3

Спасибо!


good point!


ELC_Storage_Simple (jmarrec) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Do other dependency implementations need this change too?


yes, otherwise i do get a
```
A 'declare' modifier is required for a top level declaration in a .d.ts file
```


Are you sure you want to override the value of $license variable initialized above? In any case, there should be either of these 2 statements. Can you double check?


не совсем поняла про id. это когда создаю и присваиваю id? или в самой реализации аккордиона?
оно просто ламается в некоторых случаях и не хочет работать после перерисовки.. видимо это из-за адресации по id.


Thank you, by today, thre is a updated strings.xml but i'll add it!


@dannysmith


This is a nice feature! :D


Since the time before the end of the window before which to insert the preheat command is 1 second more, I suggest you do the +1 inside the function `timeBeforeEndToInsertPreheatCommand_coolDownWarmUp` instead of here.
Also write in a comment why you do the +1: because we need to be extra sure the temperature is reached in time.


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


Unused variable.


Should be `mycomponent.scss`.


@erorcun I mentioned this commit in JIRA, but no one from CM responded. I'm not sure if this commit does anything harmful... I will wait for a few more days, and if there is still no response, I will upload this change to Gerrit.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#bv-3CdTMRs2K-ZSUiVtHCA


It's just a way of highlighting the result you want first. Thinking about it, it has benefits.


I'd rather populate this list via jQuery if possible.

```js
$("nav li[data-type='method'] a").map(function() {return this.textContent})
```


I understand bro kik is lwonardotmnt957 we been there ok ima add the

On Friday, June 24, 2016, Alexandru Simandi <notifications@github.com>
wrote:

> alex_simi_jr on skype, don't laugh i have that account for over 15 years
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/AdminOdd/POISamaritan/commit/2ce5e0b2b89c053b362e62eb7124312bbca0d52e#commitcomment-18004055>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/ASemlcSXhidoMbwIdc5pgA0njXz1E2rbks5qO-TigaJpZM4I9iPP>
> .
>


> Because then it can't be disabled by the compiler.

I don't really get what you mean here...


TeamCity OTA+ :: rvi_sota_server :: ci [Build 251](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5839&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


same here, I'd rather not have to maintain a list for new methods


disregard me, I just noticed how these were generated.


Was meinst du mit `accessoir`, @MICHAELLAUREN ?


Ok, thanks.


Yes it is, it was done to align with the IoTivity implementation. The TestSpec should be updated to reflect this if not done already. The Certification group knew of this change before hand and it should have been relayed.


why'd you remove all this?


code style: variables should be lower_case_with_underscores and they shouldn't be that long.
Please think of a concise name.


@dannysmith


Leo I ment

On Friday, June 24, 2016, DisgracedDoor . <
anonymoushacksgamesforyou@gmail.com> wrote:
> I understand bro kik is lwonardotmnt957 we been there ok ima add the
>
>
> On Friday, June 24, 2016, Alexandru Simandi <notifications@github.com>
wrote:
>>
>> alex_simi_jr on skype, don't laugh i have that account for over 15 years
>>
>> —
>> You are receiving this because you commented.
>> Reply to this email directly, view it on GitHub, or mute the thread.<
https://ci6.googleusercontent.com/proxy/OXJlLgtduZZ4pVPhHZS-2AFWaUcJONCxpLRCULrE_EWTCCDXl4MHo1R2YYZYLtHEYRgWL_-_lgvLed5oNKCRYYG-50ed_NRzfmfCXBgxdivTvIOwmzd66dHtuNnSNVlrWsLxKRDRUPvkWvE0Y3PCOaJojYVmvg=s0-d-e1-ft#https://github.com/notifications/beacon/ASemld5SBn58bEQDI1Z1aMepp4WwxU54ks5qO-TigaJpZM4I9iPP.gif
>


I am installing it now


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


You don't need assignment here.


wow


const


how so?


Where negative test?


Про id:  у тебя в строке 52
div_text.id = "accordion"+i
где i - номер элемента в списке. Если у тебя на странице будет два или больше аккордионов, то у каждого будет свой первый элемент. Получается у каждого будет свой div_text с id "accordion0". Получится что id на странице не уникальны, а должны быть уникальны.

Про ламается. Подумай, когда и куда цепляются обработчики событий


<!DOCTYPE html>

Hello World

I'm hosted with GitHub Pages.


Ok, I'm going to add u and ring u up in 30min ok

On Friday, June 24, 2016, Alexandru Simandi <notifications@github.com>
wrote:
> I am installing it now
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub, or mute the thread.<
https://ci5.googleusercontent.com/proxy/qNocYN2nW9I3GoMFPOlkuH5fTxzw3dQTN-BbYbIAkB1vUE61oWl7mIhk0ESb2BdhYfeSuP9chkjZ67it6f9Sxm2TaUvxluzcbSFLvl970190MGvUJuvGto0EIGsjkkhyPd3ADGnRr27UkCfVozc7oIval_HMdQ=s0-d-e1-ft#https://github.com/notifications/beacon/ASemlSa1DpPzsrOD_HmKQGrFCVSApeE9ks5qO-0_gaJpZM4I9iPP.gif
>


TeamCity OTA+ :: rvi_sota_server :: ci [Build 251](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5839&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 202, ignored: 3 Build time: 00:06:33


@mattbrejza got a typo here


I don't know if i can talk. I am waiting for a phone call explaining my delayed package delivery for the last 5 hours.


The tag `\return` should only be used for the actual return value. You need the tag `\param[out] var_name Description`. The `[out]` is used to say that it is an output parameter.

Parameters which are input _and_ output parameters can be tagged with `\param[in,out]`...


Hi, this looks good. Can you make sure that if the user specifies an own bundle extension it won't be overwritten by the new defaults in polyfill mode? I am on mobile and cannot see the whole file to make sure it doesn't already do this


@grs this line prevents builds as it duplicates the `friend` declaration below


> IMO it's ok to add bus_type field, or method....

ack


@dannysmith


sorry I know it's not your commit, but not sure how to notify Alan Conway here ;)


Please document the description of the `\return` value.


`supported` seems appropriate. Simple and a verb.


No reason at all. Just forgot it :)
Wanna send a PR?


Sekhar second time committing


The mental image when reading this code places the desired result before the operands.
"I need a 0 and this is how I'm doing a mask for it."


/cc @kellyredding


ack


We should not use any kind of mvc specific code in ReportUtil, here ServletActionContext is struts2 specific which will not work when report invoked using other mvc like spring so please remove this code and revert this file to previous version, freshers ask before doing such changes in infra components. TL please ensure doing code review.


a post-only can see his ticket in the global view but a user without right READALL can't ?


dddeee


If the time window is zero it doesn't take any time to warm up again, because the nozzle didn't have any time tocool down.
Why do you return the time it would take to heat up from standby temperature to printing temperature?


So, where exactly is cacheDir used again? Its creation happens asynchronously and since we depend on it later everything else we do should happen after it (in the .then part)


fix purposed: https://github.com/glpi-project/glpi/commit/8880aa1df1458bff71c1a3f3fe74c284e22f4580


Jimmy Glorial MANDABRANDJA ; June 24 ; 02:58 PM


Okay, I can see your point :)
Yes, it would be very useful if you added tests (or a test) for CreateThread.
Our specific scenario is that we have an irx to elf wrapper and it could be messing with the thread priorities.
This might be the reason why we fail to delete a thread later (because it might not have been created in the first place). Currently we just don't know.


It is not the shift, which is the problem. An enum is an int and 0xFF000000 does not fit in there.


Ja oder nein, @MICHAELLAUREN ?


Is this function actually used somewhere?


And how do you do for anonymous FAQ?


woops, sorry, thanks


Tnx for the update, but this 1/10th. of the complete attack speed for pets.


Well, it's just good practice to check all API calls for errors anyway.  You don't have to clutter the log with them, just stay silent unless they fail.


CallTags are already caught by the superclass


CallTags on GenericMessageSends are already caught by the superclass


@Shubhamsharda  You put the key in app/build.gradle file as given here in the first file. In your IDE the file will be under Gradle Scripts. Gradle scripts has two files called build.gradle. One is for the project - build.gradle (Project:Sunshine). The other one is for the app - build.gradle (Module:app) You put the key instead of 'MyOpenWeatherMapApiKey'. For example if your key is 123abc, it will go like
buildTypes.each {
       it.buildConfigField 'String', 'OPEN_WEATHER_MAP_API_KEY', '123abc'
}


If this class is now deprecated, which is the alternate class that should be used from now on ?


test comment


Hi, thanks for your super quick reply!
In the current implementation, the extension is supposed to be `.js`. I'm not sure we need to take care about  `bundleExtension` when mode='polyfill'. Should we?
https://github.com/suluke/express-middleware-rollup/blob/v1.0.1/index.js#L58


Since `time` isn't used anywhere else, you can just write it as `#{Time.now.to_s.gsub!(':', '.')`. Also, instead of using `gsub!`, you can use `strftime`, check out the documentation here: http://ruby-doc.org/core-2.2.0/Time.html#method-i-strftime


if win_shift is None we also do not need samplerate!
samplerate should get default value None. Same in the other two threshold functions.


CEci n est pas ok

https://github.com/paceuniversity/courses/commit/cdef6a4d89522b80e8117778e0768d32d56cfff5#commitcomment-18004894

@Guette


Yes! Crap.


if samplerate and winshift:


why do we need kwargs? Remove them.


abs is not needed because max is always >= min


Yeah, not sure how that would work...seems up to the network creator to not mess that up (like I did! :smile: )


I don't say that the error shall arise on the first rule. I say that the good form is include in error message location of the first rule. Many compilers does this


Here are some go benchmarks on a unit test that parsed 100,000 lines from a gtf file.

The Test:
**Old way**
for _, attribute := range strings.Split(fields[8], ";") {
_kv :=_ strings.Split(attribute, "=")
_key :=_ kv[0]
_value :=_ kv[1]}

vs.
**New way**
_var kv []string_
_var key, value string_
for _, attribute := range strings.Split(fields[8], ";") {
  kv = strings.Split(attribute, "=")
  key = kv[0]
  value = kv[1]

Old way:
2.384s
2.427s
2.428s
2.425s
2.435s
2.409s
2.413s
2.412s
2.403s
2.417s

With kv, key, and value allocations moved outside of the loop:
2.423s
2.421s
2.199s
2.434s
2.249s
2.416s
2.419s
2.436s
2.434s
1.848s

(P-value  0.16)


Don't recall making this change... will revert!


You know what, rethinking this, this is too much set up.  You can just work out what the hash code is for a password and just created a variable String expectedHash = "blah", and use that in the tests.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 252](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5845&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


I would do: String.class as the expected result not "".getClass()


This blot was my fault. I beg your pardon. Will not happen again.


replace hash(test) with a var called something like: expectedHash which contains the hash value for "test"


I did have it originally in a separate file but figured for the example it easier with everything in one script


Would it be possible to keep all these exe's out of the Source Repository? the ChromeDriver is available here https://www.nuget.org/packages/Selenium.WebDriver.ChromeDriver/ and IEDriver here https://www.nuget.org/packages/WebDriver.IEDriverServer.win32/ and the jar is available via npm here https://www.npmjs.com/package/selenium-server-standalone-jar

Pretty please!


Your first change appears to have gone through correctly. Nice job!


Fan de este commit. Especialmente del título


@orthagh  empty and update are differents for glpi_slts
in empty you delete calendars_id but it's still present in update


Pardon qu'est ce qui n'a pas marché ?


what happens if an exception is thrown?  m will be null and you'll get a null pointer exception here.
probably best to put lines 16-23 into the try block .


@rkorytkowski
After these changes maven can't resolve parameters: session and project for goals(run, setup etc.)
When I changed it back it stared worrking again for me.


@timbur I am not sure, but I don't think this is entirely correct. It would mean that if first argument isn't object with all those properties, you only set provider from a first argument but you ignore rest of it.


@orthagh if you add a field deleted by mistake, you must think to it in update (calendars_id for glpi_slas).
Thinks to ours contributors :)


looks good


We'll work on this in due time.


Thanks. Resolved.


Nice! Now all we need is Oculus to support the vive and we will not need re-vive any more!


Right.
Hmm, then we need to use `fs.statAsync()` here, because we got to return `middleware` soon, don't we?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 252](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5845&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 210, ignored: 3 Build time: 00:07:59


you are welcome.


move this to `FrontPageView#init`


![image](https://cloud.githubusercontent.com/assets/11202073/16341998/3070301e-3a30-11e6-9586-afd3e3f89d62.png)

And this picture has been added to the collection


The following doc(s) have been deployed to the [1674result](https://1674result-static-docs.pantheonsite.io/docs) Multidev environment:
- [source/docs/assets/css/main.css](https://github.com/pantheon-systems/documentation/commit/4b0fd009623c7f81c39024ca03a6de24391d5e7a/source/docs/assets/css/main.css)
- [source/docs/search.html](https://github.com/pantheon-systems/documentation/commit/4b0fd009623c7f81c39024ca03a6de24391d5e7a/source/docs/search.html)


Done for migration https://github.com/glpi-project/glpi/commit/7711c807a6bc84e0041cf3c5cc9f1edc2fb21124


It already is. https://github.com/Boris-Barboris/AtmosphereAutopilot/blob/neo_gui/AtmosphereAutopilot/Modules/ProgradeThrustController.cs#L478


should be fixed now


TeamCity OTA+ :: rvi_sota_server :: ci [Build 253](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5855&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


# Adaptando código del servidor para que funcione socketio

El index principal del servidor trabaja con express y express es el que hace todo el trabajo pero lo recomendado es que se tenga el modulo nativo de http que utiliza nodejs, por ende lo exportamos y con el creamos el servidor con el que vamos a trabajar.
Por ultimo express no escuchara mas el puerto si no el servidor creado, esto con el fin de que sea la parte nativa la que se encargue.
Estos cambios se hacen para enganchar socketio al mismo servidor http.


Code was not recognized as API Description file. Not updating.


what is it?


Similarly, removing the `key, value` allocations altogether (replacing them with kv[0] and kv[1] respectively) did not cause a significant increase in speed (p-value 0.92)


How can I get in touch? I am working on a simillar project.


Backported to `5-0-stable` via cb03f84967e3754337df37b55df65928eb90220c, and `5-0-0` via cb03f84967e3754337df37b55df65928eb90220c (same commit).


Please do the following changes
1) Change the package name from crossHierarchy to crosshierarchy
2) Change the url from crossHierarchy-xyz to crosshierarchy-xyz
3) Update all hard coded labels with properties file value
4) Move all business logic from controller to service


s/sovinyl/vinyl/


@suluke ok, it seems better to move the logic to here. (and handle `.then`)
https://github.com/cognitom/express-middleware-rollup/blob/6792c2d2e1d665f440342aac0106cc3d92b993d1/index.js#L160


This shortcut is mostly only intended for testing.  In any real use of providers, you should actually never (or rarely) use the `instantiateProvider` function.  I'll continue this discussion in #34.


:+1:

Thanks!


Aha! Thanks for the fix & updating the site. I didn't get time to fully step through and make sure I was catching everything :(


The problem with macros is that they are never actually run and are instead recursively expanded and replaced in your original code during the preprocessing phase, before the actual compilation takes place.
And that's correct, if you replace the define with the resulting integer everything will still work, but you'll lose the flexibility that macros allows. If for example `IOC_OUT ` changes in a future OS release, you will need to update your constants and will likely need to implement a new complex system to support backward compatibility for order releases.

What the constant "wrapper" (`const unsigned S_TIOCGWINSZ = TIOCGWINSZ;`) will do is forcing the _evaluation_ of that macro using it as initial value of the constant, on the C side, when the macro is still accessible and Swift will then be able to convert it in a swift constant. And that's needed because once you go to the Swift side those kind of complex macros are ignored and not exported by the language at the moment (only simple `#define name number` macros will be translated in the corresponding swift let constant).

Now, i'm not familiar with the specifics of how the layer, that among other things exports macros, is implemented (or in which block of the llvm/swift compiler chain it's located ) but implementing automatic exporting of complex macros in swift seems pretty complex, even for macros that expand to a simple numeric or string value, because, as you thought, the body of the macro would have to be _really_ executed, something that i'm sure they don't do with simple macros at the moment.

You could try opening a bug on the issue tracker to ask for Apple's opinion in regard to adding those missing constants(only a few? all of them?) on OSX (they could be worried about the exponential increase in ioctl constants inside the standard library...) or you could just add a c module to your project (a bit ugly and not a pure swift solution, i know).


We probably should use aubio's idea of sample duration for this computation - but this will do for now :-)


**IT'S FUNCTIONAL NOW!**


TeamCity OTA+ :: rvi_sota_server :: ci [Build 253](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5855&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **SUCCESS**
Summary: Tests passed: 210, ignored: 3 Build time: 00:05:48


is this necessasry?


hh


@kavdev will do :)


Looks good to me :-)


Ein fix ist es ja eigentlich nur wenn da tatsächlich ein Fehler vorlag.

Jetzt liegt aber ein Fehler vor.
Zeile 52 das zu entfernende Element ist gar nicht mehr in der Liste durch die Änderung in Zeile 25


I have users who report that they get errors about HXCPP trying to use the 4.2 SDK, on systems with only the 9.3 SDK installed.

The `BuildTool.hx` file does a search for the latest SDK on the system, if `IPHONE_VER` is not defined. I asked them to try removing this line here, and it worked for them. Somehow, it is executing before the setup process in the build tool.

4.2 is very old, and will likely break on anyone's system, so perhaps it is safer to disable this for now, and to pay attention if there is any negative effect? I haven't been able to find one, though -- I think the SDK search logic in the build tool is the best approach


As Gordian I must evaluate ???


Not quite. First of all, that call won't be made when user disables the speed control by the hotkey, so the on/off switch will be left in its "on" state. And second, calling GUI update every frame is not very good. It wasn't designed to be called every update and may introduce quite an overhead. The idea was to only call it when something changes. The uGUI system is event-based after all.


And before you ask, yes, speed setpoint does need to be updated every frame, so maybe more lightweight speed-specific method would be nice.


Nice 👍


Nice


Note that this protects against search tree explosion.

There are positions where Stockfish would only reach depth 1 or 2 without recaptures.

So I wouldn't really recommend committing this even though the test passes.


I'd be willing to revisit this once we have a way to benchmark.


Ah, i see, i didn't test turning off, that's easy to fix though.


Finally, moving the allocation of the kv slice outside of the loop and removing the key,value allocations altogether nets a significant 2% speed increase.


This line throws `gsub!: can't modify frozen String (RuntimeError)`


but what will 127.0.0.1 resolve to if there are two entries for it in /etc/hosts?


wew


Please only make one change per commit.


This is good


This doesn't work (and throws a syntax error). Perhaps you meant `if [ -f /etc/init.d/hostname ]` or `if [[ -f /etc/init.d/hostname ]]`? If you want to use `test`, you need to do it outside of a `[...]` block.


This commit added the function get_unite_theme_options twice in the theme file extras.php


Time to install eslint. :)


Thanks for this!


@orthagh You can add ID only if $_SESSION['glpiis_ids_visible']
and gettext is not correct for all languages
$title = sprintf(__('%1$s - %2$s'), $title, $_GET['id']);


python 碎片代码程序存放地


Looks good to me.


Congrats!


Are you sure this works?

I'm a bit concerned because `undiv_start` is an `os::BytePointer`. I'm not sure what that is, but it smells 64-bit. However, the return value of CompareFunctions is an `int` (which smells 32-bit). This could mask an overflow and lead to the wrong comparison result if the two functions are far away from each other.

I think it would be safer to do something like this:

```
if (fa->undiv_start < fb->undiv_start) {
  return -1;
} else if (fa->undiv_start == fb->undiv_start) {
  return 0;
} else {
  return 1;
}
```


nitpick; we try to standardize on a space around the parenthesis, granted unfortunately some code isn't this way (e.g. like 193)


Why add this to ProjectDao if it refers to chunk. Better to move to chunk DAO maybe.


TeamCity Glyssen :: Glyssen pull requests [Build *.*.129.db449853c88337cc4438dce21e59e0df93b46253](http://build.palaso.org/viewLog.html?buildId=74061&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1277, ignored: 17 Build time: 00:05:49


I'd say all of those should use NAT_IF. I think you're right about NAT_STATIC_IP. Will look into it and file a PR for it.


Yes, that is correct.


Sure. Don't think that'll solve the errors, but it can't hurt. The errors are due to the salesforce lib not handling non-ascii chars for API URL components which email address sometimes is.


We're still trying to find out if SFDC even supports email addresses with non-ascii chars.


Already up-to-date.
**** Test Results ****
{
    "suite2": {
        "id": "suite2",
        "description": "GET /",
        "fullName": "Hello World Server GET /",
        "failedExpectations": [],
        "status": "finished",
        "specs": [
            {
                "id": "spec0",
                "description": "returns status code 200",
                "fullName": "Hello World Server GET / returns status code 200",
                "failedExpectations": [],
                "passedExpectations": [
                    {
                        "matcherName": "toBe",
                        "message": "Passed.",
                        "stack": "",
                        "passed": true
                    }
                ],
                "pendingReason": "",
                "status": "passed"
            },
            {
                "id": "spec1",
                "description": "returns Hello World",
                "fullName": "Hello World Server GET / returns Hello World",
                "failedExpectations": [],
                "passedExpectations": [
                    {
                        "matcherName": "toBe",
                        "message": "Passed.",
                        "stack": "",
                        "passed": true
                    }
                ],
                "pendingReason": "",
                "status": "passed"
            }
        ]
    },
    "suite1": {
        "id": "suite1",
        "description": "Hello World Server",
        "fullName": "Hello World Server",
        "failedExpectations": [],
        "status": "finished",
        "specs": []
    }
}


For future reference: please don't merge from the master branch. Use `rebase` instead:

See here for some details: https://www.atlassian.com/git/tutorials/rewriting-history/git-rebase

And let me know if you want to do this together over hangout some time. The first few times doing this can be a bit messy, especially if you have merge conflicts.


Good to know about this parameter...


It still can't just walk off of a platform - you still stuck


/g ;)

Пятница, 24 июня 2016 г., 18:28 +0300 от notifications@github.com  <notifications@github.com>:
>s/sovinyl/vinyl/
>—
>You are receiving this because you are subscribed to this thread.
>Reply to this email directly,  view it on GitHub , or  mute the thread .


nitpick: this block of code could be restructured a little nicer.  i.e. define SolrPersistentSnapshotManager mgr; outside the try-catch and not snapshotDir, then initialize it in there, and then after the try-catch return new IndexDeletionPolicyWrapper(delPolicy, mgr);


For a thread deletion test, I think that makes a lot of sense.  I do think we should err on the side of simpler code rather than more boilerplate.  I'd rather write 20 tests that give good logging for the feature they are testing than 10 tests that give great logging for things that other tests already test anyway.

But I tend to agree, for a thread deletion test, making sure creation works should probably be considered part and parcel.

-[Unknown]


The following doc(s) have been deployed to the [1678outdat](https://1678outdat-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/scope-of-support](https://1678outdat-static-docs.pantheonsite.io/docs/scope-of-support)


on clicking list all users shows lists of users (Status:Working)


Oh also, even if you can't run on a device, you can of course compile new tests and run them in pcsx2 if you want to log more things.

-[Unknown]


uh, GUI updating every frame is always a bad news... Especially when float.toString() is involved.

https://github.com/Boris-Barboris/AtmosphereAutopilot/blob/neo_gui/AtmosphereAutopilot/NeoGUI/CruiseControlGUI.cs#L65

Here you can see how I did a "distance left" readout. A special Update() function that goes an extra mile to be positively sure that the status line update is needed before actually doing it. Something like this can be added for a speed control GUI. Later maybe, when you're done with your adjustments.

Also, are you sure you really want to re-create a SpeedSetpoint struct EVERY time a speed GUI is drawn? Which can be done several times per frame if several GUI windows are open? Like, 100 times per second? And an IMGUI system sometimes calls OnGUI even more often than once per frame, so make it 200. It's things like this that drop fps when GUI is active. You returned it back, so I guess you really need it (beats me why), but maybe not recreate it just so often? :)


This changed file might have something to do with #113


I'm assuming this change should be used for .rpm Dockerfile samples in KC docs and QSG.


If I understand correctly, you are using randomization for security. In this case, shouldn't you use a secure PRNG instead of `bsd_rand_r`, with a seed that is secure?

This seem critical... if you use a known non-cryptographic RNG, it might be enough to observe the address of one or two functions to reconstruct/guess the seed and de-randomize everything.


TeamCity Glyssen :: Glyssen pull requests [Build *.*.130.f7d92390420d43328032f4f441473d51263536db](http://build.palaso.org/viewLog.html?buildId=74063&buildTypeId=Glyssen_GlyssenPullRequests) outcome was **SUCCESS**
Summary: Tests passed: 1277, ignored: 17 Build time: 00:07:23


looks good, but please check that even on many cores, the results do not alter at all.
in the bash, you can set
export OMP_NUM_THREADS=16
to run on 16 threads, even if you have only 2 cores


辛苦了


AFAIK, it is needed to satisfy SELinux rules. Anyway, this method of starting btnvtool seems cleaner.


No, I was wonder how fast is your script :-)


@justinjaywang: Just noticed these improvements. Looks nice! But personally I kinda miss the yellow background on the developer badge 💛


To the comment on the trailing SEMI: yes it is odd, getting rid of it now.
To the comment on the grammar: where is makeListComma defined?


Добавил возможность перейти к следующему при редактировании объявления в админке.


The `session.saveDocument(doc)` should be done always. The `save` parameter should only apply to `session.save()`.


TeamCity Mobile :: Android :: Android SDK (develop) [Build 645](http://teamcity.hq.judo/viewLog.html?buildId=43137&buildTypeId=JudoAndroid_AndroidGradle) is now running


Should this be optional?


Добавлена возможность перехода к редактированию следующего объявления


arrays should not have $ sign .

Only jQuery objects should have $ symble at the beginning


Should the `(default)` be moved away from Classic since that's the part that seems misleading?


Also, I wonder how you made a new GUI work while the old GUI is enabled. That was actually the main reason why I removed the SpeedSetpoint struct: this line https://github.com/Boris-Barboris/AtmosphereAutopilot/blob/neo_gui/AtmosphereAutopilot/Modules/ProgradeThrustController.cs#L523


@zamar
I know but i have severel reasons to try this.
- in match of 1800 games on STC  against master this patch scores 7,1 elo (error 10,4) (i know that statistically not significant). So the simplification (i do them already seldom) is not my main reason. Perhaps
a SPRT[0,4] could be done.
- IMO this is the wrong place to attack the problem (its restricts moves in a very artifical and ugly way). We should instead not use an infinite window on the lower depths.

I will investigate this topic. I can remember at one position (but not exactly). Have someone one or more FEN of such a position? So i could try to compare the different approaches to limit the search explosion in combination with my patch.

In the meantime i have set prio to -10.


This functions should go on a seperate file called utils.


간단하게 가입일이랑 같이 안 나오니까 엄청 불편하다
일일이 정보 보기해야 가입일 나오고 진짜로 기술적으로 불가능한 부분?


bool should not have $ at beginning


lettuce ?


@Lynx187 Habe es IRubin schon in Forum geschrieben das er da einen fehler gemacht hat


1. Yeah, we should leave those as is to match the paper
2. That's for ES2015, which some browsers do not support yet so we should stick with using just "var" for now.
3. Sorry about that, I've been using my own linter style-guide, but I'll change it to the AirBNB one now.


application/vnd.github.VERSION.html


duration and delay different thing.
duration means how long does it take to complete the animation.
whereas delay means how much time it should wait before starting the animation .


You'll need to merge pyret-lang#horizon back onto your branch -- looks like you forked a while ago, and don't have recent changes.  It was defined in https://github.com/brownplt/pyret-lang/commit/0138fd234f813be57293f633ea230484a1bb9255#diff-7de27d6c1df2799df7abe52abb293b52R70.  I've merged most of your changes into pyret-lang#horizon already (because we wanted the tuples!), so it shouldn't be too tricky to merge back.


DB error !


don't use _ in the variable name unless it is absolutely necessary .


Db error


must use === to compare not ==


Keep in the back of your mind when NAT_IF would differ from EXT_IF, and how the plugins and NAT_STATIC_IP would be effected.

Personally, I have a 6in4-static IPv6 Tunnel (tunnelbroker.net) that adds a ```ip6tun``` interface to EXT_IF, this would be an example where NAT_IF would not include ```ip6tun``` .  It currently does not cause an issue, but it would be cleaner.


You should add `if (!string.IsNullOrEmpty(InfoTitle))` for both properties...


such autoimport, I dunno why there is zazab but it's works :D


invitations for account policies nicht joinen, denn es wird vermutlich eine separate Übersicht über Invitations folgen


@joergen7 ok, so now it's a `reduce/6` ! I think this will fit into how we're using...


Hi Benjamin,
could you please open a pull request for this feature branch when you're done?


Why has the changelog been removed? Any substitute?


I think the attry copying will decrease efficiency slightly.  I added because the object that is returned looks like the object that was passed in, except these attributes have been stripped which was making downstream code break.  Another option would be to hard code things as undirected networks (which seems pretty restrictive) and give an error if directed network is passed in.   This is what I'm currently doing upstream in the fast edgelist EpiModel fork for bipartite, since bipartite is expensive and complicated to support.  But I don't think it is safe to hardcode/delete the properties without giving an error.


I guess `int sortCol` is good enough for now? Or do we need a `ColumnMatcher`?


it worked


[messagesearch [Autosaved].pptx](https://github.com/Ellorah/messenger-platform-samples/files/332063/messagesearch.Autosaved.pptx)


TeamCity Mobile :: Android :: Android SDK (develop) [Build 645](http://teamcity.hq.judo/viewLog.html?buildId=43137&buildTypeId=JudoAndroid_AndroidGradle) outcome was **SUCCESS**
Summary: Tests passed: 164 Build time: 00:08:31


Oh, don't worry, I was just confused there for a second since I thought I had already introduced these changes so I went looking for where they were removed since last night.
You're not the first and will probably not be the last to do 'merge' commits =)


ffs


Also, make sure this is big enough. In the paper, you claim that the entropy of Selfrando is >32 bits. However, this seed is *all your entropy*, and it looks just 32 bits to me.

As an attacker, if I can observe a single code pointer, it's easy to test all possible 32-bit seed to find the one that is compatible with the observed code pointer.


Not working okru http://ellisonleao.github.io/sharer.js/#okru


I've seen this twice... a convenience method on the manager would be nice: snapshotsExist()


conversely if you are just scanning down, all you see is 0 ==, which is confusing to see, especially when the rest of the emulator doesn't do it that way.  We give the contributors enough stick about not conforming to the coding standards used around the rest of the emulator/nearby code, so it seems a bit hypocritical to break it here.


`gitnit`


That is because all the others are already checked and correct (for vanilla) - I don't do thinks half-heartedly
This should event hint towards it as well:
>  +-- Attack Speed 2.50:
+-- Nothing to fix here ;)


For now callers should rely on ColumnIdentifiers, which provides integer
indexes based on search parameters provided. Easy enough to add a different
constructor later on if we need it.

On Fri, Jun 24, 2016 at 12:09 PM, Omar Laurino <notifications@github.com>
wrote:

> I guess int sortCol is good enough for now? Or do we need a ColumnMatcher?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/ChandraCXC/iris/commit/acd32a16b8506645e8716e2f90403e9ede713e07#commitcomment-18005954>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AHmNVZNNtDAJu3TNxbOlrTcc8Ekhh_Lcks5qPAEpgaJpZM4I96EA>
> .
>

--
Erik Holum


Hmmm ... the new SPC700 code is 18 bytes longer than the old one (38 instead of 20), yet it says `cpx #50` on line 15. Am I missing something, or is this an error?


Only show members, not invited users, in the User admin


The cdn page takes about 3 days to update to the latest code, you can check with the version url `http://cdn.jsdelivr.net/sharer.js/0.2.16/sharer.min.js`


sure about that!? 😉


isn't this the `any` or `term` type? is the `_` an alias to that?


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


API dates should be returned in ISO 8601 format, we try to be consistent with that. What you are doing here is returning a the direct value of a DAO into an API response, we introduced JSON formatters to add a layer of separation to be sure that changing the return value of a DAO does change the API ( that would break clients ).


Thanks. I would strongly doubt we'll ever need support for directed networks for this, and would agree that throwing an error if someone passes one in is the way to go. As for bipartite, I'm not sure quite yet. I can see the utility of supporting bipartite networks, but not if this substantially (e.g., >100 ms) slows down the simulations. I guess I'd like to see how much it does so.

@sgoodreau may want to weigh in here too, especially about the bipartite networks. Is evonet doing heterosexual sims on a structurally bipartite network, or just with a attribute and nodematch set to 0 (or something similiar)?

If we are both jettisoning structurally bipartite networks, I would suggest dropping support for it completely.


I always use Mercurial and because I forget some things in Git=)


`print`?


Also I love the general idea behind that comment; 'thanks for the update, but it's not enough' ^^


D: why u here


I don't understand what happens if `m_in_place == false`. If the old code is still there, what prevents an attacker from just using it?


add 1 comment


Hi Ben ,
Cool, that method sounds familiar. In the field for the day but I'll check it out and update the repository with the right file.

Julian

> On Jun 24, 2016, at 9:38 AM, Ben Glassman <notifications@github.com> wrote:
>
> @julianpost Hmm it looks like perhaps we didn't upload the right code because I dont see any of the controllers or files we were working on last night here....
>
> By the way I found these articles which it looks like might explain how to implement what we are talking about... a method called prepareForSegue can be added to the controller than is being left (the one with the form) which would delegate the work to the other controller (the one that lists the records). I wonder if that player game tutorial you have uses delegate/prepareForSegue in this way...
>
> https://makeapppie.com/2014/07/01/swift-swift-using-segues-and-delegates-in-navigation-controllers-part-1-the-template/
> https://makeapppie.com/2014/08/04/the-swift-swift-tutorial-why-do-we-need-delegates/
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or mute the thread.
>


make sense, thank you for your answer :+1:


所以是只有readme


Working thanks.


Se quitaron los numeros mágicos... falta agregar funciones para Strings y datos numéricos


hey @guifelix, can you just post an issue in this repo with whatever questions you have?


Please add title


ExpressionsSingleton would be a better name.


We've also jettisoned structurally bipartite networks, and just use
offset(nodematch("sex")), coef= -Inf.

I support dropping these.  Let's focus on fast for our existing use
cases, and add back in functionality when needed.

S

On 6/24/2016 9:18 AM, Samuel Jenness wrote:
>
> Thanks. I would strongly doubt we'll ever need support for directed
> networks for this, and would agree that throwing an error if someone
> passes one in is the way to go. As for bipartite, I'm not sure quite
> yet. I can see the utility of supporting bipartite networks, but not
> if this substantially (e.g., >100 ms) slows down the simulations. I
> guess I'd like to see how much it does so.
>
> @sgoodreau <https://github.com/sgoodreau> may want to weigh in here
> too, especially about the bipartite networks. Is evonet doing
> heterosexual sims on a structurally bipartite network, or just with a
> attribute and nodematch set to 0 (or something similiar)?
>
> If we are both jettisoning structurally bipartite networks, I would
> suggest dropping support for it completely.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/statnet/tergmLite/commit/3539555c46ea79d9a998e4d5454c2912478eeaa7#commitcomment-18006079>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AH3lgl4pTc_h74Igie4wvJu8qaZgY8y4ks5qPANUgaJpZM4I9xyq>.
>

--
*******************************************************************
Steven M. Goodreau / Assoc. Professor / Dept. of Anthropology
Univ. of Washington / Condon Hall 508 / 1100 NE Campus Pkwy
Campus Box 353100 / Seattle WA 98195 / 1-206-685-3870 (phone)
1-206-543-3285 (fax) / http://faculty.washington.edu/goodreau
*******************************************************************


We already have an Expr class, so this is confusing.  If E renamed to ExpressionsSingleton, then just remove the 'as Expr'.


LGTM


TeamCity OTA+ :: rvi_sota_server :: ci [Build 254](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5890&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


🎱 "Structurally bipartite networks shall plague you no more"


TeamCity OTA+ :: rvi_sota_server :: ci [Build 254](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5890&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:27


[ci skip]


Turns out adding another constraint doesn't help at all. This will apply the allowed_pattern check after a glance.image is already selected not the glance.image name but it's ID. not so helpful.


on update you should return the `issue` object. Like we already do on create.

The JSON formatted should be updated to add the new key `rebutted_at`.

The update tries to change a property of the object that is called `rebutted_at` changing it to NULL. The accepted input param should match the name of the key you want to change, in this case `rebutted_at`.


TeamCity OTA+ :: rvi_sota_server :: ci [Build 255](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=5892&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


DATA URI scheme is indeed not URL but URI.
However, the 3 others are all URL schemes. Since URL is a subcategory of URI, I think it's good to keep URL for them to be more precise.


The reason is that establishing many simultaneous connections in a client in "not great" network conditions could lead to socket.io to fall back to polling which would eventually break the connection. Everything seems to be a lot smoother forcing websockets and retrying the connection if it fails in the first place. In the end, if you are falling back to polling you probably are not in a very good shape to begin with. Keep in mind that you are always using wss which is a lot better with proxies than plain ws.

This is not final and, depending on the feedback, we could set this as a configuration parameter or even ditch socket.io and go for plain websockets.


@FroMage Are you sure we all have these folders available locally? ;)


Why did you change Decisionguy to a forced trigger? According to the wiki, both effects are optional, unless there's another translation issue.


Se agregan cambios para corregir error de configuracion, en la definicion de beans sobre spring.


I don't understand why powerline-buffer-id MUST be trimmed.

My understanding is that before, the variable `mode-line-buffer-identification` was used with `format-mode-line` and its value was "%12b", hence the need to trim the extra spaces in case the buffer name is less than 12 chars.

But now what is given to `format-mode-line` is "%b" ([here](https://github.com/milkypostman/powerline/blob/master/powerline.el#L477)), so we shouldn't have extra spaces, am I wrong ?

I'm asking this because it seems to me that mode line updates have a significant impact on general performance (again, correct me if I'm wrong).


http://www.bdpuqvydi5vgwj4s.onion/tormap2.html
`<meta name="viewport" content="width=device-width, initial-scale=1">`
https://developer.mozilla.org/en-US/docs/Mozilla/Mobile/Viewport_meta_tag


Yo, `SetupParser()` (https://github.com/mono/CppSharp/blob/620da528a4b78d30ccbc7b209cbdaf0fe8c06981/build/Parser.lua#L13)  is defined in CppSharp's build files. If you clone QtSharp inside CppSharp's examples folder then you should get a combined build with both projects.

If you want standalone Qt# build files, then we need to copy that code to Qt# repo. What do you think @ddobrev ?


It's a bit weird to see use of Optional here, and see the previous fields initialized with null (mixed null-ness and use of Optional).  Instead, perhaps at least some of those fields could be non-null and declared final, initialized in the c'tor?  Or maybe don't use Optional; comment that it's optional.


Consider this alternative:
          IndexCommit optIc = DirectoryReader.listCommits(d).stream()
              .filter(ic -> ic.getGeneration() == gen).findFirst()
              .orElseThrow(() -> new SolrException(ErrorCode.SERVER_ERROR,
                  "Commit with generation " + gen + " is not present in directory " + indexDirPath));
          return createSnapshot(optIc);