@ChrisKitching check this code http://forums.faforever.com/forums//viewtopic.php?f=42&t=9946&p=100090#p100090
I think it could be good replacement
tested, works, just need to be set for earch faction in each mission to give just enough resources.


Why the bang?


94639630-HVACTemplateScheduleType-4860 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1650 of 1651 tests passed)

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


I agree, good catch!


nit: it's 2015 now :)


nit: I don't think you need the `()` parentheses around `self.border_padding`.


We will have to find a solution for right-to-left languages, but the css link shouldn't be in here as it interferes with the parsing of that file.

and also: indentation


mmm. This only works because you override it... SslCredentials will fail because they are channel only creds so I don't think that we should put these as an example here (and we should probably have some doc in the header to mark these are channel only creds).

In order to test this, I think you should get another IAM creds with a different value (or wait for the new Oauth2AccessToken creds that we will add at some point).


The code is incorrect. I'll submit a fix. Thanks for finding!


Hmm. I missed that. I will fix it. Thanks.

 As a side note, we are giving some final touches to our API as part of making 3.0 release. This involved changing a bunch of enums and making all inputs const yesterday. I am sorry for causing any extra work in the project you are doing.


ORM,


@Coneko bit rotted you a tiny bit with ce5f4b696015cbcfe5cdedf14f399732afbab417


1836664-ReturnWaterTempReset (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1665 of 1665 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)


username will not always be `nsroot` though that is the default admin name.

Perhaps it should mention that it is the certname of the netscaler device, not the proxy node or something? Or simply `[<CERTNAME FOR DEVICE>]`


See http://logs.glob.uno/?c=mozilla%23pdfjs#c33833. If we are going to enable this some time, then this hack is also not the right way to do it. Hopefully by then no hacks will be needed, and otherwise we can find a way to integrate them in the least invasive way.


The parameter acted as suffix in `feldsparBuilder` which I think is/was the name that got printed in the report. The reason I used the suffix was to get the "baseline" as first alternative and variations of the benchmark afterwards. The reason the suffix was used as prefix in the plugin configuration (if that was the place) was that I couldn't figure out how to use it as a suffix there since the source file was just "matmul" and the suffix was reflecting different compilation options.


@alexblair1 DWMR (done with my review). Press on!


2.1 won't perform any conversion, which...


Can you also test when multiple comma-separated hostnames/IPs are input from the command line?


`--with-fingerprint` is double


Nice!


Although your comments are great throughout your code, It might be helpful to your future self if you write a one or two sentence comment stating the purpose of each fragment.


add () after method names


looks good.


code conventions which the writer should follow


@sebmck FYI


nice localiSation bro!


This script service is supposed to be internal, it's not public yet. Please revert this change.


Oeps! Bedankt :)


as it should, the author
comma for readability (zeikfactortje)


your comments and your code structure make your code very readable!


After seeing other of occurrences of `_` , it's better to leave it now.


Graag gedaan!


Vacek, is this blowing up becaus self.inspect does not have a state?

Why not check.

if self.inspect and "state" in self.inspect and self.inspect["State"]["Running"]:
              self.d.stop(self.name)

Would that fix the problem you are seeing?


Direct fronting always goes through HTTPS.  But you need to request an HTTP URL.  If you request a 'https://' url you double up on TLS and it errors.  See [this comment](https://github.com/getlantern/lantern/blob/valencia/src/github.com/getlantern/fronted/dialer.go#L54).


objects, meervoud volgens mij


admin:admin is an F5 thing; we'll want to use `nsroot:<password>` or something similar.


done.


You should double check if the function set is the same in espes and my branch. I suspect that glo_get_extension_proc only exists in my branch. Wether we add it to all gl/ files or remove it from GLX is up for discussion. Personally I'd say adding it would make the glo code more versatile, removing the extension proc code would result in better maintainability.


This sentence is awkward. "To begin" what?


That's somewhat inefficient. NSString is really slow and upper/lower case are used a lot in some code (all case insensitive code uses one or ther other)...
I would suggest checking first for two cases:
The String is US-ASCII, the case is already the same. Assuming the former, you can use a faster algorithm without the NSString which will be way faster... Assuming the latter just return this (which is also important for Java SE compatibility).
Furthermore, if you do end up allocating an NSString might as well keep it as the NSString matched to that String stored in the nsString field for efficient conversion.


I'm not using border-right or border-left. I think this one's okay?


your .gitignore file is very organized!


I would say that we ALWAYS recommend using roles and profiles for anything that is not a proof-of-concept, such as the above example.


do you mean as long as it's not None? If so, we could chop "to a non-default value" I think.


overall great code!! I'm going to try to pull your files so i can test your app :+1:


Should the version of the plugin be 1.2.0?


Re-syncing an updated iteration of trouble.js that has been linted with JSHint [14 May 2015]. Thanks.


PR here: https://github.com/chef/chef-dk/pull/396 \cc @cburroughs


Misspelled canola


Apologies, I dropped the ball on this. I should have retracted my PR.

1. This patch seems to cause trims to hit the positive limit (10 degrees) some of the time even though a ::printf() in calibrate_accel prints a correct value that is within limits. I need to set a watchpoint in GDB and debug this...
2. Offsets should be subtracted, not added.


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

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


Okay, so i am trying this code but it does not work. It seems like we either might have some issues in state change logic or have to add some more datastructure in gstplayer to do the synchronization. I mean may be set a variable and then check if its set in state_change_cb and then do something over there. I have tried both change_state and setting element state but it did not work. Any tips ?

```c
   /* save the state and position */
 386   position = gst_player_get_position (self);
 387   current_state = self->app_state;
 388
 389   GST_DEBUG_OBJECT (self, "begin state=%s position=%" GST_TIME_FORMAT,
 390       gst_player_state_get_name (current_state), GST_TIME_ARGS(position));
 391
 392   gst_player_stop_internal (self);
 393
 394   g_mutex_lock (&self->lock);
 395
 396   GST_DEBUG_OBJECT (self, "Changing SUBURI to '%s'", GST_STR_NULL (self->suburi));
 397
 398   g_object_set (self->playbin, "uri", self->uri, NULL);
 399   g_object_set (self->playbin, "suburi", self->suburi, NULL);
 400
 401   g_mutex_unlock (&self->lock);
 402
 403   /* restore state and position */
 404   gst_player_seek (self, position);
 405   change_state (self, current_state);
 406   ret = gst_element_set_state (self->playbin, current_state);
 407
 408   GST_DEBUG_OBJECT (self, "ret=%d", ret);
 409   GST_DEBUG_OBJECT (self, "done state=%s position=%" GST_TIME_FORMAT,
 410       gst_player_state_get_name (current_state), GST_TIME_ARGS(position));
 411
```

0:00:27.822182185  4886       0xdc0cd0 DEBUG             gst-player gstplayer.c:390:gst_player_set_suburi_internal:<player0> begin state=paused position=0:00:01.956766000
0:00:27.822276922  4886       0xdc0cd0 DEBUG             gst-player gstplayer.c:2385:gst_player_stop_internal:<player0> Stop
0:00:27.975931691  4886       0xdc0cd0 DEBUG             gst-player gstplayer.c:1893:gst_player_streams_info_create:<player0> n-audio: 0
0:00:27.976136140  4886       0xdc0cd0 DEBUG             gst-player gstplayer.c:1893:gst_player_streams_info_create:<player0> n-video: 0
0:00:27.976631190  4886       0xdc0cd0 DEBUG             gst-player gstplayer.c:396:gst_player_set_suburi_internal:<player0> Changing SUBURI to 'file:///home/brijesh/gst-player/sintel_swe.srt'
0:00:27.976722527  4886       0xdc0cd0 DEBUG             gst-player gstplayer.c:408:gst_player_set_suburi_internal:<player0> ret=1
0:00:27.976754323  4886       0xdc0cd0 DEBUG             gst-player gstplayer.c:410:gst_player_set_suburi_internal:<player0> done state=paused position=0:00:01.956766000


What about `main`? Something like `Category.main`.


remover aqui também


Hey Alex, our presentation is at 1:00, I assume it is in Professor Root's
office, GMCS 531. I might make a tweak or two to the code, but it will just
be a clean-up

On Wed, May 13, 2015 at 10:48 AM, Alex <notifications@github.com> wrote:

> Not had the chance to work on actual animations, but right now it should be
> fine.
>
>
> Alex Anderson
>
> <http://theunknownartisthour.com>
> Listen In! Take Your Pick:
> Live365 <http://www.live365.com/stations/unknownartisthour>
> TuneIn <http://tunein.com/radio/The-Unknown-Artist-Hour-s241134/>
>
>
> On Wed, May 13, 2015 at 9:14 AM, David Edwards <notifications@github.com>
> wrote:
>
> > Any more additions to this or is it in the final state already???
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/theunknownartisthour/PuzzleSolver/commit/9010566b4e68ef04035b486ad9ccecf4f1e42652#commitcomment-11179599
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/theunknownartisthour/PuzzleSolver/commit/9010566b4e68ef04035b486ad9ccecf4f1e42652#commitcomment-11181346>
> .
>


@MrCrayfish yeah LOL I was too lazy to make anything quicker


You missed adding these lines to the vignette:

```
vignette: >
  %\VignetteIndexEntry{SpeClust R package}
  %\VignetteEngine{knitr::rmarkdown}
  \usepackage[utf8](inputenc)
```

Also, I'm getting an warning in both `R CMD build` and `R CMD check`, I think from the unicode characters in the new line 72 above (`x = c( "Marine Le Pen...")`).

It might have been easier to start by merging my changes (in the pull request).


> Direct fronting always goes through HTTPS

Right -- along with all domain fronting!

> But you need to request an HTTP URL.

> But you need to request an HTTP URL. If you request a 'https://' url you double up on TLS and it errors.

OK, I just don't understand why you're saying that. It's possible our code somewhere tries to "double up on TLS" but direct fronting itself is actually agnostic to this -- i.e. you're just setting the host header, which doesn't include the protocol at all.


Do you know about [pdb]()? I would put this line here:

```
import pdb; pdb.set_trace()
```


um, yeah, duh!?


Sorry ... https://docs.python.org/2/library/pdb.html. It's like gdb, gives you an interactive prompt where you can inspect state.


:lollipop:


Indentation + typo Ok
Please review


Then I target a specific test like so:

```
[gratipay] $ envtest tests/py/test_billing_payday.py -k payout_can
```


really?! brasil.gov.agenda uses the same approach (I learned it there)


It would probably be best to set up the repository so that it only includes the source files and solution/project files (and, later, any images/scripts that might be included). All the other files are going to be generated anyway when we compile locally.

I did this with one of my past projects to keep it clean and lean: https://github.com/ganonscrub/ghetto-snake


Commit of ROW I/O (ported from laufersteppenwolf)


Then I'd [read the docs](https://mock.readthedocs.org/en/latest/mock.html) and play with the API interactively to find what I need.


Ah, like [pry](http://pryrepl.org/) for ruby :)

That was very helpful, fixed.


Questa versione non va bene. Il formato POD non è interpretato automaticamente da GitHub, se vedi nella home page del tuo fork il README che viene mostrato è veramente brutto. A questo punto ci teniamo quello che c'è ora...


No `token=` here


Missing `token=`?


Add sync url
Add install instructions


h


1836664-ReturnWaterTempReset (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1665 of 1665 tests passed)

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


I didn't know.. we need to review these codes too:
https://github.com/collective/collective.cover/search?utf8=%E2%9C%93&q=getFolderContents
https://github.com/plonegovbr/brasil.gov.tiles/search?utf8=%E2%9C%93&q=getFolderContents


Questo tipo di configurazioni va inserito all'interno del file di configurazione, in una sezione specifica per questo plugin. Puoi vedere il plugin Authen per un esempio (c'è anche una sezione nel file di esempio)


will do
On 15 May 2015 5:29 pm, "Lars Andersen" <notifications@github.com> wrote:

> I doubt the problem can be found with just the namespace. The map with
> :suffix and :prefix is created by collecting symbols in the contents of
> the file.
>
> Can't you just debug this namespace and find the actual cause of the
> problem?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/clojure-emacs/refactor-nrepl/commit/a0c3047ee5120e18e8ddc4de76c6eb40a15bdbaf#commitcomment-11217490>
> .
>


AFAIK for brain it is a text property and for the object it is a method..


should be "pushing"


Cool!

> On 15 May 2015, at 18:03, Vincent Velociter <notifications@github.com> wrote:
>
> Yep :) also there is maybe another thing to do: check that variable is false instead of undefined... thus we can forget to declare a feature. I'm doing that right now
>
> —
> Reply to this email directly or view it on GitHub.
>


Se deve essere un plugin utile probabilmente la definizione non dovrebbe essere così


because it is a brain :)


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2126 of 2200 tests passed)

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


Dovremmo cercare di avere codice pulito.


I think this will give an exception


Do you have a working swift setup? Would love to find a way to make the cohabitive


Why don't you just create an originalUrl?


@cahrens Here we go, my first Jasmine test. Please review if you could when you get a chance.


Personalmente indento a 3 caratteri utilizzando solo spazi. Dovremmo adottare una policy consistente per tutti i file nel progetto


I think you can drop the parens here.


Shouldn't that be made global, in case there's more then 1 dot?

```
text.replace(/\./g, '%2E');
```


With this manjaro-tools needs "docbook2x" in makedepends


It's blowing up because if you have `--rm` in label `RUN` then the container does not exist when you run `atomic stop`. But atomic calls

```
self.inspect = self._inspect_container()
```
Which leaves `self.inspect` empty and then `assert` in `_get_args` kicks in.

i.e. the problem is I need `atomic` to load labels from image instead of container. (That would be actually easiest how to do that and just fix the check for existing container at the end somehow)


LGTM. I'd drop the optional parens, but if there's a styleguide you're trying to follow, just do whatever that says instead.


88 is WAY to high.  Even the threshold for float64 is well below that.  The cutoff point is where we identify that the function transitions into the y = x regime, not where it starts producing inf.

I'll admit I chose pessimistly by assuming the computation is done in float16 since we have no guarantee that the op will be fusioned.  Also, using float16 is usually because we do not mind a slight loss of precision so I don't see the point of going overboard by extracting every last bit we can from the fact that computation is still done in float32.


A meno che non si parli di oggetti, "$faq" è lo stesso di $faq ma con un po' più di elaborazione


got it, should be fixed in next draft


1836664-ReturnWaterTempReset (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1665 of 1665 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)


@cuddlestmonkey I must admit, I don't know.


Looks like there is second `updatePushMessageInformation()` call in this `collectMetrics` method: https://github.com/lfryc/aerogear-unifiedpush-server/blob/service-reuse-and-batch-size/push/sender/src/main/java/org/jboss/aerogear/unifiedpush/message/MetricsCollector.java#L63


ibid


good catch


Ahh, you're right – we need to increment *after* the error is thrown, not before. Good catch!

At least the way I run it, I keep calling the function, but I catch the error and log it (once every 5 seconds, so as to not overload the logs).

FYI, this is very similar to how the Java client works: https://github.com/aphyr/riemann-java-client/blob/ea9518870fa082f3f2694ac0b49bd60e90a60d76/src/main/java/com/aphyr/riemann/client/TcpTransport.java#L268-L304


sounds good!


This is for readability and to avoid mangling the case of already typed values.


Not sure `originalUrl` is the right name*, but I think your main point is that we introduce two new keys and leave the old key as is?

I think thats fair, though we will start to get messy if this is how we do versioning.

* The value is meant to be what we link to, which isn't necessarily the original url programmed into the beacon, not the url that's actually on the beacon. (e.g. we probably strip goo.gl, we may introduce our own redirect..)


I think i should go to the doctor because something is definitely wrong with me :pensive:
Thanks anyway. I will fix this tomorrow.


It was recently removed. There was a need way back in the day to have this due to PDAL's design, but this is no longer needed. It should mean that the patches can now vary in size and PDAL won't care. I don't know if pgpointcloud cares about the patch size varying though.


Credo che qui sia $! anziché #!. Lascerei anche uno spazio dopo i due punti.


Curious how this translated into that bug.. how did you figure it out?


Mi sembra interessante, non sono entrato nella "business logic". Probabilmente dovresti anche supportare una versione in cui i messaggi vengono inviati in privato - magari non voglio far sapere che sto cercando una cosa.


I discusssed this with @lamblin. Now we both think it is better to use the float32 threshold. With the elemwise fusion, my middle ground would cause lost of precision. But the threshold is there just for speed. On the GPU, it probably won't help due to the wrapsize not being 1. So as this probably don't speed up anything and could loose precision, we should be better with float32 threshold.

If you agree, update the code and document this.

When the hardware support float16 computation, we can make a mechanism similar to acc_dtype for such stuff.


Suggestions for name: `linkUrl`, `openUrl`, `pageUrl`..

I think I like `linkUrl`


The new version is OK for me.


lol wut?


 я знаю что такое md5...что ты мне шлешь за пароли?

Четверг, 14 мая 2015, 14:29 -07:00 от Vitaly Pestov <notifications@github.com>:
>md5 это сила.
>—
>Reply to this email directly or  view it on GitHub .


and you should use atan2f() instead of atan2() :)


Yes,  I meant our current implementation of direct fronting, that is, the http.Client returned by fronted.Dialer.DirectHttpClient requires an http:// URL.


Just like regular commands, but in object of array. Ex:
&!death = {
    'count': 10
}


@h-kataria You left in some debug logging.  cc @dclarizio


OK gotcha. I'll look at that, as to me that sounds like direct fronting isn't actually working like it's supposed to.


Does istanbul not work for child processes? If it doesn't write to stdout (and if it does, should be easy to fix), can't tap run tests as `istanbul cover test-blah.js` instead of `node test-blah.js`?

I did some looking at istanbul, it looks like it can be made to write its results to a json file in a format so that the outputs of multiple runs can be combined to get overall coverage, so its close to what we need from that point of view.

/cc @rmg


Der Teil des Codes wird derzeit nicht benutzt.


Looks like you get pegged for introducing this clever code ;)


Das mit dem House als Subscriber und der Mailbox als Subscriber ist jetz etwas redundant. Ich hatte es mir so gedacht das über das House abgefragt wird ob es sich nun um einen Subscriber handelt. Umgekehrt wär aber vielleicht sogar besser.

Momentan ist halt das Problem, dass man mit dem standard CollisionTask keine Kollision mit der Mailbox feststellen kann. Dazu werd ich mir morgen was einfallen lassen. Eventuell ein speziellen Task (wär am simplesten aber halt auch wieder eine neue Klasse) oder House und Mailbox aufsplitten und den standard Task nehmen (dann muss nochmal der LevelGenerator überarbeitet werden).


> O Ionic SDK é um projeto Open Source gratuito usado no desenvolvimento de aplicativos híbridos para as  plataformas Android 4.4+ e iOS 6+ com suporte parcial Windows Phone. Ionic SDK utiliza os recursos do HTML, CSS, Javascript... continua


LGTM


g_VideoReferenceClock.GetFrequency() is not the time base for nowtime. use inOutputTime->videoTimeScale


The new pull request is at https://github.com/google/personfinder/pull/51/commits

I tried to delete this one and I guess failed at that. I've fixed the comments in the new one.


Code by subogero improving my previous attempt to fix this issue (but I fixed one typo in the U2B::playback line).


Looks great!


This doesn't use border-right and instead uses border, so I think this one's okay.


my system segfaults here. I think this should not be called during switch of refresh rate.


Actually, it wasn't this line that was causing the bug. It was the change below this one. The problem was that there was a slight race condition when the progress of the upload would actually hit 100 right before the actual s3 upload had completed. This was something in slingshot where the uploader's status gets set to "done" when s3 is officially ready but the progress may be set to 100% slightly sooner than s3 is ready. So instead of checking progress, I checked the status of the upload which ensures s3 is indeed ready for me to download the image.


Yes. However, incorrect identification of the version led to a crash later which is fixed by reordering these (I don't have an exact error message in front of me at the moment).


I benchmarked this implementation on pure ascii text inputs and found the following:

On -O0 (debug) the native implementation is 10 times faster than the previous (pure java) implementation.
On -O3 (release) the native implementation is 3 times faster than the previous (pure java) implementation.

This is likely due to the fact that for loops are so slow on the new VM.


Nice ! I had open a ticket to take care of that later. What happens on AMD proc or if it doesn't have RDRAND ?


Do you need to explicitly add READ_EXTERNAL_STORAGE (typo in your code)?  An app that includes WRITE_EXTERNAL_STORAGE is implicitly granted the READ permission.  This permission is enforced starting in API level 19. Before API level 19, this permission is not enforced and all apps still have access to read from external storage.

[See this](http://developer.android.com/reference/android/Manifest.permission.html#READ_EXTERNAL_STORAGE)


TeamCity xBIM :: Xbim Geometry :: Release [Build 61](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1630&buildTypeId=XBIM_XbimGeometry_Release) is now running


Isn't it a duplicate of what we have here? https://github.com/karamelchef/karamel/blob/master/karamel-core/src/main/java/se/kth/karamel/common/SshKeyService.java#L95


?!

All typos I've made previously has been reverted :(


new config


A comment on what each timeout is for would be nice.


Users


Should we be checking the output to see if we timed out?


I see you fixed the typo :-)


What is this? I fixed this issue here, there is no need to say that: https://github.com/hephaest0s/usbkill/pull/59


Commit


@hvelarde I need your opinion on these implementations


@popcornmix issue with subtitle missing persists.
Is this fix in in your TODO list for Kodi 15?
I didn't find any related ticket in trac.kodi.tv.
This issue is really old ( http://forum.kodi.tv/showthread.php?tid=143824 )
thanks!


Bug: Cabinets and DoorBells not rotate


Do we need controller and a directive?


@RedstoneGunMade Not really a bug, I just didn't make them rotatable, but I'll do it ;)


It was better before


What is this mean?


nice work!


we typically only backport bug fixes to maintenance branches.


teste git hub pages


I wasn't advocating moving this back into Java, just doing that in C instead of Objective-C.
BTW I think the original issue had nothing to do with locale, I think toUpper might have been hiding a worse bug in the VM.


Mener du at deltaet er for stort fordi forrige bilde ble tegnet opp tidligere enn oppdateringen fra serveren? Det kan fikses ved å sette time når man mottar oppdateringer. Eventuelt kan man bruke en timestamp fra serveren isteden, siden dette sannsynligvis vil gi en mer nøyaktig representasjon av tilstanden spillet faktisk befinner seg i (slik det er nå, må man beregne at det man ser henger langt bak, som gjør at man ofte kolliderer med roser lenge før det vises). Det bør egentlig være en synkroniseringsprosess mellom server og klient som beregner hvor lang tid det tar fra serveren gjør en oppdatering til en pakke ender opp på klienten, slik at man kan ekstrapolere riktig, men det er nok litt overkill.


daktak can you resubmit your pull request for Added Transmission support #119? I merged it and my LL crashed so I reverted it now you've seemed to fix the issue but I don't know how to remerge it in without major failure.


That humans.txt thing looks pretty cool! I'll add one and people can add themselves to the list.


Initially I had it like `animationAndTransitionFinished = (transitionFinished && animatorFinished && !semaphoreATimedOut && !semaphoreBTimedOut)`. But I figured that if either `transitionFinished` or `animatorFinished` are set to `NO`, it has timed out for sure.

I can add those if you want in a *defensive coding* spirit.


Strange.. I don't know why it works both ways.. with or without method call


what is the meaning of the __init__.py data


yes, READ_EXTERNAL_STORAGE has to be added explicitly for some 4.4 devices otherwise will get permission deny.


@kasper93: this should be OK right?


@hvelarde sorry about that.. how did you find it? any debug technique or reading code?


When importing csv, shouldn't we be using CsvColumnRep?


@lucasbastianik utiliza o autor do post ao invés do site, então por exemplo o artigo do Tableless foi escrito pelo @grillorafael e o do Front in Brazil pelo @JLCarv


Is this the royal `you`?


we need a controller to get to a service, but I made it an anonymous controller built into the directive [here](https://github.com/WorldBank-Transport/edudash/commit/a99fb9cbe6df142eb8fb7ac92470dafce7e5dcda#diff-7c35e964cc23effa1511ef4b6b30f533R4)


Note to self: `copy_free<>`.


## This is a markdown file


OK.  I just wanted to check and make sure I didn't make things any worse than they were before with this change.


Note to self: no `using`.


Hi Prof. Karl,

I added the vignette lines and changed that "Marine Le Pen" line just
now.

It seems the pdf still cannot be built...

Could you please have a look?

Thanks,
Yilin

On 2015-05-15 13:20, Karl Broman wrote:
> You missed adding these lines to the vignette:
>
> vignette: >
>  %VignetteIndexEntry{SpeClust R package}
>  %VignetteEngine{knitr::rmarkdown}
>  usepackage[utf8](inputenc)
>
> Also, I'm getting an warning in both R CMD build and R CMD check, I
> think from the unicode characters in the new line 72 above (x = c(
> "Marine Le Pen...")).
>
> It might have been easier to start by merging my changes (in the pull
> request).
>
> --
> Reply to this email directly or view it on GitHub [1].
>
> 	*
>
> Links:
> ------
> [1]
> https://github.com/yzhang672/Frenchpj/commit/8a0de0bb2a27e7a00b5d9cfadc72b14f6c3db3a0#commitcomment-11219505


Yes, make sense.


Note to self: `copy_free<>` does not need `const`.


The thing is that theyre not the same ;) see the last lines of the methods.


Yes, on the GPU you will probably not get a speedup due to divergence inside a warp.

I don't like the plan of complexifying the code when GPUs will get float16 support.  I'd rather make GpuElemwise fake float32 inputs if we want the float32 limits and then change that later.


@lucasbastianik *Guia do Desenvolvedor Ionic* plz :dancers:


Can you please change `[...]` to `(...)`? I think generator expression should be preferred over list comprehensions in most cases.


Note to self: there's a shorter `decltype` in `bricks::weed`.


At last log it at debug?


Since I think this didn't work at all I doubt it.
I think we need to look into Character.toUpper/Lower as well as the String versions of toUpper/Lower.


Yes if no Endpoint with that id is found. Endpoint is an interface.


Тимофей, я не понимаю, что нужно сделать:
1) вместо user надо задать user_input_func()
2) надо написать user=user_input_func()
3) оставить user, но написать self._user = user_input_func()


These scripts were written with the primary intention of being portable and..


ok man


We should add EXT_texture_rg to the check that sets global_state.have_texture_rg in init_extensions.


Sure thing. I'm going to add a reply to your other question about the regex piece. Suffice to say, there's a piece of regex taking care of stripping out markup and then the autolinker finds urls and turns them into links. I checked most common injection attacks and everything came out clean.

More details: https://github.com/gregjacobs/Autolinker.js


Wouldn't this make more sense as a `fetchFromGitHub`?


it appears that this URL is now "fedora" and "u1404" instead of "linux", can you please update accordingly?


I'm noticing that both `shouldClose()` and `checkConsensus()` take a boatload of `int`s as parameters.  For the long term, not necessarily for this cleanup, we should look for ways to either reduce the number of passed parameters or to give the parameters types. This may help reduce the number of errors at the call sites.


$DAEMON_ARGS should be double quoted.


No, I'm not the user on the commit anymore. :)


Fair enough -- I wasn't advocating *keeping* the worse approach, just expressing hope that it accidentally worked, anyway :).  This is mainly for @leeping's reference to get ForceBalance up-to-date with the latest Gromacs (and @davidlmobley's comment here tells you how to do it correctly).


One of these days I'll get my new representation for this cleaned up and make a pull req. That uses a tree of monoid values and has easier generic infrastructure for simplifying and manipulating these trees which would make this stuff shorter to write.


Didn't you want to remove the PIDFILE here and in `zfs_daemon_stop`?


Keep in mind that this does not work in ruby 1.9.3.
This gem doesn't say which ruby versions it supports.


jade include & filter


I think you lost `freeze` :sweat:


Shouldn't this go above?


Me chamara? hehe Querem ajuda?


`btn`, `btn-primary`, `btn-block` are all bootstrap classes


not yet used in this commit


This shouldn't be a relative `href`.


"Surachi"? WHAT


94639630-HVACTemplateScheduleType-4860 (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)


PR it :)

Will fix after lunch. Sorry! :)

On Fri, May 15, 2015 at 12:10 PM, Alex Suraci <notifications@github.com>
wrote:

> "Surachi"? WHAT
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/starkandwayne/concourse-tutorial/commit/c5ab0c35360fd0225b8e86a99f0a1ae3777fc456#commitcomment-11220356


LGTM


thanks!


The basics are that we're stripping out all markup from the message before displaying it. Later on I'm autolinking urls in the message. This prevents anything from getting through.
Piece by piece explanation: http://regexr.com/3b0rq


Hey @rschmukler - thanks for looking into the bugs and fixing them so quickly. Had a question though, shouldn't this be `>=`? i.e. `el.scrollHeight >= el.offsetHeight`?


Looks great!

Couple of notes:

You can look up "clear button styles css" to remove the border around your buttons that are not in the design comp.

Also, your social media buttons look a little squished- try giving them some more space.


I'll merge it, but Perl has this wonderful feature called string interpolation:

    logger "U2B: extension=$_->{extension}, quality=$_->{quality}";

Variables are resolved in strings between double quotes. :-D


Looking forward to it!


After this commit, I get the following error when trying to run the saxon xquery:

Error on line 92 column 1 of saxon.xqy:
  XPST0081 XQuery static error near #... $mods:=$doc//mods:mods let $#:
    Prefix mods has not been declared
Static error(s) in query


closing as i submitted this PR to kilo branch per @miguelgrinberg 's recommendation. PR on kilo branch can be found here: https://github.com/rcbops/rpc-extras/pull/104


Passing the $ZED_PIDFILE here looks right to me for `daemon` as long as it matched the pid filed used by the ZED.  So we'll need to update the zfs-zed script to pass `/var/run/zed.pid` to match the ZED default value instead of `/var/run/zfs-zed.pid` which is what's happening now.


That's the point, it can't be. Because it contains several options (example: <code>-p /var/run/zfs-zed.pid</code>, we need the space here unquoted. Othervise, it would be ONE parameter, not two...


'blah' is unused here and should be removed.


Does this mean you will be switching away from visual studio and using the mingw-w64 compiler provided by msys2 soon ?
That would be awesome :D


@kalyan-kumar
it seems while migrating from Catkin to ROSBuild, you have removed the `add_executable` line from the `CMakeLists.txt` file. Have you added this line in a later commit? (Don't see it in the history though.)


Good catch!!! But the second `$method eq 'POST'` is not needed.


Looking at this function naively, it looks like it's returning two `bool`s: the return value and `failed`.  I suspect this could be reduced to returning a single `enum` which would cover the three or four different possible return values.

While we're here, it would be great to comment on what the returned value means.  I'm guessing that true means we have consensus, but I'm not absolutely sure, since the function is not named `achievedConsensus`.  Of course, if you return an `enum`, the names of the enums might provide that documentation and a comment would be less important.


Dang, I must have missed to commit that :(.


Hm, actually, it looks like the `istanbul cover` command may always write reports to stdout, but that should be fixable.


np. at least it makes the pronunciation clear


No, looking at it again, usage of it in the test and stop is correct, but not in the start (it would overwrite the file with an empty value for some reason).


If I read this, I would claim the result is still wrong given my comments in the conversation. The users asked for 700 events, you are delivering 700. You should, by your best estimate, deliver as much or more than they asked for, not less.


It's commit messages like this that make me think @burritojustice has a massive [Space claw](https://www.flickr.com/photos/straup/sets/72157625943591170/detail/) tattoo on their forearm.


double use of --with-fingerprint is intended due to the described in the gpg docs (line 53-56)


Comment added.


Yes. I was lazy, so I just ``s/url/zip/g``.


checking...


Nice


Could you revert these changes? The previous ones are more recent.


This commit seems to have broken the XBMC frontend plugin?


@hannes14 Sorry but this seems to be wrong assumption afterwards :-(
I will fix and push again...


Neuer Host Münsterland, ersetzt Freifunk Münster


This looks good to me - being able to load multiple interfaces would be great for writing things like force controllers and more.


@kalyan-kumar
Good work! Although, please avoid these kind of commits in later projects.

It is better to use a global gitignore file. Refer to [this link](https://help.github.com/articles/ignoring-files/)

I use [this global gitignore](https://github.com/icyflame/dotfiles/blob/master/gitignore_global.symlink#L1) file. Works well for me, as long as Sublime, Vim, Qt, Cmake, PYC, zip archives, etc are ignored. Do tell me if you find a better template. You can start with this template.


sgh


NetworkBy<ID|Name>() and EndpointBy<ID|Name>() return the  Network | Endpoint interface for the specified ID|Name.
The return parameter error denotes whether this was a bad request (invalid id or name passed), or something went wrong during the search (an internal error).

Therefore a valid lookup for a non existant (or no longer existant) interface will return nil, nil.

Are you suggesting a query function should return error if result is empty ?


Wouldn't it be idiomatic to always return error if something wasn't found? We're converting to an error here anyway.


asdf


94639630-HVACTemplateScheduleType-4860 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1650 of 1651 tests passed)

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


Let's format these case statements consistently across all four zfs-* scripts.


Just a followup on Howard's earlier comment, It would be good to add a comment here that the return value is (presumably) seconds...

Uh, now I'm realizing that I don't really know how the returned value should be used.  It would be good to add a comment, similar to the comment on `shouldClose` to tell the reader what we're up to here.  Without that comment my immediate next step would be to look at the implementation to see what's up.


Added. :)


But checking the manpage for start-stop-daemon, I think I've misunderstood the --pidfile option. I've always thought it would create the pidfile if it didn't exist (primarily intended for programs that doesn't create one themselves). But that would be the --make-pidfile option.

But in any case, it's not needed (as far as I've understood the man page) for the **start** option, only for **test** and **stop**.


So you prefer to use the ZED default instead of init script defaults? Not that it matters, but still. Since we're no longer using lock files, and this is the only daemon "we" start, it makes more sense to use the ZED default… ?


The leading `(` should be dropped, it isn't proper bash syntax for a case statement.  Although it doesn't seem to cause any problems.  We should fix this for all the case statements.

http://tldp.org/LDP/Bash-Beginners-Guide/html/sect_07_03.html


This commit reverted the mini aerospike ISP changes, was that intentional?


This isn't right, should be https://github.com/$travis_repo_slug


I assume we are going to require C* 2.1.5 > for this release so we can hit up the newly made statistics tables (ay) put into C*


@RyanBalfanz just to clarify, are you suggesting I include an error message in the UI if the form is empty on submission?


No worries at all, I'm still in the beginning to it's trivial to adapt. Looking forward to the release!


These queries methods are utilities used in different contexts.

For example when libnetwork is requested to create an ep with name x, a empty lookup for and ep with name x (presence check) is not an error condition, it is actually a prerequisite to move forward with serving the request.


Is this always the case?  If not setup correctly with tracking branches, won't users need to git fetch .., and then git merge ..?


Thanks, I should have caught that one.


Yaa it is still not finished. There are a few functions i need to add, and
we need to decide how the goals need to be sent. I thought I'll make those
changes when we discuss.

ᐧ

On Sat, May 16, 2015 at 12:44 AM, Siddharth Kannan <notifications@github.com
> wrote:

> @kalyan-kumar <https://github.com/kalyan-kumar>
> it seems while migrating from Catkin to ROSBuild, you have removed the
> add_executable line from the CMakeLists.txt file. Have you added this
> line in a later commit? (Don't see it in the history though.)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kalyan-kumar/Refuel-Bins/commit/f6d5570d7d1c4ca9efc9fc6104a424e17caf4058#commitcomment-11220410>
> .
>


исправил


Honestly, I could go either way here.  I don't feel strongly about it but @dun might.  To avoid confusion it make make the most sense to do both.  Explicitly pass the pid file as a DAEMON_ARG to the ZED (-p option) and have that path just happen to match the ZED default.


I'm not sure how I feel about exposing this as a public property. Basically I feel like if we can't figure out the data size we should throw an error.


This is a pretty good function comment.  Perhaps it should be moved to the header file?  It should also describe what `failed` means and under what conditions the returned `failed` value is initialized.  I'm noticing that on line 118 we return `false` without ever touching `failed`, so there are clearly conditions when the caller cannot rely on the state of `failed`.  It would be nice to document that without forcing the caller to read the implementation.  Or we could tame a bunch of this by returning an `enum`.


@ehdem What's the utility of the ```osv``` here?


Ahh, good point.


I'm reading this code https://github.com/plone/plone.app.layout/blob/master/plone/app/layout/viewlets/common.py#L305 maybe there is a better way to go..

Can any mantainer of plone.app.layout look if the way I check if I'm at edit tab is acceptable? @optilude @wichert @evilbungle @fschulze @davisagli @hannosch @alecm @esteele @mj @gotcha @witsch @scw @tisto @hvelarde

Thanks


We may want to log a warning on this casting


удалил


From the console we get away with "reboot if you can", but here we may have to prompt that a system update is due that requires a reboot so people will trigger it manually? That actually requires a package upgrade, then probing opnsense-update using "-c" command line option.


Hello Sir, the new version of the zxscan demoapp doesn't scan the qr code, the previous version scanned  properly.


Do was actually need/want to define this?  As it stands now it's unused.


RET here is also unused.


я этого не писал - файлы создались - я их добавил как только сгенерированные


what was the problem with it ? o.O


This should be 3 lines as per the style guide


`else {` should be its own line, as per the style guide


Strict equality operators please


well its called in CVideoSyncOsx::Setup(PUPDATECLOCK func) ... because its needed to setup the display callbacks. Any proposal on how to harden it to not beeing called during refreshrate change?


I will catch you there then.

Alex Anderson

<http://theunknownartisthour.com>
Listen In! Take Your Pick:
Live365 <http://www.live365.com/stations/unknownartisthour>
TuneIn <http://tunein.com/radio/The-Unknown-Artist-Hour-s241134/>

On Fri, May 15, 2015 at 11:19 AM, David Edwards <notifications@github.com>
wrote:

> Hey Alex, our presentation is at 1:00, I assume it is in Professor Root's
> office, GMCS 531. I might make a tweak or two to the code, but it will just
> be a clean-up
>
> On Wed, May 13, 2015 at 10:48 AM, Alex <notifications@github.com> wrote:
>
> > Not had the chance to work on actual animations, but right now it should
> be
> > fine.
> >
> >
> > Alex Anderson
> >
> > <http://theunknownartisthour.com>
> > Listen In! Take Your Pick:
> > Live365 <http://www.live365.com/stations/unknownartisthour>
> > TuneIn <http://tunein.com/radio/The-Unknown-Artist-Hour-s241134/>
> >
> >
> > On Wed, May 13, 2015 at 9:14 AM, David Edwards <notifications@github.com
> >
> > wrote:
> >
> > > Any more additions to this or is it in the final state already???
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/theunknownartisthour/PuzzleSolver/commit/9010566b4e68ef04035b486ad9ccecf4f1e42652#commitcomment-11179599
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/theunknownartisthour/PuzzleSolver/commit/9010566b4e68ef04035b486ad9ccecf4f1e42652#commitcomment-11181346
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/theunknownartisthour/PuzzleSolver/commit/9010566b4e68ef04035b486ad9ccecf4f1e42652#commitcomment-11219488>
> .
>


![deal with it](http://i.imgur.com/2TcOLWg.gif "Deal With It.")


Yeah, I think this is nicer and clearer. Just please add a comment about Python closing files automatically as the ``open()`` with no ``close()`` looks suspicious.


Well I (kind of) have a plan :-)
If we create a dummy package in ports to match our base/kernel, we could check if that one is in our upgrade set, if it is... we want a reboot.
Maybe I'm missing something, but the theory sounded nice.


I found this to be handy in my experience for catching this sort of stuff:
http://eslint.org/docs/rules/handle-callback-err


fixed on https://github.com/lcnetdev/marc2bibframe/commit/dc9f145ff4689ce6b1cbc1a9becacd86799865ce


The position is the second argument, not the third.  Force of habit from Dojo 1? :)


Oh, sure! I didn't realize that this was not yet completed! Sure, we can discuss and decide this!


- исправил "равно по определению", "равно по результатам измерения" (#107)


日期算法修正


The second space character is missing.


der kopiert, laut Code, verschlüsselte Passwörter.


In essence then, set ZED_PIDFILE="@runstatedir@/$ZED_NAME.pid" in zfs-zed:do_start script then. And not use the 'hack' to add it to ZED_ARGS in do_start(). Simplifies things a bit.


Hehe, I didn't assume that the "nova: use signals with irqs in core" makes it to the relase ;-) In this case please add also this fixup 1b7d0c62b9c09d87bd0e62d655ea9c56c49c0fbc otherwise the actual change get lost.


test


What do you think about adding a VERBOSE_IMPORT for use in the `zfs-import` script.  Rather than making VERBOSE_MOUNT do double duty.  This would all the log levels of import to the controlled independently which may be useful.  And why we're here a VERBOSE_SHARE wouldn't be a horrible idea.

VERBOSE_MOUNT='no'
VERBOSE_IMPORT='no'
VERBOSE_SHARE='no'


Indentation flopped here


`os.sep` is used to check sub-directory.
If using directory path without `os.sep`, it has a problem that excluding wrong file or directory.
````
>>> '/foo/bar.py'.startswith('/foo/bar')
True
````
So, directory path needs `os.sep` at the last.
````
>>> '/foo/bar.py'.startswith('/foo/bar/')
False
````


How this can get queried by Theano?


For consistency this should probably be single quotes.


I noticed that ShellCheck gives an error about this (I found a similar issue somewhere else I can't remember at the moment). But the unquoted variable is *functionally* correct, even if it's not *language* correct :(.


Unable to dowload the latest pari:
```
>>> Downloading 'http://www.sagemath.org/packages/upstream/pari/pari-2.8-1564-gdeac36e.tar.gz'
--2015-05-15 14:32:24--  http://www.sagemath.org/packages/upstream/pari/pari-2.8-1564-gdeac36e.tar.gz
Resolving www.sagemath.org... 199.27.78.133
Connecting to www.sagemath.org|199.27.78.133|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2015-05-15 14:32:24 ERROR 404: Not Found.

!!! Couldn't download 'pari-2.8-1564-gdeac36e.tar.gz'. Aborting.
 * Fetch failed for 'sci-mathematics/pari-2.8_pre20150510', Log file:
 *  '/storage/strogdon/gentoo-redlizard/var/tmp/portage/sci-mathematics/pari-2.8_pre20150510/temp/build.log'

>>> Failed to emerge sci-mathematics/pari-2.8_pre20150510, Log file:
```
I read something about broken links on sage-devel, but don't recall this one. The `upstrean` link is broken.


Should we fire the same event if callDepth is 0 for consistency?


There are many parts missing before it's even working correctly.


That looks good. Don't worry about the LaTeX problem. The pdf builds okay for me, so it's a LaTeX installation issue that we need not worry about at the moment.


Fala @grillorafael .. Vamos te citar aqui ;)


no endline


Again, this is unused.  We can keep it if useful / needed / expected on Debian but I don't think it's serving any useful purpose right now.


Thank you, fixed it!


Wow.  Nice comment.


Also a nice comment.


Does this style of customization allow the navbar to be turned off on a device that has no hardware keys like the M8?


@grillorafael Vamos te citar aqui, devido seu post lá no Tableless.. Você tem desenvolvido em Ionic ou estudado? Estamos difundindo a comunidade.. Temos o Slack do Ionic Brazil agora. Qualquer ajuda é bem vinda. :rocket:


I thought we had that already. :) Embedded in opnsense-update there is a version number and checksums for the particular set. I wouldn't recommend trying to impose this via the version number, because there is always a revision, or another hiccup maybe during the release process and we still have to store the checksum away from the package metadata. The safest and most portable way is to ask "opnsense-update, are you up to date?" after an update and that's what `opnsense-update -c' indeed does.


I would not keep unmaintained files in the tree, but up to you. Apart of that, I think these files belongs to distributors.


Yeah, I intend to correct that. I was just being lazy because it was like 2:00 AM.


Why on stop but not on start?


that should read: impose via the *package* version number


and here I realised I can edit the original post ;)


etc_t is declared in kernel/files.te, so we don't want to have that in a read_files_pattern in init.te because it breaks encapsulation.

init_t already has permissions to add entries to etc_t via files_manage_etc_runtime_files().

Giving init_t permissions to add entries to hostname_etc_t directories, even when those directories don't exist, doesn't endanger the security of the system.  I'd prefer doing it that way rather than breaking encapsulation with etc_t.


hmmmmmmm hope it would be ready soon
can i suggest to minimize refresh rate (ex: 3-5 seconds) to reduce cpu usage by the client ?


That `>&2` at the end should be dropped.  My bad.  Also we should add the extra `[ -n "$1" ] && echo "Error: Unknown command $1." for consistency with the other scripts.


Are these times in seconds or milliseconds?


There is a new repo at http://github.com/lwfinger/rtl8723bs_bt.git with the firmware and the code for rtk_hciattach. That should suffice for now. Once the driver is ready for submission, I will push the firmware to linux-firmware. I will need to investigate how rtk_hciattach will be integrated into the user code.


So my package is correct now, right?

Thanks for all your help!!

Best,
Yilin

发自我的 iPhone

> 在 May 15, 2015，2:42 PM，Karl Broman <notifications@github.com> 写道：
>
> That looks good. Don't worry about the LaTeX problem. The pdf builds okay for me, so it's a LaTeX installation issue that we need not worry about at the moment.
>
> ―
> Reply to this email directly or view it on GitHub.
>


I would rename this class to SshKeyJson to be completely camelcase compatible..


What refresh rate? The processes won't get refreshed automatically.


Question possibly for @BryanForbes to chime in on:  what do we want our strategy to be for skipping tests in non-browser environments, especially in this package?  Here are our options:

* Skip it at the `all` level like here
    * Pro: involves the least extra work, and we already do this some places in core because we have to have separate modules for browser/node
    * Con: leads to different number of tests run on different platforms
* Skip per-test using `this.skip`
    * Pro: keeps consistent test count across platforms
    * Con: requires writing `this.skip` in every single test which is annoying
* Don't even bother worrying about this in this repo since it is assumed to be browser-only
    * Pro: look ma, no hands
    * Con: `grunt test` will throw tomatos at you


Weird...not sure that this is even pixi.js related. I think it was a fix I did that just sneaked into this branch on accident!


Already done in a follow-up commit ;)


Everything looks commented properly


There's no reason to even run those in non-DOM environments. Option 3 it is!


Looks good!


@joeblew99 You should go get that missing package. It's only needed when running WebGL mode.

```
go get -u honnef.co/go/js/dom
```

I will fix instructions so this will not happen in the future, thanks for pointing out the problem!


It's hard to say then what's causing the issue.  Can you post a copy of your project on GitHub or Bitbucket?

> How much time it will take to compile with AS

To clarify here, your projects should not need to compile the Mapbox Android SDK as it's already distributed via binary `.aar` and it's dependencies via `.jar` files.  If you're looking for an example of what a project that integrates with the Mapbox Android SDK would look like in Android Studio,

1. Clone the [Mapbox Android Demo app](http://github.com/mapbox/mapbox-android-demo) locally
1. Open Android Studio
1. "Open an existing Android Studio Project" from Quick Start menu
1. Choose directory where Mapbox Android Demo source was cloned to.
1. Once loaded, you should be able to run the app either on device or emulator.


Also a nice rename.  `shouldCloseLedger` is an improvement.


Since you just return if `!newTodo.length`, one thought would be something like using [ngDisabled](https://docs.angularjs.org/api/ng/directive/ngDisabled) on a submit button in your form.

Or, a validation error message could be good too ([https://docs.angularjs.org/guide/forms](https://docs.angularjs.org/guide/forms)).


`process.on('uncaughtException')`


#2166.  test update


Esses são os port maps das instruções: LW, SW, AND, SUB, ADD, SLT, BEQ, J.


Doesn't really need to be checked in either actually. This is supposed to be checked in/by zfs-import which is run first (well, except for zed - maybe we should move it there).

On the other hand, can we count on them being run in the order

1. zfs-zed
2. zfs-import
3. zfs-mount
4. zfs-share

?


Yes it looks great, thanks.


It may be helpful to format this comment as a list of all the valid/required keys and their defaults


I'm with you, Nik, I think this is easier to read as nested `if`s than as a large compound `if`.


No, I was probably thinking about the checksystem(). Verifying that the module is loaded and if not bail is probably valid in ALL scripts...


Any aversion to changing this to an if/else?


Field and properties are the same thing


Fixed issue #99


Look up Javadoc annotations, and comment the parameters and return types that way


94639630-HVACTemplateScheduleType-4860 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (627 of 627 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-5.29%25-green.svg)


sorry for that i thought that usages are updated frequently ... i misunderstand  :disappointed:


Is the return value in seconds or milliseconds?


Huh.  The boolean state of `failed` is not what I would have expected.  So, `failed` is `true` if consensus was reached and we were part of the consensus?  That's not intuitive to me.


Thanks Ben for updating about the status of the GL renderer.

One thing we will need to document at some point are the API changes in the latest stable JOGL release (2.3.1, which the one I put back in). Nothing major so far, but they changed the package names, for instance com.jogamp.opengl instead of javax.media.opengl. This will affect only those using low-level GL calls.


Yes, I have to find where to point the mirror to. I should have paid more attention to that ticket.


I just realized we need a catch-all here just in case the incoming URL is neither `/request` nor `/confirm`. So, just add a one-liner here that does:

```
response.send(null, 404);
```


Looks good.


I think this code should be executed outside the if (!disablePropertiesAsHeaders) block. The header should be created even if the user disables properties as headers.


I especially like that one :)


Gonna need to re-introduce so stuff you removed that I just want in the guide, like line endings, I work with a lot of people that actually work and modify PowerShell code in Linux and have had the problem with line endings happen many times.

Alo I HATE comment blocks inside the function. it is a personal preference and I do not want to make having it inside the function a requirement but just make requiring a comment help book if no XML help use a must


Vendor, Frameworks and settings


0.3- means "any version at least prerelease-0.3". 0.3 means "any version at least 0.3"


Really the correct way to use resources efficiently, is to make thr STC test stricter. This `SPRT(-1.5,4.5)` is really stupid: it allows far too many zero elo patches through to LTC (where they have 95% chance to fail). Personally, I've stopped using this `SPRT(-1.5,4.5)`, and use instead `SPRT(-0.5,4.5)+SPRT(0,5)`.


Thank ya!


I see you remove the comment on use of splatting and line continuation, that is another one I want to keep in my guide.


[My expression about his commit](http://a.fod4.com/images/GifGuide/dancing/280sw007883.gif)


Nice. I'm curious, what made you think of using a regular expression (`/…/`) and the `replace` function with a string?


I'm not calling you retarded, eveo. Please direct your rage cannons somewhere useful, kthx.


JOOQ looks good. Pretty clean and probably easier to Debug than putting our entire queries in string literals. (I've been through that, it's the worst)


Ah ok. That's fair.


Make sure there is no space between the ? and php.


and it has a guide on how to set it up with eclipse

http://www.jooq.org/doc/3.6/manual/code-generation/codegen-configuration/


This method is functionally equivalent to the two below, they just differ at $watch(Group)(Collection).  I would DRY this out and pass in the correct function prototype instead.


まずは画面上からリンクをクリックしてCSVファイルをダウンロードできるところまでやってみましょう。


This RAII wrapper for FS_ReadFile could be advantageously replaced by usage of the API in FileSystem.h
(sorry, I know how painful it is to write nice helper code just to delete it after)


このファイルは編集不要かと思います。


I would rather you fail with Sys::Error (hard fail) or Sys::Drop (drop to main menu) rather than letting an exception leak. In this case I think Sys::Error would be more appropriate since dropping to the menu won't make shaders compile any better.


VEipTScheduleListからScheduleSearchResultDataにデータをセットする処理になります。


スケジュール一覧をデータベースから取得する処理になります。


周六吧，我这周末就走了


Warning the user is great for developers. You could use Log::Warn, or even better introduce a Logger for renderer.shader


Should this be `child.endAll()` so that it recurses completely? I think this will only go in one level, won't it?


that was a sneaky change


It works in the real machine only.


You can use Str::Format() instead of va(). It returns a std::string and doesn't use an internal buffer.


File foo = FS::HomePath::OpenRead(Str::Format(.....));
std::string binary = foo.ReadAll();


You should probably update the items before deleting anything, this is not going to work


Don't use .c_str(). The key is a std::string, not a char*. StringRef has an implicit conversion to std::string so you can just pass filename directly.


Why?


if i am not wrong this is low level hook not global hook


Dang, just when I thought I was getting the hang of it!
I'm not using 'git merge' directly much at all, but it does result at times when I do a pull from a remote. It probably doesn't help that I'm working off of three different machines at different times--desktop, laptop, and hetzner server. Anyway, I'll review procedures, but if you have any suggestions, I'm all ears.


Yeah, this one part of the new API changes a lot; I had to do the same thing in 80fdd5ac.


Use a "new style" cvar for that, see Cvar.h


@markguinn Sorry I suck this one in here without a PR. A fairly small non-breaking addition.


ok


nit: you use bot // * and //


it doesn't eat them but they are used for escaping, to put a \, type \\


1836664-ReturnWaterTempReset (Myoldmopar) - 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)


ouch, sorry about that. Thank you for fixing and thanks for the tip.


> The inner for block is executed at most once (d++ == 0), but we need sometimes two executions to fill the buffer (free space is at the start and at the end).

:+1: , fixed


this is hilarious


Actually, this implementation requires a explicit check against size/capacity to determine if the buffer is full or empty, since using only pointers there's no way to distinguish between those states. Fixed in the last commit


nit: remove space before )


I don't see the value in alerting in this case, as it would only happen if the developer changes the Cvar value.


Log::Warn


The upgrade worked fine.
May I suggest you remove the in-code-style in the file design/frontend/base/default/template/billmate/getaddress.phtml. It messed up the address form, covering the first name field and made the next field coming horizontal instead of vertical.
Without the styling it is ok though.


We might be leaking ShaderExceptions here, it would be good to have another try catch that just Sys::Fatal in the catch (not being able to compile our builtin shaders is a fatal error)


Ughhh global variable :(. But there is no better way without a heavy heavy refactoring :/


eveo was misdirected because he has also made some comments to this feed, but they were not left unmolested for long. (Hee hee, see what I did there.)


Thanks for the patch! It will be a great improvement for people working on the renderer.


Yes, global hooks aren't possible in the .net framework


Hi. After this change I am not able to use Sequel in openshift as it uses the postgresql adapter. There is no way of configuring this? Thanks!


I think this is making invalid assumptions that people are following certain naming conventions. I think we should just keep it simple like it was. We don't need to parse it on the server side either. We can just stack based on the name which will contain the full path like "exceptionless.plugins.EventPluginManager.run". For .NET we stack on full method signature and error type. I think for JavaScript we should stack on the full method name and the error type if we have one.


I can't spell :cry:


Whoops, this doesn't account for body attritbutes. I'll fix that.


ctype_digit will weed out decimals


We need to make sure this merge is correct


dd


Can you explain what problems you are experiencing?  It should still be possible to use `postgresql://` connection strings, this change just makes `database_type` return `:postgres`, which is necessary as that's what all of the specs are checking for.  Here's an example using a `postgresql://` connection string:

```
$ ruby bin/sequel -E postgresql:///?user=...
I, [2015-05-14T19:38:39.152151 #2661]  INFO -- : (0.000542s) SET standard_conforming_strings = ON
I, [2015-05-14T19:38:39.152674 #2661]  INFO -- : (0.000215s) SET client_min_messages = 'WARNING'
I, [2015-05-14T19:38:39.152909 #2661]  INFO -- : (0.000178s) SET DateStyle = 'ISO'
Your database is stored in DB...
irb(main):001:0> DB.database_type
=> :postgres
```


rax retry tests


Added two line notes about some missing stuff.
- Missing points after suite of keys (no final point after the sentences in the game)
- Missing space after a key that breaks the Pedia Link in the game.


Hi, thank you for the changes.

I thinks there is no need to fix collisions and pack nodes after each iterations. Maybe it's better to do this only once before `this._update_container_height();`? What do you think?


the more I think about this, the more I think it would probably be a good optimization - it might make things a little harder to debug, but removing any condition you can should make an improvement.  There might be other places you can use the same technique.


Did you run `pip install future` or rerun `python setup.py install/develop` so it can pickup on the `future` dependency?


Man I hope you get this working. I've been trying for awhile with no luck.


the dropdown is not opening. cannot change theme as a result. is it working for you?


@tonnypi  I added a method to get the closest target, and added code that tests this to your BasicGunEntity class.  You can even specify the type of entity to target!


Lessons learnt:
Bug1. self.counter is NOT same as counter.  In nutshell be very careful when we are dealing with instance variables. This was a really hard bug.
Bug2: In python reference(let's call it r1) is passed to another function by creating "another" reference(let's call it r2) pointing to "the" object (let's call o1). So if any new object (o2) get associated with reference r2 in function , we are not updating r1. It gets lost after exiting the function!
Should have spent some more time in testing.


the tabs in the header should be visible only after login.


hey


is this working for you? i cannot change theme.


Does this make you feel better?


1836664-ReturnWaterTempReset (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1659 of 1659 tests passed)

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


I can show you how to make them later! But these are preliminary, I think I can make them way cooler


Typo: `passowrd` should be `password`


`self.salt` should be `self.password_salt`


1836664-ReturnWaterTempReset (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (629 of 629 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-5.3%25-green.svg)


okay, I'll revert this change.


it doesn't work in my side too. will fix it.

Actually I was thinking to remove it. As it is not apart of User Portal @ed-xj  what  do you think?


1836664-ReturnWaterTempReset (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1665 of 1665 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)


Well, if you look at https://github.com/opencog/atomspace/network and especially if you scroll left a bit. there's this braided, tangled structure under your name; its more braided than the master branch.  So, on the 6th, you created 5 brnaches; on te 8th and 9th merged them in various ways and created a few more branches; on the 10th, more braiding, pm the 12th, consolidation to 2.   Compare to misgeatgit who has on long straight line, which is the "normal" shape.


Check last commit out, I think the dependency named argument was wrong in `setup.py` so they never were installed. Should work fine now.


Maybe this would be better to be named `ValueData` or something, since `const` and `static` aren't "really" variables. (I'm not really a fan of any particular name though.)

Could you add some (doc) comments stating that these  point to `const`/`static`s?


what


These tests are worded as if position is the last argument.  (Were they ported directly from Dojo 1?)


:-) Well,  maybe this needs some more thinking. Ideally we should now if in the set there's a base upgrade before we start upgrading.


see glx which registers into windowing


1640-AddHVACPart1 (kbenne) - 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-NA-red.svg)


The comment was incorrect, great catch. A follow up commit had already removed it by switching over to an `enum` for the return value.


tested and working


Oculus just announced they're halting Mac/Linux development and have no timeline for returning to working on it. I'm suddenly a Vive developer, look at that!


1640-AddHVACPart1 (kbenne) - Win64-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-NA-red.svg)


`(:throw-exceptions req true)` ?


It's a little weird to know we ship information about base upgrades using the packages system on top, but the design is pretty nifty: it allows us to bring in upgrade code before we execute it, e.g. when we do a larger upgrade from 15.1 to 15.7 or or 15.1 to 16.1. The downside is that ABI upgrades don't always work, but we can stage out of this by only delivering the update script, run the update and update packages from a different ABI afterwards (inside opnsense-update).


Commited I/O's for CPU Tweaks v3.0


nice :)

this would probably cause really more slow unittests on some browsers. but i'm ok with this.


Zarel intentionally changed this because he didn't like how Admins were appearing above Room Owners. He said something about Room Owners being the Admins of the specific room, therefore displaying above  Admins.


Fix pushed but I may have to make a script to generate a list of mirror on a regular basis.


I was following the convention here from batch. It handles a 0 batch size by return the entire stream as a "batch" when nil is reached. If n==0 I think personally it makes sense to emit [] and end the stream, but wanted to main consistency with batch.


@Rhythms88 https://github.com/Zarel/Pokemon-Showdown-Client/commit/426d2e6cb0e7f2f327cc98ab3449e6c89ccbb3e5#commitcomment-11208484


Seems reasonable


Hello, Its me!


These XPaths were overly complicated.


@jlucerond this code is simply removing the item from a copy of the real items. you have to remove the item from _mediaItems not a copy of it, nor from self.mediaItems as that's read-only.


Added an if loop so empty unitdate elements aren't created.


I would put a else here!! with a logMsg of no data was retrieved for this url!! ?


Minor one: We can use QStringLiteral instead of QString::fromUtf8 can't we?


1640-AddHVACPart1 (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


removed duplicate code by shifting the physdesc and extent tags outside the choose loop and using a xsl:attribute to set unit value


do_start() / do_stop() would be consistent with zfs-import and zfs-zed


Again, simplified XPath


Ничего из вышеперечисленного.

Внутри `__init__` всё должно остаться так же, как и было до этого, а передавать в `Game` как `user` нужно `user_input_func`. Когда требуется использовать `user_input_func`, нужно использовать её сохранённую в `self._user` ссылку. Поле `self._user` лучше переименовать в `self._user_func`


Markdown file for Data Science Toolkit coursera course


Memory usage may become a bit problematic if 500 bytes are allocated each time. It's easy to fix, do not change now.


Checked with Haunter and he felt strongly about admins being on top.


You can add yourself to copyright too.


I'd like to point out that intellimm is designed for newer msm8974 pro and newer cpu's and is far from fully compatible with anything older without modification to most of the core code


Just changing it to the ```setting[]``` instead fixing the offensive call will break the load names button.


No need to cast `void*` to the left hand side variable type.


I thins has been fixed in Spring-Rabbit and it's not even required anymore. Not affecting the current PR.


the kang is strong with this one. we all know you are incapable of actually doing these patches. there is no way that this could possibly be your commit


No, that's clearly wrong.


If it is modified for one s by rmbq than why is the author faux. SMH


impropper squash. but atleast half propper authorship


*can now, not (can not)


Maybe here, after `send()`,  `updateMainData()` could be added as it is in the others.


Not even close to proper authorship.


I still don't like the future plan, but changing the input types is less than trivial and I don't want to risk bad data.  So I made the ops use the float32 limits for float16 and we can revisit later.


After this change has been made my revert no longer works.
Could there be an issue?


kaputt.


ha kanged from chet -.- SMH


I fixed it! Thanks!


Only `lldpctl_k_custom_tlvs_clear` and `lldpctl_k_custom_tlv` are transmitted immediatly, right? Other changes need to be pushed by setting custom TLV atom on the appropriate port.


In pratica ogni carattere conta doppio e in più c'è uno spazio bianco, del tipo un "\n" che alza la lnghezza di 1.

Metodo "ignorante" e rapido di correggere: lunghezza = (sequence.length -1) / 2;
ma nel caso in cui anzichè emoji stampassimo di nuovo numeri oppure un misto oppure lettere non funzionerebbe più niente.

Metodo migliore (a mio parere):
prima di tutto rimuovere gli spazi bianchi (funzione trim nella stringlib.js che avevo fatto)
es. lunghezza = trim(sequence).length

dopo aver fatto il trim ci troviamo con gli emoji senza spazi bianchi in testa o in coda, ora bisogna solo contare gli emoji correttamente. Ho creato nella stringlib.js una funzione che conta la lunghezza di una stringa unicode in modo corretto, la spiegazione del perchè javascript sbaglia l'ho messa nel commento.
La funzione non fa altro che sostituire nella strina ogni carattere che verrebbe contato doppio con una "X" (che viene contata come singolo) e poi chiama la string.length sulla stringa con le sostituzioni.

quindi in definitiva

stringasenzaspazi = trim(sequenza);
lunghezzacorretta = utf_length(stringasenzaspazi);

o in breve

lunghezzacorretta = utf_length(trim(sequenza));


It's won't be needed for zfs-zed since the daemon will just wait until the `/dev/zfs` device shows up.  Which is fine.  Also not needed for zfs-import since we expect it to be loading the modules there.  But it probably does make sense for zfs-mount and zfs-share.

I was also thinking we should change the `check_module_loaded()` check.  Right now it checks `/proc/modules` for a `zfs.ko` but that assumes ZFS is built as a module and not directly in to the kernel (which is possible).  If we were to instead check that `/sys/module/zfs/version` exists it woud cover both cases.

It would probably also be a good idea to slightly rework


so you want me to convert between the timebases somehow? if so how?


One thing I noticed when testing your app is in the Horoscope Guru, when the keyboard comes out, it blocks the timer. I recommend the keyboard disappearing once the user presses the "check the answer" button or the "replay" button so the timer can be seen again


Looks good!


watch indenting, no ; at the end of functions


The structure needs to be duplicated (and then freed at the end, like the other stuff) because otherwise, it is shared between multiple ports.


Misspelled Canola to canols


@vqvu I'm not sure If i agree here with your interpretation of how sliding buffer would work at the end of the stream, because it is kind of arbitrary to apply the "sliding" part to the end and not also the start of the stream, ie:
```
[1,2,3]->slidingBuffer(3,1) == [[1],[1,2],[1,2,3],[2,3],[3]]
```
Personally if I was doing moving average or convolution, I would want this behavior to be consistent for start and end of stream( vs just end).  Its debatable if this behavior is preferable to saying "only ever emit a full window or whatever is in buffer when end of stream is reached". What do you think?


Use === instead of ==.
Everything from this if statement dow is one indent too far. Keep an eye on that stuff.


The fundamental differences between `GenericEvaluator` and `FlatteningGenericEvaluator` are:

1) `FlatteningGenericEvaluator.eval()` cannot write its results in a `WritableColumn` since the results may overflow `TupleBatch.BATCH_SIZE`. Instead, it needs to take an `AppendableTable` (concretely a single-column `TupleBuffer` in the current implementation) that does its own batch management.
2) `FlatteningGenericEvaluator.eval()` needs two columns to record its results: one for the result counts (which has the same length as the `TupleBatch` it's evaluating and hence can be a `WritableColumn`), and one for the results themselves (which needs to be an `AppendableTable` per above).

I suppose it is possible to make the existing `ExpressionEvalInterface` and `GenericEvaluator` special cases of `FlatteningExpressionEvalInterface` and `FlatteningGenericEvaluator` (presumably including optimizations that elide writing result counts, etc.). Is that what you're proposing?


Wider ein Sysout? Aber dieses mal noch nicht mal auskommentier?


Fixed in 207e9d074b4c8f6ead451eddf83db5c65ad22ea2.


This commit is actually the fix for passing it to the anon function - it's not passed anymore.


typo: cacedRccCsrValue, should be cachedRccCsrValue ?


did you ever find out the difference between the do while and the while loops?


I would rather use `unknown-tlvs` and `Unknown TLVs`. They are custom if you set them but you will also receive them from remote.


Ditto.


Aww.. :(

Sad to see that she's going... Was hoping that we could meet her again at some point.


Look at application-roles.properties in kie-wb-webapp... are you sure you want to leave this? It looks like a debug change to me.


There are several references to http://localhost in here for images; such as the logo.


you named this an array, but is it really an array. not a big deal but it might confuse another dev looking at your code.


Oh, thanks. I took this code from http://www.cmake.org/pipermail/cmake/2011-June/045092.html.


what is the Faceted variable by the way?

rethinking the window.Faceted stuff - i might have to refactor it. Faceted was first a global. I changed it for window.global for more explicitness, but window.Faceted is not the same like Faceted as global. i'm going to fix this in subsequent commits.


Yes, it's true that `colIdx` is always 0, I just put it in since `TupleBuffer` is a multi-column table, but I don't know how we'd ever use this flexibility, so I'm fine with removing it.


Way to functionalize the code.


@ehdem Why not ```product_uom = move.product_uom```?


Yay! We made pretty much the exact same change for shipping AirMech on xb1. Was just getting around to pulling out publicly releasable changes, and found this one was already done!

For reference, the uniform->static change to reduce constant buffer size netted us a 23% drop in the time spent in bgfx::d3d11::RenderContextD3D11::Submit on xb1.


typo


I will add it soon, maybe in this ticket.


Without this two lines my monodevelop refuses to compile the project.


ok


Also we support the saveAsCassandraTableEx which in another PR could be implemented here


Since there are other changes, I propose that you also fix that (but I can merge first without if you can't right now). Just change `oui_info_len` and use `MARSHAL_FSTR` which, despite its name, will work fine even if you have null bytes around (it will just add an extra null bytes in case this should be a null terminated string).

Keep that in a separate commit, this will be easier for me to review the change.

Thanks!


Attach to #143


Déjà fait  :
 - Lecture de messages
 - Affichage en gras des messages non-lus
 - Affichage des messages envoyés
 - Suppression de message individuellement
 - Ecriture d'un e-mail
 - Réponse d'un e-mail


Nice find. Can you send a PR on the original node_acl?


Fun fact:
Template magic means that you can omit `countof(moreInfo)`


@TheSeamau5 found this commit in #937. I have only a very superficial understanding of the code here, but just from reading these changes, it seems you would need an equality constraint here like `CEqual Error.List region tipe (list varIgnore)` where `varIgnore` is a fresh variable. That way an empty is of type `forall a. List a`, instead of only `a <: comparable` as seems to be the issue right now.


> Instead of layering message processing steps in a single actor, I
> would always prefer using multiple actors instead with a forwarding
> chain.

I agree that from a design perspective it keeps the architecture much
more tractable. I already have other "concept" implementations, such as
"task" and "accountant" which really work well implemented as separate
actors.

> I don't buy the "local decisions" argument, btw. Take the middleman
> actor for example: it provides an interface for publishing local
> actors and gaining references to local actors. It's implementation,
> however, is simply to establish the initial TCP connection if needed
> and forwarding all calls transparently to the `basp_broker`. It uses
> response promises to keep track of original requests. Any client of
> this actor will *never* receive a message from the `basp_broker` and
> the `basp_broker` is completely hidden away.

Sorry, I'm not following your argument and do not understand the
example. How do local decisions relate to the BASP broker? Are you
saying that you one never has to make local decisions because, ideally,
all actors works with are response promises?

In the example I gave, there exists a load-balancer with worker nodes.
This just one pattern that occurs frequently inside a distributed
system, not just at the edges. If one worker gets overloaded (because
it's a heterogeneous compute environment), I would like to remove it
from the round-robin scheduler inside the load-balancer. To this end,
the worker sends an overload message to the load-balancer. In my revised
proposal, the worker would send the overload message to the
flow-controller, which in turn then sends it to the load-balancer.

> You don't even need to keep track of the exact number of messages in a
> mailbox: you know how much pendings requests the worker has at the
> time of receiving a new task. As long as the forwarding actor does
> nothing but bookkeeping, its message handlers are fast enough for
> ignoring the "actual" situation in the worker's mailbox.

Are you saying that you can determine the load status upstream of the
worker (e.g., at the load-balancer) by performing the bookkeeping there?

> Easy to understand and to reason about, while the composition is
> *also* much easier to understand.

Yes, we're on the same page here.

> You simply add another actor to your chain. `logger <->
> flow_controller <-> worker`. Instead of complex "augmentation" of
> message handlers, I would simply compose actors. Instead of
> `spawn<worker>`, you do
> `spawn<logger>(spawn<flow_controller>(spawn<worker>()))` which you
> then can flatten using a `vast::spawn_augmented` for example.

That part I don't see yet. I was thinking of a single, global,
flow-controller as well. Though I don't see directly how that could be
hidden with spawn options. The way I would imagine the setup is like
this:

    // Inside "main"
    auto fc = spawn<flow_controller, detached>(..);
    ...
    auto a = spawn<Actor>(..);
    send(a, flow_controller_atom::value, fc);

    // Inside Actor, who may opt-in to flow control
    if (flow_contoller_ != invalid_actor && overload)
      send(flow_controller_, overload_atom::value);

It's a bit explicit, but gives every actor a chance to opt-in to
flow-control when receiving a flow-controller. If you have ideas how to
make this prettier, I'm all ears.

> If you are afraid of a few extra cycles: go measure it. I'm sure the
> overhead is not an issue. :)

No, I'm not worrying about cycles. But RTTs. (Though for a cluster
deployment even those shouldn't be a concern.)

> It's probably a good idea to spawn the flow controller detached, btw.
> If all threads of the scheduler are busy, the `flow_controller` will
> sit and wait in some queue.

Agreed. I do that already with my accountant, which performs timing
measurements which should not be perturbed by the scheduler.


That's what I did originally, but thought it was clearer to update all the arrays in one place. I'd be fine with moving it back to the bottom of the loop, though, if you think that's clearer.

`cum` stands for "cumulative" but I guess I could use `sum` instead.


user can specify it through table options. The order is user specified table size -> get table size from C* system table -> Spark SQL default size. The first available table size will be used.  System table stores the data, I will get it in next commit.


@shurcool
Thanks. I managed to work it out myself.
Managing builds with golang is amazingly easy.
I wish there was a way to debug go , as it would make it allot easier to learn it.


@Warpten A clear example of someone who hasn't any idea that means working well


I just put an ``import pdb; pdb.set_trace()`` at the beginning of the suspected method


Good catch!


Move this out of the version specific file?


Thank God!!! Thank you for doing this ... @jajones : do you know why we have this time out to begin with? I am wondering if we can remove it completely.


> Doesn't the periodicity follow from the Fourier transform? (i.e., is there any way to do an inverse FFT and get something which is not periodic?)

So it depends on what you mean by periodic. What I'm looking at here is continuity, i.e. when you take the IFFT and repeat the signal, the signal itself and the first derivative (and maybe further derivatives) are continuous. This isn't guaranteed; for example you could take the FFT of a ramp function, then take the IFFT of that Fourier representation and get the ramp back (which if repeated is not continuous).


I'd like to leave as is. Just because this might prevent future issues ;)


Actually we have a CSS on our module which overwrites a few things. Fixed there at bfa2272.


So the reason we didn't have this generic fallback is because it allows the slow version of ``var*coef`` on 0.3. At this point that probably isn't too important compared with the benefit of making the code simpler.


legAcy


Very!


sorry, in fact ``getFolderContents``  was causing here "Insufficient Privileges" issue also.

at first I replaced that with a call to ``listFolderContents`` but then I just read it can cause performance issues.

``getFolderContents`` returns brains, so it's totally fine.


So what happens now if people try to use a weird type in the expression? Missing method error for ``*`` or ``+``?


you're right again, that was caused when I changed to object


again, my fault


I just fixed that in the next commit


Pushed to pypi.


@laruence I had fixed it locally, but the push failed as there were newer commits upstream… forgot to check that after git push… Sorry about that. Fixed now.


It occurs to me that this behavior might also be good in addStructure().  I.e. when you add a new structure, automatically select it.  Too presumptuous?


@renzo @wedaly we should also move this behind the DEBUG setting. I can't think of a good reason this would be needed in production. @feanil are we using this for some special cases for ELBs?


Vielleicht besser den Hostname? Den kann man ja beliebig herumschieben.

53.168.120.37.in-addr.arpa domain name pointer fusselkater.gw.freifunk-muenster.de.


This commit broke compatibility for `riak_test` users who do not use `riak_kv`: for example, both `lasp` and `antidote`.


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


Already done, thanks!

https://github.com/OptimalBits/node_acl/pull/121

On Fri, May 15, 2015, 16:40 Prabhu <notifications@github.com> wrote:

> Nice find. Can you send a PR on the original node_acl?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/colearnr/node_acl/commit/65abef821470a83df01ca67be782c376ca85d52f#commitcomment-11221639>
> .
>


Not sure if I want to leave this in when Boswell is more stable. If your error tracker throws an exception, what are you going to do with that exception? Track it? :)

It's in for now as this is a beta, though. I'm going to include Boswell in a public app I have, and I will be using both Raygun and Boswell for error tracking there. Any exceptions thrown by Boswell will be sent off to Raygun.


Nit: No need for type-casting. :)


This is a very hard to read function name, may instead constructCassandraConnectorConfigurations() or if you don't like alliterations, buildCassandraConfs?


Yeah, it would be deferred to either a MethodError, or an error in ``@addConstraint``


:+1: :smile: :+1:


done.


Where to? Makes sense to keep it with all the other ``addToExpression`` methods.


Nit, and generally personal preference - use `types |= UIUserNotificationTypeAlert;` aka combined assignment with bitwise or.


I like that idea, it's certainly more elegant and less ad-hoc than my approach. Actually `FlatteningApply` can't use expressions for which `hasArrayOutputType() == false`, because `FlatteningGenericEvaluator` uses `FlatteningExpressionEvalInterface`, which assumes that the Java expression properly fills in the result count column. We would need to make `GenericEvaluator` a special case of `FlatteningGenericEvaluator`, as you mentioned elsewhere.


Please re-format the commit message to separate title and description.
"This specifies which methods should be fed to the Alternate JIT ..."  should be in the explanation.

You may want to squash the other two commits into this change post review.


Ahh, yeah. We could add an error method for that case on v0.3 if you'd like.


Fix confirmed.


Please note that

$links += getEventLinks($first_time, $last_time, $links, 20, true);

should be

$links = getEventLinks($first_time, $last_time, $links, 20, true);

otherwise you get each task twice...


topkek.


What do you think about making the dev-lang/python wrapper check for >=python-2.7.5 when looking at the system version, and then renaming it dev-lang/python-2.7.5?  Then we won't have to fork any upstream eclasses, which I'd prefer to avoid whenever possible.


оно как раз выше написано.
А это ловит то что в промисах не отловили https://github.com/petkaantonov/bluebird/blob/master/API.md#global-rejection-events


This looks nice


How about a test for explicitly saying `'all'`?


im not sure what happened here. I readded them

https://github.com/KSP-KOS/KOS/commit/29961cb6012759132ecc8c607fe1c55d195f11d7


hmm, well we don't really have any logic for all (since it defaults to it) so we might not need to test it


@tpietzsch
I suggest imglib2-ui?
The same for the AbstractNamedAction?


That sounds reasonable. I think the python ebuild will need to be rewritten a bit b/c ${PV} will be 2.7.5, right?


Oh yeah, that's true.


this one needs to change to ${overrideCheckColumnValue} to match what is in JdbcHdfsOptionsMetadata

should we change all overrideValue to overrideCheckColumnValue for consistency?


I think you meant `end` and not `!end`


+# 5) Go to   C:\Python27\Scripts\pip install pandas


IIRC anything > 4.6 would do, it was probably llvm-or1k/llvmpy requiring a recent gcc runtime (see the debian packages). Try running without to see whether this is still required or what required it.


auto const &


Tags: tokens, lexical.


Whoo! This is the problem, great find!


1640-AddHVACPart1 (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2130 of 2209 tests passed)

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


We may want to abstract the ItemBucketizedBySection -> NSIndexSet+NSArray conversion


Not a big deal, I've seen enough new users run into the issue that it's probably better if it just works even if it's slower. Also 0.4 will be out sometime in the next 6 months, I hope :)


Commenting to see what this does.


same as above


Okay.


Thanks! 1 beer @changetip


Hi @evz, @gorillamania sent you a Bitcoin tip worth 1 beer (14,879 bits/$3.50), and I'm here to deliver it ➔ **[collect your tip](https://www.changetip.com/collect/842619)**.

**[Learn more about ChangeTip](https://www.changetip.com/tip-online/github)**


@changetip give @folz a beer too


Hi @folz, @gorillamania sent you a Bitcoin tip worth a beer (14,879 bits/$3.50), and I'm here to deliver it ➔ **[collect your tip](https://www.changetip.com/collect/842620)**.

**[Learn more about ChangeTip](https://www.changetip.com/tip-online/github)**


Yes, i mean "if you have any doubt".

Thank you very much, maybe could do a gist for this, i haven't a blog.

Thank you.


Hi,
if I change
$row['contact_display_name']
to
$display

the names dissapear...
Where is $display defined???


I'm not sure why we have to pass the intersects here. Isn't enough to just create the  ReadConf Write Conf objects? The defaults will carry through if we haven't set anything? If not I think we should de convert all the table settings properties first (replace ("_",".") so we don't have to keep converting all the individual conf settings to match.


Hydra is getting a different hash. Is it possible it has changed in the meantime?
http://hydra.nixos.org/build/22247997


Why the conditional define? Is this sometimes slower?


The application doesn't recognize easily the name-space when there is no backslash "\"


I wonder if we should change all int/Integer to long/Long to support Big Data row counts, I see no reason to limit the max value to int. I know the original `ColumnRangePartitioner` also used int values, not sure why though. Should we remove that class BTW, it's no longer used AFAIK. If we do we should change the `ColumnRangePartitionerTest` to cover this new `IncrementalColumnRangePartitioner` class instead.


train <- read.csv("train.csv", header = TRUE)
test <- read.csv("test.csv", header = TRUE)
test.survived <- data.frame(survived = rep("None", nrow(test)), test[,])

all this work fine but the below does not

data.combined <- rbind(train, test.survived)
Error in match.names(clabs, names(xi)) :
  names do not match previous names

this is section is not working for me

this is my email john2salamm@gmail.com


I think you meant org.unit


Please can you amend/flatten with a better description than "work in progress"


need to re-generate the docs


should be indented


I actually don't know which method would be better, since both are just looking if the type is called "Mod".


what I my doing wrong?


1640-AddHVACPart1 (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Looks gooD!


There are some debuggers being built, take a look at https://github.com/mailgun/godebug and https://github.com/derekparker/delve.


Great point! Thank you!


Line 210


Thank you!


It looks like it's working. I was able to add extra parameters over the FFI (specifically all the fancy math rending stuff for Markdown -> HTML/LaTeX), so it _should_ be safe for merging. Do you mind if I add myself as a contributor on the `README`?


No, `!end` is what I meant but I suppose it isn't really required. The thought in my head at the time was to confirm that strtol hadn't set end to `NULL` before using the result of `end - start`. But considering the result in that case is unlikely to equal `input.length()` it isn't that meaningful of a check to have I suppose.


Resolves issue #23 #24


Thank you!  There is a button on the keyboard "done" for the user to hide the keyboard when he/she stops typing in. But another thing to consider is to put the countdown timer at the top of the screen so that it should be always visible! Thank you!


I’m not sure what this code is doing, but I suspect it is what causing combining marks to break shaping since the BiDi type of the mark will be ON which is pretty much always not the type of its base character. A big chunk of the BiDi algorithm is basically just resolving the BiDi types of different characters based on their context, so if this code is used for splitting text into different runs, then it is unlikely to work as intended.


Also, if there was a situation where strtol set end to `NULL` I bet there would be all sorts of segfaults in C programs doing this check via `*end != '\0'`.


Version 3.0.3 ist the first public beta. You can download it from https://github.com/osxfuse/osxfuse/releases.


The Holy Push !!!
Thank you so much !!!!


TeamCity LabkeyTrunk :: CDS postgres 9.3 [Build 9](https://teamcity.labkey.org/viewLog.html?buildId=250466&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **SUCCESS**
Summary: Tests passed: 26 Build time: 00:44:22


Alright, I changed it: #99


Oh, right. Unix time.


Listening on https://{}


I'm curios,  ./configure --help tells the following:
--enable-gauge-tools    Build auxilary gauge tools: plaquette, gauge
                          evolver, APE, extended gauge routines (default:
                          disabled)
while it seems that make.inc.in always enables it for a number of fermion operators,  e.g. see line 212 etc. Any comment?


You are right, stupid of me...
It is past midnight here and my eyes are giving up...


Could try out /**/*.* to maybe prevent EPERM error.


"pull -r" will prevent you from needing to push these merge commits.


Tuple is actually a construct in other programming languages like Python. You should probably call it a row. It is really just a hash.