The whole idea behind `IRascalResult` was that it would work for both the interpreter and the compiler.

We added that marker interface specifically for this purpose.
Are you sure you can't return a class with 2 getters?


This change is not mine. Is difference between installation of ubuntu repository  and github repository.


removegitattributes (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2288 of 2343 tests passed)

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


Shouldn't you add the $ sign before the translation?

    SKILL_20150317_002215	$Movement speed of enemy...
    SKILL_20150317_002216	Shrink Body: ....
    SKILL_20150317_002217	$Movement of ....


`self` can be inferred here if preferred.


Added input from file along with validation.


A bo mi się wizja zmieniła. ;D


Photovoltaics (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


o, w sumie można. Ale to bym sobie zrobił jako kolejny taks. :)


I just pushed this commits. I think this might help the problem. Thanks.


removegitattributes (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I'll review from now.


A lot of lines in this file are not readable. Can you indent them?


i think we can merge this one into master now ?


Эти `r` же надо по отдельности проверять. У тебя каждая строка перезаписывает значение `r`.


We should avoid using absolute values as much as possible. If we have to, then is there a way to declare these values as constants somewhere and reference the constants here? Like in Android, there is a dimens.xml


return explode('_',$id);


;p


It is awesome to see people using drive :) Thank you @pageauc for  incorporating drive into your package. I just wanted to let you know that I've made more releases since then (that you could get) and I could keep updating arm binaries for you in case you are interested.


remove


remove extra line


You can remove the commented code.


Nice. I like this.


was thinking about it. with comment people will be able to translate 'm' too.
ok


Ты это делаешь только из-за того что нет функция для перевода PixelFormat в MFVideoFormat? : )


we already have translation for 'Low', 'High' etc
you're using in code 'low', 'high' ..... adding this to translations too  would be wrong ...


You may add a TODO task saying that we need to change this to an actual server url.

Also, we should really be using POST over HTTPS. I remember you said you were looking into secure connection.


Extra space


develop (DavidGoldwasser) - i386-Windows-7-VisualStudio-12: OK (2306 of 2343 tests passed)

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


101163020-SetpointOutdoorAirTempOneDirection-4627 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1797 of 1797 tests passed)

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


It was already in the master but I've now moved the v0.2.1 tag to this commit instead of the old one.

The behavior is identical; the only reason this is a separate commit is because it wasn't compiling on Ubuntu 12.04 and I didn't catch it before I pushed the version change.


What does this do?


how can i can share the software after i do some changes?


RoomAirflowNetwork-Docs (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (750 of 750 tests passed)

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


Not sure if you want this...


You can avoid some nesting by checking the opposite -
if(name == "" || email =="" ....) {
//retrun or do nothing
} else {
// do something
}


But if status is PASS, Test will be passed


Those keywords could be reused by test cases


actually i was trying KlteRIL.java so add does line

https://github.com/MSM8226-Samsung/android_device_samsung_ms013g/commit/1cb86dbb1deee251198ed3c3a35a1e59e7f9205e#diff-0cb3afbdabb5622bd63ffed541baffa8L6


Nicely done


Maybe an idea to publish some guidelines about the "hidden" requirements:
Alphabetical order of extensions, format and source of productId ?


I wanted to test it against chrome again and finish up the remap changes for #724, since it's going to break stuff again.


ro.sec_ril.version=50 is not lollipop RIL i think is for dual sim boolean for making of one RIL for all S5 devices


change this


@mridgway eslint 1.0 disable by default all the recommended rules. I believe you should also configure .eslintrc to `extend: "eslint:recommended"`.


I'm not understanding why there needs to be an empty row.


Why do this?


hi Matti,
instead of hard-coding the change based on the module name, could you do it based on the module type ?

Something like

```
for name, module in process.filters.items():
    if module.type_() is 'RecoTrackRefSelector':
        process.setattr(name, cms.EDProducer("RecoTrackRefSelector", **module.parameters_()))
```

or

```
for name, module in [ (name, module) for (name, module) in process.filters.iteritems() if module.type_() is 'RecoTrackRefSelector' ]:
        process.setattr(name, cms.EDProducer("RecoTrackRefSelector", **module.parameters_()))
```


Seems you're right, it vanishes here too, Windows 7. The problem with moving those line to before the process.close() is that then the sf_close(sf) crashes in the 'cancel' case, at least that's what I remember from back then.
Hmm, trying now and it doesn't crash but doesn't remove the file either.

I guess we'd need to obtain and store the process.wasCancelled() before the progress.close() and do the QFile::delete(name) after the sf_close(sf)


Should probably fix this


Because two of the results don't contain any related services.


and we're testing that no extra records appear?


That seems to do the trick, I'll submit a PR


ohhhhh


Awesome thanks for the followup, glad to hear it works.  Thanks!


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-MacOS-10.9-clang: OK (1802 of 1806 tests passed)

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


Thanks @LorbusChris - this comment totally made my day :grinning:


ok. Sounds like the solution.


Correct.


No


@cviral30 I believe, you are saying everything should be semantic is its naming & should follow the flow of the page.


@cviral30 I meant, every object is square is ambiguous, it'd be a rectangle, right?


Make sure its numeric


@cviral30 We call it developer tools. :)


Add indent


@cviral30 A responsive image will always be `width: 100%;` & `max-height: auto;`


:) sorry I have a bad Japanese translation.


What is the difference？


Remove the ```use``` declarations you do not need.


I guess this won't be correct in all (especially multi-GPU) cases. The output buffer is not guaranteed to be always the same, thus in the worst case this will break and cause undefined data output.


Just checking: Shouldn't the `bin` directory there be actually something like `../.cabal-sandbox/bin`? The references to `Elm-Platform/0.15/bin` in the README from this repo stem from the fact that previously sandboxing was not done cleanly, i.e., two different sandboxes were created, one of which was just having directories under the root. The new version correctly uses only one sandbox. Its bin directory is `.cabal-sandbox/bin`.


I would check against the symbolic enum value. Just `1` is not future proof and might break.


按照这里的代码
比如我的自定义域名是 static.itjesse.cn
那么endpoint便被设置为了 itjesse.cn
这样是肯定连接不上的


ofctl_v1_2.py 側が
"(WRITE_ACTIONS)OUTPUT:2"
の形式のままのようです。


101163020-SetpointOutdoorAirTempOneDirection-4627 (rglazer2) - i386-Windows-7-VisualStudio-12: OK (1797 of 1797 tests passed)

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


see #2189


@yogthos Since we are using a did-mount function, the change to drop `.ready` makes sense to me. However, when I compile and run the code with `.getDOMNode`, the datepicker is missing some bootstrap classes and isn't hidden by default:

![image](https://cloud.githubusercontent.com/assets/8610039/9513656/34d8a456-4c47-11e5-897f-da1bb42ecef4.png)

vs.

![image](https://cloud.githubusercontent.com/assets/8610039/9513688/799cfb6e-4c47-11e5-8d74-9adab592e653.png)


Can they have an anti-Papal background then? [Is that like a beach or something?]


$ git add -A
$ git commit -m 'jackchem2003'
$ git push


有り難うございます。
あらら、次のコミット（設定側）に取得側の修正がはいっちゃったみたいですね。
コミットを作り直します。


Ich finde das ne wichtige Feststellung, dass man Bachelor und Diplom nicht zu vergleichen hat. Wie würdest du das schreiben?


also you may use Boolean.TRUE.equals(acceptance) to avoid extra null check &.


Why do this?


trivial change: this get can be moved below the files.size() validation. saw some minor code formatting issues in some of the above methods.


remember that you need to do email_text = email_text.replace(...)


added searchSubmit as argument for filter. See commit ahead for change in index.html


Sind die nicht eh bloß Debug-Ausgaben, die man am Ende eher komplett rausnehmen will?


matchWith:modelObject


nit: do we need else since we have goto in above "if"... just try to learn coding best practices...


Technically no, but I like being explicit myself.


Yes, it could be of any shape.


Did a visual inspection of this code and it looks to be correct.


Apparently I was in a branch (and unaware of how branches work)


Please add following line:
`set -e -o pipefail`


Slik jeg har forstått det, så vil ikke Docker 1.8 logrotate hjelpe oss, siden den kun håndtere rotering av loggen utenfor containeren. Problemet vårt er at vi logger til filer (og ikke stdout), og dirigerer loggene til stdout ved hjelp av tail. Dermed vil logfilene inne i kontainere vokse uten begrensninger.

https://docs.docker.com/reference/logging/overview/#the-json-file-options


Then you can drop this.


this


Please, add other type definitions here. See, e.g., [storagedbtrfstypes.h](https://github.com/storaged-project/storaged/blob/master/modules/btrfs/storagedbtrfstypes.h).


and this.


New line?


this `+` won't draw in the OSD, sure why, open to suggestions. do I need to escape it? \+?


Anyway, good job!


The encoding algo:
1. Reserve space for the header
2. Iterate defined fields and encode values
3. Iterate `['attrs']` and encode NLAs (recursion)
4. Encode header

If no fields are defined and no NLAs, then algo will work as 1 .. 4. So, only the header will be encoded, in the case of NLA it is 2 bytes length + 2 bytes type. The assigned to the NLA value (if any) will be ignored.

Decoding algo:
1. Read and decode header
2. Iterate defined fields
3. Read and decode NLAs (recursion)

If no fields and no NLAs are defined, only the header will be decoded, and the value should be `None`.

…

So, semantically it would be better here also to overload `decode()`, overwriting the value with `True`, so constructions like `if msg.get_attr('IFLA_VXLAN_GBP'): …` will work correctly.

Thanks for the heads-up, I will update the docs this weekend.


この時点で、
inst.append(parser.OFPInstructionActions(ofp.OFPIT_APPLY_ACTIONS, action))
とできないでしょうか？


上記と同様に、この時点で
inst.append(parser.OFPInstructionActions(ofp.OFPIT_WRITE_ACTIONS, actions))
はできないでしょうか？


Can we also remind users of skyhopper to make sure that they run this command ### sudo yum update
so that they could avoid old libraries/dependencies? [suggestion]


TeamCity pull requests :: netty [Build 124](https://garage.netty.io/teamcity/viewLog.html?buildId=774&buildTypeId=pulls_netty&guest=1) is now running


There are no huge changes in handling the chatchannel. The only big change is how the adjuster will get the channel. There should be no influence for other scripts.


That might be - I tried to stick to existing behavior as far as possible. I feel as though this probably is not too bad since the main use case for `BuildFromSource.hs` seems to act as an install script of last resort at the moment. I think it makes for a cleaner `$PATH` having `/home/me/projects/elm-platform/0.15/bin` in place of `/home/me/projects/elm-platform/0.15/.cabal-sandbox/bin`, a happy accident really.


Thanks :-)


You don't get to stand in front of a shiny cathedral unless you are the pope. You have to go somewhere else. :)


そうですねぇ。
他のインストラクションとちがい、
OFPInstructionActionsの第二引数はリストなので、
ループでリストの箱にアクションをドンドン詰め込んでいって、
最後に↓で入れる既存通りの形がよいかなと思います。
inst.append(parser.OFPInstructionActions(ofp.OFPIT_APPLY_ACTIONS, [action1,action2,action3,,,,,]))
もちろんリストにアクションを単一で入れて
inst.append(parser.OFPInstructionActions(ofp.OFPIT_APPLY_ACTIONS, [action])) x 複数回
もアリではありますが。


react-router 使用 1.0.0-beta 版本是不是更利于后续升级？


跑不起来，楼主能否更新一下这个项目呢


Dependency will might be broken by update.
If write as an optional command, I think it's good.


@mpinsky all of these changes to the WD have made it extremely difficult to get my script working again, because it dealt explicitly with files etc


So, nothing changed...or you didn't had time ?
I will make some suggestions for what could be changed after you merge to the master. I want to test the final version :).
Ty,


RoomAirflowNetwork-Docs (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1276 of 1276 tests passed)

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


ログアウトつくったけど，いらないのでした．
ついでに消しておいてもらえませんか．


This interface is for parsing env tac arguments. No plan to support env tac
in Rodin,so just make it dummy for now
On 26 Aug 2015 7:54 pm, "yibo-liang" <notifications@github.com> wrote:

> What is this antiquto_handler?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ggrov/tinker/commit/02cc4f0fdaaced698976f074feb36ec849c610ab#commitcomment-12910814>
> .
>


パラメータ必要！


note this version does not change tg.


Good catch, fixed in a couple places: 5b21b0f


efa2d61

Does require another suppression though :P


wow. literally a bug. :bug: :fearful:


Cleaned up here: 82f0e86


Good idea. Restructured here: e732be7


ありがとうございます。確かにそうでした...
では、 WRITE_ACTIONS の方は中身が action のみのはず、なので、
下記のように修正できる、ということでしょうか？
もし、下記のように修正するのであれば、actions → apply_actions のリネームが
不要になると思います。


https://github.com/ProgVal/Limnoria/commit/127af5db219136102f20aaec1769202c64fa2dce - it could have had more clear commit message referring here though.


You had an old version. cb426871b22b9b6f0ebb6a557095b9c21eeb83ef
When developing, always use the upstream repo as a starting point
this is where your changes will be merged into.


Андрей, ты не только моих комитов ревью не делаешь. Ты еще и свои не проверяешь. Комментирование этой строки какое отношение имеет к заявке вообще. Вообще этот файл какое отношение имеет к заявке?
Если не исправишься начну штрафовать, регрессивный комиты которые которые легко видны на ревью как этот. И за комиты файлов в которых нет изменений кроме форматирования. Пожалуйста если делаешь форматирование делай это отдельным комитом. Я то к тому что ты это комментирование мог пропустить только если закомитил только форматирование, так как других изменений в этом файле нет. Только форматирование и регресивный комит.


Thanks.


91977808_LBNL_NewVRFModel (rongpengzhang) - i386-Windows-7-VisualStudio-12: OK (1808 of 1812 tests passed)

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


```cmd.ExecuteNonQuery``` missing?


Leider kann man mit nem Bachelor ja ein bisschen mehr tun (und verdienen) als mit einem Vordiplom. Ansonsten stimme ich dir da schon zu :D Und mit dem Vergleichen hast du angefangen ;)


This is a spelling mistake. I corrected it.


ボックス化する必要はないので，Integerよりはintのほうがよいです．

+ モデルの方はlongで作ったのですが，intとどっちがいいかな？


多谢鼓励，最近在考虑项目拆分和架构调整


TeamCity pull requests :: netty [Build 125](https://garage.netty.io/teamcity/viewLog.html?buildId=777&buildTypeId=pulls_netty&guest=1) is now running


Yes. :smile: Fixed in dcecc8d9fcd4bd504a7824c14d961c96b2a9b653.


Еще один регресивный комит, зачем ты проставляешь свойство this.referenceCounterparty.CanEditReference = false;
?

Ладно, это произошло автоматически, но зачем это комитить??? Не надо комитить регресс. Зачем? Ты отвечаешь за каждую строку переданную на сервер. Если не хочешь за нее отвечать не передавай. Нахрена мусорить репозиторий, временными вещами.


It's a carry-over from Xcode 7 beta 5's compiler crashes. Also, Xcode's autocomplete couldn't identify the type of `exception` or `error`, so I figured some kind of type hint would kick it in the right direction :)


Nice, thanks!


RESOURCE_PUBLISHING is related to geoenvo/geonode#2


Just for the record, this was also a breaking change :S


Looks good.


@mcostalba, @lucasrt, Can you put (3 + depth/4) test for 20K at LTC on hold till the we know something more conclusive on my revised test at 7 threads.
http://tests.stockfishchess.org/tests/view/55de90480ebc595da94339a5


91977808_LBNL_NewVRFModel (rongpengzhang) - Win64-Windows-7-VisualStudio-12: OK (1808 of 1812 tests passed)

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


I think just an anonymous class implementing that interface would be enough.

Also, the type should not be the type of the value but the static type of the expression/statement.


line 182: Does the return type have to be `Object` and not specifically `BaseStepMeta`


Would it be useful to know which class relies on the `setDefault()` being present?


Do we need to retain the Apache license on top for our own step?


How to use this `@index` in a template ? I have tried `{{@index}}` inside an `#each` block, and it gave me "Expected IDENTIFIER" error.


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1808 of 1812 tests passed)

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


Maybe by default serialize only changes, and if true passed return full version? In most cases we need only changes I think


Wow, sick stuff! How did you find it?


Right now, (3 + depth/4) is hardly showing any ELO gain and would be a complete wastage of resources if (1 + depth/2) turns out to be a better formula.


Так как придраться не к чему, буду придираться к мелочам :)
`@base_controls` -- именование snake-case.
А кругом camel-case: `@baseUnit`, `@baseColor`...
Нехорошо...

To @robespier


msg.dataの情報でエラーの発生原因（xid=xxのFLOW_MOD,,,など）が掴めそうなのですが、本情報も追加したほうが良いのでしょうか？
![screenshot from 2015-08-27 15 53 31](https://cloud.githubusercontent.com/assets/7121715/9514302/d88d1816-4cd3-11e5-8672-3e16b95f70d9.png)


Can vmem_device_memory (src/mem/vmem/vmem_device_memory.c) used for it?
We already use it in drivers (src/drivers/net/e1000.c, src/drivers/video/bochs.c, src/drivers/interrupt/irqmp.c)
If "#ifndef NOMMU" required, it's better to hide it in vmem_device_memory or make it abstract with 2 implementation


既存の作り込みがそもそもですが...(なので、別コミット？)
to_action() の戻り値が None なら LOG.error() とするより、
to_action() の中で、None となるルートに LOG.error() を仕込むべきな気がします。

戻り値をチェックして、エラー出力をするなら、漏れる可能性がありますし、
冗長な条件分岐が増えると思います。
理想：
if action:
    actions.append(action)
else: ←to_action()内でエラー出力済みなら、不要


TeamCity pull requests :: netty [Build 124](https://garage.netty.io/teamcity/viewLog.html?buildId=774&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4748, ignored: 21 Build time: 00:29:36


Yes, this seems to be an easier solution, than mine.
But... What if the newest slug will be generated with the biggest and not optimal length? (This is most likely to happen in case of small length)
What if there will be a feature like: "all links with no clicks for the last year should be deleted"?


The problem is: If a developer (of Elm tools) uses this setup, then does some patch in, say, the `elm-compiler` directory, and rebuilds, where does *their* newly built binary go? It goes into `.cabal-sandbox/bin`! Unless they also always call `cabal` with `--bindir=...`. Then there are two versions of the `elm-compiler` executable, and only the old one is on the path, and the developer struggles for an hour before finding out why their change to the source does not seem to have any effect (because when they run `elm-compiler` they still run the old one) ...

I really don't think this is a good situation. I think that the `--bindir=...` stuff should be omitted completely (as it will most likely be in later calls of the developer), and that `.../.cabal-sandbox/bin` should be added to the path.

(All this applies only if Elm developers will use this install route. They might.)


great !!


What if, to avoid duplication, add method like:
```javascript
setField(field, value){
    if (this[field] !== value) {
        this[field] = value;
        this.setChanged(true);
    }
};

// Then
setId(id) {
    this.setField('id', id);
}
```


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@schrepfler Good point. We use `com.twitter.app.App` so we could have used flags.  The Flags implementation is type-safe and quite nice. I used it before.

One thing is, but more of an afterthought, environment variables are easy to use with docker by using the `-e` params. You can also use command-line args but in that case you also have to specify/override the default command that is being executed which makes the command-line a bit longer.


OK, I am removing `template0.css` since it's the same as `template_indico.css`


Perhaps I should also describe the idea:
The Rez parameter can only be one integer. In LSL the integer consist of 32 Bit. Prior to this change the core generates a pseudo random channel between −2,147,483,648 and +2,147,483,647. The idea is to use only 24Bit for the channelnumber. So after this change the core will generate a pseudo random channel in the range 2,130,706,432 - 2,147,483,647 (that are 16,7 million channels). This is the only change that could influence any other script.
In the slave script we combine the 24Bit of the channel and the 8Bit for a slotid and use them as the rez parameter. In the adjuster we simply split them again and we have a channel and a slotid ,


Maybe also get rid of argument? Like in extjs:
```javascript
this.enable() / this.disable()
// or
this.setDisable(true);
```


Cool


言語ファイルの変更は不要なようです。


Can we put it in Profile constructor?


なるほど。。こんなかんじでしょうか？

 def to_actions(dp, acts):
     inst = []
-    apply_actions = []
-    write_actions = []
+    actions = []
     ofp = dp.ofproto
     parser = dp.ofproto_parser

     for a in acts:
         action = to_action(dp, a)
         if action is not None:
-            apply_actions.append(action)
+            actions.append(action)
         else:
             action_type = a.get('type')
             if action_type == 'WRITE_ACTIONS':
@@ -111,6 +110,9 @@ def to_actions(dp, acts):
                         write_actions.append(action)
                     else:
                         LOG.error('Unknown action type: %s', action_type)
+                if write_actions:
+                    inst.append(parser.OFPInstructionActions(ofp.OFPIT_WRITE_ACTIONS,
+                                                             write_actions))
             elif action_type == 'CLEAR_ACTIONS':
                 inst.append(parser.OFPInstructionActions(
                             ofp.OFPIT_CLEAR_ACTIONS, []))
@@ -128,12 +130,9 @@ def to_actions(dp, acts):
             else:
                 LOG.error('Unknown action type: %s', action_type)

-    if apply_actions:
+    if actions:
         inst.append(parser.OFPInstructionActions(ofp.OFPIT_APPLY_ACTIONS,
-                                                 apply_actions))
-    if write_actions:
-        inst.append(parser.OFPInstructionActions(ofp.OFPIT_WRITE_ACTIONS,
-                                                 write_actions))
+                                                 actions))


ああ、表記がエラいことに。。消します。


I wonder if we should stop searching once we found a valid subtitle directory. If we agree we should do it outside of this PR since it's changing behavior.


I dont know. Not sure that serialize at all can has default behaviour as partially. I think serialize of object should serialize all object. "Changes" - it is our "optimization" not more. U can ask other devs.


Hi John,

what "find" is it using?

it works on nixos and cygwin, where it uses "find" from "findutils":

$ readlink -f $(which find)
/nix/store/xair9gg198gsgyh62crbdgijp93mrf2g-findutils-4.4.2/bin/find

$ man find

shows

   -perm +mode

   Deprecated, old way of searching for files with any of the permission
   bits in mode set.  You should use -perm /mode instead. Trying to use
   the `+' syntax with symbolic modes will yield surprising results.
   For example, `+u+x' is a valid symbolic mode (equivalent to +u,+x,
   i.e.  0111) and will therefore not be evaluated as -perm +mode but
   instead as the exact mode specifier -perm mode and so it matches
   files with exact permissions 0111 instead of files with any execute
   bit set.  If you found this paragraph confusing, you're not alone -
   just use -perm /mode.  This form of the -perm test is deprecated
   because the POSIX specification requires the interpretation of a
   leading `+' as being part of a symbolic mode, and so we switched to
   using `/' instead.

what does your find say?

Cheers,

Tomas

John Wiegley <notifications@github.com> writes:

> @vcunat These perm flags seem to not work on OS X:
>
> stripping (with flags -S) in /nix/store/kwr60d7n96zcdir5niqqic21jk3s61an-gnused-4.2.2/bin
> patching script interpreter paths in /nix/store/kwr60d7n96zcdir5niqqic21jk3s61an-gnused-4.2.2
> find: -perm: /0100: illegal mode string
> builder for ‘/nix/store/bc3j08gc4i5v910cb4mrh2n1n7vasyhs-gnused-4.2.2.drv’ failed with exit code 1
>
> —
> Reply to this email directly or view it on GitHub.*


Yay!


No. In any case not by default.


thanks,  https://github.com/tnataly/url-shortener/commit/a3b1c759f51add2ab7e06dde86a6eccc44a7ad81


Mind adding a comment so it is more clear taht we're not searching for archives in archives?


And it's dropped as you expected, please take a look at rxe_destroy_ah function.


Yeah, this will be important in future, especially as I've changed some things to keep consistency with newer functions in Matlab (i.e., as the Matlab version keeps updating, we may get some inconsistency). Some of this will be inevitable (and desired), other changes may creep in without knowing, and would not be detected without tests.
Could put some third-party trusted outputs as tests for hctsa (Matlab), and then use Matlab outputs as gold standard for pyopy...?


Sorry, trait is indeed not used at all. Removed it.


Good point.
btw, I suppose you mean `{k: v for k, v in settings.iteritems() if v is not None}`


ok, undid formatting change


Great idea ;-)


MemberNormalSelectList自体を変えてしまうとこれを呼んでいるテンプレートすべてを変更する必要が出てくるため、js自体を複製してそちらを呼ぶようにしたほうがベターです。


Thanks.


Correct, Fixed by creating helper function to be used by both callbacks.


Good luck


yes


isn't template_indico also the same as "no theme"?


Mixing single and double quotes!


This seems a series:

   E ((x -x/b) - .....)

Does not exists an algebraic solution?


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


> sheats


ok, moved code into a new object SslContextFactory and added class comment:
/**
 * creates SSLContext based on configuration, the SSLContext is used for communication between
 * the SJS web server and its clients
 *
 * if encryption is not activated in the configuration, then the default ssl context is used which
 * falls back to un-encrypted communication
 */


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@brnv I stole it from @vain


It's better to have good variable name


done


Hi Uwe
There now is a double break; at line 71/72. Still remaining in master. Does that make sense?
@uwetews


Committed upstream - thanks.


ok, removed PasswordRequester
Start-up now fails with an exception if either the keystore is missing or the password is incorrect (iff encryption is activated)


Start-up now fails with an exception if either the keystore is missing or the password is incorrect (iff encryption is activated).
The exception should provide sufficient information to an admin.


made it configurable


new config option


That was the first thing I tried. :) The last parameter would be {text} => Format.ToString() below doesn't add a colon because the last param doesn't contain spaces => the colon would be missing when a CBufLine is turned back to a string.


@PleasingFungus  thank you for the answer: i tried IRC twice but nobody answered me :(

i actually tried to briefly explain with an example why i believe the fleeing behaviour being more interesting than the "one of us will die" one: you can create different stories.

It doesn't matter anyway, i was very surprised by this and was wondering why this has been implemented, but nobody seem to have informations on this


>to_action() の中で、None となるルートに LOG.error() を仕込むべきな気がします。
元の作りが面白いのですが、to_action()でNoneとなったからといってエラーではありません。
たとえば、METERがアクションに設定されている場合、

1.to_actionでアクション変換
　→Noneで返却
2.順次アクションタイプをチェックしていき、METERなら
　対応した処理を実施し、インストラクションに追加
という処理が実施されます。


There is an algebraic solution. What I'm trying to do is pretty much
described here:
https://en.wikipedia.org/wiki/Annuity

However I'm not keen on introducing floating point arithmetic in Stockfish
code path...

Now, there are ways around this, like precalculated lookup tables, and
simulating floating points with integers, but it gets complicated...

On Thu, Aug 27, 2015 at 8:11 AM, Marco Costalba <notifications@github.com>
wrote:

> This seems a series:
>
> E ((x -x/b) - .....)
>
> Does not exists an algebraic solution?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zamar/Stockfish/commit/9d148843bce4d653480786cbdebade141ab75dbf#commitcomment-12919500>
> .
>


setDirty() and etc..


Is this why torch doesn't work? Where can I read about why was it removed?


Indeed it seems a continued fraction (https://en.wikipedia.org/wiki/Continued_fraction)


Correct issue: #949


No worries,

![](http://pre14.deviantart.net/1ac3/th/pre/i/2012/351/d/6/you_tried__by_the_mexican_whovian-d5od4z7.png)


should leave it, for future edits


`program/mavlink/.communication.c.swm ` should not be here


It's already released ;)


1)  I note the fabs formulas now have their checkboxes enabled by default,
that is OK, keep it that way. The reason I had them disabled by default was
that they are expensive to run and  I commonly would explore with  only one
axis enabled.

2)  Can you explain in simple language what checkbox toggle of elements
does, and should it be implemented on the other UI's?

Cheers

On Mon, Aug 24, 2015 at 3:27 AM, Sebastian Jennen <notifications@github.com>
wrote:

> wired up checkbox toggle of elements in mandelbox103
>
> @mclarekin tried to be careful with your ui, but please check if i made
> any unwanted changes.
> @buddhi1980 toggle works as expected, but does not get called on load.
> eg.:
> 1. make changes in mandelbox103 (change some toggle boxes)
> 2. save settings
> 3. quit and restart
> 3. load settings
> untoggled checkboxes do not disable connected elements.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/buddhi1980/mandelbulber2/commit/e3cbd4917fd7726a4990d2949e11b342cabd7297
>


Works great!


@Sarcasm @josteink @tsukimizake I shamelessly took this file from your project as it was the best one I could find out there. If you have any issues with that, please let us know. In general, I think it would be great if this file could be shipped with CMake.


You need latest PyQt too. I think it's not too hard to build it from sources.


I have a feeling the build may be using `/usr/bin/find`, which seems fairly limited.


感谢你对翻墙做出的贡献！


TeamCity pull requests :: netty [Build 126](https://garage.netty.io/teamcity/viewLog.html?buildId=793&buildTypeId=pulls_netty&guest=1) is now running


develop (DavidGoldwasser) - Win64-Windows-7-VisualStudio-12: OK (2305 of 2343 tests passed)

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


There is a typo here: summ. I don't know to which extent you get a notification of this.


Lets not deal with low-level URL building: https://github.com/tnataly/url-shortener/commit/e2f6b43a6d90f3f5cb97a7a65c8e80ae38ee0c1a


Using unsigned integers in some loops and signed ones in others is scary! If we don't need uints everywhere, maybe we should get rid of them. If we need them, maybe try to use them everywhere.


Hmmm - this looks like a report  / time_step fxxxup. The vector at line 66 has full length (i.e. `report_only = False`), whereas the loop later only fetches report time - and not full time. This has probably worked because most of the test data has num_time_step == num_report_step


Reminder to myself:

jsFiddle uses `<script type="text/javascript" src="http://mootools.net/download/get/mootools-1.2.6-core-nc.js"></script>`.

so maybe we could use

    if (req.path.indexOf('/get/mootools-') == -1) res.redirect(301, '/core');


Litt rart å oppdatere versjon to ganger - er dette dokumentert?


lgtm


Does it also work just with

    <div class="column">

without end-multi-columns? It's not multi columns here, so it's weird to have "end" and "end-multi-columns".


Should be:
`printf("Attempting to find %s.profile...", GENERIC_PROFILE_NAME);`


TeamCity pull requests :: netty [Build 126](https://garage.netty.io/teamcity/viewLog.html?buildId=793&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 4712, ignored: 16 Build time: 00:10:02
### Failed tests
```
io.netty.handler.codec.http2.Http2ConnectionRoundtripTest.noMoreStreamIdsShouldSendGoAway: <no details avaliable>

```


Thanks and Good luck!


Det gjøres to ganger fordi andre versjon er en patchet versjon av Koha, som bygges av jenkinsjob kohaDEV-salt-docker-100-CI og pusher til vårt lokale debian-repository.

Det er dokumentert hvordan det funker, men ikke hvordan det gjøres i vår arbeidsflyt, tror jeg?
https://github.com/digibib/koha-salt-docker/tree/master/kohadev


If the length is only ever increasing, the newest slug is generated with the minimal required length.
We don't care about deleted stuff. There's enough random strings, so we don't have to look back.


Applying to Ph.D. Programs in Computer Science


LGTM


looks good. I just committed this to the codebase. -brady


i thought in the forum discussion people did not like the 'flame' icon for gas... so thats why i did not implemented it


TeamCity pull requests :: netty [Build 127](https://garage.netty.io/teamcity/viewLog.html?buildId=794&buildTypeId=pulls_netty&guest=1) is now running


This could solve GPS reboots problem? :smiley:


All the best wishes for you！


Nå er hvertfal de inkluderte patchene i versjonskontroll:
https://github.com/digibib/koha-salt-docker/commit/19f39a7f0131403bb221eae723bd9ff1b1317a8f


copy right


There is already an `@api` annotation on the class.
With the fix I pushed yesterday in OL3-Cesium, the export should be generated automatically.
Have you tested without it? Isn't current code producing two exports?


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (760 of 760 tests passed)

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


Use local file in order to run phaser examples fastest on local computer without requirement of network.


Måste göra en rollback eller ny commit? value ska inte stå där ughhhh


Does this extension do anything GTE can't? If so it would be good to bring that functionality across to it too!


This can be simplified, as the `int rv` hides the one in the outer scope and is not needed. Recommend just making one line that says:
`custom_profile = profile_find(GENERIC_PROFILE_NAME, "/etc/firejail");`


You could also split in smaller parts:

for (int i = 0; i < 4; i++)
    table[pc][to] -= table[pc][to] * std::min(abs(int(v / 4)), 512) / 512;

or, giving up loop unrolling

for (int i = 0; i < v / 512; i++)
    table[pc][to] -= table[pc][to] * std::min(abs(int(v / 512)), 512) / 512;


Can you give the link to this library?


Looks like you found this bug, too :) Great work! I fixed it 3 days before. Are you having more issues with it?


Then why do you also drop it here?


HI there,
I was wondering if there is any chance to see all google categories.
When I use the above code, I only see 6 lines.
I am especially interested in business and the property category.

Would really appreciate some help,

AW


Out of curiosity, why do we use `angular.forEach()` instead of the standard ES5.1 `myArray.forEach()`?
https://developer.mozilla.org/fr/docs/Web/JavaScript/Reference/Objets_globaux/Array/forEach


... Comment contribuer dans ce cas ? :/


Merged. Thanks!


Yaah


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (758 of 759 tests passed)

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


`return new Crawler($html)` bez zmiennej chyba że chcemy to jakoś cachować


Nit: redundant parens


Do we want to support 3D without terrain or switching the terrain provider?
If not, I think it is better to read the terrain URL and add the terrain at OL3-Cesium creation in the main controller.


The same icon is used for the RFXMeter with SubType 'Gas'. Why not use it for the Gasmeter for the P1 Smart Meter. It is the same device, only measured with 2 different types of hardware...


The commit title should be `Add config for environment variables`, no need to mention it has been implemented using hashtable in the title. However that could be an interesting point to mention in the commit message


What do you mean by "README" ?


Nit: not sure this type is still worth having


On short term, 3D will be always with terrain.

2015-08-27 9:43 GMT+02:00 gberaudo <notifications@github.com>:

> Do we want to support 3D without terrain or switching the terrain provider?
> If not, I think it is better to read the terrain URL and add the terrain
> at OL3-Cesium creation in the main controller.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/geoadmin/mf-geoadmin3/commit/816ab2bf2c13da1d1bb9b806527b3765dfc401c5#commitcomment-12920004>
> .
>

--
Twitter: http://twitter.com/cedricmoullet
Linked In: http://www.linkedin.com/in/cedricmoullet

<http://t.sidekickopen16.com/e1t/c/5/f18dQhb0S7lC8dDMPbW2n0x6l2B9nMJN7t5XYgf6M1WN1qfVVMRYFRqW8qlQyH56dByMf1q8vLz02?t=http%3A%2F%2Ftwitter.com%2Fcedricmoullet&si=6057645868318720&pi=ee84ccb0-fc7b-4cdd-8937-2bf11742f8f3>


Autodesk_Performance (DeadParrot) - x86_64-MacOS-10.9-clang: OK (1799 of 1799 tests passed)

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


Aren't these the default values? Do we need/want to pass these values?
https://cesiumjs.org/Cesium/Build/Documentation/GeographicTilingScheme.html


You said "changed manual-sphinx" although we can not understand what you change from commit message.


This change should be ditched from the commit: just adds complexity to the review


This is the best commit message I've seen, he'd say :bowtie:


TeamCity pull requests :: netty [Build 125](https://garage.netty.io/teamcity/viewLog.html?buildId=777&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4748, ignored: 21 Build time: 00:58:45


Tutaj może bym zapisał te matches gdzieś... tak żeby nie trzeba było za każdym razem w czasie wywołania parseLatLong łączyć się do strony i parsować te koordynaty.

```php
    private function parseLatLong()
    {
        if ($this->parsedLongLat) {
               return $this->parsedLongLat;
        }
        $scriptAsString = $this->getTrm24Body()->filterXPath('//*[@id="content"]/div/div/div[2]/script[2]')->text();

        preg_match_all(
            "/(?=marker)(.*)(\s=.*)([-+]?\d{2}[.]\d{2,})(.*)([-+]?\d{2}[.]\d{2,})/",
            $scriptAsString,
            $this->parsedLongLat
        );

        return $this->parsedLongLat;
    }
```


Why `Xcode`? A copy paste mistake, perhaps?


あ、そうですね...ありがとうございます。
そうなると、今のままが適切にエラー処理されていますね。


I know this is tough but using suffix, such as ```imagename_6h1df.jpg``` is better than using prefix.


not `common-init`, but `init-common`


@p8pepe Tutaj nie kumam po co mamy lat i long zarówno w pozycji jak i przekazywane bezpośrdenio do konstruktora obiektu dockingStation


Really?! Reeeeeally?!

Comic fucking sans?!!?!

> On 27 Aug 2015, at 09:22, Johan Lindskogen <notifications@github.com> wrote:
>
> No worries,
>
>  <https://camo.githubusercontent.com/ab26d88b6cac92e6a860c0686db9029b209f20a5/687474703a2f2f70726531342e64657669616e746172742e6e65742f316163332f74682f7072652f692f323031322f3335312f642f362f796f755f74726965645f5f62795f7468655f6d65786963616e5f77686f7669616e2d64356f64347a372e706e67>
> —
> Reply to this email directly or view it on GitHub <https://github.com/cthit/bookIT/commit/bb9114ccd16875d9b0588fb1e60df936fd8a5ad2#commitcomment-12919661>.
>


`and configured it according to [system requirements](https://github.com/NativeScript/nativescript-cli#system-requirements)`

But verify that markdown links work in this context. Also ping @ikoevska for the messages.


Så virker den igen.


As you can see in each create function of resource that related to the specific PD the PD ref count increased,
This done because we don't want to release the PD before all the resources that relate to this PD released before.


Shouldn't that commit be merged with the one adding the config object?


Empty space, you haven't run the linter, have you :)


Thx! goagent , who can do better than you've done!?


Capitalised vs. capitalized


Can't you add this blank line in the corresponding commit? That would make this diff easier to read


Same here


Ow, that was sloppy coding, I'm sorry. I think I fixed it in commit 42403dd0069d17520d211ff35c2e9d2a3709f987


not relevant to this commit


Redundant line (copy paste error?)


[...] In addition, **the** Operations team [...]


:rocket: :100:


member, singular.


Add the features of onos-drivers and onos-openflow to features configuration.


ありがとうございます。
utils.hex_array(msg.data) の出力も追加いたしました。
https://github.com/iwaseyusuke/ryu/commit/f236deddb8d690c99ae3b8297e664a817b481333

ちなみに...
エラーしたメッセージをパースしようとしましたが、
OpenFlowの仕様上、デフォルトでは msg.data は、
エラーしたメッセージの先頭 64 bytes しか含まれないようなので、
パースエラーとなってしまいました...
パースできれば親切ですが、難しそうです...
WireShark はパースした結果で何かアクションを起こすわけではないので、
途中まで解析して、あとはギブアップするんだと思います。


Yes :smile_cat:


Should "write" be enclosed in double-backticks in order to make it fixed-width? Could be a mix-up between Markdown and RST?


This could also use some killSaveTimer()


Back-ticks in RST denote italicisation, not fixed-width (as they do in Markdown). Perhaps these are meant to be enclosed in double-backticks?


Yep I withdrwan them in other places, not to have two spaces.

Regards


In this PR, the `getProvider()` may return Cesium.ImageryProvider or Cesium.TerrainProvider.
Only the ` Cesium.ImageryProvider` makes sense below when we return `new Cesium.ImageryLayer(provider, layerOptions);`.

OL3-Cesium has two synchronizer objects:
- the RasterSynhronizer which we extend in this class;
- and the VectorSynchronizer.
My idea is to keep several synchronizers, each one with its `getCesiumProvider` function.
We would have:
- a `getCesiumImageryProvider` in charge of creating raster providers;
- a `getCesiumVectorPrimitive` in charge of creating Cesium primitives;
- a `get3DTilesSet`another other provider (and synchronizer) for 3DTiles.

So the current `getCesiumProvider` should be renamed to `getCesiumImageryProvider` and only handle conversion to raster.


@moolitayer please remove this change as you had to fix it later with the last patch


I remember we sort the methods by method name and do not sort them by private o public?


:+1: Thanks


Check those backticks!


this has to be removed..the one at the top is required.


oops


这个没注意，直接安装了。改过来了。


Reboot is required only for the otpmk step..so that the new fused otpmk is used by sec engine to derive it's internal keys, which will be used for blob creation.


اینجا خیلی مهم بود که متنش عوض بشه


Good for you..


I'm not sure how to force newly created link to be appeared in the same page without js reload.
I've tried to do:
    respond_to do |format|
      if @link.save
        add_url_to_session
        format.html { redirect_to root_path, notice: 'Link was successfully created.' }
        format.json { render :show, status: :created, location: root_path }
        #format.js {render inline: "location.reload();" }
      else
        format.html { redirect_to root_path, error: 'Something was going wrong. Link was not shrinked.' }
        format.json { render json: @link.errors, status: :unprocessable_entity }
      end
    end


@jackfranklin Sorry for asking. Do you know if there are plans to go beyond an RC1 for gulp-load-plugins in the near future?


Одинаковые классы могут быть. Но одинаковые классы у вложенных элементов — довольно странно.


nice.


It's an urgent critical patch. It must be a new release. Because current version v2.0.2 doesn't work at all without it.


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/4f88fe8e7f2f1212d870a6d171a1e5416a0be740


merging basho/riak_cs/refactor/request-payment-and-lifecycle-stub = 4f88fe8e into borshop-integration-1223-refactor/request-payment-and-lifecycle-stub


basho/riak_cs/refactor/request-payment-and-lifecycle-stub = 4f88fe8e merged ok, testing candidate = ded479b9


TeamCity pull requests :: netty [Build 127](https://garage.netty.io/teamcity/viewLog.html?buildId=794&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 5583, ignored: 21; resource leak detected (new) Build time: 00:35:53


Да, как вариант :+1:


@samthor as these changes impact JSDoc, would you mind taking a look at these adjustments as a sanity check?


Do you mean to commit the footer changes in this PR aimed at updating the Gulp setup? :)


Жаргонизм ) Спека — это спецификация.

Ясн, это спорный момент в общем ) Хорошо, пусть будет как в демке.


Comme expliqué dans le mail, en me demandant l'accès.


Changes here LGTM, but let's wait to hear back about the JSDoc comments.


Атрибут `name` необязательный. Значение может быть равным значению `id`. По id связываются подписи и поля ввода как раз таки.


Thanks for the extra analysis.

I don't understand what you mean by "the escape character". Does your last paragraph contain another suggestion for a solution (other than the use of SMIE)? I agree that some sort of parser is the "real" solution; unfortunately I don't have time to work on that.


That's right. This behaviour of the markdown parser is annoying: new lines in source are not enough for generating new lines in output :-(

I would revert those trailing.


Окей )


@jhrcek true, I will update that. In the future that should be automatically calculated


3rd party


stdlib


`{}` instead of `[]`


IK. that was a suggestion :). BTW, where do you get all this code?


I thought `template_indico` was the default style (i.e. already available as "no theme")?


Should it be "sink function if it is defined"? Missing an "it"?


This commit is being reviewed as a part of review issue #5.


Will try to write a plugin using this as soon as I can :+1:


dao 함수는 형태가 고정되어 있고, 1개 인자를 쓰는 경우, 2개 인자를 쓰는 경우, 3개 인자를 쓰는 경우가 동작이 달라요. 1개 인자를 쓰는 경우에는 쿼리 실행 함수를 리턴하게 되어 있고, 2개 인자, 3개 인자를 쓰는 경우에는 쿼리 실행 함수를 실행하게 되어 있어요.
1개의 인자를 쓰게 만든 경우가 편의를 위해 추가로 만들어진 방식이라고 보시면 이해하기 편하실 것 같네요.
미들웨어 함수로 등록한다는 건 제가 잘 이해를 못하겠네요.


In this case above depth 22 we don't go to zero but we keep going up (as is natural):

![image](https://cloud.githubusercontent.com/assets/1099265/9515837/8a972c3e-4ca4-11e5-947f-0aee57d58039.png)

The issue is that we have a sudden steering up of the curve at around depth 22, this is very visible with starting value 30000:

![image](https://cloud.githubusercontent.com/assets/1099265/9515893/e760ee78-4ca4-11e5-94aa-f08cbe9146ec.png)

With starting value 50000 we have even an inversion:

![image](https://cloud.githubusercontent.com/assets/1099265/9515903/019cc1d6-4ca5-11e5-95bd-6e19ef7bf7f5.png)

To overcome this I suggest to use 1024 instead of 512:

    table[pc][to] -= table[pc][to] * std::min(abs(int(v)), 1024) / 1024;
    table[pc][to] += int(v) * 64;

![image](https://cloud.githubusercontent.com/assets/1099265/9515915/318ca03c-4ca5-11e5-8341-6f727fc5231d.png)

![image](https://cloud.githubusercontent.com/assets/1099265/9515917/3e272f42-4ca5-11e5-9a28-33d6729122e5.png)


filtr – это транслитное название, правильно — filter ))


missed debug prints


Ok désolé, et merci pour l'accès ;)


Ah, I suppose that makes sense - it might make sense to start a new PR / issue for that


Conficted => Conflicting


You have gone bonkers if you think you can pretend that one cpp + headers (and without any build system) is a library, what a shame


steal => still


Is this `console.log` left in on purpose?


Thank you!


This code could be as below:
```
$full = empty( $nv_redirect ) && empty($nv_header);
```


The "no theme" is absence of any custom CSS file. The `template_indico` is the one with the commented code, that can be downloaded.


move => moved


Autodesk_Performance (DeadParrot) - i386-Windows-7-VisualStudio-12: OK (1805 of 1805 tests passed)

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


But to download it we don't need to provide it as an option since we always have the link to download the template on the management page.. right? In any case, we don't need to migrate it here as it's equivalent to the default.


greate => great
I think you should precise "adding an EXTENT to all LAYER definitions"


Stole this commit from [this](https://github.com/paulharris/boost-nowide) fork of [halex2005/boost-nowide](https://github.com/halex2005/boost-nowide) repository which has no standalone generator.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1519


Heheee... OK


Oops, no. Thanks.


This was changed to avoid any artifacts appearing to images (e.g. unwanted "borders" around shapes).


Man kann drüber reden, ob diese Funktionen in ein Modell gehören. Möglicherweise würde ich die lieber auslagern in eine eigene Klasse. Allerdings ist das bei diesen beiden Funktionen weniger klar, als bei anderen in dieser Klasse.


Moving to 1012 we push away the anomaly from depth 22 to depth 32:

![image](https://cloud.githubusercontent.com/assets/1099265/9516028/16332d32-4ca6-11e5-9fdd-83b5ebd74adf.png)


Release notes are mainly to leave users know about changes, so providing a quick reasoning why these commands where introduced would be really good. For example `purge_user` is to kick evil users :)


mapserber > MapServer


In the management page we only have the `standard.css` file, which basically applies no different layout than the default Indico. So yeah, technically I could remove the `template_indico.css` too.


Thanks for the suggestion. I will most likely write one section that contains all fs-related changes and a few words about the big picture. I agree about the fixups - they are not too interesting for the reader.


Modelle sollten "Java Beans" sein. Das bedeutet sie sollten nur Datencontainer sein, die **von außen** manipuliert werden. Hier werden die Klassen aber von Innen manipuliert und damit werden die Datenspeicherung und deren Manipulation miteinander vermischt...


What if people have uploaded a custom css file with the name `template_indico.css`?


Ooops...


But we don't use that setting for custom css files...


Ah true, nvm, brain freeze


I've rebased so that should work without. I test.


@l3l0 poprawione w kolejnym commit


@tochman You might want to wrap the structure in markdown code blocks (```) for better rendering.


What is this?


Rejected


Corrections mergée


Not very DRY to have this line in every action. I think we need to call `load_and_authorize_resource` for this controller anyway, to ensure you can only update your *own* profile? :-)


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


It should also close drawer if it is opened


There's a typo on the file name too :wink:


The second sentence (1) begins with a lower-case letter followed by a word with the first letter capitalized, (2) is incomplete and (3) has several more capitalized words mid-sentence.


jfkls


Actually we are considering renaming GeoMapFish to GeoMabFish.


Hi,
Kindly sign our CLA before patch can be accepted: http://cla.qxip.net


Adapter has context usually, no need to pass translation strings when it is easy to use ctx.getString(...). Also MyAdapter is quite weird name, OsmEditTabsAdapter or something like that?


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1283 of 1286 tests passed)

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


@djkirby Good points, I hear you. I have published `v2.0.1` with `style` and `activeStyle` support - please try it out!


wandboxでの移植結果
http://melpon.org/wandbox/permlink/Gw04ucYUqeT9tVMo


Why do we use themes for dialog? That would mean that we have completely different dialogs than in other part of application. Also it doesn't take into account dark theme


```return new ModelBindingContext() ```

should be like:

```return new ModelBindingContext```


不会不会


There is a method Algorithms.capitalizeFirstLetter


fast-forwarding develop to borshop-integration-1223-refactor/request-payment-and-lifecycle-stub = ded479b9


Only Light Theme?


Why adding the mod as a requirement since the block itself is already missing anyway?


使水电费


阿斯顿发布


o.O


It appears that I have fixedup rename of username into this commit by mistake.
Should not be hard to undo.


* returns(Classnames) -> returns (class names)
* a Object -> an object


Will both fillerSet overlap if they have the same name?


Odd. They must replaced the binary, since i have one with the previous checksum, but re-downloaded and now got the one you specified. Patch updated


Có phải Chiến FAR không nhỉ ?


Форма без submit таботать не будет.


Где-то в этом наборе блоков потерялся центровщик страницы.Надо бы его добавить.


Здесь наверное лучше подойдет абзац или подзаголовок.


no comentario


No need to create instance of Node.Create reference only.


This did the trick for AppVeyor? I'm impressed.


В этом блоке точно такиеже элементы что и в блоке promo-block. Соответственно все классы и структура должна быть идентична.


Browser support reason. I think the old android browser doesn't support it but this browser is no longer used so we could use the standard function.


Ping @Gnito


You do know that `-j` does not restrict the number of processes?

```
       -j [jobs], --jobs[=jobs]
            Specifies  the  number of jobs (commands) to run simultaneously.  If there is more than
            one -j option, the last one is effective.  If the -j option is given without  an  argu‐
            ment, make will not limit the number of jobs that can run simultaneously.
```
*make will not limit the number of jobs that can run simultaneously.*


You need to invoke it with mono if on unix


okok,好的
![Uploading 01105322_ppvt.png…]()


Пустых элементов быть не должно.


![Uploading 5032de11e949a4370b00022c.jpg…]()


No, I think it is really random.
I have the same behaviour on my VS.
Delete all build, open VS, rebuild, fail, Close VS, Delete all...**Work**


Не размечены блоки Гарантия и Кредит. Они есть в макете.


done


Карта является ссылкой.


Totally right, I fix.


Тег <nav> используется для обозначения самой главной навигации на сайте. В основном это меню. В данном случае достаточно просто списка.


Okay.

What functionality does the material slider lack? It is functionality that is need or nice to have?


Ok...even better.
I copie Moq.dll Moq.xml from Release folder and it work ><


Are you sure it is wise to open the fd twice ? What about if the input is a FIFO ?


Sorry! This is my first time contribute to an open source project. I really like MDL and would like to provide a help hand. So for every issues, only PR anything related to the issue right? Should I submit another PR for the one that has multiple changes?


Well, I'm running this on VS2015 Community Edition and since the x64 issue was resolved I get very consistent test results. I run mostly in Debug mode and get those SerializationExceptions only when adding new calls where the associated Moq setups are still missing.


TeamCity pull requests :: netty [Build 128](https://garage.netty.io/teamcity/viewLog.html?buildId=801&buildTypeId=pulls_netty&guest=1) is now running


your commit log has a typo


please just use lodash.assign.

something like `var assign = require('lodash.assign');


The four tests all do roughly the same thing - look for the first literal `nil` element in the test arrays.
Keep the first one, and change the three redundant tests to use
 * an empty input array ([])
 * an array with a variable in it somewhere that is nil
 * an array with a code element in it that returns nil


Yeah, there's a couple of very odd bugs with loading the config in certain environments I'd like to squish. That said, it's only been reported by 1-2 people so I'm leaning towards going full V1. It will 100% happen at some point!


same here, please just use `lodash.assign`


@alexanderilyin I can create a pull request so you could release 1.0 version officially )


# 泥上班都在干些啥~


aaa


I would suggest moving the .config function to another file. This is because the module should really only be used for setting up the module and it's dependencies and not for setting configuration settings.


Good thing that it works for you with the exchanged Moq files. Maybe there'll be a new Moq version in a couple of days. I saw a version 4.3 mentioned on the nuGet history page, but the associated package is still missing.


I have no issues with this and I agree it's a good candidate for an official cmake-module.

You can make a pull-request [directly on Github](https://github.com/Kitware/CMake), but Kitware prefers you use [their Mantis issue tracker](http://public.kitware.com/Bug/my_view_page.php) to initiate the process.

Create an issue there, create a pull request on Github and reference your pull request in the issue. They're reasonably responsive. Feel free to CC me if you want some backing or code-review.

(Note: I am not affiliated with Kitware in any way, but have had a cmake-patch accepted in the past :) )


please change comment too :)


del


scscscscscs


SKK-JISYO.JIS3_4およびSKK-JISYO.JIS2004の送りありエントリを
漢字変換しようとしても変換されず、送りなしエントリとして補完候補に表示される
不具合を修正しました。


Good - these four tests above test for different kinds of null values. As with the findNil test, add tests using
* an empy input array.
* an array with a variable in it somewhere that is null-valued (for example objNull).
* an array with a code element in it that returns one of the null values.


I don't particularly like explanations for a command that refer to the command itself.


removegitattributes (kbenne) - i386-Windows-7-VisualStudio-12: OK (2306 of 2343 tests passed)

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


Ok, I have reduced priority of the patch.

On Thu, Aug 27, 2015 at 8:55 AM, ajithcj <notifications@github.com> wrote:

> Right now, (3 + depth/4) is hardly showing any ELO gain and would be a
> complete wastage of resources if (1 + depth/2) turns out to be a better
> formula.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mcostalba/Stockfish/commit/19eeaf272393613bf10682a444771fe0da9d9f20#commitcomment-12919304>
> .
>


Oh you are right.
4.3 seems to add some changes that could help us :smile:
Lets see later about it


真不经看，第一个文件就写错了(╯‵□′)╯︵┻━┻，双引号让我吃了


Oups !


Solves issue #9


Final period?


What do you mean "where"?


could be better , like ...


No this is a typo...


It  looks great~ We are still improving our RBR.

Thanks very much for your related work.


Why not use `!=` instead of `not ... ==`?


Done in https://github.com/elm-lang/elm-platform/pull/66.


`recover`


I'm saying that if you go to err2, you will release the pd twice: once here, and once in rxe_destroy_ah. Or am I missing something?


已经有 uid 的是不是还用原来的。

```
this.file.uid = this.file.uid || uid();
```


TeamCity pull requests :: netty [Build 128](https://garage.netty.io/teamcity/viewLog.html?buildId=801&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 4664, ignored: 16 Build time: 00:09:07
### Failed tests
```
io.netty.handler.codec.http2.InboundHttp2ToHttpAdapterTest.clientRequestStreamDependencyInHttpMessageFlow: <no details avaliable>

```


Just by reading this I can tell that this test hasn't been tested even once. Can you see why? ;-)


should be `brick_count < 2`

Doing a replica of 0 or 1 causes gluster to error out


You have already done this test for the  findNil function.


Makes sense. ;)


fileList 也传出去？


The following feels easier to understand (less repetitive):

```python
        def get_user(username):
            try:
                return User.objects.get(username=username)
            except User.DoesNotExist:
                raise CommandError("User %s does not exist" % username)

        src_user = get_user(args[0])
        target_user = get_user(args[1])
```


Just a suggestion to go with deadline & ondemand. What's good on paper doesn't mean it's good in practice. We have fairly good & fast NAND & I know deadline will give best overlay results  when you consider all possible usage scenarios +it's ist very good when things get complicated & there lots of bandwidth (in games & so). Interactive cpu governor is simply to aggressive if you ask me.
As mentioned just a suggestion.


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


Make a pull request if you wish.


It's not possible to do it this way.


Isn't this redundant to ConfigFragment.edit() method?


After this fix,
When I drag a block to sidemenuspace from workspace to delete.
The block vanishes, hide under sidemenuspace colum.

I'd like to know where to fix the problem.


Done.


Maybe we should call it PMIX_NON_PMI since as its being said that we don't restrict PMIx to MPI implementations.


How? Src of PM-MP?


Thanks @markwaters for the tip.

The referrer thing stops some simple script kiddies, who tend to POST directly to the form endpoint. You can also disable this functionality by setting ``ignore_referrer = true``` in your config.ini


Please do not add files no end of line in the last line.


+1

I think we have a maybe a usecase we need to think about for these synchronizers.
Example with `pixelkarte-farbe` layer. It's  a WMTS  in 2D then in 3D this layer will display 2  layers: a WMTS, and swissnames 3d ( a billborad  I guess).  We can also add 3D buildings in the future.  So how the synchronizer will handle that @gberaudo ?


I'm sorry, but I don't quite understand this. I'd think that in case where ``virt-install`` is used ``disk_size`` gives the size of the whole VM image and thus / plus all the other file systems. What is the difference from the image installation where the image contains all the file systems too? Doesn't it?


@warden is it technically necessary to upgrade to a such recent version of jenkins core ?
I recently asked this on the dev list : https://groups.google.com/d/msgid/jenkinsci-dev/9050465C-3A8F-488B-A727-5FC23F37A6E3%40beckweb.net

@daniel-beck gave me this interesting feedback

> Determine how many users you want to reach. The statistics we gather can help you with that:
http://stats.jenkins-ci.org/plugin-installation-trend/capabilities.json

> E.g. if you increase to 1.532, you get 91.5% of all known installations, compared to 99.7% with 1.424. 1.580 would be 73.6% of installations.

> Related: https://wiki.jenkins-ci.org/display/JENKINS/Choosing+Jenkins+version+to+build+against

From my point of you we should try to target at least the first version of the current LTS stream (1.609.1) or maybe a previous one (1.596.1 or 1.580.1) to allow as much users as possible to use it (even if you support support only Java 6+).

WDYT ?


wtf?


hhjjh


Any release with this fix planed ?


Close scanner connection


sorry, this should not b e in here, trying to undo ....


Honestly builds are so fast it will be useless and hard to read. Btw if you are really going to use parallel, maybe it's better to even write a makefile instead.


No, I get it from a cooler place. My head.


> However, rebasing this would cascade through all the changes in further commits, so I'd prefer to do it in a separate commit. Perhaps updating all smart answers at once?

Updating them all at once, and separately to this commit, sounds good to me.


Well... I do get a bit of help from other places.


Oh https://github.com/Moq/moq4 4.3 is far from being out :cry:
https://github.com/Moq/moq4/commit/26358a6d2175ab0152a142be146b70c567a6385a


If there is generally only a single CSV which imports this data then a dedulicator isn't required.
It is only required when there are multiple references to the same object...so usually higher-level resources like Orgs, Sites and People...or for when the same data may be reimported multiple times with changes. I suspect neither applies here but haven't investigated. However it hurts vgery little to leave it in-place


Please try the following testcase:
```
#include <complex.h>
void uninitvar_cproj(void)
{
   long double complex dc;
   (void)cproj(dc);
}
```
No warning is given.


Why NewAppTheme?


:slime:


This file has never been tested in-game even once.


I do, note the "if windows"


Yeah, use original-fs instead of fs.
Still using 0.0.23, this bugged me a little bit, until I read the doc about asar on electron webpage.
Besides, I got some others errors like cannot open .dll, .pak files. Looks like swapping from fs to original fs solves them as well.


This function has never been tested in-game.


No, 4 is not the expected result.


I would be wary about using cap/info.rss since this controller is only accessible to Authenticated users isn't it whereas you need to allow access to public, which is best done via the single /eden/cap./public point of entry.
Sure this means that the cap_info will be nested inside cap_alert, but I still think this is more logical.


Add a comment saying we also modified the script to use new Recaptcha in the comment above.


:sheep: :+1:


Yeah, just saw that, too.
We'll work out something to get AppVeyor into a stable state anyway.


This is essentially a repeat of the first test, although the expected result here is correct.


Autodesk_Performance (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1805 of 1805 tests passed)

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


system("paket.bootstrapper") will work since it's the command line wrapper for the paket.boostrapper with the same type of logic.


Redundant parens on `(targetAllInputs target)`.

Can hlint this?
Can you replace my spaceUnwords with:
```
spaceUnwords words = " " <> BS8.unwords words
```
?


Why the build failing now -_-


:heavy_check_mark: added to my backlog


Thanks @rajeev3001, The following code has been changed.


"removed template" (commit msg)


Json Handling sollte auch nicht Teil der Java Bean sein. Das sollte in eine eigene Klasse oder einen Wrapper ausgelagert werden.


Merci @nico3333fr

Le souci est que je ne peux pas opérer directement des changements sur le fichier "normalize", sinon ils seront écrasés à chaque nouvelle version de normalizeCSS :/

Je vais attendre que le PR passe directement sur Normalize : https://github.com/necolas/normalize.css/pull/451


Why not $@ here? Because of the escaping needed? Perhaps buildsome should escape the $@ or provide an escaped $@?


I thought I fixed this already... I guess I haven't pushed my fix yet.


Ugh, politics getting in the way of working on this. :S


Nope, I think the gradient of the norm term cancels out.

On 20 August 2015 at 18:15, Valentine Svensson <notifications@github.com>
wrote:

> Is -log_phi really -Hgrad? What about the norm term? Doesn't it depend on
> phi?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jameshensman/GPclust/commit/4cb6c0b1e50eec15a495802463f2cd8cb51c1618#commitcomment-12812558>
> .
>


cap/info.rss works fine on my system - the items I see are headline/description just as is configured in the XSL


安得广厦千万间，大庇天下寒士俱欢颜。


Name the config "captcha_secret_key". "secret_key" is too ambiguous.


I can see that this unit test file hasn't been tested at all.


Thank you very much! The level of detail is just perfect for the release notes.


Der Unterschied zwischen diesen beiden Funktionen ist nicht im Namen gespiegelt.


Autodesk_Performance (DeadParrot) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Personally I would link the headline to the Web page (profile) for the alert rather than the CAP format as I think RSS users will more typically want that.
The RSS could include the CAP link separately or rely on people getting that from the profile page


This should have been done in my initial commit itself. Missed it that time. So doing it now. Once I squash all the commits it wouldn't really matter I hope.


Would be interesting to see if there's an issue on NPoco logged for this and/or to create a PR for NPoco to support this properly ... since we'll be using it in v8 :) We've already submitted PRs to fix the mem leak with found in PetaPoco, they'd love Umbraco if we fixed something else!


Hey! When I wrote about `ssize_t`, I *really* meant it.
It *must* be **ssize_t** (note the extra `s`). Like, verbatim:

```c
for (ssize_t i = 0; i < (ssize_t)n - 1; ++i) {
```

Otherwise this code fails in `n = 0` case.


Das geht hier so weiter. Die Unterscheidung zwischen verschiedenen States scheint klar in die Programmlogik zu gehören. Dafür sollte eine neue Klasse oder sogar Hierarchie aufgemacht werden, die regelt wer/was/wann sehen muss/darf.


Here you are declaring a new module called 'app.mortgage_calculator_service' (you can tell this because you have the array as the second argument). I would suggest renaming this module to app (while this project is small. We will create a new module later as it grows).


As with above I would rename this module to app and remove the dependency array


after you change the two modules to app you can remove the two module dependencies from here


Das hier scheint ein Factory Pattern zu sein. Dann sollte das auch in eine entsprechende Factory ausgelagert werden.


This commit is being reviewed as a part of review issue #6.


This seems like double-escaping:
` ' " ' " ' ` <- Surprised this works (and isn't parsed as: `' " '`   followed by unterminated `" '`)


Sigh, I don't see how to get those examples without selab. Wanna give it a try, @TiarkRompf ?


Why not squash this?


I would also remove '.module' from the file name


Yes... :+1:


The ```toLowerCase``` should be done after checking that the ```CT``` is not ```null```.


Offensichtlich anderes Aufgabenfeld als der Rest der Klasse. Verschiedene Logik/Aufgabe ist immer ein gutes Zeichen dafür, dass eine getrennte Klasse benutzt werden sollte.


First time I've been excited for significant API changes :laughing:


None of these values can be `null`, so you should use the lower-case `string` (primitive type)


Anyway, even that ratio of 100 is too much. [In that test result][*], the test was killed after 20 minutes.

[*]: https://cgal.geometryfactory.com/CGAL/Members/testsuite/CGAL-4.7-Ic-128/Intersections_3/TestReport_lrineau_x86-64_Linux-Fedora19_llvm-clang-with-g++-4.8.gz

Fixed by d77da3f15bb507185e80611c4ca92e530f417036: now the ratio is 10.


But the tests use the $buildsome executable, not "buildsome". Relying on $PATH is error-prone and problematic... Why not the variable exclusively?


Fantastic , browser privacy restored , thanks @mapkyca


This commit is being reviewed as a part of review issue #5.


Cool, no crashing. Any instrument GUI coming out yet?


What does the `p` do here?


same here - this was fine as-is. (could possibly be `{string|number}`, without spaces)


targetAllInputs is a bad name targetAllInputHints. It includes input hints, but not accessed files. This seems wrong?


Since we do not yet have a parallel system, I would rather update the protocol document after we create such a system, rather than reference it here.


Thanks @lasanthafdo , @keizer619 , @rajeev3001 - the code has been changed in the pull request - https://github.com/wso2/carbon-analytics-common/pull/109


This works perfectly but can you think of any way to make it slightly more efficient?


@zpqrtbnk  Actual database manipulation should not be done in a migration. Migrations are used to queue executions up so that everything is run inside of the single upgrade transaction.

To execute raw SQL for a migration, this should be done in a `MigrationStartupHander` which is an event handler. You can see an example of this in `EnsureDefaultListViewDataTypesCreated`

So this logic should be moved into a MigrationStartupHandler.


Same question for this one. It works perfectly but could it be made slightly more efficient?


Surely you meant $f and not f here?


Thanks the Author provide such a good tool to bypass GFW.


Thanks for catching this Rich. What's the best strategy with tests, given that some of the internal relevant functions are within the hyperfunction? Perhaps calculate some example values and make sure these are retrieved correctly?


Good Luck!


Thanks @keizer619  - the above code is changed due to cApp deployment validation in the pull request - https://github.com/wso2/carbon-analytics-common/pull/109


Var en delad commit. Ska nog använda pull istället och separata repositories


The best approach (in long-term view) is certainly to have our `findutils` in the stdenv. I thought such purity problems had been addressed on darwin in the last several months.


Whoops - right now, this should be `{number|!Array<string>}`, as the two values here are `0` (`STARTING_POSITION`) or an array of CSS classes.

As an aside, this whole definition kind of defeats the point of it being an enum since the inner content of `ANIMATIONS` can just be modified. Possibly moot for right now.


yang loop itu setelah tekan button checkout. seolah olah ngga mau masuk ke page veritrans nya gt.


Because i was experimenting with it (its a beta), and am not sure we like this icon.
It's not very 'gas' like, or the colors.... in the utility tab are setpoints, and they have the same icon, it is hard now to see the difference between sensor types in a quick view


LGTM


Is this because mtimes have wrong granularity? I don't think it's needed, mtimes are hi-res.


I'd be inclined to make a vector of three narea numbers (three because it won't divide through nicely) folow the approach at [line 197](https://github.com/traitecoevo/plant/commit/0d51b50cf88c499f8209137ae8385dcfa6b8e6bc#diff-7721508db52e50398f7705c8bcb49280R197) above and confirm that if you compute the strategies separately they agree with computing as a single group.


Again, this works perfectly but `element` is a curious name for the argument. Can you explain why you chose that name and can you think of any other names that might be clearer? What other names did you consider before choosing `element`?


ah right, fixed.


@zpqrtbnk  Actually, we can add this - we are allowed to break interfaces in certain namespaces because people cannot really implement these in their runtime web apps. Sorry if I mislead you down this path but we are allowed to break some interfaces, see (Non breaking):

https://our.umbraco.org/Documentation/Development-Guidelines/breaking-changes

So the changes required to `IServerRegistration` and `IServerRegistrationService` are absolutely allowed and should be made and then we need to remove the casting being done.


Isn't it safer (and simpler) to just compare the content?

diff -u ./should_be_deleted_by_make <(echo The expected content) || (echo 'ERROR: ..' ; exit 1) ?


This was a tricky question and your answer is perfect. Nice job!


I'm seeing some issues that I'v traced back to this line (eventually got removed in head). Altough everything worksl rc is TIMEOUT. Why does libusb_interrupt_transfer throws LIBUSB_ERROR_TIMEOUT in-spite of everything working?


Ditto.


Can't you just use
`if ( !empty( $this->notices[ $type ] ) ) {`


Perhaps `prependAll` would be a better name?


hej kim hvad laver du


@zpqrtbnk  you know there's an overload that you can just pass in a type to the logger, you can choose:

logger.Info<T> or logger.Info(Type t)


Added manually to the current build, too far behind to merge. \o/


Also increased this so the autocompletion doesn't pop up on every single char you write.


There's not a whole lot of date handling in here ... so I don't see it as a huge loss to use `java.util.Date` (and it doesn't seem worth a Joda Time dependency to use that either).

I'll merge this in and do up a new release in the next few days -- although I gotta say, Java 7's EOL so time for the world to move on. :)


global namespace spam


刚想上来看看有没有更新，居然没了。。。


suggested edit: "Although support for ES2015 (formerly ES6) is improving in modern browsers, the majority do not yet support the full set of features. To benefit from the awesomeness of the new ES2015 syntax while keeping backwards compatibility with Polymer's supported browsers, you'll need to transpile your JS code from ES2015 to ES5".


I recently updated master to use Polybuild (https://github.com/polymerlabs/polybuild) which should also give us crisper. Interested in whether it is still needed here.


HTC One Mini (M4) uses same name.


@basiszwo I merged your branch into my fork (since there's already a PR open). However, I excluded this commit.


There's an idiom to use here (list comprehension guard):
```
[ prepend "touch" $ makefileTargetPath tgt
| isDirectory tgt ]
```

`[x | y]` is nicer than `if x then y else []`


This will not work on windows.


done


Although it's clear from reading through the source, it might be useful to mention to the end-reader where these files are going to end up.


this indeed now got confusing with the use of the exists method


can you remove dependency of the test to `joda.time`? it is an optional dependency in the pom...


to get rid of `jodatime`, just use the `new Date().getTime()` as before and a random constant for the int value.


"Crisper extracts JavaScript that's inline to HTML files (such as imports). We need this as Babel does not support transpiling HTML files such as `<script>` tags directly.

@samccone I actually didn't know Crisper tackled this so well. Was this idea similar to what you had in mind for writing ES2015 flavoured scripts inlined that would be extracted and transpiled?


saw approval from ksauzz
at https://github.com/basho/riak_cs/commit/fb5b898bb83a4388cc3eb4218e5554088b35b9e1


merging basho/riak_cs/feature/update-clojure-test = fb5b898b into borshop-integration-1208-feature/update-clojure-test


basho/riak_cs/feature/update-clojure-test = fb5b898b merged ok, testing candidate = 34a83813


nub is `O(N^2)` (uses `Eq`),  Set.toList $ Set.fromList    is `O(N*logN)` (uses `Ord`).

Maybe write: `sortNub = Set.toList . Set.fromList` or an order-preserving `nub` that is `O(N*logN)` instead of `O(N^2)`?


Humm, ça devrait pas aller dans une autre recipe, ça? :)


TeamCity pull requests :: netty-tcnative [Build 1](https://garage.netty.io/teamcity/viewLog.html?buildId=802&buildTypeId=pulls_netty_tcnative&guest=1) is now running


Shouldn't this class be `public`?


This is not good! it should be `2.2.1` without `DEVELOPMENT`


utilery peut prendre autant de yml en entrée qu'on veut :)


Nicer style:  `(BS8.pack . showPos . targetPos) target`  to avoid having parens, `$` and application all in the same line.


Great work! Leaving some comments inline.


no


Yes, let's git tag when release is announced.


TeamCity pull requests :: netty-tcnative [Build 1](https://garage.netty.io/teamcity/viewLog.html?buildId=802&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:56


Alright, that seems like a good solution. Will fix it soon.


ok


lol... I am doing my school HW. I am not an adult... :+1:


Thanks for linking up these issues. Does this result in the build pipeline breaking while running babel in the terminal or is it a background/silent failure in sourcemap generation?


The `scripts/git/pusher.php` script did it. Is it wrong?


ran -> run


If the name is meaningless ("t") I prefer point-free:  `map (("\trm -f " <>) . escape)`


done


https://github.com/atoum/atoum/commit/162cd21d1f196e963946fd7c1c77eeee74fb596f is correct, and then a new commit put it as a development version.


For example:
$obtype = "usergroup";
$identity = "ugroup1";
#$keyvalue = "FAIRSHARE=USER_SHARES [ [user5, 4] ]";
$keyvalue = "GROUP_MEMBER=user7 user8";
$comments = "hello";
$ret1 = $b->addmember($obtype,$identity,$keyvalue,$comments);
print "addmember ret1 = $ret1 \n";

print "\n";
$obtype = "hostgroup" ;
$identity = "hgroup1";
$keyvalue = "GROUP_MEMBER=user7 user8";
$comments = "hello";
$ret2 = $b->rmmember($obtype,$identity,$keyvalue,$comments);
print "rmmember ret2 = $ret2 \n";

While using them, we need remove the prefix "lsb_" and use addmember and rmmember directly.


Can you squash the style fixes here (so stuff is indented and not aligned)?


nu ne mai trebuie asta daca am scos enumerize de jos.


a comment why we do such special handling  would be nice


message???


Squash to this commit to change alignment to indentation.


sure. This is obviously a rebase error of mine due to lack of attention. thank you!


There's `Set.notMember`, though this was already removed in further commits...


@Hywan 162cd21 is OK ;)

I'm fixing the pusher right now :) thanks for the hint!


ok great =)


Let me test that. I actually still need to test this on master. This recipe is based on v1.0.3 since I extracted it from the codelab where I tell people to download v1.0.3


### Missing semicolon.

```
return "Hello, World!"
                      ^
```


### Use '===' to compare with 'null'.

```
if (bla == null) return;
        ^
```


I know, lol XD I'm 11


Missing header?


Done.
By the way, what are the reasons for not using optional dependencies in test scope?


@rhc54
Why do we have PMIx_Get in server part?


debug?


same


lol I am 11 ~ 12 in the middle...


1. Ok. If you use `format.js`, why do you need `format.json` and `format.html` then?
2. If you need to reload a page after creating a link, why do you use  AJAX at all? Just send a basic POST


This is a pure function but you gave it an IO type (because of the weird `makeRelativeToCurrentDirectoryCwd` which is a thin wrapper which isn't useful to use here).

And because you used: ... =<< (return $ ..foo..)   which is (by the monad laws) equivalent to ... (..foo..)


Yes it should as according to their documentation, but I never had any problems with getting invalid tickets after the function returns so I'm not too sure. It is hard to find correct examples of GetAuthSessionTicket. In the steamworks example (see p2pauth.cpp: CP2PAuthPlayer::StartAuthPlayer()) they don't bother waiting for the callback. It just gets the ticket and sends it to the server.

Anyways, I refactored the code to use a async callback with the ticket in this commit since GetAuthSessionTicket may block execution within the node.js context. c3d4c9453bfdb8badaea606a34fb2303b3c00c27


Done.


already removed, refresh


A small broken(at least on mac) patch to work around crossguid library dependency
https://raw.githubusercontent.com/sobkas/something/master/builtin_crossguid.diff


@rhc54
The same comment as for PMIx_Get.


Wanna call ourselves the 11 Group?


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1522


Modifying pascal strings in such way is a bad idea. If it still works it`s a great luck.


outputs from get_device_array function, looks rather random to me

```
(process:15786): Ufo-WARNING **: device array is NULL

(process:15786): Ufo-WARNING **: location on host

(process:15786): Ufo-WARNING **: device array is NULL

(process:15786): Ufo-WARNING **: device array is NULL

(process:15786): Ufo-WARNING **: location on host

(process:15786): Ufo-WARNING **: device array is NULL

(process:15786): Ufo-WARNING **: device array is NULL

(process:15786): Ufo-WARNING **: device array is NULL

(process:15786): Ufo-WARNING **: location on host

(process:15786): Ufo-WARNING **: device array is NULL

(process:15786): Ufo-WARNING **: device array is NULL

(process:15786): Ufo-WARNING **: device array is NULL

(process:15786): Ufo-WARNING **: device array is NULL

(process:15786): Ufo-WARNING **: device array is NULL

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: device array is NULL

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image

(process:15786): Ufo-WARNING **: location image
```


authorship?


ip ro sh | grep ^default| awk '{print $3}'


Haven't had much time I'll look into it today hopefully.


[2015-08-27 17:22:32] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:22:32] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:22:56] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:23:56] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:24:35] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:25:41] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:26:20] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:28:37] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:28:48] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:29:49] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:30:53] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:31:26] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:32:01] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:33:14] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:33:16] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:33:23] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:34:45] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:36:46] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:39:45] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:40:04] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:41:05] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:43:41] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:44:53] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:45:09] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:45:12] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:45:16] failed to recv data in handshakeReceive2Callback
[2015-08-27 17:45:19] failed to recv data in handshakeReceive2Callback


Hmm, ideally we want to be keeping the same hash. Let's maybe not rush into this one - I think we can do some other refactoring to make it possible to have the best of both worlds, but I'd rather think about it separately...


That is true, sorry about that mistake.

In order to not proxy the `/pdb/cmd` endpoint I'll change it to
```
  $proxy_pass         = [
    { 'path' => '/api/pdb/query', 'url' => 'http://localhost:8080/pdb/query' },
    { 'path' => '/api/pdb/meta', 'url' => 'http://localhost:8080/pdb/meta' },
    { 'path' => '/api/metrics', 'url' => 'http://localhost:8080/metrics' },
  ],
```


We should check if "Repeat" is the label we want, or perhaps "Iteration" ? ... in Spanish "repita" does not apply here.


What was happening before this change?


But let's keep this change, since it doesn't have the same downsides as the previous one.


 I noticed that they didn't bother waiting for the callback in the spacewars example too. I'm new to the steamApi and have never seen such a poorly documented commercial api :(

I've used a similar approach to you (synchronous, without waiting for callback), and have twice received empty auth session tickets.  I'm thinking about adding a workaround in my consuming code, where if it's empty it will try again after e.g. 1 sec.


In the backend the property is called `iteration`


This just fails silently. what currently happens is that the files processed/generated by Crisper simply don't get a sourcemap file generated.


@Aloren they are used, but in tests that directly assert behavior linked to them

just didn't want an additional class / library to be used in this one since the feature tested is not directly linked to date conversion


This is like dictionary anna.
if the variable name is in dictionary then it will be added to this class.
This class is created by android studio only.


It's true, if you want I can remake a pull request correctly, sorry for the inconvenience


FYI: There are some hopes to get that fixed in time for the Summit :)


done


I didn't create this file anna  and also i didn't change form JDK_1_8 -> JDK_1_7.
Actually I have pulled the code form server and redesigned the application from the previous commit. that might have changes the settings.


このブランチには辿り着くことある？


Latest [Google Doc version](https://docs.google.com/document/d/1zgnDV54d9iWrqPBzfp41dBhj63nM8s-SZdMuGt0e4Vc/edit)


I also did some research:
The SunCE security provider's implementation of `java.security.KeyPairGenerator`for ECC currently does not care whether you try to initialize it with a curve (supported group) name (using `KeyPairGenerator.initialize(ECGenParameterSpec)`) or provide all domain params yourself (using `KeyPairGenerator.initialize(ECParameterSpec)`). In the former case it does an internal lookup in a Map matching the curve name to the internal `NamedCurve`instance (which extends `java.security.spec.ECParameterSpec`), in the latter case it tries to do the lookup by comparing each and every paramater of the provided `ECParameterSpec`instance.
This means that it doesn't make any sense to use the latter case at all (for now) and we can simply keep a map of the IDs to the curve names in Scandium because the Sun security provider does not support *anonymous* curves based on provided domain params only.
My take on this would therefore be to remove the domain params from Scandium's `SupportedGroup`enum and simply use the curve names to initialize the EC `KeyPairGenerator` and proceed as proposed in your previous comment. What do you think?


Remove this commented line.


@xuexiaokang @clowwindy 之前分析过，可以无视这个错误日志。原因可能是新版本的chrome或什么软件可能处于优化的目的，先建立了连接，然后多少秒后无数据发送后又关闭了连接。


Ok will change the application name to Bullion only.


Hi Jacek,

Could you please move these lines to one line in admin/templates/configuration/etc/sysconfig/qserv?


This is the right place to put C++-specific flags, I had the same problem with -lrumprun_unwind which is why it went in here. And, correct, we don't care about the case where someone is trying to compile C++ code with -gcc.


As i have said that I have pulled the code form server and redesigned the application from the previous commit. that might have changes the settings.


無いね。`else`を書きたくなってしまった。


I understand that we (the Yeti project) should organize the findings into numbered documents. I don't know that this needs to be part of the experiment protocol. Perhaps we should say something like:

> When completed, a report of the findings of the experiment should be made. This will be published as a Yeti document, for example `yeti-experiment-001`. It need not be an extensive document....


Sweet! I really love the plugin btw, nice work :)


Why not just `system cmd || fail`?


By the escape character, I mean "\". At the moment, it is possible to avoid it for the ending pattern, with the last argument, but not with the starting pattern. That could be a solution, but I also lack time to investigate more...

At the moment, the comments and the regexp patterns overlap too much to be usable at the same time: the regexp matches any "/" which could be the start of a comment, the end of a comment, or the math operator. This is why I disabled it, but it is true that it then leaves a lot of unescaped characters...


Remove all changes in stream_protocol.h, they are not relevant.


rename these booleans to enableZetesJob, enableRaveJob etc. since this is not only about displaying the settings


thx, removed in 533fb406697b3af4167a8109d98da72b5037c941


This should be in separate cleanup commit, not relevant to introduced functionality.


well it's not on the caf branch


So what are the requirements for pretending something is a library?

If it's an issue with our makefile we welcome patches but there's no reason to stick the files in utils.


Another thought regarding problem 1) pointed out by you:
The Java Security spec doesn't seem to mandate a minimum set of curves to support. Thus, if we use *secp 256r1* we cannot be sure that this works on every Java 7 platform as well ...


This should be in separate documentation cleanup commit, not relevant to introduced functionality.


Adding the shell to Gitgub


To docscleanup commit.


Último commit do dia 27/08/2015


That sounds reasonable. I'll re-work it.


So this puts the onus on anyone who uses this to have a frontendbuild.sh. I support that; however: what about the case where there's no reason to have one at all, i.e. there is no actual frontend build?


93651760-RowPerObjectReport (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (1812 of 1812 tests passed)

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


so this is the version to stay? because it's green


The job will fail and tell them that it could not find frontendbuild.sh file. We will no longer have to do it in SAG. Life will be better for all of us.


how do you look for changes again??


`_MSC_VER` is `1900` for VS2015. Before I made this change, the test did fail, because it was expecting double-construction, and wasn't finding it.


I am making a comment here


Maybe instead of `stop`, do

`gradient <- colorNumeric(gradient, 0:1)`

That would allow people to just do `addHeatmap(gradient=c("blue", "lime", "red"))` or `addHeatmap(gradient="RdYlBu")`.

And if the gradient argument is something really invalid then colorNumeric will throw an error for you.


another comment


Just wanted to note that this is the correct order: CommitInstantiated is setting the state of the entry before the entry is handed to the promise that uses them.


Added back in the Discussed error code and the error icon image. Google Blogs text excluded.


Thanks for quick response and the fix :-)


Hi,
Yes, that helps.

Thank you.


I was about to write you an issue for this file ( I had the older version without htmlspecialchar). It is vulnerable to XSS without the htmlspecialcharacters. I am unsure if the special characters is really fixing it. Testing this file right now.


为啥要让空标签显示出来。。


wrong commit message: actually XML mismatch problem.


My point is that the assertion serves no purpose, since it will never fail: `fast_handshake_state_free` will never change `s`.


is convertmd.jl still necessary?


how about simple "instanceof" check?


Oops, I completely missed the line `if unicodeScalar.value == 0 { break }` - might be able to add a `takeWhile` method to sequences, though