#5507-Polygonal-and-TriangularDoor-have-the-same-value (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Same here, should be `shared_ssl_ctx_t *tls_ctx_out`.


Nice, i suffer from fetch fail


It looks like this should use the `outcome` method.


(I saw this after the one below. The "Same here" refers to my comment on line 907)


我只能说 66666666


s/,/.


hi


Going away to wring Bash's neck.....

I've no idea why the cloned script fails to work unless you cd to the directory, rather than just calling the absolute path (Path lengths??)
Better part of an hour getting that to work.....

This one seems to build, although we've acquired a junk Gedit temporary file along the way......


Type in console `externalAPI.help()` at the site. This API is what you need.


Thanks mate. That's quite a relief.


Yeah I saw. Sorry. :(
We can change this on our end after the merge if you don't want to go through it again. I know you're busy.


Well cock.

On Mon, Mar 14, 2016 at 10:04 PM, Beagle <notifications@github.com> wrote:

> Tests revealed this not to be the case: echo ((int)"xyz" == "xyz" ? "fail"
> : "pass");
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub:
>
> https://github.com/BitLucid/ninjawars/commit/5355df1edac02c9a7755f849e8537b86a0ec3c82#commitcomment-16697146
>

--
+++++++++++++++++++++++++++
BitLucid.com <http://bitlucid.com/> - BitLucid, Inc. - Web & Programming
Consultants
ninjawars.net - My webgame project
github.com/tchalvak/ninjawars - My open source projects
dnaexmosn.deviantart.com - Art Gallery
585-519-7658  - Cell
"Never compare your inside with somebody's outside."


This is 2016 Spring "Data Science and Big Data Analytics" class syllabus


En esta función se vuelve a repetir otra vez las líneas de código de aprobación de vacaciones.
Se puede evitar casi toda la función, de la siguiente forma:
presence:super(wiz...) --> con el super crear todas las presencias

for (cada una de las vacaciones aprobadas)
  si está aprobada
      --> llamar a la función "aprobar vacaciones" con los parámetros requeridos


nah, I can easily find and fix them all.


NP! Glad it worked!


test


Are you still working on the sorting and font awesome path?


Thanks


Y otra vez el bloque de código repetido.

Tampoco entiendo en que momento hay que poner pendiente el estado de la presencia de forma automática. Además creo que si fuera el caso, no tocariamos el calendario, ni las sesiones, sino solo la presencia.
Explícame para qué se usa la función


Otra vez el mismo bloque


It builds on Travis' Linux and OSX workers!

Will see what happens when we try generating a DMG tomorrow :+1:


Shall we rewrite everything to python?

On Mon, Mar 14, 2016 at 10:09 PM, Roy Ronalds <tchalvakspam@gmail.com>
wrote:

> Well cock.
>
> On Mon, Mar 14, 2016 at 10:04 PM, Beagle <notifications@github.com> wrote:
>
>> Tests revealed this not to be the case: echo ((int)"xyz" == "xyz" ?
>> "fail" : "pass");
>>
>> —
>> You are receiving this because you commented.
>> Reply to this email directly or view it on GitHub:
>>
>> https://github.com/BitLucid/ninjawars/commit/5355df1edac02c9a7755f849e8537b86a0ec3c82#commitcomment-16697146
>>
>
>
>
> --
> +++++++++++++++++++++++++++
> BitLucid.com <http://bitlucid.com/> - BitLucid, Inc. - Web & Programming
> Consultants
> ninjawars.net - My webgame project
> github.com/tchalvak/ninjawars - My open source projects
> dnaexmosn.deviantart.com - Art Gallery
> 585-519-7658  - Cell
> "Never compare your inside with somebody's outside."
>

--
+++++++++++++++++++++++++++
BitLucid.com <http://bitlucid.com/> - BitLucid, Inc. - Web & Programming
Consultants
ninjawars.net - My webgame project
github.com/tchalvak/ninjawars - My open source projects
dnaexmosn.deviantart.com - Art Gallery
585-519-7658  - Cell
"Never compare your inside with somebody's outside."


o ruim de ter somente os min é se precisar depurar complica


Esto... todos esos if then else no son mantenibles, sustituye por lo que te he dicho arriba.
Mucho más simple.
Y todos esos warnings fuera. Evita los warnings. Bloquean y no dejan seguir, lo cual no ayuda en la usabilidad del sistema


Olha só, é o que basicamente aquele plugin do facebook que implantamos na redelivre faz


This whole if logic is super confusing, you're checking for 2 or 3 right next to each other so this second if statement will always overwrite the first


Exceeds Expectations！


Ahh, that's why the scores on peoples gametracker profile are so high. Thanks man.


You probably shouldn't want to commit a secret into a public code repo?


It will fail. But I'm developing a seperate module that handles the tracing and just patches `request` and `respond` method in lapin.


日期忘記改囉！有兩個 3/9。xAPIWrapper 用官方最新的 js lib，相關範例可以[參考這個專案](https://github.com/ATeamCamp/viscaWrapper_example)。


I think you are missing the latest debian/changelog entries - there have been many since version 8.7.3-1 and the previous version (now in the apt repo) was 8.8.2-1:
https://github.com/BunsenLabs/bunsen-pipemenus/blob/master/debian/changelog


Exactly.


This looks dangerous.  You are showing your access token to the world.


This data structure passes VocabularyLesson data object all the way down to the lesson selected. Exercise Items and lesson are created in the VocabularLessonActivity.


Endless while loop?


:chicken:


Probably worth dropping a comment here explaining the usage of this annotation.


![image](https://cloud.githubusercontent.com/assets/6647633/13765864/2abf8990-ea98-11e5-890b-daec8ac3a0ee.png)
又变了


Even though you reverted this commit I think you've still merged a lot of trunk in to trusty.  Can you please rebase your patches on just the trusty branch?


El wizard de confirmación del registro
En las fechas de inicio y fin deben mostrar, si el registro tiene fechas --> esas, si no las tiene las del evento y ya está. No hay que validar ni comprobar ni hacer nada ahí.

Ahora... al confirmar el wizard debe comprobar únicamente:
Fecha_inicio >= Fecha evento and Fecha fin <= Fecha evento
   y si es empleado fecha_inicio >= Fecha inicio contrato and fecha_fin <= Fecha fin contrato


priority_queue
1. constant time lookup of the LARGEST, if used for smallest, comparator needs to be ">" true
2. auto cmp = [](ListNode *a, ListNode *b){ return a->val > b->val;};
3. Definition of a priority_queue:
std::priority_queue<ListNode*, std::vector<ListNode*>, declType(cmp)> pr(cmp);


This should be it's own commit to add vmware to the list of supported vendors.


You cant use `cls.err()`
because it wont have LOG_LEVEL and stuff


For debugging purposes this works OK, but this can not be merged into the codebase.  It will cause dual boot to always be activated.  The right way to activate dual boot will be to create a file on your USB stick (/factory/preseed.cfg) with the dual_boot option set to TRUE.  That will be how FID will do it too.


No need to commit the changes to po/ directory.


Looks good


It looks to me like the whitespace might be off here.  If so this will cause problems in python since it requires whitespace to be consistent.


#5507-Polygonal-and-TriangularDoor-have-the-same-value (rraustad) - x86_64-MacOS-10.9-clang: OK (2133 of 2133 tests passed, 0 test warnings)

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


Although unlikely, it's technically possible to get multiple results from this if for some reason multiple partitions had this label.
I think it should be best to break after you find the one you are looking for.


The label should just be "UBUNTU" rather than "/ubuntu"


#5507-Polygonal-and-TriangularDoor-have-the-same-value (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1025 of 1025 tests passed, 0 test warnings)

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


#5507-Polygonal-and-TriangularDoor-have-the-same-value (rraustad) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


disregard my earlier comment, I notice you changed this here.


石井様レビューの際に、相談かもしてもませんが、
「バージョンの表記」は、あえて書かない方が無難だと思います。
（Mininet + Open vSwitch + Ryu 全てに共通して）

バージョンを書く理由としては、
「このバージョンを Ryu-Book は想定していることを示す」
ためかと思いますが、
「基本的に、最新版での実行を想定している」
はずなので、書くのであれば、こっちかな、と。


I think this log should not be here as it will be called for every minibatch.


This commit is really for hiding the profile dropdown button when taking the survey. This keeps the header intact but clean-looking, and keeps the survey associated with the brand/logo.


is this whitespace change intentional?  You might need to adjust this.


こっちを削除してますが、これでは、Ryu-Book が実践できなかったためでしょうか？


yeah, there are a bunch of variations of the game so i wouldn't be surprised if some website said that

the Game.result method in my original implementation encodes this rule


using import will require to declare two variables and two lines of code. Guess it is a trade off between style consistency and efficiency. Anyway, I am cool with either, so I changed it.


Done.  For now, I also put the first iteration from the resulting csv file to the returned object's member called sim.


Cool thanks!


Please don't commit the commented out stuff into the patch.  The VCS will show the history of what was there, so this just confuses the code.


sorted array is much easier in getting their statistics.

However, we can put the sorting in statisticUtil, but in that case, it would sort the array twice in both bin and util.


What is this needed for?


Good catch @nalimilan. Indeed I should avoid merging PRs while on vacation. Thanks for stepping in!


OK. I will delete the unused code in the patch.


直接`", "` を追加する操作をするのではなくて SeparatorInjectorを使ってください。


#5507-Polygonal-and-TriangularDoor-have-the-same-value (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2171 of 2171 tests passed, 0 test warnings)

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


latex-documentation (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (2126 of 2135 tests passed, 19 test warnings)

 * 18 tests had: AUD diffs.
 * 11 tests had: RDD diffs.
 * 10 tests had: EIO diffs.
 * 6 tests had: ERR diffs.
 * 9 tests had: ESO big diffs.
 * 2 tests had: MDD diffs.
 * 2 tests had: MTD diffs.
 * 7 tests had: Table big diffs.
 * 3 tests had: ESO small diffs.
 * 3 tests had: MTR small diffs.
 * 1 test had: Table small diffs.
 * 4 tests had: MTR big diffs.

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


削除されてるなら素直に処理スキップさせたり、そもそも検索結果に含めないほうがいいね


userForm should be checked for error. Use the method hasErrors() for dynamic form.


'Successfully set'

Don't forget to add the word set!


提交模板目录


`std::set<T>` ってrange-based forは使えないんでしたっけ


コミットタイトルが適当なままだったので修正する。


あぁ、イテレータを使いたいからなのか


#5507-Polygonal-and-TriangularDoor-have-the-same-value (rraustad) - i386-Windows-7-VisualStudio-12: OK (2137 of 2139 tests passed, 0 test warnings)

 * 2 tests had: AUD diffs.
 * 2 tests had: DXF diffs.
 * 2 tests had: EIO diffs.
 * 2 tests had: ESO big diffs.
 * 2 tests had: MTR big diffs.
 * 2 tests had: SHD diffs.

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


It should be sqr(moments[1]) here and also line 326 I think.


直接sys.module触ってしまっているので以下に修正する
self._unimport_module('mod')
self._unimport_module('ryu.tests.unit.lib.test_mod.fuga.mod')


文字列操作を明示的にするのではなくて`addCondition`を使ってください。


{object**|**function}


`.cc` では`using namespace std; をしているので `std::` の名前空間修飾子は不要です。


@theo-armour ,

I find your recipe for a good online 3D interface compelling and the use of only javascript + three.js to take care of everything very elegant.  While I appreciate the vast array of markup languages, libraries, and open source packages that web development offers, I am realizing that the downside can be a tendency to rely on all these resources to do things that can be done much more simply and quickly without them.  I intend to learn from your example for ladybug-web and I see myself translating many ladybug functions into javascript in the future in a manner that is in line with what you describe.

It seems that the three.js API has grown since the last time I checked it out!  Do you know if they have methods for computing Mesh/Ray intersections yet?

Also, Theo, I apologize fore getting your name wrong earlier on this post.  I don't know why I thought Theo was short for Theodore.

Thanks for sharing your knowledge!
-Chris


The session should be cleared if it contains information but the userID is not found. This is problem if user was somehow removed from the database after session was created. The session will contain an userID that no longer exist. Will discuss in more detail in person.


practice pushing


Esto sobra. Fuera warnings.
Si ya tiene un registro o está asignado a las sesiones, no se hace nada y listo.


agreed, moved.


確かに 0 ならfalseと同じと見なされるので間違いではないのですが、`m_impl->statuses.count(HOST_STAT_ALL) > 0`として欲しいカナ。もしくは、`m_impl->statuses.find(element) != m_impl->statuses.end();`


么么哒 周日有空不


`sinon.js` (aliased to `sinon/pkg/sinon` for enzyme in the config) expects/requires global vars; `imports-loader ` allows for those expected global vars to be injected into the module. See https://github.com/webpack/webpack/issues/304


Finally! I am an idiot!


`std::set<HostStatus> &getStatus(void) const;` としたい可能性がなきにしもあらず。


see below


really？


Esta función sobra ahora.


ok, that good


that parameter to repeat is sooo cute. I cannot believe I did it.


Igualmente sobra


Hey Remco, looks like there's a bug here: the -z switch tests whether the argument is an empty string, but from the code it looks like you're assuming the opposite.

I think you meant something like this, right?
```bash
# use BEAGL_LIB if the BEAGLE library is not in a standard location
if ! [ -z "$BEAGLE_LIB" ]; then
        if [ -z "$BEAST_EXTRA_LIBS" ]; then
                BEAST_EXTRA_LIBS=$BEAST_EXTRA_LIBS:$BEAGLE_LIB
        else
                BEAST_EXTRA_LIBS=$BEAST_EXTRA_LIBS:$BEAGLE_LIB
        fi
fi

# use BEAST_EXTRA_LIBS variable to load BEAGLE and other libraries from non-default locations
# this assumes that the library path contains all these libraries (or are set through LD_LIBRARY_PATH)
if [ -z "$BEAST_EXTRA_LIBS" ]; then
  $JAVA -Xms64m -Xmx4g -cp "$BEAST_LIB/launcher.jar" beast.app.beastapp.BeastLauncher $*
else
  $JAVA -Xms64m -Xmx4g -Djava.library.path=$BEAST_EXTRA_LIBS -cp "$BEAST_LIB/launcher.jar" beast.app.beastapp.BeastLauncher $*
fi
```


This changes plagiarism and recent edits to behave like "DYK-eligible" so all tabs in that section are uniform. This functionality can be removed for recent edits if we assume there will always be recent edits (however, it's cleaner in my dev environment this way).


good wording and check.


$ bundle install [--binstubs=PATH] [--clean] [--deployment] [--frozen]
                 [--full-index] [--gemfile=FILE] [--local] [--no-cache]
                 [--no-prune] [--path=PATH] [--quiet] [--shebang=STRING]
                 [--standalone[=GROUP [GROUP...]] [--system]
                 [--without=GROUP[ GROUP...]] [--retry=NUMBER]
                 [--trust-policy=SECURITYLEVEL]


that would work too except we need one more curly brace


prod is always true


so many params passed to populateBin. Does not feel right. Made the change anyway since it is better than what I was doing.


:)) rất nhiều cách và sáng tạo :+1:


ここはガード句にしたほうが良いよ


sobra. Los wizards, déjalos todos como estaban antes.
Ya te dije que los dejases que de momento no se iban a usar tal cual habiamos pensado en un principio.
Ya no tienen mucho sentido puesto que todo el control de la asignación se hace desde el registro.
No los quitaremos porque quizás nos valgan para hacer cosas y ya que están se dejan, pero complicarlos con controles de fechas y demás, añadiendo tanta línea de código, no merece.


bài làm đầy đủ ạ :)


いや、むしろ、これはこれでOK


what if you have this data:
[10001, 10002, 10003, 10005, 10001]

we need a talk on this.


bài làm tạm ổn, cố gắng làm đủ và dùng hàm def nhá


Se repite `en en` antes del path a scripts.


Je treba ešte pridať scss do layoutu a zmeniť v scss linku na obrázok s assetov


With this commit I'm now getting: No matching method found: write for class clojure.lang.Var


latex-documentation (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2126 of 2135 tests passed, 19 test warnings)

 * 18 tests had: AUD diffs.
 * 11 tests had: RDD diffs.
 * 10 tests had: EIO diffs.
 * 6 tests had: ERR diffs.
 * 9 tests had: ESO big diffs.
 * 2 tests had: MDD diffs.
 * 2 tests had: MTD diffs.
 * 4 tests had: MTR small diffs.
 * 7 tests had: Table big diffs.
 * 3 tests had: ESO small diffs.
 * 1 test had: Table small diffs.
 * 4 tests had: MTR big diffs.

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


Un empleado puede ser cliente o proveedor. No es incompatible. También se pueden poder registrar empresas que tengan datos fiscales.
Quita el domain añadido. No es necesario.


:+1: :+1:


ok


まだ '{' の前にスペースが入っていないことがあるようなので、入れるようにした方が良いです。

```js
while (j < 3) {
```


Ok, got it.

Reduction in the execution time of this version is about 1.251701 seconds (from 2.183744 to 0.932033) .


変にインデントが複雑だからもう少し整理したほうがいいね。
全体的な方針としてエラーは先に返して最後まで処理させないほうが読みやすくなるよ。


For your serial loop, you don't want to poll but instead take advantage of VISA events: http://digital.ni.com/public.nsf/allkb/179E61F87CF8B340862571EA0037CFA3

In Process RS232 Messages.vi, you can do "CN1", "CN2" rather than repeating the code.

As for logging, you need to add them to the queue in addition to processing them:
![capture](https://cloud.githubusercontent.com/assets/239173/13766434/03af9602-ea1f-11e5-99c3-dbf4186bb519.PNG)

Make sure any CAN messages you submit have payload of length 8 bytes. If they don't, they need to be padded.

That's all I see for now.


5113_GHX_8.2-8.3_Transition (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (0 of 0 tests passed, 0 test warnings)

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


5113_GHX_8.2-8.3_Transition (mitchute) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


I debated whether or not to just nuke the clean, not sure why you would always want to clean the dir since `jekyll build` by default will always nuke `/_site`. I wanted to chat with you first before making that change.


Потому что я потом хочу передавать сюда enum, там будет типа CUT или COPY


5113_GHX_8.2-8.3_Transition (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


@royopa @helix84 I accepted suggestions made by Rodrigo.


5113_GHX_8.2-8.3_Transition (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


__API Description file syntax error on line 316__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


5113_GHX_8.2-8.3_Transition (mitchute) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


develop (mjwitte) - x86_64-MacOS-10.9-clang: OK (1579 of 1579 tests passed, 0 test warnings)

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


Отладочные условности все это. Я еще думаю интерфейс напишу какой-нить, чтобы можно было любые файлы откуда угодно переносить. Или не напишу, хз


> The way this switches between array and non-array getters makes me nervous. What if we created separate getters for each component? A pos member with two components would be accessible as pos1 and pos2.

Good idea, that's much cleaner. 64b4fae280f7463cc1f746b3d5cd40542d86d9d2 implements this (but with pos0 and pos1 instead).

> Have you researched / benchmarked to see if getters and setters have perf overhead vs get and set methods?

Yes, I checked this a while back and there was no difference. I'll check again when I write some benchmarks


should be fixed with 0c9ff2404119bc93fe9755fa5ecdf8e8dd53b146.


@richard-scott You're welcome to open an issue on the [raspbian-ua-netinst](https://github.com/debian-pi/raspbian-ua-netinst) repository. Please include a [pastebin](http://pastebin.com) of a copy of the `raspbian-ua-netinst.log`, your `installer-config.txt` and your `post-install.txt` if/when you do.


__API Description file syntax error on line 301__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


@cypressious I'm not sure in what way either of proposed changes is noticeably better than current fix :)


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

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


Requires https://github.com/bewest/decoding-carelink/pull/175


5 had been edited out deliberately. It's the map that needs updating


@bsmithyman  Hey Brendan, during our testing today, @gerhardpratt and I made this change to your HD code. We think this properly computes the half differentiation when compared to the omega output. I tried creating a pull request but it didn't seem to recognize this change, so I opted to leave a comment on the commit instead. Let me know what you think!


Well the fiddly thing here was where to attach it in a useful and non-hacky way. In particular, attaching to GroupOperations doesn't seem like it would be helpful. Therefore, it seems to be an OperationQueue feature, but that could also be a bit intrusive.

To be honest, this may have already served its purpose to us - it proved something about one operation that I'd already suspected. It could be a useful addition, though, since the observer pattern lends itself so well to this kind of thing. The problem is making it available in a non-intrusive way, and dealing with the fact it makes some big assumptions (ie. OperationNames are basically the class of the operation, and nothing more unique).


I think the portion of the remaining margin attributed to closing out the existing position should be based on the holdings cost, not the holdings value, which would be the value of `GetMaintenanceMargin`. Maybe the holdings value comes into play on the other side (post liquidate)


Nothing in this diff appears to need this - was it just missing before or something? Or did I miss some change?


wonder if maybe we need to disable fingerprintd from ramdisk, and disable fingerprint drivers...might be dependant on TZ_ICCC


removed, will revisit with storage changes


Yeah, but are the GPIO pins you are using really on the same ports?

Spoiler:
No, they are not. ;-)


If you don't use any operations that are specific to `LinearLayout`, then I suggest using the more general type `View`. That way you can revise the layout XML file to a different kind of layout and not need to update the Java source correspondingly.


Well, I added it in a previous revision, so I could use errno in the pam_syslog on line 125.  Ryan had me change it to just use the '%m', which is apparently a GCC shortcut.  I assume the errno.h is still required, but I don't know that for sure.


__API Description file syntax error on line 317__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


There's no need to add the `OnClickListener` each time the list becomes empty -- you could add it once in `onCreateView`.


Thanks! Fernando is OK with bumping up "users" repo version in this PR, so will merge PR as is once Travis is ready (rebased on the latest master due to conflicts - Cargo.toml has been updated updated by another PR as  well).


Picky picky!
(thanks for pointing that out, it's now fixed)


Hmmm okay I was actually wrong on this. I think what happened is that we moved an assignment to a new id. So that is what messed things up.


I don't think diehard is actually a useful metric for us to apply to a stream cipher that has been subjected to nearly a decade of cryptanalysis.  But as it happens, I did apply diehard a few months ago and learned nothing interesting from it -- as I recall, approximately 1 in 20 tests failed, as expected for its significance level.  Is there anything specific you would like to see from a diehard run?

The attached program [chacha8.c.txt](https://github.com/probcomp/crosscat/files/174739/chacha8.c.txt), with the following associated files (URL and SHA-256 hash, retrieved 2016-03-15) for the ChaCha8 reference implementation as submitted to eSTREAM documented at <https://cr.yp.to/chacha.html>, compiled and run with

    mv chacha8.c.txt chacha8.c      # Thanks, Github, for protecting me from uploading .c files.
    cc -o chacha8 chacha8.c chacha.c
    ./chacha8

generates the same test vector.  Is that enough?

https://cr.yp.to/snuffle/ecrypt-config.h 8968678d12ad233bc87b4f05ab45556f9950f8d1aedf573b389a2185e922a0a4
https://cr.yp.to/snuffle/ecrypt-machine.h 3e27f0dd728d9e85e5243a0e707bfa411a5ea53f3e1e6a84ffa4067883e715b9
https://cr.yp.to/snuffle/ecrypt-portable.h
e60507ae6959366200ab51c6e4398ab6cefdfbad954812fca54599488daf9f4c
https://cr.yp.to/streamciphers/timings/estreambench/submissions/salsa20/chacha8/ref/ecrypt-sync.h d5da7674d767ea437095282c57765c747c4158aa3f9a37794d51e2914d6bae42
https://cr.yp.to/streamciphers/timings/estreambench/submissions/salsa20/chacha8/ref/chacha.c
01ccf22253e4ac84fc6565b7873862644cbf6598b606fb4267ede689a5653f35


Right, I see.

Well, I'm thinking that it would be better for the framework to allow the consumer to figure out how she wants to process the `CFAbsoluteTime` values. So, I was thinking that a `Profiler` would be an `OperationObserverType` which you could create giving it an "end point" and then attach to `Operation` instances. It would feed the end point then splits of all the timings.

This end point, is a bit like a `LoggerType` except, instead of receiving `String` input, it would be a more sophisticated structure which represented the performance of the operation.

This way, framework consumers could pipe that into their own analytics/performance systems


Hi Pradip, I did not wish to commit songcircle_cronjobs.php but chose to do so only because I wanted to include the script around line 167 to include field 'verification_key' in the SQL query to be able to include that data in the constructHTMLEmail function. Otherwise, I am not doing any other coding changes to this page until you have had time to work with it. Thanks


Add an h2 header to separate from the extension description above.
Maybe:
## Subscribe
or
## Stay in Touch
or
## RSS Feed and Twitter


oops


done: 0485a9cf1774f50d6b0817b943e50802698a326b


Yes, need to think about how to use subtitle, tho...


I made a change on `nvimux` to allow custom prefix key. you can now set `let g:nvimux_prefix='<C-a>'`.


You can create a new ArrayList of pirates instead


done: 5257bc7fa4a21606a3fdc6fc3cf4b3c20b81d0c8


This method can return the phoneNumber if an entry is phoneToUserName is not found, which makes the caller easier.


As per an *old* email exchange with Raphy. It should be noted  that I didn't update the status of the three reactions involving R-Cys-Gly because the Print_Master_Reactions_From_Files.pl threw an error (https://github.com/ModelSEED/ModelSEEDDatabase/issues/29). A quick look at the three reactions shows that they're likely still not usable anyway.


This might give me really weird text for some expressions.

But if I don't do this, for expression like `Gq|val + 3`, `lookup` actually gave me `+`.


I'm unsure as to why you only assign this in the case when auto_bump is true and then re-read it again below.

Wouldn't it be easier to just do:
```
config.auto_bump = Some(*auto_bump)
if *auto_bump {
...
}
```


Since you are using Username in other places, this variable should be phoneToUsername, or phonesToUsernames


stupid check perhaps check for the first entry which has a duplicate:

```
path = [1,2,3,4,5,6,5]
int firstDupe = find_index_of_first_dupe(path);
// == path idx of first entry that has a duplicate, INT_MAX if no dupe

...
int firstDupe = find_index_of_first_dupe(path); // == 3, INT_MAX if no dupe
for(pIt = path.begin(); pIt < path.end(); ++pIt, ++idx) {
  if (idx >  firstDupe && std::find(...) )
```

Might be a small improvement.


Hi,
What about using StringUtils.join ? (and soon plain String.join, java 8)
cu


Lines 611 and 612: To my knowledge numpy was not imported.


missing the following creature or go id's @Selphius :+1:
go_manaforge_control_console
npc_manaforge_control_console
at_commander_dawnforge ( is this an area trigger? )
npc_maxx_a_million_escort
npc_towercurse_trigger
npc_towerchannel_trigger
npc_towerchanneler_trigger
gehenna_trigger
legion_aldor_trigger
npc_socre_helper
legion_beam_trigger
demolitionist_trigger
npc_energy_ball

it can be that the spawns are also missing from a few, i dont know, and do have all text as far as i know, most where defined in the script itself, so that saved me some work.. can you also say when you have the ID if it are objects or npcs or whatever, because its not always clear in the name :P


@Travers and then??!?

I'm in suspense.


sp: Verify


@tsvtkv Slightly better performance on 2.12: One less reflection call + one less exception thrown


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


The capacity argument wasn't being used anywhere so I just removed it: a7375cc6a12f5376666018d87b79f85c59af294a


I think this needs to be a few lines earlier, the `if (isvalid) {` never closes otherwise for windows


Consider using a "fluent" style, which works because each method on `builder` returns `builder` as its value.
```Java
new AlertDialog.Builder(getActivity())
    .setMessage(...)
    .setPositiveButton(...)
    .setNegativeButton(...)
    .show();
```


If you move maxSpeed steps, better to substract that instead of carry treasure speed.


Hey man! I've been using your fork here because it supports JSON objects. This latest merge breaks on my node server. It throws an 'InvalidContent' error.
```
R-1 =============================== ERROR ================================
R-1 > ERR > 04:14:44:727[server.js:370]
R-1         InvalidContent
R-1 =============================== ERROR ================================
R-1 > ERR > 04:14:44:727[server.js:374]
R-1         InvalidContentError: Invalid JSON: Unexpected token d
R-1     at parseJson (/Users/rob/Projects/wave-app/server/node_modules/restify/lib/plugins/json_body_parser.js:38:18)
R-1     at Server.parseBody (/Users/rob/Projects/wave-app/server/node_modules/restify/lib/plugins/body_parser.js:90:13)
R-1     at next (/Users/rob/Projects/wave-app/server/node_modules/restify/lib/server.js:906:30)
R-1     at f (/Users/rob/Projects/wave-app/server/node_modules/once/once.js:17:25)
R-1     at IncomingMessage.done (/Users/rob/Projects/wave-app/server/node_modules/restify/lib/plugins/body_reader.js:121:13)
R-1     at IncomingMessage.g (events.js:273:16)
R-1     at emitNone (events.js:80:13)
R-1     at IncomingMessage.emit (events.js:179:7)
R-1     at endReadableNT (_stream_readable.js:906:12)
R-1     at nextTickCallbackWith2Args (node.js:474:9)
R-1     at process._tickDomainCallback (node.js:429:17)
```
Anything I can do to help?


I don't like `getMemberComponentString` but also don't want to have `Getter` in the name because it's also used in the setter. Any other ideas?


I like the simplification here


Oh yeah...HW3 needed to be fixed, but slip days do need to be a whitelist. (I can't think of a better way....)


This is great, especially since lodash v4+ is removing the extra contextual argument


https://github.com/Zatvobor/tirexs/pull/207 contains a completely new approach for performing many operations in a single call...


Can use \d for [0-9].  Also you probably want \d+\.\d*\.\d* to make sure you at least have a major.


@tuxitop Thank you for your input, should I simply remove the condition and return layouts[lang] then?

I just tried with more than 2 layouts and indeed it doesn't exceed 1. I am still a bit confused about how [this user](https://forum.ubuntu-fr.org/viewtopic.php?pid=14208851#p14208851) managed to get a LED mask value of 00002002.

Reading about XKeyboardState structure is not helping me much so far.


In this case it seems OK, but I don't know if I've seen a `return` in a `forEach` function. Semantically this is confusing to me. I also think `Array::find` would work here.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/find


Crashing While Adding an Item to Bank, Remove this line for TempFix Or Find a Better Way to handle it.


This is probably effectively an infinite loop.


Just re-decorated BOP Flower Field in e202cca. It features more shrubs, fewer 2-block-tall flowers (actually doing separate passes for 1-block and 2-block-tall flowers), fewer tulips, and more grass.

It's still really saturated with flowers because BOP fills in just about every available block with a tulip, so the only way to avoid total saturation is to not let BOP handle any of the decoration, but... I kinda think we need to let the base biome always do at least some of the decoration, just in case another mod depends on any of the events that get fired.

Still open to suggestions, so let me know if you think it needs anything else decoration-wise.


Good stuff. Well I'm afraid you shouldn't hold your breath for much movement on this one from me (too much deadline), but if it gives you ideas for a better upstream implementation at a later date, then all the better!


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


@jonschlinkert this shouldn't be set here since it'll disable `onLoad` whenever the plugin is added (even if `.src` isn't used).

I think it should be moved down into the `.src` method.


\cc @melgor - I think this may have been hurting our accuracy and am training some new networks now.


How creative :) Nice idea.


man, I should probably have proofread that more


Yeah, that's no worries. What you've done here has just given me some inspiration. In a previous role I wrote a bunch of internal profiling code, and it's not that easy to do. Unless of course you have a framework for executing tasks....

So, I think some sort of profiling tools would be a great feature for post 2.8.0.


@jbrooksuk It's not needed, but it's a good thing to keep it, since we can style icons in context using the ion class.


Makes sense. `createMemberComponentString` would work.


Hi there,

Can you post what error you are seeing on your Android Studio?


FixObjectCounts (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2018 of 2172 tests passed, 1 test warnings)

 * 154 tests had: Table big diffs.
 * 2 tests had: EIO diffs.
 * 1 test had: ESO small diffs.
 * 1 test had: MTR small diffs.
 * 1 test had: ESO big diffs.

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


added constants and a test


removed, will revisit


t_file = pipeline.to_string()
t_file.push_str("metadata.rb")


Resolves #41


will revisit


@GrahamCampbell I'll remove them at the end and squash :)


This does nothing because `s.decode` returns a value rather than replacing the value in place. I think you want `s = s.decode('utf-8')`.


This is a fine commit msg :+1:


moved


cc @vmg


Looks good. It would be nice to update the documentation.


would you prefer I hard code to "", use an origin or k8s specific constant, or split the rule into origin or k8s resources?


I believe I've addressed all comments for new version


Дело в том, что я на клиенте имитирую создание/удаление/изменение пользователей/компаний, потому что сервера нету. Эта функция нужна для генерации id.


Should not do this iteration in multiple places. Just do this in onLoadFinished


Надо сделать сравнимость через перегрузку оператора ==, так логичнее и удобнее


Closes #44


Oh I see what you were trying to do.  Don't you want to find the merge-base between the current branch and pipeline?


Fixed typo.


Also, comments look great! We probably don't need this depth of markup elsewhere in the code just yet, but it's good to have it here because it's a relatively specialist piece of code.

I think you wanted to mention that the Streaming will blow up if you attempt to parse it twice somewhere? Also, a small note about why you `reverse` the `results` in `parseSubJArray` maybe


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

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


Addresses note #1 from referee.


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


@RochesterinNYC added :realworld here #4274


We should only load the UI after this is done, otherwise we will get very inconsistent results depends how fast the contacts get loaded...


I think we should go back a few paces, asking for the encoding in the command line explicitly, fallback default to utf-8, and remove the reading of the data from the treatment of the data.


You also don't want to throw an error - the merge base might not have a cookbook at that path (in the case where a cookbook got renamed or a new cookbook got introduced).  Actually match the error and simply abandon the version bump.


В конструкторе сделать список из ссылок и рендерить его через map?


indexing error here.
`chimodels` has length `maxpeak-1`, but the last index in this loop is `maxpeak-1`.


not while overriding the default recycler template. will add a follow up for @markturansky


Yep, done: 5e0ea7900e5ea9468f2d72d1d247211d76a78d8b


Бери все for в {} а то как-то неочень


@randalburns Merge with microns. I have fixed the nduser issue in that.


Думаю, лучше возвращать экземпляр класса


This is the only change addressing note #8 from referee, suggesting that 2.1 needn't be its own subsection as it only serves as a summary of previous results and motivation. I included it as scaffolding so that later definitions and results may be compared/contrasted with these existing ones. If the referee feels strongly that it should be shortened/eliminated, I can think harder about how to do so without losing the motivation it provides.


> you want me to open a new case

please, do so, yes :+1:   as @Mausy5043 suggested, include a copy of the files listed (a longer description can be found [here](https://github.com/debian-pi/raspbian-ua-netinst/blob/master/CONTRIBUTING.md#reporting-issues))

regarding the "starting to think I'd broken something"... hey, we all break stuff all the time, it's the way we learn :innocent:   so even if you did, don't worry about it, it's either a chance for you to learn, or for all of us to improve the installer :wink:


![2016-03-15_20 40 50](https://cloud.githubusercontent.com/assets/10927161/13793329/4d617016-eaee-11e5-91f0-ef9af48fc170.png)


Tenian ese caracter en el codigo original?


__API Description file syntax error on line 314__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


done effe3fb54d52a9bf788129bb5cd29951836344e2


ok


It would be nice to add relevant [Doxygen-styled comment block](http://developer.lsst.io/en/latest/docs/cpp_docs.html#file-description-comment-for-header-files) describing what this class os meant to do:


@jdubois I made this changes in my jhipster application and I'm still getting "Cache-Control: no-store" in my request headers... any idea why?


Am I missing something?
I don't see what this demo achieves... i.e. I click the checkbox that says "Remove instead of highlight?" but after I check it, it simply stops highliting, it's not 'removing' anything -- just not highliting.


This is temporary to return at least something.
After I change model a bit, I'll fix that.

Also our games don't have "finished dates", only "scheduled games". We probably need to add "game finished" date, as it's theoretically (but very improbable) possible that all games are not scheduled but played, which would break the algorithm.
Also it's pretty possible that someone mistakenly set scheduled date in future (01/12 instead of 12/01), which will cause tournament to hang in "active" for a long time. So probably adding game end it a good idea.


Тут же обычные кнопки с классами. Куда еще проще? Или ты не про конкретно эти кнопки?


I wonder if it would be good to print a warning or error if it doesn't end with a ;. Or just append one if it isn't there. Doing so might reduce risk of user frustration.


Now StacklessLogging is a class I never new we had!  Good one... I'm going to use that!


Doesn't this only bump if the root directory is a cookbook?  It doesn't handle the case where you have a bunch of cookbooks in a directory.  I guess that's a future thing then...


Addresses note #2 from referee. Makes explicit that we are considering spaces `X` with a distinguished point `0`, such as a Euclidean space where the context is clear.


Small tweak to the language.


Done.
Initially it was stated in a bit complicated way ("actual tournaments are active ones, or the last one if there's no active"), then I restated it but didn't clean up.


Addresses note #2 from referee. (see above)


TeamCity LabKey :: Sprint :: Premium :: Git sqlserver 2014 [Build 26](https://teamcity.labkey.org/viewLog.html?buildId=310656&buildTypeId=LabKey_Sprint_2_LabkeyPremiumTrunk_GitSqlserver2014) outcome was **FAILURE**
Summary: Tests failed: 10, passed: 245 Build time: 00:47:20
### Failed tests
```
org.labkey.test.tests.cas.SSOwithCASTest.testSSOwithCASFromHeaderLink: <no details avaliable>

org.labkey.test.tests.cas.SSOwithCASTest.testNewUserSSO: <no details avaliable>

org.labkey.test.tests.cas.SSOwithCASTest.testSSOWithCASFromLoginPage: <no details avaliable>

org.labkey.test.tests.cas.SSOwithCASTest.testCASUnreachable: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testPublicationDetail: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testStudySummaryWindow: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testStudyCards: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testStudyCardStudyLinks: <no details avaliable>

org.labkey.test.tests.trialshare.TrialShareDataFinderTest.testFilterOnStatus: <no details avaliable>

```


Done.


Addresses note #3 from referee.


@lucianleustean I would suggest reverting this change. Peerdependencies still serve a purpose in NPM 3, namely that if you'd try to include Maji in a project without also including a compatible version of backbone, marionette etc you'd get a warning/error. Without Peerdependencies stuff will just break at runtime.


Addresses note #7 from referee.


Wouldn't the change I suggested also apply to that bug? Also when do you think the repositioning/resizing will get re-looked at? I'm planning on submitting a pr to add positions like "top left" etc since we needed that functionality.


Переделать в ключе всех предыдущих замечаний или тут что-то конкретное?


Addresses note #9 from referee.


Addresses note #6 from referee.


__API Description file syntax error on line 314__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


`**#include <iostream>
#include <string>
using namespace std;

int main()
{
	float a, A, B, b, ab,ac, Ab,Cc, AcA;
label1:
	cout << "PI=";
	cin >> a;
	if (a==314)
	{

		label2:
		cout << "PI=3.14";
		cin >> A;
		if(A==1592)
		{

		label3:
			cout << "PI=3.141592";
			cin >> B;
			if (B==6535)
			{

			label4:
				cout<< "PI=3.1415926535";
				cin >> b;
				if (b==3589)
				{

				label5:
					cout << "PI=3.14159265353589";
					cin>> ab;
					if(ab==7932)
					{

					label6:
						cout << "PI=3.141592653535897932";
						cin >> ac;
						if(ac==3846)
					       	{

						label7:
							cout << "PI=3.1415926535358979323846";
							cin >> Ab;
							if (Ab==2643);
                        	{

							label8:	cout <<"PI=3.14159265353589793238462643";
								cin>> Cc;
								if(Cc==3832)
								{

									label9:
									cout << "PI=3.141592653535897932384626433832";
									cin >> AcA;
									if (AcA==7950)
									{

									}else if(AcA!=7950) {cout<< "Gresit\n "; goto label9;}
								}else if(Cc!=3832){cout<< "Gresit\n "; goto label8;}
							}//else if(Ab!=2643) {cout<< "Gresit\n "; goto label7;}

                           }else if(ac!=3846) {cout<< "Gresit\n "; goto label6;}
					   }else if(ab!=7932) {cout<< "Gresit \n"; goto label5;}

					} else if(b!=3589) {cout <<"Gresit\n "; goto label4; }

				}else if(B!=6535) {cout<< "Gresit\n "; goto label3;}
			} else if(A!=1592) {cout <<"Gresit\n "; goto label2;}

		} else if (a!=314) {cout << "Gresit\n "; goto label1;}

}/**`


Addresses note #4 from referee.


this particular one only takes effect for docker-in-docker... should we be setting it empty everywhere?


>  We're not just saying it should be dropped as a browser standard, we're saying it should be dropped (or split and modified) as a web standard.

Personally, I think the future of MathML is an orthogonal issue. I would like to see MathML develop as an XML spec for publishing but I also don't want it to prevent progress on the web where we need something different, imho. But that's getting too deep, I think, for this post.


actually, looks like the defaults are empty


If not all rounds are started, there is still games to play so tournament will be active, and "end_date" not used at all.

end_date is only used when there are no active tournaments, and it contains start date of the last round.
Then the tournament which has the latest end_date will be returned.


FixObjectCounts (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2018 of 2172 tests passed, 0 test warnings)

 * 154 tests had: Table big diffs.

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


test


> Peter, do you mean Safari 9.0.1 not 9.1?

I don't know. Whatever Saucelabs gave me  (https://saucelabs.com/beta/tests/09cc67d582ac482e8dd13a83aba2c79d/watch)


Will proceed with option 1, but as described above, I'll add "finish date" to games first.


Done.


So deprecating dbh.disconnect which is a sane, well named api is ok but deprecating a Perl 5 DBI relict (RaiseError) ported to DBIish isn't?


То-есть, всюду, где передается или конектится юзер, мне полностью прописывать его поля? Или можно вынести их в какой-то файл?


doble import de boto. El sts lo importaria al principio, o lo sacaria llamando a boto.sts


I think that will be good. How does it look from the ground, which is how most players will see it? Those shrubs might look like hedges, which would be a good effect.


Happy to have you on the [Slaask](https://slaask.com) family! :rocket: :heart:


FixObjectCounts (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1980 of 2134 tests passed, 0 test warnings)

 * 154 tests had: Table big diffs.

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


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

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


oh, it seems like theirs, hmm, I'll look some more


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

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


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


There is no such version yet? :-(
https://github.com/markus-git/language-vhdl/blob/master/language-vhdl.cabal#L5


???? como acabas teniendo foo barr ahi?


Тут имитируется загрузка юзера по id из routeParams, а дальше объект юзера расширяется, чтобы у каждого поля(firstName, birthDate...) было не только значение, но и атрибут валидности.


Verify that you actually have a patch version.  vec[2] might not exist if you tool my regex edit above.  Even otherwise, you can receive a version string like "1.." which will cause this to panic if you unwrap.


DockerDaemonContainer -> RuntimeContainer with a default of `/docker-daemon` in https://github.com/kubernetes/kubernetes/commit/38efc837b906d475daf683beb559b90b9a03f5af

Default changed to empty string in https://github.com/kubernetes/kubernetes/commit/51e4ccf10657e0649f9c038e2621915b6ccd647d

RuntimeContainer -> RuntimeCgroups in https://github.com/kubernetes/kubernetes/commit/575812787db7149ed458fda14544dc74a5e4f6e4

So I don't think this is needed any more


closes #602


No changes have been made to address note #5 from the referee. The fact that the index `\alpha` appears without being used in the product does not seem like a problem to me. Similar notation is used all the time, e.g. `\sum_{1\leq i\leq n} 3 = 3n`. It seems most natural notationally to have `X=X_\alpha` imply `\SigmaProd_{\alpha<\kappa}^{z}X_\alpha=\SigmaProd_{\alpha<\kappa}^{z}X` without having to remove the `\alpha<`.


This feels dangerous :(  Unfortunately, I can't really think of an easy way to do this in a safer way.  Do keep in mind that metadata.rb is proper ruby code with comments and all and can do all sorts of hilarious things inside it.


needed until the UI does discovery, see https://github.com/openshift/origin/issues/7998


@benjamg Correct.
So why going the wrong direction? zmq_ctx_shutdown was the first step - almost there ;-) Just adding zmq_ctx_term and everything's fine (hopefully). Why falling back to zmq_ctx_destroy?


Try to store the script hash in the script object and just read it here.


@dillaman  Testing on the latest master (so with your patch included), I still observe my issue with rbd-mirror daemon stops replaying. Some tests that early got stuck succeed now, not sure if it is not just accident though.

The position where it hangs:

```
registered clients:
        [id=, commit_position=[positions=[[object_number=3, tag_tid=2, entry_tid=3], [object_number=2, tag_tid=2, entry_tid=2],
[object_number=1, tag_tid=2, entry_tid=1], [object_number=0, tag_tid=2, entry_tid=0]]]]
        [id=1458b207-6827-4a31-9eee-c91565135985, commit_position=[positions=[[object_number=3, tag_tid=1, entry_tid=95], [objec
t_number=2, tag_tid=1, entry_tid=94], [object_number=1, tag_tid=1, entry_tid=93], [object_number=0, tag_tid=1, entry_tid=92]]]]
```

And after the restart:

```
registered clients:
        [id=, commit_position=[positions=[[object_number=3, tag_tid=2, entry_tid=3], [object_number=2, tag_tid=2, entry_tid=2], [object_number=1, tag_tid=2, entry_tid=1], [object_number=0, tag_tid=2, entry_tid=0]]]]
        [id=1458b207-6827-4a31-9eee-c91565135985, commit_position=[positions=[[object_number=3, tag_tid=2, entry_tid=3], [object_number=2, tag_tid=2, entry_tid=2], [object_number=1, tag_tid=2, entry_tid=1], [object_number=0, tag_tid=2, entry_tid=0]]]]
```


work in progress


Please add boilerplate header to `BinnedWcs.cc` including `// -*- LSST-C++ -*-`


agree modo L


no, eso es una mezcla de tener tres teclados de diferentes idiomas y tamanyos pese a la torpeza natural


What's *guard profiles*?


test


-J was an option that was introduced in FreeBSD 9.0 so this change currently is breaking process promises on FreeBSD 8


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


done in 53b33d7675bec5d4a895182ff4cc2cf61e25ae20


Filled the pig with lead and hollowed out enough space for the collimator+jacket to slide out.


Loving the comments


FixObjectCounts (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1026 of 1026 tests passed, 0 test warnings)

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


develop (mjwitte) - x86_64-MacOS-10.9-clang: OK (1580 of 1580 tests passed, 0 test warnings)

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


Removing this from latexonly temporarily


Could we turn this onClick callback function into a new function? We repeat the same thing three times. :)


Great use of the switch statement - v easy to read. Switches are also v fast.


Shouldn't this only skip the loop for pypy if ser_id is the dill serializer


needed somewhere else?


I just made a new text-based faculty rating question type in Academy and I couldn't figure out why it wasn't displaying. Tracked it down to this special case handling of text questions `question_type = 4`. Either we have to override this view in Academy with `question_type in (4, 102)`, or change how this works.


And removing it from thinkjava.tex


@ChrisMayfield See next commit.


@rfranco10
This is only for displaying whatever the current stamina is.  In the PlayerAttack script is where the stamina regen is located.


Removing this from thinkjava.sty because it should only apply to the latex version.


I like it, though I would keep using `-a`, `-b`, and `-t` flags. It is more logical for me, i.e. for any operation `-t` is a transformation, but whether it is input or output depends on the command (and since someone is going to use this command he is supposed to know what it does). Anyway, that is of course only one of many possible solutions. Feel free to impolement whatever you like.


arch/arm/mach-omap2/board-espresso-connector.c:25:#include <linux/platform_data/fsa9480.h>
arch/arm/mach-omap2/board-espresso10-connector.c:25:#include <linux/platform_data/fsa9480.h>


It really should be "withColumn" (not "with_Relative_Column") here...


appended


Nice commit name? :P


thanx for the cred


back*


@ChrisMayfield

Ok, I'm going crazy here.  I have removed every definition of the code environment I can find, and plastex is still picking up the old definition and ignoring the one I added in the plastex-only section of thinkjava.tex.

If I change the name of the env to jcode, it works as expected.  But not when it is called code.  So it seems to be picking up a definition of code from somewhere.

One option is to replace code with jcode throughout, but I would like to fix the problem rather than work around it.  If you can see something I'm missing, I'm all ears.


Let's invert the condition and use `@AssertTrue`. It'd be confusing to return `false` for an `isValidXXX` method to actually indicate validity.


What's up with all the capitalization changes in Mods/Clutter_SK/Defs/ThingDefs/Books.xml?
Looks like bad grammar there to me (book names starting with small letters etc).


Same as https://github.com/ilayaperumalg/spring-cloud-stream/commit/2823abc50fbf4d16a293ab02afecff1497967cff#commitcomment-16717930


Makes sense @Michael, however for the our current timeline, i have decided
to get this upgrade since it was much easier work than having to go through
the 48 failing tests after upgrading openmrs platform in addresshierarchy
from openmrs.

Thanks

On Tue, Mar 15, 2016 at 8:56 PM, Michael Seaton <notifications@github.com>
wrote:

> Personally, my view would be that, if we are doing significant new work to
> upgrade to 1.11.5 that we consider abandoning this module in favor of the
> widely used "addresshierarchy" module. The addresshierarchy module was
> written based on addresshierarchyrwanda, and is a part of the reference
> application, so moving to it will make any future adaptation of OpenMRS 2.x
> or Bamni more straightforward.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly or view it on GitHub
> <https://github.com/PIH/openmrs-module-addresshierarchyrwanda/commit/55c1e718be94afa407bf49792ba7b38bd4748001#commitcomment-16714681>
>

--

*Kaweesi Joseph*

*Jembi Health Systems NPC* *|  **Uganda*

*Mobile: **+256784994313** | Office: **Software Developer** | Skype:*
*kaweesi.joseph*
*E-mail: kaweesi@jembi.org <kaweesi@jembi.org>*

*This e-mail contains proprietary and confidential information some or all
of which may be legally privileged. It is for the intended recipient only.
If an addressing or transmission error has misdirected this e-mail, please
notify the author by replying to this e-mail and then deleting same. If you
are not the intended recipient you must not use, disclose, distribute,
copy, print or rely on this e-mail. Jembi Health Systems NPO, its
subsidiaries and associated companies is not liable for the security of
information sent by e-mail and accepts no liability of whatsoever nature
for any loss, damage or expense resulting, directly or indirectly, from the
access of this e-mail or any attachments hereto.*


Because it behaves differently than javascript's regular array push method.
`array.push(1, 2, 3)` adds three elements
`structArray.push(1, 2, 3)` adds one element with three members/components

`emplaceBack` is borrowed from c++'s vector where it is used to construct elements in-place at the end of the array. It's not a great name, but I prefer it to something that already has a different meaning.


better leave it unitialized and set the assignment in do turn. that way no need to clear the list


set meeting with Norwegian prime minister appointment agreed on e-mail at anon@getmail.no
time 16.03.2016


what do you need the loop for? you can check availableTreasure.length or size


what options object are you referring to?


Verified unit tests are passing and are no longer storing test templates in the project files.


Some more context: earlier versions of Relay allowed products to manually mark that a mutation error had occurred on a particular record in the store. This was easy to forget and therefore inconsistent. The standard way to check for this now is:

```
function recordHasMutationError(record: RelayRecord): boolean {
  // get the pending transactions that affect a given node
  const transactions = this.props.relay.getPendingTransactions(record);
  // determine if any of them had an error
  return !!transactions && transactions.some(tx => !!tx.getError());
}
```

This diff removes the last vestiges of support for the error status flag.


because indexOf uses strict equality, not deep equality (your line 422 there)


You two should really align your formatter settings :P


Yeah, it looks pretty good from the ground. Not as many tall flowers so it's easier to see, and there does seem to be a bit of a bias towards that hedge effect, which does make for a pretty nice effect, though it was completely unintentional. The terrain could probably do with some variation, but I reckon you've probably already got a few ideas for that. I tried messing around with the surface, giving it the podzol treatment, but it just didn't look right.


getSailOptions and checking the returned location isn't occupied is done in three places in the code.
If it was done in one method, it will be easier to improve the logic later on.


I sent a new pull request.


so where would i find this version?


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


We just reverted the state of the 2 LEDs. Now green indicates power and we use the red for warnings (eg. at 1st boot). If you're not satisfied: we ship with bin2fex/fex2bin since ages and you can easily adjust that yourself. In case you build your own images there's the userpatches directory to adjust this stuff on the fly...


checkout the work tomer has done at
https://github.com/tomerrs/bot/commit/98b7be9a6c70d5f8f7346925ad2a7d87fac83efb


- Minor styling consistency is to stay consistent with usage of single or double quotes for element attribute definitions.
- Good id names are generally more descriptive than a single value. So instead of `id="1"` better practice would be `id="btn1"`. Also I believe only alphanumeric characters are valid id's even though the browser doesn't really complain when that rule is broken. I like to stick to JS variable naming conventions for element id's.
- You don't need spacing between attribute names and the values. If you use Emmett shortcuts it will fill out the preferred styling for you nicely. So instead of `value = "1"` you would have `value="1"


See above about id naming conventions and the use of special characters here.


set to stop spraying any kind of chemicals from air plains or any other machinery,


Too many spaces here.


In retrospect, I am wondering if we can merge this into the existing `MessageConverterConfigurer` and reuse the interceptor there. That would also ensure that the order in which the conversion and header population is deterministic (right now it isn't).


Could we turn this into a function?


Tab line.


Tab line


Set all prices around the world -20% from 16.03.2016


Clean up extra line


I'll comment on that file as well but best to get in a good habit of placing even temporary `.js` files in the `/js` directory. You also don't necessarily need to use dot notation here for path resolution. You can just use relative or absolute paths such as:
- Relative `src="js/cash_register.js`
- Absolute `src="/js/cash_register.js`


Added some tests in 9cdecfeb18beec51c5c7ae9e20d98d9d9f2a4d13 and 556a6e92f880d83d32613d86a1102c2ee5d2d793.  Are they satisfactory to you?


Remove commented out code.


I would have opionions, smh :laughing:
```ruby
msg = <<-ASCII
                 __
    ____  ____ _/ /____  ______ _      __
   / __ \/ __ `/ //_/ / / / __ \ | /| / /
  / /_/ / /_/ / ,< / /_/ / /_/ / |/ |/ /
 / .___/\__,_/_/|_|\__, /\____/|__/|__/  v#{Pakyow::VERSION}
/_/               /____/
ASCII
```


I love the memo area, but why do you have it? Extra feature? :+1:


@MountAthos

Would you be able to test this script with your SHT sensor? I would like to incorporate the code into the new Mycodo, but I don't have a SHT sensor to test with.

You will have to manually edit the pin and click pin at the bottom (pin, clock_pin, voltage):

```sht = SHT(16, 17, '3.5V')```


Set Isis off from 16.03.2016


Accel remarche maintenant, mais bien plus lentement, par contre toujours le fatal error Immédiat :/


Hi
First of all, let me thank you for your work!
I've found your webpage looking for a TR64 implementation of Fritz commands.
I'm skilled in bash, posh and vbscript, but never used perl..
So I've installed it, downloaded and installed all the dependencies, executed build.pl without any error.. but now I'm lost. I expected an executable, but it seems I'm wrong.. so it builds a library it teems?
But once I run example.pl, I got:
"Can't locate object method "find_service" via package "Fritz::Error" at example.pl line 44"

sorry for the newbie question but.. have you got any idea of what's wrong? :)
Thanks!


Remove spaces


Revealing Module pattern - love it! <3


Docstring explaining what should go here vs. the actual L3-plugin would be nice. Also that this is a reusable piece used by out of tree code, so need to maintain backwards compatibility.


no longer a "private" method, so drop the leading underscore?


Need unit-tests for the new ApicDriver class since it will be used by out-of-tree code.


remove commented out code.


@jef-n FYI I'm also seeing the same error @nirvn reported now


Yes. Thank you.


Great job! The app looks like it's mostly working. :+1:

Looking at your commit history - great job. Couple of notes: It would be great to see more commits, and more detailed commit messages. Sometimes, it's difficult to tell what your commits mean.


Heftig bro :+1:


it was fundamentally broken. fixed


безпечніше використовувати .text()


When returning your object literal from the revealing module pattern you don't want to invoke the function on the right side of your key/value pair. You're merely just pointing to a _reference_ to a named function within your closure that you want to expose by a certain name and it will be invoked by that public name from the calling function.

With this example here `keyClick` would be invoked as soon as the compiler reached line 82 but you don't want it invoked until someone says `module.keyClick();`


така конструкція з ретурном то є зле


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


Remove commented out code.


This is ok but always keep in mind that you want to validate that your input is what you expect it to be before performing an operation. Like what if firstNum is 'taco', that would break your program...


This might be better with `get` not `is`.
As many other our booleans.


explain why moves are a necessary variable, remaining actions is defined in the main.


Looks good!


But then you have to escape all the `\` characters and you have extra space on the LHS :-)


Why do this if in `STATE_TRY_UTOPIC ` we will retry all `USED_GUARDS` not in `PRIMARY_GUARDS ` - which includes the dystopic?

Is there any reason for trying again the dystopic `USED_GUARDS` not in `PRIMARY_GUARDS ` which have just failed on the previous state?

I can't see a good reason for doing this unless we skip the dystopic `USED_GUARDS` not in `PRIMARY_GUARDS ` in the first paragraph of §2.2.2.

cc\ @tcz001


FixObjectCounts (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1026 of 1026 tests passed, 0 test warnings)

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


Hmm, interesting, I'll take a look at this later. EvenWurse client isn't even updated to 1.9 yet but I'll see what I can do.


true. getting best location should be a separate function.


This function works, but it appears you removed the input validation that we had added when we first did js-calculator. It's best practice to add input validation to all of your inputs. People WILL abuse your web app, and it's best to be protected from them if they try. :smile:

If you have time add input validation back into your code.


The variable `output` is not defined here so it will actually create it at run time and put it in the global scope since there is no `var` declaration so watch out for that. I think that `return memory` is really all you need from what I can tell.


щ（ﾟДﾟщ）


@trociny What test should I run to locally repeat this? Looks like it fails to switch to the next tag when it should be available.


Where did you get `_screenItemList.size()` as the right operand from? It’s `g_sci->_gfxFrameout->getScreenCount()`.


Same comment as above in regards to `output` variable.


We're not any closer to matching the interface of `emplaceBack` than we are `push`. I'd rather use a slightly different interface for an established domain concept than introduce C++isms to our codebase.


Would the `createVertexAddMethod` and `createElementAddMethod` be moved into buffer.js?

---

I think I prefer them separate since Bucket doesn't need any part of Buffer when creating the arraybuffers. And none of the StructArray methods are needed for rendering. If the StructArrays were moved to the inside of Buffer there would be a worker/data-creation side of Buffer and a rendering side of Buffer that wouldn't really share much.

I might be missing how this fits into the long-term vision for Bucket


Same as above in regards to input validation.


It appears we have a main.js, but it is blank. I would move your code form clickBtn.js into main and remove clickBtn.js. Main is a more common name for the js file that ties everything together. Also - it's bad practice to have blank files in your code.


Please remember to document all the member functions before you are done.


@liggitt was this meant to be a check for `docker`?


I didn't really do anything to the terrain. There were some side effects from various changes I've been making but it was fairly flat before and it still is now. I'll take another look at it once our changes have been merged


Обновления по UI - видел, стало конечно, выглядеть аккуратнее. Здорово.

 Я правильно понимаю, вы просили посмотреть еще задание с clean code? Все остально вроде как у вас отмечено и зачтено.
Если по clean code, то замечания следующие:
а) Именуйте переменные без '_', если это только не
public static final String SAMPLE_OF_CONSTANT_NAME = "...";
> String key_word_in_history = (String) json_object.get("message");

По правилам, переменную лучше назвать _keyWordJsonArray_. А по логике, вообще лучше _jsonMessage_
То же самое касается функций

б) Закрывайте ресурсы. У Вас местами закрыты, а местами - нет :)
> functions.write_history(new FileWriter(FILE_NAME), chat);

...

    public void read_history(FileReader a, ArrayList<Message> al) {
        Object obj = JSONValue.parse(a);
        JSONArray json_array = new JSONArray();

        if(!al.isEmpty()) {
            message_array_to_json_array(json_array, al);;
        }
        System.out.println(json_array);
        json_array.clear();
        json_array = (JSONArray) obj;
        for (Object anArray : json_array) {
            System.out.println(anArray);
        }
    }

в) Не выбрасывайте наверх исключения, обрабатывайте их. Как и с ресрсами, где-то вы обрабатываете исключения, где-то на самый верх выбрасываете и все.

Насчет Вашего вопроса с JS, я к сожалению сегодня не успел, завтра утром постараюсь сделать пример и сбросить Вам


Makes sure docker can talk to a daemon before trying to make docker calls


Too many spaces


#5406-High-Humidity-Control-=-Yes-in-ControllerOutdoorAir (mjwitte) - x86_64-MacOS-10.9-clang: OK (2133 of 2133 tests passed, 0 test warnings)

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


Thanks for making this commit! I totally forgot that I mentioned this fix.


Tab issues on the function declaration.


try 22


Why is the presence of `docker` that can't talk to the daemon not a failure case?


@jdubois Are you sure it works with maven? I think we have a multiple problems with

```java
if (getClass().getProtectionDomain().getCodeSource().getLocation().getProtocol().equals("file")) {
            if (env.acceptsProfiles(Constants.SPRING_PROFILE_PRODUCTION)) {
                container.setDocumentRoot(new File("build/www/"));
            } else if (env.acceptsProfiles(Constants.SPRING_PROFILE_DEVELOPMENT)) {
                container.setDocumentRoot(new File("src/main/webapp/"));
            }
        }
```

in ``WebConfigurer.java``:

* DocumentRoot is gradle specific
* it doesn't seem to work with cloudfoundry

Removing the document root works fine inside cf: http://atomfredejhipster.cfapps.io/#/


I don't have a problem with this at all but just an FYI. Repos generally consist of code files for running the project and not personal notes. For notes, you can comment in code so that you follow your own steps as you write actual code. Again...toootally fine for this but as we go forward people like to see clean repos and we'll get more into ignoring files and what not but just wanted to point it out. The main project README is a good source for other developers on things _they_ should know about your repository.


This is my suggestion:
```
    def _xset(self):
        """
        Check using setxkbmap >= 1.3.0 and xset
        This method works only for the first two predefined layouts.
        """
        ledmask_re = re.compile(r".*LED\smask:\s*\d{4}([01])\d{3}.*",
                                flags=re.DOTALL)
        layouts = self._get_layouts()
        if len(layouts) == 1:
            return layouts[0]
        xset_output = check_output(["xset", "-q"]).decode("utf-8")
        led_mask = re.match(ledmask_re, xset_output).groups(0)[0]
        return layouts[int(led_mask)]
```
notice that I changed the regular expression. and I also changed the "led_mask = ..." line. this would show the first layout if any unexpected number was there and prevents any IndexError.


Great commit - v small and specific. However, I have no idea what you worked on here! Add more detail to your commits so others have a better idea of what you worked on.


This will always return an empty array if the condition is true above. We should just return the converted list.


Tread lightly with me :laughing:
https://gist.github.com/iamvery/5191b18f7b53f8ff8047


Hm. After a little more reading, I agree that `emplace` is a pretty good word for this use case. How do you feel about calling it  "`emplace`"? cc @jfirebaugh

> Inserts a new element into the container by constructing it in-place with the given args if there is no element with the key in the container.
Careful use of emplace allows the new element to be constructed while avoiding unnecessary copy or move operations. The constructor of the new element (i.e. std::pair<const Key, T>) is called with exactly the same arguments as supplied to emplace, forwarded via std::forward<Args>(args)....
No iterators or references are invalidated.


(no this is probably not better at all :joy:)


related to: https://github.com/mapbox/mapbox-gl-js/commit/d40a059ef0f94b10a93f0578658213ac5a21229c#commitcomment-16714675


We can just use `targetTypes` instead.


Hi,
Curious what this is.
Cheerio!

Gabe


Я бы добавила дополнительный словарь синонимов: выбрать по каноническому представителю для каждой группы синонимов и в качестве атрибута вопросу сопоставляла бы не встреченное слово, а полученного по нему канонического представителя.


2086-plant-load-schemes (kbenne) - i386-Windows-7-VisualStudio-12: OK (2370 of 2394 tests passed, 0 test warnings)

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


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


That's great! :+1:


I see there is a `doOver.js` and a `js/doOver.js`. That can be confusing so you'll want to remove probably this one from the repo. You can do `git rm doOver.js` to remove a file that has been committed to the repo previously but not delete the file on disk or just `rm doOver.js` which will delete the file and git will see that it has been removed but you still have to add/commit that change.


Czemu final skoro zmienna zmieniasz w następnej pętli? Chyba że znowu coś zapomniałem xdd
I można użyć googla, Lists.newArrayList();


This line can be first in the method and I think it will read a bit cleaner.


@arbetts I think this change breaks some tests (It is a fast check from my smartphone so perhaps I am wrong)


Seems like I cannot add inline notes on a smartphone...
 -  I suggest you add a log.info in [line 197](https://github.com/mrworf/iceshelf/blob/master/iceshelf#L197): copying may take a while.
 -  In [line 495](https://github.com/mrworf/iceshelf/blob/master/iceshelf#L495) a `if... or True` disables the if (if I'm not misunderstanding python)
:)


Main method for creating the object and space to occupy


Aby komunikat nie pojawiał się zawsze, proponowałbym sprawdzić, czy ten wpis nie jest już pierwszy:

```
def make_cover_item_first(opftree):
    meta_cover_id = opftree.xpath('//opf:meta[@name="cover"]',
                                  namespaces=OPFNS)[0].get('content')
    cover_item = opftree.xpath('//opf:item[@id="' + meta_cover_id + '"]',
                               namespaces=OPFNS)[0]
    manifest = cover_item.getparent()
    if manifest[0] != cover_item:
        print('* Make cover image item first...')
        manifest.remove(cover_item)
        manifest.insert(0, cover_item)

```


I don't think we should use the Thread context classloader directly here.


This should just be `session[:name] = email`


Test for drawing a letter and hiding it at the end.


Hubot, of course.


Object draws a question mark with a defined color specified by a parameter


Validation was good. Was there a reason to remove?


I think this needs some sort of handling for Linux users so they don't feel left out for now. Maybe direct them to https://github.com/brave/browser-laptop/releases where they can download packages. Not ideal but better than nothing.


For maximum flexibility, we should revert to passing the classloader as argument, and use the application context classloader in the invoking class. But the static method should not make a decision on the classloader to use.


Generally you validate the input to a function and not a module variable but validation is always good!


Object draws an exclamation mark with a color specified by a parameter.


This is a LOT easier to debug with Valgrind now, thanks! :-)


Those two methods are still/again included in the main file.


why are you pushing directly to master????


It's generally considered good practice to not commit `console.log` statements. Just calling it out as an FYI, I know this is work in progress.


Wait, the damn thing is just not working right now :D


![2016-03-16 00-36-12](https://cloud.githubusercontent.com/assets/1431904/13794880/5312a4b4-eb0f-11e5-9c90-df5fc3d68b1e.png)


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


closes*
;)


We could if the types in shader interfaces were strings. https://github.com/mapbox/mapbox-gl-js/blob/d40a059ef0f94b10a93f0578658213ac5a21229c/js/data/bucket/symbol_bucket.js#L41
Is that preferable?


great job


fjgsldjfglsd


5521-EMS-InsideHVACSystemIterationLoop-AirLoop (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2138 of 2138 tests passed, 1 test warnings)

 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.
 * 1 test had: MTR small diffs.
 * 1 test had: Table small diffs.

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


Object draws the letter 'M' in my last name.


The issue with the stock version of ENet is that it's really designed to have enet_host_service() called periodically (as it would be in a game at the start of each frame. Specifically, the retransmission logic requires multiple calls to enet_host_service() to work properly, because the first call won't wake when it needs to retransmit unless it receives a packet from the other host. This obviously won't happen if the lost packet was the initial connect packet. This is possible to solve in common-c by creating some function to encapsulate enet_host_service() but that's not what we're doing yet.


Object draws the letter U in my last name.


@garth
The whole point of debounce is meant to throw away the former events. Definitely there should be a throttle vs a debounce, though.

Note https://gist.github.com/makenova/7885923 each implementation clears the timeout.

http://drupalmotion.com/article/debounce-and-throttle-visual-explanation has a good visual explanation (see the bit where you need to mouse over).

This change turns debounce into what it's meant to be. What you have currently is throttle.


Object draws the letter H in my last name.


Even though this properly finds the element with `id="display"` properly we want to be more explicit with something like `var elem = document.getElementById('display');` so that you know you're referencing the HTML element and not a `var display` that may be in scope.


Object draws the letter A in my last name.


Since this is not querying all the known types anymore we can restrict this to fail an assertion on the wrong MimeType.
E.g. passing "application/json" shouldn't return Object.class silently.


Why did you c&p all that code ?


Unnecessary variable declaration.


> Would the createVertexAddMethod and createElementAddMethod be moved into buffer.js?

No, `Buffer` would be a wrapper around `StructArray` (via composition)

> I might be missing how this fits into the long-term vision for Bucket

Every method that remains on the `Buffer` class in this branch I have moved to `Bucket` in the data-driven styling branch, in the spirit of "inline then refactor." How would you feel about entirely dropping the `Buffer` class?


So I think I forgot to add the &-sign to this row. I think this would work..


I also believe we can move this away and isolate inside CompositeMessageConverterFactory.


teehee yeah not sure that's better. I see your point, though (and agree there could be a better way!). Here's how Sidekiq does it:

```ruby
    def self.banner
%q{
         m,
         `$b
    .ss,  $$:         .,d$
    `$$P,d$P'    .,md$P"'
     ,$$$$$bmmd$$$P^'
   .d$$$$$$$$$$P'
   $$^' `"^$$$'       ____  _     _      _    _
   $:     ,$$:       / ___|(_) __| | ___| | _(_) __ _
   `b     :$$        \___ \| |/ _` |/ _ \ |/ / |/ _` |
          $$:         ___) | | (_| |  __/   <| | (_| |
          $$         |____/|_|\__,_|\___|_|\_\_|\__, |
        .d$$                                       |_|
}
    end
```

But I hate having to break the code flow to avoid leading whitespace. We could use the `<<~` heredoc syntax but we need to support < 2.3 (and I still think we'd having the escaping problem).


Does this boolean indicate that a user typed into the search? Where is this boolean set?


to tweak the output. Not sure it was worth, though...


Shh... I accidentally pushed my changes out twice and couldn't find a way to delete the commit.


done 2f78b4196c75aadab701cae536666993107a0bf1


I'm not sure. I know I have problems with my MLS addon in that way as well. Maybe its something dumb I'm doing. I say send a PR and I'll include it. We can go from there.


I don't care either way, I'll change it.


Methods from this line to 610 are practically the same, the only real difference being each method after the 'MUHA' has an added parameter for bold, and size of the leter.


Same here.


Added explanations for the methods used.


It's ok as it is now.


Subtle but you may or may not actually want to return here. If this function is just to update a memory value then you would not include a return and the function would just return `undefined` as opposed to here where it will actually return the new total value in memory. Maybe that's not what you want?


@jakkubu These changes should merge cleanly with your branch.


> We could if the types in shader interfaces were strings. Is that preferable?

Yes, they should be consistent with other `StructArray` declarations.


But why a squirrel for :shipit:


This is the master file


Is `this.props.searchJobs.list` a data structure containing the jobs matching the search terms?


Because a docker daemon isn't required for 90% of our tests. A docker client binary does not imply a connected docker daemon.


The complaint it has is "one line use {}" vs "more than one line use do/end" but it does look weird. :)


> Every method that remains on the Buffer class in this branch I have moved to Bucket in the data-driven styling branch, in the spirit of "inline then refactor." How would you feel about entirely dropping the Buffer class?

sounds good to me. If it's going to be inlined with data-driven styling, does it make sense to just leave it as is until then?


But it seems to me that this is equivalent to what is in [ogit.ml], but the latter reuses the functions of Logs_fmt which is much better. Note that there is a bug in the current Logs_fmt.pp_header, only `Logs.err_style` is being used.


Can't build on Windows since this commit.


Why isn't the function declaration inline here?


What is the reason for emiting an array instead of a dict as is done by the other data providers?


@kostrse Afterglow if I remember correctly. :)


I didn't say I would update it to 1.9, I just wanted to see if it is possible to put this into Wurst 2.20, just for personal purposes.


Ok, problem between chair and keyboard. Have misread the last statement.


Yup, ok, let's merge :)
I see no Open PR for that?


Uważaj z tym. :) SecureRandom jest wolny oraz synchroniczny (wiele wątków nie może go jednocześnie używać). Dla przykładu zobacz sobie czas generowania UUID poprzez `UUID.randomUUID()`.


Fett ass


Daniel, so it could be <field name='name'>digits</field>.. did you tried this approach?


Ah, quite right.
I've just noticed that if the additional sound files are installed, there are Allison recordings of individual letters, so it may sound better to use those instead of flite... if they are installed.
I've corrected the error you point out, and will take a look at using the recorded files, and failing into flite if they fail.


Because each of the methods on `builder` returns `builder`, you could also string them together in the "fluent" style.


Tak się nie robi. Nie rejestruj klasy w jej konstruktorze.


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


Yes, Fritz is a perl library and no real application.
The ``example.pl'' is my personal testbed during development, so it’s not very well documented… There is more documentation available in the modules itself (in the ``lib'' directory).

I have a real application here: https://github.com/mmitch/fritzdial
The readme contains a (very) short installation instruction.

``example.pl'' uses the same file for user/password as fritzdial (as I said, it’s not documented) and it has no error handling as well, so it might be that you got an "unauthorized" error but did not see it.
You could uncomment lines 35/36 and 30/40 to see what’s going on.

Most actions on the FritzBox need authentication, it’s best to add a second user for UPnP acces via the web interface.

The security-port thingy in lines 115-139 should work without authentication, so you could try to remove/comment lines 42-113 and see what happens.

Regards
Christian


Masz to już w Apache Commons, która jest w Bukkicie - `RandomUtils.nextInt(int start, int end);`
http://commons.apache.org/proper/commons-lang/javadocs/api-3.3/org/apache/commons/lang3/RandomUtils.html#nextInt(int, int)


DAMMNNNN 400% better


i'm getting emotional

On Tue, Mar 15, 2016 at 5:21 PM Adam Pearce <notifications@github.com>
wrote:

> https://www.youtube.com/watch?v=zMRh5JPGuqo
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/bizweekgraphics/swoopyarrows/commit/04835d35cd2ad8834fa00dc97d5560465c162616#commitcomment-16718207>
>


Or we can just use null instead and rely on the default classloader supplied by the framework.


Should be mostly handled, I'll watch for these update markers.


FixObjectCounts (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1584 of 1584 tests passed, 0 test warnings)

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


Masz to już w Apache Commons, która jest w Bukkicie - `RandomStringUtils.randomAlphanumeric(int count);`
http://commons.apache.org/proper/commons-lang/javadocs/api-3.3/org/apache/commons/lang3/RandomStringUtils.html#randomAlphanumeric(int)


Uważaj z tabami.


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

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


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

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


@FaridSafi Could you bump the version so we can update the component through npm?


Masz to już w Apache Commons, która jest w Bukkicie - `StringUtils.join(list, ',');`
http://commons.apache.org/proper/commons-lang/javadocs/api-3.3/org/apache/commons/lang3/StringUtils.html#join(java.lang.Iterable, char)


@vaa25 why do we need these `doNothing()`?


It's the same for all types so it doesn't need to be recreated. Recreating it wouldn't make a difference performance-wise but I think this is clearer in some ways. And probably less clear in other ways. If you prefer it inline I can move it up.


couldnt find another place where this is used, but its a good thing to make your code modular nonetheless


- [ ] #2


I'm fine with `emplace` but I think the `Back` makes it a tiny bit clearer about where it is emplaced. Do you think we might ever want to `emplace` to a specific index for dds or anything?


Hm, hier sollte aber ein Punkt hin.


evo ga moj doprinos :)


That was quite a patch jump! Probably worth going to `0.2.0` in the next release.


oh well


> does it make sense to just leave it as is until then?

Yes, I think so. Let's leave it be :+1:


@rafaelwendel Se desculpar pelo que?? Você está correto.


> Do you think we might ever want to emplace to a specific index for dds or anything?

We can cross that bridge when we get there.


Is this necessary?


it's in movePiratesHome.
You can also create another method called move which gets destination and maxMoves willing to spend.
It should call getNoColideMovement, and then update occupied positions, remaining moves and call game.setsail


`emplaceBack` is ok. It gave my little Javascript brain a shock initially but, after thinking about it, it is a good, semantically accurate name.


As we discussed off line, because BaseTypeVisitor is a scanner, super must be called.  So, instead we are going to file a bug against the BaseTypeVisitor to use the correct valueType when checking assignments on string concatenation compound assignments.


@gburton Gary, works to the extent that I can edit other fields in a product and the GTIN will not be filled with zeros.  However when entering a test GTIN, the attempting to delete that, it will not delete.  Keeps populating with the last GTIN entered.


@Nerezine gratuluję startu :)


Given that each of the methods on `builder` prior to `show` returns `builder`, you could also string these together in "fluent" form:
```Java
new AlertDialog.Builder(getActivity())
    .setTitle("Delete Alert")
    .setMessage("Are you sure you want to delete this crime?")
    .setPositiveButton(...)
    .setNegativeButton(...)
    .show();
```


Ah. I forgot this was within a closure. The way it is written is good.


Oh, then sure, I don't mind (take anything you want :P).

On Tue, Mar 15, 2016 at 5:47 PM, ThisTestUser <notifications@github.com>
wrote:

> I didn't say I would update it to 1.9, I just wanted to see if it is
> possible to put this into Wurst 2.20, just for personal purposes.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/null-dev/EvenWurse/commit/a4b77607b42d388a7ffabebd29a389f8802ec4c7#commitcomment-16718649>
>


This does not guarantee that copy was persisted as a new object. It may have updated the column `aasm_state` of a DB row with `id` == `source.id`, however, the `source` object didn't get reloaded from DB, so it still has old, stale, `aasm_state == 'alpha'`.


This is a wrong merge, but should be fine since 0 changed files.


I'll make a PR on the official client later if I have time...

On Tue, Mar 15, 2016 at 6:05 PM, Andy Bao <contactnulldev@gmail.com> wrote:

> Oh, then sure, I don't mind (take anything you want :P).
>
> On Tue, Mar 15, 2016 at 5:47 PM, ThisTestUser <notifications@github.com>
> wrote:
>
>> I didn't say I would update it to 1.9, I just wanted to see if it is
>> possible to put this into Wurst 2.20, just for personal purposes.
>>
>> —
>> You are receiving this because you authored the thread.
>> Reply to this email directly or view it on GitHub
>> <https://github.com/null-dev/EvenWurse/commit/a4b77607b42d388a7ffabebd29a389f8802ec4c7#commitcomment-16718649>
>>
>
>


I'll clean that up. But is there any way to align the `pp_level` output?

```
+3638us git.value     [DEBUG] add_inflated
+3706us git.memory [INFO] Writing b847860c76c1e79eb482da6881a25a5a829e2e3d
+3717us git.value     [DEBUG] add_inflated
```


right. that thing we are doing constantly everywhere


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

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


@koppelaar I published new npm version
Thank you for your commits


<!-- copy and paste. Modify height and width if desired. --><iframe class="tscplayer_inline embeddedObject" name="tsc_player" scrolling="no" frameborder="0" type="text/html" style="overflow:hidden;" src="http://www.screencast.com/users/flybyray/folders/Camtasia Studio/media/cff060d9-0fcb-45bc-85a3-49594f0a8b27/embed" height="720" width="1280" webkitallowfullscreen mozallowfullscreen allowfullscreen></iframe>


Thats expected behaviour.

Question:  Why would a shopowner ever change a GTIN ?


You may find the code simpler if you don't treat both cases as being "OK" with differing values, but rather treat one case as "OK" and the other not.


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


inline comment


5113_GHX_8.2-8.3_Transition (mitchute) - i386-Windows-7-VisualStudio-12: OK (0 of 0 tests passed, 0 test warnings)

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


Consider whether this addition of the `Crime` to the `CrimeLab` might be part of the responsibility of `undoDel`.


프로그래밍 언어 비교 소스코드 저장소 개설


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


Conventionally all-upper-case names are only used for `static final` members, which are used as constants.


The http://www.access-for-all.ch/en/pdf-lab/pdf-accessibility-checker-pac.html link resolves for me and the APH link on visual-audio-contrast7.xml should be http://www.aph.org/research/design-guidelines/ (now titled "APH Guidelines for Print Document Design")


too much blank lines


comments should be in english


In general true.  Couple scenarios though:
1 New shop owner gets to playing around with entering test numbers then finds out he/she is stuck with the test GTIN..unless they phpMyAdmin it
2. Shop owner's eyes are tired and enters the wrong GTIN only to find out that product never had a GTIN to begin with.


A `Toast` might be another option to consider for how to convey the information that nothing was available to undo. Alternatively, might it be better to simply disable the undo menu item when there is nothing to undo?


`$form->isValid()` already calls `$form->isSubmitted()` so the second is superfluous


not blank lines here


remove this copyright


FixObjectCounts (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1584 of 1584 tests passed, 0 test warnings)

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


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