good progress! However, can we do the same using `sparseMMFromFactor` instead of `model.matrix`, thus avoiding the memory overhead of generating a (potentially huge) dense matrix to represent a matrix that's fundamentally sparse?


By the way, I think this change (from commit abe4a4f04970) technically involves undefined behaviour, because the `unsigned char` gets promoted to (`signed`) `int` and thus any value above 128 in `b4[3]` gives a shift into the sign bit.

(Or to put it another way, the test suite stops being [UBSAN](http://clang.llvm.org/docs/UndefinedBehaviorSanitizer.html)-clean with this change;
casting `b4[3]` to `(png_uint32)b4[3]` make UBSAN quiet again.)


Это костыль для чего-то, может написать коммент?


Man, I should have done TDD, but for now I've some tests as last commit, which should fail on your repo.

Anyways, have a look when you get time. I'm moving forward with the changes locally, since I badly needs to ensure data stays same between conversions :smile:


OK


@takahirom
Yes it happened on genymotion only...
![marshmallowpgenymotion](https://cloud.githubusercontent.com/assets/6959245/11876576/c18e51ec-a4ea-11e5-98c5-223cfbfadaf5.png)


Is this supposed to be commented?


this map doesn't seem to make sense to me. why would you want to replace all unique GlobalValues G in Module M with the same GV?


oopsies


fetch doesn't work as you expected. Look [here](http://apidock.com/ruby/Hash/fetch/)


@justenwalker This is what the output from my `net.Interfaces()` call:

[127.0.0.1/8 ::1/128]
[192.168.0.7/24 fe80::12c3:7bff:fe45:a2ff/64]
[]
[]
[10.0.3.1/24 fe80::7801:23ff:fe0a:6a1f/64]
[10.99.99.1/24 fe80::250:56ff:fec0:1/64]
[10.88.88.1/24 fe80::250:56ff:fec0:8/64]
[172.17.0.1/16 fe80::42:27ff:feff:b2cc/64]
[127.0.0.1/8 ::1/128]
[192.168.0.7/24 fe80::12c3:7bff:fe45:a2ff/64]

This is on Ubuntu 14.04.


Magic Number, может лучше вынести куда-то наверх?


We don't have `WorkerGlass::Worker`, do we?


Хах, в одном месте используется 1024, а вдругом 1000, специально чтоб врагов запутать?


@Thunderbottom
Yep, I really recommend it.

I used it when my old android phone was dead and I needed to talk with friend etc. on whatsapp.

It worked flawlessly.


what do you mean?


s/lookup/look up/


値がNULLかどうかで分岐するには大きすぎる違いだと思うので、
```
struct-or-union-specifier
: struct-or-union-definition
| struct-or-union-declaration
;
struct-or-union-definition
: struct-or-union identifier.opt '{' struct-declaration-list '}'
;
struct-or-union-declaration
: struct-or-union identifier
;
```
みたいな感じで分けたほうがいいと思う。


git log for this part of the code was mostly broken by the "initial commit"
anyway. The only change was your PR, I tried to merge b4 but I wasn't fast
enough :-(

On Thu, Dec 17, 2015 at 3:11 PM, Flávio Vinicius Alvares Ceolin <
notifications@github.com> wrote:

> i really don't like this kind of commit, it's really bad for log. I
> usually use log/blame a lot to figure out when/why something was made.
>
> Otherwise, our code style tool was not even complaining about it, but ....
> as it is already done go ahead.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mbelluzzo/soletta/commit/7e7c3ca5aad484b016804955c5e02be2e998b01a#commitcomment-15047178>
> .
>


All right.


Можно в аргумент функции.


remove `callback` block if not needed


s/was/were/ ?


Sympa le commit qui annule mon dernier commit. ^^


OK, I can confirm that this change is outputs the same, but I forgot
to add $MENHEIGHT. Fixed in
https://github.com/wdlkmpx/woof-CE/commit/6ac7ac6a90e993380f178910bc127e343b2c3c1a

I have tested in tahrpup & slacko. Put fixmenus and fixmenus-mod in a
direct, and run this..

Use this code:

	#!/bin/sh
	(
	echo '# ./fixmenus'
	time ./fixmenus

	cp /root/.jwmrc /root/.jwmrc.orig

	echo
	echo '#./fixmenus-mod'
	time ./fixmenus-mod

	diff /root/.jwmrc.orig /root/.jwmrc > fixmenus.diff

	) &> fixmenus.time


pedantic: s/lookup/lookUp/


Not sure I'm following. We have multiple reference in this module to the GV in the shadow module, we want to replace all of them.


OK, I guess I need to test and see if the HTML report still behaves properly.


should it fail should it silently ignore the leftover?
should this be configurable?


I'm not sure what you're referring to with `sparseMMFromFactor`.


No, you still want to make sure it gets revisited.


add translations to `виберіть клас` і т.д.


`ast_make_init_declarator_list` と同様に `vector` を活かしてください。


@masensio be aware that you need to add the exploded aars with an empty src folder to the libs directory so the ANT builds won't fail :smiley:


@mattleff review please!


that s why i said "this kind of commit", anyway, it was my opnion, maybe the others prefer the code style fixed :)


Hmm, yeah you're probably right.


I have no idea what happened here


whups! lost attribution when cleaning up the debug! http://stackoverflow.com/questions/660836/django-way-of-specifying-channel-image-in-rss-feed


Should it be named `shiftedCalendarDateForDate:`?


Looks good.


This `tmpDate` should be `autorelease` to avoid memory leak.


jhjhjhjhj


Usually I also dont like commits fixing coding style, but In this
particular macros, style was a little bit over the top in terms of how
messy it was.

On Thu, Dec 17, 2015 at 3:26 PM, Flávio Vinicius Alvares Ceolin <
notifications@github.com> wrote:

> that s why i said "this kind of commit", anyway, it was my opnion, maybe
> the others prefer the code style fixed :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mbelluzzo/soletta/commit/7e7c3ca5aad484b016804955c5e02be2e998b01a#commitcomment-15047573>
> .
>


why do you need `match`. look [here](http://guides.rubyonrails.org/routing.html#segment-constraints)


look [here](https://github.com/zdubzdab/schoolTest/commit/4934270b5df333d319055213d48ba5e0371d95b2#commitcomment-15047629)


`ast_is_struct_declarator_list` のチェックが抜けています。


Both there :(
But for me **firmware/atmel/E2A47P2_2_04.fw** is missing lol.


I had to change this code to build (gcc (Ubuntu 4.8.4-2ubuntu1~14.04) 4.8.4):

```cpp
        iterator it = std::lower_bound(
            map.begin(), map.end(), key,
            [&](Pair lhs, Pair rhs) -> bool {
                return lhs.first < rhs.key;
            });
```


Hi Jet, how did you do this?


Hmmm, last digit is 4 because of rounding, would it really work?


Why don't you use global variable `utcTZ` from `MAPIApplication.m`?


Ok, hows [this](https://github.com/RByers/dom/commit/fb875c874f270ef0a4f91d981d941aeae5a9d0e6) instead?


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 857](https://teamcity.labkey.org/viewLog.html?buildId=292169&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 24 (2 new), passed: 223 Build time: 00:26:34
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.util.JUnitHeader: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.util.JUnitFooter: <no details avaliable>

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

org.labkey.test.Runner: org.labkey.test.tests.ScrumtimeTest: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testDatasetExport: <no details avaliable>

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

org.labkey.test.tests.DataFinderTest.testSelectingEmptyMeasure: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnStudyNavigator: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testGroupSaveAndLoad: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

##### there are 14 more failed tests, see build details
```


да, я на время тестирования убрал отрицание. Заметил потом. Но из итогового реквеста я вообще убрал эту проверку, тк загрузка файла по сути происходит через обычный сабмит формы в iframe, то есть не по аяксу. Но это можно будет использовать для именно аякс-запросов.


Remove all LM_W(("KZ: traces).


Shouldn't it be required to `autorelease` to avoid memory leak?


Leftover to be removed?


The original intent was to avoid creating the second stack trace, but I see now how that changes the file name and line number of the actual log statement.
I will remove the exception check and always generate the backtrace.


Sorry for the cryptic shorthand. It's defined in [RItools/R/Design.R, on the clusters branch](https://github.com/markmfredrickson/RItools/blob/clusters/R/Design.R#L348).


See the abovementioned comment.


это константа, обозначающая то, какое действие мы делаем. Загрузка картинок на аву, потом добавятся может загрузка обложки поста, и тд


ща напишу комментарий


Consider this for improved style: http://www.w3schools.com/js/js_switch.asp

Unfortunately, ```d3.scale.ordinal().domain(['C','N','O','S','H']).range(['gray', 'blue', 'red', 'yellow', 'white'])``` does not have a default value for inputs outside the domain (they are implicitly added to the domain, recycling values from the range, so
```d3.scale.ordinal().domain(['C','N','O','S','H']).range(['gray', 'blue', 'red', 'yellow', 'white', 'orange', 'orange'])``` will colour the first two "bad" inputs orange and then start over with gray, blue...


Not only LM_W(("KZ:", also this one (look for LM_W in general).


There is risk of forgetting to add this step in the future. A "FIXME" make it more explicit.


I missed the part where the lifetime of the map is the same as the lifetime of the argument G (recursively for all functions they get passed to). ok, yeah this makes sense.


curious, what was the reason for this?


согласен. Я думал так и сделать, но решил другой подход использовать в итоге. Доработаю этот момент


This is a .test created during the merging process? Or a .test previously exsiting in feature/1259_... branch?


Levels change everything...


Yes, it should be using png_get_uint_32, but that's actually a significant change because it swaps the byte sex (and this is what the original bug encountered).   I don't think there is a guarantee that png_uint_32 is 4 bytes in size, but the suggested fix is safe.


Issue: http://www.bforartists.de/content/clean-delete-menu


да, это мой классик старый добрый. По сути, без разницы. Вроде итак понятно его значение. И писать быстрее


Please note that this is the commit responsible for making it *seem* as though there are tons of changes to a few files, however, these are only line-ending changes, and have no  substantive impact on the code.


that is working good but there is a problem...profile owner and viewer  timeline contains public posts too.... and that's not good...users experience is decreasing...


Seems better. Still not quite consistent with dictionary language elsewhere in DOM, but I'm not sure it's in a very good state elsewhere. 😟


Makes sense. Would you agree, @mindfreeze?

Of course it would be nice if we didn't have to use the extra memory when it wasn't necessary...what would the impact be to resize the sketch canvas when the viewport is rotated?


Now it's in there twice -- I just moved it to be in alphabetical order.


Replaced all the ifs with this:

    	switch(sign){
		case 'C':
			return 'gray';

		case 'N':
			return 'blue';

		case 'O':
			return 'red';

		case 'S':
			return 'yellow';

		case 'H':
			return 'white';

		default:
			return 'orange';
	}

Didn't know that JS had switches, but it looks better :-)


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


I ACTUALLY DID THIS CLAYTON LIED


I'm fine with this change. I just ask that we break the line between the clause and expression, as these lines are very long.


print "hello world!"


да, так быстрее. за счет того, что не вычисляется buttons.length на каждой итерации в отличие от традиционного `for (var i = 0 ; i < buttons.length; i++) `


how ican make
tesets on decaf emu
and how to open emulator


I don't think you need this and this would cause a huge memory leak


Please explain what is the goal of these changes?


Ok


ça serait plutôt `git rebase develop`. Si on fait un merge, et s'il y a eu des nouveaux commits sur la branche develop, il y aura un commit de merge sur `branchname`. Cela reste en local, donc on peut refaire un `rebase` à la fin. Mais autant le faire directement.


Does ist make sense to add an err-object to the callback? This callback is later passed to `openGroupSocket` which itself doesn't expect an error-object, only a parser.
So we need to patch `openGroupSocket` to allow an error-param?!


WTF???


@xrmx It actually doesn't make sense to do the uwsgi_signal_registered check as the signal might have been registered, but there might be no cron handler registered.

Adding a loop similar to the one in del_cron that throws an exception when trying to register cron handler for the same signal repeatedly.


Also added more tool tips, and increased default update timer to 5 minutes. (Up from 45 seconds)


I'm currently in the same situation as you were, I have had no phone for almost 2 years now :(


Don't forget ```break;```.


Thank you @wenkesj
I merge it now


Hi Masurov,

I was looking into an issue posted about 2.3 parsing. I had determined that the issue was with that the event map for 2.3 seemed to be backwards, so I made a change in my fork for it.

When I looked at pull requests for the main repo, there was an earlier pull request for the event map file with the reason being that those files are automatically generated from the HL7 message databases. Since I had other changes that would be beneficial to make use of I made a branch for the message changes and kept the master branch on my fork in a good state for my pull request.

Without having those databases to figure out and fix the generation of that 2.3 event map, we can only fix it manually each time until the models/maps are regenerating.


приведение к булену для поддержки любой писанины:
*`data-multiple="true"`
*`data-multiple="1"`
*`data-multiple="multiple"` (как у инпута)
*`data-multiple="yes"` и тд


Version Changed


In fact, neither of them need the equal sign. Thanks for catching this!


fix the indentation.  The style we're following in this project is indent 2, expand tabs as spaces.


**System Functions - Critical**

Your API routes are not structured to standard.
Some of your routers are `/api/roles/:userName` as opposed to `/api/roles/:id` to GET, DELETE nad PUT (read, delete and update).

Check out this [post](http://www.vinaysahni.com/best-practices-for-a-pragmatic-restful-api) to get a clear picture of how  to structure API routes. You could also refer to your Simulation Project https://github.com/andela/vvida


`ghc-report-errors` is a global variable; it should work anyway.


Ok.  I reviewed some of the discussion of other dictionaries (eg. `EventInitDict`) and don't immediately have any ideas for making my text better.  I'm happy to make further changes if you have suggestions though.  This at least seems very precise in terms of defining the algorithm unambiguously (if maybe overly verbose).


There is no self.get_mangled_physical_resource_id, so this line just crashes the lambda


This commit shouldn't be accepted.  You can't, and shouldn't, simply change a migration file especially one for a validated phpBB extension among other reasons.


This looks good


I think we want to keep the BOXED_COMPLEX type


ScalaDoc please


Are we actually installing VS 2013 community? Is there something i am not seeing in this diff?


это я так по-быстрому сделал описание ошибки не в байтах, а в мегабайтах


Cool, parser combinators!


да, для ie


I don't think this trick works with [`and`](http://clojuredocs.org/clojure.core/and)


This should call ```r = PyComplex_FromDoubles(0.0, d)```


I'm not sure, I didn't notice that line.

It was crashing for me without explictly defining the resource_type, so I will investigate that more


r+


copypasta?


Linea 32 deberia ser favorite!!


Less delay in rc control...nothing else. Delay seems to be a big thing now.


Хорошо, но я подразумевал, что окно просмотра обычного кода (которое выше окна для IL-кода) будет растягиваться под освободившуюся область)


I have not tried though, just wrapping my mind around it made me feel if is clearer to me :smile:


**Code Quality - Critical**

You used the function form of `'use strict';` in other scripts, expect in your models.
Your code style and formatting should be consistent.
Also, the function form (inside a closure) is recommended as it put your code base in a private scope.


Just had to follow this wisdom on SO - http://stackoverflow.com/a/17410119/5499522


What about this one?
it's a lower costume


you should remove NonMovableOrCopyable in derived classes to avoid warnings/multiple inheritance of NonMovableOrCopyable


Not required when we return from the function, but i'll add it just so we
follow the spec.

On Thu, Dec 17, 2015 at 6:45 PM koefoed <notifications@github.com> wrote:

> Don't forget break;.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cnrd/IDE15-16/commit/861b41441f3e1895141807423ffe0eca43d10f5f#commitcomment-15047970>
> .
>


@Malabarba oh, of course - sorry! (I'm too used to work standards where long lines are fairly common...)

Feel free to make the change, otherwise I will this evening when I get home.


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


Se borro esta funcion? o se movio a otro lado?


#5252_TinDiscrepancyNewVRFModel (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1450 of 1450 tests passed)

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


Да, я это понял сразу :D Сейчас разбираю сплит-контейнеры. Хочу переделать под них.


should*


It seems that the the repeated PT_UCNC case exists only in CXXR and I ended up deleting the wrong one.  I will push a fix that will make this file identical to the one in CR.


Yay!!!!!


Так вот как они называются...


That's bad.

I am currently waiting for OnePlus One 64 GB to be in stock.


**Code Quality - Critical**

The variable name `mongoose` shouldn't be a predefined variable name. As it should be `required(…)` in any script it's to be used in.


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


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


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


$ - jQuery нельзя использовать!!!


lenght => length


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


Just thinking in the core and containers there is this now as well but it is a method isReady()

Maybe these could be the same for consistency or the others switched to like this?


Pour le diagramme UML, j'ai changé juste une majuscule en minuscule. Rien de bien grave.
En revanche, je pensais utile de définir "lobby" mais  il n'apparait que dans le diagramme UML. Est-ce que ça semble bon comme ça ? Ou est-ce que je rajoute le mot dans le texte ?


Never mind, its because I was using a different class name and resource name.

I will remove this and resubmit


Please don't forget to include *why* something was reverted, so we don't have to go to GitHub to find out! :-)


**Code Quality - Nitpick**

You'll want to use environment variables to handle such parameters.
This will be key as you will be deploying Checkpoint 4 on the cloud.


с 56 по 82 строчку убрать и заменить на getParam. Получится гораздо короче и нагляднее. getRegex можно использовать и с getParam.


One package will cause monstrous dependencies.
* weechat depends on libcurl, libopenssl, zlib, libgnutls, libnettle, libgmp, libidn, libgcrypt, libgpg-error, libncursesw, libintl-full, libpthread, librt,  libiconv-full.
* weechat-plugin-aspell depends on libaspell, libstdcpp (~0.9MB).
* weechat-plugin-perl depends on perl (~2MB).
* weechat-plugin-python depends on python-light (~1,2MB), python-base (~0,7MB), libffi, libbz2

Not sure I have to install this bunch of libs if I not going to use perl or python hooks.


This doesn't seem to work well with multiple attributes.
The SDK will raise "One of the required keys was not given a value" exception.


Это наверно стоит перенести в GetPath. GetResource довольно общая функция, в ней не должно быть таких частностей


:+1: Thank you for this, @avelis!


This would be a proposal for non-blocking `_loadSub()` (only for **packages** under a subthread):

```diff
--- a/lib/module.js
+++ b/lib/module.js
@@ -829,6 +829,13 @@ var jxt = process.binding('jxutils_wrap');

 Module.runMain = function() {
   if (process.subThread) {
+    var isJXPackaged = null;
+    if (!process.isPackaged && !process.isEmbedded) {
+      var jxmain = process.argv[process._MTED ? 2 : 1].trim();
+      isJXPackaged = jxmain && jxmain.length > 3 && jxmain.slice(-3).toLowerCase() === '.jx';
+    }
+    if (process.isPackaged || isJXPackaged)
+      _loadSub(true);
     Module._load('_jx_subs', null, true);
   }
   else if (process._Monitor) {
@@ -846,10 +853,10 @@ Module.runMain = function() {
   process._tickCallback();
 };

-var _loadSub = function() {
+var _loadSub = function(dontWait) {
   if (process.threadId == -1) return;
-
-  if (!isThreadLoadingRoots()) {
+
+  if (!dontWait && !isThreadLoadingRoots()) {
     // since process.reloadModules is called only once per thread,
     // we should wait a little for NativeModule.Roots
     var _start = Date.now();
@@ -1900,7 +1907,7 @@ Module._extensions['.jx'] = function(m, f, pa) {
     Module._extensions['.jsonx'](m, f);
     return;
   }
-
+
   _loadSub();

   if ($uw.existsSource('@' + f)) {
```


 getParam(html, result, 'service_package', /пакеты услуг[^]*?<table[^>]*>[^]*?<td[^>]*>(.*?)<\/td>[^]*?<\/table>/i, replaceTagsAndSpaces);
Так вроде понятнее.


Why would travis run the tests on firefox 31 when 39 was in the config? https://travis-ci.org/medic/medic-webapp/jobs/97495214#L766


omg so that means developement builds are coming again for ppc OSX 10.4 and 10.5? thank you so much :) :+1:


Can you explain this a little more? What is the `pattern` part for?


hey, this is a great catch :) If you'd like to raise a pull request back to my repo I'd be more than happy to push out this fix! Thanks


it wouldn't - i introduced v 39 in the previous commit


Fixes #2


Мне кажется, эти два словаря можно слить в один, минуя строки типа "path"  и т п


Nope we don't have buildbot builds for OS X PPC for completely different reasons.


getParam понимает вложенные массивы. В данном случае можно написать так: [replaceTagsAndSpaces, /[^\d:\-]+/, ' ']


**Test Coverage - Critical**

You should have tests that cover the authorization the User roles pose, as used to limit the user’s ability.


Oh I see.  And what code is failing on firefox 31?


I did and tox works fine but doing `python setup.py install` (in a virtualenv) breaks when trying to install glance. Can you try the same?


#2 This should allow secondary VIPs in the same hsrp group to work properly.   The use case where multiple hsrp groups is going to be problematic, as the BD can only have a single custom mac address to my knowledge.


Or in fact we should add `isEmbedded` too (we might have tasks there):

```js
    if (process.isPackaged || isJXPackaged || process.isEmbedded)
      _loadSub(true);
```


Hello. I am using this package. But i can not add item to transaction. Please help me.
```javascript
public function getCheckoutPaypal()
    {
        $payer = PayPal::Payer();
        $payer->setPaymentMethod('paypal');

        $item = Paypal::Item();
        $item->setName('Pro Plan');
        $item->setDescription('Buy Pro plan from website www.registratiebeheer.nl');
        $item->setPrice(50);
        $item->setCurrency('EUR');

        $itemlist = Paypal::itemList();
        $itemlist->setItems(array($item));

        $amount = PayPal:: Amount();
        $amount->setCurrency('EUR');
        $amount->setTotal(50); // This is the simple way,
        // you can alternatively describe everything in the order separately;
        // Reference the PayPal PHP REST SDK for details.

        $transaction = PayPal::Transaction();
        $transaction->setAmount($amount);
        $transaction->setDescription('Buy Pro plan from website www.registratiebeheer.nl');

        $redirectUrls = PayPal:: RedirectUrls();
        $redirectUrls->setReturnUrl(\URL::to('upgrade/paypal/success'));
        $redirectUrls->setCancelUrl(\URL::to('upgrade/paypal/cancel'));

        $payment = PayPal::Payment();
        $payment->setIntent('sale');
        $payment->setPayer($payer);
        $payment->setRedirectUrls($redirectUrls);
        $payment->setTransactions(array($transaction));

        $response = $payment->create($this->_apiContext);
        $redirectUrl = $response->links[1]->href;

        return Redirect::to( $redirectUrl );
    }
```


vectorを活かしてください。


But what code does that point to, you have it handy?


I'm working on it. Sorry about that.


I don't think so. Probably if it impact enough people, one P1 can be "upgraded" to a P0?


Prove me that offset + newPos is not negative,  and not only because of the silly internal cast to uint64_t that coverity assumes


This needs to go in https://github.com/Event38/ardupilot/blob/E386_Autolanding3/libraries/AP_BattMonitor/AP_BattMonitor.cpp


You bet! Thanks for observation.


the substitution done here
```js
    var replaceJavarosaMediaWithLoaders = function(formDocId, form) {
      form.find('img,video,audio').each(function(i, elem) {
        elem = $(elem);
        var src = elem.attr('src');
        if (!(/^jr:\/\//.test(src))) {
          return;
        }
        // Change URL to fragment to prevent browser trying to load it
        elem.attr('src', '#' + src);
        elem.css('visibility', 'hidden');
        elem.wrap('<div class="loader">');
        DB.get()
          .getAttachment(formDocId, src.substring(5))
          .then(function(blob) {
            var objUrl = ($window.URL || $window.webkitURL).createObjectURL(blob);
            objUrls.push(objUrl);
            elem.attr('src', objUrl);
            elem.css('visibility', '');
            elem.unwrap();
          })
          .catch(function(err) {
            $log.error('Error fetching media file', formDocId, src, err);
          });
      });
    };
```

i'd guess it's to do with the number of cycles of promise resolution performed in the unit test.


aww dang :/  yes I know of some issues http://forums.scummvm.org/viewtopic.php?t=13856 had hoped this would be it. Do you know of progress on this? Would really like to test the new implementations on ppc.


hmm we aren't manually telling it how many promise-ticks to perform


Do we need to know the minimum version of firefox we support?


here's the test, fwiw:
```
    it('replaces img src with obj urls', function(done) {
      Auth.returns(KarmaUtils.mockPromise());
      UserSettings.callsArgWith(0, null, { contact_id: '123' });
      dbQuery.returns(KarmaUtils.mockPromise(null, { rows: [ mockEnketoDoc('ok', 'form-9') ] }));
      transform
        .onFirstCall().returns(KarmaUtils.mockPromise(null, '<div><img src="jr://myimg"></div>'))
        .onSecondCall().returns(KarmaUtils.mockPromise(null, visitForm));
      dbGetAttachment
        .onFirstCall().returns(KarmaUtils.mockPromise(null, 'xmlblob'))
        .onSecondCall().returns(KarmaUtils.mockPromise(null, 'myobjblob'));
      createObjectURL.returns('myobjurl');
      enketoInit.returns([]);
      FileReader.returns(KarmaUtils.mockPromise(null, '<some-blob name="xml"/>'));
      var wrapper = $('<div><div class="container"></div><form></form></div>');
      service
        .render(wrapper, 'ok')
        .then(function() {
          setTimeout(function() {
            // need to wait for async get attachment to complete
            var img = wrapper.find('img').first();
            chai.expect(img.attr('src')).to.equal('myobjurl');
            chai.expect(img.css('visibility')).to.satisfy(function(val) {
              // different browsers return different values but both are equivalent
              return val === '' || val === 'visible';
            });
            chai.expect(transform.callCount).to.equal(2);
            chai.expect(enketoInit.callCount).to.equal(1);
            chai.expect(createObjectURL.callCount).to.equal(1);
            chai.expect(createObjectURL.args[0][0]).to.equal('myobjblob');
            done();
          });
        })
        .catch(done);
    });
```


Hi @feugy - let me know when this pull request is ready for another review.


Why not use `pkill` instead, which is guaranteed to be available (it's from the same package as `kill`, `uptime`, `top`, etc)? It supports doing `pkill -a collectd` just as well and you don't have to go through all that trouble.


= sadl2_path


lalala


great idea


long term support version is 38, and tests pass on that


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


That's going to look weird on the output messages, a random number like that


やはりここは構造を分割すべきだと思います。
```
struct-declarator
: declarator
| bit-field-declarator
;
bit-field-declarator
: declarator.opt ':' constant-expression
;
```


Why removing all noexcept even on clearly noexcept functions?


ohno!!! all those precious milliseconds wasted when looking up _G.string then _G.string.format!!!
:-1:


5379-IDDCurveReferences (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


This is needed for compilation on osx when SDL2 are found on OS X in the system frameworks dir but sadly_path is not set.


Do note that the "base exp yield" is 8 bit.
Also, that it uses "unsigned bites"
So the range of these values are 0 ~ 255.
Anything outside this range will cause a error upon running the "make" command.
I have noticed this and will be pushing the corrections later on today.
Anyhow, here is what has been updated :

-- Modified starter pokemon base stats, level 1 moves, and level up moves

-- Modified route 1 pokemon base stats, level 1 moves, and level up moves

-- Modified hidden items in viridian city and viridian forest

-- Modified celadon's pokemon mart selection

-- Modified the power, type, accuracy, and power points of pokemon attacks

-- Modified the pokemon that rotate at the title screen

-- Modified the pokemon encountered on route 1

-- Modified the title screen to read "ProudEyes"


Great catch, thank you!


Sorry, but this is already done in branch javafx. So please wait until this branch is merged.


addChild
 * should be made non-public as its usage is heavily dependent on mFirst/mLast/mNext (and is probably unsafe to call from anywhere)
 * may be renamed into something else as it could be confused with addChild from WorkParent (I know, different signature, but still)


I would say yes for the general case. We might run into an exceptional situation someday that would justify this, but I wouldn't document it.


#5252_TinDiscrepancyNewVRFModel (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1985 of 1986 tests passed)

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


note: assigning to an empty value is totally valid, there's no (real) need for `$(NULL)`.  It's commonly used as the last element on a `\`-using list so all useful elements end with a `\`, but it's otherwise not very useful.
Not harmful either, don't get me wrong, that's purely a note


Hello,

I guess you should also track remote="cm" name="CyanogenMod/android_packages_services_Telecomm" as this commit breaks the build at the end of installing the telecomm services apk...


This does more than check, it also stops them.


short desc could mention "converts SGR-colored Input to W3C conform HTML-Code"


This is not a very accurate way of checking. If the user has a `vi /etc/collectd/collectd.conf` open, you'll think it's a collectd process.


The emulator is not intended for public consumption just yet.


ここも `struct-or-union-specifier` と同様に構造を分割したほうがいいと思います。
```
enum-specifier
: enum-definition
| enum-declaration
;
enum-definition
: "enum" identifier.opt '{' enumerator-list '}'
;
enum-declaration
: "enum" identifier
;
```


you may want to check the return value of insert and assert if there was a duplicate: while addWork enforces that we don't have duplicates wrt unique names for Work, you should probably enforce a similar invariant at this level as well (insert will not replace an existing item)


Should we change the comment to explain that the neutron stuff isn't to match your physical LAN but the virtual one that you want openstack to setup? They could of course be the same.

Do you also want to move db_pass and rabbit_pass up?


The scaling does not seem to change the actual x, y, z coordinates.

Only affect the actual sphere size.


@cathay4t Is there any reason we can't try something like:

```c
static int vpd_lookup(const char *vpd, char *sd_path, size_t sd_path_len)
{
	int rc = -1;
	char tmp[255];
	char tmp_sd[255];
	snprintf(tmp, sizeof(tmp), "/dev/disk/by-id/wwn-0x%s", vpd);
	if( !access(tmp, F_OK) ) {
		ssize_t result = readlink(tmp, tmp_sd, sizeof(tmp_sd));
		if (result > 0 && result < sd_path_len) {
			const char expected_pre[] = "../../sd";
			tmp_sd[result] = '\0';

			/* Make the full path by making some assumptions and doing some
               sanity checking, readlink -f functionality would be ideal */
			if (0 == strncmp(expected_pre, tmp_sd, sizeof(expected_pre) - 1 )) {
				result = snprintf(sd_path, sd_path_len, "/dev/%s",
                                  &tmp_sd[sizeof(expected_pre) - 3]);
				if (result < sd_path_len - 1) {
					rc = 0;
				}
			}
		}
	}
	return rc;
}
```
first and then fall back to other more involved methods?


vectorを活かしてください。


@afanfa Do you know what the status of the `non-caf-compiler` branch is? Looking at the network this is one of two commits that were never re-integrated into another branch. Are these commits integrated in some other fashion? Would it be safe to delete this stale branch?

CC: @rouson


crucial*


ruleが変わっています。


@akalicki does this need to be changed for consistency's sake?


@afanfa : This is the other commit on `non-caf-compiler` branch that has not been re-integrated...

CC: @rouson


So I just pushed the ```.clang-format``` file that I used then executed

    find . -name '*.cc' -exec clang-format -i {} \;
    find . -name '*.h' -exec clang-format -i {} \;


У меня при попытке отправить письмо не получается. Он просто даже ошибку не выводит, а просто так доолго стоит и обрабатывает..( можете посмотреть пожалуйста.


$sleep needs to be wrapped in double-quotes for paths containing spaces


あっ、こっちが正しいのか。


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1002](http://teamcity.monogame.net/viewLog.html?buildId=12593&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 558, ignored: 6 Build time: 00:23:39


Looks like there was already a `wp` namespace that you can use for this ^


I guess you're being a dick but why wouldn't I keep the same conventions for an entire addon? I don't use .. anywhere in the code, and regardless of whether or not it's faster, there's no point changing it for a pull request


ok i forgive


Why not just define the field as ISO 3166 and then parse everything according to it? So if you find 2 characters, 3 characters, a number or a long string you always know how to parse.


however this is the same exact thing as concatenation, because of luajit


@alexandramedway the `concat` function should not be added to `is_sugar` `func_name` pattern matching because it's not syntactic sugar! It's just a function like `print` or `head`


this is fine. Squash into one of the other four commits for merge.


but now it's not `w.pl`! :O


@chaals Normative definition is at http://ian.hixie.ch/bible/english (first google hit for [hixie english]).


The program actually gets the json. it is written into the String response.


Oui, on peut mettre lobby dans le glossaire, ça serait mieux :)


your main method should contain only one statement:
```
new Directory("c:\tmp").walkFileTree(new FileVisitor());
```
where walkFileTree will play same role as "accept"  method in classic Visitor implementation.

You can see how this pattern is used in this javadoc:
https://docs.oracle.com/javase/7/docs/api/java/nio/file/FileVisitor.html


5379-IDDCurveReferences (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1986 of 1986 tests passed)

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


Вставьте сюда ещё, пожалуйста
```

%%% Продвинутое управление групповыми ссылками (на формулы и т. п.) %%%
```
отбивка пустой строкой и описание группы пакетов ()


I love these fixes :D


clearInstance() still causes us issues each time we build with a custom config.


thanks, it fixed my issue


Was too hasty with the first setColor, this is where it should go.


I am not sure what is the motivation for this. Could explain?


Looks fine.


OK


In builder.py the actual node name is unique and has a _1 appended to it.  Here, the code was trying to find a node with the class name, so it caused an error.


because let's fix what's not broken


Nice!


This makes the 'producerQueue' not thread safe. Think of what would happen if two threads get here at the same time. . .. In any event it is related to the previous comment of making it not final.


offset = size_t
and newPosition >= 0 cause of the || evaluation


Thanks for bringing this up. Can you seems me an example I can work with?
On Dec 17, 2015 1:02 PM, "Tiago Brito" <notifications@github.com> wrote:

> This doesn't seem to work well with multiple attributes.
> The SDK will raise "One of the required keys was not given a value"
> exception.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nicktacular/KeyValueStore/commit/d002331bb201941fd98645d8350271da7fee2834#commitcomment-15048355>
> .
>


hey @nska02. Please take a look at my latest comment under issue #7587
you're probably using ruby-sass just like me. Foundation uses node-sass to transpile scss to css.


```
Используя команду \verb|\labelcref| из пакета \verb|cleveref|, можно
красиво ссылаться сразу на несколько формул
(\labelcref{eq:equation1,eq:equation3,eq:equation2}), даже перепутав
порядок ссылок \verb|(\labelcref{eq:equation1,eq:equation3,eq:equation2})|.
```
Вот на это заменить - исправление одной опечатки и отсутствия двух запятых.


When will this fix be released?  We've run into this all the time with self-hosted OWIN applications, and it's really kind of frustrating to start going through a view to figure out if we've got something broken on a model, or if it is just a partial view that Nancy couldn't find because it didn't get copied over correctly.


So many tests :heart:


can you add some sort of test for these?


This whole block repeats at line 266. Perhaps moving it to a private method for reusability and maintanance?


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


rs=me


@jszczepkowski What would be the reason for moving the HPA example to user guide?
IMO it is very convenient to have sample artifacts in the examples folder. May be the user guide can point to the artifacts in the examples folder. WDYT?
Thanks


this string needs to be capitalized, and also match the @Param field (which is capitalized correctly). It also needs to move to the library because it's not a landing thing, it's a battery thing.


In Rails, the proper way to delete a model is to use the destroy method versus the delete method. The basic difference is that destroy will also call a bunch of callbacks while delete bypasses all that stuff. Stick with destroy.


Ah, good change. I had overlooked that on error the API returns -1 here.


DCO 1.1 Signed-off-by: Vincent Dowling vdowlin@us.ibm.com


welcome back :-)
please, before you start more let's talk,
what we wan't to do next.
I think it's better if we communicate much about each steps of us.

The old RN structure was very terrible an I think that was a result
of too less communication.
Ok ?


Not that a precise value matters, but you can also do this for a very accurate version:

```python
from math import e
```


Also, keep in mind that even if you were to synchronize this method it would still not be thread safe. Because you removed _final_ and not made it _volatile_ there would be no visibility guarantees as far as the updates to this variable. In other words the value set by one thread may not be visible by another.


@kostyfisik Может быть, доработаете ```make distclean``` в соответствии с расширениями, перечисленными в https://github.com/github/gitignore/blob/master/TeX.gitignore (без pdf, eps и подобных файлов рисунков, естественно)? Там они подписаны блоками, так что будет понятно по смыслу, какие именно включать на удаление.


> There is just one thing I don't understand. Why are you suggesting to add the edit back to the diff after removing the conflict?

My original design for this was, to resolve a conflict, you delete all of the edits belonging to that conflict, then re-insert the edits you want to keep.

The analogy with git would be, to resolve a conflict, you first delete the entire conflicting block:
```
<<<
foo
===
bar
>>>
```
Then add either the "foo" or "bar" edit back into the diff.

> Am I overlooking something?

What you said sounds good, but IIRC, `-removeConflict` fully removes *all* of the edits related to the conflict., analogous to deleting the `<< foo === bar >>` conflict block. That's why you have to add the edit back, unless you want to discard both alternatives that were proposed by the two sides of the diff.


Ca, c'est fait en faite :)
Mais, il n'apparait pas dans l'index en réalité. Puisqu'il n'est pas dans le texte mais dans le diagramme UML.


Seems like it is scheduled for Version 2.0


I recommend not including this in the result if the signer list is zero.

Since this is not actually a count, I suggest renaming to your other approach "multi signing": true"


what are the lines 18-26 for?


s/^s/S/


It looks so nice though :) maybe we could ask him to keep it :)


+1 We definitely need more tests especially as comprehensive as this one


Actually, your initial intuition was right. Ruby will evaluate all expressions at run-time, including all parameters sent to methods. Even though `performance_promise` itself short-circuits in production, since `Article.count` would be sent as a parameter, it would be evaluated. This helper will prevent that evaluation in prod.


@Nekotekina  plese don't remove "please report this to a developer." it is a important for user opened issue


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


5.times do | i |
Project.find(i).update(user_id:i)
end


So close!! The reason why this is not working is because you neglected to tell rails what HTTP method to use, so, by default, these links are issuing GET requests versus DELETE. If you include 'method: :delete' I think that would fix it.


I think you're right actually. If you return it like I said above you always get an answer and the for-loop will always exit.
It won't continue up the tree as you'd expect. Sorry!


Thanks, seems that gcc-4.8 doesn't support this sort of type inference. Did 4.8 have trouble with other parts? We use some C++14 features so generally gcc-4.9 or newer is necessary. I'll add this to the README.


I have changed this comment from "Main" to "Master" Sound Output.


Code is fine and You can create pull request for this branch.

My personal personal preferences (you are free to skip them =) ):

1) Don't understand why you need to create instance of Main class. You can easily rewrite your main method without loosing a sense:
```java
public static void main(String[] args) {
    GUIManager guiManager = new XGUIManager(new WindowsDrawApiImpl());
    guiManager.setDrawAPI();
    guiManager.createWindow();
    guiManager.createPanel();
    guiManager.createButton();
}
```

2) I would put some comments in bodies of empty methods:
```java
public void createWidget() {
    // some widget creation code
}
```


no remove this line, the "break" not prevent us if a missing return value.


Thanks, I'll look forward to it!


This condition is not actually good because the Cygwin env may have ncurses installed and thus colors should work properly.


Zet deze buiten de form


I don't see any levels in these files, though...


The lack of a space following identifier in the format is intentional. If an identifier has been defined, there will be a space at the end of the identifier string. This allows the majority use case which does not use the identifier to not have their output affected.


Any reason not to just use `std::string` here?


Looks like it can be fixed by introducing strfile port.


Last update: 17.12 , 20:40 , Roy 🤔


Does the lookup table need both major and minor version numbers? The setup for swiftmodules was intended to eventually become a semver-like version for persistent interfaces, but the lookup table will always be rebuilt with a new compiler anyway.


incorrect , "Unimplemented" it is more clear, so correct this "Unimplemented or unsupported"


I would suggest making the element names lower case.


I think it's pretty clear what this PR is trying to do. See https://nodejs.org/dist/latest-v4.x/docs/api/http.html#http_class_http_agent for more info about http.Agent


Should this be a Set or Collection instead?  List implies some sort of ordering.


Use `file_inode()` here it's the preferred upstream interface and a compat wrapper is already provided for older kernels.


This test I had already made in the feature branch.  Changes was made to it during the merge though


Yeah


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


```
— Otherwise, if the operand that has unsigned integer type has rank greater than or equal to the rank of the type of the other operand, the operand with signed integer type shall be converted to the type of the operand with unsigned integer type
```


Actually, look close at this if 0 / endif.
I wanted to show you this one.

Note that we are in the middle of MongoCommonUpdate.cpp ...


This seems like a good time to get away from the kind of awful name `LookupWtElements`. `elementWeight` would work, or `getElementWeight` if you like function names to be verb phrases. (applied to both, and deprecate the old name).


The scan id is a *path* parameter, not a query parameter.


Why do we need need 'and' and 'or'?  I can only envision a scenario where ALL health indicators must be true.  The composite health indicator should be false if any one of the child indicators is false.


Yes, definitely


enable a ping pong protocol to know if the connection is alive for all websockets. The extension is disable when `false` (default value).The extension is enable when an object is instead provided in parameters.


I'd suggest using octal here, I think it's what users will expect when working with a umask.  The kernel helpers should understand octal so I don't expect any problems, but I haven't verified this.


not sure what `output` means here...


Maybe an error?


Похоже, что надо именно ```\usepackage[russian]{cleveref}```. При компиляции pdflatex после предварительного удаления всех лишних файлов без пометки о языке вставляло частицу to.


Let's make this `static` to make it clear this is a private function.


ti ohuel udalat DRONa?


tente la commande `\glsaddall` avant `\makeglossaries`


It would be good to be more specific: *returns metadata for a single scan, all of its series and all series details*


5379-IDDCurveReferences (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (912 of 912 tests passed)

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


Either `elementSymbol` or `getElementSymbol`, to be consistent with the above.


That's like saying why would I upgrade a Human tier 1 broadsword to a tier 8 broadsword. They have the same dps through all the tiers. Just the Level is different...


Thanks!


And one other thing. Exposing 'producerQueue' to the outside world, breaks encapsulation and exposes this component to a variety of loopholes. Even if its 'protected' it means that my extension can access it and do something with it (e.g., drain it while the parent component is in the middle of some logic that expects it to be not empty). I see that you did it to help with some testing, but that is not the correct motivation for breaking encapsulation. Reflection is perfectly fine to be applied here to access its value if necessary.


We should validate `params: {id: joi.number()}`


It only displayed the changes to the files


Number: the time laps in milliseconds between a pong response and a ping request. default: 2000.


aaaa


Не нужно было, ну да пофиг, если уж народ кричит прям об этом)


no newline at the end of file


You were probably planning on filling this in later. Just making a note here so that we don't forget.


Number: the maximum waiting time between a ping request and the pong response before closing the socket. default: 10000.


#5252_TinDiscrepancyNewVRFModel (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1985 of 1986 tests passed)

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


two empty lines before class definition


The logic feels backwards to me, I think a `zfsdev_cred_check()` function would be clearer and then we check for a failure of that function.


Missing space between `){` (yeah, I know, it's nitpicking).


I would think that we want to get the `Scan` model, `withRelated: 'series.seriesData'`


Похоже, что надо именно ```\usepackage[russian]{cleveref}```. При компиляции pdflatex после предварительного удаления всех лишних файлов без пометки о языке вставляло частицу to.


It looks like a entererly new file in this commit...

Anyway, it doesn't matter. NTC


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


you can declare the return type of the block to be `id` to obviate redundant casting: `^id(WMFTitlesSearchResults* searchResults) { ... }`


Also, remember that the `Scan` model is protected by bookshelf shield, so you need to use the `read` method instead of the `fetch` method.


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


This was using reference equality to compare two boxed primitives. Essentially the previous timestamp would NEVER match the current timestamp.


@Vaibhav-Vyas @rohithvsm  Take a look, i have written the logic of scheduling
We assume spark.cores.max and spark.executors.cores  are specified and not left to defaults [ALL]. Having ALL doesnt make sense when bin packing.


ok


wouldn't "link" be closer to what's going on here? it's as if the user is visiting the disambig page and tapping on a link—but we're presenting it natively.


I think I would rather write `return aWTable[atomicNumber+1].atomicWeight` than modify `atomicNumber` to be something other than the atomic number.


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


@spencerhakim I'm glad it was immediately clear what the PR was intended for. As someone who doesn't use Node, nor is intimately familiar with its constructs or API, it wasn't immediately clear, absent an explanatory note, which the PR lacked. Thanks for pointing me in the right direction.


You can pass in any tags you want.  So for example, this is what I'm calling at one point in this sample:

``` mHub.register(mRegistrationId, "MyTag", "AndroidUser", "AllUsers");```

So if you wanted to be able to push to a specific user, you would first need to have an identifier that uniquely identifies them (username, email address, facebook ID, etc).  You would then just use that as a tag in the register method above (so for example, replace "MyTag" with the username value).  Then on the backend when you want to push, you push to that tag.


After looking closer, that check should be removed. It is the one that ensures that attributes without value, type and metadata are not used in the NGSIv1 API (in NGSIv2 this is allowed, but not in NGSIv1).


I'm wondering what else about an Analysis can we test? Although the analysis schema is part of the "metadata" section, we need to test that something of meaning can be retrieved here. For example, for the WASH7P file, should we expect the VariantAnnotationSet to have an analysis with at least one software record: "SnpEff  -hgvsTrId hg19 - "?


I mean, the check should be keep, sorry.


use `WMFJoinedPropertyParameters` for this


> Without having MT32 at all?
You asked about a real GM instrument, not MT32.  It's not dosemu's fault if a user tells it to use real hardware that does not exist.  (But as I own a MT32, even this use case is valid for me.)

> Both GM and MT32.
I don't understand your objection "having both to virtual".  Why should dosemu care what my MIDI configuration is?  Many people using dosbox are using Munt as a daemon.  dosemu should let me tell it exactly what I want, with defaults that work for most people.

> Yes you do. Others do not.
"Others do not" != "Stas does not".

> How?
I'd show you with a patch, but I have very limited time to work on this and I assume it would be rejected, so my motivation is low.  So here's just my imagination:

blaster.com /ML:
ALSA) ALSA rawmidi default
ALSA_VIRT) ALSA rawmidi virtual
Available output connections:
   0) Timidity daemon on 128:0 (GM type)
   1) Munt daemon on 129:0 (MT32 type)
FLUID) Fluidsynth plugin
TIMIDITY) Timidity plugin,
[..] (munt plugin?)

blaster.com /MALSA
ALSA rawmidi default output selected.

blaster.com /MALSA_VIRT1
ALSA rawmidi virtual output selected (connected to port 128:0 Timidity).


AnalysisResult is unused, but perhaps not for long!


`new` is undefined when `INFO`-level logging is not enabled.


Fixed in ccb9536


Looks good please merge with Master  Branch


testing


Magic numbers should be avoided.  When doing any operations with the mode bits make sure you use the S_* macros in `linux/stat.h` to mask out the bit you need and do comparisons.


Documentation should be a docstring within the function. The only case where we have to use the `#:` notation is for variables, not for functions.


Watch the capitalization of `seriesData`


Should be `const ImagePlug *image`, since this method does not need to share ownership in any way. So we can remove the RefCounted include and the forward declaration change.


there's a convenience method that does the promise adaption for you: `wmf_GETWithSite:parameters:`


ok.
Now, if I keep this check, an attribute update with { } will 'stop working again (for v2).
We need to skip this only if v2 perhaps?


"config/current" hasn't existed for a very long time?


It isn't necessary for this project, but it would be good to add `series: function() { return this.belongsTo('Series', 'series_id'); }`


How about just rename it to RepoTags, which is where this originally comes from https://github.com/fsouza/go-dockerclient/blob/master/image.go#L23

We'd better also rename `kubecontainer.Image.Tags` as well. `Tags` sounds really confusing.


then I can usually compile this code you are here


I think you may be right.
This test was 'un-added' on my feature branch.
Added for this PR.
That's why it 'looks new' - because it is :-)


Nataly, please exclude this log from the repo.


You may need to specify the foreign key: `series_id`


r=MonsieurNicolas


exposed in `boost/math/tr1.hpp` already, should be possible to be removed now.


I think that we should stick with property, it makes a lot more sense!


saw approval from MonsieurNicolas
at https://github.com/graydon/stellar-core/commit/90e75eb520ff10bb55f4bde84ef9f09b9b0a9fe9


merging graydon/stellar-core/history-system-redesign = 90e75eb5 into auto


graydon/stellar-core/history-system-redesign = 90e75eb5 merged ok, testing candidate = 58b8bb44


aacid on the poppler mailing list told me that though there's no way yet, they do have a patch in the queue that adds write-back support for metadata. What do you reckon I should do? Wait for the patch to land or first implement external extractors and use those?


FIXME (and #if 0/#endif removed) in 50fec8e.

Now, perhaps we should try to do something more than just a FIXME ?


I would find a way to symlink the exceptions JSON description into the client and use a named check here, that is the advantage of encoding our exceptions the way we did. Given this is 15.10 this is fine for now, but for future use I'd prefer not using magic constant numbers.


This should be entirely disabled by default to avoid user confusion.  This is a work around, we should be able to trust the permission bits on the device under normal circumstances.


hello


first one


Great start!

Before you can run tests, you will need to add support for this route to the API client (halfpenny). See https://github.com/MRN-Code/halfpenny/blob/master/CONTRIBUTING.md


I don't know if is reliable, as the way in which we're getting the array of "search results" is by grabbing the values of a dictionary, so order isn't defined.  Instead, you should be associating search results w/ title objects, and deriving order from the array of titles. For example:

``` objc
NSArray<MWKTitle*>* titles = [article disambigTitles];
self.disambigTitles = titles;
[self getPreviewForTitles:titles].then(^(NSDictionary<MWKTitle*,MWKSearchResult*> indexedPreviews {
  self.indexedPreviews = indexedPreviews;
  // reload table...
});

// in cell configuration block
// get the title at the specific index of the cell
MWKTitle* titleForIndexPath = self.titles[indexPath.item];
// grab the preview associated w/ that title
MWKSearchResult* preview = self.indexedPreviews[titleForIndexPath];
[cell wmf_setImageURL:preview.thumbURL];
// etc...
```


Habia quedado del xls export, no se usa mas esta funcion, se reeemplazo por la del dashboard.js


avoid submit feature comented


Maybe... I don't like to much but by the moment use if (apiVersion != "v2") or something like that. Keep the FIXME to be sure that this code is refactorized and cleaned in the future


Resume with lesson 21


Do you mind if these data are in the TestData class? I think that's the existing convention.


changing the file


line note


its ok


notes about change


ok


Starts a ping pong protocol to check if the connection stays alive. The protocol closes the connection if the client doesn't reply to the ping request before the end of the timeout. `enabled` boolean, `interval` default: 2000 the time laps in milliseconds between a pong response and a ping request. `timeout` the maximum waiting time between a ping request and the pong response before closing the socket.


I think we'd want to have an "assert Thread.currentThread() == channel.getIoThread();" kind of thing here.  I suspect we may need shutdowns to happen on the write thread via an execute method like was done for the other methods.


DCO 1.1 Signed-off-by:Thomas Wall <walljt@us.ibm.com>


there was a change here introduced after 0.13.9 was cut, with the `???`. That's making it very hard to cherry pick this fix onto 0.13.9.


minor nit:  I would describe this as number of previous intervals to resample.


```python
if LOG_SIZE > 100:
    LOG_SIZE = 1
```


Warning : The websocket client must respond to the ping request with a pong message. That is not necessarily the case if you use an other websocket client than WS.


I have already done a quickfix
The user can active select, which path is the actual path on the settings tab.
But its not uploaded at the moment


after break it'll throw on line 23, so it shouldn't matter.


by description you mean `name`? E.g. `USER_OBJECT_NOT_FOUND` ?


please wait until I have uploaded my last changes


agreed


so, i just realized we already have `WMFArticlePreviewFetcher`, but it's not meant for what you're doing here. It has a bunch of extra stuff for progress & analytics headers, since it was left over from the native view experiment.  That said, I'm thinking you could probably delete that one, and rename this `WMFArticlePreviewFetcher`.


Intelligent mode is always IRQ2 *by default*.  That's because MPU401 was *hard-wired* to IRQ2.  Only later revisions, when it was nearly obsolete already, could be jumpered to a different IRQ (example: MPU-401AT)

A real MPU401 after reset does not work like UART, unless having been placed into UART mode with a 3F command.

I guess some UART mode game you're thinking of will allow the user to select a separate MPU IRQ from the SB16 IRQ.  In that case, I don't think you can prevent the user from doing the wrong thing.


only for Windows ? :(
what about linux and mono-sgen
https://bugzilla.xamarin.com/show_bug.cgi?id=27756

but I think it does not work because
https://bugzilla.xamarin.com/show_bug.cgi?id=30594
:cry:


:+1: Thank you sir!


enable or disable the keepAlive protocol


IMO we should assert that all titles' `[title.site isEqualToSite:site]`. it's possible through the API, but forcing callers to extract all the title's "text" properties is too inconvenient. an assertion should cover the accidental programmer error case


I wasn't actually sure what this did, so I was hoping somebody would correct my comment here.

Maybe we should be renaming this `RESAMPLE` or something like that?


La moitié des membres qui doit approuver, cela risque de beaucoup limiter la prise de décision non ? Tout dépend aussi quelles décisions passent via ce process, mais je te conseille plutôt 1/3, voir pas du tout, si personne s'oppose, on avance. Peut-être proposer une solution si il y a blocage et que plusieurs décisions ont été prise sans s'en sortir. Chez nous, on appliquerait dans ce cas un vote au 2/3 au bout de 3 sessions de prises de décisions non réussies. On a essayé de définir quelques trucs proches pour notre lieu à Lille: https://www.loomio.org/d/9mjdhLf4/choix-du-processus-de-prise-de-decision-de-la-coroutine, que tu retrouves aussi dans les statuts que l'on crée pour une structure juridique porteuse de communs  qu'on doit mettre en place : https://docs.google.com/document/d/1Eh9UCw3kxRCHr4ivYwXL51oC6lYAHXhTPbpU9mmJsLM/edit?pli=1


What's the problem?


the interval of time in milliseconds between a ping request and a pong response. default: 2000.


この2つは一応区別しておいたほうがいいと思う。
下は多分K&R時代の宣言だよね。


@jensens fixed. Still, it is the responsibility of the person who breaks the build to make sure it get's green again, even though if the CI server is responsible.


There were 5 commits on #2297


the maximum waiting time between a ping request and the pong response before closing the socket. default: 10000.


@Viniciuscarvalho O ponto final é melhor ficar depois do link. Senão vai ficar algo como `... e seguir as instruções.página do wiki`


typo in "canolars". Please fix


Если только управляет ссылками на формулы, то весь этот блок описывать "...групповыми ссылками на формулы", а не на "формулы и т. п.". Ведь "и т. п." не настраиваем.


saw approval from paulplace
at https://github.com/basho/eleveldb/commit/e8bac8a10a4f806217ef9f58909b4b00e197597b


merging basho/eleveldb/bugfix/streaming_fold_memory_leak = e8bac8a1 into borshop-integration-175-bugfix/streaming_fold_memory_leak


basho/eleveldb/bugfix/streaming_fold_memory_leak = e8bac8a1 merged ok, testing candidate = e03e8362


thinking about this some more, you could actually simplify a lot of that by instead an array of objects with titles in the same order of the titles that were given. for example, given an array of `title1`, `title2`, `title3`, the caller would get their respective previews back in the same order.

**however**, there _might_ be a case where a preview for a specific title can't be retrieved. i have no idea what the API would do this case, but we could fake it out injecting a phony title in w/ some real ones in the API sandbox to see what we get back. if the API marks the entire request as a failure, we're OK.  OTOH, if it just omits that preview from the response, we might be in for some binary gymnastics to handle that gracefully.


`value="{{EVAL({{VAL(viewOffset)}}/{{VAL(duration)}})}}"`
As far as I can see, you just missed the CMD markers for EVAL. See also /TVMLTemplates/Default/Movie_PrePlay.xml, line 15.


repmgrd shouldn't terminate just because the receive position is before the replay one.

For example, if the standby is stopped and then starts again it will start reading from the beginning of the segment.

Here are logs for what I just explained:

LOG:  consistent recovery state reached at 31DE/DEFEC168
LOG:  invalid record length at 31DE/DEFEC168
LOG:  database system is ready to accept read only connections
LOG:  started streaming WAL from primary at 31DE/DE000000 on timeline 4
LOG:  restartpoint starting: xlog


does this play well with aspect adjustments?


Can we drop the `/` prefix?


So you running without any Hotplugs? IS a 3.4 kernel able to do this properly? I can see with a 3.10 based on as scheduling is different. But with 3.4 based, I still see hotplugging being more efficient than leaving all cores online. But I am open minded and truly want to know what kind of results you are getting. Thanks bro!


use https://github.com/hashrocket/decent_exposure


そうですね
なんと名前つけましょうか
standard-function-declaratorと
k-and-r-function-declarator？


Also, even if the standby has disconnected, we should still keep repmgrd running. Else we have to start it back up each time the standby is stopped (example, a point release update)


Commit v5 local


We generally guarantee that pulling on a property twice returns an identical value, unless the return value is an immutable array.


Could we do `for index in oldArguments.indices` here instead?


this is a great thing :+1:
buuut unaligned code is unaligned


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


when keepAlive is disabled websocket should be pinged 0 time in 700ms


also, if you're returning the promise from `fetch` (which you don't have to, as nobody's using it currently), you should either return the results you were given or `[NSError cancelledError], not `nil` in the case the data source was deallocated.  i've been returning the cancelled error in this case, as IMO it's closest to the intent: somehow this object was deallocated, therefore, we don't care about its fetch results anymore.


Looks like Abizer Nasir is missing in speakers.yml. Could you add him?


Why do you expect `\S` before `\[\d+\]`? It may cause false triggers. Did you mean `\s` instead?


is this a TablePaginationController constant or a file constant? I'm not quite sure how we choose the difference.


use `wmf_setImageWithURL:detectFaces:`, not SDWebImage's category. you'll probably want to detect faces here, as we do in the home view


Oh, just a short experiment with the 1st Gym Leader.

-- Modified Brock's pokemon and levels


when websocket doesn't reply to a ping, WebSocketServer should close it


`function-declarator` と `old-style-function-declarator` でどうだろ。
`array-type-declarator` も `array-declarator` でいいと思う(cf. 規格書の6.5.4.2 Array declarators)。


Ojo que para instanciar la clase Person debes estar en otra clase, en este caso, estabas haciendo la declaración en el mismo archivo, pero fuera de la clase.
Lo mejor es instanciar esta clase dentro de la clase en la que la vas a usar, por ejemplo, en la MainActivity


sorry, i meant to post that on the fetcher class you wrote, not here, but same applies: this should be a preview data source


なるほど。


Lo mismo para este método, debes implementarlo en la clase donde lo vas a usar.


without parameters, setKeepAlive should be executed with default values


Aquí dejé el método que habías escrito en la clase Person


i thought we were going to present article footer "detail views" modally? e.g. language picker, history, etc.?  this was one of the reasons why disclosure indicators were omitted from the cell styling, IIRC


Weird, I really can’t get rugged to play nicely at all.

Could you explain what the two flags were doing:

-DCMAKE_INSTALL_PREFIX=C:/ -DWINHTTP=OFF

Understanding that might help me figure out why my version isn’t running properly.

Sam

> On 17 Dec 2015, at 17:03, Robin Newman <notifications@github.com> wrote:
>
> Yes. I’ve just completed a complete rebuild from scratch using the new ruby location. Worked perfectly (once I corrected win-build-app.bat to reflect the change (see submitted pull request :-)
>
> > On 17 Dec 2015, at 14:52, Sam Aaron <notifications@github.com> wrote:
> >
> > BTW , did you get rugged working correctly? I've tried with Ruby 2.1 and 2.2 and both times it segfaults when I attempt to initialise a repo.
> >
> > —
> > Reply to this email directly or view it on GitHub <https://github.com/samaaron/sonic-pi/commit/e7fde3a9632d22b2ea8c92cb66dbf265aafb7a3e#commitcomment-15043088>.
> >
>
> —
> Reply to this email directly or view it on GitHub.
>


The only consideration I have here is for more complicated expressions (which I know aren't a thing yet).  For example, when definining a function, the second argument has to be a symbol right? So if you try to type in a "string" is should not let you.


oh... nevermind...


if i'm not mistake about modal presentation, they should be wrapped in a navigation controller so that they can have a top toolbar/navigation-bar


cambios agregados...
-icono principal
-icono en el footer
-fotografia del grupo


does this need to be a nsobj?


Aquí haces la declaración de la lista de Person


Congrats you win a prize for finding the Easter egg! (The prize is only virtual) where should I send the PDF?


with true parameter, setKeepAlive should be executed with default values


En esta parte se hace el llamado al método initializeData(), quien es el que crea la lista de personas.


Nice! This means that I can safely remove all the splice() calls in Feathers 3.0 too. :smiley:


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


The idea of the rawTheme is to build the muiTheme.
So I think that the `zIndex` has nothing to do inside the rawTheme, as the `isRtl`.


should set keepAlive with default values


This looks good please merge


黎大神开搞 NLP 了就是吊


Thanks @emmettu!


Hrrrm. You mean like landscape is way smaller shorter so we might want to do something more clever?


should disable keepAlive with false option


Yeah, it needs it to be a stream listener (since autoremoval depends on associated objects).


When keepAlive is disabled websocket should be pinged 0 time in 700ms


did you forget to check in the changes for this?


I looked at pkill documentation and found that it uses regular expression. I did not see that it has an option for exact match.
```pkill -x collectd```  searches for an exact match, similar to the expected 'killall' behavior. I will use this.


I will change the name of the function


`# 8192`


when websocket doesn't reply to a ping, WebSocketServer should close it


Yeah, there were bunch of other similar issues in this file.


`# 4096`


I missed removing this before the commit.


sorry about that, left in my local repo.... will drop it


done


я прям щас пытаюсь раздуплиться


similar quoting..


all tests pass:
success: http://buildbot.bos1/builders/test-eleveldb/builds/553


I think generally we should scope things inside the class where appropriate. I'll move it.

(It's a constant, so ideally it would be static inside TablePaginationController, but you can't have static fields in a generic type).


Yes, ```pgrep``` would do a better job. Thanks.


butter is misspelled


du musst die Datei dann aber auch in deinem commit mit drin haben!


@rohithvsm @Vaibhav-Vyas Take a look at the scheduling logic. We assume we know ``spark.executor.cores`` and `spark.cores.max`, Bin packing does makes sense for their default values which is ALL


GHT_Bugs (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1986 of 1986 tests passed)

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


5379-IDDCurveReferences (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1986 of 1986 tests passed)

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


OK, I'll cherry-pick this onto `api-v3-groups` and continue from there.


+ Dmitri.

The general idea is that there will be a side-car file describing the changes that occur on stdlib symbols. ‘pattern’ is to refer to a common set of changes for multiple symbols, e.g. if you have method

insertElement()
—> base_name_renamed “insert"

and you have insertElement in multiple types, then you specify the ‘pattern’ id to refer to the common change, instead of duplicating it for each symbol.

Things are very TBD at this point.

> On Dec 17, 2015, at 10:03 AM, Jordan Rose <notifications@github.com> wrote:
>
> Can you explain this a little more? What is the pattern part for?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/apple/swift/commit/042efbfb267febc4bf598cc65ce87996b8222a37#commitcomment-15048377>.
>


Should this be applied to master also?


@Memphiz PR'd, now it's out of my control if 6.0.1 is actually ever built ;)


where is the class that has code for converting pdf to images


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/97516880


here you can use sol_str_table_lookup_fallback()

then your code will be like:

    static const struct sol_str_table table[] = {
        SOL_STR_TABLE_ITEM("POST",    SOL_HTTP_METHOD_POST),
      SOL_STR_TABLE_ITEM("GET",     SOL_HTTP_METHOD_GET),
        SOL_STR_TABLE_ITEM("HEAD",    SOL_HTTP_METHOD_HEAD),
       SOL_STR_TABLE_ITEM("PUT",     SOL_HTTP_METHOD_PUT),
        SOL_STR_TABLE_ITEM("TRACE",   SOL_HTTP_METHOD_TRACE),
       SOL_STR_TABLE_ITEM("DELETE",  SOL_HTTP_METHOD_DELETE),
       SOL_STR_TABLE_ITEM("OPTIONS", SOL_HTTP_METHOD_OPTIONS),
       SOL_STR_TABLE_ITEM("CONNECT", SOL_HTTP_METHOD_CONNECT),
        SOL_STR_TABLE_ITEM("PATCH",   SOL_HTTP_METHOD_PATCH),
        { }
};
if (!method) return ....;
return sol_str_table_lookup_fallback(table, sol_str_slice_from_str(method), SOL_HTTP_METHOD_INVALID)
};


GHT_Bugs (mmatts) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@kmiscia Done.


Could you add a feature that when "Copy Link Text" is used it highlights that link.
Reason: as a visual of what link was last copied..
Thanks


You can avoid this by using `TableRegistry::clear()` in tearDown()


Since we have a `PRESERVE` env var above - might want to follow the same conventions it's using and call his `PRESERVE_TESTFILES`.

Also, probably want to use the same approach there with


YOU LEAVE PABST ALONE, you monster


Typooooo


That's just a warning. Not urgent but would be nice to fix. See https://github.com/facebook/react-native/issues/4547


You can use count on the query itself :)


So rather than checking a regex for `/true/` here, just the existence of `PRESERVE_TESTFILES` should be sufficient.


should you set status = SOL_HTTP_STATUS_NOT_IMPLEMENTED as well?


Thanks Ben. Clarified this in 7176e56


Make this variables describtive


We don't want to remove `.so` - those are actually DLLs.  I'm not sure if we're using `lth_cat.exe` anywhere in production (I think just for tests) - but @MikaelSmith would know for sure.


#14 做一半ㄌ


Shouldn't `package.json` also be updated to match?


fast-forwarding master to auto = 58b8bb44


This is what caused #4215 @TheBB


Tried that as well, no dice. Why it does not work is beyond me :)

On 17. prosince 2015 at 20:13:33, iBaa (notifications@github.com) wrote:

value="{{EVAL({{VAL(viewOffset)}}/{{VAL(duration)}})}}"
As far as I can see, you just missed the CMD markers for EVAL. See also /TVMLTemplates/Default/Movie_PrePlay.xml, line 15.

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


FYI @lexpar @ahankinson


```
 Bank Account
    ✓ can be added (11229ms)
    ✓ cannot be added when identity is not set (3648ms)
    ✓ cannot add identity if entity type is invalid (3621ms)
    ✓ cannot add identity if tax id is invalid (3182ms)

  Brand
    ✓ can be retrieved with api key (182ms)
    ✓ cannot be retrieved when using an invalid key (46ms)
    ✓ can get available list (51ms)
    ✓ cannot be retrieved if it does not exist (1113ms)

  Checkout
    ✓ is possible (6730ms)
    ✓ is not possible with invalid cart (4249ms)
    ✓ is not possible with bad url (4400ms)
    ✓ is not possible with bad url (5429ms)
    ✓ is not possible with bad url (5370ms)
    ✓ is not possible when the grand total is to large (3699ms)
    ✓ is not possible with invalid shipping address (4039ms)
    ✓ is not possible when shipping information does not match Paypals shipping information (5397ms)
    ✓ is not possible when state field is to long (4177ms)
    ✓ is not possible if not logged in (3491ms)

  Checkout Confirm
    ✓ is possible (13843ms)
    ✓ not possible when an product in order is tampered with (12971ms)
    ✓ is not possible with invalid order (7355ms)
    ✓ is not possible when logged out (7097ms)

  Checkout Donation
    ✓ is possible (7498ms)
    ✓ is not possible when cart is invalid (7995ms)
    ✓ is not possible when a bad url is given (7284ms)
    ✓ is not possible when an invalid item is in the cart (8825ms)
    ✓ is not possible when grand total is to large (7117ms)
    ✓ is not possible when using an invalid shipping address (7885ms)
    ✓ is not possible when given an invalid field (7215ms)

  Checkout Donation Confirm
    ✓ is possible (15677ms)
    ✓ is not possible with an invalid order (8574ms)
    ✓ is not possible when the product in cart has been is modified (12428ms)

  Checkout Rates
    ✓ can be generated (21405ms)
    ✓ cannot be generated if grand total to large (20448ms)
    ✓ cannot be generated if shipping address is invalid (18494ms)
    ✓ cannot be generated if product is missing (18931ms)

  Deal
    ✓ can create deal product through API (4338ms)
    ✓ cannot create deal product (4467ms)
    ✓ cannot create deal product when the start date is after the end date (4240ms)
    ✓ can checkout deal product (14286ms)
    ✓ cannot checkout deal product when deal is over (9644ms)
    ✓ cannot change a deal to a regular type product (4946ms)

  Event
    ✓ can subscribe (1582ms)
    ✓ cannot be subscribed to if missing a field (1392ms)
    ✓ cannot be subscribed to if field is invalid (1406ms)
    ✓ cannot subscribe to a webhook that has already been subscribed to (1803ms)
    ✓ can retrieve list of subscriptions (1683ms)
    ✓ cannot unsubscribe if it is not subscribed to (1486ms)
    ✓ cannot unsubscribe if missing a required field (2155ms)
    ✓ cannot unsubscribe if a field is invalid (1693ms)
    ✓ can be unsubscribed from (1892ms)
    1) can get webhook
    ✓ can retrieve by id (12146ms)
    ✓ cannot be retrieve by id if it does not exist (1430ms)

  Login
    ✓ can be done (1277ms)
    ✓ cannot be done with username (57ms)
    ✓ cannot login if a parameter is invalid (55ms)
    ✓ as a guest can be done (1132ms)
    ✓ cannot be done as a guest with a missing parameter (63ms)

  Media
    ✓ can be uploaded (2199ms)
    ✓ cannot be uploaded without a purpose (1377ms)
    ✓ cannot upload with a purpose that does not exist (1698ms)
    ✓ cannot upload with an invalid checksum (1378ms)

  Message
    ✓ can be sent (5469ms)
    ✓ cannot send if product id is needed with selected context (4125ms)
    ✓ cannot send if order item id is needed with selected context (3768ms)
    ✓ cannot send if missing context (4429ms)
    ✓ cannot send if context is invalid (4777ms)
    ✓ cannot send if product id does not exist (2149ms)
    ✓ can reply (6966ms)
    ✓ cannot reply when missing a field (5669ms)
    ✓ cannot reply when a field is invalid (6312ms)

  Product
    ✓ can get available list (709ms)
    ✓ can be created (3631ms)
    ✓ cannot be created when the title is made of whitespace (3259ms)
    ✓ cannot be created when the title is too short (3166ms)
    ✓ cannot be created when the title is too long (2934ms)
    ✓ cannot be created when the description is made of whitespace (3176ms)
    ✓ cannot be created when the description is too short (2903ms)
    ✓ cannot be created when the description is too long (2684ms)
    ✓ cannot be created when the price is too low (2818ms)
    ✓ cannot be created when the price is too high (3064ms)
    ✓ cannot be created when missing shipping information (5843ms)
    ✓ cannot be created when free shipping is not selected and no other shipping information is provided (2882ms)
    ✓ cannot be created when stock is in an unacceptable range (2809ms)
    ✓ cannot be created when return policy "name" is an empty string (2910ms)
    ✓ cannot be created when return policy "policy" is an empty string (2861ms)
    ✓ cannot be created when return policy "policy" is filled with whitespace (2767ms)
    ✓ cannot be created when return policy "name" is filled with whitespace (2955ms)
    ✓ cannot be created when set as deleted (3081ms)
    ✓ cannot be created with type set to donation (3174ms)
    ✓ cannot be created with type set to deal (2976ms)
    ✓ can be updated (4438ms)
    ✓ cannot be updated with type set to donation (4566ms)
    ✓ cannot be updated with type set to deal (4405ms)
    ✓ cannot be updated with an invalid price (4299ms)
    ✓ can be deleted (4816ms)
    ✓ cannot be undeleted (8265ms)
    ✓ cannot be updated if it does not exist (1300ms)
    ✓ can be retrieved by id (4749ms)
    ✓ cannot be retrieved if it does not exist (1291ms)
    ✓ auction can receive bids (4259ms)
    ✓ cannot post a bid larger than the max (3994ms)
    ✓ cannot post bid smaller than the minimum (4199ms)
    ✓ cannot post bid less than the current bid (4790ms)
    ✓ cannot be bid on it if does not exist (1573ms)
    ✓ cannot receive a bid if it is not an auction (3780ms)
    ✓ can create 5 products in parallel (4899ms)
    ✓ can create product with shipping options (3533ms)
    ✓ cannot create product with shipping options price bellow limit (3651ms)
    ✓ cannot create product with shipping options and free shipping (3106ms)
    ✓ can put product with $10 shipping options to $0 (5494ms)
    ✓ cannot put free shipping on a product with shipping options (5519ms)

  Store
    ✓ can get available list (4856ms)
    ✓ can be retrieved by id (2897ms)
    ✓ cannot be retrieved if it does not exist (1664ms)
    ✓ can be created (2453ms)
    ✓ cannot be created using an invalid name (2354ms)
    ✓ cannot be created using an invalid email address (1838ms)
    ✓ can be updated (5430ms)
    ✓ cannot be updated with an invalid email address (4973ms)
    ✓ cannot be updated with an invalid name (3905ms)
    ✓ cannot be updated if it does not exist (1432ms)
    ✓ can be deleted (3137ms)
    ✓ cannot be deleted if it does not exist (1502ms)
    ✓ cannot be deleted by an unauthorized user (2653ms)

  Store Address
    ✓ can be created (1544ms)
    ✓ cannot be created when a field is too long (1277ms)
    ✓ cannot be created when a field is invalid (1335ms)
    ✓ cannot be created when a field is missing (1391ms)
    ✓ cannot be created when not logged in (4177ms)
    ✓ can retrieve list (2383ms)
    ✓ can be retrieved by id (2230ms)
    ✓ cannot be retrieved by id if it does not exist (1387ms)
    ✓ cannot be retrieved if not owner (2085ms)
    ✓ can be updated (2241ms)
    ✓ cannot be updated if it does not exist (2130ms)
    ✓ can be deleted (2387ms)
    ✓ cannot be deleted if not owner (1924ms)

  Store Return Policy
    ✓ can be created (3913ms)
    ✓ can be validated (3327ms)
    ✓ can retrieve list (3702ms)
    ✓ cannot be created with an invalid id (3165ms)
    ✓ cannot be created if it already exists (3732ms)
    ✓ cannot be created when a name is not provided (3679ms)
    ✓ cannot be created when a policy is not provided (3774ms)
    ✓ cannot be created with an empty name field (3712ms)
    ✓ cannot be created with an empty policy field (3081ms)
    ✓ cannot be validated when a name is not provided (3889ms)
    ✓ cannot be validated when a policy is not provided (3896ms)
    ✓ cannot be validated with an empty name field (3708ms)
    ✓ cannot be validated with an empty policy field (3815ms)

  Store Sale
    ✓ can retrieve list (16091ms)
    ✓ can be retrieved by id (17262ms)
    ✓ cannot be retrieved if it does not exist (3131ms)
    ✓ can be updated (19887ms)
    ✓ cannot be updated with a forbidden status (19271ms)
    ✓ cannot be updated with an invalid status (19444ms)
    ✓ cannot be updated if it does not exist (3641ms)
    ✓ cannot be updated when user is not logged in (19554ms)
    ✓ cannot be updated when incorrect store id is provided (21180ms)

  Store Subscription
    ✓ can be done (4624ms)
    ✓ cannot be done if missing a required field (3545ms)
    ✓ cannot be done if a field is invalid (2892ms)
    ✓ cannot be done if the intent is invalid (3237ms)
    ✓ cannot be done if already subscribed to a plan (5218ms)
    ✓ can be paid for (15450ms)
    ✓ cannot be paid for if bank account is not present (7805ms)

  Store Transaction
    ✓ can retrieve list (16124ms)
    ✓ can be retrieved by id (17618ms)
    ✓ cannot be retrieved by id if it does not exist (3561ms)
    ✓ can retrieve holds associated by id (17803ms)
    ✓ can retrieve hold by id (19749ms)
    ✓ cannot retrieve hold by id if it does not exist (3433ms)

  Tag
    ✓ can retrieve list (3915ms)
    ✓ cannot retrieve list if it does not exist (1337ms)
    ✓ can be retrieved by name (3905ms)
    ✓ cannot retrieve by name if it does not exist (1369ms)

  User
    ✓ can register (2194ms)
    ✓ can register with a password that includes whitespace (2370ms)
    ✓ cannot register if password is < 6 (55ms)
    ✓ cannot register with an incorrect 'action' parameter (62ms)
    ✓ cannot register with an invalid field (91ms)
    ✓ cannot register if already registered (2261ms)
    ✓ cannot register with an invalid email (76ms)
    ✓ can reset password (2699ms)
    ✓ cannot reset password if an account does not exist (96ms)
    ✓ can be retrieved by id (2311ms)
    ✓ cannot be updated if it does not exist (1053ms)
    ✓ can be update (3478ms)
    ✓ cannot be updated with invalid field (2569ms)
    ✓ cannot be updated by unauthorised user (2089ms)
    ✓ can change password (3556ms)
    ✓ can change password to one that contains whitespace (4176ms)
    ✓ cannot change password to an empty string (3274ms)
    ✓ test (2697ms)
    ✓ cannot change password if unauthorized (3858ms)

  User Address
    ✓ can be created (1807ms)
    ✓ cannot be created when a field is too long (2329ms)
    ✓ cannot be created when a field is invalid (1693ms)
    ✓ cannot be created when a field is missing (1334ms)
    ✓ cannot be created when not logged in (3777ms)
    ✓ can retrieve list (1911ms)
    ✓ can be retrieved by id (2191ms)
    ✓ cannot be retrieved by id if it does not exist (1475ms)
    ✓ cannot be retrieved if not owner (1857ms)
    ✓ can be updated (2564ms)
    ✓ cannot be updated if it does not exist (3089ms)
    ✓ can be deleted (3552ms)
    ✓ cannot be deleted if not owner (2739ms)
    ✓ cannot create more than 20 (18075ms)

  User Auction
    ✓ can retrieve list (6091ms)
    ✓ cannot retrieve list if not logged in (7057ms)
    ✓ can be retrieved by id (6787ms)
    ✓ cannot be retrieved by id if not an auction (6741ms)
    ✓ cannot be retrieved by id if not logged in (6819ms)
    ✓ cannot be retrieved by id if it does not exist (1298ms)
    ✓ can be updated (5553ms)
    ✓ cannot update watchlist with an invalid product (4615ms)
    ✓ cannot update watchlist if item does not exist (1463ms)
    ✓ can be removed (6195ms)
    ✓ cannot be removed if it is not being watched (4817ms)
    ✓ cannot be removed if not logged in (4709ms)
    ✓ cannot watch a deal (6611ms)been bid on (4851ms)
    ✓ cannot watch a virtual product (8943ms)

  User Notification
    ✓ can retrieve list (16994ms)
    ✓ can be retrieved by id (16777ms)
    ✓ cannot be retrieved by an unauthorized user (16788ms)
    ✓ cannot be retrieved if it does not exist (1768ms)
    ✓ can be updated by id (19450ms)
    ✓ cannot be updated when tampering with email flag (19302ms)
    ✓ cannot be updated by a forbidden user (20153ms)
    ✓ cannot be updated with an invalid field (19206ms)
    ✓ cannot be updated when not logged in (19074ms)
    ✓ cannot be updated if it does not exist (1775ms)

  User Order
    ✓ can retrieve list (23622ms)
    ✓ can be retrieved by id (16859ms)
    ✓ cannot be retrieved if it does not exist (1674ms)
    ✓ can be updated (19188ms)
    ✓ cannot be updated with an invalid field 1 (20324ms)
    ✓ cannot be updated with an invalid field 2 (19353ms)
    ✓ cannot be updated with an invalid field 3 (18325ms)
    ✓ cannot be updated by an unauthorized user (17510ms)
    ✓ cannot be updated if it does not exist (1337ms)

  User Order Item
    ✓ can retrieve list (22134ms)
    ✓ can be retrieved by id (16609ms)
    ✓ can be updated by id (18769ms)
    ✓ cannot be updated because order status prohibited (20859ms)
    ✓ cannot be updated if it does not exist (1795ms)
    ✓ cannot be updated by an unauthorized user (1243ms)

  User Transaction
    ✓ can retrieve list (17376ms)
    ✓ can be retrieved by id (17103ms)
    ✓ cannot be retrieved by id if it does not exist (1603ms)
    ✓ can retrieve holds associated by id (19181ms)
    ✓ can retrieve hold by id (22523ms)
    ✓ cannot retrieve hold by id if it does not exist (1637ms)

  Vanity URL
    ✓ can be retrieved (6539ms)
    ✓ cannot retrieve if it does not exist (1612ms)

  Virtual
    ✓ can create virtual product (10239ms)
```


@mikeflynn Thanks for pulling this in! Would you mind cutting a new release, please?


What if max=100 in mako?


While it's OK to generate the array multiple times, the VariableDeclaration node should be created only once. ConditionalWeakTable is our mechanism for accomplishing this. Given that the BoundMultiple case below will have to keep track of multiple VariableDeclaration nodes, maybe storing an array as the CWT value is OK.


This turns `flyspell-mode` on in all programming buffers when we at most want `flyspell-prog-mode`


I like how you styled the constructor arg like a type


Ha. Moved it and it crashed the compiler.


Посмотри, вроде то, что ты заказывал.


thanks!


Yeah.


Oh


@blawrence719 This looks great, but I'd just delete the very last sentence--it's enough just to say (in public anyhow) that she was discouraged from continuing. You could add, though, that Ierardi is working on a novel based on a Dickinson poem now (in 2015), and add the blog link about it: http://deaddarlings.com/life-death/


Again, these cases need the CWT trick to avoid creating equivalent nodes multiple times.


@tisto tnx for fixing it!
but how can the person who broke the build but does not have access to the CI servers fix such an error?


It's just for tests.


This was me my git client got messed up...


То, что нужно. Браво) Крой задачу.


This closes issue #7


Solved in: e96597b417d68fab64e54a4c6ca60d6308206602


spaces between


remove `as np` and use numpy instead...


Implemented SinZ's feedback


Опять проблема с именованием переменной :cry:


Потеряли пробел перед {


Why did you create a branch for that ?
And please, do not put style attribute into your html tags. Only for test, then use stylesheets when you have a stable version.
I added your changes in my commit and pushed it into the master branch.


I put it here so that it could issue a warning and because it is specific to embedded cue sheets. Should I create something like ```check_embedded_cue(struct mp_log *log, struct cue_file *f)```?


Is the motivation for eliminating the implement-both-interfaces trick and introducing the extra node here so that all cases are completely regular and walking won't encounter a node twice? If so, that's cool, so long as the new node is gaining us something.


Yes, `pkgs.lsof` sounds right. You can always open a pull request to have the change checked on Linux, btw.


Fixed the writing paramBase to where the last iteration and mixture assignment were actually saved. Added a string that specifies what model is being written. Also updated the load method to where it only takes a file and creates the parameter in the function by reading the model value I added.


This is a good change that cleans up a mess I left behind.


I was using `DataTableSpace` which is a property in the interface. But then i discovered we can set custom attributes - which is a better choice, since datatablespace will be a property set and used by another connection type.


remove commented code


check this assertion move


The TRACK command is issued without a following TRACK 01. EAC does this for data tracks.


Please change comment to `NewEmptyDocument creates new empty document`.


Same as above


The changes spawn both connection, db level, logging and ui changes. That said, I am unaware of a prescribed pattern. @overtron Where can i find it?


should be PATH_CONFIG instead of Path_Config as @gabr mentioned


Redundant constructor. Will remove


Redundant function. Will remove


Please correct mentioned variable naming. Other than that it looks good


w.pl is the core part of the program... I'm going to tweak it so that you only include e.pl, f.pl, and other bits if you need them.

I have mixed feelings about the Perl modules, and may move some stuff back into w.pl. We'll see.


This update should allow the "make" command to run.

* fixed the base exp yield error *

-- Modified base exp yield for starter pokemon

-- Modified the wild pokemon type and level on route2

-- Modified the wild pokemon type and level on route22

-- Modified the wild pokemon type and level in viridianforest


Ah, that's a good suggestion, thank you.


New users (as of when?) will have the new modules?
How do you switch back to the old modules?
What if a module you need is not represented?

Will this change in default affect Blanca users as well?  I assume it'll break all their automation and many of them use older versions of software.


Haha, much better!


Unfortunately, the latest `git-annex` fails tests on Darwin now too. I'm going to report that with Joey.


umm theres a bunch of spaces here <insert clarkson smirk here>


This seems wrong. Missing the dot.


eRest


Hi Kushank,
A small update on the max line length. Although PEP8 uses 79 characters as its max line limit for Python, the [Django](https://github.com/django/django) project has made an exception to it as mentioned [here](https://docs.djangoproject.com/en/1.8/internals/contributing/writing-code/coding-style/) to have the max line length as 119 characters.
I noticed that the migration files generated by Django also contain lines upto 119 characters, which is why it would be recommended that we also keep the max line length to 119 characters. I have updated details in README.md.
Sorry  for the confusion, let me know in case of any suggestions or concerns.


delete this line please


Fixed in next commit


remove line


Please remove `AddCluster` function from `DBSession` interface and move the logic from [obinary/commands_db.go:358](https://github.com/ernestas-poskus/orientgo/commit/59dc9a8c57948745a5e7141d68e17367467b326f#diff-2fbfa4ac6d5fcccba0a8e8a4e17756cfR358) to [client.go:286](https://github.com/ernestas-poskus/orientgo/blob/59dc9a8c57948745a5e7141d68e17367467b326f/client.go#L286). Thus we can drop `obinary.Database.AddCluster` entirely.


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