Congrats on all of this so far.

The output you sent over last night is unbelievable. We should illustrate with this output all over the docs as much as possible.


@ichord thanks! I checked your last commit and, although I'm a bit confused by the coffee layer, I think this should fix the issue, since now each timeout belongs to the controller instance and they should no longer overwrite themselves.

Thanks for your work on this plugin :+1:


This change seems to get confused when dir-paginate is applied on custom directive with ng-if in its root element. Under such circumstances addNgRepeatToElement function receives a "comment" node as element[0] that has no hasAttribute method causing "TypeError: undefined is not a function".
The previous version was not affected by this problem and was able to get proper element there.
Unfortunately I failed to produce succinct enough snippet to put it on Plunker.


When I updated to 2.3.7, I am seeing the code data-rel="prettyPhoto"> and the name of the picture above the single product picture.   What would be causing this?


@magnayn do you mean @abayer ? He doesn't do core development according to commits. And he has no idea why/how he did all this changes. Instead of resolving issues people just spread bugs to other plugins.

You said that pom shading copy-paste from jclouds just works. And now you did another one copy-paste and says again that it "just works". So previous "just works" was not "just works".
I can help with development and resolve issues. But i don't want to do debugging and fixing after somebody copy-pastes, that even didn't do changes through PRs.

Nobody even tried to fix or understand what happens with hpi and shading. You will always add new bugs without understanding what do you actually do.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1241
success: http://build.servo.org/builders/linux2/builds/1239
success: http://build.servo.org/builders/android/builds/1275
success: http://build.servo.org/builders/mac1/builds/1454
success: http://build.servo.org/builders/mac2/builds/1457
success: http://build.servo.org/builders/gonk/builds/420


@malukenho can you actually give a shot fixing this?

Marco Pivetta

http://twitter.com/Ocramius

http://ocramius.github.com/

On 1 April 2015 at 14:35, Jefersson Nathan <notifications@github.com> wrote:

> @GrahamCampbell <https://github.com/GrahamCampbell> that's code is
> generated by couscous when accept any PR on the master branch. Couscous
> is run by travis and deploy the web site on the gh-pages branch.
>
> You can revert this one and try to make a if/else here
> <https://github.com/Ocramius/ProxyManager/blob/master/doc-template/default.twig#L4>
> on the master branch.
>
> /cc @Ocramius <https://github.com/Ocramius>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Ocramius/ProxyManager/commit/a3135cb378d09f587385476baa02a5574a075f67#commitcomment-10522246>
> .
>


Yes this is the Woodstock document


this change is wrong. I am calling the "timeout" command, not a flat "timeout". Basically the entire CLI call is being wrapped in the "timeout" command.


See in line comments.


@Montellese Yes thank you, what I wanted to see is how the icon looked, so I replaced a existing icon with this and turned out to be wrong.

Please use this one below instead, both fits with style of skin/color and looks much better than the draft icon I originally done.

![icon_events](https://cloud.githubusercontent.com/assets/3521959/6942145/7545d81c-d87d-11e4-9e7f-9c86ae5ebd2d.png)  <-- its there I promise :+1:

this is how that looks with a dark BG for test purpose only
![test](https://cloud.githubusercontent.com/assets/3521959/6942190/c99b707a-d87d-11e4-8f63-dd981e1c7e3a.png)


I see,thank you !


In my opinion logic should be different:
Right now default value is more important than actual one.
Should be:
//try to read actual value from content
Object value = mapResourceToField(resource, valueMap, field);
if(value != null){
 FieldUtils.writeField(field, object, value, ReflectionHelper.FORCE_ACCESS);
}
//otherwise fallback to default if field is initialized (nothing to set actually)


fast-forwarding master to auto = f25604c7


You sure?


Seems like the link is broken?


经过上面的代码处理后，可清除的Bitmap会驻留在 Ashmem 堆中。不管发生什么，垃圾回收器都不会自动回收这些 Bitmap。当 Android 绘制系统在渲染这些图片，Android 的系统库就会把这些 Bitmap 从 Ashmem 堆中抽取出来，而当渲染结束后，这些 Bitmap 又会被放回到原来的位置。如果一个被抽取的图片需要再绘制一次，系统仅仅需要把它再解码一次，这个操作将会被完成地非常迅速（这个过程非常快）。


@cpaxton this needs to be updated to `hog_desired`


Don't close over the parent context -- use [`bind`](https://github.com/petkaantonov/bluebird/blob/master/API.md#binddynamic-thisarg---promise) in the Promise chain instead


Thanks @luke-


saw approval from jdm
at https://github.com/servo/servo/commit/4e9bb15ba73fa3a1c8a744003042fc29a9489bc1


merging servo/servo/load-failure-panic = 4e9bb15b into auto


servo/servo/load-failure-panic = 4e9bb15b merged ok, testing candidate = 6824bc93


good catch!


You mean there's a "S" missing in ``+= ${VARS_FILE}``?


how to use this nesbox-companion?


Are these documents queried based on creation date filter input?


so my point was more around removing the code, and not the hack. Can the functionality of the code be achieved some other way?


Che benja... claramente aca comiteaste sin probar... y despues mandaste el fix.. *recuerdos


在Android设备上面，快速高效的显示图片是极为重要的。过去的几年里，我在如何高效的存储图像这方面遇到了很多问题。图片太大，但是手机的内存却并不大。每一个像素的R、G、B和alpha通道总共要占用4byte的空间。如果手机的屏幕是480*800,那么一张屏幕大小的图片就要占用1.5M的内存。手机的内存通常很小，特别是Android设备还要给各个应用分配内存。在某些设备上，分给Facebook App的内存仅仅有16MB。一张图片就要占据其内存的十分之一


解码的过程是在GPU上的密集操作 -》 CPU

UI会一直停滞-》UI会一直假死

现在它们推荐使用另外一个标志位——inBitmap -》现在它们推荐使用另外一个特性——inBitmap

除非说App里面所有的图片都是相同大小 -》 除非 App 里的所有图片大小都相同


It return dif value, plez Trees


@DeSerg Без особых оснований не стоит делать свойства приватными, лучше - protected. Тогда к свойству смогут иметь доступ классы-наследники.


Blame @maggiben!
ping @fansworld/deciders.


Yep, that would be more cautious. On the other hand, the whole process could be SIGKILLed during that write, which would also result with a partial file.

What about using the atomic rename trick here: write to file `x.ts.part`; rename to `x.ts` afterwards?

I also thought about adding a new command named `clean` to remove all the pytoutv temporary files in a given directory.


Is that prefix can be stored in String constants?


You triggered my null aversion! *Barf.* How about passing an empty collection instead?


is there any special reason to roll back here? Thanks!


See commit message.


No, I mean't `geoportailv3` has nothing to do here.
Sorry, it was clear only for me.


Used $config['auth_ldap_groupmembertype'] and $config['auth_ldap_groupmemberattr'] for compatibility reasons, see http://www.observium.org/wiki/Authentication_modules#OpenLDAP .


很小


I would use a maven variable here instead having to maintain the versions in 2 different places


Now I see: this requires that test.txt contains double-quoted text. Our .style files don't, that would be an excess complication. I don't see how to include unquoted string yet.
This will require an extra build step - appropriate text file generation with `xxd` or something. With your permission, I'd leave this for now.


The indentation in the code you write is often funky. Can you check your Eclipse preferences?


@DeSerg В таком виде функция `sprintf` лишняя. В подобных исключениях хорошо бы указывать, какие данные недопустимы и почему.


Yep that's sounds good.


Why not check the actual value?


It looks like glProgramUniform is only available as of gl 4. We cant use it.
On Mar 31, 2015 8:35 PM, "HoneyHazard" <notifications@github.com> wrote:

> Aparently using GL.ProgramUniformX on a Mac refults in invalid GL
> operation. I'm investigating further
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jeske/SimpleScene/commit/63726bc28907fb1fcf198e4ea95d129c7e66ad24#commitcomment-10509910>
> .
>


Ohhhhhhhhhhhhhhhhhhhhhhh thanks.


Not sure what you mean by that. The code before this was the result of a pull request. I got rid of all that if-statement stuff there and cleaned up the rest.


Ah damn. Writing to the config is a bit annoying to do.


Yes that's what I intended.


Yeah I knew that when I added it.  It's an easy addition though.


@dereuromark This is not required. The action method is still executed without `echo`. An `echo` is nice if we have some HTML to output, but in our case we don't need it.


>Something about that sentence tells me that you are not trying very hard to resolve any conflict,

It seems haasn already decided that he will keep his fork no matter what, and that a fork of mpv has to exist because of my past mistakes (which was to get into an argument with him). I haven't seen a single attempt from haasn to resolve this conflict, in fact he seems to be very glad about it.

There isn't even a real conflict. In the past I've merged virtually everything he did. He's just mad that I called him names and disregarded his argument that doing something broken is ok because ASS subtitles need it. (Though I didn't change how ASS colors are handled. Hell, I've even implemented the vsfilter bt.601 hack myself personally, so shitty weeaboos can have their shit.)

If even a normal patch review doesn't do it, what do you want me to do? Merge changes from a foreign fork FFmpeg-style?


Hi. I'm getting "Maybe: fromJust" error from snaplet-acid-state(http://hackage.haskell.org/package/snaplet-acid-state-0.2.6.1/snaplet-acid-state.cabal) since the cabal file has "x-revision" but not "x-cabal-file-hash".


You missed one case here: what is it?


It doesn't work with the emulator embedded on other websites, only here http://nesbox.github.io/emulator/
I'm going to remove the restriction in near future


Haha


Just what I needed!


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


reverted at https://github.com/MrSwed/MODXEvo.plugin.ManagerManager/commit/2d47c9ef9a6838daed2e046eda2b93dba1efff59


The logging is quite verbose in the case of the unreachable graph API, we should discuss it maybe. @Aranir


Please continue the work as described above.


May I sugest `' '.join(('openSUSE' , self.openqa_version(), self.openqa_group())).strip()`?  It does not avoid the `.strip()` but is a bit more clear IMHO.

In either case +1


I see.thank you for your help!


Mas isso é pra testar exatamente esse comportamento, não? Que se o cara não tiver autenticado vai ser redirecionado pro login


Any chance of moving this into main spigot resources? Seems to be abandoned on bukkit, and everyone is crying out for it. If you do a job anything like chat control, basically we know it's a sure fire winner!

(I've had a go at compiling, but need more of a crash course or guide on how to do it first  :)  )


OK


What is the WPT job PLR supposed to mean – total probability of the roundtrip ploss, or the link-level probability on the UP _and_ the DOWN links?

Ptotal = 1 - (1 - Pin) * (1 - Pout) = Pout + Pin - Pout * Pin

So for small Pout and Pin <<1, the Ptotal would be roughly their sum. This might not be what the test owner expects, if by PLR they intuitively mean the rate that the ping command reports – the _roundtrip_ loss rate...


One of my Support cases last year resulted in incorrectly exported dates when using `LIST_DATE_FORMAT` with the default setting in an `ExportList` API call.

Dates in the export file appeared as e.g. _56/15/2014 07:56 PM_
A Support Representative replied with the following on 24 August 2014: "You should only use the LIST_DATE_FORMAT if you wish the exported format to be something other than the standard MM/DD/YYYY."

I need to implement recent changes into my fork, as I've specified the different formats used by Silverpop (for `ScheduleMailing` it includes an AM/PM time) as constants and pass any dates internally as timestamps (to avoid confusion).

	// Silverpop date format
	const SPOP_DATE_FORMAT	= 'm/d/Y H:i:s'; //2 digits with leading zeroes for month/day/hour/min/sec, 4 digit year
	const SPOP_TIME_FORMAT  = "m/d/Y h:i:s A"; //see SPOP_DATE_FORMAT, hours as 00-12 with leading zeroes, added AM/PM

For `ExportList`, the `DATE_START` parameter would change as follows

	if (!empty($startDate)) {
		$params .= "\t<DATE_START>".date(self::SPOP_DATE_FORMAT, $startDate)."</DATE_START>\n";
	}

I sometimes use different date formats for different API calls (e.g. `ExportList` vs `RawRecipientDataExport`) so have made it part of the endpoint instead of defining it as part of the connection (although I suppose you can have both a default setting and overwrite where necessary).

I am not sure if they actually fixed the dates and formats yet (and keep hoping for easy locale settings...)


not needed, there's no possibility to come other different type, cuz we invoke this function internally and do not expose it to the api directly


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


Cosmetic: Maybe be explicit here with comparison precedence to have better maintenance?
`if (isArray(value) || (isObject(value) && jQueryMode)) {`


TeamCity xBIM :: XbimExchange :: Build [Build 178](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1268&buildTypeId=XBIM_XbimExchange_Build) is now running


注意对齐


The login button works, the window close down after pressing login botton.


Thank you for your review.

I'm going to fix those common mistakes what you mentioned and resend pull.
Regarding of test cases I will need to learn how can I make test for ui mistakes but I will do it.

Thanks again,
 Zsaga


Doh! My bad. Fixed.


Fixed all of those.


`"cells": [0, -1]`  would be better to remove the CSS injection cell from the published NB.


esse teste então não estaria em lugar errado?
onde poderia colocá-lo para não repetir isso em todo comportamento?


Fixed via d3b1e3f56a8d3b2bccb36a973299c1f85985e48f


@snopoke
In the  shell on staging I'm seeing that row['time_end'] should already be timezone-naive.

```python
>>> fd = FormData.objects.filter().all()[0]
>>> fd.time_end
datetime.datetime(2012, 6, 17, 16, 8, 2)
```

Any reason for `.replace(tzinfo=None)`? If not I'll remove it just to reduce confusion.


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


This should be left out of the main repo.


Did you already import the `FindMatlab.cmake` from CMake? Does it work? If so I will import it in YCM, even though this patch means that I cannot just download it :(
Anyway according to my [evil plans](http://wiki.icub.org/wiki/YARP_Supported_Distributions), we are bumping dependency on CMake 2.8.11 in a month or so... ;)


Fixed via 414af0f56d59d8999d202a7cb976b68b9b97434a


Doesn't this need to trap processes that exit normally also?


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


nice. i like it when it's an easy change


Misstänkte det, inga problem :sweat_smile:


There are some oddities here that I would like to discuss, as mentioned yesterday.


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


hey @sivakuna-aap , what can be used to move an item to different stage on the same desk?
seems like move blocks this


As I've already said, I don't agree with your assertion of 'without understanding what you actually do' - the issue (the way jenkins interacts with classloaders) is understood precisely. It's also pretty well understood that the the classloading infrastructure is pretty hairy, leading to sub-optimal solutions like shading.

Please explain what bug is now present, and raise an issue if neccessary.

I hear that you would rather "try to fix or understand what happens with hpi and shading". That's great, because I won't be doing that. I agree that doing so would represent a better solution. But I won't hold up not apply fixes waiting for a solution that may or may not materialise.


Sorry to be so picky, but `do..while` is missing :(


@Glenpl Takie warunki odwracaj – czyli, żeby nie bylo `!` – mniej kodu do czytania, mniej kodu do wykonania ;)


The violates the interface described by the super class, which while not critical is "nice".


Can you place `"sqlite"` at `notebooks[0]`.

The order  in `references.json` corresponds to the order in the list here:

![image](https://cloud.githubusercontent.com/assets/6675409/6942809/f26b8556-d857-11e4-8d82-5495a40e028e.png)

So the latest NB should correspond to the first item.

Sorry about that, this part is poorly designed.


Unfortunately this patch is necessary for CMake 2.8.11 (it is not necessary for CMake >=2.8.12).


TeamCity xBIM :: XbimExchange :: Build [Build 178](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1268&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:05:59


You could, for example, parse your response to a dom in cleanup so that you are only doing it once for multiple times round the loop of entries in the test_key.

I agree that cleanup() is the wrong name.


:+1: a pro-rated URL


Brilliant.


@julianorrish Tip: include the abbreviation for the book title at the start of your comment, e.g. 'FG: Update links on copyright page'. That way, others can see which book you've changed in our commits feed.


How I should correctly translate the lines 37  and 41? Maybe, need to remove?


int limit = null?


So in short, we always track anything with a protected type anywhere in its type, and make sure it gets correctly closed with an appropriate close method.


@Glenpl Może jest w cpp jakaś składnia shortifów?

Coś w stylu `return (x>8) ? "" : " ";`

Zamiast `if(x>8) return ""; else return " ";`


@brian0305 Since when does trees code the Core?


dfs


dfd


```bash
Cloning into 'projects/rb'...
remote: Repository not found.
fatal: repository 'https://github.com/feelpp/rb/' not found
Clone of 'https://github.com/feelpp/rb' into submodule path 'projects/rb' failed
```


对于上面提到的“解码操作致使 UI 假死”的问题，我们找到了一种同时使 UI 显示和内存管理表现良好的解决方法。如果我们在 UI 线程进行渲染之前把被抽取的内存区域放回到原来的位置，并确保它再也不会被抽取，那我们就可以把这些图片放在 Ashmem 里，同时不会出现 UI 假死的问题。幸运的是，Android 的 NDK 中有一个函数可以完美地实现这个需求，名字叫做 AndroidBitmap_lockPixels。这个函数最初的目的就是：在调用 unlockPixels 再次抽取内存区域后被执行。


dont think that is whats gonna be hard coded...believe it was just to test the commit process


创建一个内存安全


Added the heading, 'Kannada'.


@Glenpl `->string {if(x>8) return ""; else return " ";}`

Takie coś widzę identycznie już w conajmniej czterech miejscach: ktoś tu robi kopiuj–wklejki ;)
Poszukaj gdzie można tu sensownie jakąś funkcję wyciągnąć.


Make this plate to Plate with a capital P.. BTW what happened to the stuff we worked on yesterday?


there was no Hydro-devel branch


Oops.


@MacDada kurka, no wiadomo, że jest ternary operator! :D zbyt się podnieciłem tym, że pierwszy raz używam funkcji lambda i kompletnie o tym zapomniałem. Zaraz wrzucam robię commita, bo po tym w którym wyrzuciłem własne funkcje konwertujące int/string program zaczął się wywalać przy podaniu nieprawidłowych danych wejściowych


@Glenpl BTW, nie widziałem wcześniej takiej składni – to jest jakiś rodzaj funkcji anonimowych, closure?


@Dinuuu ahi cambie lo que me dijiste y rebasee con master porque habia conflictos


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/88e765bd4ec518cd81c17c54f6e5684abc2848cb/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/88e765bd4ec518cd81c17c54f6e5684abc2848cb/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/88e765bd4ec518cd81c17c54f6e5684abc2848cb/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/88e765bd4ec518cd81c17c54f6e5684abc2848cb/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/88e765bd4ec518cd81c17c54f6e5684abc2848cb/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


The version is empty? Is this expected?


This sounds like a pretty reasonable change, and it seems like you can point to a case where the existing implementation really did create a problem in Silverpop.

If you get a fork set up with this working the way you suggest, please send over a pull request.

I would recommend that the default be left as-is, but allow you to override it on a case-by-case basis. Whether we use the default or an override passed on an individual call, only including the format if it is non-empty (and perhaps setting the default format to NULL) would resolve your issue, but still allow other users to specify a different default if needed and then not have to worry about passing a date format with every call. Right?


hola probando esto


No reason for it, likely a copy paste issue from https://github.com/dimagi/commcare-hq/blob/03d1126baa23dc55ecac6c897cfae52c3e331003/corehq/apps/reports/standard/monitoring.py#L726


saw approval from seancribbs
at https://github.com/basho/riak_kv/commit/8722eb7580503f328dbe17e316f53736aa127d8a


merging basho/riak_kv/feature/fd/RIAK-1534 = 8722eb75 into borshop-integration-1100-feature/fd/RIAK-1534


basho/riak_kv/feature/fd/RIAK-1534 = 8722eb75 merged ok, testing candidate = d5f85fec


@MacDada, tak, anonimowe funkcje wprowadzone w standardzie c++11
http://en.cppreference.com/w/cpp/language/lambda


Good work.


One reason to use a stream here is to reduce memory usage. You don't need the entire stream in memory at once in order to compute the sum. That said, this is at most 80kb, so not a huge deal. But if there were some way to do this both in parallel and lazily, that would be ideal.


@shinnn, if we want to keep empty directories. Adding `stripDirs(entry.fileName, opts.strip) === '.'` to this `if` statement would work, but would it affect anything else you think?


Ah, that makes a *lot* of sense. Dateutil always returns timezone-aware datetimes, even when it's inappropriate (which is really annoying).


That's duplicate code... A simple ?: would suffice, on the left/top assignments.
Same goes for the horizontal version.

Btw, these buffer classes do not provide you any extra benefit, you could use buffer arrays on the same renderer class without all the extra complicated stuff


Thanks! Couple of comments:
1) Looks like you accidentally committed the Emacs backup file `PharoTour/PharoTour.pillar~`. Can you remove it? (We should add `*~` to the `.gitignore` file, as well)
2) Can you do further additions as Pull Requests? Easier to comment on them, that way.

Thanks again.


centos-release restored[1] /etc/redhat-release as a symlink so this change is not necessary but also does not hurt i.e. I won't revert it but won't push rdo-release updates either.

[1] https://git.centos.org/commitdiff/rpms!centos-release.git/f3a9a7fb3d640e80ed979be291330537a174d6b9


I'm somewhat losing track what I have changed and what not, but currently it shows right numbers and doesn't complain about undefined variables, so I guess it's okay by now.


Use the internationalizations instead of "Cannot have blank values"


Ah - thanks! I assumed it included that for me :/ Will do in future :)

On Wed, Apr 1, 2015 at 4:17 PM, Arthur Attwell <notifications@github.com>
wrote:

> @julianorrish <https://github.com/julianorrish> Tip: include the
> abbreviation for the book title at the start of your comment, e.g. 'FG:
> Update links on copyright page'. That way, others can see which book you've
> changed in our commits feed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/electricbookworks/bettercare/commit/7f4c3e242a4da8587b771e648d3a5755e4cdba53#commitcomment-10523409>
> .
>

--

Kind regards

*Julia Norrish*

Intern | Electric Book Works | 3rd Floor, Sunclare Building, 21 Dreyer
Street, Claremont, Cape Town, 7800 | T: 021 671 1278 | C: +27 (0)74
464 5588 | E:
julia@electricbookworks.com | Website: www.electricbookworks.com

*We develop new and better ways to publish in emerging markets.*


the setUp method is called before each individual test so its identical, i did it locally and the only issue i had was with the user still existing because you forgot to delete him in the teardownafterclass method.

But all tests still work, the mocks your modifying a fields and can be modified for each individual test like you wanted.


It's supposed to be the overall likelihood of packet loss on the link, regardless of direction.  A "ping" traversing in both directions is 2 packets and each has equal likelihood of hitting a loss event.


sdsdfsdfdsdsf


sfddsfdsfdsdsds


sdfdsfdsds


@Glenpl Wyciągnij do funkcji nazwanej.

W JS można zrobić tak:

```
setTimeout(function () {
    // tutaj jakiś kod, który wykona się po upłycięciu czasu
    // jest on w funkcji „anonimowej” (bez nazwy)
    // cały jej kod jest skopiuj–wklejkowany powiedzmy 5 razy w różnych miejscach,
    // ale na przykład za każdym razem z innym czasem
}, 1000);

// gdzieś dalej w kodzie „prawie” to samo
setTimeout(function () {
    // tutaj jakiś kod, który wykona się po upłycięciu czasu
    // jest on w funkcji „anonimowej” (bez nazwy)
    // cały jej kod jest skopiuj–wklejkowany powiedzmy 5 razy w różnych miejscach,
    // ale na przykład za każdym razem z innym czasem
}, 2222);
```

To można taką funkcję zrobić jako „nazwaną” i reużyć w dwóch miejscach:

```
function namedFunc() {
    // tutaj jakiś kod, który wykona się po upłycięciu czasu
    // jest on w nazwanej funkcji
};

// odwołanie do samej nazwy, bez odpalenia funkcji,
// dzięki czemu nie kopiuj–wklejkujemy
setTimeout(namedFunc, 1000);

// gdzieś dalej w kodzie „prawie” to samo
setTimeout(namedFunc, 2222);
```


Nope, I left the master in a broken state when I went to sleep.  I'm fixing
it right now, another 10 minutes and all should be well again.

On Wed, Apr 1, 2015 at 3:49 AM, JonWel <notifications@github.com> wrote:

> It doesn't compile for me on Ubuntu 64 bit.
>
> To be sure not having any submodules issues, I deleted the previous
> directory and downloaded all freshly:
>
> git clone https://github.com/CoolProp/CoolProp --recursivecd CoolProp
> mkdir build && cd build
> cmake .. -DCOOLPROP_64BIT_SHARED_LIBRARY=ON
> cmake --build .
>
> And it gives this:
>
> [ 20%] Building CXX object CMakeFiles/CoolProp.dir/src/AbstractState.cpp.o
> In file included from /home/johndoe/CoolProp/src/Backends/IF97/IF97Backend.h:6:0,
>                  from /home/johndoe/CoolProp/src/AbstractState.cpp:17:
> /home/johndoe/CoolProp/externals/IF97/IF97.h:316:5: error: braces around scalar initializer for type ‘double’
>      };
>      ^
> /home/johndoe/CoolProp/externals/IF97/IF97.h: In function ‘double IF97::Region3Backwards::Region3_v_TP(char, double, double)’:
> /home/johndoe/CoolProp/externals/IF97/IF97.h:1517:83: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>                      throw std::out_of_range::out_of_range("Unable to match region");
>                                                                                ^
> /home/johndoe/CoolProp/externals/IF97/IF97.h:1517:83: note: for a function-style cast, remove the redundant ‘::out_of_range’
> /home/johndoe/CoolProp/externals/IF97/IF97.h: In function ‘double IF97::Region3Backwards::DividingLine(IF97::Region3Backwards::DividingLineEnum, double)’:
> /home/johndoe/CoolProp/externals/IF97/IF97.h:1713:90: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>                      throw std::out_of_range::out_of_range("Unable to match dividing line");
>                                                                                ^
> /home/johndoe/CoolProp/externals/IF97/IF97.h:1713:90: note: for a function-style cast, remove the redundant ‘::out_of_range’
> /home/johndoe/CoolProp/externals/IF97/IF97.h: In function ‘char IF97::Region3Backwards::BackwardsRegion3SubRegionDetermination(double, double)’:
> /home/johndoe/CoolProp/externals/IF97/IF97.h:1720:68: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>                 throw std::out_of_range::out_of_range("Out of range");
>                                                                     ^
> /home/johndoe/CoolProp/externals/IF97/IF97.h:1720:68: note: for a function-style cast, remove the redundant ‘::out_of_range’
> /home/johndoe/CoolProp/externals/IF97/IF97.h: In function ‘char IF97::Region3Backwards::BackwardsRegion3RegionDetermination(double, double)’:
> /home/johndoe/CoolProp/externals/IF97/IF97.h:1764:78: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>                  throw std::out_of_range::out_of_range("pressure out of range");
>                                                                               ^
> /home/johndoe/CoolProp/externals/IF97/IF97.h:1764:78: note: for a function-style cast, remove the redundant ‘::out_of_range’
> /home/johndoe/CoolProp/externals/IF97/IF97.h: In member function ‘double IF97::Region3::output(I97parameters, double, double)’:
> /home/johndoe/CoolProp/externals/IF97/IF97.h:1977:74: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>                  throw std::out_of_range::out_of_range("bad key to output");
>                                                                           ^
> /home/johndoe/CoolProp/externals/IF97/IF97.h:1977:74: note: for a function-style cast, remove the redundant ‘::out_of_range’
> /home/johndoe/CoolProp/externals/IF97/IF97.h: In function ‘IF97::IF97REGIONS IF97::RegionDetermination_TP(double, double)’:
> /home/johndoe/CoolProp/externals/IF97/IF97.h:2075:65: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>              throw std::out_of_range::out_of_range("Out of range");
>                                                                  ^
> /home/johndoe/CoolProp/externals/IF97/IF97.h:2075:65: note: for a function-style cast, remove the redundant ‘::out_of_range’
> /home/johndoe/CoolProp/externals/IF97/IF97.h:2082:69: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>                  throw std::out_of_range::out_of_range("Out of range");
>                                                                      ^
> /home/johndoe/CoolProp/externals/IF97/IF97.h:2082:69: note: for a function-style cast, remove the redundant ‘::out_of_range’
> /home/johndoe/CoolProp/externals/IF97/IF97.h:2087:69: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>                  throw std::out_of_range::out_of_range("Out of range");
>                                                                      ^
> /home/johndoe/CoolProp/externals/IF97/IF97.h:2087:69: note: for a function-style cast, remove the redundant ‘::out_of_range’
> /home/johndoe/CoolProp/externals/IF97/IF97.h:2101:69: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>                  throw std::out_of_range::out_of_range("Out of range");
>                                                                      ^
> /home/johndoe/CoolProp/externals/IF97/IF97.h:2101:69: note: for a function-style cast, remove the redundant ‘::out_of_range’
> /home/johndoe/CoolProp/externals/IF97/IF97.h:2111:65: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>              throw std::out_of_range::out_of_range("Out of range");
>                                                                  ^
> /home/johndoe/CoolProp/externals/IF97/IF97.h:2111:65: note: for a function-style cast, remove the redundant ‘::out_of_range’
> CMakeFiles/CoolProp.dir/build.make:195: recipe for target 'CMakeFiles/CoolProp.dir/src/AbstractState.cpp.o' failed
> make[2]: *** [CMakeFiles/CoolProp.dir/src/AbstractState.cpp.o] Error 1
> CMakeFiles/Makefile2:63: recipe for target 'CMakeFiles/CoolProp.dir/all' failed
> make[1]: *** [CMakeFiles/CoolProp.dir/all] Error 2
> Makefile:116: recipe for target 'all' failed
> make: *** [all] Error 2
>
> I also tested to update submodules git submodule foreach git pull origin
> master, it update some externals modules but the error remain the same.
>
> Did I miss anything?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CoolProp/CoolProp/commit/48fe5b972f3b2b90b4f21d46f7fc166af8c506a1#commitcomment-10515765>
> .
>


that is not what i mean...
I mean it return dif value so Trees should be remove Obsolete


Line is too long. [132/100]


@Glenpl Druga opcja to callbacka przypisać do zmiennej (w JS funkcja nazwana i funkcja anonimowa przypisana do zmiennej wyglądają w użyciu identycznie, ale różnica de facto jest znaczna):

```
var anonFuncInVariable = function () {
    // tutaj jakiś kod, który wykona się po upłycięciu czasu
    // jest on w nazwanej funkcji
};

// odwołanie do samej nazwy, bez odpalenia funkcji,
// dzięki czemu nie kopiuj–wklejkujemy
setTimeout(anonFuncInVariable, 1000);

// gdzieś dalej w kodzie „prawie” to samo
setTimeout(anonFuncInVariable, 2222);
```


Fully trust your feeling on 'more clear';  I myself am undecided what feels clearer


@elfring I agree with you.  So far I have been treating this project as a proof of concept (PoC) but probably should follow proper development practice given there is more interest from the community.


:+1:

хоча як колишній його студент, то рахую за необхідне поставити +:100:


@Glenpl Lambdy wymiatają, też się cieszyłem jak „wreszcie” wsadzili je do PHP :D


Sim, aqui deveria ser "^([a-zA-Z]+)$".


Wut?
Create ```address_list```, then free it immediately?


I've just tested and the LIST_DATE_FORMAT error I described is exclusively caused by case sensitivity (the API guide - even the most recent one Winter '15 - specifies the format as all lowercase mm/dd/yyyy) - it is correct in the implementation above.

I've updated my original comment and will get a fork with the date formats set for the API calls that are implemented.

Thanks for all the great work (:


reference the other user value


this feature description appears to be wrong


some reasons for it to work only in PhpStorm 9 (that's not even released) and not PhpStorm 8 (which 1.0.0.alpha#201 supported) ?


Not sure but that doesn't look intentional ;)


Is this a typo?


In doubt do not change the code : )


r+


@lucasart  Is it really so hard for you to have some respect for people that actually have ideas that contribute elo to Stockfish? Recall that it took many years before someone finally succeeded in getting some elo out of the easy move concept. For me this is far more important than the fact that his code may contain a line that can perhaps be removed.


Mir fällt grade der Namenskonflikt mit data-on-color/data-off-color auf. Wo sind die Attribute eigentlich definiert? Gibt es evtl. bereits allgemeingültige Attribute für Hintergrundfarben?


The problem is solved, you cant highligt any header-title!


Sure Sir.
I have removed the .pillar~ file. Yep , committed it accidently.
Will take care in future.


This doesn't actually toggle the item state. How about:

```js
var completed = typeof checked === 'boolean' ? checked : !todo.completed;
return Utils.extend({}, todo, {completed: completed});
```


however, I started testing a first oo-version, began it some weeks ago...


saw approval from seancribbs
at https://github.com/basho/riak_ensemble/commit/c5063aaa20f397b1de0918110f96350f9a190fed


merging basho/riak_ensemble/rebar/fd/eleveldb-dependency = c5063aaa into borshop-integration-62-rebar/fd/eleveldb-dependency


basho/riak_ensemble/rebar/fd/eleveldb-dependency = c5063aaa merged ok, testing candidate = a69f4842


some tests failed:
failure: http://buildbot.bos1/builders/test-riak_kv/builds/1984


Rename to `an_attribute` or something a bit more descriptive (`:arr` is a bit confusing for me).


@Glenpl BTW, ogarniasz jeszcze jakieś języki (w takim stopniu, żeby móc czytać ich kod) oprócz cpp?

Bo wcześniej dawałem Ci przykład w PHP, teraz JS – może wygodniej by Ci było w ruby? Sam C* ogarniam raczej średnio, na codzień piszę appki webowe, stąd przykłady w językach używanych najcześniej do webu ;)


This breaks all words letter-by-letter, should be word-break: keep-all. See also: http://caniuse.com/#feat=css-hyphens


Here we need a "+1" to get the ceiling. Otherwise, it works.

Thanks for the quick fix (:


The tests should clear local storage before each test, otherwise running subsequent tests in the same browser (like refreshing an instance of Intern's browser client) will just keep adding items to the todo stores.

```js
beforeEach: function () {
	localStorage.clear();
}
```


This change was done in order to allow easier maintenance of the plugin and an easier release model due to changes in the Platform. You can use IntelliJ Community, which is free, meanwhile.


I believe you can add the following and replace the code below line 16

```ruby
context 'validation scenarios' do
  subject { described_class.new }
  [
    { values: [], valid?: false } ,
    { values: [''], valid?: false } ,
    { values: nil, valid?: false },
    { values: '', valid?: false },
    { values: ['work', ''], valid?: false },
    { values: ['work', nil], valid?: false },
    { values: ['work'], valid?: true },
    { values: 'work', valid?: true }
  ].each do |scenario|
    it "will validate #{scenario.fetch(:values).inspect} as valid? == #{scenario.fetch(:valid?)}" do
      subject.arr = scenario.fetch(:values)
      expect(subject.valid?).to eq(scenario.fetch(:valid?))
    end
  end
end
```


This commit is April Fool's joke :wink:


The reason is changed API. We don't want developing two separate versions of plugin for 139.* builds (PhpStorm 8) and 141.* builds (PhpStorm 9). So until the go-plugin 1.0.0 release we will target it on the latest stable platform version which is 141 for now.


done


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1242
success: http://build.servo.org/builders/linux2/builds/1240
success: http://build.servo.org/builders/android/builds/1276
success: http://build.servo.org/builders/mac1/builds/1455
success: http://build.servo.org/builders/mac2/builds/1458
success: http://build.servo.org/builders/gonk/builds/421


I had just copied your code :)
Can you create a PR?


what?


This commit, although following the RFC 2616, can cause trouble in a real word scenario with poor designed servers. For instance, I can't fetch this image anymore:

```
curl -v http://www.supercars.net/gallery/119513/3020/1072343-1024.jpg
* Hostname was NOT found in DNS cache
*   Trying 174.37.136.130...
* Connected to www.supercars.net (174.37.136.130) port 80 (#0)
> GET /gallery/119513/3020/1072343-1024.jpg HTTP/1.1
> User-Agent: curl/7.37.1
> Host: www.supercars.net
> Accept: */*
>
< HTTP/1.1 200 OK
< Date: Wed, 01 Apr 2015 12:50:32 GMT
* Server Orion/2.0.7 is not blacklisted
< Server: Orion/2.0.7
< Last-Modified: Sun, 25 Aug 2013 20:42:35 GMT
< Accept-Ranges: bytes
< Content-Length: 424995
< Connection: Close
< Content-Type: image/jpeg
< Transfer-Encoding: chunked
```
The `http-client` rises an error `ResponseLengthAndChunkingBothUsed`. Would be possible just to ignore `Content-Length` in the presence of  `Transfer-Encoding: chunked` and treat it as `chunked`? Is there any workaround to retrieve the content?


Needs `|add_class:"form-control"`?


You can't use a colon in values.


master (kbenne) - x86_64-MacOS-10.9-clang: OK (480 of 481 tests passed)

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


1: M111 unrecognized component name: OAIRepository


Lol, you asshole


I just added these two lines for supporting NSData when sharing something.

I used it for printing a PDF file;
```C#
NSData data = new NSData(Application.streamingAssetsPath + "/" + pdfFilename);
SocialXT.Share(new object[] { data });
```


BTW, you can use [PhpStorm EAP](https://confluence.jetbrains.com/display/PhpStorm/PhpStorm+Early+Access+Program), it's also free.


```objc/deprecated/PromiseKit+#{framework}.h``` not allow me to use cocoapods ```use_frameworks!```. Please remove it if not needed.


I think this might broke apt sources addon. After this commit it checks `config[:sources]` for sources, but there is no place where `config[:sources]` is set. In .travis.yml it's added as `apt_sources` and previously it was set up here: https://github.com/travis-ci/travis-build/commit/2b04f92530e5c5108ffec636c7ea37d03f275675#diff-2f9482e29687eb9bce2951201c32b9cfL11


PS: also I apologize for being rude. I can be extremely rude when I'm grumpy - it's a bad personality trait, don't take it too seriously. (I personally don't think saying "fuck you" to someone over the internet is the worst offense.) Things like this really shouldn't be a reason, should they?

And while I'm annoyed about all the dozillion corner cases the color management stuff and some of the placebo crap creates (and over which I long lost the overview, especially after the vo_opengl rewrite), let me remind you that it was me who implement the initial colormanagement, the high quality scalers (except ewa and antiringing), dithering, and correct chroma location. I also didn't try to prevent pigoz from implementing smoothmotion, but very much encouraged it.

Now it's haasn's turn. If you ask me, it's not necessary to merge all of haasn's work - just some of the popular bits like the pending debanding implementation - so cooperation should be in his interest too, maybe. (Or maybe he doesn't care in his attempt to split the project in two. I don't care either, even if he succeeds. I now have very specific reasons to work on mpv, and it's not because of users of placebo features.)


The joystick event is firing and it is returning some values but I am not yet sure that they correlate to how I tilt my iPad.  I'll investigate more tonight.


Yes, good catch


fast-forwarding master to auto = 6824bc93


We could also raise here. Any thoughts?


Hey @KernCheh! Thanks for the advice! :) Hmm, but would it still be dangerous even if there are no input fields but only a button to execute the POST request?


Perfect!


excellent change


Oh, and I don't see the mouse events.  I thought they might happen for touch.


I am not sure on the correct terminology, but the tutorial refers to "project directory" as the one containing `manage.py`.


Yes I think so; sorry.


I'm OK with changing the behavior in that way. Did you want to take a crack at a PR so you can confirm that it works for your use case?


this comment belongs to `BASE_DIR`.


ping @vforge ping @Warkot


Here it is, good.


Back in business I think

On Wed, Apr 1, 2015 at 8:30 AM, Ian Bell <ian.h.bell@gmail.com> wrote:

> Nope, I left the master in a broken state when I went to sleep.  I'm
> fixing it right now, another 10 minutes and all should be well again.
>
> On Wed, Apr 1, 2015 at 3:49 AM, JonWel <notifications@github.com> wrote:
>
>> It doesn't compile for me on Ubuntu 64 bit.
>>
>> To be sure not having any submodules issues, I deleted the previous
>> directory and downloaded all freshly:
>>
>> git clone https://github.com/CoolProp/CoolProp --recursivecd CoolProp
>> mkdir build && cd build
>> cmake .. -DCOOLPROP_64BIT_SHARED_LIBRARY=ON
>> cmake --build .
>>
>> And it gives this:
>>
>> [ 20%] Building CXX object CMakeFiles/CoolProp.dir/src/AbstractState.cpp.o
>> In file included from /home/johndoe/CoolProp/src/Backends/IF97/IF97Backend.h:6:0,
>>                  from /home/johndoe/CoolProp/src/AbstractState.cpp:17:
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:316:5: error: braces around scalar initializer for type ‘double’
>>      };
>>      ^
>> /home/johndoe/CoolProp/externals/IF97/IF97.h: In function ‘double IF97::Region3Backwards::Region3_v_TP(char, double, double)’:
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:1517:83: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>>                      throw std::out_of_range::out_of_range("Unable to match region");
>>                                                                                ^
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:1517:83: note: for a function-style cast, remove the redundant ‘::out_of_range’
>> /home/johndoe/CoolProp/externals/IF97/IF97.h: In function ‘double IF97::Region3Backwards::DividingLine(IF97::Region3Backwards::DividingLineEnum, double)’:
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:1713:90: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>>                      throw std::out_of_range::out_of_range("Unable to match dividing line");
>>                                                                                ^
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:1713:90: note: for a function-style cast, remove the redundant ‘::out_of_range’
>> /home/johndoe/CoolProp/externals/IF97/IF97.h: In function ‘char IF97::Region3Backwards::BackwardsRegion3SubRegionDetermination(double, double)’:
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:1720:68: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>>                 throw std::out_of_range::out_of_range("Out of range");
>>                                                                     ^
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:1720:68: note: for a function-style cast, remove the redundant ‘::out_of_range’
>> /home/johndoe/CoolProp/externals/IF97/IF97.h: In function ‘char IF97::Region3Backwards::BackwardsRegion3RegionDetermination(double, double)’:
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:1764:78: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>>                  throw std::out_of_range::out_of_range("pressure out of range");
>>                                                                               ^
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:1764:78: note: for a function-style cast, remove the redundant ‘::out_of_range’
>> /home/johndoe/CoolProp/externals/IF97/IF97.h: In member function ‘double IF97::Region3::output(I97parameters, double, double)’:
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:1977:74: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>>                  throw std::out_of_range::out_of_range("bad key to output");
>>                                                                           ^
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:1977:74: note: for a function-style cast, remove the redundant ‘::out_of_range’
>> /home/johndoe/CoolProp/externals/IF97/IF97.h: In function ‘IF97::IF97REGIONS IF97::RegionDetermination_TP(double, double)’:
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:2075:65: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>>              throw std::out_of_range::out_of_range("Out of range");
>>                                                                  ^
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:2075:65: note: for a function-style cast, remove the redundant ‘::out_of_range’
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:2082:69: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>>                  throw std::out_of_range::out_of_range("Out of range");
>>                                                                      ^
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:2082:69: note: for a function-style cast, remove the redundant ‘::out_of_range’
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:2087:69: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>>                  throw std::out_of_range::out_of_range("Out of range");
>>                                                                      ^
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:2087:69: note: for a function-style cast, remove the redundant ‘::out_of_range’
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:2101:69: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>>                  throw std::out_of_range::out_of_range("Out of range");
>>                                                                      ^
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:2101:69: note: for a function-style cast, remove the redundant ‘::out_of_range’
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:2111:65: error: cannot call constructor ‘std::out_of_range::out_of_range’ directly [-fpermissive]
>>              throw std::out_of_range::out_of_range("Out of range");
>>                                                                  ^
>> /home/johndoe/CoolProp/externals/IF97/IF97.h:2111:65: note: for a function-style cast, remove the redundant ‘::out_of_range’
>> CMakeFiles/CoolProp.dir/build.make:195: recipe for target 'CMakeFiles/CoolProp.dir/src/AbstractState.cpp.o' failed
>> make[2]: *** [CMakeFiles/CoolProp.dir/src/AbstractState.cpp.o] Error 1
>> CMakeFiles/Makefile2:63: recipe for target 'CMakeFiles/CoolProp.dir/all' failed
>> make[1]: *** [CMakeFiles/CoolProp.dir/all] Error 2
>> Makefile:116: recipe for target 'all' failed
>> make: *** [all] Error 2
>>
>> I also tested to update submodules git submodule foreach git pull origin
>> master, it update some externals modules but the error remain the same.
>>
>> Did I miss anything?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/CoolProp/CoolProp/commit/48fe5b972f3b2b90b4f21d46f7fc166af8c506a1#commitcomment-10515765>
>> .
>>
>
>


This will break if you define a different $PREFIX directory for installation than /usr.

Have a look at the file org.knopwob.dunst.service.in and the service target in the makefile for a way to work around this.


Is it possible to have the string here specify Stay Armed or Away Armed?


This looks good to me!  I think @anarchivist and/or @AudreyAltman should give a :+1: before merge, since they both know this app better than I do.


@MacDada tak żeby czytać to nie ma problemu, ale mogą byc problemy z bardziej skomplikowanymi konstrukcjami językowymi czy też jakimiś rzeczami które są nieintuicyjnie odmienne od innych języków. Bez problemu oprócz rodziny c mogę czytać kod pythona.


On that line, you introduced some extra indentation that's not consistent with the rest of the file (Bloc's formatting is not so good, for it can make you feel like you need some extra spaces, when actually you don't).


This script is one of my 'dirty' ones that is meant to run on the cron, so this probably isn't a file we want others calling into. This is a function that needs to exist on both the client and the server, I googled around to try to find what other people do in a case like this and couldn't come up with much. I ended up just copy/pasting the function into both but now I'm wondering maybe grunt could help us out with the copy/paste.

As for what to do in the short term, maybe drop this function into the Wagers model with a comment that it is also used on the client and we need to figure out what to do long term? If you actually want to try to figure out the grunt step to do the copy I am all for that as well but I'm not sure how that is going to turn out, it's just an idea.


Shouldn't this be "GeoKret" instead?


Shouldn't this be "GeoKrets" instead?


saw approval from jdm
at https://github.com/jdm/servo/commit/1baf2effd78a5d8b646c13bb29c1f5531a8dfe95


merging jdm/servo/advance = 1baf2eff into auto


jdm/servo/advance = 1baf2eff merged ok, testing candidate = 455fb21b


Good catch!


@itewk `(x || y)` returns the first element which is not "falsy", which in ruby is `nil` or `false`. In this case, it is an easy way to specify a default value.


I found a better way to do this not too long ago, you can use $stateParams to get the groupId.


yes, that was the reason to add it, though only partially related.


Nice, I just ran into this today.  Thanks for the fix.


Yes, it compiles now.


My intent was to remove the `apt_sources` addon in favor of `apt.sources`.  I left a shim in place for `apt_packages` because it's been in the wild since January, whereas `apt_sources` landed last week.


Actually the filter version would end up using 5 lines except of 6! It would be shorter. The list comprehension syntax is clear, but not common in usage, and can be confusing to readers less used to Erlang. When I add that kind of stuff, I usually end up writing a comment explaining why it's done that way to avoid confusion. If the explanation makes it longer than the other 'boring' manner, then I switch to the boring one :)


all tests pass:
success: http://buildbot.bos1/builders/test-riak_ensemble/builds/172


If there are 10 wagers on games that haven't started yet you will end up returning nothing right?


I think the `Module` instance should also have value object semantics. The anima generated ones have.

If you write this class like:

```ruby
class Attributs < Module
  include Concord;;Public.new(:names, :defaults)

  def self.new(*attrs)
    defaults   = attrs.last.instance_of?(Hash) ? attrs.pop : {}
    names = (attrs + defaults.keys).uniq
    super(names, defaults)
  end

  # ...
end
```

You end up with the same result, but reduce the need to write to ivars directly (can easily contain a typo gone by unnoticed), and you get `Attribs.new(:foo) = Attribs.new(:foo)`.

Also I'd consider to freeze the module after it was build. To reduce variance introduced by faulty 3rd party libs that try to monkey patch around.

`concord` is AFAIK a dependency of anima, so you do not pull a new one.


I am experiencing a regression here concerning issue [1729](https://github.com/driftyco/ionic/issues/1729). On Android 4.4.2 drag gestures will be canceled immediately, unless I pass `{prevent_default_directions = ['left', 'right']}` as fourth argument to `$ionicGesture.on`. Although I don't use native scrolling.

I think `preventDefault();` should be the default, if `'overflow-scroll'` isn't present? With the current behavior it breaks drag gestures when using Ionic scroll.


I personally think you should almost always use the attr_reader and not the ivars. Unless there is a benchmark that tells you the ivar access is substantially faster. Rubies behavior to accept typoed ivars and defaulting their read to `nil` is very annoying and you remove the full bug class with that.

This comment applies to all other ivar reads.


@katcipis I created the store.Cache interface that is used by engine to cache the leveldb instances open.
For now, exists only the LRUCache implementation and then they are on the same file. In the future we can split this into cache.go, cache_lru.go and so on, and use go build -tags lru OR go build -tags other_cache for performance tests.


Nope, I'll return all 10, you just won't be able to see the game that was bet on (you'll still see who did the bet and how much). That logic is handled down on line 173.


Not par of my PR, we will address it later.


Gotcha. I'll revert this and dig around a bit to see what options there are for making a file generic between node and browser.


Anima provides `Anima::Attribute#get` you could use to remove the need to compute the ivar name per call to `append_to`. As the ivar name is cached internally this will produce less garbage.


Yeah, I think your form doing an explicit mapping of `modified -> true` is good enough. I'm not a fan of using the `A andalso side_effects()` form because I tend to think of `andalso` as guard syntax, but I certainly don't dislike it as much as the list comprehension stuff.

Though with a boolean it's easy to use
```erlang
if Modified -> store_erlcinfo(G, InclDirs);
   not Modified -> ok
end
```
Making it explicit we're going for side-effects only. The `andalso` form only works because digraph operations are destructive.


What is the status of this? If it is no longer needed please remove it entirely.


Ha-ha! Maintained? Perhaps... Developed? I don't think so...


Why the 1 at the end?


In case one of your attributes *without* defaults is intentionally `nil` it'll not be included in the resulting hash, is that intent?


Oh. Cool! That's awesome.


int OR null


TeamCity Azmq :: Build (Linux) [Build 82](https://78.46.71.115/viewLog.html?buildId=441&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (Windows) [Build 108](https://78.46.71.115/viewLog.html?buildId=442&buildTypeId=Azmq_Build&guest=1) is now running


TeamCity Azmq :: Build (OSX) [Build 66](https://78.46.71.115/viewLog.html?buildId=443&buildTypeId=Azmq_BuildOsx&guest=1) is now running


TeamCity Toolbox :: Pull requests [Build 42](http://10.48.34.86:8111/viewLog.html?buildId=7909&buildTypeId=Toolbox_PullRequests) is now running


Yes, for the demo after retrieving the document we are adding the creation date from/to filter.


TeamCity Toolbox :: Pull requests [Build 42](http://10.48.34.86:8111/viewLog.html?buildId=7909&buildTypeId=Toolbox_PullRequests) outcome was **SUCCESS**
Summary: Tests passed: 165 Build time: 00:00:18


You read `self.class.attributes.defaults` two times in this class, it might be helpful to move that to an private instance method to deduplicate.


cc @Resseguie


@meatballhat so it should work if it's specified as `apt: { sources: [ ... ] }`?


Will change to remove developed.

On Apr 1, 2015, at 10:54 AM, Dmitri Smirnov <notifications@github.com> wrote:

> Ha-ha! Maintained? Perhaps... Developed? I don't think so...
>
> —
> Reply to this email directly or view it on GitHub.
>


Too many `=`.  The other READMEs make these match.


These have always been missing the extra ws.


The literal `50` is present two times this method, I think its better moved to a constant, that also gives the magic number a name.


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


If we move the label outside .globalsearch, then the input will be first element and will get border-radius.
Not sure if the move is valid. Wdyt?


Reviewed and good with the PR merge.


@Resseguie we should add this signature to Johnny-Five's `Led.RGB.prototype.color(...)`


`value.map(&format)` gets called for both branches, I think it should be computed once just before the if statement, reducing duplication.


Sorry about that, for the next reader, it does 3 things:

* Move the first 4 attributes of AndroidProject to AppProject.
* Create a constructor for AppProject from what was in ModelBuilder::android_project_for.
* Move the redef of AAnnotation to the same module as AppProject.
* Rename the annotation `java_package` to `app_namespace`.

I didn't use more than 1 commit because it would have meant a lot of code rewrites in the same PR, but I'll probably go for that option next time.


TeamCity Azmq :: Build (OSX) [Build 66](https://78.46.71.115/viewLog.html?buildId=443&buildTypeId=Azmq_BuildOsx&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:02:54


TeamCity Azmq :: Build (Windows) [Build 108](https://78.46.71.115/viewLog.html?buildId=442&buildTypeId=Azmq_Build&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:02:57


Needs a copyright header


TeamCity Azmq :: Build (Linux) [Build 82](https://78.46.71.115/viewLog.html?buildId=441&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:03:22


Since we have more developers now it would make sense to give credits to all of them


this was not a minor fix, api would not return more than 100 members of a groups cause of this... took me two days to figure out


not following this change... shouldn't statusStore be a copy of store

```
store.UdpateStrategy = persistentvolume.Strategy
...
statusStore := *store
statusStore.UpdatesStategy = persistentVolume.StatusStrategy
```
I assume we let user do persistent volume updates to modify object meta or any future modifiable fields in spec, and that persistentVolume.StatusStrategy should just be status updates?


Also we could remove the 'onfocus' and replace 'value' with 'placeholder'.


Also the default value being returned by `Hash#[]` might not be in your control, as the original `defaults` Hash is constructed at the call side, it might make sense to use `defaults.fetch(attr, nil))` to overcome this.

This is more a design decision, not something I'd flag.


similar comment to the other area... its confusing to have a normal update strategy for a non-status store to use a status strategy


Sorry for raking up this commit.. I was browsing the code and it was odd..
Why is the method signature is like this,
```java
public <E extends Model>  LazyList<E> limit(long limit)
```
instead of
```java
public LazyList<T> limit(long limit)
```
is this something related to the *fluency* of the API ?


@joshnuss This commit breaks the dropdown module created for the variant selection. The list of variants is no longer being populated (see attached).

![screen shot 2015-04-01 at 11 00 33 am](https://cloud.githubusercontent.com/assets/865198/6943807/5fd3026c-d85e-11e4-8665-5065fd74abd3.png)


no, the correct option could have been to add this `it does 3 things` in
the description on the commit

On Wed, Apr 1, 2015 at 9:58 PM, Alexis Laferrière <notifications@github.com>
wrote:

> Sorry about that, for the next reader, it does 3 things:
>
>    - Move the first 4 attributes of AndroidProject to AppProject.
>    - Create a constructor for AppProject from what was in
>    ModelBuilder::android_project_for.
>    - Move the redef of AAnnotation to the same module as AppProject.
>    - Rename the annotation java_package to app_namespace.
>
> I didn't use more than 1 commit because it would have meant a lot of code
> rewrites in the same PR, but I'll probably go for that option next time.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xymus/nit/commit/169436242ced86587d4227a23774fe3d520ab179#commitcomment-10525512>
> .
>


Huh, funny, I read that as an L. Oops.


fast-forwarding develop to borshop-integration-62-rebar/fd/eleveldb-dependency = a69f4842


yep!  I verified before merging to master.  Is it not working?


TeamCity Toolbox :: Pull requests [Build 43](http://10.48.34.86:8111/viewLog.html?buildId=7911&buildTypeId=Toolbox_PullRequests) is now running


If you want to merge, the file to be translated is `src/i18n/en.json`, not `dist/i18n/en.js` (see CONTRIBUTING.md)
Thank you


TeamCity Toolbox :: Pull requests [Build 43](http://10.48.34.86:8111/viewLog.html?buildId=7911&buildTypeId=Toolbox_PullRequests) outcome was **SUCCESS**
Summary: Tests passed: 165 Build time: 00:00:17


Good point. I definitely will add Dmitiry Kalinkin. DimaK, do you have a preferred link to associate with your name?

On Apr 1, 2015, at 10:58 AM, Dmitri Smirnov <notifications@github.com> wrote:

> Since we have more developers now it would make sense to give credits to all of them
>
> —
> Reply to this email directly or view it on GitHub.
>


@plexus You did *not* asked for a review. But I was just curios when I spotted this repo.

I than was triggered by the commit message:

> First, and possibly final, version

And thought: Hey, this is probably mutation covered, so lets see if I can find something I'd flag on a commercial code review, even after mutation testing. And I just did for fun. Feel free to ignore *all* my noise here. Or ping me back on the flags I did in case you want a discussion.


Raising or not ... This is the wrong test. In principle, nothing forbids log tasks to have a model, but the configuration application would fail because it cannot write properties.


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


@plexus Also I tried to limit myself to semantic, not stylistic flags. In case you also want to have the style flags: Just ping me.


Looking at the name of your branch "activehdl" I suspect you want to add support for aldec active hdl.
There is already a fork at https://github.com/oesvends/vunit where oesvends has implemented support for aldec riviera pro which should look the same as active hdl. We do plan to add aldec support to the mainline vunit repo but we are just waiting for aldec to give us tool provider licences so that we (the main authors) can maintain it. We are in the final stages of the process of obtaining such licenses.


correct, it is changed to sorted() in a later commit to make testing work consistently.


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


It's an 'L', to ensure that the default value is returned as a long rather than an int. This was causing some crashes for me on untextured default-color models.


With your changes, the #ifdef on line 285 of the unit test should be removed.


@meatballhat it works, sorry, I just thought that the older version should still work, all fine!


Does this enable automoderator by default on all sub reddits ?


@mbj thanks for the review Markus <3 I wasn't sure if I had done anything in particular to upset you :stuck_out_tongue:

I pulled this out of Yaks because I wanted to use it elsewhere, and yes it's 100% mutation covered, obviously, so I wasn't expecting to do much on it any time soon.

Good review though, you raise some valid points, I might look into them. I'm flattered that you take the time to comment, as it means you still believe I'm able to do better :)


Yeah, I think they made a mistake in overloading {}. Should have left sets as set().


You said that plugin do 2 shades: 1) for docker-java library and it works, 2) for guava.  If you see any class/guava issues that mean that shading doesn't work. And seems it doesn't work, because of hpi plugin ignore execution, i'm debugging it. You can see executed mojos here https://jenkins.ci.cloudbees.com/job/plugins/job/docker-plugin/202/com.nirima$docker-plugin/executedMojos
No shade, but it defined in pom.
So instead of checking why it doesn't work, you just changed classes to shaded, that eventually matched to shaded from docker-java-shaded (1). I got some advices from @jglick and will try proposed hack to do .jar packaging with maven-jar-plugin that works fine with shade-plugin.

Without any review of commits you just hiding real issues. I very hate dealing with fixing/debugging post-factum.


@MacDada w c++ da się zrealizować oba przykłady które pokazałeś, ale w pierwszym przypadku trzeba paprać się z obrzydliwymi wskaźnikami do funkcji (jeśli by miała być użyta w tym settimeout, jeśli jak zwykła funkcja to nie trzeba). Problemem jest to, że to nie są dokładne kopiuj-wklejki, zależnie od sytuacji zwracane wartości są odwrócone i podobne lambdy są 4, ale tylko 2 są takie same. Różnią się zarówno rodzajem porównania, jak i zwracanymi wartościami, więc żeby wszystkie działały prawidłowo musiałbym napisać funkcję interpretującą kilka argumentów... jednolinijkowe lambdy z jedną instrukcją return i operatorem trójargumentowych są tutaj najlepszym rozwiązaniem.


Good morning (at least it is here in California).   I fixed my problem.  This morning I manually cleaned all Fann files from my Pi and started over.  I reloaded FANN-2.2.0 and started pulling errors on make runtest.   Somehow the links to the .so files were not being properly set up by the compilation process for FANN.  I ran ldconfig which fixed the problem with make runtest.  Then got the fann2 bindings using pip and installed.   It works!!!

Thanks for your help.

pi@raspberrypi ~/Fann $ sudo pip install fann2
Downloading/unpacking fann2
  Downloading fann2-1.0.7.tar.gz (63Kb): 63Kb downloaded
  Running setup.py egg_info for package fann2

Installing collected packages: fann2
  Running setup.py install for fann2

    building 'fann2._libfann' extension
    gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes -fPIC -DSWIG_COMPILE -I./include -I../include -Iinclude -I/usr/include/python2.7 -c fann2/fann2_wrap.cxx -o build/temp.linux-armv7l-2.7/fann2/fann2_wrap.o
    cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for Ada/C/ObjC but not for C++ [enabled by default]
    fann2/fann2_wrap.cxx: In function ‘void init_libfann()’:
    fann2/fann2_wrap.cxx:11294:21: warning: variable ‘md’ set but not used [-Wunused-but-set-variable]
    g++ -pthread -shared -Wl,-O1 -Wl,-Bsymbolic-functions -Wl,-z,relro build/temp.linux-armv7l-2.7/fann2/fann2_wrap.o -ldoublefann -o build/lib.linux-armv7l-2.7/fann2/_libfann.so
Successfully installed fann2
Cleaning up...
pi@raspberrypi ~/Fann $ cd /
pi@raspberrypi / $ python
Python 2.7.3 (default, Mar 18 2014, 05:13:23)
[GCC 4.6.3] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> from fann2 import libfann
>>> ann = libfann.neural_net()
>>> train_data = libfann.training_data()
>>>

From: tty02-fl
Sent: Tuesday, March 31, 2015 4:30 PM
To: FutureLinkCorporation/fann2
Cc: jfperez
Subject: Re: [fann2] fixing issue #6 -- bumping version (beb24b3)

Also you do not need to create all the links, if you compile http://sourceforge.net/projects/fann/files/fann/2.2.0/FANN-2.2.0-Source.zip/download with one parameter it has to select where place the compiled libs.

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


직접 파이썬3에 호환 가능하게 패치한 것입니까? 대단합니다!


So when non-player entities enter a portal that goes to an unloaded world, what happens to that entity? Is the Entity simply not teleported?


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


I don't think I have anything more to add.

On Wed, Apr 1, 2015 at 4:06 PM, Kanstantsin Shautsou <
notifications@github.com> wrote:

> You said that plugin do 2 shades: 1) for docker-java library and it works,
> 2) for guava. If you see any class/guava issues that mean that shading
> doesn't work. And seems it doesn't work, because of hpi plugin ignore
> execution, i'm debugging it. You can see executed mojos here
> https://jenkins.ci.cloudbees.com/job/plugins/job/docker-plugin/202/com.nirima$docker-plugin/executedMojos
> No shade, but it defined in pom.
> So instead of checking why it doesn't work, you just changed classes to
> shaded, that eventually matched to shaded from docker-java-shaded (1). I
> got some advices from @jglick <https://github.com/jglick> and will try
> proposed hack to do .jar packaging with maven-jar-plugin that works fine
> with shade-plugin.
>
> Without any review of commits you just hiding real issues. I very hate
> dealing with fixing/debugging post-factum.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jenkinsci/docker-plugin/commit/bdde16c512c9f72eabc209b7d369f258c987a7af#commitcomment-10525728>
> .
>


@jayeshka, would you mind renaming this variable to something like `HAS_UPSTART` instead of `FLAG`?


Hi,

Thank you for your feedback, can we please have a look to your template code ?

Best regards.


Some errors may occur if you guys rename the API Get10kURLwithCIK() in GetURL.java to GetURLwithCIK(), but I will look into it.


I mentioned this to @ulferts previously (see https://github.com/opf/openproject/pull/2455#commitcomment-9505251), but we forgot to change this.


Out of interest, which types failed? I think it might be better just to do it unconditionally, and suppress the exception if  the type doesn't get auto-converter - will that work?


The value BITCOIN_SCHEME should be changed instead.


Something I should mention here is that RubyMotion didn't support subclassing `Module` a year ago (maybe that changed) so it might be worth checking that and reconsidering using this technique as libraries like this one could be super handy in RubyMotion-land.


Agree. Three arg and object.
https://github.com/rwaldron/johnny-five/issues/681


You are welcome! It's really difficult to test every combination of input options. So some problems will be fixed after the bug report. I made the fix after reading [the question](http://stackoverflow.com/q/29385548/315935). So if you would find more problems, then just post the issue here (it's better to include the demo which reproduce the problem) or write the question on the stackoverflow with the tags "jqgrid" and "free-jqgrid". Typically I create the fix very quickly.

Best regards
Oleg


I dont get this line
why do u need it?


It sounds good. I'll let you know when I have something.


Thanks!

On Wed, Apr 1, 2015 at 6:12 PM Edgar Gomes <notifications@github.com> wrote:

> It sounds good. I'll let you know when I have something.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/snoyberg/http-client/commit/ed0b0b71f7bdd1a769a864bedd73c059cc1676e4#commitcomment-10525843>
> .
>


The hole configuration stuff assumes that there is a model_name. Therefore, it should also test if there is really one.

If a log task has a model at some point it should raise when someone is trying to change a property. At the moment the proposed test acts as safety net.


:ship:


why do u use pytest11?


You're right. Fixed in 20e283b59a53d6e9099cf381161ccbb2351c641d.


Just wondering what your thoughts are here? Maybe that this might be useful for all PlugValueWidgets, so we could set this property in the base class, and do things like also style the text in text fields and so on? Or something else?


this is unintentional, will push a fix!


FYI
(it is not related to childify-me !!)
https://make.wordpress.org/core/2015/03/30/query-improvements-in-wp-4-2-orderby-and-meta_query/


In general use, wouldn't you usually put the scripts under the stylesheet?


This bit isn't equivalent.   It used to set both lengths to zero if xdr->x_base was NULL, even if pixdr->xdr_len was non-zero.  You probably need the check for len too.


Coolio.


@aravindc26, to make a long story short: yes. This is because Model is not parametrized (as `Model<Person>`) so it cannot propagate the type information to LazyList, so

    LazyList<Person> johns = Person.where("first_name = ?", "John").limit(10);

would not work without this twist in the code... Discussion can continue at #357.


So...this "just" adds default values on top of anima?


I dont like regexp at all, can we do it moar readable with https://docs.python.org/2/library/string.html#string-formatting ?


fix pushed!


That's an interesting point, since a lot of the more interesting micro libs subclass Module AFAIR, e.g. Concord, Anima, Equalizer, Procto (I think), ...

Do you have an example of that style of lib that does things in a different way?

So far this is only used for Yaks, and I don't expect to push it far beyond that, but I might use this in an upcoming "feature toggle" lib which might be handy e.g. for RubyMotion


some tests failed:
failure: http://build.servo.org/builders/linux1/builds/1243
failure: http://build.servo.org/builders/mac2/builds/1459


@solnic Plus pretty printing, collection attribute appends.


I left out the '#0' purposefully so that when posting quotes the bot will have a typical non-mobile client color.


> Something I should mention here is that RubyMotion didn't support subclassing Module a year ago (maybe that changed) so it might be worth checking that and reconsidering using this technique as libraries like this one could be super handy in RubyMotion-land.

Thats a good point. But actually: Lets spend the energy to fix ruby motion. Not working around its deficiencies.


@JessWallace94, that's right! I actually made this because our teacher made us do it this way.


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


But dbus should be reloaded before others, otherwise changes to configuration may break other units. That's why I moved dbus reload before other reloads.


Were these pulled in via transifex?  If Transifex is up to date, probably better someone with full commit accessto just pull them in directly via the Transifex client.


MarkdownDocs (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1674 of 1676 tests passed)

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


yeah the pretty printing is the main functional addition. When used throughout you can do `puts foo.pp` and have something you can paste straight into irb or in a unit test. Yaks uses this throughout.


If you move the MC jar to the output you can tell zip to write to that instead of the zip and it will just overwrite the files that we changed..


Usage in another template


Should be fixed.


ok, nice option btw using the regex.


I actually went with "yara_hits" originally, but since 0 hits will still put a row in the table I decided to go with just "yara". Maybe a better name is "yara_scans"?


Ah excellent, just wanted to make sure - it was odd for me to see it this way round. Are you planning to add some more snippets?


I dont get this thingy, why do we replace all predicates, such as ```and``` and ```or``` with empty string?


@MMelouk94
if you have any comments let me know


Maybe the whole case statement might be better moved into a private utility method (one that does not access instance state). This would make this much more easy to follow and review.


fair enough


First of all I need to check if maybe it is already supported. If it's not I may try to find an alternative solution. I was crying when I couldn't use Virtus and friends on my RM project last year ;)


Sure, I guess my thoughts were mainly around using an SSL cert with multi domains for an assets subdomain rather than for different envs


I would raise for now, then


More an overall impression... I would push it all into master basically and no add more changes here..


I think you will likely want to do something similar with the masks used?


Normally I will, but I noticed that Emmet replaced my need for most snippets


Thank you for your follow up. Which file do you want? I can send you a private email ftp access.

Best Regards.


But we can't distribute a modified .jar... (Actually distributing the modified classes is also something modders are not suppose to do.)


@ChalkPE 다다익선


One important point of the Parsec paper is to avoid this, and I wonder whether the same technique can be used to avoid it in bash.


Ah yeah, I forget about Emmet - it's been useful!


u can use ```pytest_runtest_makereport``` instead
that hook will performed right after particular test run


Because "CITEXT" is not a normal type in PostgreSQL. You have to install it as a separate extension, which we currently don't have installed on the production server (and we probably don't since it's harder to maintain and upgrade).


How can `currentAverage` ever become a floating point number when it's always calculated from two `long` values?


Почему ты юзаешь Mat в итоге, а не InutArray, OutputArray и тд,?


ooops ...

ta


그리고 커밋 이름은 다시 영어로 쓰여지고 있었던 것이었던 것이었던 것이다!


Вот тут почему Mat, а не InputArray, OutputArray?


@gormanb Could you make it upper-case so its more clear what the intention is?


Ah thanks for the clarification :+1:


MarkdownDocs (jasondegraw) - 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)


Hum.
Really surprising result.

So when it was added, it was certainly < 5 ELO.
But now it is much more, since everything else had been tuned around this ???.


I think you are right, a `?` should return `true` or `false`.


@cchalmers I'm going to check whether anyone plans to fix linear.  I don't think 7.4 support in Diagrams is important enough to do so myself.  See https://github.com/ekmett/linear/issues/71


This link is used by dependency manager as base url for artifacts. For example [link to .jar 1.0](
https://github.com/danikula/AndroidVideoCache/raw/mvn-repo/com/danikula/videocache/1.0/videocache-1.0.jar)


Looks like there are some small typos in this. Mind giving it a look over @jhonnymoreira? Cheers dude :smile:


doesn't this some what resolve https://github.com/Wurst-Imperium/Wurst-Client/issues/70?


duh! :)


Space after ,


No space after ( and single quote


Note: this should probably be changed to this / this scope instead of sidebar variable, so I'd consider this a hotfix right now.


This seems like a good idea.  My one issue is that not everyone is feeding photos so if you could wrap it in a a configatron option that is defaulted to false and add some documentation for the options I will merge it.


what if one have same ```test_method``` name in different modules?


Achtung, the `.cmxs` is missing :)


Adding option for proxy_ports_per_gear for nodes


This was a really nice comment


There are mixed tabs and spaces - please use two-space indentation here.


from line 135 onwards, the code is not placed correctly. it needs to be placed within <script> tags, and it needs to be within the <body> tags somewhere too, (or the <head> tags)


I’m basically just trying to make sure the code is predictable.  I added a few methods in this class that can be overridden by a sub-class, but they’re limited and have obvious return values.  If we allow someone to completely replace init() or render() in a stock page class, for example, we can no longer guarantee that certain properties will be correctly populated or passed along to the view script at all.

Ultimately, I’d like to avoid inheritance as a way of extending stock pages at all, really.  Instead, you’d compose an object that handled the logic and view assignments related to the page.  Make the stock page classes in the \Dewdrop\Admin\Page\Stock call this composed object so simply that it is obvious to someone hoping to extend the functionality how they can do so without inheriting from that stock page class itself.

> On Mar 30, 2015, at 8:31 AM, Darby Felton <notifications@github.com> wrote:
>
> Why do we need final here? Wouldn't it be better to allow extending classes to override, or at least provide a hook by which additional initializations can be done?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/DeltaSystems/dewdrop/commit/36be2a50231ddc5678c265f4a65a716afcf4a986#commitcomment-10467658>.
>


Thanks ;)


missing a space here


perhaps we are missing something because I have read this 10 times and am still working through the logic. The fact that it is this hard to understand tells me that we should probably fix the implementation.


Awesome job!


:smile:


This will not work for regular objects. They will end in `.type`. This should work: `tpe.typeSymbol.fullName`


scss-lint also points out
```
skins/oasis/css/core/PageHeader.scss:199 [W] MergeableSelector: Merge rule `.wikinav2.oasis-one-column .WikiaPageHeader` with rule on line 119
skins/oasis/css/core/PageHeader.scss:203 [W] MergeableSelector: Merge rule `.WikiaPageHeader.wikia-page-header` with rule on line 6
```
I think it shouldn't break anything, but it's better to ask. @bognix @vforge


Still I'd like to keep it as a double, just adding the explanation for it in another commit:

the denominator is always 1 because we rely on the schedule of the periodical being 1 second. Otherwise we'd need to store timestamps as well, which is overkill for this use case, I think.
To reserve the option to make it more precise, I'd like to keep the doubliness.


TeamCity Toolbox :: Pull requests [Build 44](http://10.48.34.86:8111/viewLog.html?buildId=7913&buildTypeId=Toolbox_PullRequests) is now running


TeamCity Toolbox :: Pull requests [Build 44](http://10.48.34.86:8111/viewLog.html?buildId=7913&buildTypeId=Toolbox_PullRequests) outcome was **SUCCESS**
Summary: Tests passed: 165 Build time: 00:00:16


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


Need to specify exact path where js and html files are, relative to /src


Can we give arguments as list instead of string? A space can be escaped and this code will split arguments incorrectly.


Redundant `typeof input === "object"`


master (kbenne) - x86_64-MacOS-10.9-clang-Debug: OK (475 of 481 tests passed)

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


It's a good idea that you try to detect null values but I think printing the error in console won't be of much use. Instead what we should do is throw an error so the code using this method can act accordingly. If we leave it like this the error will fail silently and we'll never be aware of it.


:crying_cat_face: without tests or at least explanatory comments this seems like pure magic especially with `Fixed 'typeDefinition of undefined' error` name


It's a shame that placeholder is not supported by IE < 10...


I like it :+1: . The problem with it is that we can't include this into the config-schema-scheme (Oo) so its impossible to autogenerate gui / docs / config checking for it.

But we could do this manualy. I would prefer a additional indirection like:

```
    {
      "id": "temperature_out",
      "name": "Temperatur außen",
      "class": "HomeduinoRFTemperature",
      "protocols": [
        {
          "name": "weather4",
          "options": {
            "id": 198,
            "channel": 1
          }
        }
      ],
      xAttributeOptions: {
        "humidity.displaySparkline": false
     }
    },
```

It's harder to remember, but we can its better seperated from the regular configs. I would handle this as an DeviceConfigExtension (Example: https://github.com/pimatic/pimatic/blob/master/lib/devices.coffee#L787-L802).


Fixes #340


I think maybe if assign some meaning to the non-zero exit codes we could probably get this combinator to be properly streaming.  Something like `exit 1` means we failed to parse but didn't consume input, so `(<|>)` can go on to the next parameter; and `exit 2` means we failed to parse and did consume some input, so `(<|>)` must fail overall.

But this gives the problem that I'm not sure how you'd implement `try` in that scheme.


I am going to add tests :smile:


Counting indexes with `for` loops is difficult to read and maintain. It looks like all of the for loops in this code can be easily replaced with the jQuery's `each`: http://api.jquery.com/jQuery.each/


What happens if none of the above set `values`?
(i.e. if internalRGB is passed `null`)
Should it set a default? pretend it's the same as `arguments.length ===0` ?


Ok, I see, sorry :)


<VirtualHost *> matches all IP addresses, and "the same port as the most recent Listen statement of the main server"

<VirtualHost *:*> matches all IP address, on any of the ports specified with `Listen`.

http://httpd.apache.org/docs/2.2/mod/core.html#virtualhost

I don't know which is "more" correct for our use case. Either way works. But I liked the rest of your commits, so I merged the whole kit and caboodle.


seems looks moot


@rwaldron I'm working this and self assigned at that issue. (also note the additional inline comments below..)


you're going to fetch for each resource, even if they use the same confdir. Get the list of all confdirs from `resources` first, then iterate on them and use that as the input instances.


Yep.


Change Where 'setprompt.sh'  Looks for the 'promptconvert.sh' file.


Indeed, from the styling point of view it would be better.
Now, having a label outside the div looks strange from the semantic POV.


Dit is niet de wijziging die ik bedoelde. Maar ja wie ben ik.


Can we skip those outer ``"`` in the parser?


Dit is een line note?


use `lodash` instead


Oh, that's a tough one. I'm not sure we should give any special semantics to `null` or `undefined` (jQuery found itself in a corner trying to do this). Maybe just throw?


Ah, right.


- skip this -


MarkdownDocs (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1675 of 1676 tests passed)

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


Maybe keeping the div "globalsearch" like it is, and adding a new div for "input-group" ?


Shouldn't this check be done outside of the `if...else if` block so it covers `savedInstanceState` as well?


I'd prefer a symbolic link, `ln -s ~/.vim/vimrc ~/.vimrc`


Ok, just making sure we weren't missing something...


I'm going to duplicate the behavior in the update to five.Led.RGB.color()


You should cherry-pick instead of copying commits as if they were yours. Bad habits!


jellybeans should be installed by Vundle automatically...


hey i just copied what you chatted me


> Ok, just making sure we weren't missing something...

Nope, just me missing something ;)


then why didn't you chat it to me with `-s`?


instead of moving after cloning, you could do it in one step with: `git clone https://github.com/mstyles/vim.git ~/.vim`


It looks like there are two very similar blobs of logic. After things are simplified down using iterative constructs like jQuery's `each`, it seems like there should be a way to abstract out the differences and simplify this code more.


but it isn't


then why didnt you say that in the first place?


This is true of the existing function as well, but why is it `<` when the function name is ge (which to me implies equals)?


See: https://github.com/alvaro-prieto/colResizable/issues/26


Since the logic is basically the same, I would have these two functions call another function that does most of the work.


sounds like user error :P


Uhm, so now simply importing `odeint` changes the global settings for the warnings.
I'd find this surprising:

```
>>> import warnings
>>> def rrr():
...     warnings.warn('!', RuntimeWarning)
>>> from scipy.integrate import odeint
>>> rrr()
__console__:2: RuntimeWarning: !
>>> rrr()
__console__:2: RuntimeWarning: !
>>> rrr()
__console__:2: RuntimeWarning: !
```
which is consistent with the filter 'always'. But if I do not import `odepack`

```
$ python runtests.py --python
Building, see build.log...
Build OK
Python 2.7.3 (default, Sep 26 2013, 20:03:06)
[GCC 4.6.3] on linux2
Type "help", "copyright", "credits" or "license" for more information.
(InteractiveConsole)
>>> import warnings
>>> def rrr():
...     warnings.warn('!', RuntimeWarning)
...
>>> rrr()
__console__:2: RuntimeWarning: !
>>> rrr()
>>> rrr()
```

which seems to be consistent with python's (unfortunate) default of only warning once.

I think it's best to avoid messing with global settings unless really necessary.


yeah, the user that sent the instructions in the first place!


yes you are right. After some test i found this way very dirty.
I have seen all device with attributes are build the same way.
So we can add the attribute field to the config.

```
    {
      "id": "temperature_out",
      "name": "Temperatur außen",
      "class": "HomeduinoRFTemperature",
      "protocols": [
        {
          "name": "weather4",
          "options": {
            "id": 198,
            "channel": 1
          }
        }
      ],
      attributes:[
       {
        "id": "humidity",
        "displaySparkline": false
       }
    ]
    },
```
But wit this we have the problem, that the plugin can overwrite the config entry while the device creation.
Your solution with the xAttributeOption looks really good.

I have the feeling, that the mobile frontend inst not enough configurable for the user.
A other way would be to link all possibilities to the device-config-schema. And then to the front-end. This needs some more work but would work too.


Not really any more than before. It's *enabled* for all subreddits in that anyone that wants to can use it, but it won't do anything automatically unless that subreddit sets up some rules in the config/automoderator wiki page.


receiving


how do we have this many comments on 4 lines of documentation but when I commit 40 lines of business logic all i hear are crickets?


Did the `ࠌ` cause any problems on Linux?

I put it there so it says "Brandschaden" in combat log and the item description *and* still leaves the cyclopedia link intact. (The cyclopedia parser otherwise treads 'Brandschaden' as a single word and can't find the entries for 'Brand' or 'Schaden' anymore)


libtool versions are C:R:A

libcrmcommon gets C++:0:0 b/c attrd_update_delegate()'s final argument changed from a gboolean to an int; libcrmservice gets C++:0:A++ because services_action_cleanup() was added; all others get C:R++:A if their source code changed.

The versioning scheme is described at http://clusterlabs.org/wiki/Point_Release_Checklist -- I had a bunch of ideas bouncing around in my head regarding point releases, so I wrote them down to get it to stop ;-)


Another helpful thing to do would be to run this code through jshint after making your edits to see if it gives you any code quality tips. It'll catch things like mixed tabs and spaces, trailing whitespace, double declarations, and so on.

I'm sometimes surprised at how well it catches scope-related probleme.


@nsoranzo Is this extended list drawn from some source - I am trying to compare this with the Picard list and there are some here that cannot be set in Picard. This is fine I was just wondering if there is another standard or something somewhere.


Space after comma, please.
Ditto for <space>binary-op<space> for string concatenation above.


Spelling - `programmign`


I am more concerned with the second argument given against the use of probability estimates, i.e. that they may be inconsistent with the scores. Maybe we should have a run on the attachment subtask to compare `predict_proba` with `decision_function`.


spelling


Remove if no longer required


Nevermind - found bam spec :).


remove unused debug code


I'm trying a new approach closer to scalac where I do the rewiring of value classes methods (step 2 of SIP-15) after erasure instead of doing it in ExtensionMethods, this way, calls like `new Meter(3).isDefined` introduced by patmat get rewritten to `Meter.isDefined$extension(new Meter(3).underlying)`, this also means that `new Meter(3)._1` gets rewritten to `Meter._1$extension(new Meter(3).underlying)`. I did this to avoid unnecessary boxing done by patmat but this also means that I don't need a special rule for  `_1` any longer, which as you've pointed out could be dangerous.


Obviously, this is my baby, I live by these settings


We could be even more accepting here, and take output plugs provided their Node isn't a ComputeNode - see `Switch::variesWithContext()`. I wonder if we should move `variesWithContext()` to a PlugAlgo.h file so we can reuse the logic - it seems like the sort of test we might want to use for other optimisations elsewhere.


It's for Wurst-Bots only, so it doesn't.


Can you add a short comment explaining this "magic value"? Is there no other way to find out of there are more trips to be queried?


receiving


understandable, i remember my vimrc being a masterpiece, but that was over a year ago and i don't know where it went :(


c'est une ancienne consigne que j'ai oublié de virer. les tests sont dans processor.


Hahaha!!! Nice!

On Wed, Apr 1, 2015 at 6:49 PM, Stelios Savva <notifications@github.com>
wrote:

> +1
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stelios313/simple-search/commit/70175b6891f8bb6fa7369c5e65ea12727c1d60a6#commitcomment-10526754>
> .
>

--
George Eracleous
Avocarrot co-founder

W. http://avocarrot.com
E. george@avocarrot.com
T. 6504229317
uk.linkedin.com/in/geracleous
t: @geracleous


I'm not going to do anything about this quickly (I'm still learning about error handling strategies in haskell), but in the meantime I've opened an [issue for this](https://github.com/seagreen/hjsonschema/issues/10). Thanks for the suggestion.


ditto :+1:


On Android, the safest way would be to use Activtiy.onPause and Activity.onResume. They are always guaranteed to be called.

http://developer.android.com/images/activity_lifecycle.png

1.) Activity.onResume --> enableGLCalls
2.) Activity.onPause-->disableGLCalls,
2.) Activity.onDestroy-> map destructor..


c'est le cas


MarkdownDocs (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1675 of 1676 tests passed)

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


Can we delay this computation until we actually need to use the path matcher? As a general principle, dependency graphs shouldn't really compute anything until something is pulled on from the outside.


ouaip, en fait, c'est plutôt ici "l'ennemi de mon ennemi est son ennemi" :P


ok, je vire


We need a binary compatibility note in the commit comments, since this changes the size of PathMatcher.


`setup`? seems a bit loaded, but the only alternative I can think of is `init`...


I figured it was worth keeping as a proof to you that everything was working fine.  I can get rid of it if you'd like.


Heya ! .. did you measure the difference this made to performance ?


Can you open a ticket? I use PromiseKit with CocoaPods and I `use_frameworks!`.


Something here isn't right.

I think it's related to a specific issue with these commits:

5072d51dc3b15d0f3283e133d844d3da4438f302 01f650dc622619a7f412b1b8f8e8aed1e7382c24 b8fee71d1b83f08321433a66a81f73fd2d24a11e 65e31b9f20e0407a73f1c5ab96371750bd3edd4e e7dce82eb72af35b56b0e67c47903ecfb75d318d ec9d40d7df8470c4b4825d762b1405ec2ac0da67 f0a3bc24be4dd4a084f77772ff892c0acb85b79c d243756866ae57eaa7df6d6c0055b0e6c06acca1 4e4f4ac775d9253cdd0796225a7204e82078c704 403d5a05caa42d797006b79510ecfb6a88b26e35 bf00dc9d2249723f3efe47776446fdf841108fdd 27560fa81096ec6096351403d9577da6a11d5a42 27b36f07f54c04682998b7d75a089725836f5e14 d1e97aeaffa8fced6533e609aa04688d7d57f94a 96cbd9ed2e2e87747bd2a973c151488bc8622a6b 7ff26168879bc760024cf714f975d7fffc5440a5 a9c56310c586b259cc3e4055e595e98e199a3be0 4fbb35e896cd3441bab214d689a48570af38e1d9 e517576ff931c66ff885f393eba31459a652c462 5ed390c58c7a4ae9d0fdf6baee1077362bd0a93d 3173f8b225e5f9cfaad2a04eb88feeb65c1f0033 74ff2645af2735291112cf8896a594217dc6bf0c 38642eba260e8b0fae8c13f06ae13f088ec3819e 3c496ca4b73a5866cc151d260b385c649532aa8a d39c642946c158ace272b1c6151ee6f175892de2 914839f70cbd03f4727f016fa0e93dc4b7b3e7aa 727502c37fbd60ce9d66164512bf07756aaa33f4 3076d0baa3d31454a2371b0fc6f57d1e0c9835c0

but I can't be sure.


I get that, but if you look at it from the perspective of an author, having that dummy argument seems at least equally awkward? one way around that might be to pass a positional index, but that seems like cheating


This'll create a bit of conflict between `_.inRange` and `_.range`:

```javascript
_.range(-6, -2); // => [-6, -5, -4, -3]
_.inRange(-3, -6, -2); // => true

_.range(-2, -6); // => []
_.inRange(-3, -2, -6); // => true
```

Yah, I know you're supposed to supply a negative step to `_.range`...


Maybe leave a comment here? "Workaround for bug 1108166, in which the server returned an incorrect clientid when registering with no assertion"