Don't you think it should be renamed `CeTaskProcessorRepository` in order to be consistent with `CeTaskProcessor` ? Or the latter could be renamed.


unexpected token tLSHFT


unexpected token tEQQ


unexpected token tCONSTANT<br>dynamic constant assignment


unexpected token tRSHFT


Фильтрация по выбранному набору не осуществляется. Ранее в данную переменную попадало системное имя набора, теперь id набора. Поэтому далее в коде  выражение
$model->applyDatasetFilters($datasets[ $dataset ]);
работает не корректно.


Line is too long. [108/100]


Can be replaced by:
```checkState(taskProcessor != null, "No CeTaskProcessor found for CeTask type %s", ceTask.getType());```


ok


Yes, sonar lint was complying :)


lol


Indeed


Hi,

I understand, please let us know when your tests are over.

Regards,


On line 107 the code didn't change, see my compile fix from last time;

https://github.com/zander/Bitcoin/commit/2c56924614e4ed105c0e61055547074d033feebc

I belief you can use int everywhere instead, is there any reason to belief you need a 64bit value?
An int (signed) allows for 2GB/s max, don't you think that is enough?


Would at least be nice if we could be given an option to return the rope ladder to its old glorious ways.


For #648.


@jcfr Did your patch solve the issue?


good job! :+1:


Probably given context should overwrite existing. Here ```$this->_extra```'s values  will be used in case of keys conflicts


it should be static class, not  a prototype class


I think Optional should be returned instead of throwing IllegalStateException. Here is the reason: imagine that views plugin is installed, tasks are in the queue then the views plugin is uninstalled (no more license, ...). The tasks should fail but without technical stacktraces in logs. That's a valid functional case that should be handled by the worker. A single log should be printed, not a stacktrace.


Done.


:+1: to this commit.  Might be worth pulling out to a separate PR to reduce noise when reviewing the substantive changes.


should be a static method Geometry.appendVertexAttributeToList


I mean this is a functional case but not a technical failure.


Looks correct to me.


:tada:


same as above


@daniloegea I don't understand why an intermediate char* should be needed.
Your new patch does QString -> std::string -> char* -> std::string.
Why does it work better than only QString -> std::string as I did?


**no-unused-vars**: builder is defined but never used


**no-unused-vars**: Title is defined but never used


**no-multiple-empty-lines**: Multiple blank lines not allowed.


**Eslint**: 'return' outside of function


snap to level should probably count steps by calculating distance between starting and ending point rather than just vx


**semi**: Missing semicolon.


could probably remove this comment ;)


Außerdem Fehler weitere Fehler behoben, z.B. in FakeSUPArtikelErmitteln()


Okay. Was needed then [but not now](https://github.com/LimeSurvey/LimeSurvey/blob/master/application/helpers/qanda_helper.php) but still needed in [v2.051](https://github.com/apmuthu/LimeSurvey/tree/2.051).


És curiós, per a això he fet un commit que eliminava tots els directoris svn (de fet, a la branca corrector no en queden .svn). Per què apareixen aquí?


sugestia: publiczna metoda z cyklem, wewnątrz wołana prywatna metoda "one-way"


The checks to $this->member want to be above / before the $this->item stuff (currently lines 39-42)


Potrzebny jest ten `if {} else {}` ? Moze da sie to jakos skrocic? ;)


Yeah Odoo 9 !
--------------------

![will-and-carlton-dance](https://cloud.githubusercontent.com/assets/7599886/10216296/93b23518-6845-11e5-8afe-f3eedb84d92c.gif)


ok


It looks like you changed lots of whitespace ;)
I think I indent all of the other files with tabs but there are now spaces.
You should poke around with the netbeans style settings a bit :)


sugestia od MPI: rozszerzyć zadanie o kolejne preferencje różnego typu. Tak, aby można było poćwiczyć rozszeranie modelu w nietrywialny sposób. Np. lewo/praworęczny, w tym samym mieście...


Haha woops! That was from my testing to see what was making copy.php fail! :P Will remove tonight.


yep


The filter on the company is done by the test

if ($this->socid)
		{
			$this->where.=" AND f.fk_soc = ".$this->socid;
		}

The test
if (!$user->rights->societe->client->voir && !$user->societe_id)
must be kept to say, if you don't have permission to see all or if you are an external user, then view is restricted to your invoice only.

So old code looks good to me.


I'm pretty sure I'm not a fan of this. From neither the yaml file nor the command line is it possible to know what is being included: you have to go look at the module file which could be _anywhere_ in the current PYTHONLIB.

Or to be it more concisely: This violates explicit is better than implicit.

To put it less concisely: Maybe it is the right thing to do but it feels like it is an unwarranted optimization that is coming about as a result of typing gabbi-run on the command line lots of times rather than having a gabbi-run command in a shell script, Makefile or grunt hoopa joop.

I'm very happy to be convinced otherwise and probably can be quite easily but you'll need to address whether or not the lost explicitness is worth it.


indeed, it was to not forget it... In fact I'll remove it.


Wäre denkbar, aber bisher war der Bedarf nicht so groß. Warum den Benutzer vorher mit so Optionen belasten wenn er in der Bildbearbeiten per Copy’n’paste das einfach richten kann! Andererseits mags Fälle geben, wo es geschickt wäre...

Spannender wäre doch eh, GIMP-Patterns zu exportieren, dann kann man einfach Formen füllen.


Estava fent la revisió commit a commit, per a vore si així es podia vore tots els fitxers sencers... Oblida-ho.


@florianeckerstorfer for my curiosity, which tools do you use for changelog generation and release tagging?

Thanks.


sugestia od MPI: gdyby imię miało identyfikować developera, to wtedy nie da rady tutaj be euqals/hashcode.


En altres sistemes, les funcions que afegeixen js a les pàgines, s'ocupen de fer el merge dels fitxers js (i també css). Des del meu punt de vista, això li manca al wordpress. Ja que estem utilitzant una funció «wp_enqueue_script», per què aquesta funció no s'encarrega de recollir tots els js que s'han afegit i fer un merge? He vist que hi ha algunes extensions:
- https://wordpress.org/plugins/wp-minify/
- https://wordpress.org/plugins/combine-js/

Què en penses?


You'll need to refresh the managed job entities if there is and if you'll use them after this (in order to have updated counters).


Yes, i got your point. Very good catch.
Thx a bunch.


On Thu, Oct 1, 2015 at 2:08 PM, Amit Dodiya <notifications@github.com>
wrote:

> [image: wow_gif]
> <https://cloud.githubusercontent.com/assets/7599621/10216317/bd231872-6845-11e5-8550-1f071c9e5e5c.gif>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/odoo/odoo/commit/b5ae7659ff4386b9d9098b5c8e9738b5db4697f2#commitcomment-13535892>
> .
>

--
Thanks & Regards,
Parthiv Patel


Are there any additional restrictions that you think we should capture in the spec? I don't think we need a really detailed definition of this stuff, but what other things does the typechecker _not_ enforce for `native`s, and what _additional_ interesting rules _does_ it enforce for `native`s?


Okay will move it up. Got a question about $this->get( 'Item/Member/etc' ). Remind me tonight.


Ah, I only saw your last comment, so it seemed cryptic. I agree with you about getting rid of the x in this case.


ok


Thanks for the patch.   Can you also update the automated test to check for this bug?   There's a _KeyNavContainer.html test file that you could probably augment.


sugestia od MPI: robiąc `getClass() != ` ograniczam porównania do tych samych klas. Jeśli chcę pozwolić na równość instancji klasy `Developer` z odpowiednią instancją klasy `JavaDeveloper` (gdyby taka klasa zostawał wprowadzona), to potrzebne jest użycie ` instanceof Developer.class`. Użycie `getClass().isInstance` nie rozwiązuje sprawy, bo wtedy porównanie wraca do równości klas, jako że `getClass()` zwróci nam `JavaDeveloper.class`.


We really should properly fix this in near future instead of silencing errors...


Well yess I do need to look at my NetBeans settings. But was I was referring too was that git thought all 200 or whatever files had been modified and not just these 3 files! :/


indeed


@anrishitko


Hi @tortuetorche,
it was simpler to just fix it, thank you for offering to do a PR though.
Thank you.


ワキタ先生，

脇田です．フルネームに修正して，提出しました．ご確認をお願いします．


sugestia od MPI: tutaj trochę naginam zasady Object Callisthenics poprzez skorzystanie ze streamów Javy 8. Fajnie byłoby rozbić to dla ćwiczenia. Wtedy efektem ubocznym będzie zapakowanie hash setu `people` jako collection object z metodą `filter`.


nice


Probably it  better (and commonly) could be done like
``` null === $level ? CM_Log_Logger::DEBUG : (int) $level ```
e.g. with casting. Certainly in this case casting could be unnecessary because value will be casted in setter, but nevertheless we should cast it as early as possible.


Commit note changed to:

> To employment_details_of_mother_2.
>
> This follows the renaming of employment_1 to employment_details_of_mother_1.


sugestia od MPI: w tej metodzie tracimy interfejs `Matches`. Gdyby chcieć używać `Matches` chainowo, np. `matches.onlyBeginners().onlyOlderThan(18)`, to trzeba aby `all()` także zwracało `Matches`. A w ogóle MPI sugeruje zaimplementowanie interefejsu `Iterable`, zaś filtrowanie uzyskać poprzez tworzenie nowych instancji `Matches` (np. z podzbiorem osób lub ze wszystkimi osobami i filtrem).


Commit note changed to:

> To employment_details_of_partner_2.
>
> This follows the renaming of employment_2 to employment_details_of_partner_1.


Fixes #234


Commit note changed to:

> To employment_details_of_mother_3.
>
> This follows the renaming of employment_1 to employment_details_of_mother_1, and
employment_1_more to employment_details_of_mother_2.


Ok i see, i will revert this commit


I've added the following to the commit note:

> I've added the leading hyphen to the relevant Smartdown scenarios to indicate
that we expect this question to appear on its own page.


Well there's a whole bunch of things I need to document for `native` of course, but I don't know what you think would be interesting to mention in the spec. You could wait until I finish write the docs and then extract from that the things you want perhaps.
The thing that I can think of right now is for example that shared declarations require a header while private declarations do no not.


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


It is a good start @tkrish but the hard-coded values are a bad idea :)

Something like this where we retrieve some IDs from the organisation connected might be a bit better:

<?php

            $response = $XeroOAuth->request('GET', $XeroOAuth->url('Invoices', 'core'), array('Where' => 'Type=="ACCPAY" && Status=="AUTHORISED"', 'Page' => '1', 'Order' => 'Guid.NewGuid()'));
            $invoices = $XeroOAuth->parseResponse($XeroOAuth->response['response'], $XeroOAuth->response['format']);
            echo "Found invoice ID: "  . $invoices->Invoices[0]->Invoice->InvoiceID . "
";
            echo "With LineItemID: " . $invoices->Invoices[0]->Invoice->LineItems[0]->LineItem->LineItemID. "
";
            echo "Will try to create a billable expense on this invoice...
";

            $xml = "<LinkedTransaction>
                       <SourceTransactionID>".$invoices->Invoices[0]->Invoice->InvoiceID."</SourceTransactionID>
                       <SourceLineItemID>".$invoices->Invoices[0]->Invoice->LineItems[0]->LineItem->LineItemID."</SourceLineItemID>
                       <Status>APPROVED</Status>
                       <Type>BILLABLEEXPENSE</Type>
                    </LinkedTransaction>";

            $response = $XeroOAuth->request('PUT', $XeroOAuth->url('LinkedTransactions', 'core'), array(), $xml);
               if ($XeroOAuth->response['code'] == 200) {
                    $billableExpense = $XeroOAuth->parseResponse($XeroOAuth->response['response'], $XeroOAuth->response['format']);
                    echo "" . count($billableExpense->LinkedTransactions[0]). " billable expense (linked transaction) created in this Xero organisation.
";
                   echo "LinkedTransactionID: " . $billableExpense->LinkedTransactions[0]->LinkedTransaction->LinkedTransactionID;

               } else {
                   outputError($XeroOAuth);
               }
?>


publishDate


![Release the Kraken](https://media.giphy.com/media/E3cQDsj7pUpwI/giphy.gif)


Aligned with the HTML: https://github.com/whatwg/html/pull/72.


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


SF :)


great


我是说，这repo根本不是我创建的。。应该可以删掉了


Поправим, спасибо. Коммит ваш отклоняю, т.к. он неверен.


well done


اینو ننویس!


I think it has been executed very well.


probably ```self``` matches better than ```static``` for that case and for ```hasLevel```'s occurrence too.
I suppose we don't want to use LSB here


Мы не оставляем семантические блоки без классов


Тоже самое navigation - все что ниже - производное от него


Haha no worries, apologies for being ambiguous!


why call( undefined ????
call and apply should be avoided if not necessary (slower)


navigation__item navigation__link


I would tend to agree with the change to ``bool`` type.
Should I go ahead and amend for all variables/function definitions affected in this commit ?


tag tag-date tags__link - перебор

тогда уже tags__link tags__link--date


An uneven number of -a results in -a being set; the -c option is there to exercise the default switch clase; the multiple -b clauses are there for showing that last flag wins.


@colm2 We should probably add this to the model


@Soullivaneuh I don't use any specific tools, just writing them down by hand. However, I use the *x comments to master since this release* functionality of Github to get down changes.


Not sure if "restore in" or "restore to" ?


I'm still getting errors like
```
[ERROR] /Users/ingmar/workspace/marytts/marytts-runtime/src/main/java/marytts/server/http/SynthesisRequestHandler.java:54: error: semicolon missing
[ERROR] * @author Oytun T&uumlrk
```
at 55aad6a6279900afe538abc94800dd08a4e1a0ed.
I thought you fixed the malformed HTML entities, specifically the umlauts, using your perl one-liner?


что это за -1 -2 -3 они все одинаковые footer__column


Please remove duplicated casting


You're right, sorry for the confusion, the test you added looks correct.

Are you sure about Cryptography discarding data when serializing? I'm not able to reproduce this. If it does, then this must be a bug, as the signature would no longer be valid for the certificate.


column-1__link удалать


btn- удалить


Interesting! Were you planning on doing a PR? If not, I'll just include this myself.


Зачем?


author


Aren't there some specific null pointer value in C++11? Shouldn't that be used?


плюсую. Зачем? Ломается структура.


Подтверждаю, на чистой cms данная ошибка не наблюдается. Прошу прощения.


we use _ (underscore) to prefix non public methods and attributes.


Does the add of a "/" is necessary, because there are a few other `post_url` not starting with a "/"
i.e:
       |
       v


Мою косяк, мне показалось, что у вас есть общий Quick start.
Но его всяко надо добавить. Должно быть:

## Quick start
### .NET

### Java

### ...


This is one way. You may want to split the middleware into `Authenticate` for instance. But this is fin :smile:


If I recall correctly, that's what the browser was sending when I looked at it. Any reason we need to remove those ? As this changed ? Or is this just for consistency ?


You should return Bad Request response.  NOT_ACCEPTABLE is when the client ask for a format not handled by the server


IIUC the warmup time is the time during which a creature can't use any skill at all, even if it has skills ready to use (to prevent spamming e.g. 3 skills at once)?

If so that's a good concept IMO, but should we keep it defined per-skill, or should we just make it a global parameter? In other words, what would be the use case for different warmup times in different skills?


as said in the issue, I would prefer to not silently fail here, but le the client try a few times and fail hard if it can't.


You should create a service for that.


Nice


I'm making a commit removing the temporary fix for this and going to make the pull request just fix the app submission issue, sorry it's my first pull request :smile:


> I use the x comments to master since this release functionality of Github to get down changes.

Don't know what you're talking about. Do you have a link to it?


:clap:


Removing this, but when I was looking at the browser request, it didn't send these two fields, don't think it matters


I doubt there will be conflicts (on JVM at least) for the `ctors` branch, so I'm OK with merging this now. But the loss of an abstracted `apply()` (that works across values and value constructors) is a shame.


If you go to the [releases page](https://github.com/cocur/slugify/releases) there is a link under the name of the release (only if there have been new commits.)


Em thêm 2 file index.html và index.html2 để so sánh canvas và webgl. Một cái em dùng PIXI.CanvasRenderer và một cái em dùng PIXI.WebGLRenderer sao em thấy dùng canvas nhanh hơn nhỉ


I think we should remove this in master?


Looks fine to me @gavinking. Thanks.


I was about to say this argument is not used in the function, but actually the function gets removed in a later commit :P


Oh yeah, this things.

Thanks you for your time. :+1:


1: M111 unrecognized component name: tests


1: D100 Missing docstring in public module


32:1: I100 Imports statements are in the wrong order. from invenio_base.helpers should be before from invenio_ext.cache
32:1: F999 redefinition of unused 'unicodifier' from line 28


26:1: I100 Imports statements are in the wrong order. from invenio.testsuite should be before from invenio_celery


58:1: I100 Imports statements are in the wrong order. from invenio.utils.date should be before from invenio_ext.principal


25:1: I100 Imports statements are in the wrong order. from invenio.testsuite should be before from invenio_ext.sqlalchemy
25:1: I201 Missing newline before sections or imports.


looks good to me, can i get a +1 from Jim / Lalatendu as well ?


Van


@anrishitko


Bootstrap юзает jQuery, если не подключить все время логи падают в консоли


ok, make more sense if we do the opposite... see my last commit 438f3c8b5e85788b5b80496ebb55b986d870e5d7


@AnriShitko


Commit note changed to:

> This was the 1st question on the employment_details_of_partner_2 question page
> so I've renamed the question page employment_details_of_partner_2a.


Actually I removed these tests because it is just by luck that we have a good score: the anomaly rate is in fact 66% ! (both labels 1 and 2 are considered as abnormal). I think the test below is enough...


Commit note changed to:

> This was the 1st question on the employment_details_of_partner_1 question page
> so I've renamed the question page to employment_details_of_partner_1a.


41ae586dbd01fb1b733cd6d0b51a096c12b48851
應該是鞦韆魔術家的腳本的問題


Both arguments are no longer useful.


Nice!


複数の内容が1コミットにまとまってませんか？


Commit note now reads:

> To its own page.
>
> This was the 2nd question on the employment_details_of_mother_2 question page
> so I've named the new question page employment_details_of_mother_2b.
>
> I've added the leading hyphen to the relevant Smartdown scenarios to indicate
> that we expect this question to appear on its own page.


Commit note now reads:

> This was the 1st question on the employment_details_of_mother_2 question page
> so I've renamed the question page employment_details_of_mother_2a.


Ну, в идеале должны же отправляться) Тут конечно не критично и можно не использовать форму, но вроде как желательно? В общем я не уверен если чесно...


Commit note now reads:

> To its own page.
>
> This was the 4th question on the employment_details_of_mother_1 question page
> so I've named the new question page employment_details_of_mother_1d.
>
> I've added the leading hyphen to the relevant Smartdown scenarios to indicate
> that we expect this question to appear on its own page.


the new indentation is actually the correct one
but, correct indentation of comments after an empty line comes only when Eclipse 4.4 is enable in the Intellj IDEA plugin (which I activated not so long ago)
![selection_018](https://cloud.githubusercontent.com/assets/11717580/10217042/a15e21d0-682d-11e5-8e70-268d7b8cd144.png)


There's "nullptr", but Bjarne did say in his FAQ that he prefers plain 0. I don't know if there's any benefit with nullptr.


Để tối a xem. Browser của e có support webgl ko

On Thu, 1 Oct 2015 4:09 pm minhpl <notifications@github.com> wrote:

> Em thêm 2 file index.html và index.html2 để so sánh canvas và webgl. Một
> cái em dùng PIXI.CanvasRenderer và một cái em dùng PIXI.WebGLRenderer sao
> em thấy dùng canvas nhanh hơn nhỉ
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/minhpl/leaflet-maskcanvas_needfix/commit/b8ad8c684921b91142d456d67eb7b059cb9b11a6#commitcomment-13536479>
> .
>


Commit note now reads:

> To its own page.
>
> This was the 3rd question on the employment_details_of_mother_1 question page
> so I've named the new question page employment_details_of_mother_1c.
>
> I've added the leading hyphen to the relevant Smartdown scenarios to indicate
> that we expect this question to appear on its own page.


ок, я обязательно добавлю документацию


comment


After a Draw game it will be bugged


Commit note now reads:

> To its own page.
>
> This was the 2nd question on the employment_details_of_mother_1 question page
> so I've named the new question page employment_details_of_mother_1b.
>
> I've added the leading hyphen to the relevant Smartdown scenarios to indicate
> that we expect this question to appear on its own page.


It is not required. I will remove this signal.


Commit note now reads:

> This was the 1st question on the employment_details_of_mother_1 question page
> so I've renamed the question page employment_details_of_mother_1a.


Okay.


Phew! :D


Thank you for supporting XBus system.  I still not familiar with GitHub...Oh, English, too :-).  So I'm sorry for any inconvenience.

It looks like some confusion between ANT module and Receiver unit.  XBus system has 2 mode and mode B is for some compatibility for other :-) command system.  Transmitter will send the mode B command packet on our standard XBus command system.  This means that we do some capsulized for the mode B command packet.

ANT module (like RJ01) receives the standard XBus command packet and output it without any mod... Yes, it's still capsulized.  That's why you need to offset to get the real mode B command packet.

(Of course, the mode B command packet is de-capsulized when it's from Receiver unit )

By the way, we have a standard XBus command system (mode A) and it's already open.  We have a sample code to drive our XBus servo from Arduino.  So you can understand the mode A command packet from this source code.

http://www.jrpropo.co.jp/jpn/dl/data/XBusServoLib.zip

I can answer some question if it's possible :-).  Feel free to ask me.

Best Regards,
Zak


I think if you want add all symbol paths to symbol search path, you should remove duplicates from string.


ok but either we remove the full test or we test something with assert_*

Running the code is not enough.

what do you suggest?


Remember that this brace needs to be on the same line as the `function` keyword itself.


>  you have to go look at the module file which could be anywhere in the current PYTHONLIB

But that's the case regardless of whether you use this shorthand or not? This shorthand just saves you the class name (by virtue of the module providing a reference via the pre-defined `gabbi_response_handlers` contract).

>  typing gabbi-run on the command line lots of times rather than having a [script]

Sure, but you want to allow for both to be painless. For example, I generally have some `py.test` command in my `Makefile`, yet still regularly run a slightlty different command (e.g. `py.test -x`) directly.


````
 TORPKGSCOMMON='$TORCOMMON $HTTPSPKG'
````

The problem here is the use of single quotes. Those cause the variables to not be expanded. The correct way would be

````
 TORPKGSCOMMON="$TORCOMMON $HTTPSPKG"
````

An example:
````
$ REASONS="¯\_(ツ)_/¯ #YOLO"
$ echo $REASONS
¯\_(ツ)_/¯ #YOLO
$ WHY='$REASONS'
$ echo $WHY
$REASONS
$ WHY="$REASONS"
$ echo $WHY
¯\_(ツ)_/¯ #YOLO
````


Попробовал этот плагин. К сожалению, он мне не подходит. Поэтому, если нет желания, можешь этим не заниматься.

Проблема в том, что он всегда показывает только один слой. А было бы здорово, если бы он показывал одновременно все слои, которые в него запихнули. И если они все - overlay-и (т. е. с прозрачным фоном), то можно было бы менять их порядок (с `editable: true`). Я именно так собирался его использовать :)

Но не зря написано:
> It does not support overlay layers.


Yeah, freu mich darauf!


Original GetProcAddress sometimes lock `loaderLock` and because of that we try reduce using of _RGetProcAddress.
These changes are really necessary?


s/activie/activity/


> But the loss of an abstracted `apply()` (that works across values and value constructors) is a shame.

Well you're very welcome to add an abstracted `apply()` method that simply doesn't accept `Set`, and have an additional one on `Value` that _does_.


Upgrade to CanJs 2.2.4 before SS2 live. Needed for SSD. Check if working with SCC. "scope should be renamed"


Same question


> The thing that I can think of right now is for example that `shared` declarations require a header while private declarations do no not.

There is also the thing about `shared` members of `native` classes needing to be declared `native`. Though I now have some doubt about that one...


ここの部分が謎。82行目に@set_property()と記述しても@set_propertyが正常に動かぬ…


I confirm, for pyflakes the ``# noqa`` line has no effect. I have configured syntastic in vim to use ``flake8`` ( https://pypi.python.org/pypi/flake8 ), which combines pyflakes, pep8 and some McCabe complexity checkers. For ``flake8``, ``# noqa`` works fine. I'm quite happy with flake8.
We should define the preferred way for code checking in https://github.com/plone/plone.api/blob/master/docs/contribute/conventions.rst (or are the conventions meanwhile managed somewhere else? /cc @svx )


Có anh ạ, nếu dùng mặc định autodectectRender thì nó tự động dùng webgl
On 1 Oct 2015 16:15, "ducthangho" <notifications@github.com> wrote:

> Để tối a xem. Browser của e có support webgl ko
>
> On Thu, 1 Oct 2015 4:09 pm minhpl <notifications@github.com> wrote:
>
> > Em thêm 2 file index.html và index.html2 để so sánh canvas và webgl. Một
> > cái em dùng PIXI.CanvasRenderer và một cái em dùng PIXI.WebGLRenderer sao
> > em thấy dùng canvas nhanh hơn nhỉ
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/minhpl/leaflet-maskcanvas_needfix/commit/b8ad8c684921b91142d456d67eb7b059cb9b11a6#commitcomment-13536479
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/minhpl/leaflet-maskcanvas_needfix/commit/b8ad8c684921b91142d456d67eb7b059cb9b11a6#commitcomment-13536600>
> .
>


I think, its auto-gen list, so due to webui commit, im here as averrin and Alexey Nabrodov. So, changing author of commit - difficult operation, and this comment is just notice=)


yes, but the Perl one-liner was only used under marytts-signalproc, because that was being tested, now the errors under common and runtime are being fixed and they should be okay soon


Tiens? `kuler` ne me semble pas déclaré, ou alors j'ai loupé un truc?


PR? Would you also be able to rearrange the options a bit, hide some of the less used ones?


*exrta*


We should remove the full test I think.
Gilles suggested that testing grid_search may be now part of the common tests.
If not, we could add a gridsearch to the test below.


この行意味なーい。のでリファクタリングします。


Yes, I'm adding documentation to this class. It's definitely lacking.


at last I truly see


Bruke me ein aen database nå?


Ja, Gimp-Pattern wäre super... Der Bedarf ist bei mir da, da ich es trotz euren Anleitungen noch nicht hinbekomme, die OID's selber auszudrucken, wenn ich die in paint.net oder gimp (fill-pattern) verwendete.


Why on earth would you exclude the composer.json? It contains vital information, and assuming it is only ever fetched via packagist and or used internally in composer itself is wrong… Flow e.g. uses it to look at dependencies on it's own, and I'd always assume a composer package I download–whichever way–contains a manifest.


While following the instructions as prescribed in book, I get error message as attached herewith:
![captureerror](https://cloud.githubusercontent.com/assets/11568906/10217351/dbec80de-684c-11e5-9c86-2b4b1a3bbced.JPG)

At another time, I wrote a code to send mail and in that case also, I got similar error. I understand that Gmail isn't letting me sign in through this code (they sent me a mail regarding that).

How to overcome this?


Browser gì, số lg items là trong 1 canvas là bn. Nếu ít items thì canvas
nhanh hơn webgl. Nếu dùng chrome thì canvas có tự đọng dùng card màn hình
nếu canvas đủ lớn, do vậy khả năng là pixi sẽ chậm hơn. Nên test trên cả
firefox nữa. Nhớ check bộ nhớ xem thế nào nhé

On Thu, 1 Oct 2015 4:25 pm minhpl <notifications@github.com> wrote:

> Có anh ạ, nếu dùng mặc định autodectectRender thì nó tự động dùng webgl
> On 1 Oct 2015 16:15, "ducthangho" <notifications@github.com> wrote:
>
> > Để tối a xem. Browser của e có support webgl ko
> >
> > On Thu, 1 Oct 2015 4:09 pm minhpl <notifications@github.com> wrote:
> >
> > > Em thêm 2 file index.html và index.html2 để so sánh canvas và webgl.
> Một
> > > cái em dùng PIXI.CanvasRenderer và một cái em dùng PIXI.WebGLRenderer
> sao
> > > em thấy dùng canvas nhanh hơn nhỉ
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/minhpl/leaflet-maskcanvas_needfix/commit/b8ad8c684921b91142d456d67eb7b059cb9b11a6#commitcomment-13536479
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/minhpl/leaflet-maskcanvas_needfix/commit/b8ad8c684921b91142d456d67eb7b059cb9b11a6#commitcomment-13536600
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/minhpl/leaflet-maskcanvas_needfix/commit/b8ad8c684921b91142d456d67eb7b059cb9b11a6#commitcomment-13536764>
> .
>


Well that's more or less the solution we used for `Class` with `apply()` and `classApply()`. But  I'm not even sure I can do that in a source compatible way, since `Value` has already taken `apply()` to mean the more specialized one with `Set`, so I'd need to call it `valueApply()` (thus we end up with an API that's the reverse of the one in Class where it's the more general one that's called `apply()`). It just starts to look like more and more of a mess.


?Very nice!

1. Please add me as contributor to all repos
2. Do you do C#?

________________________________
From: SAC-CS112-NGUYEN-TUAN <notifications@github.com>
Sent: Thursday, October 1, 2015 2:15 AM
To: SAC-CS112-NGUYEN-TUAN/Rock-Paper-Scissors-game
Subject: Re: [Rock-Paper-Scissors-game] GUI input (36a4dba)

After a Draw game it will be bugged

-
Reply to this email directly or view it on GitHub<https://github.com/SAC-CS112-NGUYEN-TUAN/Rock-Paper-Scissors-game/commit/36a4dbaff23666efe7b60575b0e8578aa6c07b2e#commitcomment-13536608>.


I'm just following the same best practice everywhere, I prefer short syntax too...


>> you have to go look at the module file which could be anywhere in the current PYTHONLIB

> But that's the case regardless of whether you use this shorthand or not? This shorthand just saves you the class name (by virtue of the module providing a reference via the pre-defined gabbi_response_handlers contract).

In the old way: You have to go look at the module if you want to know what each response handler does.

In the new way: You have go look at the module to know what response handlers are being used.

That is not the same.


In C++, there is no GC ;-)


Thanks :+1:


I'll rename it.
Using the 'Ce' prefix or not is a question of its own...


Should we use `"cannot use keyword '%s' here."` message for all of these?


this is the first check in by suriya


done


no use for


Thanks :D


Hey! Don't hesitate to PR this :+1:


Oops, this was meant to go straight into master..


Thanks :grin:


IMHO, we should always use sized types. Why ? Because it is architecture dependant. Believe me, when you have a but because you send an int (32 bits) from a computer and got everything messed on the receiving computer because it read it as a 64 bits (corrupting all the following data), you prefer sized types ^^


Why two calls? Why can't the store method just store both sides?


agreed


Oops :P What happens to e.g. a local uint32_t or std::string when the function terminates then?


I wanted to, but I saw you recently changed from coffee to es6.
It's in my todo list to rewrite it in es6 style and submit a PR ! :)
Great plugin btw.


Hi there!
Quick questions about locking cores to 6:

Is there any reason for LKVM to run on a single core?
Is 6 picked at random, or it could be any core?

This doesn't look good for machines with only 5 cores :)

Thanks!


Actually it is not, but I wanted to get this uniform everywhere, and it's more natural to add it`/` for root of the project


Indeed :( I should stop merging pullreqs late at nights :)


In reality, warmup is the time during which the creature cannot cast another skill or even move. Practically, it can be seen as the time needed to play the skill animation. It (can) avoids fast units to always hit and run (unless we want them to by decreasing warmup).
IMHO, it must be a per skill because we might have a very strong attack for a level 30 tier 4 that takes more time to do than the level 1 skills ^^


Not the same : https://github.com/LimeSurvey/LimeSurvey/commit/6a2a42753c155ee5f1f7125024f1947832dbb0a5 ???? gloups ....


Does this fix the problem I mentioned here? Or shall I write a ticket for this?


We have problems with the new version.

The speed for the request is reduced tremendous.

Look at the picture. Every select takes 500 ms. For simple tables without FK!

![yii_kcu_select](https://cloud.githubusercontent.com/assets/4319814/10217572/0328692c-6831-11e5-9a86-be6a4c76dbd5.png)


Yes but they will later be when we will have the "frightening trap" ^^


user can now*


Hi,

I believe was missleaded by your statement. This is not true or false for the 8th parameter like you stated. It is Configuration::get('PS_REWRITING_SETTINGS'). Like in the above commit.

The correct call is {$link->getProductLink($color.id_product, null, null, null, null, null, $color.id_product_attribute, Configuration::get('PS_REWRITING_SETTINGS'), false, true)|escape:'html':'UTF-8'} so the current implementation.

- This is working on 1.6.0.9.
- This is not working on 1.6.0.14 and 1.6.1.0 when url rewrite is off, the anchor is not right. In this case you can not replace Configuration::get('PS_REWRITING_SETTINGS') by true, because otherwise all the product link is url rewrited (when rewrite is off), not only the anchor. So the second initial problem which leads me to add $add_anchor, there is nothing you can do here to fix this case. {$link->getProductLink($color.id_product, null, null, null, null, null, $color.id_product_attribute, Configuration::get('PS_REWRITING_SETTINGS'), false, true)|escape:'html':'UTF-8'} will not work (ie their product.js will not react) for this two versions when url rewrite is off, all others versions are ok whatever url rewrite.
- This is working above 1.6.1.0.

So according my tests, you have indeed to edit your modules with {$link->getProductLink($color.id_product, null, null, null, null, null, $color.id_product_attribute, Configuration::get('PS_REWRITING_SETTINGS'), false, true)|escape:'html':'UTF-8'} but you do not have to make 3 conditions to handle backward compat.

Regards


Well, it does not make sense to have a hash that is not consistent with eq/neq.
As it is, the eq/neq in INET/Labelled/IPVPN are not consistent with this hash.


The ANSI C standard doesn't recommend to use underscore characters at the beginning of function names if they are shared between modules:
http://c-faq.com/decl/namespace.html
>The rules, paraphrased from ANSI Sec. 4.1.2.1, are:
>    1. All identifiers beginning with an underscore followed by an upper-case letter or another underscore are always reserved (all scopes, all namespaces).
>    2. All identifiers beginning with an underscore are reserved for ordinary identifiers (functions, variables, typedefs, enumeration constants) with file scope.
>
> ...
>
> What do the above rules really mean? If you want to be on the safe side:
>
>    1,2. Don't give anything a name with a leading underscore.


it will come, pls be patient....


looks like a constant value + array initialization is a resource-intensive operation = better to initialize once at constructor/define as a protected(static) property


Lines 174 - 175, where it's setting the error messages.  The first line makes sense because the PDF file is secure so it can't be scanned, but then the message_content says "please choose a valid file extension."  That would be a different problem altogether.  Need to make sure the messaging matches up with what happened.


Ah thanks, that makes sense. I wonder if the "warmup" name is clear enough though, for me "warmup" is the time needed to prepare an action (e.g. let a railgun warm up before it can fire), so it's a bit misleading IMO to use it for the time needed to "do" or "recover from" an action.

Not sure I have a better idea though. IIRC we already discussed this point regarding spells, but I don't remember what came out of it :P


Nei, la inn feil, sko prøva koden i min egen database


Yeah, did a commit amend, but had already pushed. Sorry.


In that case, I believe your issue is with *multiple* response handlers being loaded from a single module? That I can understand.

So would you be fine with this if `gabbi_response_handlers` wasn't a list, but just a single response handler class?

More concretely:

    $ pip install gabbi-html
    $ cat .../gabbi_html/__init__.py
    ...
    gabbi_response_handler = HTMLResponseHandler
    ...
    $ gabbi-run -r gabbi_html


@quintesse actually I think I would prefer not to get into the whole question of "headers" in the spec.


hardcoded 250MB should use the same function and the environment variable


Sounds good :D
A good ol' scarecrow trap :)


new changes


D'où sort ce db ? Ce code a-t-il été testé ?


> so I'd need to call it `valueApply()`

Yes, I realize that. I think it's acceptable.


Ce serait pas plutôt console.error ?


Maybe this error message could be slightly more verbose, like saying the actual space needed and the space left.

At least it needs an upper case letter.


Read another inline comment. Don't use hash. It won't work on browser with disabled javascript.


Nice, this is really accurate.


as arrays are not passed by reference, each call to getCode initializes and copies huge array into a temporary variable. As this operation is done in scope of a class, you can use property access instead of method


@kdambekalns tools running locally should rely on the metadata about installed packages (or on the lock file) IMO, as these are the metadata used by composer (if there is a mismatch, which can happen if you use a ``package`` repository while the code has a composer.json, the repository wins, not the file in the downloaded code)


Read another inline comment. Don't use hash. It won't work on browser with disabled javascript.

Use query string instead:

```google.com/search?q=memahami+arti+cinta```


True. I think I should try to start setting up as many query scopes as possible for Event; the controller is getting overkill


Yes, ok, I agree that passing a string to new is better.


no usages in this interface


This commit implements and fix #30


Polished the Program. DHCP and VLANS


getsolve is not defined properly (Note : l is missing in solve)


Ok, so by now you've probably realised you're typing this out a lot. There is a better way of doing this using something called Dependency Injection, you can find some docs about this on the [Service Container](http://laravel.com/docs/5.1/container#introduction) page.

There is a class called `Guard` which is a contract of the `Auth::user()` method. Look at `./Http/Middleware/Authenticate.php` for how to use Guard.


This commit includes a lot of changes. Please provide details why / why he does it.
What tests he fix?


getsolve not defined but used in list set-up


It is my understanding that you always update ``*patchentry->original = g_vld._RGetProcAddress(module, procname);`` on line ``1783, 1792, 1800`` since ``if (patchentry->original != NULL)`` is expected to be true.

Maybe the test condition != is wrong or my understanding is wrong?


Why you comment this?


I don't know C# , Java is my first coding class.


getsolve is used again but not defined because of spelling mistake


We can improve upon this with something like:

```php
$user->update([
    'about' => Binput::get('about'),
]);
```


> There is also the thing about `shared` members of `native` classes needing to be declared `native`.

Actually that restriction only applies to `native` implementations, not to headers. Given that I want to avoid talking about that distinction in the spec, I'm inclined to just call this done, at least for 1.2, and document the finer points of it elsewhere.


Wouldn't it be better to update mPhyAtk and mMagAtk each time a creature levels up, instead of recomputing it at each attempt to deal a blow?


There is no client code which needs whole codes configuration, as it need a description of a concrete code (getCode($type, $code)), so this method can be removed from interface


Cả chrome với firefox đều chậm hơn khi em vẽ 1 triệu tam giác
On 1 Oct 2015 16:30, "ducthangho" <notifications@github.com> wrote:

> Browser gì, số lg items là trong 1 canvas là bn. Nếu ít items thì canvas
> nhanh hơn webgl. Nếu dùng chrome thì canvas có tự đọng dùng card màn hình
> nếu canvas đủ lớn, do vậy khả năng là pixi sẽ chậm hơn. Nên test trên cả
> firefox nữa. Nhớ check bộ nhớ xem thế nào nhé
>
> On Thu, 1 Oct 2015 4:25 pm minhpl <notifications@github.com> wrote:
>
> > Có anh ạ, nếu dùng mặc định autodectectRender thì nó tự động dùng webgl
> > On 1 Oct 2015 16:15, "ducthangho" <notifications@github.com> wrote:
> >
> > > Để tối a xem. Browser của e có support webgl ko
> > >
> > > On Thu, 1 Oct 2015 4:09 pm minhpl <notifications@github.com> wrote:
> > >
> > > > Em thêm 2 file index.html và index.html2 để so sánh canvas và webgl.
> > Một
> > > > cái em dùng PIXI.CanvasRenderer và một cái em dùng PIXI.WebGLRenderer
> > sao
> > > > em thấy dùng canvas nhanh hơn nhỉ
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/minhpl/leaflet-maskcanvas_needfix/commit/b8ad8c684921b91142d456d67eb7b059cb9b11a6#commitcomment-13536479
> > > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/minhpl/leaflet-maskcanvas_needfix/commit/b8ad8c684921b91142d456d67eb7b059cb9b11a6#commitcomment-13536600
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/minhpl/leaflet-maskcanvas_needfix/commit/b8ad8c684921b91142d456d67eb7b059cb9b11a6#commitcomment-13536764
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/minhpl/leaflet-maskcanvas_needfix/commit/b8ad8c684921b91142d456d67eb7b059cb9b11a6#commitcomment-13536848>
> .
>


What problem does this fix?


> Read another inline comment.

Where? :')


Oh, got it, thanks


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


I won't replace it... since I will remove the check completely :p


The swift compiler cannot infer the return type...


Yeah, they are superfluous.


SendGrid можно прописать как бин в конфиге сразу с нужными пропертями (ключ) и здесь инжектить через @Inject


worderrorでました。

Error Found in d:\mozja\l10n\aurora\ja-JP-mac\browser\chrome\browser\devtools\debugger.dtd:
@line 46: <!ENTITY debuggerUI.autoPrettyPrint     "[WORDERROR[最小化]WORDERROR]されたソースを自動的に整形表示">
@line 101: <!ENTITY debuggerUI.autoBlackBox           "[WORDERROR[最小化]WORDERROR]されたソースを自動的にブラックボックス化する">


You could bump it up to 4.2.4. It's good to start with the newest packages whenever you start a new project.


m_format | FORMAT_WORDBREAK is always true, isn't it?. So this else if is actually an else clause running the wrapping algorithm even if not needed I guess.


but we use it.


Would it be better readable to keep using `SubscriberList` everywhere instead of `klass`? Using a let here only saves us a couple of characters.


oh I am dumb, because of the other parts it's not else but the mformat | FORMAT_WORDBREAK and the other is still always true


If exception occurred because of us, no one will handle it.


nice work. for git, it makes sense to make more commits. so when you finish your work in the storyboard, make a commit. when you finish a minor feature, make a commit. it's a good habit so that you don't lose your work, and so you can undo any changes if you screw something up. for example if you make some autolayout changes that break things, you will wish that you saved a commit before adding autolayout but after adding in the image and views. this also helps other developers see what progress you made, because all i see is "initial proj version" but it's actually your final proj version as well.


@sheelio hey sheel, I change this to be LEFT instead of RIGHT (your example had RIGHT join). We don't want rows with just months and not any cases, right?


<?php
 echo "Hellow World";
?>


Very clever implementation overall, it makes everything so much simpler that it is all skill-based.


good autolayout stuff


Ok, để tối vè a xem

On Thu, 1 Oct 2015 4:52 pm minhpl <notifications@github.com> wrote:

> Cả chrome với firefox đều chậm hơn khi em vẽ 1 triệu tam giác
> On 1 Oct 2015 16:30, "ducthangho" <notifications@github.com> wrote:
>
> > Browser gì, số lg items là trong 1 canvas là bn. Nếu ít items thì canvas
> > nhanh hơn webgl. Nếu dùng chrome thì canvas có tự đọng dùng card màn hình
> > nếu canvas đủ lớn, do vậy khả năng là pixi sẽ chậm hơn. Nên test trên cả
> > firefox nữa. Nhớ check bộ nhớ xem thế nào nhé
> >
> > On Thu, 1 Oct 2015 4:25 pm minhpl <notifications@github.com> wrote:
> >
> > > Có anh ạ, nếu dùng mặc định autodectectRender thì nó tự động dùng webgl
> > > On 1 Oct 2015 16:15, "ducthangho" <notifications@github.com> wrote:
> > >
> > > > Để tối a xem. Browser của e có support webgl ko
> > > >
> > > > On Thu, 1 Oct 2015 4:09 pm minhpl <notifications@github.com> wrote:
> > > >
> > > > > Em thêm 2 file index.html và index.html2 để so sánh canvas và
> webgl.
> > > Một
> > > > > cái em dùng PIXI.CanvasRenderer và một cái em dùng
> PIXI.WebGLRenderer
> > > sao
> > > > > em thấy dùng canvas nhanh hơn nhỉ
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > > >
> > >
> >
> https://github.com/minhpl/leaflet-maskcanvas_needfix/commit/b8ad8c684921b91142d456d67eb7b059cb9b11a6#commitcomment-13536479
> > > > >
> > > > > .
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/minhpl/leaflet-maskcanvas_needfix/commit/b8ad8c684921b91142d456d67eb7b059cb9b11a6#commitcomment-13536600
> > > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/minhpl/leaflet-maskcanvas_needfix/commit/b8ad8c684921b91142d456d67eb7b059cb9b11a6#commitcomment-13536764
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/minhpl/leaflet-maskcanvas_needfix/commit/b8ad8c684921b91142d456d67eb7b059cb9b11a6#commitcomment-13536848
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/minhpl/leaflet-maskcanvas_needfix/commit/b8ad8c684921b91142d456d67eb7b059cb9b11a6#commitcomment-13537219>
> .
>


cc @TylerSheffels


No, the issue is with visibility: Answering "What has happened as a result of doing this?" requires deeper inspection.

However, you've convinced me not too worry too much about it. Your suggestion in your most recent comment, which effectively removes the most useful power of your initial proposal, highlights that power well enough to make me be willing to capitulate.


Hello,
I have an issue with this commit:
```
remote: error: object 7b5e259b1d3c54ee33b94f04d7881616857cc593:invalid author/committer line - bad time zone
remote: fatal: Error in object
error: unpack failed: index-pack abnormal exit
```
Any idea?


Thanks @sjmulder for the feedback. Wasn't quite sure where I got that, but I just remembered. So when I was running `gem build ipa.gemspec` it gave warnings and recommendations for fixing them using compound requirements and I applied them. What do you think is best: Revert, change or keep?
```sh
WARNING:  pessimistic dependency on CFPropertyList (~> 2.2.0) may be overly strict
  if CFPropertyList is semantically versioned, use:
    add_runtime_dependency 'CFPropertyList', '~> 2.2', '>= 2.2.0'
WARNING:  open-ended dependency on bundler (>= 1.0.0, development) is not recommended
  if bundler is semantically versioned, use:
    add_development_dependency 'bundler', '~> 1.0', '>= 1.0.0'
WARNING:  See http://guides.rubygems.org/specification-reference/ for help
  Successfully built RubyGem
  Name: ipa
  Version: 0.1.3
  File: ipa-0.1.3.gem
```


It's so easy now that you've made it all skill-based :D


tout va bien en faite, j'avais une erreur de rebase, j'y est corrigé et c'est niquelle maintenant :)
en faite  certain de t'es commit que tu avait effacé étais encore la, je les est supprimé en y rebasant et c’était bon, fausse alerte dsl ^^


BTW, this would be a very odd situation.  Basically a developer would have
decided to change a persistent checkpoint into a nonpersistent one.

On Thu, Oct 1, 2015 at 4:58 PM, Andrew Stone <stone@openclovis.com> wrote:

> if the checkpoint is not persistent, we do not want to look on the disk
> for the data, even if it exists.  A non-persistent checkpoint means no disk.
>
> On Wed, Sep 30, 2015 at 9:03 PM, hungta <notifications@github.com> wrote:
>
>> initDB should be called even if this checkpoint is not persistent. The
>> purpose is to copy data from disk (if it exists) to checkpoint (please see
>> the initDB() implementation)
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/OpenClovis/SAFplus-Availability-Scalability-Platform/commit/ec8149fc39d712a7afbf15c36d2d05cba22234fa#commitcomment-13533176>
>> .
>>
>
>


This modifies the contents of the entire collection, right? not just "an asset or source file"


@davidsh - Okay - I have an additional change to make based on other comments anyway.


needs a trailing slash, since we're addressing the entire collection?


My point was - I think I have gotten that same thing working. We just force
the moduleResolutionKind to Classic and then we don't need to fix any
imports - but the generated .d.ts files have relative paths so loading
seems to work for users with moduleResolutionKind Node

On Thu, Oct 1, 2015 at 4:10 PM Wesley Wigham <notifications@github.com>
wrote:

> There's still a part of me that enjoys the lack of absolute names this
> commit has. Makes it compatible with node module resolution without
> depending on your own .d.ts file. Which is mostly why I did this - I
> wanted to try out commonjs/node bundling, and I wanted it all to resolve
> via node module resolution.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/weswigham/angular/commit/6bf51ddd4a4a8d8f037766730bf6e6ba2785dce4#commitcomment-13553076>
> .
>


Good call.


Nope, I just did `gem install asciidoctor` and generated html as `asciidoctor readme.adoc`. Wondering if the submodules are even required?


So I'm providing a trailing slash from the caller, but I need the slash and the query params. The other option would be to provide the query param as an argument to patch as well, though it feels a little bit that we do that here but not in other api calls.


I would use the same identifier in vagrant and aws.

I'd prefer `virtualenv`.


should VBoxManage be /usr/local/bin/vboxmanage?  Is it immediately available on the path after install?


Of course, I meant "bit 2" for KForceUseHandler.


well, I take that back, it doesn't have relative paths in the generated
.d.ts files - but it still works, I think because it knows how to find
those under the 'angular2' node module?

On Thu, Oct 1, 2015 at 5:02 PM Alex Eagle <alexeagle@google.com> wrote:

> My point was - I think I have gotten that same thing working. We just
> force the moduleResolutionKind to Classic and then we don't need to fix any
> imports - but the generated .d.ts files have relative paths so loading
> seems to work for users with moduleResolutionKind Node
>
> On Thu, Oct 1, 2015 at 4:10 PM Wesley Wigham <notifications@github.com>
> wrote:
>
>> There's still a part of me that enjoys the lack of absolute names this
>> commit has. Makes it compatible with node module resolution without
>> depending on your own .d.ts file. Which is mostly why I did this - I
>> wanted to try out commonjs/node bundling, and I wanted it all to resolve
>> via node module resolution.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/weswigham/angular/commit/6bf51ddd4a4a8d8f037766730bf6e6ba2785dce4#commitcomment-13553076>
>> .
>>
>


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (822 of 822 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-24.31%25-green.svg)


issue5221_memoryAllocation (rongpengzhang) - x86_64-MacOS-10.9-clang: Build Failed

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


change this to "Error : {exception}".format(exception=exception.message)


This fixes #53 btw.


This could more compactly be written as

```python
l = [x for x in zip(*seqs)]
```


@Anmol-Singh-Jaggi convert your tabs to spaces and make sure one tab = 4 spaces


Ohh I see. I'd be fine with providing the query params here even if htey are not in other api calls. people can add them following this example as they are needed.


Reviewed and everything looks good to go


Not what the instructions stated. This is not valid.


jj


The default value should always be native to the type: IE: https://github.com/creesch/reddit-moderator-toolbox/blob/master/modules/modmailpro.js#L81 and https://github.com/creesch/reddit-moderator-toolbox/blob/master/modules/modmailpro.js#L67


I would expose availability in `FunctionRegistry` by splitting `getScalarFunctionImplementation` into 3 functions (1 private, 1 returns `ScalarFunctionImplementation`, 1 returns `boolean`).


Nice commit message typo.


how about `patch` --> `patchAll`, since this affects the entire collection. otherwise we'll have to rename this if/when we want to patch an individual item, which is something we are considering doing eventually for the channels api: https://www.pivotaltracker.com/story/show/95126834


I'm just a fresh to FRP, and not very familiar with the optimization in FRP. I was wondering why don't realize that thing in that more elegant way, when first met the source code. Maybe I need to do more work in internal optimization of FRP.
I never care about honor or something else, so keep it free for changing my request.
Ps. Looking forward the new version of Yampa. :P


Fixed in 5f623b8784a6f6c45437726c3f0b73f998f4145c.  Thanks!


:zap:


コメントだけ大文字で残ってました。
修正しときます。


Nit: First sentence is a little too vague. Suggest changing to "ApplyOptions stores cmd.Flag values for apply."


nit: I'm not too crazy about camelcase in URLs. Let's do something like '/api/artwork/url/original/:filePath' instead.


'/api/artwork/url/thumbnail/:filePath'


'/api/artwork/url/original/:filePath'


In fact I think lines 263-274 can be more clearly written with a couple splats

```python
seqsBis = []
for seq in zip(*seqs):
	seqsBis.append([reduce(operator.__mul__, [AC(q,c) for q,c in zip(qubits, pulseNums)]) for pulseNums in zip(*seq)])
```


not much place to break anything ;)


ok, take  a look at latest commit: ebaf0a353bad71de52ab29dbf0373306db3542f7


Orleans taxonomies 0.0.0.1

Board
	Agendae
	Daft Minutes
	Approved Minutes
	Bye Laws
	Contracts
	Executive Committee
Members
	Member List
	Gossip Sheet
	Stuff at the camp
	Guardiens and contacts
	Reservations
	Trip Log

Accounting
	Billing
	Dead beats
	Guardiens invoices
	Bank
	Accountant


I don't know neither. I just tried to avoid external references in this commit.


MINC1: https://github.com/BIC-MNI/libminc/blob/master/libsrc/hdf_convenience.c#L1322
MINC2: should be around here: https://github.com/BIC-MNI/libminc/blob/develop/libsrc2/volume.c#L621


Hi,
Thanks for the pull request. AFAIK, You see only one InfoWindow open if you define only one infoWindow. I feel this is reinvention of wheel, but I may not know your exact use case.


GHX_Vert_IDD (mmatts) - x86_64-MacOS-10.9-clang: Tests Failed

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


So where's my comment? Totally truncated..


Hiện tại anh đã sửa lại code benchmark vì gần như vẽ theo kiểu dùng graphics là cách vẽ rất chậm và ko nên dùng. Chủ yếu là phải đo tốc độ vẽ image lên.
- Hiện tại Pixi webgl nhanh hơn Pixi canvas (chứng tỏ webgl nhanh hơn thật)
- Tuy nhiên, nếu mình chỉ vẽ các ảnh cố định lên trên canvas thôi thì canvas thường còn nhanh hơn webgl. Có thể là do cách làm của PIXI, hiện anh thấy lần nào cũng phải load Sprite từ 1 cái texture có sẵn như thế thì ko thể bằng canvas thường chỉ vẽ đúng 1 cái Image ko phải load gì hết.

- Em thử xem còn cách nào khác với PIXI webgl ko, nếu ko thì có lẽ dùng Webgl thường (tự viết các shader) thì sẽ nhanh hơn cái canvas thường.


I think this should just return an error if there is no object read from input, rather than defaulting to server-side. You're using a resourcebuilder with `FilenameParam` so `info.VersionedObject` should never be nil.


Similarly to what you do on delpoyed machines, I think `python setup.py develop` should be run in Vagrant. Do I miss a reason so that you did not do it ?


When `setup.py` does not pin versions and does not use `requirements.txt`, will we have multiple versions of the depending packages installed ?


This is great, but can you break it out into a separate test suite with a setUp method that creates the explorations in the datastore (and publishes some)? Then the search queries can be one test, the paging limit can be another, the rights can be another, and the deletion can be another. I think it's cleaner that way.


Fixes "Resume" playback option does not work #7


Ceci est mon premier vrai commit sur Github.


Chabon has tirado muchos commits en poco tiempo


I think an unknown activity type merits a real exception...

How about (this is just a sketch):

    if activity_type == ACTIVITY_TYPE_EXPLORATION:
        try: ... except: ...
    elif activity_type == ACTIVITY_TYPE_COLLECTION:
        try: ... except: ...
    else:
        raise


// err is not handled here because it is handled by app.Deploy


Typo:  Propety


Trailing whitespace


Then maybe just sanitize with the appropriate generic function, mysqli_real_escape_string (or mysql_real_escape_string if we aren't using mysqli yet).


cool !


This is an application not a library. One key difference is applications have auto-generated resources accessible via the `R` constant.


this reads as a commit message...


Dependencies should always be specified in the `group:name:version` String format for consistency. Most of these don't appear to be used for anything.


Not sure what this means. Maybe give examples.


Dependencies for Android apps shouldn't ever be included by classes jar. They must be pulled in from jcenter.


Good job!

It's not a must but I recommend you to comment the ai related files in the karma config file to make the test looks more clear (just showing the result of gameLogic tests).


junit/hamcrest/mockito are all used for testing, not for compiling into the actual apk file itself.


Did you update the test to check for webOS using the capabilities?


This should be `validateArgs` I don't think you meant to call [this ValidateArgs](https://github.com/kubernetes/kubernetes/blob/master/pkg/kubectl/cmd/create.go#L73), even tho they happen to be exactly the same. I recommend changing the name to `validateApplyArgs` or hanging the method off of `ApplyOptions`. I'll send a pr to make the one in cmd/create.go less generic.


Suggest separating into line items (prefixed by hyphens) -- hard to tell where the break is. Also, put keys of dict underneath the first part. I.e.

```
    A tuple with two entries:
    - A list of dicts ... the keys are:
       - type:
       - activity_id:
    - A list which contains ... feedback threads
```


I think it should? As per node module resolution in node proper? but I'm not sure if it actually does, is the thing. Or rather, thinking it through, if my working file is `modules/angular2/test/core/directives/ng_class_spec.ts` and I'm trying to import `angular2/angular2` in it, according to node module lookup, I'll look for an `angular2` module in my node modules folder and attempt to include the `angular2.js` or `angular2/index.js` file from it. (Likewise, `angular2.d.ts` or `angular2/index.d.ts` for types). None of those things actually exist before a build, afaik... (and even after a build they won't be in a node_modules folder[1](https://nodejs.org/api/modules.html#modules_loading_from_node_modules_folders))

Actually I don't think that style of nonrelative imports works particularly well with node resolution here.


User B


Good job!


@elia Thanks for this; even though I use `Native` all the time, I occasionally forget what some things do. Plus, this will be a great help for newcomers who don't want to wade through all the code :)


No, if we installed a package via pip install -r requirements.txt and then setup.py has install_requires, it won't install a second version


Long story short, the handling or nonrelative module identifiers differs between resolution kinds (whereas relative is the same). Node looks inside `node_modules` folders, classic does not.

Out of curiosity, what editor have you guys been using for typescript? Part of the reason I wanted this was because swapping it to all relative imports meant that the editor became actually able to resolve all types inside the editor (since the tsconfig says that they should be commonjs modules, it uses node resolution to try to find things in the editor).


@DamianEdwards FYI


I was trying to match what was being done for the other app types, but I believe that has already changed since I wrote this.


As a rule, absolute paths begin with `/`. This tells whatever is parsing
the path the starting point: `/`, the root directory.

An absolute path, e.g., `/etc/hosts`, will point to the same file
regardless of whether we're in `/home/jacob` or `/usr/bin`. (These paths
may not exist on a Mac. I think the home directories on a Mac are in
`/Users/username`.)

On the other hand, a path like `projects/unit1` will resolve differently
depending on where we are currently: if we're in `/home/ashley`, it'll
resolve to `/home/ashley/projects/unit1`; if we're in `/tmp`, it'll
resolve to `/tmp/projects/unit1`. We call `projects/unit1` a relative
path for this reason.

If this doesn't clear anything up, come see me during a break tomorrow
(or ask about it during review!) and I can demonstrate it live.

On 10/01/2015 03:38 PM, J.S. Toy wrote:
> Im a little confused still. Are you saying that an absolute path will
> begin with '/Users/...'?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/toymechqm/unix-cli-intro-assessment/commit/ecc5b30d3b3519dee2aa7f7ed40d8f6922b73b34#commitcomment-13549370>.
>


The proper way to bring down your mesh interface is terminating your wpa_supp.


Should this be || instead of && ?


Sorry, should be `\tex_end:D` here to stop tex running.


Okay, here's the editor oddidy: If you open the editor on the root of the repo, the editor can resolve the references, but if you open it on an individual module folder, you cannot. Fun.


Or I could be imaging things and this was only an issue before an PR got merged that fixed it.


Thanks.
I don't know how I missed that!


Doesn't _really_ matter. There's a reason I just made this branch for myself. :3


What do you think about converting this to options {body: , headers: , client_timeout} rather than body, headers, client_timeout? I think this would make this much easier to deal with going forward. I almost did it the last time I was refactoring this code.


Should this be !=?


Yup, but `setup.py` is ran first, iow `requirements.txt` will come too late.


this just magically works? why is there no corresponding change?


This looks like an ABI break in RC stage (or 7.0 branch at least).


See the comment above. The hash options here will make these methods a little more friendly.


You have my blessing. 🙏


Makes sense!


OK, here's a PR that does that. https://github.com/jjjake/internetarchive/pull/98
I left them in the test code (where I think they serve more of a purpose) and used for classes (where they are infrequent enough to be worth leaving, I suppose).


ha, with ArrayMap from android, interesting.


Good Catch!


If the method moves to options, then you can trim these expectations down with http://www.rubydoc.info/github/floehopper/mocha/Mocha/ParameterMatchers#has_entry-instance_method. This will probably save your hands quite a bit of expectation wrangling.


Not a fan of using regexes to parse the header. The readability is kind of low, and probably error-prone to maintain. Note that the legacy Ruby parser has a tokenizer for this scenario, though in the case of C, it should be fairly easy to tokenize (split by `","`, split by space, collect all names, etc.).

Other than that I like this.


good suggestion for the Map object, definitely gonna do that !

today we close #549, i'm excited, ahahah :P


Good catch!


so do we make of the platformer 2 the new platformer ? and remove the old one ?


Good job with the unit test.

You should change the README file of your repository from TicTacToe to the name of your game:)


Looks pretty good, but would be nice to clean up the call method signature.


however, there's a few caveats to using `ArrayMap` that I hope you're aware of.

>Note that this implementation is not intended to be appropriate for data structures that may contain large numbers of items. It is generally slower than a traditional HashMap, since lookups require a binary search and adds and removes require inserting and deleting entries in the array. For containers holding up to hundreds of items, the performance difference is not significant, less than 50%.

Not appropiate for large number of items (< |999| => performance hit < 50% hmm, I don't know @aldenml ) .
Slower than hashmap due to binary search, not sure if I like that.


Have functions to create 30 cars, set their locations, figure distance to car in front, make moves (accelerate, decelerate, stop, or continue cruising. Also have a function that moves all cars ahead based on car speed and updates location of each, along with passing updated info to car behind.


I don't know if the case of zero length data could happen or not. But if it happens, the web socket will be closed as an error. I'm asking this because if the msg is an NSString type and the msg's length is zero, the web socket doesn't get closed as an error.


Does this distinction matter? The name will be the same in either case.


I see where you're coming from. This code already made `offset` a public member of the object though, so technically it's not hidden any more. [What I ended up doing](https://github.com/sjmulder/nbt-js/blob/02860e00fe88cba274ceb0f83118abc957474c1a/nbt.js) was to keep it an attribute so it's a two way street, mirroring `nbt.Writer`.


This is a ready run example of a jndi configuration with maven jetty plugin.
The require installation:
1. Access to an instance of Oracle database (e.g. XE Edition)
2. Maven 3.x.x
3. Oracle JDK 1.8


Random words


* Added the Job.GetSearchResponseMessageAsync which enables you to fetch raw search results in a specific format: JSON, CSV, or any of several flavors of XML.
* Add the search-response-message-stream example which shows you how to execute long-running search jobs and how to use Job.GetSearchReponseMessageAsync.


Another comment


# MORE COMMENTS


@zassmin would you consider listing the Flask resources under the 'Python' section?

Something like:

Python
* Flask - a lightweight Python web framework
    *  http://flask.pocoo.org/docs/0.10/tutorial/


Figaro or dotenv. They do essentially the same thing. Pick one and go with it.


I do think that starting without the utility classes is a lot more educational - I was planning to rely on that to show how to add fields to schemas, and I think it also documents what the utility classes actually do pretty well.

I was also hoping I might convince the tutorial-runners to make an exercise out of rewriting the example plugin in C++ and/or using the utility classes.  If not, I'll provide a less-annotated example using the utility classes myself.


The `araphic` package which is depended on `cjk` is required since we want to use `gbsn` instend of `zhmetrics`. So I think our list is
```
tlmgr install adobemapping araphic cjkpunct environ fandol lualibs luatexja ms
tlmgr install trimspaces ulem xcjk2uni xecjk xkeyval zhnumber
tlmgr install --no-depends ctex cjk
```


will_paginate and kaminari. They do essentially the same thing. Pick one and go with it.


I changed it because the type changed from a pointer to a Handle (from ctrulib):

https://github.com/cpp3ds/cpp3ds/blob/26c1836d833cb0384402ef4935b5a26a6095256d/include/cpp3ds/System/Thread.hpp#L209

I just forgot to update this to reflect that change.


Based on our conversation, you likely don't need omniauth and friends any more.


This is a breaking change, please use `BREAKING CHANGE:` in the commit message.


@sjjf seems like this was obviously a typo but just wanted to double-check with you...

Also I think it is clearer to put the units in the column names ("size_gb" etc.) rather than naming the column "size" and putting the units in a comment. But this is what's done in openstack, so I guess it was a conscious decision to rename the columns and I'm wondering about the rationale behind that?


It looks like we are only supporting integer values, maybe add `step` attribute so html5 form validation will allow it? http://stackoverflow.com/a/19012837/744228


Hi @jdubois  just out of curiosity, what does this .log do here?  And where is the install method called?


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1360 of 1360 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.36%25-green.svg)


Good job!


It's done


:+1: !!


No no no no... :-1:


Ví dụ vẽ cell thì dùng graphic dễ hơn, vì cell chỉ là một sector của hình tròn, và góc quay không cố định.
Nếu không thì có thể dùng một ảnh cell là sprite rồi xoay ảnh đi.


package namespace should be under edu.rpi.shuttles.android


Already fixed 😊

-----Original Message-----
From: "Eilon Lipton" <notifications@github.com>
Sent: 10/1/2015 5:50 PM
To: "aspnet/benchmarks" <benchmarks@noreply.github.com>
Cc: "Damian Edwards" <damian@damianedwards.com>
Subject: Re: [benchmarks] Fixed benchmarks to not use multiple libuv threadsdue to blocking issue on Windows. (16a41fc)

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


Typo introduced here.


Decided against mentioning EBS given that it doesn't work on CoreOS right now. The other thing we will have in the future is using native VPC routing rather than an overlay network (flannel).


Hey. Setup a call with me. You are going to need to explain this to me.


:dickbutt: :eggplant: :peach:


Hmm. I'm not sure I actually like this change any more. I think I'll stuff the schema into db/models.py instead, and have an `ensure_db()` function that's called by create_db.py. Might also call it from trivia.py and rename this to reset_db.py.

Only thing is that it'll make it slightly harder to run this script. Could either move it to the project root or do some filesystem existence checks instead.

@paked what do you think of this idea? This is definitely not inspired by how Kenni's rewritten for 2015 murder website works :stuck_out_tongue_winking_eye:


This is great


Unless you're using the raw map for a specific reason, I recommend using `testapi.Default.Codec` so you can work with the actual api objects. See example [here](https://github.com/kubernetes/kubernetes/blob/master/pkg/kubectl/resource_printer_test.go#L209).


actually, it looks like bcolz installs are broken now. Hopefully there will be a new release soon.


this is just copying the definition of the `scss` target. if you call `make scss` it's quite slow because it runs venv_update first.


Should use `testapi.Default.Codec` to serialize/deserialize so you don't have work with the raw maps. See comment below in e2e/kubectl.go.


SGTM. @auscompgeek


It seems on gh that multiple commits get merged into the same PR.

Changes to Fed 1040
   - input file takes filename containing personal details which will be used in all US forms as well as state forms.
   - input file takes 2 new input corresponding to key inputs of form 8959
   - input file takes 1 new input corresponding to key input of form 8960
  - dumps 6251 form's xfdf file, to be used to fill its pdf

Also note that the .h file containing the mapping of pdf fields to their keynames may contain extra struct members.    They will be ignored if identified by double-quotes.   It is extremely workflow friendly, as dumped pdf fields can be read in entirety, but only configured subset of fields are populated in tthe program.    The others are just left as is.


custom3="metakeywords" & custom4="siteid" was removed from both cfindex updates


Excellent!


## This is a markdown file


> Let's try the short version and expand only if it causes confusion. =)

From the conversation above I reckon @markokr got confused. I'd confuse too frankly speaking.


This is a previous commit that was superseded by this release https://github.com/nuxsmin/sysPass/tree/1.2.0.03-rc3 which takes care about that issue.

Thanks for your support!!


Won't that always evaluate to true?
```
var display = false
display = display || true; // true
```

You want

```
display = typeof display !== 'undefined' ? display : true;
```


I created a commit for that, but got caught up with real life. :laughing: Definitely want to do it, though.


What confuses me about this is: this means that `debian_enabled?` was *never* called before but this was its only call site. So is this code block actually needed? It's not a case of code drift because this call site and `debian_enabled?` were added in one commit: https://github.com/puppetlabs/puppet/commit/7fe61647f23650fc4c93cd6b54c654a20ff7c9f9.

If this is useful, great. It just seems odd that this code path has apparently never been executed. I may also just be missing something obvious :)


This description doesn't say anything...the `@return` documentation is better


@kalbasit why do you have the `-o -` here? Isn’t that redundant?


Đấy là 1 cách. Em cũng có thể dùng shader của webgl vẽ còn nhanh hơn rất
nhiều. Hoặc lưu trước ảnh của tất cả các cung được làm tròn theo số nguyên
(góc của cell ko cần tuyệt đối chính xác lúc hiển thị). Việc vẽ cell sẽ chỉ
còn là vẽ ra 3 cái ảnh của 1 cung.
Tuy nhiên vì các cells gần như luôn cố định nên có thể tạo 1 layer chỉ chứa
các ảnh của cells, lưu trên server. Khi nào cần hiển thị cell thì chỉ cần
load các ảnh này đặt chồng lên tile của bản đồ nền là xong. Khỏi mất công
vẽ vời nhiều. Cells rất ít khi thay đổi

On Fri, 2 Oct 2015 8:19 am minhpl <notifications@github.com> wrote:

> Ví dụ vẽ cell thì dùng graphic dễ hơn, vì cell chỉ là một sector của hình
> tròn, và góc quay không cố định.
> Nếu không thì có thể dùng một ảnh cell là sprite rồi xoay ảnh đi.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/minhpl/leaflet-maskcanvas_needfix/commit/77e1c7fa785707f1cbeaa773a34c4660bbbe97cd#commitcomment-13554388>
> .
>


No sé si es bueno eso jaja. La quiero sacar buena.


incorrect operation.


[non-binding] - ObjectVector.java: getBufferSizeFor() - I don't like the idea of any ValueVector method calling UnsupportedOperationException because these end up getting propagated to the end user. Why do we have this class?


[non-binding] -  Should we consider parameterizing OUTPUT_MEMORY_LIMIT?


Можно/нужно грохнуть все console.log, чтобы в master'е не болтались.


I cleaned up the script I used for testing. It is now available as `Build-SubtitleEditBeta.ps1` in [this folder.](https://www.dropbox.com/sh/thgk5507tteg6ze/AABgb8jgWBnz9LLgDHsECfika?dl=0)
After unpacking the most recent SE master source archive from GitHub, it sets REVNO and GITHASH, and replaces `build_helpers.bat` before invoking Visual Studio.

Note: To allow starting powershell scripts in explorer (context menu item “Run with PowerShell”), the default execution policy must be changed: press Win-R, type `powershell.exe`, click **OK**, execute following commands:
```
Set-ExecutionPolicy -Force -Scope CurrentUser -ExecutionPolicy RemoteSigned
Get-ExecutionPolicy -List | Format-Table -AutoSize

exit
```
This change is persistent (stored in the Windows registry).


У себя в консоли вижу
```
cookies.js:57 has item!
cookies.js:58 no item!
```
и фильтр не сохраняется (в куках пусто)


These are absolute now because the panel is relative to ```chrome://browser```.

@Standard8 I don't know if ```resource://``` or ```chrome://``` is the right scheme to use here.


No idea! Try it out!


コメントが正確ではありません。


許可することで何が出来るようになるのかまで書いて下さい。


!! ไม่มีกล่อง AsianFactor


That seems better.
Although, as it is a static data, I wiil put this property on Gitter# (the NuGet package) data : POCO values.


Just a comment; I think this would be better set and made configurable in the upstream bonita package, make Forms.php overridable or something.

Problem with direct tweaks is that they have a tendency to get clobbered every time we update from the upstream package.


I need to execute cadabra from jupyter with Ipython kernel. I intend to try make this with help from my friend @padawanphysicist.
However, today I found a [old post in the maillist of sympy](https://groups.google.com/d/msg/sympy/d6kcwDznzlE/hWsYrjGn0bEJ) with a claim about development version fo cadabra version 2, that seems to have just the solution for this.

Please, could you to make public the development version of cadabra version 2?

cc/ @kpeeters


コメント


我在博客园的ID是 http://www.cnblogs.com/upwifi, QQ是55763，

在创业中，已经拿到投资，游戏产品已经完成，现在在处理游戏细节，等细节处理完成后做上线钱的准备。

可以交个朋友么？


この`post`は具体的にいうと何ですか？


具体的に何を呼び出しているのか、詳細に書いて下さい。


I rewrote the mocker and it is now using strict mode.


もう少し具体的に書いて下さい。`@post.comments`は何ですか？


コメントを記載して下さい。


From passports' doc, try using the `successRedirect` as opposed to the last middleware in lines _24_ to _26_, and see if there's success.


Defines URL to load dictionary from


commit reverted


Can we change the count(*) to use count(episode_id) ? Both return the same, but the first one queries the primary key of the DB, instead of a full table scan:

'''
sqlite> .schema tv_episodes
CREATE TABLE tv_episodes (episode_id INTEGER PRIMARY KEY, showid NUMERIC, indexerid NUMERIC, indexer NUMERIC, name TEXT, season NUMERIC, episode NUMERIC, description TEXT, airdate NUMERIC, hasnfo NUMERIC, hastbn NUMERIC, status NUMERIC, location TEXT, file_size NUMERIC, release_name TEXT, subtitles TEXT, subtitles_searchcount NUMERIC, subtitles_lastsearch TIMESTAMP, is_proper NUMERIC, scene_season NUMERIC, scene_episode NUMERIC, absolute_number NUMERIC, scene_absolute_number NUMERIC, version NUMERIC, release_group TEXT);
CREATE INDEX idx_tv_episodes_showid_airdate ON tv_episodes(showid,airdate);
CREATE INDEX idx_showid ON tv_episodes (showid);
CREATE INDEX idx_status ON tv_episodes (status,season,episode,airdate);
CREATE INDEX idx_sta_epi_air ON tv_episodes (status,episode, airdate);
CREATE INDEX idx_sta_epi_sta_air ON tv_episodes (season,episode, status, airdate);
sqlite> select count(episode_id) from tv_episodes;
3650
sqlite> select count(*) from tv_episodes;
3650
'''


Need to check whether URL was set and it's correct


These functions cause the list to not be generic. How to do this in a generic way?


Also please make sure it's not loaded on every analysis as it's quite expensive


Nice :+1:


Finally managed to track this down. Line 109 means that on the first run it'll use the FQDN, even if people are using hostnames to define their nodes, which causes issues on the second run of Chef. Please can we revert this? If people have specified their peers they should choose to use FQDN's or hostnames there.


Done in df1a93eacea3855c544177bca52a411779efe139


Here is the change https://github.com/Odonno/Modern-Gitter/commit/d0b9787039b8dedbff09539d750cf13c34ac1caf


hej


My commit message was autocorrected by the lovely github app: Should read ''Add test post for downloads"


used in #4


Also it's only required if you want to use pulsecontrol as a backend. which is only used in our lab.


I'll go ahead and merge these changes into my fork, but out of curiosity, why disable retries on the error and finish workers?


I've switched RTD to go off the 3.0.x branch, rather than the tag, so that people will get the bits of the docs that we didn't manage to get into the release.


CAOCAO


The naming of `tryNever`, `tryNow` and `retry` is a bit more active than what they actually do.  Maybe `nextRetry{Never,Now,Backoff}` ?


I have contacted you by email to sort this out. If anyone else reads this and is interested, please get in touch by email.


SDFSFS


Does Qt check UF_HIDDEN on Mac?


Hello!,

We are getting very good results thanks to your library and our fuzzer (QuickFuzz based on QuickCheck): we reported bugs in Firefox, gdk-pixbuf, optipng and more. If you want to talk, drop me an email to gg ( a t ) cifasis-conicet.gov.ar


@rande can you send me such a file via mail or gist, so i can add it to the docs here


Great, as long as only 3.0.x-specific changes go in that branch… we're trying to avoid publicizing the next release's changes c.f. https://github.com/python-pillow/Pillow/commit/4d4dc99fa88ff6c3852c6d3d5ebacadab63d2e16


Karl Heinz, please feel free to change the version numbers afterwards.

I am just attempting to go the path of least resistance and notice
constantly, that Maven plugins have become a Jar hell.

On Fri, Oct 2, 2015 at 10:17 AM, Karl Heinz Marbaise <
notifications@github.com> wrote:

> Why not using an uptodate version see
> http://search.maven.org/#search%7Cgav%7C1%7Cg%3A%22org.codehaus.plexus%22%20AND%20a%3A%22plexus-io%22
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mojohaus/xml-maven-plugin/commit/ad410f4d49073c15b8e1694a2025ac6aec7d858b#commitcomment-13557456>
> .
>

--
The next time you hear: "Don't reinvent the wheel!"

http://www.keystonedevelopment.co.uk/wp-content/uploads/2014/10/evolution-of-the-wheel-300x85.jpg


Hi Micron10,
could you recommend me some server for new gateway (something what you already got tested)?
Situation:
- 1Gbps uplink to internet
- aprox. 1000 home users
- NAT
mail: 120(and)centrum(dot)cz

Thx


Accidental comment?


This won't work because `get_post_type` will return the post type as a singular value like `post` or `page`. The capability is plural like `read_private_posts` or `read_private_pages`.

I personally prefer the `! current_user_can( 'read_private_pages' ) && ! current_user_can( 'read_private_posts' )`. This makes it possible to search for used capabilities in the code when it's needed.


Tester doesn't exist in this context. This always fails, which breaks Nova unit tests.

Also, I'd expect pep8 to need some kind of quoting.


Yeah. The 3.0.x branch is only for changes that are in line to be the next 3.0.1, etc. We don't generally commit to those branches unless we're planning a release. But it's an ideal place for documentation only stuff that we want to push prior to the next semi major release.


Hi,
Since you made this change, we can no longer find older versions of ngDialog in bower as "ngDialog", which breaks our builds that rely on it. Anything you can do about this? We're working towards a release and if we have to rename ngDialog references in our code, then we will have to go through testing again.
Bruno


This can be simplified to:
```
<?xml version="1.0" encoding="utf-8"?>
<selector xmlns:android="http://schemas.android.com/apk/res/android">
    <item android:state_pressed="true">
        <shape>
            <solid android:color="@color/translucent_grey_lighten_20" />
            <corners android:radius="2dp" />
        </shape>
    </item>
    <item>
        <shape>
            <solid android:color="@color/blue_wordpress" />
            <corners android:radius="2dp" />
        </shape>
    </item>
</selector>
```


If both pkgs can share the patches why did not you make a symlink to the patches directory?


Fine. I've disabled 3.0.0 then (and back ported the docs badge so they'd show up on 3.0.x dcos)


Just found, that this will not work properly. You will have only count of seconds instead of splitting to days, hours, minutes etc.

```php
$interval = new \DateInterval('PT300S');
echo $interval->s; // 300
echo $interval->m; // 0
```


This should be "const char *".  Also, your compiler should have given you a warning when you tried to return "" as type "char*".


I know it's still very much alpha, but this should probably read `$this->t('Cancel')`

Thanks a lot for your work!


These tests are missing the main point of testing sandboxing: making sure that the sandbox can actually allow or forbid system calls.  They do not distinguish a correct sandbox implementation from a broken one that always returns success.


It's proabably safer to use TT_FORK along with MOCK.


Thanks, will remove; was keeping what was in the already in the file.


Thank you very much for your offer.

I'm KMY.
I'm developing jStorybook.

jStorybook ver.5 is based on your oStorybook,
but ver.6 is created from zero.

> I'll want to help you to develop this version. Please contact me.

I want you to help me.

I have two questions.

1.
Do you understand MVVM model?
This program is written with MVVM design model.
Within not MVVM partly, but almost is MVVM.

2.
What can I support for you?
I'm Japanese, and all of documents are in Japanese.
I'm not good at English.


The author said that 6.x has a hard dependency on systemd user units.

Please revert this update.


This doesn't test the success case.


Thanks @IcyApril I think I understand this change, do we still have to uncomment this when wishing to access Magento Connect?


@bc-2 I've added `ngDialog` to bower again for now, but I would like to have only one lib in registry.

I don't know what build system is on your project, for example in require.js you can provide an alias for the library.


请求合并


> Can assume P#T <: Q#T if P <: Q

@odersky : I wonder if this means that we don't need to set `variance = 0` anymore in [`TypeAccumulator#applyToPrefix`](https://github.com/lampepfl/dotty/blob//3ae9347bca8edbd46aa296570bb2b718a4849665/src/dotty/tools/dotc/core/Types.scala#L3019-L3025) ?


Why get rid of the first method? Or will dependencies always be filtered?


This does not really solve the multi-lookup problem


I not sure the patches will be the same. Two maintainers and thus probably diverging versions.


I thought they were neccesary, but they are not; will remove.


En théorie, c'est facile ... en pratique, j'ai fait pas mal de tests et je n'ai pas trouvé une solution qui marche avec les 2.
Il ne faut pas oublier que derrière, il y a la lib d'openzwave qui fixe aussi ses besoins.
Si ça t'interresse de creuser sur une solution qui marche sur python2 et python3 cas-y :)
Pour ma part, je n'ai pas tous les outils de python 3 sur mon poste pour tout tester ;)


yup thanks for that!


Right, this is not needed either. thanks!


See OD_UNUSED in odintrin.h


We should be carefully with this. If truncate failed, we may get a failed operation with a TEE_Success.


Modal


I was experimenting with using py.test (it's built in to travis) to see if it found all the same tests.
```bash
pip install pytest
py.test -x -v ---doctest-modules
```
This finds 2 more tests than `python setup.py test` does.
Without this change, the py.test version breaks; since were not using py.test here though, will remove.
(Sorry for the noise)


Don't know if the table can fit it, but could this be in its own column?


nvm, fixed it


I like this a lot, especially the comments
But should it not be:
overhead += 20;
for ether-phy, as we expect people to also add ether-fcs, and that might have been specified before ether-phy?


我在博客园的ID是 http://www.cnblogs.com/upwifi, QQ是55763，

在创业中，已经拿到投资，游戏产品已经完成，现在在处理游戏细节，等细节处理完成后做上线钱的准备。

可以交个朋友么？


Yes, good catch! Do you want to test the change?


Sure, I'll open a PR on top of this one.


"Path"?


So, we should use a `var_export`?


You just put your mouse over it.


Hi CoolEx

For this situation you need for normal work :

Intel E5 2620V3
Mainboard i recommend Supermicro x10 series
for Network adapter : Intel i350 Dual or Quad port if need more speed may
be need add 10G Intel 599 with Dual SFP+ port for 10G connection.
2x 500Gb WD RE4 Sata3

And this is a normal machine to work safely without any problem.

Best regards,
m.

On Fri, Oct 2, 2015 at 1:27 PM, CoolEx <notifications@github.com> wrote:

> Hi Micron10,
> could you recommend me some server for new gateway (something what you
> already got tested)?
> Situation:
>
>    - 1Gbps uplink to internet
>    - aprox. 1000 home users
>    - NAT mail: 120(and)centrum(dot)cz
>
> Thx
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gfreewind/linuximq-HighPerf/commit/2fb440f1b70c86841489663e9fbd1b00f488df73#commitcomment-13559436>
> .
>


@FabianHoltkoetter Da hast du unser tolles Logo einfach bei Imgur hochgeladen. Wir müssen es doch vorher noch markenrechtlich schützen lassen :smile:


why the fuck does this destroy `widget::main_menu::set_visible` ?
The next `widget::main_menu::set_visible` will fail when `m_settings.reset()` gets used..


This needs to be MACHINEID_SIZE + 1


Hi,
This code has problem. When the model is embed_in, it is impossible to issue the proper sequence.

Please see:
Your code I was allowed to reference. Very thank.
https://github.com/yamitake/mongoid-sequence/blob/master/lib/mongoid-sequence.rb#L26


Warning, if you pass "3" here, `machine_id` will not be zero-terminated. Given that the buffer comes from the stack, it might not be initialized to zero...

I would use either `strcpy` or use `strncpy(machine_id, "rkt", strlen("rkt") + 1)`.


Do not merge yet, we need to propagate python3 here on Fedora 23 and Fedora rawhide.


Right, thanks!


Understand what you're saying and if I follow that argument then 'ether-all' should be done in the same way (I may need to add a vlan tag/s)

Personal view:  I view these options as 'base' options to which other optionals are added.  Usage instructions should specify that 'base' options need to be applied first, with optionals later.

Incidentally, see the skbstats branches on both sch_cake & tc-adv ;-)


copypasta :D


`dialog::chat` and `dialog::settings` both are detachable windows.
At the moment code is duplicated..

`dialog::detachable_window` will include all the logic for attach/detach.
Independent of `dialog::main`


goooooood


You can also use `pos += snprintf()` if you want to avoid the `strncat`. Then, you can assert that `machine_id+pos` does not go over `size`.


Hmm actually I don't think that will work, we can replace `P` by `Q` in `P#T` only if `T` is a member of `Q`, so we have to set `variance = 0` to be conservative.


npm run eslint-fix


idem


디버그성 로그인듯 합니다. 나중에 빼주세요 ^^;


Indeed `var_export` seems better than `print_r` because it's valid PHP code :+1:


Haha, I noticed it. Good idea!


I'm not sure what's correct, but the z3/grok app that I'm looking at does not specify the `{prefix}` in the password either.

I believe the `IPasswordManager.encodePassword` always prepends the prefix.
Thus I think the value from ZCML stanza is encoded and stored with the prefix before being saved in the persistent principal registry (I've seen this via debug sessions ).


I guess you wanted to check `r == cs` instead of checking `r < 0`? If the file is truncated, you want to return an error instead of just taking the first few characters (and write uninitialized characters in `strncat` below).


`$expectedElement = (new DOMDocument)->importNode($expectedElement, true);` is allowed syntax too on 5.6. Would you accept a PR to change that, or do you think it's fine as is?


Temp solution, best would be to update event_params


See chriseldredge/git-slack-hook#20.


@odersky, I did not get the intention of this change. Could you please elaborate?


Its good to build the message within the isDebugEnabled itself, as it will be not used on other cases


But I would read the file in one step without the loop and without using a chunk_sizes array.

Then, `snprintf(machine_id, size, "rkt-%8s-%4s-%4s-%4s-%8s", buf, buf+8, buf+12, buf+16, buf+20)` (please check the numbers)


Thank you for quick reply.

Dne 2.10.2015 v 12:51 micron10 napsal(a):
> Hi CoolEx
>
> For this situation you need for normal work :
>
> Intel E5 2620V3
> Mainboard i recommend Supermicro x10 series
> for Network adapter : Intel i350 Dual or Quad port if need more speed may
> be need add 10G Intel 599 with Dual SFP+ port for 10G connection.
> 2x 500Gb WD RE4 Sata3
>
> And this is a normal machine to work safely without any problem.
>
>
> Best regards,
> m.
>
>
>
>
> On Fri, Oct 2, 2015 at 1:27 PM, CoolEx <notifications@github.com> wrote:
>
> > Hi Micron10,
> > could you recommend me some server for new gateway (something what you
> > already got tested)?
> > Situation:
> >
> > - 1Gbps uplink to internet
> > - aprox. 1000 home users
> > - NAT mail: 120(and)centrum(dot)cz
> >
> > Thx
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/gfreewind/linuximq-HighPerf/commit/2fb440f1b70c86841489663e9fbd1b00f488df73#commitcomment-13559436>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gfreewind/linuximq-HighPerf/commit/2fb440f1b70c86841489663e9fbd1b00f488df73#commitcomment-13559755>.
>


Fixed, thanks @mattheu


Why `O_CREAT` instead of `O_APPEND`? Do we want to remove the file that exists in the image?


Rename the file to `index.html` or the HTML code won't render.  You'll also have to remove the first line as that's [markdown](https://en.wikipedia.org/wiki/Markdown).


Uhm, achtung: genau dafuer wurde die continuous sqrt gebaut.


@suhothayan : this method returns the message. therefore we can't build the message within the isDebugEnabled block.


We only do this when the file doesn't exist


You need to give the file a `.html` extension or it won't register as an HTML file. This was also not the point of the assignment. The template was actually provided in the git repo, all you had to do was add HTML / modify the CSS.


I guess this will not work correctly if target creature has indestructible or regenerates.
The destroy will return false, and so the damage will be dealt also but it shouldn't.

So this code should work correctly:
```
permanent.destroy(source.getSourceId(), game, false);
return true;
```


Perhaps this is a usual python thing but this .0 and .1 just feels a bit ewww to me. also, thinking about it, party_list makes me expect that it will be a list and not a flag.


`<title>` needs to be in the `<head>` tag.

```html
<html>
  <head>
     <title>About Me!</title>
  </head>
  <body>
  </body>
</html>
```


This was driven by microbit (nRF51) development.


Redundant `</style>` tag and `</head>` tag underneath.

Actually looks like line 48, 49, 50, 60, 61 are extra. You need to change line 50 from `<body>` to `</body>`. Remember for every tag you open, there should be a closing tag.


Since this source file uses the `*at()` functions, we could use `faccessat()` here for consistency. Then, no need to do the `snprintf` just above.


Didn't know about that function :)


:+1:, but `.backdrop` isn't working. Can you see what's wrong?


This line of code won't do anything since there's nothing inside the `<center>` tag.


me neither. I just read the manpage of access :p

On 2 October 2015 at 13:17, Iago López Galeiras <notifications@github.com>
wrote:

> Didn't know about that function :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iaguis/rkt/commit/3c6cf6ab2ee3617fb0e5d8530d92aec4f1595e1c#commitcomment-13560087>
> .
>


renamed the makeSuper method to highlightGems as we have not yet touched upgrading a gem to a super gem.


The `</ol>` tag has no start tag.


There's no end `</ul>`.


Again, this feels like it's not very obvious what's going on.


The content between a `<ul>`, `</ul>` can only be `<li>` tags.


You need the end `</ol>`.


Totally irrelevant to this request but this block feels like it should be in a template.


여기 함수 정의한 부분에 주석좀 달아줘!


You are! This is great! Yes, google is your friend :)


Really? I thought it was driven by ESA...


The message vanishes after a reorg. That's imho a minor issue though.


The `<br>` has to go inside the `<li>`. You can't have anything other than `<li>` tags after the `<ul>`.

```html
<ul>
  <li>Any tag can go inside the li thought! <b>BOLD ME!</b><br/>Line break!</li>
  <li></li>
</ul>
```


Nope, not this time :)


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: tests


Thank you Freeztime ... I correct it