The interesting catch here is that since there's no more actual loading of the module, we can provide bogus imports and avocado won't notice them :) That sounds good, given that being more secure against badly behaved test modules is better.


I think it is correct as written.


@SeanFeldman see the description at https://github.com/Particular/NServiceBus/issues/2956 - it describes my experiments when reproducing the problem.


Please fix indentation


it might only happen when there aren’t any newlines between new content and quotes?

On Oct 6, 2015, at 10:47, Dmitriy Likhten <notifications@github.com<mailto:notifications@github.com>> wrote:

let me know what happens! :)

—
Reply to this email directly or view it on GitHub<https://github.com/jdpopkin/email_reply_parser/commit/bf9d5ce00f52f9096acf87a60f3ed6134338f03e#commitcomment-13616669>.


Boss, this is awesome layout for the article. How's the progress on this so far?


It seems like the `coerce` methods should be the ones that raise, that way it can provide an appropriate error message.  Also, `nil` might be an acceptable value, unless it is a non-null type.


Yep, I don't mind moving it into another library.  As long as we can discuss this and come to a consensus quickly.  As I said earlier, I think we need a new vehicle class, like "AP_Airframe" which would handle stuff like this.

I agree that a proper GUI in MP will be critical to making this usable.  I already know what we need to do, which is basically a recreation of a common screen that exists in the Heli profile of most Tx today.  Old school heli users will be very familiar with it.


Please fix indentation so it's consistent, there's a lot of uneven spacing between lines 2-9


yep - dont worry i just wanted to grab a simple po file to use

the good news that it mostly works

only 1 test fails - something to do with obsoleting units requiring the file on filesystem i think


I will see how bit a change this gives me (likely HUGE) later tonite.


@nrc I know but it seems like they should be :) (at least for the rustc ones). But whatever.


@AngieHinrichs typos fixed.


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


Can you also add change sets that drop the active list tables


@dwaynebailey - see here - it overrides the revision with the current max_unit_revision


Why do you have two form-groups?  Are you not able to use one?


Ah, nice catch thanks.

Yeah, I tried using LEAK_MIN as the name, so we would have RATE_ROLL_LEAK_MIN, but that's too many characters, so I went to the crappy _I_L_MIN suffix.  It's all I could figure. Any other suggestions?


Note - this issue doesn't arise with Portable PDB format, since we associate the dynamic bitmask not with a slot but directly with a variable or a constant.


This is definitely the direction I am going in.  I originally implemented something much rougher and what code you see here is me pulling bits and pieces of what worked out of the original as the basis for where I'm going here.  Basically, what you see in the branch is not what the API will look like, it's more like my playground for getting a few implementation details right (such as ClrType letting you cast to dynamic to get static fields, etc).  Things will start shaping up when I have a real chance to sit down and work on it.

The eventual plan is:

1. Publically expose ClrDynamicObject (or whatever I name it).
2.  It has to work with V2 (so there will be a ClrObject for v2 which doesn't get "dynamic" support, but still has type/size/etc.
3.  Add convenience stuff like ClrType will be able to (optionally) give you static fields with because it's dynamic too.
4.  ClrObject (or whatever I call it), needs to be very light weight.  "IEnumerable<ClrObject> ClrHeap.EnumerateClrObjects()" will really suck if ClrObject is a heavy-weight object with a lot of fields..."

Or something like that.  I have a lot of other work in ClrMD to get to before I really flesh out and complete dynamic support...

Thanks for the feedback though!  Please let me know if you have suggestions and such when this starts coming together.


@derekcollison Maybe? This is how the clientOpts is specified within the gnatsd server. https://github.com/nats-io/gnatsd/blob/fe3b8f2aa1826d85d78ee4d17502ae03e27afb26/server/client.go#L73 and it is used via auth.Token : https://github.com/nats-io/gnatsd/blob/fe3b8f2aa1826d85d78ee4d17502ae03e27afb26/auth/token.go#L13


Ok, no problem.

On 6 October 2015 at 02:02, Randy Mackay <notifications@github.com> wrote:

> I think set_using_interlock() should be moved out of the arm_checks
> function perhaps to the one-hz-loop. I don't think a consumer of the
> ap.using_interlock flag would expect that it's only correct after the
> arming checks have been run.
> I think it probably all works fine but it's slightly brittle.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/R-Lefebvre/ardupilot/commit/64746cb20bd469b07b856fda942ab910c2fdce81#commitcomment-13607600>
> .
>


Note that this resolves an unreported bug, wherein the number 1 would always be reported as prime, a known universal falsehood.


Thanks I will try that out.

Zachary T. Murphy
Computer Technology Major
WCCC

Mobile: 724-516-7201

Sent from my iPhone

On Oct 6, 2015, at 3:01 PM, Katrina Siffrinn <notifications@github.com<mailto:notifications@github.com>> wrote:

A thought to ponder. It's possible to simplify your methods a bit more. You could declare the Address and Dimensions in the beginning, after your instance of the Scanner. for example:
Address address1 = readAddressInfo(keyboard).
Each subsequent method entry point can then be simplified to read something like:
public static Dimensions readDimensionInfor(Scanner keyboard){
By doing this, you lose the line of repeated code in each method calling the new Scanner.

-
Reply to this email directly or view it on GitHub<https://github.com/ztmurphy21/CPT-163-27-F2015-Assignment-6-DRY-Refactoring/commit/cb68d7c574bae8857e2d40d3ef5bc0c2c88d6d35#commitcomment-13622498>.


In the same vein as what was already said, I thought it was interesting how there was so much input and pushback on removing the customization of the both the scroll bar and menus, but after a while designers got used it, and removing the decisions regarding these technologies inevitably helped them move on to more aspects to develop. This is another contradiction in the "free market" design aspect and have the decision taken, so that you can move on to more customizable features and to develop more aspects to a project or the the Andrew Project in general. At least, that is how I saw this.


TeamCity Colore :: Main Build [Build 188](http://tc.sharpblade.net/viewLog.html?buildId=736&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:51


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


Is there any point in keeping the attribute if we're always building the mir in any case?


Can you also remove the problem list section on the patient dashboard?


@miigotu ok removed python3-openssl and install python-openssl, restarted sickrage and still same fault

2015-10-06 22:49:11 INFO     SEARCHQUEUE-MANUAL-281617 :: [KickAssTorrents] :: Unable to find a download for: [Empire (2015) - 2x02 - Without a Country]
AASSLError: hostname 'kat.cr' doesn't match either of '*.kickass.so', 'kickass.so'
AA    raise SSLError(e, request=request)
AA  File "/opt/sickbeard/lib/requests/adapters.py", line 431, in send
AA    resp = super(CacheControlAdapter, self).send(request, **kw)
AA  File "/opt/sickbeard/lib/cachecontrol/adapter.py", line 46, in send
AA    r = adapter.send(request, **kwargs)
AA  File "/opt/sickbeard/lib/requests/sessions.py", line 576, in send
AA    resp = self.send(prep, **send_kwargs)
AA  File "/opt/sickbeard/lib/requests/sessions.py", line 464, in request
AA    return self.request('GET', url, **kwargs)
AA  File "/opt/sickbeard/lib/requests/sessions.py", line 476, in get
AA    resp = session.get(url, timeout=timeout, allow_redirects=True, verify=session.verify)
AA  File "/opt/sickbeard/sickbeard/helpers.py", line 1623, in getURL
AATraceback (most recent call last):
2015-10-06 22:49:11 ERROR    SEARCHQUEUE-MANUAL-281617 :: [KickAssTorrents] :: Connection error to getURL https://kat.cr/usearch/ Error: error hostname 'kat.cr' doesn't match either of '*.kickass.so', 'kickass.so'
2015-10-06 22:49:10 INFO     SEARCHQUEUE-MANUAL-281617 :: [KickAssTorrents] :: Performing episode search for Empire (2015)
AASSLError: [Errno 1] _ssl.c:504: error:14077438:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert internal error
AA    raise SSLError(e, request=request)
AA  File "/opt/sickbeard/lib/requests/adapters.py", line 431, in send
AA    resp = super(CacheControlAdapter, self).send(request, **kw)
AA  File "/opt/sickbeard/lib/cachecontrol/adapter.py", line 46, in send
AA    r = adapter.send(request, **kwargs)
AA  File "/opt/sickbeard/lib/requests/sessions.py", line 576, in send
AA    resp = self.send(prep, **send_kwargs)
AA  File "/opt/sickbeard/lib/requests/sessions.py", line 464, in request
AA    return self.request('GET', url, **kwargs)
AA  File "/opt/sickbeard/lib/requests/sessions.py", line 476, in get
AA    resp = session.get(url, timeout=timeout, allow_redirects=True, verify=session.verify)
AA  File "/opt/sickbeard/sickbeard/helpers.py", line 1623, in getURL
AATraceback (most recent call last):
2015-10-06 22:49:10 ERROR    SEARCHQUEUE-MANUAL-281617 :: [ThePirateBay] :: Connection error to getURL https://thepiratebay.gd/s/ Error: error [Errno 1] _ssl.c:504: error:14077438:SSL routines:SSL23_GET_SERVER_HELLO:tlsv1 alert internal error
2015-10-06 22:49:10 INFO     SEARCHQUEUE-MANUAL-281617 :: [ThePirateBay] :: Performing episode search for Empire (2015)
2015-10-06 22:49:10 INFO     SEARCHQUEUE-MANUAL-281617 :: Beginning manual search for: [Empire (2015) - 2x02 - Without a Country]

But i am hijacking your kat issue sorry for that.


The above fixes were done to the file paths on these kinds of slides:
![image](https://cloud.githubusercontent.com/assets/12142968/10322215/a3b980a2-6c31-11e5-8165-795b6cc0b6e7.png)


@nrc

> Is there any point in keeping the attribute if we're always building the mir in any case?

well I thought it's still nice to see the graphviz output, though really I guess it should be a compiler flag


No problem - there is a bug with the hipchat acknowledgement message which I'm just about to fix, and lots of improvements possible - if you have any good ideas please file as issues.


Can't you simply write **THREADS="$(nproc)"**?


TeamCity Colore :: Main Build [Build 189](http://tc.sharpblade.net/viewLog.html?buildId=737&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:49


I took the "Scientific Truth" part of the title to be referring to the (wrong) idea that technology automatically moves towards a "best" state. This way, each part of the title could refer to different discussions on technology and its context within the chapter.


This should probably be `STATUS_CREATED` since nothing actually starts until an upload event has been emitted.


This should be inline with the last div's end tag


gute sache mit exclude bower.
schau dir mal issue #1 an. lass das mal so machen wie in der psd datei vorgeschlagen.
@chk1: ist das jetzt überall die Brix Slab Schriftart?


ah I see what is going on


does this test just check that nothing ICEs?


Interpolation can be can 50% slower than concatenation on my machine. However I think readability (I am personally liking interpolation more and more) could trump this in some cases in non-performance critical code paths.


Don't include commented out code in commit unless this is for a future functionality


On the note of histories being objective, does this in some way take away from the area of deciding who and what gets the credit for the development of these ideas? I know that this is can be subjective, especially in the area of computer development and web design. I know that is why, when code is created, or changes are made, there is always descriptions to accompany this, so that anyone may work on it, but I am going to assume that anyone who created a systematic change would want to be recognized for their work.


@NateWr, you asked about this -- you should now see `index` in the page/op name when viewing the homepage even if it's not explicitly specified in the URL.


yes


looks like `ENABLED()` just concatenates a `1` with whatever the feature is, and then returns `1` or `0`.

Seems like the same as `defined()` to me, but we should be consistent.


@ajithcj Compiled with debug=yes and tried several bench runs. With different number of threads and different depths. Looks ok. Also ran in terminal, 5 minutes (go movetime 300000) search on the start position with 6 threads, and another one up to depth 28 (go depth 28). Both finished without problems. Only thing I noticed was in the last run, nps suddenly dropped by 1/4.


From what I've seen in web discussions elsewhere, many people are moving away from using applications like Javascript and Flash. For Javascript, at least, I think there is a question of security. I wonder if there are ways to make HTML5 backwards compatible without using Javascript, or if the alternatives are less pleasant or non-standard?


nice catch


@calvinmetcalf this ok?


I'm not sure manifestTimingSources should be modified here - the manifest contents should be used regardless of whether the protocols match. It is the content provider's responsibility to set the URL correctly.

Perhaps UTCTimingSources could be modified and cat'ed if it's actually required?


>       variant.stock_items[0].send(:write_attribute, :count_on_hand, 0)
> (that's just ridiculous code!)

`write_attribute` is made private by rails, not spree.

I think your warning is a nice idea, but I'd rather it go further and raise rather than warn. One can always use `stock_item[:count_on_hand] = 0` if they really know they want to bypass the correct working of things.

A `StockAdjustor` would of course be nice!


it was increased to not fail ternary condition


@joergoster Thanks for trying out! I wonder what went wrong on CoffeeOne-23cores.
In anycase I've launched a new one with only helper threads splitting. Not sure what to do with the original test, I'll ask for tests on high end machines on the forum.


Yeah, a similar change (from 12 to 24) to core sped up my loading of 9M bcrypts from "gave up after half an hour" to a *literally* a few seconds.


hey foo


@CLC-4662W I found the mention of the Brontes really interesting as well. In context, Haas was referring to the creation of literacy tools and sign systems—do you think she was referring to a particular way of writing, or the creation of a new language by the Brontes? I'm still a little puzzled that she would use that example.


ok guys i did some digging found this thread https://github.com/SiCKRAGETV/sickrage-issues/issues/1231#issuecomment-97815695

Will look at it tomorrow there is probably my solution thanks for fast reaction


Yes, you can style the gist container


Maybe add a line introducing this section, something like:

"You can use rspec-page-regression either headlessly (using PhantomJS and Poltergeist) or in a browser (using the selenium driver).  Instructions are given for each:"


why not use a stream? ☺


I'm a bit sad it's five lines longer than before. owell.

$ flake8 --max-complexity 3 pgctl/daemontools.py
pgctl/daemontools.py:38:1: C901 'SvStat.__repr__' is too complex (5)
pgctl/daemontools.py:70:1: C901 'parse' is too complex (6)
pgctl/daemontools.py:89:1: C901 'svstat_parse' is too complex (7)


@ajithcj You're welcome. Maybe there is a problem which only shows up with a higher number of cores. Good luck with your test!


For consistency I'd change "with" to "using"


I could, but by using $(()), THREADS will be initialized to zero in case
nproc returns non-parsable integer value.

2015-10-06 22:54 GMT+02:00 rapperskull <notifications@github.com>:

> Can't you simply write *THREADS="$(nproc)"*?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mc1100/opengapps/commit/5bf48d2760f318a1853d704880add7b9a4016d3b#commitcomment-13624898>
> .
>


@diegobz cleaned up regex in favor of string formatting
Now the format string would be set like this: "urlformat":"http://%(language)s.transifex.com/%(page)s"


For consistency I'd change this to

"### In browser, using the selenium driver"


RE:

> @lliss What do you think about getting rid of processing and change complete to completed?

> I certainly don't mind changing the tense for "complete". But why remove "processing"? I think it is good to know that it actually made it to the processing job.

We should keep `STATUS_PROCESSING`. I just wanted to point out that it is in a different tense than the other statuses. But I can't think of anything better.


Trying one from my phone to see what source looks like in a successful post.

On Oct 6, 2015, at 10:48 AM, Dmitriy Likhten <notifications@github.com<mailto:notifications@github.com>> wrote:

let me know what happens! :)

—
Reply to this email directly or view it on GitHub<https://github.com/jdpopkin/email_reply_parser/commit/bf9d5ce00f52f9096acf87a60f3ed6134338f03e#commitcomment-13616669>.


hey this is somethign


? https://en.wiktionary.org/wiki/awakeness


I like the idea of blockquotes. I think the creation of elements that allow blockquotes speaks to some atypical audiences in the HTML field. I wonder when the blockquote element was created, and if it was introduced early on or once a new community of HTML users came along.


Mind writing a test for this one?


try removing the `vcxproj\.nuget\nuget.tagets` it should restore without these if you use the commandline restore. MSBuild is probably having issues because these targets are manually added to vcxproj` here.


please, delete 'app.loadReason === "startup"' from condition, it's annoying, open new tab on each browser start


On Tue, Oct 06, 2015 at 09:57:47AM -0700, ericsnowcurrently wrote:
> That makes sense then. :)
>
> Out of curiosity, why is Exec closing them?  I wouldn't expect that to be the case (hence my original comment).  Also, I only see where stdin gets closed.  That's only done to send the EOF for the sake of the websocket, right?

Yeah, we've had a lot of issues getting things to close in the right
order in this code. If we're not calling Close somewhere on something,
we could relax the API restrictions, although I haven't looked that
closely.

> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/lxc/lxd/commit/0a1d9fab0c3fb9cf5fab96966004d1781bfb0a6b#commitcomment-13619885


I think this whole paragraph could be replaced with just

"To run in a browser, you use the [selenium-webdriver](https://rubygems.org/gems/selenium-webdriver):"


A little lifting to the mood factory :)


This returns an `int`


That would be cool to know if there are any other sorts of history behind the other colors like the "rebbecapurple" we talked about in class. I do not know why there are different options for coding in the colors because you can us the word tag, rgb, and also the color hex option. I think the easiest is to use the color hex which is the three double digit numbers starting with the # sign. It is hard to know all the different color names that are used and the rgb coding is confusing to me.


Thank you, I was not aware of the zero initialization.


Would `message` work here?


Yes


It is tricky... it's basically to add a consonant between two vowel sounds. Because "LLDP" is pronounced "ell-ell-dee-pee", it starts with a vowel sound ("ell") so you switch "a" to "an".

I think the same thing is happening with "an historic event" because the "h" in "historic" is almost silent. Therefore you switch "a historic" to "an historic" so that there's a clear consonant separating the two words.


Yea, this makes sense. I like to encourage extending the class rather then writing hooks/methods to work around visibility, which has been requested.


parameter is not necessary. `cache` is class member parameter


I think it'd be awesome if we could have a feature for this as well, just to specify how it works and ensure we don't break the CLI :)


Tried turning off proxy? That's the problem we have discovered. We are going to strip out glype proxy support and allow specifying custom mirrors-proxies for each provider.


Our convention is not to use wild card imports https://wiki.openmrs.org/display/docs/Coding+Conventions


I'm not certain where to put this paragraph.  Where it is now it seems like it's specifically part of the selenium instructions.  Maybe it should be in its own section "## Directory Structure" just before Usage?


I think there's no such thing as `null` input for graphql. Although there is an open PR for it, then this will need a refactor: https://github.com/facebook/graphql/pull/83


so it fixed the world loading error? huh, when i downloaded the mod again, my world still didn't load?
I'm using latest version of forge 1.7.10


no way this will work


Why did you re-introduce the above? It was relocated in bbef2e9 due to problems.


I remember us talking about this when you first made the plugin :)


TeamCity pull requests :: netty [Build 375](https://garage.netty.io/teamcity/viewLog.html?buildId=2074&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 485 (485 new), passed: 5130, ignored: 21 Build time: 00:49:21
### Failed tests
```
io.netty.channel.epoll.EpollChannelConfigTest.testOptionGetThrowsChannelException: <no details avaliable>

io.netty.channel.epoll.EpollChannelConfigTest.testOptionSetThrowsChannelException: <no details avaliable>

io.netty.channel.epoll.EpollDomainSocketFdTest.testSendRecvFd: <no details avaliable>

io.netty.channel.epoll.EpollSocketConnectionAttemptTest.testConnectCancellation: <no details avaliable>

io.netty.channel.epoll.EpollSocketConnectionAttemptTest.testConnectTimeout: <no details avaliable>

io.netty.channel.epoll.EpollSocketConnectionAttemptTest.testConnectRefusedHalfClosure: <no details avaliable>

io.netty.channel.epoll.EpollSocketConnectionAttemptTest.testConnectRefused: <no details avaliable>

io.netty.channel.epoll.EpollSocketEchoTest.testSimpleEcho: <no details avaliable>

io.netty.channel.epoll.EpollSocketEchoTest.testSimpleEchoWithVoidPromise: <no details avaliable>

io.netty.channel.epoll.EpollSocketEchoTest.testSimpleEchoWithAdditionalExecutorNotAutoRead: <no details avaliable>

io.netty.channel.epoll.EpollSocketEchoTest.testSimpleEchoNotAutoRead: <no details avaliable>

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


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


I also remember you hating me or something.


Weiirrdd. I swear I got 12.4.4 today via a knife-bootstrap. It stood out to me, I went looking for a release announcement and found none, etc. Maybe I'm completely losing my mind. I only see 12.4.3 now.


I don't think we need the "Note on versions" information now that we're moving up to 1.0 -- I think we can just presume rspec 3 and forget about the old ones.  (Technically that'd be a change that's not part of this PR; but just thought of it now as I was reading this, so mentioning it here.  Could sneak it in with this PR or do it it separately.)


hey now


I think it's better to put this as the first member of the structure.


# TODO: Write tests


@Einarin lets try to use rebase instead of having merge commits.


@cehallen DWMR. Press on!


TeamCity pull requests :: netty [Build 376](https://garage.netty.io/teamcity/viewLog.html?buildId=2075&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 340 Build time: 00:02:53
### Failed tests
```
io.netty.util.HashedWheelTimerTest.testExecutionOnTime: <no details avaliable>

```


Ignoring exceptions silently is extremely hard to debug.. you'd wanna catch the appropriate exception or at least log the exception always.

    import this

> Errors should never pass silently.
> Unless explicitly silenced.


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


Fixes #1


One can run Php on Paginas FEUP?


I don't think we could consider `Favourite Point Counter` as a `User setting`.
You should better do as it's done for `possibleLogTypes`


I just did the git stash, pull, pop.   Sorry if that was re-introduced. I will smoke it.


should be public?


TeamCity Colore :: Main Build [Build 190](http://tc.sharpblade.net/viewLog.html?buildId=738&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:51


git you


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


Perhaps one of these?
```
$ git grep -B1 -A2 "undef inline"
autoconfig.h.in-#ifndef __cplusplus
autoconfig.h.in:#undef inline
autoconfig.h.in-#endif
autoconfig.h.in-
--
blake2.h-#if _MSC_VER
blake2.h:#undef inline
blake2.h-#define inline
blake2.h-#endif
--
blf_plug.c-
blf_plug.c:#undef inline
blf_plug.c-#ifdef __GNUC__
blf_plug.c-#define inline __inline
--
escrypt/sysendian.h-#if _MSC_VER
escrypt/sysendian.h:#undef inline
escrypt/sysendian.h-#define inline
escrypt/sysendian.h-#endif
--
jumbo.h-#ifdef _MSC_VER
jumbo.h:#undef inline
jumbo.h-#define inline _inline
jumbo.h-#define strupr _strupr
--
jumbo.h-#pragma warning (disable : 4018 297 )
jumbo.h:#undef inline
jumbo.h-#define inline _inline
jumbo.h-#endif
--
misc.h-#ifdef _MSC_VER
misc.h:#undef inline
misc.h-#define inline static
misc.h-#endif
--
sunmd5_fmt_plug.c-#ifdef _MSC_VER
sunmd5_fmt_plug.c:#undef inline
sunmd5_fmt_plug.c-#define inline
sunmd5_fmt_plug.c-#endif
--
unicode.h-#ifdef _MSC_VER
unicode.h:#undef inline
unicode.h-#define inline _inline
unicode.h-#endif
--
unrarhlp.h-#ifdef _MSC_VER
unrarhlp.h:#undef inline
unrarhlp.h-#define inline
unrarhlp.h-#endif
```


no cast


Dead code?


I see your point. I'll add a `logging.error` call.


It's a good point. I guess we should do due diligence and check if the changes are binary compatible before doing so. \cc @jroper


Why was this removed?  It should be right the way it is now.

benchmark_level is ONLY set if --test-full is used.  Thus, benchmark time is cleared out.

If you remove this, then --test-full will ALWAYS be run with -test=1


Streaming-API. Nice. Reads good. Don't know how the case with the colon (`Word: SmallerWord`) can be handled. What about the things with `Wo{RD}`? The test cases 2 to 4 by @matthiasgeiger at https://github.com/JabRef/jabref/pull/176#issuecomment-142723792. Do we need an ANTLR grammar? :sweat_smile:


Store the result in a `class member` for later use


I think the stop_ids hers cannot refer to park and ride or kiss and ride lot ids because GTFS validators will error on them as unknown stop ids. => Needs new supplemental transfers.txt (transfers_lot_ft.txt?) with the same format for stop id/lot id transfers.


TeamCity Colore :: Main Build [Build 191](http://tc.sharpblade.net/viewLog.html?buildId=739&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:49


git foo


Unrelated to this commit, but it looks like this comment is now out of date with the code.


Typo: indirectness


For reference, talking about this PR: https://github.com/mafintosh/electron-prebuilt/pull/64


Was there a specific reason for this change, other than aesthetics ?

Was this not working the other way (I would have to be shown how/why it is NOT working).


I agree that HTML5 is better in this case than HTML, but as @alexandradepalma said people are reluctant to change. It took me forever to decide to upgrade my computer to Windows 10. I had a discussion about this in my other class where it is the job of browsers and different companies to make people want to upgrade to the newer technology. Older browsers do not support a lot of new technology. Even when looking at things for the historicizing project, I still saw a lot of newer elements were not supported in all browsers. So it is interesting that people took the time to make HTML5 work in older browsers, because they did not have to.


I tried to propagate up the notification by mounting /run/systemd/notify but that doesn't work.

Dan Walsh explains "We have sd_notify working for runc containers but not for docker run clients. Problem is docker run is a child of systemd unit file, while container is a child of docker daemon."


why name is phone validation?


Только наверное всё-таки rows.


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


- Код содержит слишком много ошибок. Вот некоторые из них:
1. main.js:42 Uncaught TypeError: Cannot read property 'getElementsByClassName' of null
2. Uncaught TypeError: this.removeItem is not a function
- Код не обурнут в модуль (http://largescalejs.ru/module-pattern/)
- addEventListener() использует инлайн функции. лучше выносить их отдельно
- new Array() -> []
- alert() лучше вообще не юзать...
- Не очень понял зачем нужно было усложнять с подлистами и их элементами. Именно это и усложнило код и вызвало много ошибок.
- Как на счет создать два туду на одной странице? Нужно сделать рутовый элемент, относительно которого мы будем искать все вложенные теги. Например: rootEl.getElementById()...
-


@NOTtheMessiah


versione lagemer us id property files. und zxing gits inere vill neuere version?!
http://mvnrepository.com/artifact/com.google.zxing/javase/3.2.1


Do not add Two parameters, but just a boolean.

L111-L121 could be much simpler,
ex:
```
            postResult = GCParser.postLog(cache.getGeocode(), cache.getCacheId(), viewstates, logType,
                    date.get(Calendar.YEAR), (date.get(Calendar.MONTH) + 1), date.get(Calendar.DATE),
                    log, trackableLogs, fav_check.isChecked());

```


Nit: I'd prefer a comment like "// Ensure that this is not considered a move, but rather a reborrow." (even though I can learn this from #28839)


Nice cleanup!


hey foo


I am pretty sure this is why the salted (with a HUGE number of salts) dyna is like watching grass grow.  Actually I DID mow my back yard waiting over the weekend (no lie, lol).


typeof(a, b) is invalid; only typeof b is returned


du chasch doch ned es file abspeichere..? das isch ja anderst unperformant. du müsstischs direkt mitgä im xml als base64 wenns wörkli eso wetsch mache.


TeamCity Colore :: Main Build [Build 192](http://tc.sharpblade.net/viewLog.html?buildId=740&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:48


Would it be possible to document this new feature?

I have some uses for that because on some of my computer my user is not home directory has symbolic links to a /mnt/data directory on which I mount some disk.


remove system.out.println


Verificaciòn jefe de familia


can this be made as the else part of line#25


exceptions münd gfange werde und sinnvoll witerverarbeitet werde


This should be an `@Override` from `ILoggingManager`.
A function called getXXX should return something.


@JakeDaBoss DO NOT MERGE


yes. But I'm not very proud of https://github.com/hwoarangmy/OpenDungeons/blob/5437c42a186e60993e8f336062ab6eb98e65383f/source/gamemap/GameMap.cpp#L2861

I will try to something else


ИМХО, суффикс `_tmp` лишний, нет же другой переменной с именем `min_sum_tab`.


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


another comment


readded mup.json


:coffee:


This function should return value stored at L86
Do not save in `Settings`


hey hey


Add here only a single parameter `final boolean addToFavorites`


No, it didn't work properly. I did not remove it, I **moved** it.


foo


TeamCity Colore :: Main Build [Build 193](http://tc.sharpblade.net/viewLog.html?buildId=741&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:49


Paginas FEUP runs PHP and ASP.NET. No database though.
https://sigarra.up.pt/feup/pt/web_base.gera_pagina?P_pagina=21231


Change that test to something like
```
if (addToFavorites) {
    params.put("ctl00$ContentBody$LogBookPanel1$chkAddToFavorites", "on");
}

```


Repeats the same words from the introduction...


Not really useful


When I run my RSpec tests with carrierwave's enable_processing set to false, this code is resulting in a call to model.send(:"process_#{mounted_as}_upload", which is breaking my tests.


R is a bit weird in that it thinks functions with "." in the name are S3 methods (e.g. `summary.lm`) so probably good to use "_" here.


This was why Travis failed OpenCL. You set it too late, I moved it to earlier in the process.


barcode4j hanni nid zum laufe bracht. viellicht schaffsches du ja :)


Yay Lego metaphors. That actually makes it sound more useful as a layout,  even when you don't see "div" used as much in the coding because there are now elements made from its pupils (I kinda want consider things like nav and article as its pupils or students, since they kinda grew up into elements of their own in HTML, haha). I like that there's something that flexible to work with. Thank you for that insight.


@nbp: Yep, that's the plan, see https://github.com/NixOS/nixpkgs/issues/10204. Specifically, [this comment](https://github.com/NixOS/nixpkgs/issues/10204#issuecomment-145842931).


add data  outside from tag file


I bet you are right.  Thus, the multiple include 'undid' some of these other 'hacks'.  It did not make sense as to why a proper wrapper caused grief, but now it shows.

jumbo.h gets included VERY early (in arch.h I think). Thus, if the define got blasted later, (such as in misc.h), then I would get the error about double pre-declares (static and inline used). VC is happy with static, or with static _inline, but NOT with static inline.  That is because (I think), vc is the same compiler for C and C++, where gcc you have g++.   _inline is 'C' for vc, while inline is C++.  Vc has allowed inline to 'work', but it fails under certain conditions.

Now that I see this, I am pretty sure I can figure things out properly, once I get a chance to look.


need to update to chage input to `x` ?


ah, I removed `BaseType`


I think this deserves a mention in the documentation - something like "avocado tests are not loaded at detection stage for security reasons, which means bugs due to supporting libraries not installed and other subtle bugs may not be noticed. Please use static and care whenever possible".


I have reverted the formats.c, but even there it would have caused no problems (it would already have been 0).  But as long as it gets done, things are 'fine'.


These are also useful for other drivers that use PCI and need to edit the command register. What do you think about putting them on PCI files?


but isn't `BaseType` a part of a `MemberOrTypeExpression` anyway? can it be part of something else?


Test


We can either export the functions to the user or keep them hidden and use them internally.  If we remove this @export line then it will be internal.  Internal functions usually don't have documentation...


This doesn't look so good because snippets don't have spaces between them - can probably swap the `hbox` and `vbox` and put some `hskip` between columns?

I don't think this example is all too simple to understand for a user not familiar with `map` and some of the more Julia-specific Julia syntax. Might even be nicer to dumb it down to do one thing in a line, this need not be exhaustive for all possible combinations of fontweight and fonttype - just a few examples as long as you mention allowed options in the text will do just fine.


This was just something that git stash pop jammed back into the code. I knew I was working on formats.c, and simply did not see the comparison difference.


@maothao I definitley found this surprising. I would think that the engineers/designers would want to make the best scrollbar they could, but instead were satisfied with what they had. They did not want to change to a newer/better version of what they already created. @alexandradepalma I can see that this would be a problem in other industries. I think this can be a problem with a lot of things, in a broader sense. Where people in their specific field do not take into account people who do not have any knowledge of that field. I think because the scrollbar is not a consumer product, the students and user advocates who may actually use the feature were not taken into account at first.


Sorry about my delayed response. When applying these changes in the other template files (post, page and loop) the ghost platform seems to be struggling:
![image](https://cloud.githubusercontent.com/assets/3368018/10323190/a78a74e8-6c81-11e5-803e-ac72f56aa779.png)

Ill have to delay the PR until I have some more time to explore the limitations of this feature. I'll keep you posted!


instead of triple ticks you can use single ticks to get inline code-looking font.


> I'm not quite sure what you mean by only one language. There are multiple languages in the xml, I find the one that matches the user language, and set it as the default.

You're setting the default language pre-translation - does the generated HTML form contain only one language?  I was thinking if it contained all the languages, then the current language could be set post-translation.


Good idea; I'll do that.


be nice to get a working example here.  i still don't have a good sense about the work flow we're suggesting for the user


The way the line height remains the same is very counter-intuitive, certainly a :bug: not a feature.


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


s/mutual/common/
mutual = взаимный
commоn = общий


Move the parsing logic to `GCParser` and call it from `GCLoggingManager.onLoadFinished()`


Why does this need to be out of the function anyway? Isn't it calculated in WaitSeconds, used once, and then discarded?


Remove this, it's fine to repeat this stuff here.


Yeah, I became interested in them because I work at the MNDaily, and we can add pull quotes to stories through Adobe InDesign. Granted, we use them more as "page filler" than for "emphasis", but when I read magazines, those pull quotes do serve their purpose well and get me interested in articles through a first glance.


Doesn't exist in http://www.collinsdictionary.com at least :)


Thanks @ches , lots of changes so I'm sure there's some stuff wrong as you've noticed. Would love some pull requests!


Thank you sir :)


I would say to create a function to do this since other drivers might need it. see `x86: Add PCI command configuration` in my GPIO PR.


@pkdash I am trying to make the logic (marked by {if..else..endif} blocks) in this template into 3 parts, the first part (up to line 24) deals with the button display, the second part deals with rating info display, e.g., x others +1 this, etc., and the last part shows login info for anonymous users. In the first part of logic, the internal if block distinguishes between logged in users and anonymous users and show +1/withdraw +1 button or +1 text accordingly, so I am keeping the if-else logic together in the same block to make them easier to read.


`exclude bower_components` hab ich genutzt, weil das sonst lange zum kompilieren braucht (1-2 Minuten statt 7 Sekunden), aber leider sind dann die Javascript Dateien nicht mehr da

An der Schrift habe ich nichts verändert, ist weiterhin überall Brix Slab. Da müsste man nochmal experimentieren, welche anderen Schriftarten da wie gut zu passen usw.


Why remove the ability to get the connection and game profile?


FWIW, here's how I modified my version of the file to operate with a single-level of array returned:

 if (!is_array ($result)) {
        $result = htmlentities((string)$result, ENT_QUOTES, 'utf-8', FALSE);

      } else {
        foreach ($result as $key => $value) {
          if (is_array ($value)) {
            trigger_error ("Unsupported return value from $classFile (" . $_GET['method'] . "): multi-level arrays are not supported." . print_r ($result, true));

          } else {
            $result[$key] = htmlentities((string)$value, ENT_QUOTES, 'utf-8', FALSE);
          }
        }
      }


Thanks. done


@madankb See my comments. Do not store favourite remainingCcount in Settings. Globally, take example on what's already done with `possibleLogTypes`. Good luck :)


why would you need this `mmio` variable? you can use `c_this->mmio`


А где же нагрузочный тест, сравнивающий геном человека с чьим нибудь ещё геномом? ;-)


你確實挺娘的


It's OK to get rid of the `*_str` variables and just plug the string literals directly wherever they should go in the outermost giant `vbox` in main. It becomes easy to access the text a snippet of code is surrounded by.

This is looking great in general.


Removing this would remove the ability to specify custom `ChatType`s


I've moved to using manual.html (link in Settings tab) for usage, and keeping the Readme for install procedures. But yes, it is behind. This month I have my PhD dissertation proposal and qualification exam, a seminar I'm speaking at, and I'm traveling out-of-state for a week to set up Mycodo to regulate humidity in an 80-foot underground bunker for bat research. So, my plate is kind of full at the moment.


Yes the final HTML contains both languages, eg:

```
<span class="question-label active" data-itext-id="patient_id:label" lang="en">Patient ID</span>
<span class="question-label " data-itext-id="patient_id:label" lang="fr">Identification du patient</span>
```

So we could find all `[lang="en"]` and remove the "active" class, and find all `[lang="fr"]` and add the "active" class but that feels even more hacky to me than modifying the default language in the form.


Awesome @aranhoide


Awesome! If you'd like, make this a Pull Request and I'll merge it into the addon.


porque não utf-8??


It was to make the code a little more concise, but I'll be removing it anyway when I switch to a macro-based style of accessing MMIO registers in response to the comments from @jeez.


Seems like "wakefulness" would be the most appropriate ^^


Удобно добавить ещё таргет:
```
test: autotest_compare
        ./autotest_compare
```
Тогда для сборки и запуска теста достаточно будет набрать `make test`.


Só NetunoTestConfig, porque você pode usar essa classe pra teste de integração também.


TeamCity pull requests :: netty [Build 374](https://garage.netty.io/teamcity/viewLog.html?buildId=2073&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new); tests passed: 4522, ignored: 21 Build time: 01:30:33


worst blockquote ever


ditto about `mmio` variable


No, never! `BaseType` and `BaseTypeExpression` are disjoint things!


r+


ah si alli fue que borre sin querer mucho y como tenia sueño ya no notaba donde iba cada comilla


Out of scope for this commit, but "Creates" on Line 2 is spelled wrong.


Ooops, that was me being completely distracted. Thanks for spotting it, fixed now.


Кажется, эти инклюдники (строки со 2 по 9) не нужны.


I sent a PR here: https://github.com/playframework/playframework/pull/5152

One of my projects has stopped working. I'm not sure we can just leave Play broken for people even if there were a binary incompatibility. It seems like people would be far more likely to hit the u60 issue than a binary compatibility issue with typetools which I doubt many people would be using directly in their projects. We had to update to u60 to fix another issue so we're kind of stuck between a rock a hard place.


Classe de page-object.


Blame! Sorry David I just really wanted to use this feature :stuck_out_tongue:

This here breaks the networking part of the game since the World shouldn't have to know anything about the Renderer. I've left a comment in the code suggesting a possible fix, happy to discuss alternatives with you though :smile:


Actually sounds less hacky, but also less efficient to me.

We might also consider doing this at the XSLT level.

But does it really matter?  If what you've demoed is good enough, let's stick with that.  A comment explaining the approach taken might be helpful, though.


Added in 8.0.0.r1759.g2895db6 (commit 2895db607c381fcf393e31d42b54102d466c271c)


O que faz esse pageSource? o_o'


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


merging graydon/stellar-core/catchup-retry-assert = bbc098b3 into auto


graydon/stellar-core/catchup-retry-assert = bbc098b3 merged ok, testing candidate = 6834145f


Cara, tem um monte de coisa que você pode extrair pra uma classe de page-object. Mas já melhorou muito


Если в командной строке задать только один аргумент, то в 28-й строке будет выход за границу массива `arg_vec`.


@dianafa You left $ref (line 26) which is not used anymore. I'm cleaning it up.


Тоді треба юзеру пояснити, що для продовження треба ввести 1.


POST


no need for the `build_url()` call


No performance problems detected.


Usually we use the `success` key instead of the promise syntax unless we actually need promise-like behavior

```
complete: IndicoUI.Dialogs.Util.progress(),
error: handleAjaxError,
success: function(data) {
...
}
```


r+


`input:text` is shorter and does the job
And I'd add a linebreak after `{` and before `}`


Це, як і змінна `resp`, зайве. Можна зчитати рядок просто за допомогою `scanner.nextLine();` і результат нікуди не записати


"global.next_check= global.next_check or tick+interval"

Should be:

"global.next_check= global.next_check or game.tick+interval"


It's always a GET argument -> `request.args`


@dianafa This hook's name does not make sense anymore if it's called from here. Btw. do you know where is code that is subscribing to this hook?


```use something....``` but I don't know which.


That won't overwrite the existing settings


Можна просто `System.out.println();`


Yeah, I thought I'd add in a second "order by"  if the words happen to be the same length. That was just me experimenting :)


-Also added network testing code


If I want a project where I get to look int how writing practices in history have their influence on web writing and turned into its own HTML, this would definitely be it. I have a passing interest in typography, so to I know a bit about the print side of things. Typefaces being directly transferred from something printed on paper to pixels on a screen both "preserved" (old fonts) and reinvented (really fancy computer-made fonts), that is something pretty interesting in a technology. This is kind of tangentially related to font-families, but the fact there are families, not a one single family for web writing, shows a perspective of technology that isn't so rigid.


Since you don't care about flash messages, success=True, etc just use `jsonify()` (if you use jsonify_data, use `flash=False` unless your JS code handles flash messages)


Answer to why such wierd name (`0_config`) -
Web2py runs the files lexicographically (sorted in ascending order). We want the values in the `current` object to be set before they are used anywhere else.


IDs/classes use lower-with-dashes


Oops, double "meta".


Якщо ви використовуєте тут `break`, то нащо перевіряти умову на рівність одиниці у `while`? Зробіть тоді `while (true)`, заодне і перший коментар втратить актуальність.


I'm at a total loss why this doesn't work.

The `transaction_id` and `amount` always come through empty to RCP, even I've verified that they are all correct multiple times over. Initially I thought the `$event->type` was wrong, but I've confirmed it to be correct in both instances via the web hook response to Stripe.


I feel like there could be a bit more comment here. The idea is that we will replay the node-ids from the (cached) starting point, because we (presumably) will do the same thing each time? One thing I am missing (sorry if it's obvious) is why we are lowering the same code multiple times?


@rodowi según entiendo flag es otra acción como votar


Лучше писать в `cerr`.
Потому что если программу запускает интерактивно человек, перенаправив её `stdout` в файл или поток, то ему будет непонятно что пошло не так.
Ведь он не увидит это сообщение.
А вот дальнейший пайп у него может сломаться.
Или не сломаться, что может быть ещё хуже – он будет думать что всё выполнилось как задумано.


You don't actually need to write it this way.
You can do:

"game.on_init(On_Load)" and it will be called the same way. The same applies for everywhere else you've used "function(event) someMethod(event) end" - you can just use "someMethod" directly and it will be called with event passed in.

See: https://github.com/Rseding91/Landfill/blob/master/src/control.lua#L25


since when do we align values? (it might look nice, but you'll hate it as soon as you add/remove something and end up changing many unrelated lines just to keep everything aligned)


Ok, thanks. Changed it.


`</td>` in a new line


Mojo::Message::Serializer doesn't seem correct.


Лучше `exit(1)`.
Если программу запускает скрипт или другая программа, так они смогут определить что что-то пошло не так.


I'd call it `registrant_id`


Goddammit Jordan!


- Далее есть проблема привязки к определенным тегам в DOM дереве. Нужно сделать рутовый элемент, который будет передаваться в модуль и относительно которого можно будет искать все вложенные ветки.


oh hmm maybe this is the comment I wanted :) let me read


sorry


Fixed in 4868b27044307aa4bf75d6c8642fce9c7994df37.


typo: s/is/id


Valgrind now points to this memset() as the most time consuming function when searching for 0 in zeroed memory. I'm not sure yet if it can be further optimized. Maybe a `recalloc()` meaning `calloc()` -> `memmove()` -> `free()` could be worth a try. Or this `memset()` is just called too often. Maybe @coolwanglu or @taviso knows.


In datawindow.h, there is a missing semicolon after the assert statement. Besides that, no compilation issues.


(we can also assume that HIR node ids begin from the AST ones, I guess?)


yes, thank you


ele pega o html da pagina inteira.


Why not hide pci_addr from the user? You can initialize and set the address fields inside this `quarkX1000_eth_init()` function, just like I did in my GPIO driver.


"one of the node's id"? I guess you mean the id for that node and all cloned nodes.


"this" ;)


Hm, I think a similar patch was attempted earlier and it broke something (possibly due to chroot building). We'll see how it goes.


@Decad3nce you realize this derp? all the changes in ota_from_target_files are in the wrong file, when cherry-picking you removed the symlink and added in lollipops ota_from_target_files which is vastly different than marshmallows


Fun no answer , I asked help about this more then two months ago. result , .... nada.
If now you're not happy , well provide seld a 100 % working code and realy 100 % working .

If you can't .... wel keep the remarks for you


Hahaha oh no, you caught me! Never thought you'd notice my little addition to the dependencies ;)


This is just a single commit, not the full PR: https://github.com/Wikia/app/pull/8614/files

This hook's name has been changed: https://github.com/Wikia/app/blob/dev/extensions/wikia/PortableInfobox/services/Helpers/PortableInfoboxRenderServiceHelper.php#L87


Add


It would be good to test this mechanism SOMEHOW. Perhaps we can add a `#[test]` that just lowers the AST twice for a bunch of random snippets or something?


Thanks, I didn't know... I'll remove the '?'.


@Scottmitch ouch


Hehe valeu, eu não queria transformar tudo em page-object
porque se não demoraria muito para executar os testes, pois para cada classe o spring boot iria subir o servidor uma vez. Porém irei trabalhar em uma maneira de botar eles em page-object e organizar a execução via teste suite, talvez assim eu consiga o comportamento que eu quero.


looks like `string.h` and `assert.h` includes are missing.


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1895 of 1904 tests passed)

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


typo: s/sayingt/saying


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


> I have reverted the formats.c, but even there it would have caused no problems (it would already have been 0).

Yeah I was mostly worried you still saw problems. We'd have to get to the bottom of it if that would have been the case. Also, it's better not having the extra one there because it would confuse us in the future...


I'm getting an Android Studio error complaining about the directory structure not matching the package declaration.  Also, should we change the package to match the Android app?  Currently we have:

package org.physicalweb.collection // PwoCollection
package org.physical_web.physicalweb // Physical Web app


Good catch, git just told me file mode changed.


24-29: зачем так сложно? Строку можно создать из `char *` напрямую. Не очень понимаю чего можно добиться делая это через `istringstream`.
```
string file_name1(arg_vec[1]);
```
или
```
string file_name1 = arg_vec[1];
```
(Второй вариант отличается только записью, никакого `operator=` вызываться не будет. В обоих случаях вызовется конструктор от одного аргумента.)


Like we discussed, this else if should be removed, along with the power tile and the fingerprints with 0B04 and 0702


Will this make it into other lts kernels?


Can remove zigbee.simpleMeteringPowerRefresh() + zigbee.electricMeasurementPowerRefresh()


uno strume?


Can remove zigbee.simpleMeteringPowerConfig() + zigbee.electricMeasurementPowerConfig()


possibly change to something like
```bash
is_git_repo() {
    git rev-parse --is-inside-work-tree &> /dev/null
}
```


Fixed in a26bde0.


I should have been clearer. I want to be able to do:
```
let values = [1.0, 2.0, 3.0]
let scaled = values * 2.0 // [2.0, 4.0, 6.0]
```
also
```
let values = [1.0, 2.0, 3.0]
let scaled = 2.0 * values  // [2.0, 4.0, 6.0]
```


Yes. But not really sure why the `Accept` header needs to be cleared? Isn't it empty when you new it up?


E porque você precisa pegar todo o html da página se você só vai usar 2 input e 1 botão pra fazer o login? o_o


abstrair credencial do codigo


Кажется, эти инклюдники (строки с 6 по 9) не нужны.


Looks like I went too far the other way then.
I'll go back to declaring return types. In fact, I'd type everything if I could have my way, but I'm trying to be Groovy... :wink:


fast-forwarding master to auto = 6834145f


Вы бы лучше микро-лендинг выложили в котором используете данный функционал, чтоб с первых секунд понимать "нафиг все это надо"


naaaaaaaah, vai precisar carregar não, depois te mando um exemplo


You could use an early return here `if (uip_len <= 0) return;`. In order to save some identation


wow.


Haha. Sorry you had to see that. Just cobbling examples together for the exam, and there are a bunch of details that I don't care about. And "no", seeing this won't help any of the students. :-)


'LOGIN_20'? Did you mean 'LOGIN_2'?


Was ischn das für a Haltig? "villicht schaffsches ja du?!" Entweder du machsch das Züügs recht oder du lahschs  aber di ganz Ziit am Sebi arbet ahdreia findi biz schwach!


This patch causes atom to take 5-15 seconds to startup from my home folder, depending on how many subfolders are expanded by default. There are has dozens of git repos to track.

Without this patch this package takes just 181ms.

When my home directory has a .git folder, this package takes just 265ms to start up.

Finding repos for the hundreds of paths must be very expensive.  It seems to be synchronous, too.  Need to find out how to write performance specs.


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1901 of 1910 tests passed)

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


Reverting this change as 1) we now have an exam schedule, and 2) a non-closed comment caused the Exams category to be removed entirely

The new exam schedule will be added in a separate commit by a new developer in our team (via pull request)


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


merging graydon/stellar-core/bug-376-and-772-track-publishing-in-db = 62a1f403 into auto


graydon/stellar-core/bug-376-and-772-track-publishing-in-db = 62a1f403 merged ok, testing candidate = 5f74c324


@aleph7 would I extend all three of Float, Double and Vector for the * operator.


I've changed it to null. Thinking about it, zero is a legitimate return value (it is the product of some numbers), so it would be an incorrect result for an empty or null list.


@cynthn FYI I had submitted #4685 already to address this, and also include the ARM sizes as "ExtraSmall" is not a valid machine size when using ARM (Standard_A0 needs to be used, for example).


@hyi made the changes as per your review comments


It wont duplicate `skipAllTestsOnFailure` because `skipAllTestsOnFailure` works across context

Example
with `skipRemainingTestsInContextOnFailure ` on, the way it works in the above code
- context 1
  - test 1 (pass)
  - test 2 (fail)
  - test 3 (skip)
  - test 4 (skip)

(skipNextTest is reset here)
- context 2
  - test 1 (pass)
  - test 2 (pass)
  - test 3 (fail)
  - test 4 (skip)

with `skipAllTestsOnFailure` on
- context 1
  - test 1 (pass)
  - test 2 (fail)
  - test 3 (skip)
  - test 4 (skip)

- context 2
  - test 1 (skip)
  - test 2 (skip)
  - test 3 (skip)
  - test 4 (skip)

I tested this and showed a co worker and it seems reasonable to me.  I can change it though since its the feature you are requesting.  Just make an example that illustrates what you want if it differs.


Awesome. I'll look into merging this after I'm done going over your old spaghetti code for project loading. Heh.


* TODO: if failed to restart php-fpm


hm tuet mir leid wenn das falsch überecho isch. aber ich denke mir wüssed beidi dass de sebi da meh erfahrig und know-how mitbringt als ich. darum chas ja so si dass er's schafft :)


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

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


Hi there, I see you've changed the indenting here as well, can you revert that.

Also, we actually use https://review.gerrithub.io/#/admin/projects/repaint-io/maven-tiles for reviews rather than pull requests so would be good to submit via there if you can.

    $ git push ssh://USER@review.gerrithub.io:29418/repaint-io/maven-tiles HEAD:refs/for/master

Mark


@zzlei I was hoping your figures from `vternarylogic()` would be usable as-is, and they are! Saved me a lot of thinking :+1:


Yeah let's update this and make it consistent with the filters in `templates/partials/filters.html`. Currently they look at `$scope.facilities.length` which may be sufficient but you might be able to come up with something better.


Mier hennd sus auno as logging framework wo wunderbar funktioniert, het zum biispiil en logger bim AuthenticationEntryPoint...


@hyi It's not a big issue for me. I was thinking with my suggested change it might be easier to follow the logic.
+1


@dimoschi as you are changing the bit.ly links anyways, I suppose we should just point to transifex.com URLs directly. I don't see much benefit using bit.ly in this case.


But camunda-commons-typed-values needs the reference to the joda module!
Building current master, unzipping and starting the wildfly distro I get
23:57:01,319 ERROR [org.jboss.msc.service.fail] (ServerService Thread Pool -- 54) MSC000001: Failed to start service org.camunda.bpm
.platform.process-engine.default: org.jboss.msc.service.StartException in service org.camunda.bpm.platform.process-engine.default: j
ava.lang.NoClassDefFoundError: org/joda/time/LocalDate
        at org.camunda.bpm.container.impl.jboss.service.MscManagedProcessEngineController$1.run(MscManagedProcessEngineController.ja
va:97) [camunda-wildfly-subsystem-7.4.0-SNAPSHOT.jar:7.4.0-SNAPSHOT]
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) [rt.jar:1.8.0_60]
        at java.util.concurrent.FutureTask.run(FutureTask.java:266) [rt.jar:1.8.0_60]
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142) [rt.jar:1.8.0_60]
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617) [rt.jar:1.8.0_60]
        at java.lang.Thread.run(Thread.java:745) [rt.jar:1.8.0_60]
        at org.jboss.threads.JBossThread.run(JBossThread.java:122) [jboss-threads-2.1.1.Final.jar:2.1.1.Final]
Caused by: java.lang.NoClassDefFoundError: org/joda/time/LocalDate
        at org.camunda.bpm.engine.impl.core.variable.type.PrimitiveValueTypeImpl$LocalDateTypeImpl.<init>(PrimitiveValueTypeImpl.jav
a:128) [camunda-commons-typed-values-1.2.0-alpha1.jar:1.2.0-alpha1]
        at org.camunda.bpm.engine.variable.type.ValueType.<clinit>(ValueType.java:60) [camunda-commons-typed-values-1.2.0-alpha1.jar
:1.2.0-alpha1]
        at org.camunda.bpm.engine.impl.variable.ValueTypeResolverImpl.<init>(ValueTypeResolverImpl.java:50) [camunda-engine-7.4.0-SN
APSHOT.jar:7.4.0-SNAPSHOT]
        at org.camunda.bpm.engine.impl.cfg.ProcessEngineConfigurationImpl.initValueTypeResolver(ProcessEngineConfigurationImpl.java:
1279) [camunda-engine-7.4.0-SNAPSHOT.jar:7.4.0-SNAPSHOT]
        at org.camunda.bpm.engine.impl.cfg.ProcessEngineConfigurationImpl.init(ProcessEngineConfigurationImpl.java:548) [camunda-eng
ine-7.4.0-SNAPSHOT.jar:7.4.0-SNAPSHOT]
        at org.camunda.bpm.engine.impl.cfg.JtaProcessEngineConfiguration.init(JtaProcessEngineConfiguration.java:54) [camunda-engine
-7.4.0-SNAPSHOT.jar:7.4.0-SNAPSHOT]
        at org.camunda.bpm.container.impl.jboss.config.ManagedJtaProcessEngineConfiguration.init(ManagedJtaProcessEngineConfiguratio
n.java:34) [camunda-wildfly-subsystem-7.4.0-SNAPSHOT.jar:7.4.0-SNAPSHOT]
        at org.camunda.bpm.engine.impl.cfg.ProcessEngineConfigurationImpl.buildProcessEngine(ProcessEngineConfigurationImpl.java:510
) [camunda-engine-7.4.0-SNAPSHOT.jar:7.4.0-SNAPSHOT]
        at org.camunda.bpm.container.impl.jboss.service.MscManagedProcessEngineController.startProcessEngine(MscManagedProcessEngine
Controller.java:177) [camunda-wildfly-subsystem-7.4.0-SNAPSHOT.jar:7.4.0-SNAPSHOT]
        at org.camunda.bpm.container.impl.jboss.service.MscManagedProcessEngineController$2.run(MscManagedProcessEngineController.ja
va:131) [camunda-wildfly-subsystem-7.4.0-SNAPSHOT.jar:7.4.0-SNAPSHOT]
        at org.camunda.bpm.container.impl.jboss.service.MscManagedProcessEngineController$2.run(MscManagedProcessEngineController.ja
va:129) [camunda-wildfly-subsystem-7.4.0-SNAPSHOT.jar:7.4.0-SNAPSHOT]
        at org.camunda.bpm.container.impl.jboss.util.Tccl.runWithTccl(Tccl.java:53) [camunda-wildfly-subsystem-7.4.0-SNAPSHOT.jar:7.
4.0-SNAPSHOT]
        at org.camunda.bpm.container.impl.jboss.util.Tccl.runUnderClassloader(Tccl.java:45) [camunda-wildfly-subsystem-7.4.0-SNAPSHO
T.jar:7.4.0-SNAPSHOT]
        at org.camunda.bpm.container.impl.jboss.service.MscManagedProcessEngineController.startInternal(MscManagedProcessEngineContr
oller.java:129) [camunda-wildfly-subsystem-7.4.0-SNAPSHOT.jar:7.4.0-SNAPSHOT]
        at org.camunda.bpm.container.impl.jboss.service.MscManagedProcessEngineController$1.run(MscManagedProcessEngineController.ja
va:90) [camunda-wildfly-subsystem-7.4.0-SNAPSHOT.jar:7.4.0-SNAPSHOT]
        ... 6 more
Caused by: java.lang.ClassNotFoundException: org.joda.time.LocalDate from [Module "org.camunda.commons.camunda-commons-typed-values:
main" from local module loader @736e9adb (finder: local module finder @6d21714c (roots: C:\tmp\server\wildfly-8.2.0.Final\modules,C:
\tmp\server\wildfly-8.2.0.Final\modules\system\layers\base))]
        at org.jboss.modules.ModuleClassLoader.findClass(ModuleClassLoader.java:213) [jboss-modules.jar:1.3.3.Final]
        at org.jboss.modules.ConcurrentClassLoader.performLoadClassUnchecked(ConcurrentClassLoader.java:459) [jboss-modules.jar:1.3.
3.Final]
        at org.jboss.modules.ConcurrentClassLoader.performLoadClassChecked(ConcurrentClassLoader.java:408) [jboss-modules.jar:1.3.3.
Final]
        at org.jboss.modules.ConcurrentClassLoader.performLoadClass(ConcurrentClassLoader.java:389) [jboss-modules.jar:1.3.3.Final]
        at org.jboss.modules.ConcurrentClassLoader.loadClass(ConcurrentClassLoader.java:134) [jboss-modules.jar:1.3.3.Final]
        ... 21 more


Thanks for the comprehensive reply - I've just returned from a short vacation and I see my changes are already merged. What you say makes sense and I can still make another PR to re-word the changes as it appears I misunderstood the meaning of afterEvaluate. In fact the whole option name could be renamed since it is not entirely accurate.

And, of course, solving this in our own build-scripts would be ideal however pre-empting the evaluation order in the gradle-dependency-lock-plugin was the most reliable way to catch fix the issue for our whole multi-project build; I suspect others with large multi-project builds might encouter the same issues.


Awesome :) :+1:


Note to self: If we ever want to actually enable this, we also need some way to detect that the driver is new enough to support this (it has to be CUDA 7.5 or later, be it within the OpenCL driver or not). I have yet to find something usable - nvidia does define some PP macros in CUDA but not a single one (that I could find) for OpenCL.


I haven't tried any GCC 5.x versions to see if the warning appears on them. Here is what I did later to avoid the false positive warning on the standard CM gcc: https://github.com/sultanqasim/android_kernel_motorola_msm8916/commit/fd9ea163f5fd8600caa4472664bccb5b5e039c34


Hi, just glimpsed over it and it looks awesome. Will try to test and merge today or tomorrow. Thanks the request!


A usage message here would be friendlier to future us and future users.


still an extra cstring to remove will open a new MR


The code comments/names are in English.
README is in English.
Every comment/issue is in English.
Every other commit message is in English.

This one is different.

Bel lavoro comunque


@bradspry looks like a code task JIRA ticket to me: https://github.com/Islandora/islandora_bagit/blob/7.x/CONTRIBUTING.md#report-a-bug


@alloy @orta: I could use some feedback, as I'm a bit confused as to how to test this. This just creates a gray square and verifies that snapshot - which isn't really testing much other than that the VC is created without error.


A usage message would be friendly and helpful for future us and future users.


###### В **C**, не в **C++**
Лучше единообразно, либо `return ...;`, либо `exit(...)`.
В случае `main()` это одно и то же.
Но вообще, `exit(...)` – более мощное средство, так как работает из любой функции.

###### В **C++**
Чтобы аварийно выйти в стиле C++ не из `main()` кидают исключение, а в `main()` его ловят, точнее ловят все исключения подряд.

###### ИМХО
Я бы предпочёл вариант с `return` – не нужно подключать `cstdlib` и это смотрится более по C++.
И `exit(...)` в каком-то смысле похож на `goto`, он разрывает стек вызовов функций как `goto` разрывает контекст блоков кода.
В частности, при этом не будут вызваны деструкторы локальных переменных этих функций:
```
~$ cat 1.cpp
#include <iostream>
#include <cstdlib>

class A {
public:
    A() {
        std::cerr << "A::A()\n";
    }
    ~A() {
        std::cerr << "A::~A()\n";
    }
};

void f() {
    exit(0);
}

int main() {
    A a;
    f();
    return 0;
}

~$ g++ 1.cpp -o 1
~$ ./1
A::A()
~$ sed 's@exit@//\0@' < 1.cpp > 2.cpp
~$ g++ 2.cpp -o 2
~$ ./2
A::A()
A::~A()
~$
```


The difference is that this now specifies 1.1.1 as the minimum allowed version.


This breaks on Pi1 when built without cmake. Any reason why this is now an error?


Rather than bailing out, how about selecting a default set of tests to run. As u user, I'd like to do as little setup as possible before running the script.


Что-то с отступом.


@erikPrime Is everyone on board with this change?


2nd note to self: We might get away with
```
#if SM_MAJOR >= 5 && (DEV_VER_MAJOR > 352 || (DEV_VER_MAJOR == 352 && DEV_VER_MINOR >= 39))
```


Hej :)


@codewarrior0 This `new` variable causes tracebacks when logging level is different than `logging.INFO`.


At line 207 and 210 TIP_3 and TIP_4 are mentioned, but the corresponding strings are not listed here


Looks good, I checked the paper and confirmed that your formula matches theirs.  Thanks for finding that.


I was way down in `Open3.popen3()` and `isatty(3)` before noping out of making a feature for emulating term behavior. The explicit flags are thoroughly tested, the automatic behavior is not.


hmmm it seems ofRemoveListener() doesn't work after this?


but will the benefit outweigh the cost of 7 characters?


what about testing with shared history/countermoves ? Mikael confirmed that he did thread specific history based on a few bench runs. It maybe that using original approach was better.
I'm not  comfortable enough with git to merge lazy changes into my code. else I would have tried it.


some tests failed:
failure: https://travis-ci.org/stellar/stellar-core/builds/83987989


just....
```
super(ImmutableBreakableData.class, ImmutableSet.copyOf(breakable), Keys.BreakableBlockTypes);
```


because there's a funny test in Calendar that fires 'localized' but MockL10n doesn't have it.


Never modify static obects with non static statements!!!!!!!!!

This will add the color predicate to the static filter obejct as often the card is created on the whole server. And it never will be go away until the server is restarted.


and here.


Awesome, thanks! I'll patch it really quickly.


#DobryCommit #Wow #10tysiecyNettoPotwierdzone


I feel like you're trolling me right now :)  Consumer code might want to wait for the `translateFragment` to finish. E.g. the SMS app localizes iframes with `Promise.all` over `translateFragment` calls.


It is worth noting for other users that the Jackett project allows Sonarr users to use SceneAccess:

https://github.com/zone117x/Jackett


"function type parameter" can be read as "a type parameter of the function". Maybe change to "parameter in/of function type"?


k


Commit Message of the Week :tm:


Set the texts of the effects with setText() method to the correct texts for the tooltip.
First effect:
{this}  Lancer gets +1/+0
Second effect:
and gains first strike until end of turn


Do you find that compiling takes longer after the big stagefright 2.0 fix? My pc seems to be stuck at some audio policy part for at least 30mins...


/cc @steelbrain

I thought you might want to know about this in advance, since IIRC, linter uses `instanceof TextEditor`. We're deprecating the `TextEditor` export; I'd recommend using some kind of duck typing to check if an object is a text editor:

```js
if (typeof editor.markBufferRange !== 'function') {
  throw new Error('Given editor is not really an editor')
}
```


#heheszki :dancer:


Ok for wakefulness, as for me.


102867486-FixRectangleShapeTest-5173 (mjwitte) - i386-Windows-7-VisualStudio-12: Tests Failed

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


@posita

> Ah, now there's a different problem. What if one code path is still using resource2 after another code path calls resource.close()?

resource2 will loose the internal reference. The expensive resource does not longer exist, and if you call **getExpensiveResource** it will generate a new expensive resource

> Is this always the case?

No.

> What if two higher level modules that don't know about each other are each using the NAMESPACE entity to access similar expensive resources?

> For example, what if the resource was a database connection, and two different modules were using the connection to create its own cursors and perform its own queries?

I dont think this code is similar to this scenario. But in this case, the database connection shouldnt be handled on **NAMESPACE.resource** level, maybe in **NAMESPACE** level

> Should those modules have to coordinate with each other if they want to share a single connection?

How I told before, I dont think to handle a database connection at *NAMESPACE.resource* is a good idea. For me, yes, the database connection shall to be shared, but it is not a problem to be solved at **NAMESPACE.resource** level

> Also, there are a few unanswered questions.

> From e6cb8cc#commitcomment-13580894:

>>>>>  No, it does not require browser functionality ... it will require only if you use window to declare globals
>>> How would I run the tests without using a browser or phantom?

Command line using several tools like mocha, grunt ...

>>>>> I'm not sure I understand. Are you suggesting there's no way to hide shared state from the global namespace, or are you suggesting that in this case, it is unnecessary because you've eliminated _all_ids?

There is no way without sharing a state.

>>>>> or are you suggesting that in this case, it is unnecessary because you've eliminated _all_ids?

Yes, I suggested it because on that point, I had removed (by mistake) the _all_ids dependence. But at that point, I really had a bad idea about the requirements. I think we should agree the requirements were not clear all the time ... the code by itself is very confuse .... :)


Added a comment.


language : json
built with : node.js


Indeed, Valgrind says that this `memset()` is called for almost every byte! This can't be right! Man, and I wondered why finding matches takes **tripple** the time than finding no matches at all!


Agreed. I'll update the docs and the the diagram after I deal with the
Python 2.6 issues.
On Oct 6, 2015 6:27 PM, "Lucas Meneghel Rodrigues" <notifications@github.com>
wrote:

> I think this deserves a mention in the documentation - something like
> "avocado tests are not loaded at detection stage for security reasons,
> which means bugs due to supporting libraries not installed and other subtle
> bugs may not be noticed. Please use static and care whenever possible".
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/clebergnu/avocado/commit/dbf6737e5866e8a33962ecdde411f8c19e12d14a#commitcomment-13625600>
> .
>


Added a comment


Also, I must confess I did wrong assumptions about the code several times ... I think I should ask for more clear requirements maybe


commenting!


window or global


Consider adding a test for annotation with an explicit target `@field:Ann`.


NAMESPACE


This needs to reference `TrailImage` now instead of `Trail`.


and it there is no public access


我還是照以往翻譯用法改成製作團隊了，backer也算金主，當成團隊一員


我暫時翻 "衝突戰" 了


Seems alright!
Check out the static method [String.join](https://docs.oracle.com/javase/8/docs/api/java/lang/String.html#join-java.lang.CharSequence-java.lang.Iterable-), probably familiar to you from Python.  It was added on Java8.
For instance:
```java
String.join("\n",words);
```
would have given you same results without having to use substring and replaces.

Cheers!


Needs a `goog.exportSymbol()` too


Infact, I already have a .gitignore containing:
  build/*
  *.3dsx
  *.elf
  *.smdh
  .gitignore
Thanks anyway, I'll add it! :)


handle


To be honest, I got the annotation from the Groovy implementation of collect. But I've just added some tests to my code and now I understand what it does: it is ensures the type check happens for the first parameter of the closure.


Took opportunity to change all "tab" characters to "two spaces" as per issue #600.


Looks OK to me… It would be better to use figure/figurecaption in this case but I think we can live without that until someone has time


QUnit.test( "getTotalExpensiveResourcesCreated inacessible", function( assert ) {
	  assert.ok( typeof NAMESPACE.getTotalExpensiveResourcesCreated === 'undefined', "undefined" );
);


Holy !?*@?# !!!! I just contributed to MOOSE...


White power xD :facepunch:


Addressed all issues brought up so far, awaiting review


@vadymmarkov - would you guys consider pointing at the current commit on ASDK master?  For all of the API surface area in 1.2.3, it is actually more stable (a few key bugs fixed, mainly with table / collection), and more optimized (a few pretty impactful performance improvements).  The only reason it hasn't been pushed out in an actual release tag is that some of the new-in-2.0 APIs may be renamed in the next week or two.


I temporarily changed this to dworgs.json to speed up testing


Yes I think that should also be tested separately.


[super excited to see you guys are doing some work with ASDK — would love to help if you have any needs, either questions with stuff you're implementing /or/ feature requests for the framework itself!]


I like this change to always turn on joint norm along tv dimension. But I wonder if we should dedicate a dimension for the derivatives, say dim #15 of `mri.h`, instead of secretly stashing it in dim #16.


I was trolling. Sorry for lack of semantic tags


Right! hehe.


Changed these to init to empty string, so people don't wonder if those values have a specific purpose.


Would be good for @frankong to double-check this


`agent` doesn't seem to be used.  I init'd token to an empty string.


This would be more like `Module.nesting[0].name` instead of `self.class.name`


`hostname` now uses the `config.js`, `path` defaults to root.


@ajithcj : you don't need to merge. To get lazy_smp branch from official repo:
git fetch origin
git checkout origin/lazy_smp   -b lazy_smp_official


It's getting hot in herre!


`self.class.name` will not work here because the code is outside the namespace at the top level of the Ruby environment.


I replaced `var a` with the `keepAlive` parameter in `optionsdb`


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (836 of 836 tests passed)

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


This pep8 standard makes a lot of sense, since lists (and dicts) are pass by reference, so this arg will keep state when called multiple times and you'll spend hours tracking down really odd bugs. :)


This change allows you to use the token from a registered application instead of a personal OAuth token.  OAUTH has to be passed via the header, whereas an app token is passed with the querystring.  I just commented this out for now, to make it easy to swap back if needed.


yolo


why not simply check for 'STOPPED'? That seems like its cleaner.


Not sure if `gitstats-a` was purposeful or not.  I changed it to default to the name of the db from `config.js`


random whitespace.  Only place I'll comment on this, since it pops up elsewhere.  More cleanup needed in this regard.


TeamCity pull requests :: netty [Build 377](https://garage.netty.io/teamcity/viewLog.html?buildId=2095&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5621, ignored: 21 Build time: 01:18:17


Trailing newline


" " => "-" so that it's more apparent that it's being used as a join.


New delete function.  This will delete the entire couch db.  I may add another function to bulk delete docs and call purge/compact.


@abuiles while updating the [ember-cli-scaffold](https://github.com/marcioj/ember-cli-scaffold) addon which uses an older version of ember-router-generator, I notice that when adding several routes just the last is used.
For instance:

Using

```js
    var newRoutes = routes
      .add('users', { resetNamespace: true })
      .add('users/new')
      .add('users/edit', { path: ':user_id/edit' })
      .add('users/show', { path: ':user_id' })
```

Generates

```js
Router.map(function() {
  this.route("users", {
    path: ":user_id"
  }, function() {
    this.route("show", {
      path: ":user_id"
    });
  });
});
```

Instead of

```js
Router.map(function() {
  this.route('users', { resetNamespace: true }, function() {
    this.route('new');

    this.route('edit', {
      path: ':user_id/edit'
    });

    this.route('show', {
      path: ':user_id'
    });
  });
});
```

This happens because each time the add method is used a new copy is created, but this copy always use the `this.source` which is immutable, just the `this.ast` is transformed by the visitor. I think this commit introduced this change.

Is this the expected behaviour?


I'm not sure about this. Should check whether short-circuit applies, and revert if it does.


Seems fine. Though best to verify it with an actual test in Azure.


LOL I would rather have it in the center


New create function.  If the db exists, it says so.  If it doesn't exists, it creates the db, and continues.


This has to match the behavior of Cube.

But what you're saying is also exercised in ExtendedCubeTest (with semicolon) and CubeTesterTest (with underscore)


The mods on gen_gdl90.go to the Ownship message won't work - need to convert to an integer to make this message. Would rather not do that once per second and only once per change via the interface or config load. I'll need to work on that part before merging. Looks good!


saw approval from macintux
at https://github.com/basho/riak_kv/commit/51afd8c3fa74e4c92a1166db44d078156546e3b2


merging basho/riak_kv/bugfix/bch/update-ql-repo = 51afd8c3 into borshop-integration-1208-bugfix/bch/update-ql-repo


basho/riak_kv/bugfix/bch/update-ql-repo = 51afd8c3 merged ok, testing candidate = b433990a


@cptvitamin @explorerleslie This adds the rating XBlock. I added a Studio view and attempted to fix accessibility. If this could be reviewed, we could launch it in the Dartmouth course.

Leslie: From what I understand, if this could be reviewed in time, we could also push this as a hotfix to the platform before their course launch. This is relatively easy and self-service now.


I think this should be name `exerciseGuava` as this isn't a test java file. This helps with distinguishing it from `testExerciseGuava`.


webMAN MOD 1.43.06
- Fix cache issue on PS3 with certain http commands (e.g. restart.ps3). Thanks to Dino05.
- Added delete mode to File Manager
- Added option to save plugins currently loaded to /boot_plugins.txt and other boot files.
- Installer now enable stage2.bin if it was disabled.
- Updated simplified chinesee.

src: http://aldostools.org/temp/test/webMAN_MOD_1.43.06.rar


Your code, but wrapped as a function so it isn't called until couch is ready.  I also modified line 257 to expand `user` repos, which are pretty much an `org`.


fair enough


> /* fix this url lmao */

URI


this sure is ugly >.<


should be "it brings whole IWD:EE and first part of IWD2 to work inside EET". It wouldn't make sense to bring first IWD twice :P


I added a few event listeners, should be pretty self-explanatory.  I use `once` so the listener is deleted after it is called.  By default I am calling `delete_db()`, so the environment is reset every time.  Feel free to change this to `create_db()`


This file was not formatted correctly, and one of the repos was spelled incorrectly.


You might want to apply the same fix to the other entries in UsersController.php:

                    $update_assets = DB::table('accessories_users')->whereIn('id', $accessory_array)->update(
                            array(
                                'assigned_to' => null,
                    ));
                }


I understand some of the confusion here. Maybe it should be `exerciseGuavaExtended`?


Although we don't have any dependencies, I added a `package.json` for completeness and assigned an MIT license.  We should probably add an IBM author attribution as well.


That's a good suggestion, but I'm trying to make this as similar as possible to cpu/6502/net/ethernet-drv.c.  That way anyone that is familiar with that code will be able to understand our new code more readily.


I think it is fine as exerciseGuava.  After the change it reads better anyway


Duh! You are right! So much for a quick fix. Goid thing we have pull requests that allow for peer reviews.


Sure, I can do that.


Good idea.


Try writing this as `for d in (commandline -ct)*/`.  The trailing slash should make it match only directories, which should be faster and eliminate the need for the `-d` test below. The `-x` test is still appropriate I think.


Self.class.name should also return CommunityExtensions::STL::Exporter...
Doesn't It?


Sorry, it's a mistake ...


Please update this comment.


It doesn't really matter what the value is since mutability currently only matters if there are parent fields, and there are trivially no parent fields in this case.


Code tested and working after I modified in my current install (v2.0-95)

:-)


Oops, typo, oh well


`removeAdmins` doesn't exist, so this throws and dumps a message in the debug console instead of displaying the changes message


likewise here and two more


Fe de erratas: El nombre correcto del commit es: ""Utilizando HTTP 1.1 y cambio de tiempo de vibración a 20 ms"


Nice touch :)


:boat:


twas sitting around for quite some time waiting for suggestions other than the flag and exceptions.


Need review by @rustd.


`TPOS` should be the disc number. See http://id3.org/id3v2.3.0 and the [Picard tag mappings](https://picard.musicbrainz.org/docs/mappings/).

> TPOS
> The 'Part of a set' frame is a numeric string that describes which part of a set the audio came from. This frame is used if the source described in the "TALB" frame is divided into several mediums, e.g. a double CD. The value may be extended with a "/" character and a numeric string containing the total number of parts in the set. E.g. "1/2".


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


Done.


Where's the branch for this?


Maybe you could add `UNHEX(REPLACE(UUID(),'-',''))` for `BINARY(16)` columns?


Yep, that should do it.


This is intentional, to show there is no `.build()` and/or `.evolve()`.


`self` is the module `CommunityExtensions::STL::Exporter`. self.class is `Module`, and `self.class.name is the string`"Module"`


OK, I will track these enabled and disabled events as `LocalCommand`s, using stuff I built to handle the reset engines.


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


Nice catch. Fix pushed.


@bbcrddave ... not sure I agree.  It will not harm the manifest timing sources if they are proper and already have https if needed and will prevent downgrade for user that may not get it BUT I can go either way on the manifest based items.   Now for the default and added time sources it is critical to match the protocol as per the many reports of this issue.

I can easily modify this if we come to some agreement.


Changed it to avoid collision with default shortcut.


fsdfs


Added compton to autostart.


Let me bring this to the attention of the group


Script for updating menu cache.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2468


Script for keeping screen on (when using xscreensaver) when watching fullscren flash videos, vlc etc.


It could be useful to add a comment here, something to the effect of:
```js
// Adds attributes missing from lib.d.ts
// See https://www.khronos.org/registry/webgl/specs/latest/1.0/index.html#WEBGLCONTEXTATTRIBUTES
```

So that it's clear that there are more properties defined in lib.d.ts.


This call to GetTypeInfo() must be added manually each time the code is generated (known issue).


Ach... I should know better than to squeeze a hotfix between busy hours...

On Wed, 7 Oct 2015 01:11 Jim Foltz <notifications@github.com> wrote:

> self is the module CommunityExtensions::STL::Exporter. self.class is
> Module, and self.class.name is the string"Module"`
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/thomthom/sketchup-stl/commit/392aef10d814ed3b5df046389c5e0922f0086ffb#commitcomment-13627324>
> .
>


Could I be referred to `Shane S. Anderson` instead (with middle initial)?


Try to keep the styling in the CSS file. It's best practice to 'separate the concerns' and keep the HTML file for content and CSS for styling.


Line 138: Should isFirst be (i==0) rather than (i==1)?


han barcode4j zum laufe bracht. das teil isch aber völlig veraltet und brucht mega viel dependencies (teilwies au sache wo gar nüme gwartet werded und scho anen neue ort gmoved worde sind). das isch darum ned ideal.

am beste packsch das bild wo du generiersch ines base64 bild und übergisches im XML. darstelle cha xsl-fo das ganz eifach. de dave het usserdem en library wo der en QR code als bild mit 1nere zile erstellt. das chasch denn zumene base64 umwandle und fertig.


Are you going to release the new version to CPAN?


This looks okay.


LTGM


<img width="564" alt="screen shot 2015-10-05 at 2 52 41 pm" src="https://cloud.githubusercontent.com/assets/14878809/10325422/6687bdc6-6c4e-11e5-98cd-a25c10f9ccb2.png">


looks good to me


Completed 10/6 Alex Senko, Lauren Pope


Cool, let me know how it goes after some time. We had it running on the
client site from yesterday and it was good.

Denis

On Wed, Oct 7, 2015 at 7:25 AM, Justin Ramos <notifications@github.com>
wrote:

> Woo hoo! I got it working. Thanks for this!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DigdeepDigital/snowplow/commit/31cc095de07167ca3b6023bc70571bad8d99ee19#commitcomment-13624321>
> .
>

--
Denis Mikhalkin | CTO | Digdeep Digital <http://www.digdeepdigital.com.au/> |
0421 171 928


Got it, nice addition!


tutorial for prework


looking good! :+1:


wow, this is weird, I was doing something similar today with areas of 3d cubes ( in C++ of course)


:imp: hihihihi


Have you tried several cycles of dragging both gutters in the three-pane example left and right?

When I do that, the right edge of the right pane creeps gradually to the left.

Don't know whether this is due to the isFirst i==0 vs i==1 issue or something else.


This commit looks good, I don't see any problems.


take a look!   in theory

Flex may not supported on explorer  9.

http://caniuse.com/#search=flex


3 crashes after 53 games


In most cases you'd want to try and keep it to one html tag per line. You can put the opening and closing tag in the same line, but when there are more I'd split it up. It helps with readability and helps to try and avoid errors where you accidentally close a tag 2 times, or forget to close it. Here, you closed the `<i>` tag twice :)


LGTM


I really like that you have a landing page and put in 2 separate links for your portfolio and your 'about me' page. Very nice!


I tested ur previous solution, on the pr page and everything worked smoothly ^^


:angel: bebok!


More specifically, in the three-pane example:

Call the two gutters GL and GR, for left and right.

From initial position, drag GR to the left. The right edge of the right pane move left. Holding the mouse button down, moving the mouse back and forth, the right edge does not move more, but releasing the mouse button, re-grabbing and dragging left again makes the right edge move left again.


After trying to open passworded zip there is Opening file... popup, and then nothing.

Messages log, nothing related or fancy (I think):
...
151007:012706 luajit: E lipc:ds::Duplicate service name (com.github.koreader.activityindicator)
151007:012706 luajit: C def:dbusi::Failed to init dbus
...


Слова "минут", "часов", "дней" необходимо склонять


This is wrong, is it not? We no longer fall back to "Linux"?


Ainda esta faltando o exercício 5 do slide 4. Sera postada o mais breve possível.


use jCommander.usage


If it works as described above, it will be perfect - that's exactly what I want. Thanks very much for your effort


There are other tags for disc number, e.g. `DISCNUMBER` in Vorbis Comment. The [Picard tag mappings](https://picard.musicbrainz.org/docs/mappings/) is a really useful reference for making sense of this mess.

It might be a good idea to have an API in libgroove for reading at least the most commonly used tags (artist, album, track, disc, etc.) without worrying about the bizarre naming of ID3 tags or how 'disc' is spelled with a 'c' in APEv2 but with a 'k' in MP4. TagLib does this.


Don't use File, it is not portable. Use e.g. [Class#getResourceAsStream](http://docs.oracle.com/javase/7/docs/api/java/lang/Class.html#getResourceAsStream(java.lang.String))


если твит сделан час назад, то нужно печатать "1 hour ago"


@wlbragg

My friend.
Sorry by lating it so much. I was out (everybody needs some money, so i was working).

As soon i arrived i home again, i done a closed look on all.

Removed some duplicated verts, made some tweaks, give smooth to some parts, remapped some others, etc, etc, etc.

The result is this :

![image](https://cloud.githubusercontent.com/assets/10703842/10325734/6f891da4-6c6a-11e5-8116-7f467d80a183.png)

Look. you don't need to use this version. It is only my choice and was made very fast.

Now, how to push, bro ?

I am into the "skis" branch. I done

git checkout skis
git pull

And then, done my changes.

How can i push ?

If i do a "git push", i says i don't have permission to do it.

If you prefer, i can put all on gdrive.

Cheers.


I'll stop the test if it keeps happening again.


It might already do this. At least, the ffmpeg API claims to do this.


You are courageous and enterprising. Good luck !


python3_3 has been announced for deprecation, please don't add it to new ebuilds


Addressed review comments.


you should declare `dirname` a local variable


Nice one.

So the `-clean` parameter will make chocolatey think it failed? Not sure if you should include it then at all. From the package managers perspective I'd not allow a package that could crash me, even if it is optional.


This comment got really messed up


@nick22891 could you change the Label from `Cell #` to `Contact Number`?


:cat2: approves


Use of 'get-down' here is limiting deployment due to the default options array in the 'get-down' call to the node 'request' package (see https://github.com/tschaub/get-down/blob/master/lib/bower-util/download.js#L44). Specifically, environment preferences for CA's, proxies, and SSL verification are not considered. I would strongly consider a refactor if you'd like openlayers to disseminate.


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1374 of 1374 tests passed)

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


Thanks @allanhicks for fixing sloppy mistake caused by temporary variable left in workspace


looks like `src_install()` can be omitted, since the defaults pretty much do the same afais


Commenting out this `memset()` shows that it consumes 9% of the overall scan time.


this looks like it needs a revbump


I prefer `??` for such things, but whatevs.


Looks good.


closes #109


Hi @kellijohnson
I've tried to replace the use of paste in directories with file.path, but obviously missed this one. Do you see any reason why the new block of code on lines 33-37 couldn't be replaced by the line below?
`outfile <- file.path(dir, file)`


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


@wlbragg easy enough to test, i guess... take a roller-like object, a drinking straw for instance, and tape a flat piece of cardboard to it that is similar shaped to the skis... try to allow for different positioning of the cardboard on the straw so that it may be balanced, nose-heavy or tail-heavy... then hold it in front of a fan or blower so that it gets wind like it would when flying...

i'm guessing that up to a certain angle, the tail will be pushed up bringing the front back down... but you are right, it may be a battle of forces and the bungee is needed to pull it up and the cable to hold it so that it doesn't go vertically flat to the wind... depending on the balance point, of course...


Good job. Using year limits the results to one column.

I recommend always using ['year'] instead of .year for a selector. There are a couple reasons.
1) Someone always names a column with a space in it - this breaks the dot method
2) If you have a column named `count` or another reserved python function - this will also break


var shoppinglist: [String]

// shoppinglist = []


You are relying on value_counts to produce the right order (which it does in this case). If you wanted to do the least killings or more advanced source, you need to do this a different way.

```
state_killings = killings[['state', 'armed']].groupby('state').count()
state_killings.sort('armed', ascending=False).head(5)
```


gayet  güzel olmuş kuyruk artık daha okunaklı ve anlaşılır :D


These seven lines change them by this
```python
                line_ids += invoice_ids.invoice_line.mapped('id')
                lot_ids = lot_obj.\
                    search([('invoice_line_id', 'in', line_ids)])
                lot_ids = lot_ids.mapped('id')
                element_searched = lot_ids
```


FreeBSD's bsd.prog.mk has

    ${OBJS}: ${SRCS:M*.h}

which makes every object file depend on any .h files listed in SRCS.  I'm guessing NetBSD's bsd.prog.mk doesn't.

And yes, the dependency I put in there explicitly is also unnecessary in FreeBSD for the same reason.  Just a case of not thinking it through.


'lines_id': [(6, 0, line_ids)]   by  'lines_id': [(6, 0, lot_ids)]
'lines_list_id': [('id', 'in', line_ids)] by 'lines_list_id': [('id', 'in', lot_ids)]


Nice, although - you don;t need the second ascending (you are sorting names alphabetically - if there is a tie for median. Actually, that's a nice touch - but it may be the default.


TODO: this is hardcoded to make it work with docker


nice plot.


I'd split in the middle of the function arguments.

Also while I'm looking at this, you don't need parentheses around something which is being typecasted.


outstanding. The `as_index = False` is a great practice.

one pandas tip for doing multiple aggregations:
```
majors[['Unemployment_rate', 'Major_category']].groupby('Major_category', as_index = False).agg([np.mean, np.median, np.var])
```