Yes, but if set is 0, one produced (0, 0, 0, 0) and the other (255, 255, 255, 0). Maybe you'd be able to see the difference if you did linear filtering of the font, perhaps? Like (0, 0, 0, 0) might produce a black halo. I'd have to test it out to know for sure.


@mostaphaRoudsari ,
Thank you for correcting my bad habits.  I will try to remember this more in the future.
-Chris


Sprzątać.


I recommend exporting setPrototype, so everyone can do whatever you said, without code modification.
We're going to need a better name for that function.


Please rewrap this verbatim part; otherwise it looks ugly in the manpage.


"Credentail" typo.


Can you also send a source with description of that vulnerability please?


I'm disagree with this change: multiple key selectors approach. It's not based on LINQ specs.


Hi Toke,

since I was there anyways, I just changed ipt() to honor SQM_DEBUG and log to $SQM_DEBUG_LOG instead of /dev/null. Seems to work okay, I had to teach ipt to also list the arguments otherwise its a complete unintelligible mess ;). Also /etc/init.d/sqm stop ; SQM_DEBUG=1 /etc/init.d/sqm start ; SQM_DEBUG=1 /etc/init.d/sqm stop works like a charm ;)

Best Regards
	Sebastian

On Sep 7, 2015, at 20:27 , Toke H�iland-J�rgensen <notifications@github.com> wrote:

> Looks good. Nice job! :)
> �
> Reply to this email directly or view it on GitHub.
>


Hey @ClementPadovani nice one. Can you also add a test for that please? Cheers!


stimmt, es ist nicht mehr verpflichtend. war es aber mal, und falsch ist es auch jetzt nicht. Besonders wenn das Subjekt sich ändert, ist es schön, ein Komma zu haben.


I think with 0 alpha that is unlikely, unless (255, 255, 255, 0) produced a white halo.


I'll have a proper look if there is some other way to remove the warning when Apache resarts.


I really appreciated that. We can also provide a pluggable solution, so everyone can configure default comparer, the way he prefers. For example using underscore equality as a default comparer.


Require fat static libs.


Hello, I think the "Add supplier to product if there is less than 10 suppliers" arbitrary logic could be asbtracted away in a specific method open for overriding. My 2 cts.


##add one comment


I was thinking the same thing


It has, however, the amount of times we had to edit this changeset (over 5) is unreasonable.


Looks like this `a` is never used? Left over from debugging, maybe?


Cool! :)


Ok, hier kann es auch wieder raus, jetzt, wo ich weiß, dass es optional ist. Und die Struktur des Satzes ist einfach genug, e sbraucht keine zusätzliche Strukturierung, um Klarheit zu schaffen. Also nehm ich es wieder raus, wenn du magst. Aber ich warte noch, falls noch 'was kommt.


@erickwilder this looks good to me but I'd love another approval please.


+1, this also lets subclasses add other things like the snapshot window etc


Nobody has access to the dictionary on Production, so there's no chance of conflict. Even if there were conflicts, a single conflict is possible to resolve manually. This is (conflicts) however an ongoing problem with dictionary for everyone


The paragraph has a typo, it should be "GeanyProxyFuncs::load() should check [...]". The difference is that ::load can only be called if the plugin was visible in the PM.

The paragprah is also wrong (the next one too) :\ not calling GEANY_PLUGIN_REGISTER() results in the PM *not* showing the plugin. It is equivalent to not calling it in geany_load_module() for native plugins. Actually ::load and geany_load_module() serve the same purpose.

I dont know what made me write this.


Разве это не значение по-умолчанию? Если да, то можно убрать.


r=jedmccaleb


It still could break code. So far we used 255/255/255/0 and 255/255/255/255 for mono fonts, with that change we use 0/0/0/0 and 255/255/255/255. If someone uses a blending mode where 128 is added to alpha (not sure we support that though...), the background would be transparent white before, and transparent black after the change.


:cop:


right, the paragraph tells bullshit


True. Do we support undefined behaviour, though?


Вроде не обязательно указывать. Разве название WAR-файла не берется из `finalName`?


This fixes issue #35


Hah Nice :+1:


It shoudl return a 'list' of (string,int) pair. The same way you created
the list in 'read_candidate(filename)' method.
And it works

On Mon, Sep 7, 2015 at 2:06 PM, shreeshbhat <notifications@github.com>
wrote:

> It should be a list of (string,int )
> On Sep 7, 2015 2:04 PM, "ramyarao242" <notifications@github.com> wrote:
>
> > Hi,
> >
> > Should the score_republicans() method return a list or a dict?if its a
> > dict it has key value as a republican name and value of it as the
> > democrat_count. Is this correct?
> > now when i am trying to return the dict with the above properties i get
> > "TypeError: string indices must be integers" , any suggestions on this
> one?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/iit-cs579/main/commit/6381616b84dcf0b6921711d9c3d52170dacae4ae#commitcomment-13100522
> >
> > .
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iit-cs579/main/commit/6381616b84dcf0b6921711d9c3d52170dacae4ae#commitcomment-13100547>
> .
>


Rename


@jnweiger And btw, I´m working with you, not against you. :)


Настройте IDEA, чтобы она не вставляла подобные бесполезные комментарии.
Дата не нужна -- есть в коммите и вообще мало полезна. Авторство, если хотите указывать, то указывать надо с помощью @author -- тогда она попадет в javadoc.


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/e5dbcfb5d38fee24afc2f355c34358699074d939


merging MonsieurNicolas/stellar-core/offerPrice = e5dbcfb5 into auto


MonsieurNicolas/stellar-core/offerPrice = e5dbcfb5 merged ok, testing candidate = b9bb2cb0


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 537](https://teamcity.labkey.org/viewLog.html?buildId=270634&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 30, passed: 9 Build time: 00:23:16
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGrid: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGroups: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridColumnSelector: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyItem: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridCheckerboarding: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudies: <no details avaliable>

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


With a shader that is certainly possible, but I don't think that would ever be desired and it's undefined behaviour. I can revert the patch though.


А сами файлы?


Should this zip file be here?


I'd propose to add a short `If you're a ROS user`-section here,
which tells the reader to add the git repositories below to his catkin workspace and build it.

At least this would ensure the reader knows about this option
before diving into "let's create a catkin workspace by hand
without telling the user it's a catkin workspace..."


typo ;)


Hey Spp, I've been wondering for some time now. What was the reason for changing uint32 to int32 for MoneyPerDay and SlotsPerDay data?


@danieldietrich it seems I should change extendable class to Traversable<T> because lot of clashes


Yes, it's undefined in the sense that we never documented it, so we could change it now. Still, it seems the old way is more consistent to what we do with grayscale fonts.


This change introduced a bug with 'summary' variable on line 42...


I suggest reverting it... summary *is* used fro linkedin link


The `ceil()` method was added to `nn.SpatialMaxPooling` in https://github.com/torch/nn/commit/929cfc57c88952b597bec77046582b90d1122380; make sure your version of `nn` includes this commit.


Is there some reason you want to change it, or just code cleanup?


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 1](http://ci.blueocean-net.de/viewLog.html?buildId=2&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


No reason, just code cleanup. To me a clear pixel should be (0, 0, 0, 0) but say the word and I'll revert it.


i prefer int when I'm sure INT_MAX won't be exceeded


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 1](http://ci.blueocean-net.de/viewLog.html?buildId=2&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:47


I somehow missed staging some stuff here. It's just supposed to be a rebuild call.


Hi @ejci

It's just that submitting an add-on to the Mozilla website (addons.mozilla.org) containing your lib causes their code validator to yell it, ie:

```
Warning: `setTimeout` called in potentially dangerous manner
In order to prevent vulnerabilities, the `setTimeout` and `setInterval` functions should be called only with function expressions as their first argument.
	Automated signing severity: high
	Suggestions for passing automated signing:
Please do not ever call `setTimeout` or `setInterval` with string arguments. If you are passing a function which is not being correctly detected as such, please consider passing a closure or arrow function, which in turn calls the original function.
	Tier:	3
	File: js/vendor/favico.js
	Line:	447
	Column:	18
	Context:
	> }
	> _drawTimeout = setTimeout(drawVideo, animation.duration, video);
	> link.setIcon(_canvas);
```

I think the main concern here is that the validator has no way of knowing if `drawVideo` is really a function or a string, being the later indeed forbidden for security reasons.


@lots0logs : THANKS! I really didn't know how to fill this one.


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


The tools for v0.12 uses 1.15 already, so it's safe to bump.
I suppose, that @diunko did that, because new Docker doesn't support old version (just a number) of API.


@ruslansennov ok, is there now s.th. todo or can I merge your PR?


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 2](http://ci.blueocean-net.de/viewLog.html?buildId=4&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 2](http://ci.blueocean-net.de/viewLog.html?buildId=4&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:18


I'm the fucking cuntlord bitch


This prevents a search for closed tasks by guests. (think of changelog..)

it gets joined to
```sql
WHERE ( is_closed = 1 ) AND t.mark_private = 0 AND p.others_view = 1 AND t.is_closed = 0
```


Your code had too many unnecessary memory allocations and memory accesses. Specifically, in your code:
* Lines 253, 275, and 290 add extra memory allocations when the `std::vector` object is used to initialize the the output `Permutation` object. The memory copy from the copy constructor called in the return statement should be taken care of by RVO. Your method would be OK if the vector had been moved into the result permutation objects, though my implementation avoids the overhead of the move.
* Line 271 has a double dereference, which creates causes extra memory loads due to aliasing.
* Line 251 and 252 access the memory of the result permutation twice.

`identity()`, for example, had 2 memory allocations and 4 memory accesses per element (not counting the return copy). My version has 1 memory allocation and 1 memory access per element (again, not counting the return copy).


why?  all integers in C have dubious behaviors when they reach their limits, but at least with an unsigned you know it just can't be negative (more than often, it allows to only check if it's `< max` instead of both `>= 0 && < max`).


fast-forwarding master to auto = b9bb2cb0


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


Не скупитесь на пробелы.


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 3](http://ci.blueocean-net.de/viewLog.html?buildId=6&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 3](http://ci.blueocean-net.de/viewLog.html?buildId=6&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:10


if the originally selected line moved due to new plugins arriving or disappearing the selected line remains at position N ( which is a different plugin now).


Thank you for that, it's good enough.

this is how choumiko does it:

![image](https://cloud.githubusercontent.com/assets/13031901/9722236/00104d40-55b2-11e5-9033-53e4ffc61cbb.png)

 he put a mention on the release notes - and in the forum post ( he has a very short readme.md file)


`JstlView` там по умолчанию, насколько помню. Можно убрать.


A bit shorter, don't you think? :-)


@karasu Actually, I didn't know either. POEdit made the correction automatically when I fixed the header :satisfied:


This is causing way too much controversy. I'll just revert it unless you've change your mind now? :P... no real good reason to have changed it other than repeated code always bothers me.


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 4](http://ci.blueocean-net.de/viewLog.html?buildId=8&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


Возвращайте интерфейс `ViewResolver`, а не его реализацию.


(libdir)\" \
 	-DTEST_DIR=\"$(abs_top_srcdir)/test\" \
 	-I $(top_srcdir


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 4](http://ci.blueocean-net.de/viewLog.html?buildId=8&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **SUCCESS**
Summary: Running Build time: 00:00:19


I assume you meant to type "excluding," not "exclusing" in the Lexicon entry.


Почему комментарий слипся с импортами и не относится к самому классу?


@Carbon-Ghost that new gui is larger. ^^


Зачем два параметра для одной версии?


The reason is that the filter func is now also called for the empty list (before being populated). This couldn't happen before because of missing hierarchy.

Remember the filter func is called for items on the same hierarchy level. We now have new hierarchy levels coming while the the PM is open (before the main list [hierachy level 0] was populated before the filter func was installed])


please explain this, need context


Вы в одном коммите смешиваете исправления для нескольких тикетов. Старайтесь, чтобы каждый ваш коммит был атомарным и содержал ровно те изменения для которых он подразумевался -- ни больше, ни меньше.


106 tests failed of 308 :) it's too late


`<version>` обычно пишут после `<artifactId>`


@silashansen - please open a separate issue with your question. It's very unlike we'll be able to track your feedback/request on a comment on a very old commit.


oh ok, hehe :)


I just tested the latest build and it works fine. May be we don't need getRepeatCount at all?


Желательно возвращать интерфейс, а не реализацию.


will fix


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 5](http://ci.blueocean-net.de/viewLog.html?buildId=10&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 5](http://ci.blueocean-net.de/viewLog.html?buildId=10&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **SUCCESS**
Summary: Running Build time: 00:00:17


Faster, faster.
That `MaxIterCharSequence` saved our butts.
nondeterministic finite automatons ftw.


Crazy diff!


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 6](http://ci.blueocean-net.de/viewLog.html?buildId=12&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 6](http://ci.blueocean-net.de/viewLog.html?buildId=12&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **SUCCESS**
Summary: Running Build time: 00:00:16


Йоба, если ты не против, я запилю свои исправления касательно проблемы. Я тут много чего пофиксил.


for some reason it had weird windows line breaks


Aah


What platforms don't support a long long literal?


copied from demoplugin.c, i think it's not true anymore.


Давай.


Please update your master for future tests.


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 7](http://ci.blueocean-net.de/viewLog.html?buildId=14&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 7](http://ci.blueocean-net.de/viewLog.html?buildId=14&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **SUCCESS**
Summary: Running Build time: 00:00:16


This was reverted.


This could be a level 2 heading (two `#`'s) to separate it from the API section. (Apologies for the low quality of my criticism :)


Don't explicitly specify the `className` propType at all - our propType validation explicitly skips `className` because otherwise we'd have to put it almost everywhere.


again, i prefer int when the value range is not a problem


Ok, fine with me :-)


Probably this or next week, given I don't receive critical bug reports


// Lame FREEBOX xmlns declaration !!!!
As attributes depends on what the server provides, i think it's up to the server to filter expressions using her own capability, as you do here. So i doubt this should depends on node namespace. (can't find anything like this in the spec, but i can be wrong)


On Sun, Sep 6, 2015 at 11:52 PM, Mohamad Balaa <notifications@github.com>
wrote:

> Why did we change the default ip here, @Juul <https://github.com/Juul> ?
>
Because a lot of people are using 192.168 on their LAN which makes it
annoying when your newly flashed router comes up on a conflicting IP. I
used 172.22 because it's not commonly used, we don't use it for the mesh,
and it's a reserved-for-LAN range that's easy to remember.

--
marc/juul


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 8](http://ci.blueocean-net.de/viewLog.html?buildId=16&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


expect(page).to have_content(@user.name)


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 8](http://ci.blueocean-net.de/viewLog.html?buildId=16&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **SUCCESS**
Summary: Running Build time: 00:00:19


i dont think this should have been committed


same here with spacing dont need it


The approach works for me @OrKoN do you see any issues?
The only issue I see now is that we cannot have something like optimized mode.


index.html


Would `.strip()` be better/cleaner than `.replace()`?


whoops, second statement in the commit message is not true, this still stops the bug I was squashing, it's just not the user-visible bug


Okay, give me a heads up when it's good to merge :+1:


Sheepdog driver supports these operations


On Sun, 06 Sep 2015 15:02:42 -0700
Julian Ospald <notifications@github.com> wrote:

> hey my friend, we could also add
> ```xml
>     <natural-name>pyDeComp</natural-name>
> ```
> to `metadata.xml` of `dev-python/pydecomp`
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gentoo/gentoo/commit/cb56c159eb4c1d5368495cdf74b7e6362a6ecb82#commitcomment-13088088

OK, thanks, I didn't know about that one

--
Brian Dolbec <dolsen>


93651760-RowPerObjectReport (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1831 of 1831 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)


恩是的，代码提交时疏忽了。新版本会删除这些无效代码


@Orvid  - Can you please rebase this pull request? Thanks.


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - x86_64-MacOS-10.9-clang: OK (1824 of 1825 tests passed)

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


EMCのページにある, Get volume statisticsは出来ます。


Hi viren,

It was meant for Mike & Dave. I'm probably going to move this repository over to the one that now holds the rest of the annotator, so confusion like this doesn't continue.

By the way, thank you for all of your hard work on the server set up.

I hope you have a great holiday!


snapshot listの取得用のオペレーションは無かった気がしますがいかがでしょう.


はじめまして、東急コンピュータシステムの湯浅と申します
貴殿が、お作りになられたEVMのプラグイン
https://github.com/momibun926/redmine_evm

を使用させていただこうと、手順どおりやってみたのですが
AbstractController::Helpers::MissingHelperError (Missing helper file helpers/baselines_helper.rb):
のようなメッセージが出て先に進めません
ぜひ使用させていただきたいのでアドバイスいただけると助かります

不躾ながら、他に頼る知識がございません、よろしくお願いいたします


@azturner Any thoughts how to support this with Excavator?  I'm comparing string values now but performance-wise using the Id or Guid would be faster.  I'd have to pick one though, I can't dynamically switch between the field needed for lookup.


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1830 of 1831 tests passed)

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


My inclination is to say no - that we should stop them before they fail.

`X with Y` and `X after Y`, both require `Y` to be active - `on-fail=block` is saying "leave it failed" which would logically imply that the constraints cannot be satisfied and `X` should be stopped.

Have you got a particular use case in mind?


```
RepoMan sez: "You're rather crazy... doing the entire repository."
```
I prefer the word lunatic.


Even in condor(msm8610) Bluetooth share fcs


Cool, I will merge it in.  Thanks!


Scanner should be in main class to take input.User should not be taken in generic classed.


thanks a lot @istr , yes, I can solve this problem through fix my environment, add a luarocks share path to my package.path, rather than this change to gin/cli/console.lua.


Looks like you have a typo here, `applicatio`


注意，返回的序列从 开始位置 开始 ，刚好在 结束位置 之前结束。即开始位置是包含在序列切片中的，而结束位置被排斥在切片外。


This allows staff room auth to read modnotes :|


Move this method to main class to take user inputs.


newNode is not used in this method.remove this.


This is a comment made in the commit.


修正します。


I talked to TI about it; it's intentional and it makes sense. We trust kota with modnotes.


以python 3重写


リスト系操作はデータベース上の参照処理なので、ドライバ側で特に機能を用意しなくても、サポートされます。（あえて削ってもよいですが、他のドライバに倣っておくため、現状のままとします）
なお、REST-APIおよび、cinderコマンドで実際に動作することの確認はできています。


We shouldn't be thinking of this as a "kota" rank. It's supposed to be "staff room access".


Looks good.


I didn't try.
Works on my local. Might be useful if we need to switch to production DB.
Do you remember if NCBI has postgres as production, except MSSQL?


This code logic to take input and check the input,should be written at a single place where input is going to be taken.No need to create separate methods to take input for left and right node.call addNode method.


Same as above for addRightChild.


指摘ありがとうございます。追記しておきます。


Les recomiendo usar mejor let*, el código queda mucho mejor


@opdavies good point. Created #93


שיניתי מלא דברים


Меньше текста набирать.


hi, i will be joining in 10 min

On Sun, Sep 6, 2015 at 10:23 PM, Phani Kumar <phaniwithu@gmail.com> wrote:

> Hi,
>
> I made a commit to add phones for emp, please check at below
>
> https://github.com/phaniwithu/Manish-Angular/commit/f018c6584999839e36a65867b1ea714136f9f352
>
> we can discuss more tomorrow
>
> thanks
> Phani
>
> On Sun, Sep 6, 2015 at 6:57 PM, manishagarwal7 <notifications@github.com>
> wrote:
>
>> phoneArr is getting pushed with count. Not actual numbers.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/phaniwithu/Manish-Angular/commit/4c051c451429eab0b0c9c71c11317ecb059788f5#commitcomment-13086252>
>> .
>>
>
>


Okay.   As discussed in PGXL community, `#ifndef` directive could be confusing.   I'd like to use `#ifdef` instead.


注意：列表的赋值语句不创建拷贝。必须使用切片操作符来建立序列的拷贝。


#4854-ZoneHVACOutdoorAirUnit-does-not-autosize-child-objects-correctly (rraustad) - Win64-Windows-7-VisualStudio-12: OK (1791 of 1823 tests passed)

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


Pragma there's a thing to update in doc/python kvs whatever in kvirc-www


Closes #4


Hello Howard,   sorry for not having mentioned that, yes, you are correct. The library must be specified in the DESCRIPTION  file.   There is another important point, source () should not be used either in packages and this us the reason I have had to move everything to inst. In any case, using source with local as you do seems to work fine as we are not polluting the environment with our definitions.   To avoid source, we may need to move all the files to the R folder without subfolders, but honestly I prefer your style of keeping the files tidy in different subfolders.   Let's see if there is feedback regarding this when we submit to CRAN and the reviewer sees the package.
     Best regards
       Jose

Sent from my Samsung Galaxy smartphone.

-------- Original message --------

From: Howard <notifications@github.com>

Date: 7/09/2015  11:56 am  (GMT+10:00)

To: VisualExploreR/shinyDE <shinyDE@noreply.github.com>

Subject: Re: [shinyDE] remove library imports (b72b3de)

Ah. I figured out. Never import libraries using library() function when writing packages. Instead specify needed libraries in the DESCRIPTION file.

Instead of using runApp() function, I used visualToolkit() and it worked. Thanks!

---
Reply to this email directly or view it on GitHub:
https://github.com/VisualExploreR/shinyDE/commit/b72b3de1d8bb590dd416676d68d47548d2bf417f#commitcomment-13088883


Why do we use scope.$eval instead of JSON.parse?


Here, IDF is implemented densely, i think it can be sparse way is better.


No need to create separate classes and main method for each functionality.Create methods and use them in single main method.


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1830 of 1831 tests passed)

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


#4854-ZoneHVACOutdoorAirUnit-does-not-autosize-child-objects-correctly (rraustad) - i386-Windows-7-VisualStudio-12: OK (1791 of 1823 tests passed)

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


Much better to read things from resources... I love it.


Set Name


LGTM


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


If/when you publish this, it will be better to pin to a released version than this branch.


This is a test!


I would like to see logic which prevents an interval of less than 15s also.


If you're already using Pathname, why not use `Pathname#dirname` and `Pathname#basedir`? Also `Dir.pwd` already resolves symlinks, so resolving it to the realpath is a noop.

Short version:
```ruby
def maybe_make_path_relative_to_app_root(app_root, path)
  realpath = Pathname(path).realpath
  if Pathname.pwd == realpath.dirname
    realpath.basename.to_s
  else
    realpath.to_s
  end
end
```
I don't see why we need app_root at all, because passenger already changes into the app root in the spawner.

Even the code above seems overkill, because as long as we use the fully resolved path were fine:
```ruby
require "pathname"
# replaces maybe_make_path_relative_to_app_root
def realpath(path)
  Pathname(path).realpath.to_s
end
```
The above solution works fine for me.


Hello!

This

Is

Such

A

Great

Job

You

Good

Guy!


Ohh muchas gracias, justo al momento de hacerlo pensaba
como hacer algo así.
Saludos.

El 6 de septiembre de 2015, 21:30, Hector E. Gomez Morales <
notifications@github.com> escribió:

> Les recomiendo usar mejor let*, el código queda mucho mejor
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JuanHerreraBrito/lenguajes20161_JuanHerrera/commit/5cda467bec9072942c1e915f39baca5fc87b3b53#commitcomment-13089091>
> .
>


Using the timestamps in the logs we can replay the inputs. A tool to do this does not exist yet.

http://192.168.10.1/logs/stratux-uat.log

http://192.168.10.1/logs/stratux-es.log


`length` is simpler, isn't it?


読みやすく素晴らしいです。良い仕事ですね。
Controllerのクラス名、~Controllerじゃなくて~Applicationなのが気になりましたが、Playの標準なんですかね～


FavoriteAdapter.java
Variable: mList should => mListWords


Weird, I'll apply this patch and see if it still works on falcon.  Does this solve the issues on condor and titan?


#4854-ZoneHVACOutdoorAirUnit-does-not-autosize-child-objects-correctly (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1295 of 1295 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.06%25-green.svg)


Are you using a release version of Racket? It turns out the location of some of the functions in unstable got moved for the next version. So it probably makes sense to keep this `racket/function` and to keep a commit on the 6.2 branch that makes this change.


Darn you, I was in the middle of porting the commit as well! Without running it and just giving it a look over, seems good to me! +1


s/plugin use/plugin uses/


s/repository the plugin/repository to which the plugin/


Consider adding some examples of values for this repository. It's not clear what format someone should be using to specify the repository.


LoginActivity.java
Line 22: variable name not correct with convention
SplashScreenActivity.java
Line 24: variable name not correct with convention

activity_testing.xml
Use dimen


scope: :user (untested)


nevermind my last comment then :+1:


I have filled it as #5 so you can easily find the pointers above.


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 9](http://ci.blueocean-net.de/viewLog.html?buildId=18&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 9](http://ci.blueocean-net.de/viewLog.html?buildId=18&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **SUCCESS**
Summary: Running Build time: 00:00:17


IMO it's better when route url is just expressing the resource not the operation. HTTP Verb (POST, GET, etc) should express what is being done with the resource.


plugins.c isn't completely consistent about it and HACKING has no mention about it. I'll change it, but I wouldn't say I was wrong here.


http://www.sarahhagstrom.com/2013/09/the-missing-django-allauth-tutorial/
https://www.youtube.com/watch?v=fYz3WJLWFVI
https://www.youtube.com/watch?v=SWaNEcckjqI


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 10](http://ci.blueocean-net.de/viewLog.html?buildId=20&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 10](http://ci.blueocean-net.de/viewLog.html?buildId=20&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **SUCCESS**
Summary: Running Build time: 00:00:16


Why? Useless checks that wrongly suggest pa or pb can become NULL?


Is there a reason why this function does not have the guard statement:

```c
if (!self || !self->handle) {
    return 0;
}
```

(Same for `php_smb_ops_seek`)


Thanks! It's one of those things where I'd explained it to people in person so often that I could swear it was written down.


Handling errors in nodejs can be pretty annoying. When sending a response to user it's also difficult to reason about the code with a lot of if else statements. Try something like this:
```javascript
if (err) {
  return res.send(...);
}
return res.redirect('expenses');
```

By returning from function you make sure that the response is sent only once and that every path actually returns reponse to user.


Ooops - you're right. I'm sorry. How do you want to proceed?


The compare func here is called per-hierarchy levels, which is handled by GtkTreeModel


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 11](http://ci.blueocean-net.de/viewLog.html?buildId=22&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 11](http://ci.blueocean-net.de/viewLog.html?buildId=22&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **SUCCESS**
Summary: Running Build time: 00:00:24


Ok then


About the return values: are `0` and `-1` the canonical return values? In for instance `php_stream_smb_mkdir` and `php_stream_smb_rmdir` they are `0` and `1`.


@piotrpalek I would call the module just 'selectable'


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 12](http://ci.blueocean-net.de/viewLog.html?buildId=24&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


okay


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 12](http://ci.blueocean-net.de/viewLog.html?buildId=24&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **SUCCESS**
Summary: Running Build time: 00:00:17


In future it might be better to serve static content under separate prefix:
```javascript
app.use('/public', express.static(...));
```
It's easier to set-up production environment for that (for instance with nginx) and it doesnt get mixed up with your custom routes.


Thank you Yuri.  That was quick.  Even if it is work in progress, I pulled it into master so that we can check that nyquist-workbench translation is doing what we expect.  Will update again later for RC1 if there are further changes.


I guess you can just update your branch, this pr should be updated automatically.


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 13](http://ci.blueocean-net.de/viewLog.html?buildId=26&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 13](http://ci.blueocean-net.de/viewLog.html?buildId=26&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **SUCCESS**
Summary: Running Build time: 00:00:15


no, it shouldn't... travis tests never works... travis has bad network...

This has to do with timeout of connections that are kept alive... otherwise the process won't shutdown when it calls it self...


What is this last line doing?


@bf4 Hey, I was looking for forks of my repo and by accident I found yours fork. I was curious about your changes and that's how I found this commit. :)

I like the idea with 0/1. You can update spec file spec/knapsack/config/tracker_spec.rb to cover all cases for `generate_report` method and then please do pull request to my repo so I can merge it. Thanks for help!


Do we have cxx and c overrides defined in OE-lite/core?


Ah ok, we should maybe be ignoring events which aren't of type `EventKindUp` or `EventKindDown`


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 14](http://ci.blueocean-net.de/viewLog.html?buildId=28&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


This line is why I reopened #49 (wrongly because it's not really this issue but a comment referencing this issue). The purpose of this code block is to **strip the last section of the file path**, this check breaks the logic and it's the problem.
Moreover your don't fix the *empty filename* issue here, see existing check line 96: `&& p[1]`.

@Liryna Could you review and remove this line if you agree? Thank you.


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 14](http://ci.blueocean-net.de/viewLog.html?buildId=28&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **SUCCESS**
Summary: Running Build time: 00:00:17


I was. Sometimes we get an EventKindUp that is missing that data.


weird spacing


don't need the `{` here just put text in


Will check this, thanks. I'll have a look at gtk_tree_model_filter_convert_path_to_child_path().


Hey
Thanks for the catch :)
It's already fixed here: https://github.com/idris-hackers/atom-language-idris/commit/e87a3075ee87c98c1ea4fb10671903c4cf925174


I didn't bother yet because I wasnt sure what's the right way.


I am not surprised by the result, it might make sense to use bare copy if this is a performance bottleneck. However, my guess is that you will not even notice the impact. It might be twice slower to use optics but it is still super fast, if you change the annotation of your benchmark to `@OutputTimeUnit(TimeUnit.MICROSECONDS)` you can see how long does it take to `modify` a case class.


Perfect. :+1:


Beautiful. :+1:


I wanted the logging in the methods, and didn't fancy doing `tickChannel(ch, duration, "sync jobs")`


The indent level is getting high. Maybe set `ret` to `-1` at the start of the function and only set it to `0` if the operation succeeds? Then you can do:

```c
int ret = -1;

...

if ((smbc_open = smbc_getFunctionOpen(state->ctx)) == NULL) {
    break;
}
if ((smbc_close = smbc_getFunctionClose(state->ctx)) == NULL) {
    break;
}
if ((smbc_utimes = smbc_getFunctionUtimes(state->ctx)) == NULL) {
    break;
}
if ((handle = smbc_open(state->ctx, url, O_EXCL|O_CREAT, 0666)) != NULL) {
    smbc_close(state->ctx, handle);
    ret = 0;
}
if (newtime) {
    ...
    ret = smbc_utimes(state->ctx, url, times);
}
break;
```


I agree.


What compiler checks are you refering to?

Your suggestion has a flaw: "\n<i>Other plugins depend on this. Disable them first to allow deactivation.</i>\n" would be displayed without markup (instead <i>, </i> would be printed verbatim. That's the reason for complicating like I've done.


Here you should do `where answer.status != active`


Understood. Carry on :+1:


I wanted the logging in the methods, and also when looking through the code I preferred the explicit methods, marginally easier to grep for :wink:. I'm not concerned with the duplication, given the simplicity and the low chance of change.


Sorry for replying this late...
@marcacohen I think renaming the manifest.json file to something different (ex. webAppManifest.json or web-app-manifest.json) inside the PSK will remove a step in this recipe and could be considered reasonable, but I it's more of a CDE issue to assume that the manifes.json file is for a ChromeApps ...
maybe CDE could/should use a dot file to store a project's properties (ex. the prj type, maybe the git credential, etc...) but I'm going OT.

The main "issue" I see with changing the name is that if you look up Web App install or Web app manifest articles and docs all are using manifest.json and that might confuse some green developers...


Use sec:CryptographicKeyCredential. Everything else uses urn:bedrock:test: but this one was previously using sec: so still is until we figure out a proper long term naming scheme.


1. Открываем консоль внизу вьюпорта браузера
2. Открываем "Настройки".
3. Ресайзим область консоли вверх, пока у окна "Настроек" не появится скроллер.
4. Видим, что окно гибкое и ресайзится на толщину скроллера!
5. То же проделываем с "Избранным". Оно тоже ресайзится, причем длинные названий тредов сокращаются и появляются троеточия.
6. Со "Скрытым" не придумал, сейчас посты тупо обрезаются, и видны половинки кнопок ответа (все раво они там не рабочие).
7. Проверял в Лисе, хроме и некроопере. В последней заметил особенность - при ресайзинге инспектора, страница не перерисовывается, и скроллбар у окошек не появляется.

TODO:
1. Исправить вкладку Инфо, сделав ее на флексбоксе.
2. Сделать редактор спеллов на флексбоксе.
2. Сделать троеточия для длинных названий в "Видео".


It's my personal preference, and habbit. also makes introducing -1 as a special value later on easier.


Documentation/2- Card Library - Effects.md

what is meant by a behavior-specific resource?


You left a lot of debug staff.


Here also.


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


This CSS needs to be in the body of the embedded document, not on the iframe, to get rid of the vertical scroll bar.


maybe add a note that mods are free to change resources as they see fit. Not all resources are guaranteed to exist in all mods.


and here


One more


entire file


Пришлось посокращать некоторые названия опций. Возможно вернуть им длинные имена, сделав где надо `text-overflow: ellipsis`, как с избранным. Сейчас желательно, чтобы текст не доходил до правого края Настроек.


Agree


should this be 0.0.2?


lol, he left that there.


```newobject = createObject(..., 'group', groupId) specifies the group  context in which the```
missing text


The only line that actually changed is the 331 in the .m file. Yes, it is related to the #221.


nice :)


@wesner0019 sorry I must have missed that mention of the electron. If you are rolling your own electron design then you'll need JTAG to add the bootloader, and/or a combined bootloader/system binary and your user app. I'm not aware of any plans to offer preprogrammed STM32's but even if we did not offer that I'm sure this is something that could be done by ST or a third party. That said I think a testing/programming rig with JTAG would be just as easy as one with serial so probably not worth the cost unless there is something I'm not understanding about your application. Options are good though ;-) if you need more detailed info on this please contact support at hello@particle.io. I want to make sure I'm not blocking you by being slow to check my github emails. I get a lot of them!


I will test it from Git tomorrow the entire day from the Lab, I'll be solving support tonight to have enough time tomorrow...


ah ok, ja machi

han leider nöd so vill chöne mache, han denn ja ds back-end nu müese apasse und denn bini im back-end hange blibe willi nu es paar sache gseh han

morn isch au wider en tag;)


Fixed


```'group ```` why removing ```'```


want to submit a pull request?


This is just a demo project. It is not required for the demo to work. In a real life case you might want to list all projects linked to a certain tag, so in that case: yes ;)


Yes, quite a useful commit message. I see the light.


Parte inutil!


Thanks! BTW, does RVO apply in nested situations? Move might be safer then...


I will implement some commenting for this tomorrow.


ok, want to replace the TODO with a comment to that effect?


It seems you are implying everyone should use "Add Lowell first meeting" as their commit messages here


No alt? Shame on you!


Might I suggest a black background?


TeamCity pull requests :: netty [Build 184](https://garage.netty.io/teamcity/viewLog.html?buildId=1201&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5194, ignored: 21 Build time: 00:15:22


pull and request e617433


Would it be worth a note about adding a futher description?


no, `url.js` is merged. Please delete the file if using URI module from npm works.


This comment is a bit confusing/typoed.

I experimented a bit – is it by design that ~E allows interpolation but that it's not actually performed at that time?

```
iex(12)> ~e"hello #{1+2} world" |> Phoenix.HTML.safe_to_string
** (ArgumentError) interpolation not allowed in ~e sigil. Remove the interpolation or use ~E instead
…
iex(12)> ~E"hello #{1+2} world" |> Phoenix.HTML.safe_to_string
"hello \#{1+2} world"
```

By not actually performed, I mean that I expected to see "hello 3 world".


pull request


pull and request e6174335


The ```sk``` argument is used for BUG_ON statement only, so I think compiler complains to unused argument if you compile Tempesta w/o DEBUG option.


> It might be twice slower to use optics but it is still super fast

Sadly, Gatling is not your regular web application where great functional code prevails over raw performance because you can just throw more hardware in your cluster. People expect to get the best possible throughput from their hardware, and compare with other tools.
When you want 30.000 rps from a single machine, you have to hunt down every allocation and excessive processing without hurting design too much... :(


@jameskoster Digged in https://github.com/woothemes/storefront/pull/227


Interesting...


Ok.


Actually hidden on small layouts too - would need to set position on this bit to display it again:

    @media screen and (max-width: 765px) {
    .main h1 {


Verdade. Vou adicionar :)


I agree with you, you definitely have stronger performance requirement for a library like gatling.


`float: none !important;` отсюда можно убирать, в флекс-контейнерах `float`/`clear` и так не работают.


**require-jsdoc/require-jsdoc**: Missing @param description in the jsdoc for componentWillReceiveProps


**valid-jsdoc**: Missing JSDoc @returns for function.


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


**valid-jsdoc**: Missing JSDoc parameter description for 'event'.


**valid-jsdoc**: JSDoc syntax error.


**valid-jsdoc**: Missing JSDoc for parameter 'newProps'.


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 15](http://ci.blueocean-net.de/viewLog.html?buildId=30&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 15](http://ci.blueocean-net.de/viewLog.html?buildId=30&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **SUCCESS**
Summary: Running Build time: 00:00:14


>Сделать троеточия для длинных названий в "Видео"

Вот без этого лучше бы постараться обойтись - лучше переносы, чем обрезанные названия.


А, точно. Я забыл про него.
Нужно избавиться от всех `float`'ов в скрипте. `Float`'ы это настоящее css зло.


Use `ng-submit="todoCtrl.todoAdd()"`


Should be `ng-model="todoCtrl.todoInput"`


От переносов ресайзится окно по вертикали. А оно не должно "скакать" при переключении вкладок. Его высота должна быть фиксированной.
Так что, либо сокращать смысловую нагрузку текста, либо вариант с обрезанием. Никаких переносов.


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 16](http://ci.blueocean-net.de/viewLog.html?buildId=32&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 16](http://ci.blueocean-net.de/viewLog.html?buildId=32&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **FAILURE**
Summary: Tests failed: 3 (3 new), passed: 62 Build time: 00:00:20
### Failed tests
```
tests.test_genrequest.TestGenrequest.test_genrequest_batch_invalid_xml (Create a batchrequest only using the Communication-object,): <no details avaliable>

tests.test_genrequest.TestGenrequest.test_genrequest_batch_xml (Create a batch-request only using the Communication-object (): <no details avaliable>

tests.test_genrequest.TestGenrequest.test_genrequest_check_response_batch_xml (Create a batch-request only using the Communication-object): <no details avaliable>

```


Might I suggest giving the assets a name conforming to one single naming convention? The use of underscores, whitespaces and capital letters for example is inconsistent.


This is to prevent possible issues / cause less work for community translation projects where they might change the aegis items in item_db.


Hi, my goal is to adapt this sketch for an arduino mega + 16 relay card and two opto iinput card. The output pin goes from 22 to 37 and the input pin to 38 to 53. how  i can define this? What others change will be necessary to run on my hw? thanks and greetings for our huge job guys


Is there no option in openwrt to configure the name prefix?


9 changed files


3 musics ? are you sure ? i mean 1 to 2, personnaly, with of course the music choosen by Obani :)


Это да, лучше бы сразу флексы сделали вместо этого дерьма.


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 17](http://ci.blueocean-net.de/viewLog.html?buildId=34&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) is now running


TeamCity Zimbra-Community :: python-zimbra :: Test Py2.7 [Build 17](http://ci.blueocean-net.de/viewLog.html?buildId=34&buildTypeId=ZimbraCommunity_PythonZimbra_TestPy27) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 62 Build time: 00:00:18
### Failed tests
```
tests.test_genrequest.TestGenrequest.test_genrequest_batch_invalid_xml (Create a batchrequest only using the Communication-object,): <no details avaliable>

tests.test_genrequest.TestGenrequest.test_genrequest_batch_xml (Create a batch-request only using the Communication-object (): <no details avaliable>

tests.test_genrequest.TestGenrequest.test_genrequest_check_response_batch_xml (Create a batch-request only using the Communication-object): <no details avaliable>

```


Beep, beep, beep


tested this by manually comparing before/after doc counts and it looked good. @dannyroberts let me know if  there was a better reason you didn't change it at the time you wrote the comment


Done. Перенес генерацию панели с содержанием в скрипт. Планирую перенести и генерацию лэйаута всей странички туда.


Исправил на второй вариант, т.к. он ближе к эклипсу.


ACK


Э, какие вкладки в Видео? В других местах согласен, можно и обрезать. Но для окна видео тогда где-то обязательно нужно показывать полное название, хоть в тултипе первой строкой, а теперешние данные - во второй.


Why chop this paragraph? Is it just stating the obvious?


Player now basically "teleports" to a fixed location, then falls down.


> в "Видео"
А, читаю одним местом. Ладно, там пусть переносится. Я думал вы про Настройки.


we could use --use-leap-wheels in here


I'm uncertain about this API change.

Argument for adding `dryrun`: it's easier for the programmer to see what's happening.
Argument for simply relying on `S==NULL` to indicate a dryrun: it saves memory?  it's slightly "neater" in terms of avoiding unnecessary variables?


Can you update the copyright to 2015


Provisional ACK, if my question above is answered.


I'm so glad that Google stopped that preview nonsense with Chrome.


Having -Ofast and -ffast-math is really redundant. -Ofast is essentially -O3 and -ffast-math combined :)


>Пришлось посокращать некоторые названия опций.

Кстати, многие текстбоксы в настройках тоже можно ещё уменьшить по ширине, они много где раза в два шире чем занимают места значения.


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


**valid-jsdoc**: Missing JSDoc @returns for function.


**quote-props**: Unnecessarily quoted property `medium` found.


**yoda**: Expected literal to be on the left side of >.


**no-var**: Unexpected var, use let or const instead.


**quote-props**: Unnecessarily quoted property `small` found.


**quote-props**: Unnecessarily quoted property `sizeBorder` found.


**no-warning-comments**: Unexpected todo comment.


**yoda**: Expected literal to be on the left side of !==.


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


**react/jsx-sort-props**: Props should be sorted alphabetically


Was fixed in second commit.


could it be default? like ...

```lua
     -- gin init
     local gin_init = [[
lua_code_cache ]] .. convert_boolean_to_onoff(Gin.settings.code_cache) .. [[;
lua_package_path "]] .. (Gin.settings.package_path and Gin.settings.package_path or './?.lua;$prefix/lib/?.lua') .. [[;;"; # NOTE: ;; is default package path
 ]]

     -- add db upstreams
```


ACK


yes. Both from context (you can typically see the list of methods directly after this doc), and also because it's kind of implied anyway with vector being array-like.


develop (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1307 of 1307 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)


Photovoltaics (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

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


develop (mjwitte) - x86_64-MacOS-10.9-clang: OK (1304 of 1304 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)


Photovoltaics (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1307 of 1307 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)


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

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


Can you remove the back-ticks from the headings here? We try to avoid these inside headings.

(Same for the other options below)


ACK. I wonder how some of this stuff slipped through...


@m-kuhn nice! One thing I'm not sure about is whether this is a good idea to have this behaviour by default. I think it should be a non-default option for the widgets, so that users have to opt-in to storing relative paths.

My rationale is that otherwise this is making data dependant on a project's location. I think in general it is bad practice to make data dependant on a project, as it breaks portability of the data.


Notes on the branch, and when to merge back to master:

When I finish the work on fading. At this point it's just modifying the previous fade system to use the new idiom, and cleaning up old references. Once that's finished I'll submit a PR and branch off of this for (again) playing with physics movement. With the new changes it might be possible.


there is a service for this now. please use `$this->container->get('zikula.doctrine.schema_tool')`


ACK... surprised that this is all that needed editing, though.


That's fine.


New version.


(added it back in given the issues on staging)


0 is allowed for the reference counter, is it right?


This isn't going to work.  need to be able to update the category as well.


The "mutable" classes are "experimental" code which has been around for a long time and has never graduated to non-experimental. No offense to the author, but I consider them pretty useless and would like to see them removed.


But ACK.


![jenning consulting logo](https://cloud.githubusercontent.com/assets/9747050/9722715/4e9111d2-55aa-11e5-827f-b978ce7541ba.jpg)


Metoda isAmountPositive() vyhodi exception, kdyz amount neni kladny, tak jsem myslel, ze dalsi vyjimka je  zbytecna.
Nerozumim casti napr. -1 ? Rizene selhani mam otestovat metodou Assert::exception ?


if you remove something with rm -(r)f you dont have to check if the file is present, simply remove with -f


Does this really qualify as private? I wouldn't consider it bad for client code to access it directly.


Thanks, this is very useful :+1:.  I did not know about the `family` keyword.

Please add the removed lines `##' @rdname cred_env-methods` and `##' @rdname cred_user_pass-methods`
I have tested and we need the lines to pass `R CMD check` without warnings


Well and good, but it is likely that `Hamster.hash`, `Hamster.vector`, and so on will be removed. @krainboltgreene raised the point earlier that there are too many ways to initialize collections in this library. Do you agree?


As above.


Why get rid of this one line and replace it with 2 different ones?


Good catch.


La función isEmptyObject detecta si en el objeto Consulta la respuesta es vacía:
-->si es vacía es porque nadie la respondió aún. Así que envía la respuesta al alumno que preguntó.
-->si no es vacía es porque ya la respondieron, sólo emite un log en la consola.
---------------------------------------------------------------------------------------------
Noté que hay un problema cuando se tiene mas de un alumno... los números de preguntas que hacen los distintos alumnos empiezan a coincidir


Trying to reduce the number of "notes" on this page; perhaps;

```Markdown
The `awslogs` logs driver requires you to set a region. Either the `aws-region` log option
or the `AWS_REGION` environment variable must be specified.
```


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

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


as this is a ring buffer, it couldn't have worked this way...
It should be implemented like in e.g. the `uart_check_free_space` function: http://www.promptfile.com/l/2301D0A2BA-3B1379FE8C


@vanthome @piotrpalek I don't think that the solution is complete w/o the 2nd thing Peter mentioned is solved.  I have several concerns: 1) meta object needs to be replaced each time something is changed. This does not look like a nice pattern because new object needs to be generated each time 2) I am not sure whether the proposed solution of running `notifyPropertyChange` would work. I assume that although it will mark the metaObject as dirty, the new data will not get through still because the old meta object === new meta object. As far as I know `notifyPropertyChange` just fires observers on the current object and don't cause re-computation of bindings downstream.

I think it may be a good idea to have a typed meta object that exposes methods like selectItem, deselectItem etc and ensures that this causes updates in the template. The advantage is that one does not have to decide whether he must replace or not replace the meta object.


Photovoltaics (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


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


@yrashk
Thanks for letting me know it.

I released it as 0.1.2.

https://github.com/benelog/multiline/tree/master/maven-repository/org/adrianwalker/multiline-string/0.1.2


I think it's better to store a `raptor_stop_event` in the visitor no ?


NAK.

README.md is there specifically for people to see on the main GitHub page. It's not YARD documentation. YARD-README.md is processed by YARD and appears among the YARDocs.

Go to this branch in GitHub and look at what the homepage looks like.


nevermind...


Thank you!


It's basically the same as [`.empty`](https://github.com/xaviershay/hamster/blob/3d9c035c89302cf7744a71e90c214e07e370ff54/lib/hamster/hash.rb#L92). Per above comment about too many ways to initialise - I don't see a benefit to having both the class method and an constant to construct an empty hash, and I prefer public methods to constants because the former are easier to deprecate/change. Might be clearer to implement them thusly though:

``` ruby
# private
EmptyHash = Hash.new

# public
def Hash.empty; EmptyHash end
```


That's awesome! :smile: You are welcome!


Yeah I don't feel like they are all needed.


Fair enough. ACK to this commit, then.


Can you use our standard "Note" formatting for this longer note? The format is described in our docs style guide; https://docs.docker.com/project/doc-style/#notes


ah derp, yeah you're right.


Sir, in Utils, I can't find VerifyUtils method. so, there is error in BasePage.


+1 Jesse


@aharley40 the child class doesn't need everything following extends duplicated


I think you mixed up high and max here.


Fair enough. Makes sense.

The problem with your proposed implementation is that when called from a subclass, it doesn't return an instance of the subclass.

Actually, I think the only reason we have the constant is that there are a few other places in the library which need an empty `Hash`, and I thought `EmptyHash` would likely be faster than `Hash.empty`. If that is wrong, then the constant is unnecessary and can be removed. But for now, sure, let's go ahead and make it `private`.


This leads to a new warning
```
warning: No SupportedSourceVersion annotation found on org.adrianwalker.multilinestring.MultilineProcessor, returning RELEASE_6.
warning: Supported source version 'RELEASE_6' from annotation processor 'org.adrianwalker.multilinestring.MultilineProcessor' less than -source '1.8'
```

Which means it still assumes the RELEASE_6 if `SupportedSourceVersion` is not supplied. Quick googling suggests that it is rather a common problem. I will update this issue if I'll find a solution.


TeamCity Colore :: Main Build [Build 67](http://tc.sharpblade.net/viewLog.html?buildId=597&buildTypeId=colore_mainbuild) is now running


TeamCity Colore :: Main Build [Build 67](http://tc.sharpblade.net/viewLog.html?buildId=597&buildTypeId=colore_mainbuild) outcome was **FAILURE**
Summary: Compilation error: Corale.Colore\Corale.Colore.csproj (new) Build time: 0:0:39


Oh and Coarse Dirt, forgot that in the commit name :D


@MechWarrior99 I'll continue on 1.8 and 1.9 tomorrow, could you list what you have done so far?


added


Fixed: a56f44553b688a4cd0be61d86f1ab6886a74de59

Please file proper issues for that kind of thing


Thanks @addyosmani and @SmokyBob! I agree a rename it's too dangerous. Was thinking it might be ok to copy instead of rename but SmokyBob makes a good point. Instead of fixing PSK to accommodate one app (CDE), we should fix the app. :) I'll write an issue on CDE for this.


@kucrut maybe test for a `nav_menu_item` to be removed from `wp_get_nav_menu_items()` when its underlying taxonomy term or post deleted.


I feel like we should leave it named breakpoints since there aren't any other options that you can currently pass


You just need to change the first lines **bconf** and **conf** to match a standard Arduino, there are several example for that.

If you need further help, please use the mailing list available here:
https://github.com/souliss/souliss/wiki/Community


@dalderliesten: Is it possible to make the white transparent in the GUI?
@jessetilro: This is just a first commit, I will make everything consistent


Exceptions do not have a `message` attribute (on Python 3); you can use `str(e)` here.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 538](https://teamcity.labkey.org/viewLog.html?buildId=270645&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 30, passed: 9 Build time: 00:26:05
### Failed tests
```
org.labkey.test.tests.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBasic: <no details avaliable>

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


@yrashk
I will find the solution, too.

I suspect that it is influenced by source and target version in pom.xml

```xml
			<plugin>
				<groupId>org.apache.maven.plugins</groupId>
				<artifactId>maven-compiler-plugin</artifactId>
				<configuration>
					<source>1.6</source>
					<target>1.6</target>
					<compilerArgument>-proc:none</compilerArgument>
				</configuration>
			</plugin>
```
Thanks a lot.


@aharley40 this shouldn't be needed because the same code that checks for toggleOnDblClick also checks if there's an editor with clickToEdit==2 already


That line is blasphemous. Change it immediately, or else.


@aharley40 don't include "bonus" changes in commits, this change wasn't described in the first line of the commit and should have been in its own. the title of this commit could be better too, like saying what was actually changed and/or referencing the issue #


Build will fail until I implement a client in the root directory of the project (next on my to-do list).


@aharley40 this { needs to be on the previous line too


@aharley40 leading commas should go to when pulling old code into slate-admin


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

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


@benmcmahon100 Please lint your changes. This else needs to be on the line above.


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


Will do and re-roxygenize. You will then see my dilemma (I did try this before my PR), as there are some entries in the family that quite opaque S4 stuff. But obviously that's how it must be or there is yet another trick that I don't know ....


Next time please at least delete the original files if you copy-paste ;)


Ah that was a debug thing I added that I forgot to remove.


**valid-jsdoc**: JSDoc syntax error.


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


**valid-jsdoc**: Missing JSDoc parameter description for 'event'.


**require-jsdoc/require-jsdoc**: Missing @param description in the jsdoc for componentWillReceiveProps


**valid-jsdoc**: Missing JSDoc @returns for function.


**valid-jsdoc**: Missing JSDoc for parameter 'newProps'.


Aegis names should be NEVER changed. They aren't translations. I want to revert this commit, it is wrong.


@Tsunamical , this commit has all the changes I was testing before.
I didn't test the commit itself, but these changes, when in different commits, were working.
I'll compile again, but I'm 99% sure about it.


So the same as the variable above?


Pretty dope!


Indeed, I forgot about that one.


Why is this change being made at all?

We need to upgrade TwitterOAuth and you're not making it easy for us.


Hello,

When the client specify "dc:title", the server can not know what the "dc"
prefix means  (which xmlns is it ?)

Minidlna and UpnpPlay(android) declare the namespace/prefix associated to
each filter.
It is not the case of the Freebox revolution.

I too have found nothing about this declaration problem ...
I have forced the declaration of all known xmlns, I hope, it will fix the
problems.

Good night,

Olivier

2015-09-07 21:58 GMT+02:00 s-leger <notifications@github.com>:

> // Lame FREEBOX xmlns declaration !!!!
> As attributes depends on what the server provides, i think it's up to the
> server to filter expressions using her own capability, as you do here. So i
> doubt this should depends on node namespace. (can't find anything like this
> in the spec, but i can be wrong)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/oeuillot/upnpserver/commit/0cee30a505a497d7548845c9f32099ede3bba563#commitcomment-13101263>
> .
>


Which now? Is the indent wrong?


@MishimaHaruna I know what you mean and while I completely agree, it does happen in the community I've realized. I've also been noticing some aegis name changes on official server variants lately. So it's clear to me that there is not a constant between official builds. Quality control is falling hard in aegis.


Is this change really intended? I'm unable to find official information about this.


Can't we say `mvn -q exec:java -Prunner -Dtacoco.target ...` instead ... just like we did for the analyzer and reader CLI?


I don't think I have.


+1 (I noticed this and wanted to report...)


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 539](https://teamcity.labkey.org/viewLog.html?buildId=270648&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 30, passed: 9 Build time: 00:24:28
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBasic: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisBasic: <no details avaliable>

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


No.... Not required


> Планирую перенести и генерацию лэйаута всей странички туда.

Готово. Теперь вся боковая панель целиком генерируется из скрипта. Если строчку с инклудом скрипта из странички удалить, никаких следов боковой панели не останется и тело странички будет занимать всю ее ширину.


Awesome! Will test later tonight :)


Maybe a good Idea is not to expose your token in the URL, but instead simply listen to `/webhook` anyway you can send any URL to the setWebhook method, remember the token is the key to modify your bot.


But line 149 in _ _ init _ _ .py is still
```
EXTERNAL_IPV6 = False
```
Is that correct? Is it useful at all as ```EXTERNAL_IPV6 = test_ipv6()``` is always run?


@dholm, I wonder if was something like this: https://github.com/dimitri/el-get/pull/2238#issuecomment-138383276


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


Translating constants is at least as stupid as translating numbers. If they wish to do so, they shall also edit their scripts.

The real fix would be to make sure a wrong constant (or variable name) causes an error when loading the script. I have been suggesting that for years.


This:
```js
if () {
} else {
}
```

Not this:
```js
if () {
}
else {
}
```


I think it's much more useful to both notify users of functions that they could make themselves useful by finding out what official messages are, and offer an easy search for anyone looking to help out with such messages.


No need to pass obj, methods can directly use Bank(DB_FILE) as it is a singleton class and same instance will be returned everytime.


Ha Ha Come one now! That's a fairly nit picky stylistic thing! :p It's all getting minified anyway when being served to the client right?


@emmanuelbernard FYI I just pushed this one.


The uppercase sigils never perform interpolation. That's how all uppercase
sigils work in Elixir (no interpolation, no escapes).


It's not a mistake? It should be `UserManagementController` for this case, no?


```python
msg = Bank().addAccount(Account(name, email, password, typ))
```


caret needs to be removed, because it install beta4 with breaking changes


Don't we need ACCOUNT as well for the account balance?


This seems to break ncurses 5.9 builds on ARM systems, not sure if this also breaks x86 architectures or not -- but given the compiler error I am seeing I suspect it is broken universally.

```
gcc -DHAVE_CONFIG_H -I../ncurses -I.  -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64  -DNDEBUG -I. -I../include -I/opt/chef/embedded/include -I/opt/chef/embedded/include  --param max-inline-insns-single=1200 -c ../ncurses/./base/lib_initscr.c -o ../objects/lib_initscr.o
_1099.c:835:22: error: expected ‘)’ before ‘int’
Makefile:1550: recipe for target '../obj_s/lib_gen.o' failed
gmake[1]: *** [../obj_s/lib_gen.o] Error 1
gmake[1]: *** Waiting for unfinished jobs....
_1099.c:835:22: error: expected ‘)’ before ‘int’
Makefile:2075: recipe for target '../objects/lib_gen.o' failed
gmake[1]: *** [../objects/lib_gen.o] Error 1
gmake[1]: Leaving directory '/var/cache/omnibus/src/ncurses-5.9/ncurses'
Makefile:109: recipe for target 'all' failed
gmake: *** [all] Error 2
```

```
cd /var/cache/omnibus/src/ncurses-5.9
gmake -j10 all
-------*snip*---------
gcc -DHAVE_CONFIG_H -I../ncurses -I.  -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64  -DNDEBUG -I. -I../include -I/opt/chef/embedded/include -I/opt/chef/embedded/include  --param max-inline-insns-single=1200 -c ../ncurses/./base/lib_initscr.c -o ../objects/lib_initscr.o
_1099.c:835:22: error: expected ‘)’ before ‘int’
Makefile:1550: recipe for target '../obj_s/lib_gen.o' failed
gmake[1]: *** [../obj_s/lib_gen.o] Error 1
gmake[1]: *** Waiting for unfinished jobs....
_1099.c:835:22: error: expected ‘)’ before ‘int’
Makefile:2075: recipe for target '../objects/lib_gen.o' failed
gmake[1]: *** [../objects/lib_gen.o] Error 1
gmake[1]: Leaving directory '/var/cache/omnibus/src/ncurses-5.9/ncurses'
Makefile:109: recipe for target 'all' failed
gmake: *** [all] Error 2
```


Aah, then I get it. So ~E never interpolates, and ~e could be assumed to interpolate – but it specifically raises because I guess one should do <%= %> instead. Thank you for clarifying!


@MishimaHaruna Yeah I know. Keep it how it is for now please, I wanna do a item constant update later so it all comes in one commit.


INVOICE_AND_PAYMENTS, contain account tables i suppose (after reading the fields from the Group enum), so in that case scratch that comment.


@benmcmahon100 The style of this repo was set a long time ago. Lint you files and submit PR's.


Looking at that code it screams to be refactored into a function or macro. I wouldn't change the way  ALLEGRO_NO_PREMULTIPLIED_ALPHA works, but with some clever refactoring we can probably avoid the code duplication.


I wonder if a clearer error message might be something like: "interpolation not allowed in ~e sigil. Remove the interpolation, use <%= %> to interpolate, or use ~E to show the interpolation literally"


tests well with -n


Yep, I'm not reverting this (while it still holds true that I want to). My point is that, just like a wrong constant becomes 0, a missing item db entry would not be detected at startup. That's why constants were added to begin with.

I'd love to start working on that script engine update I mentioned, some time soon.


Right so I'll just leave it then next time


They'll do that all by themselves when they notice a message is wrong. And until someone does, others aren't confused by a debug message^^


TODO this could be abstracted, these lines could go into session class


@MishimaHaruna Agreed. I've been keeping a lists going on update we need, recently I dug into how syntax is handled and threw up a bit in my mouth. There's also a ton of missing commands we'll need at some point. I also wanna have our own extension for scripts for better working with it in syntax highlighters.


For some reason they chose to do it in the confusing reverse order. Expected goes first, then actual, in EXPECT_EQ


TODO add documentation for the developers what this line does and that it's optional


Remove unused `glob` module


please make it single-exit-point method


jg-security CANNOT have jg-rest-ws as a dependency! (REST are interfaces which agregating all modules). Note that you failled order of mvn modules buillding


Remove `glob` module, isn't used.


without `return` in front of the DB call to `save`, it'll break any `.then` chaining functionality


Change this to a description of the component itself (that's the source of the showcase component's description)


please add to Input test case from issue to prove that case is covered.


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


By taking ownership of the files away from SYSTEM don't we risk permanently breaking the Diagnostic Tracking, in case someone should want to revert? If you've run this on yours, right click the Diagnosis folder in C:\ProgramData\Windows, go to properties>security>advanced, click Continue to see permissions and make sure SYSTEM still has [Allow, SYSTEM, Full control, None, This folder, subfolders, and files]

As long as that is the case, we're golden!


should you be linking to babels fork of acorn?


@Ruined1 Pretty much, I'll implement a revert feature for the diagtrack log later.


Worth discussing/linking to `babelrc` here?


Yep! Also there was a memory leak in the negative part of this hunk where the signal watcher Lwt threads are created each time around the loop and then they all fire on signal. Now there is just one.

I hope the rest of it is tail recursive and that the tail-call elimination is not borked by the Lwt bind?


Photovoltaics (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


TODO maybe it's useful for the beginning to comment this out as session-concurrency prevention might be a deal-breaker for many users


`(high + low) >> 1` may overflow


In a 1 based array, the index at count is not past the end. Should be ```self.current_index > #patients```


@benmcmahon100 I suppose it's a matter of consistency, they're not being used anywhere else. And technically, we're saving a byte or something of downloading for the user per removed character, so there's that ;)


That sounds great, please send a PR!

--

*José Valimwww.plataformatec.com.br
<http://www.plataformatec.com.br/>Founder and Director of R&D*


Telegram's docs specifically suggest using the token.

> If you'd like to make sure that the Webhook request comes from Telegram, we recommend using a secret path in the URL, e.g. https://www.example.com/<token>. Since nobody else knows your bot‘s token, you can be pretty sure it’s us.


Would you re-consider this? Many of us have used Timecop for years without this rounding behavior, and it seems strange to account for a limitation of MySQL by compromising part of Rails that isn't really related to the database (preventing us from employing `#travel_to` fruitfully on sub-second timescales).


I should've pointed out that the line I've attached my comment to writes `/dev/tun%d`, which is unexpected because the macvtap driver creates a device node matching `/dev/tap%d` on my system.


Cool, then will change my bot to use this method, thanks


Hecho!


What was specifically changed from the previous code in the file "RoverCodeNoDriving.py"?


What happens if someone makes a symlink that points to `.git/refs`?


Photovoltaics (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@Ruined1 Oh I misread your comment, nevermind. This wont be a problem because it takes ownership away from the SYSTEM and gives it to the User/Administrator, so to revert it, all the user has to do is right click on the file>Properties>Security>Edit>Tick "Allow" for Full Control for SYSTEM


It looks OK. No side effect I think.
On Sep 6, 2015 7:02 PM, "Pierrick Le Gall" <notifications@github.com> wrote:

> @modus75 <https://github.com/modus75> can you please take a look at my
> changes and tell me if you think it won't break anything? As you can read
> in #348 <https://github.com/Piwigo/Piwigo/issues/348> this forced sorting
> order avoid unexpected decrement in remove_computed_category
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Piwigo/Piwigo/commit/195791c4e284204c4ff7d9ead9e7448d1cfc20ad#commitcomment-13086932>
> .
>


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


The main reason for this being that if someone else finds out the IP/port your bot is running on, they can send requests to it that aren't from Telegram, which could be potentially abused. With the token, it is nearly impossible to do this. If worst comes to worst, you can just revoke the token and create a new one.


:beer:


Whitespace fixes should be a separate commit.


default:rail, this is useless?


Should probably be `false` instead of `not true` to make it easier to read. In general, I think I'd prefer `ignore_hostfile` and `ignore_cache` instead of a `no_` prefix.


Awesome stuff :D (I was wondering where this module was in 1bcc6a6e)


Drastic unwanted changes to the workspace.


I commented out the lines that took joystick input and put it into the Left
and Right variables, and made them always equal to 128, effectively
disabling driving. This is so children won't get too excited and drive the
rover, but can still control the arm.

Also, I didn't see the original python code that we used in Utah on the
github, so I added what Robert had in an unmodified file.

Trevor Hedges
On Sep 7, 2015 3:16 PM, "Thomas Pryor" <notifications@github.com> wrote:

> What was specifically changed from the previous code in the file
> "RoverCodeNoDriving.py"?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/huskyroboticsteam/2014-15/commit/35df4fe84a930de5593667e27fba5491ea81ee69#commitcomment-13102143>
> .
>


2 useless crafts?


a bit of debug code leaking into the commit here?


What's the significance of adding checkpoints on 7199,7200,7201,7457 and 7540? Why not just one at 7540?


Should probably just be affirmative `["hosts" => $truth, "cache" => $nope]` instead of negation etc.?


Failure on my side. tap%d is correct name. I didn't noticed that this tun is from older commit which should not be pushed. On morning (now there is 00:30) i'll recheck if there is nothing other already fixed on my local-work tree not pushed there...


Понял - для каждого тикета - свой коммит.
Про возврат интерфейса не очень понял, почитаю. Если можете посоветовать
где - будет здорово.

2015-09-07 22:51 GMT+03:00 Slava Semushin <notifications@github.com>:

> Желательно возвращать интерфейс, а не реализацию.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AleksSPb/Spring-demo-App1/commit/f0937b457a7131b7dd3dbcfebb2e36f3a4c22d23#commitcomment-13101217>
> .
>

--
Всего наилучшего, Пархоменко Александр


furnace is ABM and useless,nobody use it, there isn't material for cooking?


Alright sounds good! Was this code tested during the meeting?


error prematuro


error permaturo


Not very important, but is there a reason you are using the `@doc "" ->` syntax as opposed to the plain strings that can be used on master nowadays? http://docs.julialang.org/en/latest/manual/documentation/


The `hasDuplicate` logic look for notification that are pending (future) or in-processing. The case where there is a future notification makes sense, but the case where the notification is in-processing seems flawed: When the notification shows as in-processing, there is no guarantee whether i/ the analytics handler has not been called yet, ii/ is being called and running, or iii/ has already been called (but notification was not yet moved to processed state).

The last case is an issue in the logic of the [handleKillbillEvent](https://github.com/killbill/killbill-analytics-plugin/blob/master/src/main/java/org/killbill/billing/plugin/analytics/AnalyticsListener.java#L180) because we will not create a new notification and may end up missing the latest state (as reflected with the current bus event). I think it only makes sense to avoid creating new notification where there is a future notification (not in-processing case).

Now i am unclear about whether this is also the case for the logic in [handleReadyNotification](https://github.com/killbill/killbill-analytics-plugin/blob/master/src/main/java/org/killbill/billing/plugin/analytics/AnalyticsListener.java#L126). I think in theory we have the same problem, but in practice because the of the `refreshDelaySeconds ` we introduce which is >> time to process notifications, that should not be an practical issue.


Not completely sure i follow the check on `futureUserToken`. seems like should only happen if if we have 2 threads processing the same notification. Is that possible?


@Brit200313 Working on getting my coverage back up to 80% at least.

I'm getting a new error that seems to be related to `user`/`current_user` and all that: `NoMethodError:
        undefined method 'authenticate' for nil:NilClass`

for instance:

```
PersonalEventsController GET #edit returns http success
      Failure/Error: get :edit, id:test_personal_event
      NoMethodError:
        undefined method `authenticate' for nil:NilClass
      # /Users/admin/.rvm/gems/ruby-2.2.1/gems/devise-3.5.1/lib/devise/controllers/helpers.rb:120:in `current_user'
      # ./app/controllers/application_controller.rb:8:in `set_current_user'
      # /Users/admin/.rvm/gems/ruby-2.2.1/gems/activesupport-4.2.1/lib/active_support/callbacks.rb:432:in `block in make_lambda'
```


That is total bullshit!


Oh! I wasn't aware; got used to having the `@doc` and didn't realize it has changed. I'll update accordingly


I hate your job


Great job


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 540](https://teamcity.labkey.org/viewLog.html?buildId=270658&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 35 (2 new), passed: 4 Build time: 00:31:19
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyGridColumnSelector: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridCheckerboarding: <no details avaliable>

org.labkey.test.tests.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

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


How to use を README.md に書くのだ。 ちょっとMarkdown覚えないといけないけども。是非。


@Brit200313 interesting, though, because the `set_current_user` method passes!


maybe add unbreakable group torch?


Do you still want to be listed on the libsass implementation page? There's another libsass wrapper that is up to date with libsass here: https://github.com/wellington/go-libsass


It looks tail-recursive to me. Or at least, my mental model of `bind m f` is that it tail-calls `f`. I suppose that's really a property of the lwt implementation. Might be worth checking out the lwt source to satisfy yourself!


This is completely wrong; it refers to a version of the function that was replaced.


You're also changing the std:: qualification here.
This matters as std::string may sometimes be different from ::string.


@dynasource if you're sure that https://github.com/yiisoft/yii2/issues/9288#issuecomment-134597913 is better, would be great if you'll create another pull request against master.


For transferring file contents, there's the 'Load' command in the CLI protocol - would it be better to use that?


The URL is simply ` http://localhost/index.php` now.


Adding "brand"


Ah. Yes.


Should we just remove the document altogether then?


Good catch.


Even `http://localhost:8000/index.php`


Would oclock be better here?


added an extra test case


Still want to work with just flex-box at the moment. Will tidy up the @media queries for the larger viewports then will have one more go at working only with min-width. Need to complete this course and will do a bootstrap version if I can squeeze in some time.


Done


I'll fix it.


Thx


Heh, I thought I already started getting used to it, but then EXPECT_THAT() came into play...


Urgh!!!

I have vim set to automatically kill whitespace. ;)

Fixed


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 541](https://teamcity.labkey.org/viewLog.html?buildId=270661&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:10:31
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

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

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

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

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

```


Why not `m => ...`?


ditto


Do you have the updated branch somewhere? I was going to rebase the branch today but I don't want to loose the fixes you already did.


I managed to connect the display, but the results are not too promising yet!

![image](https://cloud.githubusercontent.com/assets/6170837/9723573/e34844b0-55bb-11e5-8e8e-3a04ebdf2bcf.png)


@artstorm the above PR just got merged does it still affect us?


This revision is clean.


it doesn't appear that there are instructions for creating `blog.md` although the file exists in the [part1 source](https://github.com/realpython/flask-by-example/releases/tag/part1) with part7 selected


¿¿Y Diego??


Can you delete the temp files generated in that partition before raising the exception?


messy, redoing.


Trailing comma


A note on this, you are relying on a reference still, if someone mutates that reference you will still get the changes.

The reason this works right now is because the _removeConsumer does a reassignment. Cloning is still a safer way to go.


Assuming all other non-trait-proxy classes are available through the KieBase. Is this assumption safe?


Much.


Oops
From what I can infer, the feature being implemented here is intended to fix the need for the second IP address. This is actually already finished in the [testing](https://github.com/NICMx/NAT64/tree/testing) branch. It was implemented as a fix to NICMx/NAT64#36 and [this](https://github.com/NICMx/NAT64/issues/67#issuecomment-23980186).

The code looks stable; we're waiting for some more testing to be completed to release.

Documentation snapshot [here](https://dl.dropboxusercontent.com/u/95836775/doc-pool4.zip) (see `mod-run-stateful.html`, `op-pool4.html` and `usr-flags-pool4.html`).


これは入門サイトでセキュリティ対策として使った方がよい、ということが記載されていたためです。
具体的にどんな脅威に対する対策なのか、使わないとどうなるのか、使うべき個所はどこなのか、といったことまでは理解できていません。


Should this information be in the Header?
Would any more specific structure have been rebuilt already by the time this information is needed?


Hi!

I am getting a error compiling this version:

[ 60%] Building CXX object CMakeFiles/pastec.dir/src/requesthandler.cpp.o
/opt/pastec/pastec/src/requesthandler.cpp: In member function 'bool RequestHandler::testURIWithPattern(std::vector<std::basic_string<char> >, std::string*)':
/opt/pastec/pastec/src/requesthandler.cpp:99:9: error: name lookup of 'i' changed for ISO 'for' scoping [-fpermissive]
     if (i != parsedURI.size())
         ^
/opt/pastec/pastec/src/requesthandler.cpp:99:9: note: (if you use '-fpermissive' G++ will accept your code)
make[2]: *** [CMakeFiles/pastec.dir/src/requesthandler.cpp.o] Error 1
make[1]: *** [CMakeFiles/pastec.dir/all] Error 2
make: *** [all] Error 2

The sequence I am using in ubuntu:14.10
sudo apt-get update
sudo apt-get install -y libopencv-dev libmicrohttpd-dev libjsoncpp-dev  wget
sudo apt-get install  -y cmake git
cd  /opt/pastec
git clone https://github.com/Visu4link/pastec.git \
&& cd pastec \
&& mkdir build  \
&& cd build  \
&& cmake ../   \
&& make \
&& echo "End of install"

To work I have to revert to the version:
git checkout   c5e536adccaa34c925bc4bce3a7b74daa835af9d

Is it possible that was a issue on the commit of this version?

Thanks in advance, very nice api! +1

Regards,

Cesar


Can you make this `Notes:` to be consistent with the others. That might be helpful once we parse this file to generate the component guide


ありがとうございます！一度オフラインで相談させてください。


Is this a joke?

```
2015-09-07 16:09:28 ~ $ shasum -a 256 libelf-0.8.14.tar.gz
8022ccbd568c02e0b36331e533f78eeb3d0d1fa5637adac9b8b816d1ea358ebf  libelf-0.8.14.tar.gz
2015-09-07 16:09:22 ~ $ tar tvzf libelf-0.8.14.tar.gz |head
drwxr-xr-x  0 0      0           0 May 12  2003 libelf-0.8.4/
-rw-r--r--  0 0      0        2657 Oct  7  2001 libelf-0.8.4/acconfig.h
-rw-r--r--  0 0      0        7091 Jan  2  2003 libelf-0.8.4/aclocal.m4
-rw-r--r--  0 0      0       23810 May 12  2003 libelf-0.8.4/ChangeLog
```

Indeed, all of the files in libelf-0.8.4 are very old.


Can you change this to ` * - ` instead of ` *  *   ` ? While this is clearly better, we should try to be consistent with the other blocks to ensure that we can consistently parse when generating the component guide


$item = $_GET['item'];
$item = str_replace("\"", "", $item);
$item = str_replace("\'", "", $item);
$item = str_replace(" ", "%20", $item);
$item = str_replace("\\", "", $item);
@include_once ("set.php");
$rs = mysql_query("SELECT * FROM items WHERE name='$item'");
if(mysql_num_rows($rs) > 0) {
	$row = mysql_fetch_array($rs);
	if(time()-$row["lastupdate"] < 3600) die($row["cost"]);
}
$link = "http://steamcommunity.com/market/priceoverview/?currency=1&appid=730&market_hash_name=".$item;
$string = file_get_contents($link);
$obj = json_decode($string);
if($obj->{'success'} == "0") die("notfound");
$lowest_price = $obj->{'lowest_price'};
$lowest_price[strlen($lowest_price)] = 0;
$lowest_price = str_replace("$","",$lowest_price);
$lowest_price = (float)($lowest_price);
mysql_query("DELETE FROM items WHERE name='$item'");
mysql_query("INSERT INTO items (`name`,`cost`,`lastupdate`) VALUES ('$item','$lowest_price','".time()."')");
echo $lowest_price;

cost.php

maybe better that ur code


Ahhh, sorry I misunderstood this, but original code always apply `add_const` to initial state, doesn't this yield same result?


Does this work everywhere without adding `position: relative`?


A look at the source (test/dhtest.c) tells me that this algorithm check is quite correct, really...


Same here about `position: relative`


Yeah, as a general rule I agree. It's not a difficult change to make and it means things "Just Work" for even less technical users. Though having said that I'm not sure what if any performance issues there are associated with using my way instead of just iterating through them all on load once.


I know, but cloning causes memory pressure, and I'd rather not do that in a function that is called every time an element is emitted. It's better to just make sure that changes to those arrays does a copy than to defensively copy all the time.


If we are trying to discard the data here, and we have a known content length for this request (not chunked input), should the `discard` method accept a total number of bytes to discard?


Above comment aside, you're quit correct.  We could have the ecdh and ecdsa tests depend on no-ec, but the test app does that check internally anyway, so...


Checked
https://github.com/ruiztulio/backupws/blob/leo-space/lib/utils.py#L209


...doesn't work because you're never calling `a()` :P


fix this
<link rel="stylesheet" href="css/uath.css">


I'd be interested to see the actual memory ramifications of creating a new array with existing objects inside it, but overall I can buy that argument.


I don't necessarily have a better idea.  If that underscore is common in Monocle, then I think it's the right thing to do.


coglione


Fine too, as long as defaults are ensured to be `true`.


Bilgileri içeren sınıf


Veriler


Yes. We take humour very seriously at Homebrew.

We rely on a wide community & contributor base to keep packages up-to-date. We don't have the humanpower or time to vet every single submission intensively. In this case, the fault seems to be with upstream who for some reason have a redirect enabled from `0.8.14` to `0.8.4`, which is the cause of the tarball contents being a little old.

```
==> Downloading http://www.mr511.de/software/libelf-0.8.14.tar.gz
==> Downloading from http://www.mr511.de/software/libelf-0.8.4.tar.gz
```

Sometimes mistakes happen. We appreciate people notifying us of those mistakes so we can fix them.


レビューありがとうございました！大変参考になります。
私もこのあとデータ指定にIDが使えた方がよいと思い、ID対応版を作成してみました。(9/7commit分です)
2点目の・については、リンク先の手順通りにsqlファイルを作成してプッシュしました。もしイメージと違うものでしたら、教えてください。
宜しくお願いいたします！


@VisualKei
```css
float: none !important;
```

Дабы не было вопросов - зайдите на http://8ch.net/b/, откройте Избранное, и попробуйте отключить это свойство. Достаточно убрать `!important`. Увидите неожиданный лизурьтат.


Hihi
我不是歪國人~
2015年9月8日 上午1:25於 "adioscat" <notifications@github.com>寫道：

> hi~
> 請問你懂中文嗎?@@
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/k19x1234/Trove-AHK-AutoFish/commit/3910462f5adbf7154e250f735d23fa26958669dc#commitcomment-13100238>
> .
>