This is not directly related to the oracle change, but...

If I got this right all `detect_vm_*()` functions return 1 on success.
Agreed? So this should be the correct code, no?

Do you want me to split this into two commits?


Yeah, and I don't think you should force unload RVM in the CLI -- this can break when you actually want to use RVM, and can be extra confusing because it's not clear it's doing that.

If you need another entry point, add it to https://github.com/conjurinc/omnibus-conjur/blob/master/config/software/conjur-cli.rb#L14


Did we end up in an infinite loop here then?

This fix reintroduces another bug, and I'm beginning to think the architecture here is a little flawed.
The bug it reintroduces is that now stream flow control updates (in writable_window_update_stream, which is iterated on line 107) can be sent before the stream is announced to the peer, since in this loop headers are allowed to go out even if there's no transport level window - because headers aren't flow controlled.

I thought about a fix where we check transport_global->outgoing_window before re-adding the stream to the writable list, but that would imply needing to scan all streams whenever the transport window is replenished, which is probably bad behavior we'd want to avoid.

Maybe it's worth introducing another list to track 'writable streams that are waiting for transport window to open up'.

It's likely worth merging the stream window update list and the writable list to make this more robust.

The specific fix here is likely going to be to keep track of the first stream that we re-add to the writable list, and to terminate the iteration if we reach it again, but ensure that we re-add that stream to the head of the list to ensure fairness.

I think all this ends up touching #2606 also.

I'm happy to pick this fix up if you'd like - let's talk when you're in maybe.


For doing this mod, you'll need Dan's permission. I asked him once, and he said "no" "because it's a violation of my IP"


Noyo codo, best codo


Consider moving this model to models.js


you should add a test on two wildcards mappings


There's 0% chance he'll aggree with u.


@bendavid The only place it is needed is here. All other conversions to GBRForest have been for grad boost so far.


just to confirm I understand: what this is saying is we must zero the place we are moving FROM, because we don't have enough precision to know what drop flag to use, if indeed there is one? If so, I think the comment could be clearer.


rsa是用别人写好的，最新一次commit有原作者的声明信息。ps我第一次看到的时候也惊呆了→_→


No When ?


@jv110 Its fine just help please. I was the one who had the initial idea of it on pe so complain to me


Hmm...it shouldn't be cleaning up baselines, only the non baseline files. Let me double check.


This looks wrong, and also redundant.

Redundant, since loop_write() already does this.

And wrong, since UDP messages must be written in a single send() or write(), if you issue them in multiple, then this will result in two dgrams on the wire.


I love your get and fetch solution!

I'm ok with .set being async since it's consistent.  In a sense, it's the .get call that is the outlier: almost everything else in hoodie-land returns a promise (but no, I'm not suggesting to make it async as well). I think set/get/fetch is a great solution.


Please add accessControl="#managers"


This is very good for code sanityzing, runtime performance, and also, to avoid later missbehavior/programming mistakes!
I like the const change! :)


What? Someone else made a mod like this before?


OK although the change to line 172 isn't about the copyright year. :-)


@onebone <3


no When ?


Recursive works only on address mapping ?


Where do I read about the Emacs conventions about extension keybindings?

Without taking them into consideration, do you think `M-n`/`M-p` are inappropriate?

My choice of a single-chord is motivated by the ergonomics of navigation between the errors, necessitated by the process of discovery of how your program might be typed.


我觉得可以不用导入包


Please be over protective and also check here if the current user is in the #managers group, just in case there is some sort of modifications in the policy at the content management level


@RandomAltThing it doesn't even make sense to make this mod; if you do it, it will get deleted for not having permission


@tommygithub this is indeed a bit better, you've simplified it by removing one "if" statement.

Now I'd like you to think if there is a way you can get rid of this "if" statement too.  Hint:

https://msdn.microsoft.com/en-us/library/ty67wk28.aspx


Same check as read method please


Unless I am mistaken, this change drops binding of the dgram emission to the ifindex, no? I am pretty sure we should continue to ensure the dgram leaves the host on the iface the DNS server belongs to though, and thus replacing the sendmsg() with write() cannot really work. Or am I missing something?


Okay, Thanks.


I'm sorry, it slipped.. and somehow I didn't notice it -- probably due to latter commits.


Who's permission?


Whoops forgot to switch to master.

On Thursday, July 23, 2015, Philip Shilling <notifications@github.com>
wrote:

> Did you mean to push this to the new branch? :P
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RedstoneLamp/RedstoneLamp/commit/55dd7c8572570d2b31d3690870253c9ec724bfb7#commitcomment-12318727>
> .
>

--


i think it would be good to suffix this with "_usec" or "_timestamp" or so, to indicate that this is a timestamp... the same way as we prefix counters with "n_", and so on...


Will do.


Nope, I just updated the helpers for ``photutils`` and wondered whether indeed there wasn't any change to ``ah_bootstrap.py`` (as usually there is) and thus found out about it.


@CrazyWolf23 No. That's because ComputerCraft is a PC mod (my favorite mod :D) made by dan200 and to do a PE version you'll need his permission. That's exactly why there's no mods like this in PE, because Dan doesn't grant permission.


My attempt via github desktop


@1000miles You should only one style of string concatenation at the time. So:
`"Comment by: " + @comment.user.name`
or
"Comment by: #{@comment.user.name}"


Very nice!
I had it crash on me as well but thought it was gsdumpgui's fault.


I have doubts if I want to have to NULL or just a plain "-" ... nevertheless this is not a critical issue.


I have considered using it, but wasn't it introduced in emacs-24.4?


ok


Not sure this matters, but food for thought: shouldn't this be the timestamp when we *began* the failed attempt, rather than the timestamp we timed it out?


:+1: on this commit


The `as.logical(seqnames(exons) == chr)` step solves the subsetting error you were getting @nishika.

I also had to add the `if(!is.null(coverage[[chr))` part because the bigwig files might not have coverage data for all chromosomes you are looking at.


I like! Thanks!


@informatic code
Trying to remove bug and present full working examples somewhere online.


@snopoke this is how I have done the workaround. See also tests for more info https://github.com/dimagi/commcare-hq/commit/17272a2870b6465b6a3f3b2032ca9c8cfacb7dce


@1000miles Are you sure you want to have those `{` and `}` here? I'm not sure how they will behave when rendered.


@Gr2m - you might find it interesting to see how the ampersandjs folks dealt with this: https://github.com/AmpersandJS/ampersand-rest-collection/blob/master/README.md
(Perhaps that's why I like your solution!). They do just what you suggest: fetch goes to the server, get retrieves locally, and then they also have a convenience getOrFetch - if you're not sure if the collection has the item you want yet (if it does just return it, if not fetch it).


static哪里去了？


The new output looks good. I merged your new branch into the main one.
GitHub sure makes things easy.

On Thu, Jul 23, 2015 at 12:36 AM, Matt C. Wilson <notifications@github.com>
wrote:

> Hey @Xayrn <https://github.com/Xayrn>, here's the revised patch for the
> CSV class. I saw that card 47, Bless, has two quoted fields back-to-back,
> which was what was tripping up the previous version. I have a test for that
> case too now.
>
> I'd appreciate any review feedback on this.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Xayrn/CardHunterUtils/commit/6294939829ea75d90e375c464794c5ee74a120a1#commitcomment-12313358>
> .
>


Cool! Haven't seen this before! Where did you read about this?


The job is to skip `Warning\n     `, and it does it without validation that it's indeed the substring being skipped..
But I thought that throwing a regex at the thing is an overkill. Is it?

Alternatively, it could be refactored into something that indicates this purpose more clearly, like:

`(cl-subseq text "Warning\n     ")`

What do you think?


It will check and set current user for model in all controller action

and it is thread safe, check the related model


"" is the default


Please unify with guava usage, thank you


577: I had a switch on different colorspaces, but I found out I was best to default on sRGB.


It's actually really useful, because different systems will have different port names.  So the way to make it go on a new machine is to run the program, see what's printed, choose from the list, and update the config file.

Ideally we'd present a menu or something but I think this is fine.

Is this OK with you?  Should I add a comment on why it's in there?


I think true/false parameters are not the best or at least you should add the parameter name.


:+1: on this commit


The default value will be the stringified json, for example:
```
c:'{c={"x":1,"y:2}:json}'
```


it returns negative if something fails (maybe due to EPERM). It returns 0 if the check worked, but no VM was detected. It returns positive if the check worked and a VM was detected. The != 0 check hence means: "return if things failed, or when we detected a VM". you turned it into "return if we detected a VM".


@Kelketek a part of "Vanilla edx-platform support"  task. Added


Thanks for this. I'd been meaning to do something like this for a while.


how about we do `.get`, `.fetch` and then `.update` instead of `.set`? I think that'd make it more clear, because `store.update` is async, too. And I don't know about other front-end MVC thingies, but [Backbone](http://backbonejs.org/#Model) has `.get` / `.set` methods and they are both sync. It has a `.fetch` method which is async, and a `.save` method which is async, too. I prefer `.update` over `.save`, it makes more sense in our context


Ah, okay.  Thanks again for pointing it out.  I was negligent this time about ah_bootstrap.py myself (I looked over the changelog and thought there was nothing changed that would have affected it).


We use it in other places and Emacs 23 tests go through. So I gather it is
ok to use it.


did you build it?


check ok


A unit test on this case would be awesome.


ah indeed, right I think I even wrote it that way ^^


ack


Yeah, it had a different name in the spec but I noticed it was the same
as h2 when I started to refactor.

We have only found use for it when generating AKE and Session keys, so
its on the context of generating such keys.

On 7/23/15 7:08 AM, Ola Bini wrote:
> Isn't this exactly the same as h2? Also, why have it be a method on
> keyManagementContext? Not better to have it as a free floating function?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/twstrike/otr3/commit/c10e355eaa09e4f726e038648251eef0015743a0#commitcomment-12319261>.
>


@xiphiaz I got it sorted. `putenv()`:

> The environment variable will only exist for the duration of the current request

Which I wrongly assumed would be per test. Though I guess the phpunit run technically is one request then, so when the queuetest changed the variable to beanstalkd, the change survived for following tests. So I added a driver restore in the `tearDown()` for that test. And now it's fine, no need to define the sync driver in other classes.


Расскажите немного чем закончилась с историей с не удаляемым кешом?


Fixed in d818bea77a0c8b0565b7c49b8292adbc88f10e78


Which bits do you mean? Stuff like 'minuend', 'subtrahend', etc. are the correct terms for objects in arithmetic.

`rg` is a prefix for _range_; as in a range of values.


Why this assert()? Not obvious to me... (might mean there should be a comment in the sources about it, not that it should be dropped...)


does our linter take care?


Hm, I find it really confusing to compare unsigned values with < 0... But sure, if that's what we are doing I'm sure I'll get used to it ;-)


If you're changing the Gulpfile can you copy the changes into the other apps as well? You should be able to replace the whole file and see only your changes, trying to keep them absolutely consistent.


this is my first repository


the "if (r< 0) {" blocks for both protocols could be unified too and be placed after the big if switch.

Also, might be a case for a "switch (t->scope->family)" statement.


:+1: this is a great improvement. I'll work on the keyword keys separately (probably after my upcoming cyanite  release)


:+1: on this commit


Might be good to add a debug log message here.


@anikalindtner I'm changing the date back, as it is in fact July 23rd and not July 7th ;)


:+1: imo, this is a huge improvement for the settings, thx @NobodysNightmare


This needs review


Hehe, this is indeed retarded. Will change.


still think the timestamp should be reset here too ;-)


wrong  comment


Looks good.


+1 to what @warner said.


If you still want to use this macro can you change it to implement the toolkit, rather than its own HTML?


:boom: This is awesome.


Having difficulty when running `vagrant provision` after changing the above script "p4-setup.p4". Hangs at
Cloning into 'p4factory' and then timeout error.


The application controller method set_current_user  is needed when two things occur:
 - the broker is acting for the employer or employee
 - @person is derived from current_user

Controllers where @person is derived from params do not need set_current_user

For example the broker does not act for the person in the controller directories broker_agencies, exchanges, so I'll look for ways to move this to  a before action on specific controllers but it cannot be at application controller.


@rase- would you be able to release your forked XMLHttpRequest to npmjs.org and then release a version of this library that is not relying on github?

I'm working in an environment where we don't have access to the Internet but are able to bring in packages released to npmjs.org. Currently I'm not able to build a project using this library as npm fails to connect to github to get the dependency.


Looks good, but the while() loop should use a "<" check probably, and return an error if we exit with != offset + rlength i figure. Also, it should be its own function (see other thread).

I think this patch ended up in another series too? There it lacks the bitmap_isclear() check though, doesn't it?


If I understand this correctly, the IP_UNICAST_IF we apply to the socket in dns_scope_socket() is equivalent to PKTINFO on each packet, so the behavior should be unchanged.


Looks fine.


Checking.


ahhh ok.. didnt realise that...


shouldn't there be a define for the "1 << 15" thing, or so? called something like EDNS0_OPT_DO or so?


IMHO If you do a provision, you expect to be nuking everything from orbit. It feels contrary to vagrant practices to assume anything else.


Perhaps vagrant practices aren't always the most practical :) ... if you
have 20 sites and 20 databases you work on consistently. Re-seeding
everything kinda sucks.

On Thu, Jul 23, 2015 at 10:16 AM, Svpernova09 <notifications@github.com>
wrote:

> IMHO If you do a provision, you expect to be nuking everything from orbit.
> It feels contrary to vagrant practices to assume anything else.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/homestead/commit/f2931d94d9598c330925e3ceb45fccd50c66bcae#commitcomment-12323174>
> .
>


still believe the timestamp should be reset too, here! ;-)


FacilityTabGridView (evanweaver) - 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)


Good work, loving it! keep it up

	_____.___.               __      __.__      ._.
	\__  |   | ____  __ __  /  \    /  \__| ____| |
	 /   |   |/  _ \|  |  \ \   \/\/   /  |/    \ |
	 \____   (  <_> )  |  /  \        /|  |   |  \|
	 / ______|\____/|____/    \__/\  / |__|___|  /_
	 \/                            \/          \/\/


Totally agree :)


OK. I'll throw pipe.base.TaskError


Change
```
 +                      help=_("preview the command entered, command will not execute"))
```
to
```
 +                      help=_("preview the command that %s would execute" ) % sys.argv[0])
```
In both places.


no se dbee usar _ y luego mayuscula. son palabras reservadas del C


Missing changes to man pages.


Right, I just meant that as long as we don't need polymorphism, and as long as we don't use any fields in the context, there is no real point in having it be a method.


I have a suspicion that we should be able to remove some of [these](https://github.com/alphagov/smart-answers/blob/08d6cc04a0193ee969ae858cee300f87a5065613/test/data/register-a-birth-questions-and-responses.yml#L33-L42) inputs without loosing any test coverage, but I'll hold off, because now there's a fair amount of logic in the views and we don't have coverage data for `erb` files.


Please implement the following interface:

retrieveObject(String txId);  <-- does not exist yet
retrieveObject(); -> delegates to retrieveVersions with false
retrieveObject(boolean onlyModifications);
retrieveRelations();
retrieveRelations(String txId);
retrieveTransactions();


It should still work fine with ic2


Do we want to return if an error occures? Then we need to add another condition above, no?

```
        if (r < 0 || (r > 0 && !streq(_id, "kvm")))
                 goto finish;
        else
                _id_cpuid = _id;
```


@bluej100 adding browserify as a dependency doesn't really help. For example, you may choose to use webpack instead.

Ideally, NPM modules should never use the browserify-transform field in the package.json. Instead, compiling should be done as build step before the module is published to NPM.

That is exactly what is happening here as the es5 version is created and is the file the package.json file points to.


Ah, if we use that and it works for UDP too, then we should be fine, indeed!


FacilityTabGridView (evanweaver) - 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)


Can you make up a more descriptive commit message? We already see you updated tor.js, so the message is redundant. Also we use present tense to describe changes, because for the readers of your comment it will not lie in the past.


@lwyszomi Thanks for flagging, that's very strange.  This was a standard automated translation update.  Do you know what strings were affected?  Can you submit a PR to revert or fix that change?

I just added Hindi to [transifex](https://www.transifex.com/projects/p/commcare-hq/language/hi/), which is where we manage translations.  Here's our [help page on that](https://wiki.commcarehq.org/display/commcarepublic/Translating+HQ%3A+Transifex+Instructions+for+Translators).  I imagine that will be an easier way to manage translations.  We pull updates from there every Tuesday.  Does it make sense for you guys to use that for Hindi custom reports?


yes, we always want to return if an error occurs. let's not eat errors.


it would probably make more sense for toggle to call `hide` and `show`,


I rarely use the provision command. But my use case is almost entirely based around Homestead configured per project. I think it's dangerous to stray too far away from vagrant practices primarily because I expect vagrant and vagrant projects to be configured a specific way.

With the default Homestead workflow being so primarily abstracted away from the everyday vagrant user I don't necessarily see harm in this, just possible confusion. When i do a provision I'm expecting to go from a clean slate to empty tables ready for me to migrate and seed. Maybe I'm the minority in that thinking.


Excuse my ignorance, why is machine name a series? It's both a series and a tag after this change.


> Perhaps vagrant practices aren't always the most practical :) ... if you have 20 sites and 20 databases you work on consistently. Re-seeding everything kinda sucks.

This. I've spoken to @joecohens about this on several occasions (we both have several big projects) and it's just impractical.


I don’t think this actually tests anything… I believe the intended usage is

    assertThatException(ex).hasType(`RegExpException`);


Would produce a warning iff the tt_news table doesn't have an l18n_parent column. Might want to change this from "$newsRow['l18n_parent']?" to "@$newsRow['l18n_parent']?".


I think you might be :) heh... To me vagrant provision simply asserts that
the machine matches the config... it says nothing about the "contents" of
the database themselves. Only that they exist. Vagrant "destroy" to me
implies that everything has been destroyed and will be recreated fresh on
the next provision.

On Thu, Jul 23, 2015 at 10:25 AM, Svpernova09 <notifications@github.com>
wrote:

> I rarely use the provision command. But my use case is almost entirely
> based around Homestead configured per project. I think it's dangerous to
> stray too far away from vagrant practices primarily because I expect
> vagrant and vagrant projects to be configured a specific way.
>
> With the default Homestead workflow being so primarily abstracted away
> from the everyday vagrant user I don't necessarily see harm in this, just
> possible confusion. When i do a provision I'm expecting to go from a clean
> slate to empty tables ready for me to migrate and seed. Maybe I'm the
> minority in that thinking.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/homestead/commit/f2931d94d9598c330925e3ceb45fccd50c66bcae#commitcomment-12323395>
> .
>


FacilityTabGridView (evanweaver) - 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)


you mean set_current_person? @jfinucane

I agree, the new before action for set_current_person should not be in application controller. and for limited action, that will reduce the duplicated code


Oeps, die dubbelingen waren idd niet de bedoeling. Zoals je zelf ook al constateert: terug nr oorspronkelijke tabel voor status en valide geometrie.


GG


To influxdb, what is the difference between a tag and a field


Brilliant codes


@Lewis-Kang Copy and paster error frmo the as5712


Yeah, I had thought about it... this way is a bit shorter, but you're probably right: It's better having just two functions


Ebben az avidemuxban 2.6.1 -es belső ffmpeg van,én aközösbe egy ffmpeg frissítéstől egyelőre ódzkodom.
Én használom aktívan az avidemuxot, az x265 frissítés csak a fordíthatóság miatt kellett.
Próbaképp tömörítettem x265 -el, de semmivel nem lehet azt lejátszani, csak az avidemuxszal.

A gtk rész is fordítható egy patkóval, csak futási hibával még mindig elszáll.

diff -Naur orig/avidemux/gtk/CMakeLists.txt mod/avidemux/gtk/CMakeLists.txt
--- orig/avidemux/gtk/CMakeLists.txt	2015-07-23 16:24:13.000000000 +0200
+++ mod/avidemux/gtk/CMakeLists.txt	2015-07-23 17:06:31.392084915 +0200
@@ -34,6 +34,7 @@

 SDLify(../common/main.cpp)
 ADD_SOURCE_CFLAGS(../common/main.cpp "-DADM_SUBVERSION=\"${ADM_SUBVERSION}\"")
+ADD_SOURCE_CFLAGS(../common/main.cpp "-DADM_VERSION='\"${AVIDEMUX_VERSION}\"'")

 ###########################################
 # Version Info


@milythael

do you have any suggestion here? I just though, there are some duplicated code, could be before action.


I'm fine with that, (being in the minority and with your explanation of provision VS destroy)


Hm, redundant, possibly (had missed the helper function we have), but the behavior should be unchanged compared to using manager_send(), the only difference is that we no longer pass explicit control data along.

Regardless of that, I don't see how it is wrong, this is not about splitting up the packet (unlike loop_write()), but about retrying to send it if it failed to send altogether. Am I missing the point?


Frankly speaking, I still don't understand why your compiler would complain about `math.h`. Here it is always referred to as `lemon/math.h`. In no way can this be in conflict with the system `math.h`.


cause you do this check in the constructor of the iterator, I suggest you remove it here…


chez moi, immolation s'affiche bien. le "." ne changera rien.

pour rhoven, la "qualification" est purement textuelle, donc la changer ne permettra pas d'intégrer le tier. pour ça il faut changer le code dans le fichier tiers_menoth.xml et rajouter l'entrée dans la zone qui va bien.

<only>
        <entry id="xxxx"/>

pour permettre qu'il soit intégré, mais ça demande de recompiler l'appli et republier. je me le note en "issue" sur github pour ne pas l'oublier à la rentrée.


No, there's not a recent enough version of tornado packaged, hence we require the `-P` flag to be passed.
SaltStack is trying aleviate this issue by providing packages as soon as possible.


typo on required.


Jshint is a program to help us find JavaScript errors (missing ;, etc). The jshintrc is the config file. I just copied one from another project. I should clean it a little bit...


This looks like a nice addition to me. @nickloman any concerns?


You're absolutely right, thx! (Fixed)


How does this change work? Under what conditions does matlab.lang.makeValidName fail? Why can we assume that if matlab.lang.makeValidName fails, the varName is still valid? Shouldn't this be an error?


why if element hidden you can access it's text?


@armadillu no it has not. I got the coding style backwards. I am going to fix this soon. Good catch anyway!


Lo cambio por RTCS_H_?


looks great!


got it thanks


gd


oh, indeed, i am an idiot. ignore me!


오여긴댓글달아짐


Aqui seria legal extrair a chave "livro" para uma constante em uma classe específica para esse fim. Desse modo, não precisamos ficar decorando as chaves e evitamos erros de digitação, sem contar que ganhamos também o auto-complete da IDE.


maintenance branch ?


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


Couldn't resist...
https://github.com/StephanHoyer/translate.js/commit/6ec38a533ccc2f806a70958a6a3b804723e6109a
Nice to catch this nasty performance regression in non-chrome browsers.


note.
0)Inerface - typo
1) in java conventions, camelcase looks like IBeaconInterface


I disagree with the change: now you compute many times a linear problem in the loop. The original code isn't efficient either. invSigma should be computed only once and increase should appear at the denominator of original line 84


pls rename


> I've found where chg_unknown_bat is used
That only say that battery gauge will not work if it's disabled (default) --> constant 100% ...
But probably also no charging..: check this:

https://github.com/TeamCanjica/Samsung_STE_Kernel/blob/ada5cb730330749534576adcaad0df8ac9062e5a/include/linux/mfd/abx500.h#L427

That's why i've enabled this.

The Charger identify an "org." battery with an 1.5k resistor .. so maybe up to now the non-org. batteries was also misdetected as org. battery ? And exactly this is the problem.. because then a 3.7V battery with max. allowed Volt.: 4.20V will charge @ 4.35V .. and that's why such batteries will life much shorten then an org. battery:

https://github.com/ace2nutzer/Samsung_STE_Kernel/blob/3465af24f0972350cd8b764f45699a0d98ed0d11/arch/arm/mach-ux500/board-codina-bm.c#L375

So maybe a good idea is to ask in Forum if someone with such non.org. battery @ 3.7V the max. voltage will be 4.10V (100%) (correct) or wrong: 4.25V ... then we know if that works or not ;-)


El template dice por eso TEMPLATE_FILE_H, yo diria RTCS_H por ende. :)


Uncaught TypeError: Cannot read property 'update' of null

Running latest master. This should check for scope.chart existance, as per latest comments here: https://github.com/krispo/angular-nvd3/issues/109


We should never advertise smaller packet size than the default 512 bytes. We enforce this elsewhere, but assert here just in case. Is it the constant name that is confusing? DNS_PACKET_UNICAST_SIZE_MAX is now the "default/legacy max", but we can (and should) advertise larger sizes.


Lembre-se de deixar a sua aplicação sempre preparada para internacionalização. Aqui seria legal você utilizar aquela ideia de extrair todos os textos da sua aplicação para um XML. Depois basta referenciar esses textos através de seus IDs. Verifique o resto do seu código também para substituir todos textos hard-coded por referências às strings do XML.


@paveldruzyak it check if element is visible `this.isVisible()`


GJ


Page published


Solves https://github.com/yhoogstrate/segmentation-fold/issues/7


ㄷ


it seems awful NPE-prone code. Is it really necessary?


Don't spamming here.


Final


holy place


Let's add a `flag` or `promote` OP to note that it's an admin making changes not the user.
This will make it easier to update the cron that retroactively runs to search for instances of reportbacks being flagged/promoted & we can add the reason


>But we don't have time to make all that right now they just crashed the server two times in a row with fucking buttbots, come on guys.

10/10


@tavriaforever ничем, потратили некоторым количеством человек overdofiga времени, оставили матч с ошибкой на сам модуль https://github.com/enb-make/enb-diverse-js/commit/7879d37e11d408ca3ff40da9af61bc0a2a401474#diff-b57fdf30ce49f0637363fc0c68699588L184


ㅎㅇ


Consider using servers.empty() instead.


Does this handle rename/save-as?
eg: blacklist *.old, rename file.old -> file.php


Добави и scope-а `Issue#open`, за да не се показват затворените


Fuck PHP


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.405.1d9cc6b868228fb3f33a369becf9c22b6f38b8bb](http://build.palaso.org/viewLog.html?buildId=59958&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3319, ignored: 130 Build time: 00:09:23


Hey @keithgee and @kayiwa, can I indeed delete this repo now?


Looks good to me, let's merge!


New change themes


Não é o foco agora mas depois certifique-se de utilizar o padrão View Holder no seu adapter. Vai servir como treino do conteúdo do 59 e também vai tornar a interação com a lista mais agradável para o usuário.


Thanks for being so polite and friendly. Will accept pull req only because you're so nice.


@jv110 ComputerCraft isint a copyrighted name. I could just make it *ComputersCraft"* as that name is COMPLETELY different from the PC version.


I am not terribly happy with these changes. What do browsers do?


Will not work if object was delete. Results in:

  File "/home/jens/PyLucid_env/src/django-reversion-compare/reversion_compare/admin.py", line 105, in get_related
    if hasattr(obj, self.field.name):
  File "/home/jens/PyLucid_env/lib/python3.4/site-packages/django/db/models/fields/related.py", line 572, in __get__
    rel_obj = qs.get()
  File "/home/jens/PyLucid_env/lib/python3.4/site-packages/django/db/models/query.py", line 357, in get
    self.model._meta.object_name)
tests.models.DoesNotExist: Factory matching query does not exist.

Fix is on the way...


No. We are not sure if we want to bring @InjectPromise back. It did not work well with Angular components and directives because they are created synchronously.


This was just changed in the spec. 's how it is now...


D'oh!


It had something to do with git checkout, I was able to fix this by doing this:

https://github.com/xamarin/monodroid/commit/c028f73cc8862f15dbc0c4be2981bb6f5e7a6155

The build is now failing somewhere else.


@hlee I think we don't need specs to test removal of a feature/text.
There should be a team-wide policy for this.


But we know the spec is deficient in file URLs. How do browsers behave?


@route after I updated poltergeist to v1.6.0 this method starts returning empty array:
`all('.visible-to ul li', visible: false).map(&:text).uniq`
I checked it, if found elements and has all attributes, but `text` is `nil`. I think because of this change.


Future project: use URLmaster + similar code to that deleted here to auto-generate a ton of web-platform-tests.


i think if we have a comment here in the sources explainaing what you just explained, then we should be good.


And so it begins...


@moy Even with the new implementation which Junio has suggested, has this problem, will see what I can do, thanks for pointing this out :+1:


Well they still treat file as a schema, similiar to http.

The spec treats it now like about: basically.


What are the use cases for this?


Please, put the scalar at the end of the matrix expression. Moreover, chaining / and * and relying on precedence of operators is confusing


"view read"


We should also catch `IOError`, as those can occur as well:

>  If the connection cannot be made the IOError exception is raised.

source: https://docs.python.org/2/library/urllib.html


Can we rename `s_file` to `cached_session_file` since the replacement's scope is much bigger than the original?

As a result let's also add a default value in the setup section at the top of the file and a quick comment about what it's used for (you should probably rebase as I added a few comment strings).


Daylighting_Speed (DeadParrot) - 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-87.01%25-red.svg)


今天主要是捣鼓这些软件怎么用


That's no good then. We need to avoid regressing, so we'll need to float some patches on the spec :-/.


还有熟悉一些怎么用markdown


fix here: https://github.com/jedie/django-reversion-compare/commit/75c9a4852b13edb44e5a41cec3d1837b59dc652a


Daylighting_Speed (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


Michael, in the ModelEdit.js there is already a hook to go back to the previous page after a save. Rather than forcing the user to go back every time we had made this configurable. Also, your code change blocks other logic in the save() function to work.

What you should do is in your config.json options property set the `returnAfterEdit` property to `true`

Check out the contents of the save() function:

```
if (callback) {
  callback();
} else if ($scope.action.options && $scope.action.options.returnAfterEdit) {
  $window.history.back();
} else {
  //reload data
  if (!$scope.section) {
    //No section identified, so likely not called from main navigation via config.json
    //Instead likely called from Modal Popup
    if (modalInstance) modalInstance.close();
  } else {
    $state.go("dashboard.model.action.edit", { model: $scope.section.path, action: $scope.action.label, id:response[$scope.action.options.key] });
  }
}
if (modalInstance) modalInstance.close();
```


Can you revert a commit to development?


O que o código da linha 30 até a 42 faz? Aparentemente, ele cria uma instância de Livro e preenche com os dados contidos no JSONObject. Podemos deixar essa parte do código mais clara com uma pequena refatoração. Para isso, crie um novo método para fazer apenas isso: criar um novo livro a partir de um JSONObject. Assim o código do fromJSON vai ficar bem fácil de entender prá quem for olhar o seu código.


additional file


Aqui vale a pena extrair também as chaves do JSON para constantes em alguma classe. Desse modo, se mudar alguma coisa no formato do JSON, fica fácil para qualquer um fazer as alterações necessárias para ajustá-lo ao novo formato.


build passed: https://travis-ci.org/jedie/django-reversion-compare/builds/72317865


This seems cool, but, tests showing what it actually affects?


Motivation?


enumerable: true, configurable: true


"Hello,World!\n"


Daylighting_Speed (DeadParrot) - 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)


Lembre-se de tratar essa exceção e mostrar alguma mensagem no seu aplicativo (no local adequado, claro). Caso contrário, a sua aplicação irá fechar e o usuário não ficará muito feliz. :)


Thanks for the report -- you can change your keyboard shortcut for either plugin, or maybe just use both for a while then decide which one meets your needs better (I think mine achieves the same goals with far fewer steps).

Regarding keeping the button centered, that would be a new variant of the plugin.  I wish I had time to write that for you, but it may be several months from now!


update


The lower.cpp and lsra.cpp can be in the common JIT sources above.


It's probably worth adding a note to the docs that people should test their confirmation e-mail links after this change. There could be potentially serious consequences here if, say, before it was directing you to www.example.com/... and now it directs you to example.com/... for the confirmation. The reason being that site.domain may not contain "www" since doing so can cause complications with e-mails sent out from webmaster@example.com.


Yeah, the url_resolution tests with base fail otherwise.


add sailfish-weather
and do not forget about jolla-settings-layout


w3c resolution tests use it now


@siemieniec thanks. i added the "+" and split the string into 2 to test/learn, if there are stylings possible. :D


Do we need to reward score if there isn't more_than_one comboattack?   I expected : 0;

score += more_than_one(comboattacks) ? ComboAttack * popcount<Max15>(comboattacks) : 0;


I'm removing all of ProQuest's styling so that I didn't have to overwrite it with new CSS. Could you be more specific about what kind of problem you're having with the script at that line? If you've got an iframe in there somewhere you might need to specify the first head (head[0]) or something?

-M


done


Oh OK neat.


What's this magic here?


Why can I only do comment...


Let's add `l:file` back and use that locally. If the caching option is enabled, let's set `s:cached_session_file` here.

My main rationale is I'd like to keep the scope of variables as small as possible.


Awesome OK


closes #86


Yes on the infinite loop. I was not sure whether this will introduce other bugs thus I included a test which can reproduce the infinite loop. What you said makes sense and please take the fix. Thanks. I will be in a bit late today.


Added. Thanks!


I do need to return the rewritten token here, don't I?


Do we need that check for other TileEntities, too?


Haha do you have a watch on my entire username??

On Thu, Jul 23, 2015 at 9:31 AM, Bryson Hill <notifications@github.com>
wrote:

> typo on required.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BrockNeilson/devkittens/commit/be5114923b6bc9c782ded3320ca46d2dbb379534#commitcomment-12323521>
> .
>

--
Brock M. Neilson


@jkotas, do you have any idea why there are ifdefs for AMD64 in the codegen file for ARM?


Nice Work!!!


Consider replacing c style cast with time_t(-1) instead. Or simply use -1 as we are doing is comparing two integers.


FacilityTabGridView (evanweaver) - 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)


Same as `GitSessionSave`, delete cached version if cache option is set, otherwise keep the old behavior.


Any chance we can make this cross platform?

Also could you please update the /doc/compilation.dox with the steps necessary to use python (in Ubuntu) and boost. If all packages are preinstalled then at least list python and boost as depdencies now.


:ok:


Ditto `GitSessionSave` and `GitSessionDelete`.


Hey @pei-jung , why did you need to do this?


it'd be really great if you could change this to

> sloth's party

i think sloth would appreciate having the ownership.


Nit: The if should be on a new line, or you should use elif


I know however this reduces the changes for developers to use the legacy
JIT whilst RyuJIT's backend is non functional. That said having looked the
other files are guarded by #ifndef LEGACY_BACKEND so I could still add them
to common, if you think that'd be better.

On Thu, 23 Jul 2015 16:53 Jan Vorlicek <notifications@github.com> wrote:

> The lower.cpp and lsra.cpp can be in the common JIT sources above.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/benpye/coreclr/commit/8eb4dcd4b5287283996d0e6291595de90e3788b4#commitcomment-12323968>
> .
>


Didn't we agree to keep the generated files checked in?


Can this be declared before the lock?


Te most leforgattad a 2.6.1-et az avidemux-hoz, én a 2.6.3-at az xbmc-hez (kodi).
A rendszeren meg van 2.2.4.
Minek?


I'll answer myself, it's because it cannot access it from the parent tag. Right?


I normally don't like repeating stuff, but if you think it is a good idea, I don't mind.


Hmm, generally I don't like checking things in DebugBuild. Can't we put this script somewhere in IS/test/scripts/.


If we check this in, we wouldn't need the hack :)


Can you describe the point of the hardReload argument and how its used?


I think this needs a rebase on origin/master. It is missing the recent scollector pprof changes


git bisect showed this commit to be the cause.


Accepted.


@tbuchok thanks a lot, landed at https://github.com/tbuchok/fun-with-gitz/commit/4ac3556bd8959f4f92bcc63f0d6352a26ebba936#commitcomment-12324086


It would be nice to add comment with the stack layout like the AMD64 specific file does. It helped me many times when debugging stuff to visualize the expected layout.


You should walk me through these calls over here and what each of them does.


Yeah, if I don't pass in a bean, there's no bean available from the parent tag.


LGTM other than being out of date with current master


how array length can be less than 0? :)


Instead of using `skipSpaces`, would it be more straightforward trim `raw` before registering the implicit header identifier?

Actually, I note this is a general problem for reference keys. I prefer the commonmark behavior here, which ignores leading and trailing spaces in the keys:

http://johnmacfarlane.net/babelmark2/?normalize=1&text=%5B+hi+%5D%3A+url%0A%0A%5Bhi%5D%0A

I think the whole problem could be solved if we simply stripped off the leading and trailing square brackets before calling `toKey` (everywhere, not just here).  Note  that `toKey` already normalizes internal spaces; the edge case is just space after `[` or before `]`.

Doing this would remove the need for the `skipSpaces` here, and improve reference link behavior too.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.410.906d1df991304ea46024abe54f48da36e54b7097](http://build.palaso.org/viewLog.html?buildId=59961&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 859, ignored: 54 Build time: 00:14:32


space


Why did you remove the upload-button and clear-button?


Consider moving this to right after the lock and changing it to servers_.empty().


Sorry, I'm definitely behind the current version, but what is the significance of the change to ProtGenerics and the adaptation of xcms peaks()?


@tbuchok thanks, here's where I landed https://github.com/tbuchok/fun-with-gitz/commit/582470979a342c50473c5e69c3fd33e431f72d6e


maybe we should just use namespace boost and leave the python explicit in all the calls below. Make it slightly easier to sport what comes from it and what parts you've coded.


Hm probably.. It was long ago, but from the first sight I can remember that this behavior had been changed in capybara and I had to do this change in order to make them compatible, because of a few falling tests. So overall you have to go back, revert my change and run poltergeist specs, it will show you those failing specs


Also added ability to pass SecurityType in SetBenchmark


ok, looks good


Why did you mark these functions as not stack safe? They recurse on `Free`'s `>>=`, which is stack safe.


Some of this seems to already be documented at https://github.com/jitsi/jitsi-videobridge/blob/master/doc/rest-videobridge.md


I definitely need an explanation here.


Thanks for this and the packages (tickets 4402 and 4403)


@Acefox17 confused about this change–don't we need the allSettled so it doesn't hang when trying to load deleted records?


I realize now, that when someone introduces another framework, it's really difficult to do code reviews as one of the people has no clue what's going on.


Fix move/copy


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.406.1d9cc6b868228fb3f33a369becf9c22b6f38b8bb](http://build.palaso.org/viewLog.html?buildId=59963&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3319, ignored: 130 Build time: 00:04:55


It seems it would be better to use #ifdef _AMD64_


I don't need it.


Keep it simple. Why duplicate code when it's easily avoidable?
```java
return (isTests || !root.isTests()) && file.getName().endsWith(".erl") ? file.getAbsolutePath() : null;
```


I'm surprised by this deletion.


The same here


har bir qatordan oldin * qoysang list itemday bo'ladi, bo'lmasa hammasi bir qatorda yozilganday ko'rinadi: https://github.com/Polinbek/uzbek-words/blob/master/README.md


oxirigisa ham birlikda bo'lmasinmi?


Why is it important that python determintes the memory where the QString goes?


yumshatish emas ayirish belgisi bo'lishi kerak emasmi?


@lowe0292 , The change on line 338 to line 347 makes a record deleted error resolve instead of reject so we can reject other errors.


Oh oops, maybe you're right... @paf31 suggested they weren't. `mapF` is implemented in terms of `liftA1` so that should be safe too, right?


A nit - please add newline at the end of the last line


Let's squash this into the previous commit


This is my first repository.


What is the meaning of sqrt_controller? I don't know know why the equation is safe_sqrt(2.0f*second_ord_lim*(error-(linear_dist/2.0f))). Can you give me more document?


Uf, why :)


Yes. They're implemented in terms of recursion on Free's own `Apply`/`Applicative`/`Monad`, which is stack safe.


In previous version of this pull request I wrote about this %s, yet you've copied it again.

In these branches you only have a single call with second argument different in each branch. Introduce a variable for it and use a ternary operator - it will get a lot prettier.


This should make use of `foreach_reverse` added in https://github.com/awesomeWM/awesome/pull/340.


No, it isn't.  I lumped two fixes into a single commit, which is bad form on my part.

I'll strive not to do that in the future.


The string on this line get's set as a shared variable in the next commit to make this less fragile.


Looks nicer. I will take this approach. Thanks!


@rakatyal Can we just add this preference via the plugin.xml?


Heck yeah! Been waiting for this.


Thats a good point. This may be a moment when we need to start weighing
different strategies against others to see what gets better results. I
might make a branch to try this out, but I suspect our score would go down
by only replacing infered playables with reservable clues. although we
still havent burned fuses, so I may try it with both first and see what
happens.

On Thu, Jul 23, 2015 at 12:41 AM, Mark Spurlock <notifications@github.com>
wrote:

> Debug mode, nice.
>
> I was just playing with something like this, noting that we discard a lot
> of reserved cards. Working on improving that, but it's tough because the
> new inferred_playable stuff is working against us trying to give "reserve"
> clues.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Spurlock/Hanabi/commit/b1b24eea3bddb244e16e8a821e23d803ec1a400d#commitcomment-12313425>
> .
>


I will need some time to consider this and don't have it today.  I will get back to you.


#MoRequiem 2015
#The following line will make you insert the cost of your meal, not including taxes.
meal = float(input("-What is the cost of the meal?    \n"))

#The tax is set on kenosha restaurant tax, edit if needed; also tip is 15 percent, later will add question for increase or decrease in tip
tax = 0.055
tip = 0.15

#The following two lines will calculate the tax on your meal, and the total with taxes
tax = meal * tax
meal = meal + tax

#The following two lines calculate the tip to give and the subtotal
tip = meal * tip
total = meal + tip

#The following set of lines displays the finished calculations in: total, tip, and subtotal
print ("""-Total: ${0:10.2f}
-Tip: ${1:10.2f}
-Subtotal: ${2:10.2f}""".format(meal, tip, total))


I did this just for the main prompt but there is a lot of potentially translatable text in the classes of the ``magic.model.choice`` package which I have no intention of looking at for now. In any case, I would be inclined to only load these strings (and all gameplay-related strings) while a game is running, keeping them separate from the non-game UI text and unloading from memory once a game is finished.
![screenshot00199](https://cloud.githubusercontent.com/assets/2443519/8855623/146a44b2-315e-11e5-9c25-36be4488f6f5.png)


I'm ok with renaming factor. But, I really dislike the name pose. Is there
any reason to have factor at all? Any code that uses it can be rewritten to
use
weight (what's now called pose) instead.

On Wed, Jul 22, 2015 at 8:05 PM, wren gayle romano <notifications@github.com
> wrote:

> I pulled that name from somewhere else in the code base. But, as we
> discussed at the PI meeting, the name "factor" is horribly overloaded. I
> just pushed a patch which renames things to avoid the word "factor"
> entirely, and added documentation about why that's a terrible word.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hakaru-dev/hakaru/commit/05fdf7e637a3452bbc5346a2678978bdbd7a37bd#commitcomment-12310709>
> .
>


Oh dear. That was not the intention.


I didn't notice, what do we need this for?


#MoRequiem 2015
#The following line will make you insert the cost of your meal, not including taxes.
meal = float(input("-What is the cost of the meal?    \n"))

#The tax is set on kenosha restaurant tax, edit if needed; also tip is 15 percent, later will add question for increase or decrease in tip
tax = 0.055
tip = 0.15

#The following two lines will calculate the tax on your meal, and the total with taxes
tax = meal * tax
meal = meal + tax

#The following two lines calculate the tip to give and the subtotal
tip = meal * tip
total = meal + tip

#The following set of lines displays the finished calculations in: total, tip, and subtotal
print ("""-Total: ${0:10.2f}
-Tip: ${1:10.2f}
-Subtotal: ${2:10.2f}""".format(meal, tip, total))

tipOk = input("Is the tip amount (15%) ok?   \a")

tipOk.uppercase()

if(tipOk == N):
    tip1 = .5
    meal1 = meal * tip1
    print("5% :   ${0:10.2f}}"


Az avidemux az szállítja a 2.6.1 -et, nem külön forgattam le.
2.7.2 -nél tart az ffmpeg. http://ffmpeg.org/download.html
Ekkora ugrástól tartok, van vagy 30 csomag, amit első körben le kellene forgatni vele.
És persze kipróbálni..


Gotcha, thanks!


Avoid nesting where it makes sense to do so. In this case it would be a lot easier to read something like
```java
if (erlangModulePathsToCompile.isEmpty()) {
  reportProgress(context, "Source is up to date");
  return;
}

reportProgress(...);
runErlc(...);
```

It's like checking your preconditions, and then taking an action. Does it make sense?


For some reason, I do not know if it's related to code, but tested with some SRT subtitles ... when there an accent, common in the Portuguese language, it cuts the rest of the sentence, eg "você é legal" appears only "voc"


Fuck yeah removing old code :metal:


sorry, mistake...


commit 2


Consider replacing with the following...

        auto & best = *servers_.begin();
        for (auto & e : servers_)
        {
            if (e.second == time_t(-1) || e.second < best.second)
                best = e;
        }


hi,
i tested your api but i get the error
Strict Standards: Only variables should be passed by reference in /home/admin/web/dirohost.com/public_html/login/modules/addons/sageone/sage.api.php on line 130

this is the line with rawurlencode

i try to get

$data = array(
));

$result = $client->get('/tax_rates', $data);

hope you can help


Shouldn't be public.


I think this makes more sense to extend TestMetadataTestCase.


Not needed.


Weird spacing.


You need to preserve the original signature here. So you should have one function like this, with the original comments:
public static PhoneNumberUtil createInstance(MetadataLoader metadataLoader) {
  return new PhoneNumberUtil(new MultiFileMetadataSourceImpl(metadataLoader),
      CountryCodeToRegionCodeMap.getCountryCodeToRegionCodeMap());
}

And then a second one like what you have.


setInstance(createInstance(new MultiFileMetadataSourceImpl(DEFAULT_METADATA_LOADER)));


You don't need to import com.google.i18n.phonenumbers.Phonemetadata.PhoneMetadataCollection here.


public MultiFileMetadataSourceImpl(MetadataLoader metadataLoader) {
  this(META_DATA_FILE_PREFIX, metadataLoader);
}


muss das sein? so macht der tooltip das design auf smartphones kaputt.


I would put a newline here.


This shouldn't need to be public.


We should talk about this.


This should be moved to the .ipp file since it isn't used outside of the wamp_invocation_impl. Preferably in an anonymous namespace.


Agree with the current implementation (literal and input:file only, marked as non-web resources)


Almost done, just a few small changes.


Can we verify it using the test suite?


Call the other constructor:
public MultiFileMetadataSourceImpl(MetadataLoader metadataLoader) {
  this(META_DATA_FILE_PREFIX, metadataLoader);
}


Our naming convention uses all lower case and underscores for argument names.


A nit - please add newline at the end of the last line


Sure!


Depending on how you use this, you're almost certainly going to need a virtual `clone` method.


unexpected will put it again


Oh man, boost is going to slowly creep up everywhere now :)


Very wide


Yes! This is what a meant. Straight and simple.


Float the liftIO out as far as you can here


what the


Prefer:

using wamp_tcp_session = autobahn::wamp_session<boost::asio::ip::tcp::socket, boost::asio::ip::tcp::socket>;


A newline before isTests. It's the fourth time I comment on this.


I put it again, that being said, it's doc generated from vertx-core.


Our naming convention uses m_all_lower_case style.


no, bc this is the only one that modifies and/or queries the world around it. There's no harm when the other machines (vanilla TE's even) do an extra tick clientside before being destroyed because they don't care about the world state.
But for this, IPedestalItem.updateinpedestal passes world, x, y, z, which the item uses to re-retrieve the tile entity to do things with.
And in the World tick method, TE's are ticked before the "pending removal" ones are removed due to a clientside fault in how it checks for if a chunk is loaded, so we get an extra tick clientside :p


Maybe we should go for the abbreviated names here (Mo, Tu, We, ...)? Would save some space on the screen.


looks like you like canola oil


<h1>hello world<h1>


Put the comma at the end of the line above, please.


What is the meaning of 10957, 30 years? Descriptive const ?


Report progress.


The problem with .update is that it presumes something already exists.  So Hoodie has .add, .update, and .updateOrAdd (somewhat like Ampersand's getOrFetch).
So personally, I would lean towards what you say Backbone does: .set/.get for sync and .save/.fetch for async (preferring 'save' to 'update' since it can be for an add or an update).  To me, the subtlety of add vs update is not really valuable - I've found myself using (or writing my own) "upsert" functions for database calls.  Upsert either inserts or updates depending on if something's already there.  But many/most times, I don't *care* if it was there before, I just know I want the thing in the database.  Of course, this does put a bit of burden on the developer, so that they don't try to save a second copy of something (without using the pre-existing id) when they really should be saving using the existing id.


Is this really part of arrays @lieuwex :P


diao big others 是什么意思啊，这 commit 我也是醉醉的了


Our file naming convention is lower_case_with_underscores.hpp


final


This can throw out_of_range. Maybe catch that? :)


This totally killed my data connection on Candy for the Skyrocket.  The SIM wasn't detected, and the empty triangle icon would flash briefly every 8-10 seconds...


HotFix-ReadVars (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1756 of 1757 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)


Daylighting_Speed (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (742 of 742 tests passed)

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


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.411.906d1df991304ea46024abe54f48da36e54b7097](http://build.palaso.org/viewLog.html?buildId=59966&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 859, ignored: 54 Build time: 00:14:59


The string concatenation is problematic, because you never know what the space you've chosen for your language is best represented as in other languages. For instance, in German I would use (day of week), (date). So, don't hard wire this. Use a  string resource of the form "%s, %s" (and document it very well).


I believe the term that wamp favours is actually "component" as there is no traditional sense of client/server in wamp.


Yes


@lieuwex If EvaluationResult has a destructor that destructs varVal (it should, don't want to check now), then this goes wrong. Maybe copy `this`


See my previous two comments. Same applies here.


Use ```List``` instead of ```ArrayList``` where possible. It's always better to use less restricting types unless you have to use specific ones.


Still 3 processors waiting for deletion :D


Your logic isn't quite right. `last_login=date.today()-timedelta(7))` only finds logins which match that timestamp. You should use `last_login__gte=date.today()-timedelta(7))`.


I went for the long ones to match the Mondays/Tuesdays etc... strings used in the summary field.
Also there is masses of space in the current dialog, although maybe there won't be when someone replaces it with something which shows related EPG data... ;-)
I'd be happy to go with the short ones (the long names come from the 'Weather' section in localized strings) if it makes a difference to getting this merged, but I still prefer the long ones.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.409](http://teamcity.monogame.net/viewLog.html?buildId=9101&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


On the other hand, if it does not destruct varVal, I'd like to see what valgrind makes of the program once arrays are more mature


I'd prefer having common stuff in common list, so please move those two files there.


(y)


You also need to add it to the LD_FLAGS or it will cause problems with avr-gcc 5.1 and LTO at least. Not tested with older compilers.


You want the latest ones. You are currently looking at the first ones. Use `User.objects.all().order_by('-last_login')`


Fair enough. I was just scared about having a function named "h" in the
global namespace for the package. It seemed to be something that can
easily go wrong.

On 7/23/15 10:20 AM, Ola Bini wrote:
> Right, I just meant that as long as we don't need polymorphism, and as
> long as we don't use any fields in the context, there is no real point
> in having it be a method.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/twstrike/otr3/commit/c10e355eaa09e4f726e038648251eef0015743a0#commitcomment-12323274>.
>


Strange, when I did this before I just got a different warning about casting const char* to char*
I was using gcc 4.9 however, there are a lot of new warnings etc


Needs definition of time constant, is this configurable?


good idea


Also, need to define precise algorithm.


Hey @savvas , could you provide me with those Google Analytics stats? Thnx,bb,luv.


Shouldn't that be something like `m_bt.bt_body[i]` here?


The commit message needs to adapted to reflect the actual changes. E.g.: "Turning off 'Any time' defaults to now()" is no longer in.


Ah, my mistake. I naively assumed `== struct` was true for primitive value types as well.


Module name and dependencies should be made final - they can't be reassigned.


In one of previous reviews I asked you to pass module name explicitly - the node has no need to be aware of ```ErlangModuleDescriptor``` at all.


картинку можно  не отправлять


Why is this removed?


Blanks.


Suggestion: call this method `isSyntheticCastMethod` or `isValueClassCastMethod` instead (in several places in the documentation I call them "synthetic cast methods")


Blank line.


I'd use new fields in the deserializer too when it possible


Daylighting_Speed (DeadParrot) - 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-87.22%25-red.svg)


Consider moving before the lock


97902934_ZoneMixing (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


better `if (! ...) ... // without else`


What is the migration plan? Do we simply advance the ABI Version and drop the old `id` field? // cc @udalov


Closes #9


Thanks for this workaround, I just hit this problem today for the first time and your workaround helped.


It is good practice to place libraries before application specific files to ensure there are no `undefined` errors.


@tomsmeding 44dd81229584484698043cafe626fe7d81317351


Typo. Missing )


This does not recurse into lists or dicts (not sure whether that's needed).


Why do you need this? cd $PWD should get you to the same directory you were before.


But then I couldn't use the same comment for this line:
https://github.com/philipwalton/solved-by-flexbox/blob/master/assets/css/components/holy-grail.css#L22

I think it's fine as I'm using `flex: none` mainly because it's shorter than `flex-shrink: 0`, and the comment explains what I'm actually intending.

This is similar to using `background: red` instead of `background-color: red`. In the former case you're actually setting the background image property, background repeat property, etc, but that information isn't really relevant.


Space.


What is the purpose in adding this class? Is it for future testing purposes? Just wondering, since the commit doesn't give any reason for it...


> I'm thinking that there shouldn't be state in the cursors when the range is single-pass.

There's a trade-off here as well. There is better locality when the state is in the cursor, and it makes the job of the optimizer much easier to not have to worry about indirections. When benchmarking the comprehensions test, I found the result was very sensitive to the indirections and ended up moving state *into* the cursor for better perf.

Also, if mutable state is stored in the range instead of the cursor, then the range cannot have const begin/end members. This is a lesser issue.


Okay, good points, let's stick with the long ones. idrc.


Consider using built-in [el](http://backbonejs.org/#View-el) attribute.


Remove trailing space after '


@mpc-hc: I have updated LAV so now I think everything is ready.


Tested and worked for me.


Yeah this mess of a makefile passes the CFLAGS also to the final link call, so doesnt matter for me.
(Also I'm thinking of reworking it to be single-call compilation someday (so no time wasted multiple compiling all the files when i anyways have LTO), so... I mean its kinda tiny for the whole dependencies and all that stuff it uses now.)


Why have you left a name argument here? Also, why is it named 'erlang'? Why is it named "erlang-production-test"?


what is this? return a random falsey value?


it seems to me that "class" word must be used in this case instead of "clas"


I searched on the internet if it was best to use CDN or local for libraries like Bootstrap and jQuery. Many posts suggested CDN over local, but I wanted to use both because if the CDN failed to load the library, then the webapplication couldn't be used. I found a post on stackoverflow where a user suggested to use both but with a local fallback, based on a post of Scott Hanselman. http://stackoverflow.com/questions/26192897/should-i-use-bootstrap-from-cdn-or-make-a-copy-on-my-server
http://www.hanselman.com/blog/CDNsFailButYourScriptsDontHaveToFallbackFromCDNToLocalJQuery.aspx


@batt Facciamo un repo separato per la demoboard? In teoria le modifiche applicate a questo repo vengono propagate a tutti i progetti che utilizzano la develboard (adesso solo cdg)


> I'll go work this out for myself instead of making you painstakingly explain every detail of the design.

But don't assume that every aspect of the design has been thought about deeply. I'm sure there's lots of room for improvement.


I removed the .gitignore @zenorocha


ContainerUtil.newArrayList


I love this


the prototype object of EventEmitter is modified?
or it should be
new EventEmitter() ?


97902934_ZoneMixing (macumber) - 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)


`// Adjust system_clock::time_point for NetClock epoch`


Several fixes to get rid of the CHEF-3694 warnings, removed duplicate code and unnecessary user accounts.


Extra tabs entered here.


And here.


we don`t have to metion "px" or "%" for zero values


You don't need this field.


Just making this was supposed to be removed since it seems unrelated to the commit description.


LOL :tada:


Blank.


FacilityTabGridView (evanweaver) - 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)


Is the function header (like int getchar ( void );) a explicit label?


Thx, I'll resquash.


It might be easier that way, but it seems better to parse `raw` correctly without leading/trailing spaces the first time, instead of trimming it after the fact or hoping that other functions will trim it.


I am missing something .. I dont see any changes


Sure!


s/mid/messageID/


Yeah, as noted in the commit, let's try to remove this as part of the next PR.


Good idea - especially since datasets will also need to keep information about which subsets (i.e. continents) are being displayed with interior boundaries and which aren't.


Shouldn't this be writable, but with ReloadRequiredWriteAttributeHandler?


Uhm, virtus doesn't feel like a good fit here. It's a pretty large library. What was the motivation behind adding it as a dependency?


ContainerUtil.newHashMap()


This could be put on one line like this:

`attr_accessor :title, :priority, :completed`


we can avoid using this line of code, just use 'main.min.js' in previuos line


97902934_ZoneMixing (macumber) - 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)


That looks good to me.


<img src="//www.customink.com/temp/vendor/6036661/PO6036661-front.png">


If you feel that branch is ready. I'm ok with merging it.


It moved the creation of the vsync observer to BaseCreate, instead of it happening elsewhere as before.


I think we can drop `constructor` field and `Type.Constructor` message after next advance of the ABI Version.


Please, reorder methods of this class. It's way more convenient to see it's overrides and public methods first, and then all the utility methods. It's especially true for this particular class - it's main contribution is setUp and tearDown, which are buried under these utility assert*** methods.


html files must not be merged, they should be just moved to dist folder


Just curious. Why is this 5? If this is deterministic, can we document it as well? Makes it more maintainable if we modify acs handler.


This commit fixed #28 and not #27 as specified.


It really needs more testing and I'm leaving in like 5 minutes, not sure when I will be back.


Yeah, it's been a loooong time coming, but with everything out of the way (decent GBA support, re-integration of GB/GBC support, and a spiffy new GUI) it's time to tackle this thing. Once this gets finished, it'll be released as a 1.0 version.

Not much happening here (no significant code) but I'm just setting things up atm :P


Protected?


Eventually I will add more functionality to it (i.e., friends list permissions) but I'd like to get this working first. Let me know what needs to be done before it will work. I'll be happy to see it in OMT 2.2.0


copy & paste, I guess.


There are a couple of ways one can guarantee that such errors don't creep in.
1. Somebody can review the code. Search for all lines that contain a ` and see diff whether it is done correctly.
Also I use a linter while writing the code. It prevents me from doing a large class of errors. Any string that is not a vaild string is likely to be caught.
2. I can use a parser like Acorn to find and convert all expression that join a string and variable with template strings.   But this looks like a bit overkill to me. And I don't know how to use acorn in the first place. So, I can't do this right now.

I just want the codebase to be clean and more legible by using the nice syntatic sugar provided by the language. If you are not still convinced, I am happy to revert all such changes.


So I pulled out and I checked your code for a suspicion I had! It looks like we are using 4 spaces and you are using tabs for indenting. If you switch your editor to make tabs 4 spaces instead of the literal tab character I think this would make your issues go away! You would also keep consistency with us which would be great!


you should use the completed attribute that you defined above as apposed to using the @complete instance variable here.

Example: `self.completed = true`


I wasn't sure the file validation (csv vs txt) was working so I was trying test.csv to be sure. Once I find the error message I'll use this instead of test.csv and test.txt.


you should use the completed attribute that you defined above as apposed to using the @complete instance variable here.

Example: `self.completed`


Nightly build are about testing. Anyway merge it when you are ready.


FacilityTabGridView (evanweaver) - 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)


这里改为options.scope更符合命名习惯。


Should be fixed now.


By way of documentation:

How I generated the secure token...

- Generated PAT here https://github.com/settings/tokens
- Created a Digital Ocean Droplet with the Ruby on Rails image. ssh'ed into machine

``` bash
gem install travis
sudo apt-get install git
git clone https://github.com/tjmahr/rprime
cd rprime
travis encrypt GH_TOKEN=token_i_made_earlier
```
- Pasted yaml output into the .travis.yml file
- Destroyed the droplet


But for that we should new fields add to `Type` instead of to `Type.Constructor`


please use 'dist' name for folder instead of 'dest' (the same for all other cases)


engine is downloaded into the download dir! imo you can't block this.


Add config file hook-in to determine presence of Towny.


Ocn does not spawn a horse, and cardinal should not either, it just edits the players' data to have a Riding nbt tag, with an armor stand as the entity the player is riding


unwanted e


LGTM


File permissions on the deploy script were wrecking the build. Hadn't figured out yet that I was using the wrong token.


Yes, this sounds more reasonable // cc @MichaelNedzelsky


No, still wrong - for example, Thames Wharf machine works completely insane.


Does it take rotation into account?


ты же в этом месте знаешь точно сколько поинтов ты передал, проверь что число правильное


Make this assertion before you do anything else. I'd even move it to constructor in order to forbid making instances of this test case for unsuitable OSes.


Extract a constant for sdk path.


Clear code is of great value. Also messages might change and it should do
something sensible even in case of surprising message.

Use regexp to remove 'Warning:[ \n]*' and it will be okay.


Does this help on battery? Have you guys discovered what's causing battery drain or how to fix? Thanks


Hm. Then I will remove this at once!


Alternatively you could get rid of the completed attribute and just use the @complete instance variable but you'd want to set that in the initialize method instead of the completed attribute.


initialize is spelled wrong


FacilityTabGridView (evanweaver) - 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)


The code was copied, a refactor is beyond the scope


qweqwsadsdas
dasd
as
das
das


I'm not sure what I am supposed to be doing right now.


Unnecessary braces.

Also, I've already asked you whether you need commands, and you responded negatively, if I remember well. Why haven't you replaced WriteCommandAction calls with ApplicationManager.getInstance().runWriteAction then?


I recognized I need the `attribute` helper to define DSL-specific nodes.

```ruby
class ExcludeKeys < AbstractMapper::Node
  attribute :keys, default: []

  def transproc
    Functions[:exclude_keys, keys]
  end
end
```

I have two possibilities: either to define this feature in `abstract_mapper` (but that is definitely not the responsibility of `abstract_mapper`, but more general), or use the external library. I just haven't find any one with better API, than `virtus`.

I also tell to myself, that `virtus` is too heavy. But it will work only at the time of DSL definition. Later it can also slow down the AST creation and optimization. But IMO it doesn't matter, because the time-critical stage is mapping, where no node is used at all. Only AST-provided transprocs.


isn't this still needed?
m_DownloadDirectoryTextCtrl->SetValue(TowxString(SlPaths::GetDownloadDir()));

(+thanks)


`index` is masking `Bot::index` if I'm not mistaken. Not that much of a problem, scoping works in C++, but not so neat.


now you should use sources only from dist folder, not from bower_components or any other folders


+++ Ophir Lifshitz [Jul 23 15 09:43 ]:
>   It might be easier that way, but it seems better to parse raw correctly
>   without leading/trailing spaces the first time, instead of trimming it
>   after the fact or hoping that other functions will trim it.

If we trim in `toKey`, we only have to do it once, and we're
covered.

Note also that to avoid parsing the spaces before a closing
`##` sequence in an ATX header, we'd need to constantly
lookahead over spaces to see if they're followed by that
closing sequence.  This is inefficient parsing.  Better, I
think, just to include the spaces in what we parse, and
strip them off later.


Too much `auto`. How about `bool`?


Yes, removed too much))


Please remove this.


I think this might be a too long line?


This one is unnecessary


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.409](http://teamcity.monogame.net/viewLog.html?buildId=9101&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 522, ignored: 6 Build time: 00:27:15


I've been informed that I typo'd this commit message.

![](http://k46.kn3.net/taringa/2/D/6/A/4/C/vagonettas/C72.gif)


This doesn't get the history that I wanted.


Blanks.


final


@romartin I believe this change is safe, but could you please double check?


As the rotation transform should be in `getBasis()`, it should take rotations into account. Yet I think there could be missing a conversion of TR coordinates to internal ones, I'm not sure which one `m_transform` uses.


And how about just saying `if (index < 0 || index >= (int)arrVar.arrVal.size())` below? (Mind the addition of `index<0`)


Sure, I think I had my remote pointed at the wrong repo. Sorry for the inconvenience!


Should be more like: "A string which is contained in labels if ToDo items to find."


oh, I see.. you made it fail for the generic case.

Then, you can just do `return context.defaultRewrite(node)`


you need to have git ver 2.4+ for followTags to work you mainline mofo, miss you


I think you should take advantage of this change to use `mysql/query` instead of a string. You will need to implement something for the "ON DUPLICATE ...", but otherwise it's pretty straightforward.

That, of course, if you see value on continuing using it. Otherwise, that is fine.


capitalize SELECT


That's a good idea.  I'll try returning `Try[InputStream]`.


The transformation matrix and the vector should be switched, I think: ent->m_transform.getBasis() * btVector3(dx, dy, dz)


Add a comment to the commit explaining what's "fixed"


@vsavkin ok, I figured that `@InjectPromise` won't be back, but I thought there will be some kind of mechanism to load dependencies async.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.410](http://teamcity.monogame.net/viewLog.html?buildId=9105&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


What happened to createInstance(MetadataSource metadataSource)? I think you misunderstood my comment?


getInstance should use the new createInstance.


Yeah, it’s definitely better than writing everything out manually. I’m a bit confused at to what it actually generates, though – does it do both the proper sum type (`object $includeF`) and the fixpoint wrapper? If so, we probably need a way to distinguish between fixpoint and non-fixpoint functors (EG, 2- vs 3-parameter versions of `Instance`). Both the actual case class and the sum object should probably have the same name (no conflict, since one is in the nested scope), so we can generate with `Instance("$includeF", "$instance", ListMap())`, using the first parameter for both of the non-fix names, and the second for the fix. And GroupOps with `Instance("$push", ListMap("value" -> ParamType.Rec))`, since they’re just a functor, not expected to be fixpoint.

I really think this can be done with almost no boilerplate if anyone ever figures out macros. Not that this syntax works, but something like `fixpoint[ExprOp](removeF(_))`, where `removeF` is some function that translates the functor name to the fixpoint wrapper name.


If you can currently play around with the code, do it! I'm probably doing nothing for today.


give some comments


What if the services don't have `release` command?


Thanks for the bug fix.

Howerver, the three branches are actually important for performance. I only added them after fairly rigorous testing. They seem to trigger varying levels of Java's array bounds check elimination logic. I can concoct the benchmark code (or you can run it with `sbt "project benchmark" "run-main breeze.linalg.MappingUFuncBenchmark`). But it seemed like having the three-way branch was the best way to optimize all common cases to their greatest extent.


please add .idea, bower_components, node_modules to .gitignore file


(The common cases being: DV with stride 1 and offset 0 and DV with stride 1 and some other offset)


Why i need `attributes`? Because it simplifies nodes definition (and implicitly rules definition too).

```ruby
module Faceter
  class MergeExcludes < AbstractMapper::PairRule
    def optimize? # checks if optimization needed
       nodes.map { |node| node.is_a? ExcludeKeys }.reduce(:&)
    end

    def optimize # merges two consecutive nodes
      ExcludeKeys.new nodes.map(&:keys).reduce(:+)
    end
  end
end
```


Here's a solution which adds the day of the week and doesn't involve inventing yet another date format to localize. Unfortunately it is not exactly frugal when it comes to screen space...
![screenshot007](https://cloud.githubusercontent.com/assets/12870817/8856682/32690290-3164-11e5-9d4d-c3cc16802f57.png)

I don't mind it as a solution and there is certainly screen space to spare, however if you're still not keen I'll simply drop this commit until after the rest is merged. It was probably making the PR a bit 'big' anyhow.


@tomsmeding e3f96a733f8aa704678e94c68f4d5c1bb06df9c0


here you can make this simpler:

node_output, cand_output = pair[:2]


So now the models estimated can actually be deployed to make new predictions beyond the predictions made by providing a data.frame in the “test_data” argument of evolve_model(). This predict method is super fast because it pulls out the estimated state matrix and action vector (from the S4 “ga_fsm” object that a user gets from evolve_model()) and passes them and the new data to predict to C++ code to make the actual predictions.


The `configure.vars` needs to be hand configured (from `configure.vars.noconf`) for the paths of all of the external dependencies in your system.
These properties files are configured for your system from `configure` and `replace`.


Thanks. I hope to implement some other changes next week which will make it even faster.


My thought was: Right now we only eventually use `raw` for the `toKey` function, but what if later we send it to more functions? Wouldn't it be inefficient to trim it separately inside each of those functions? Anyway I trust you to make the right change.


simpler:
for ...
pairs.append(pair[:2])


this can (and should) be done entirely inside of `render`. No state or event listeners are required.

```
var labelClass = this.props.value ? 'has-content' : '';
```


this would also cover the case of when all the content is removed and the label fades back out.


I refactored this later, and the pattern match is simpler.

Style is ignored except when rendering PNGs (though allowed).


This file was really just renamed to `UnanimusProject/module_unanimusproject.xml` and drastically changed to compile our program.


Agreed. I'll move new fields to `Type`.


TODO: need to extend the support to Cholesky decomposition


Honestly, this was a change from a long time ago on my rig computer that I had to commit to sync the code with recent developments.

Trying to rehash my thinking, though maybe this wasn't it...
matlab.land.makeValidName may not exist in all matlab versions, or may behave differently in different versions. Since it's not a 'standard' function I'm also doubtful that most users would understand how this function works.

I don't know when something would go wrong with makeValidName, which is why this change is structured as it is:
(a) If nothing ever goes wrong with it, this modification has no effect.
(b)If something does go wrong with it, I'm not sure that I want to throw an error, because I don't know if this is preventing behavior I do want or not. This modification alerts me to the fact that something's up, and points the user to the relevant code, but it also allows the operation to go through in case this is something with makeValidName not working as expected.

I think this may have actually been the case on my rig computer a few months ago, since I was using matlab 2013a at the time, and makeValidName might not have even existed?


gfhfghfg


I'm going to checkout `${sdk.dir}/tools/ant/build.xml`, maybe it will yield some insight into packaging our APK.


gold


I don't see anywhere where you create pairs of lenght 3 as discussed. Where is this?


Great to see this merged into develop!


also, these should stay as `onChange` to have the same interface as an input.


This file can also be configured by hand, if you know what you're doing.


sorry, i couldn't resist: https://github.com/springlobby/springlobby/commit/eedf78aba427687e8fc95c36a0cfb7f8ed0d9c24


This is the build file for our project.  It does most of the work.


Yes. I had noticed it and submitted the second commit to correct it.


Here it reads the corresponding properties file configured with `./configure` from `UnanimusProject/module_unanimusproject.properties.noconf`


That's better))


To the best of my knowledge they are unified.  I'll look into that, though.


Here the configure script reads the configure variables from `configure.vars`.  This file needs to be edited from a copy of `configure.vars.noconf`.


It we add another module, we just need to include it in this array here.


Routing loop detection?! How neat is that!

How do you end up in a scenario where multiple nodes create a loop automatically?


Correct, I've removed it in a newer commit.


Replace does most of the work, formatting the lines in all `*.properties.noconf` files  using bash subshells and variables.


Daylighting_Speed (DeadParrot) - 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-86.99%25-red.svg)


You can exclude this line, as `HOME` is an environment variable.


The tilde is an implication for your home directory, it should actually read `${HOME}/.m2` after configuration, or wherever your maven cache is.


References issue #149

Still to do on this issue - add a config flag to specifically enable this logging, as it does introduce both security and performance concerns.


Yeah, looking over this more, I think doing this should work fine.


You probably have `java-7-openjdk` as I do, however it's just important that it is a JDK 6 or (preferably) 7 version.  Android does not currently support JDK 8.


You can also point these to `UnanimusProject/libs`


I don't understand that part, so it's hardly possible I'll be able to fix this! :smile:


This is the real configure script.


I took care of it, now it should behave fine.


fixed


test


Wow. That's a lot of changes. Looks good for the most part.

I'm going to merge this... really hope it doesn't break anything too badly. =)


renamed


Ok.


HomePath corresponde al directorio del RStudio Project en local


These files are actually executable with `replace`, but they require the environment variables from `configure.vars`, best to stick to `configure`.


It may be annotated as `[deprecated=true]`


What a nice fix


See comments from `UnanimusProject/module_unanimusproject.properties.noconf`.


Insert a blank line before line 357.


This manages most of the system dependant properties.


This pretty much is just a lower-level interface for `UnanimusProject/module_unanimusproject.xml`.


"bin15" ?


It will if you make this return the interface per my other comment :)


LGTM


Nitpick: can you remove the newlines above and below expectedDate?


I was able to verify the issue and the fix, thanks. If you address the small nit, I'll merge it.


что за ТЕСТ?


OK, I can live with two PRs :) Easier to review.


s/GM/GA/ here and in the commit description


yea, I figure if there is no string we should just return, but I guess it would be better to just return '' or null or false not whatever we feel like. I will make the change.


@tomsmeding 7a0dbe6ccee8e163334c72ff54595598d9cc385e


I'm fine with a longer name. But keep in mind that the package is part of the name. You could even call it "Interface" if you have it in a "storage" package.

But the name for this interface definitely can't have "etcd" anywhere in it. Because this interface is (should be) storage layer agnostic.


Oh please add a comment here why this is necessary.


I'll do it.


Visto che male non fa, onde evitare casini, io lo lascerei ovunque. Così abbiamo un kernel che va su entrambe.


lol @tomsmeding that bool was because I was originally thinking of doing iterator stuff. After that I realised a boolean would be much more logical. After that I just forgot to remove the `auto`


You can remove this comment :) I put it here as a note for darrin at one point and somehow we checked it in.


I think "you" "accidentally" misspelled canola


The number is not deterministic, it's just trying to express that it's okay if there's minor churn in number of goroutines due to other async processes (e.g. seelog async logger goroutines). The leak was on the order of 100 and this is just meant to be a number well under that.


`g_autoptr(GList)` here.


Typo: Sllow [sic] instead of Allow.


"mapped onto" sounds awkward. I've never heard anyone use that phrasing. [GoogleBattle](http://googlebattle.com/?domain=%22mapped+to%22&domain2=%22mapped+onto%22&submit=Go%21) agrees; 11.8M results vs 827k results. It may be proper (not sure) but I'd rather read it how it was phrased before. The other changes to this file make sense.


The argument "relay_state" and "mtype" is never used. Can they be implemented or removed?
https://github.com/rohe/pysaml2/commit/9b8ef8ac38d7f6b00da2d7ccbb55f0160bda1b5b


I agree we should do decision carefully, it doesn't hurry.
I open this question now to get some OW rule for ARM architecture, because bld/plusplus/h/pcheader.h
is first candidate for it.


Pet peeve of mine here, but if we have a function to free an entry, I prefer to always use that function.


same for the other do() method below.


I'm not sure about that, it looked to me like the two calls (getrsakey and then dologin) were what was causing the duplicates.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.410](http://teamcity.monogame.net/viewLog.html?buildId=9105&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 522, ignored: 6 Build time: 00:27:24


Heh. I don't think this is actually used anymore.


Загрузка обстановки реализована.


My subconscious made me add IDs to headers in this commit so I could b7cd33b


s/to run the pod created by this build./to run the build pod./


Not as far as I know


Do we have a way to check that all sub-directories of `./data/` are writeable? In particular in this case, the new `./data/PubSubHubbub/` directory and its descendants.


In a newer commit I already switched to doing a flatmap.


Daylighting_Speed (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1259 of 1260 tests passed)

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


Looks good to me


97902934_ZoneMixing (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


@bleskes I put a new issue: https://github.com/elastic/elasticsearch/issues/12428 and i'll let you guys decide :)


but in our case, an `account.profile` always exists though, so I read `account.profile.update` as update the profile with these property changes.

We used to have `.save` in `hoodie.store`, too, but it was rather dangerous, because it always replaced the entire document, similar to what CouchDB's `PUT` does. In Backbone land, `.save` makes sense, you can call it without any options, it will then persist all the local changes you did before using `.set`. But as we don't have that, I think `.save` would rather confuse, at least more than `.upadte` in my opinion


I added (privately, this isn't pushed yet) an inferred_reserved thing that was basically analogous to inferred_playables, but it wasn't working out. Too easy to either (a) fail to infer the intention of the clue, or (b) misconstrue it as implying the card is playable.

So I'm thinking of scrapping that (for now) and just relying on the "players try not to discard stuff they have info about" behavior. So at the point where we decide to give a "reserve this card" clue, all we have to do is make sure the clue can't be misconstrued as "play this card". So the clue would have to meet one of two conditions: (a) the clue is about more than one card, meaning the receiving player won't try to infer anything about it, or (b) if it is just one card, the receiving player will try to infer that it's playable but there won't be any playable cards that match it. In that case, he won't infer anything about it, but he'll be less likely to discard it since he has information about it.

The other part of this is knowing how much to prioritize "reserve card" clues. I think it's generally better to tell a player "play this" as opposed to "reserve that", but we may need to experiment to find the sweet spot. And in general, the decision making process that goes on during take_turn() probably needs to get smarter.

I was planning on doing this work tonight, but it sounds like you might be toying with some similar ideas, so if you'd like to take a crack at it instead just say the word.


s/Strategy/build/


Perhaps a dumb question, but what are characterization specs?


testing readme


Thank you!


Yes, this is good!

I use VM for multiple projects, and it was annoying to use serve command.


In a newer version `cause` is no longer `Option[Throwable]` but just `Throwable`, so the logging is unconditional.


Nope, works only on Grabbers, Me and Tech


> somewhen after the release

So is only `Unique` going out with 2.068, and this with the next release?


The (optional) revision information from the source repository used in the build as a SourceRevision.


Ya, I changed that right after checking in. :)


Wait do not change it, I am working on validation


happily noted, thanks :)


Git is good, we love git.


Human, не humen.


Also, EPEL should be used very carefully! The EPEL repository should not be
left attached by default as it can break updates. That can make it a bit
problematic. Also, installation of packages from EPEL can cause other
updates to fail even if the EPEL repository isn't attached.

On Wed, Jul 22, 2015 at 6:00 PM, Tim Daley <tim.daley@cru.org> wrote:

> I also have james looking at a centos image that we can keep up to date
> for you.
>
> Sent from my iPhone
>
> On Jul 22, 2015, at 3:50 PM, Matt Drees <notifications@github.com> wrote:
>
> Side note: I didn't realize you could put a comment here. In a bash
> script, you can't, because it breaks the above and below into two separate
> statements. Docker must strip out this line entirely, not just everything
> following the #.
>
> Very helpful to learn.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CruGlobal/syslog-splunk/commit/afcc449718b06323025bd9509c68132b37160bb5#commitcomment-12308642>
> .
>
>

--
*Tim Daley*
IT Specialist-Operating Systems
cru | Technology | Platform Team
o: 407-826-2911 | m: 407-716-0284
tim.daley@cru.org


I'd add canbuild() and buildingenabled() here as related functions


Заменить на NULL


Isn't that redundant?


Its confusing to me that you set this transform to identity and then change it right after. Is that right? Do these multiply somehow?


Зачем здесь приведение типов? Если одно наследует другое, то оно автоматически приведется к требуемому типу.


I currently don't have enough time to spend on this, I have things with higher priority that I cannot simply neglect. I'll try to start working on it again in a few weeks time.


That works for me. :+1:


@csrwng Made changes


Sorry for the slow answer... I am not currently working on uPD (though I might get back to it eventually). I invite you to go through the example scenes and reading through the available documentation in the PureData.cs script. Otherwise, I've given a bit more information on this forum: http://forum.pdpatchrepo.info/topic/9056/replacing-unity-s-audio-engine-with-pure-data-and-libpd/2


Все nullptr заменить на NULL.


yeah, seems to be a github bug... as you can see, the file itself is connect: https://github.com/Cranke/Magento-PageCache-powered-by-Varnish/blob/master/app/code/community/Phoenix/VarnishCache/Helper/Esi.php


Player.


Git is great.


И \return нужен тоже.


Nice, very nice!


I realize I'm splitting hairs, but I suggest rewording this as
```
Likewise if no detached metadata exists for the commit, ...
```
since it may someday hold more than just GPG signatures.


As a side note: There's a nasty nashorn bug when sharing ScriptEngines upon different threads even if the evaluated scripts are thread safe:

https://bugs.openjdk.java.net/browse/JDK-8075006

This bug can easily destroy your servers thread pool as I had to experience lately. ;)

It's fixed for 8u60. I wouldn't recommend to use a shared ScriptEngine prior to this fix.

I fixed this issue by using a pool of engines (instead of thread-locals).


Clever chap. :-) Much better than my idea with a string resource. Like it.


I am not sure if I should move the data logging action into the MBean itself or leave it here.


Los cambios hechos a la clase * se aplican a todas las clases del documento css.


Are we ok with instantiating the mbean tracker here?


src/script/cbottoken.cpp in main repo should be updated with new path to documentation of these functions


sorry wasn't clear... wanted to remove 'created by this build'. (Implies the pod is somehow the output of the build, but it's actually what runs the build).


Is this the correct domain to register this MBean?  ("com.rackspacecloud.blueflood.io"), or should it be "com.rackspacecloud.blueflood.service"?


Should these copyright header comment be 2013?  or 2015?


Thanks! If you are interested, you can directly send a pull request to fix it:)


Ok, there is some standard language arround "here's how you set up your gulp/npm proj." so i'll ⌘C ⌘V


What does it return for invalid input? nan?


This is necessary for "localhost" server to have secret keys, the better fix would be to add them in hostconfig with config generation.


Подсчет товаров при повторном добавлении товара


@csrwng Made that change, sorry!


En exploradores antiguos que no son chrome puede no aparecer la propiedad "box-sizing: border-box", así que para asegurarnos de que se aplica usamos los prefijos -webkit (Safari), -moz (Mozilla) y -ms (Microsoft).