See my comment here: https://github.com/graphql-python/graphql-core/issues/18#issuecomment-146878614

This isn't working by design.


@DHrpcs3 should probably be added to rpcs3 org if you haven't already.


You send 2 parameters but the minimum function just receive 1


We are, but 2.5 is relatively minor - just API enhancements. This change won't break your implementation.

*Before*

Hash = 'wc_variable_prices' + product_id + hash version + display flag (etc)
One product = shit loads of transients

*After*

Hash = 'wc_variable_prices' + product_id
One product = one transient with multiple hashes inside

Does that make this change clearer?


@dschafer, this is the test that I think might be in violation of the spec. Can you confirm/refute my reading?


Excellent!


I only see:

Error details: Build uploaded has been partially compiled. We recommend at least 85% units should be compiled and ready for analysis to avoid false-positive. As per last few lines of cov-int/build-log.txt, compilation unit ready for analysis is 1%, which is less than the expected 85%...


If you think it makes sense, go ahead.


Default <title> tags from all pages now states: "Contact | " When page name/title is Contact.


:-) lazy sounds familiar to me too... hi bro!

What about Airtable, seems enough, so you'll spare the time to create the app, these days are so sunny :-/


It is already included by python-module.h. Also if you consider my extra-minimalistic alternative PR, I have introduced a kludge into that header. (Please check other instances of this include as well, I won't duplicate this comment)


Is this correct? This break is not inside an statement block, so start += docCount will never be executed.


No copyrighted files in the repo!!!


Alignment


Ah yeah I was going to split it out and change my mind. You just run test. I'll clean that up.


No {} on single-line if blocks please, see CODING_STYLE.


... How is this going to change anything? Just delete the method!

I made it so that `@ExposeToLua` ignores Java access modifiers


Please update the file header to show:

````
(C) 2002-2015
````


:) I don't have a number. But it could be a problem if you have lots of them emitting very frequently (corner case).

But the main (and concrete) issue with USB/FTDI is the latency issue. As I said before, if we want to optimize downlink programming, we need to have a low latency (order of 10ms) between host and sx1301. Which is not verified at all when using FTDI.


That looks worse now (in github view).
Indentation should be spaces rather than tabs (no guarantees that tab settings are the same on different systems).


It's better to use self.question_percent_id


thanks for the comment, I would not of figured out the point of doing this property otherwise


It's better to use self.label_men_id because you did a create


lol, I live in England, its not sunny :P

I guess I have specific ideas about how I want to receive contributions - via github by editing a simple file like now.  But this same file be used for the DB that runs the web via some means.

I don't want accepting contributions to be outside of github or harder than editing a obvious file, I believe that'll discourage contributions.

I know how to achieve this goal, but have to write something


It's better to use self.label_women_id because you did a create


must have rake


I don't understand why these extra lines are needed or why you need to do the init twice. Neither do I know what you refer to in the commit message. I have not experienced this sleeping issue myself.


It's better to use self.input_0_men_id because you did a create


It's better to use self.input_0_women_id because you did a create


Should this java 6 unwrapping be inside your while loop?


It's better to use self.input_1_men_id because you did a create


I know this is definitely a work in progress :smile: but I've just found that receiveMessage is undefined...


Thanks. Will look it up

Sent from Yahoo Mail on Android

From:"Anthony Nandaa" <notifications@github.com>
Date:Fri, Oct 9, 2015 at 16:51
Subject:Re: [factorial] Create factorial.js (ffb16cc)

Read up about styling guides, it's advisable to leave space after the closing ) and before opening ( i.e. for (var c = 1; c <= n; c++) {. Check out for complete list here - idiomaticjs

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


dammit @RabidOtter62845
you messed up the link its broken you pleb
i shall fix


Dit kun je beter met kleine letter schrijven: 'aaron', dan wordt het herkend als object.


This can just be `File.read(private_key, 'rb')` I believe, otherwise you leave a file open until GC runs


Thanks for the fix, had one actual question about java 6 Wrapper interface - please let me know what you think.


Same remarque


Hier straks toevoegen addComponent(aaron)


Dan is dit niet meer nodig


This resolves #6.


well, the signedness matters a lot. if the types are uint32_t here, und you compare 4 and 5 and determine the difference, then 4 - 5 is (uint32_t) -1, which is 65535, which is > 0, hence 4 is larger than five with this function!

This only works correctly for signed types, such as int32_t, where 4 - 5 is -1, and -1 is < 0...

Actually: the line above is even completely broken as it compares a1 with a1, I figure you actually mean a1 and a2 here...


Is this intended to be different from the "H12" that is used further down in the test?


Can you stick up a quick list of current features we can expect in the alpha?


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


`$operatore()` fallisce. (Chiama `__invoke` su Utente?).


congraduate


Is not a init daemon


Hierin heb je alle veranderingen van Aaron teruggedraaid :(


is not a login pty


Yeah but if it's an opt-in feature in the config most people won't even notice, and if you really would like to have it on your server you can still use it. Like seeing low level quests, not blizzlike, but a nice-to-have feature.


Wenn mehrere Professoren gehackt werden sollen, könnten die Aufgaben dann in Abhängigkeit von deren Lehrfächern stehen, bei Mathe-Prof Mathe-Aufgaben usw. (oder vielleicht stell ich mir das leicht anders vor :))
Auf jeden Fall eine interessante Idee!


Did you do this to ensure the cpp-build refers to the same Phonemetadata.java as the other (ultimately, the generated nano one)? We had a discussion about this (you were debugging), just want to make sure I understand it right.

Also, can you make the <source> only for Phonemetadata.java rather than the directory?


No you didn't actually see me push to the master directly, shoo! :D


But OK, if the this call is never used for sorting or prioq stuff or so, then I guess it really doesn't matter if the stuff orders sanely, or just compares... But the a1/a1 issue is definitely something to fix...


@lgray
This is in principle a *geometry modifier*, and this is the reason why we need "unconst" stuff to handle them.

The main "const" problem is that the map in the geometry have "const" module and "const" triggerCells,
[here](https://github.com/amarini/cmssw/blob/topic_neighbors/L1Trigger/L1THGCal/interface/HGCalTriggerGeometryBase.h#L140)
in the idea that they are "frozen" after being "constructed",
that we may want it to be the case, when people will call it later in the code; this is why I didn't modify that part.

 It is unclear if we want other plugins to be able to modify this const objects (non explicitely).


Hey @tremainebuchanan.... i see a LOT of work went into this.  Apologies ahead of time for the news I am about to break to you :(

The angular app should do the following:
* capture demand and commodity allowing the user to enter arbitrary weight units
* display demand and commodities rendering the user's previously entered weight units

So from the front-end perspective absolutely nothing would change.  During the matching at the API level we would first query based on the dates and the list that's returned would get converted to the base units, then sorted.  The conversion to the base unit only facilitates the sorting of the matching demands/commodities.  Also the conversion doesn't modifying the units presented to the user at the application layer.

Let me know if I am missing anything.


Overlooking a PEP 8 rule is a slippery slope ... can this not be express like this:
<pre>
if not (remote_url
    and request.POST.get('_content_type') == 'application/json'):
</pre>
... in which case the actual parenthetical statement of `and` is clarified, the is `\` not needed, and everything is pep compliant.


I think that reusing the code this way adds one more function call and decreases the code readability


Is this really needed? The docs say that you must not call any C API before init: https://docs.python.org/dev/c-api/init.html#c.Py_Initialize


Is this real life?


Very nice


Hi @MisaelSalcido, I think that at line 444, the description should be something like: "...that an operating system is not supported" instead of "is not to be".


And that's what I did. If the ship doesn't have enough FTL energy, it can't use hyperdrives/slipstream - but it still needs to take FTL crystals from the empire in order to generate that energy.


Your changes are functionally correct as you mention. However, all of the fields are listed for didactic purposes and I think it's not a bad idea to preserve that.


I just hit this roadblock.


is not a pid file


@tofumatt  That's removed in master now. Thanks for catching it!


Whoops, was copying from the above line


@amarini
Hmm, then an appropriate solution would be to pass a list of geometry modifiers to the initialize function of the geometry (where you can then pass non-const), or have the geometry object keep a list of geometry modifier functions that are applied when init is called (like this option better).

Would propose you make a system of plugins that we can configure from python like the geometry itself.


create_socket_perms should be enough


Et non, pas besoin, c'est ça la magie systemd!


It goes without saying that the comment should reflect what the method is doing. Currently, both - `block_for_signal()` and `wait_for_signal` have exactly the same description: *Block until a signal is received*


@romani

What about this example:

````java
case 1:
  // fall though -  violation (should have the  same intend level as previous line)
case 2:
    int a = 5;
    // fall through
case 3:
    int b = 10;
 //  fall though -  violation (should have the  same intend level as previous line)
default:
               // no violation ? What part of code does this comment describe ?
````


Thank you for your contribute.

I will be honest and this project is far away to be finished. It was only an experiment.


Thanks! Now I checked tests are working.
But not the command
python -m unittest test_biro_v4.TestBiroV4.test_sum_coherencies_double
as I get
  File "/usr/local/Cluster-Apps/python/2.7.5/lib64/python2.7/unittest/loader.py", line 91, in loadTestsFromName
    module = __import__('.'.join(parts_copy))
ImportError: No module named test_biro_v4


The first rule of PEP8 is don't talk about PEP8.

The second rule is "[a foolish consistency is the hobgoblin of little minds]"(https://www.python.org/dev/peps/pep-0008/#id10). In this case, since it's likely just a few characters over (and not, say, double the length) it's OK to stretch it. The rules on [line length](https://www.python.org/dev/peps/pep-0008/#maximum-line-length) make the concession that it's OK to stretch it to 100 characters.

However, you are correct. The preferred way of splitting up long statements is through the implied line continuation inside parentheses.

We're also painting the [bike shed](https://en.wikipedia.org/wiki/Parkinson%27s_law_of_triviality) red.


Looking forward for this part of the API :)


This removes, for instance,  the ability for http code 302 to be properly handled in the future, where the user agent should transparently use the Location URI in place of the original for subsequent requests (not continue to hit the URI that forwards you)


Why do you accept count == 2. Is that a fillable polygon?


`.then(() => nativeMethods.formSubmit.apply(form, arguments));` ??


@marziarivi You probably need to run that in the tests directory, which has the test_biro_v4.py file in it.


.then(filesInfo =>  UploadInfoManager.prepareFileListWrapper(filesInfo))


semantics of "similar to" is not what is meant by "no variation"


why do you accept count == 2?


leave this code in the script. was removed because it needed to run multiple times.


We can tweak this as necessary over time.  It's only necessary at this point so that we can catch tests that reach out over the internet and mark them accordingly.  It actually doesn't have to be watertight or perfect.  It's stolen shamelessly from astropy.


Yes I did.


no comments in commit


@amarini The geometry should be constructed and then remain static until it needs to be rebuilt from (probably) scratch due to conditions change. So really it's fairly justified to stick any modification into the initialize function. It's just a matter of how we want to factorize and implement what we're doing.


You should pick this too:
https://github.com/CandyDevices/device_htc_himawl/commit/2e8913a2ca00dcb237d1bf72bc1e05546f508476


?? je comprend pas tout à systemd.


Things I'm working on for the next commit:
- Finish form validation (MLH code of conduct checkbox)
- Inserting the rest of the acquired data into the collection (maybe not race and sex since those are supposed to be in another collection...? A little confused on the specifics of that).
- Some small but important visual things - error box shakes too much, delay between first transition is too long, spacing of checkboxes / radio buttons, next / back buttons are reversed

Figuring out how to do document uploading with GridFS, but that's likely not going to be next commit.


Ok! There was a mistake in my path... Sorry.
Now it works.


... then they are guidelines rather then rules. Super red! :-P


I never used it on spartan6 but it seems to be similar:
http://www.xilinx.com/support/documentation/user_guides/ug380.pdf / P109.
So it can probably be reused with no or little modifications.


The problem is that Racket's numeric tower isn't generic enough to accommodate the SICM code as written. Do you really want to do

```racket
(define (quaternion-+ x y . ys) ...)
(define (up-+ x y . ys) ...)
(define (down-+ x y . ys) ...)
(define (literal-number-+ x y . ys) ...)
...
```

And do the same thing for ALL numeric types that they define? The amount of boilerplate involved in that is immense. We also would need to errata their book and replace every instance where they use these basic functions on different types. I foresee this being a considerable amount of effort.

Why not consider an alternative like
```racket
(define-syntax-rule (generic-math)
  (require
    (prefix-in racket/base: (only-in racket/base <numeric-tower>)))
```
Then people could just incorporate our functions in with\
```racket
(generic-math)
(require mechanics/<some-library>)
```
This way the SICM examples work as written for people following from the book, and people who may want to use some of the libraries in other projects can choose to do so with 2 lines of code instead of one. I think thats a pretty good tradeoff. Besides all of the `number` math operations dispatch to the usual behavior anyway so nothing will conflict.


I should probably have used it.


Should probably extract this into `OAuthApplicationPresenter` and use that here and in the controller above.


Should be extracted into a view helper.


I actually found a couple of bug in the code thanks to the stricter types


Should use translation interpolation here, `%application has access to:`


@matjames007 thanks for the comments. Just to be on the same page with you.
1. When the user enters the demand/commodity, should the amounts be converted to the base unit (where applicable) at the api level and stored in the db? This was my understanding from the issue created around the accommodation of the multiple units by the application. See ref: https://waffle.io/slashroots/jas-mis-app/cards/55bbbdb3c42bc51900a7b6be


This commit didn't change the schema.


This script takes too long to execute. Needs fixing for it to work properly.


Yep, that's right.


what is this ?


"".equals(patch) = > patch.isEmpty()


LOLOLOLOLOL


blank node is not used


I don't see how this tests the view. Also should have one for the authorized applications list.


no commented code


Ok Jon, I know you're still cleaning things up, just some comments to the current state: one compiker file contains paths that are obly valid for you, referencing paths out of the scope of the repo (/Xipho's or so). And I do not like that object ids are generated randomly. That was the case for characters initially and I threw it out to make multiplayer work. We need another way.


@selectusamicah Hey


we will miss you


ddd


I'm perfectly comfortable with 4 > 5, as long as it is consistently so ;-) All this gives is "some order", not necessarily an intuitive one (sorting would still be fine, though it is not used). a1 v. a2 is indeed broken. Will fix.


Good catch. Should be fixed in [d1d2618](https://github.com/h5p/h5p-drag-n-bar/commit/d1d2618ab570c241420b4362d8a95347420faf3f#diff-1).


Klingt auf jeden Fall nach einer schönen Spielidee. Kennst du Escape Goat? Kannst du sonst ja mal als mögliche Inspiration anschauen, da es dort auch darum geht den Ausgang aus einem labyrinthartigen Ort zu finden/öffnen, es kombiniert dabei Geschicklichkeit mit Logik.


Great! Congrats to all devs for some awesome work. v0.4 has really introduced some great feature that I use all the time.


@phanimahesh, I am not sure. The emqttd_pubsub process's work is to write/read mnesia database which should be scheduled first if the VM is busy.


Ta soeur !


Nice, thanks! I like the first text more for Intro, but chunks of the second will be good for the abstract too. Committed.


I wish I could get intelij to draw 2-3 rulers for me - I want a 72 col ruler and an 80 col ruler and a 100 col ruler. Then I could get a very visual notion of how max-dangerous I was being.


we probably don't need the delay on startup anymore as the triplestore is managed externally


@vtjnash Before:
```julia
julia> @time for i = 1:10000
       Y = randn(100,10);
       cov(Y)
       end
  0.156312 seconds (200.00 k allocations: 171.661 MB, 13.23% gc time)
```
after
```julia
julia> @time for i = 1:10000
       Y = randn(100,10);
       cov(Y)
       end
  0.529782 seconds (10.71 M allocations: 334.778 MB, 8.80% gc time)
```
Hopefully, the Nanosoldier will soon have received enough training to detect such a change.


So both are going to be one month instead of two?


@tremainebuchanan We want to store exactly what the user enters and display what the user expects.  The issue described how to do the conversion but it didn't specify where to use the conversion.  There was another thread of discussions floating around, where I described how and where to use the conversion.


don't commit commented code


I really think that we should make it so that the boss doesnt fire regular bullets, and a new kind. I have already made the texture, and I'd be willing to write the code to make BossBullet class a usable object.


Could you space things out a little bit more? ` + 1` instead of `+1` would be nice.


m) dit hätte ich auch sehen können


Actually not sure `redirect_uri` is a good name here, how about `application_url`?


Same for here: `[False] * num`


what if id == null


And here `[True] * diff`


And here `len(split_line) - num`


Wenn travis sagt das ist kaputt, dann ist das kaputt :P


maybe don't pass `stdout` if its not going to be used.


Ich habe in meinen Übersetzungen bisher immer "Kompilierzeit" verwendet.


:+1: Django docs favor inheritance over using `{% includes %}`, but I find this much more flexible and easier to understand.


Small improvements on checkstyle. nice


ok!


use Double.parseDouble and catch exception


You mean creation of the `ByteBuffer`? Done.


I realise that Rubocop will have other opinions, but `zone` is redundant here.


May we put the Exception on the first place? In general, users should be encouraged to not forget eventual exception handling...


link_to doesn't need the `method: :get'


You mean that I should get rid of the internal buffer altogether and write directly to Writer?


is Start an id ?


Do we need `secure_compare` at all now?


done.


I think it would be more appropriate to have this insertion in the next paragraph, since this gives a general overview, but the next paragraph starts talking about command line arguments.


@tfennelly Hi Tom.  At the time of committing this change, this call to updateBuildHistory() was failing due to a syntax error and so was not completing anyway, so this change does not affect the fact that the updateBuildHistory() call does not succeed in this case.  From my understanding, the condition of this block is met only if the build history is _collapsed_ and no build history is being shown, in which case the build history list is hidden and thus refreshing it isn't needed.


Yeah, I had it without before, and did this solely to shut Rubocop up.


Please link to diffs of the 2 JAR files. If not here at least at the PR level.


Nice catch :+1:


iLAB Editor: Khadija Sharife


Would be cool to not commit anyting in msautotest, so patches apply directly on tar release.
Helpful for packagers ;-)


AIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIIII!!!!!!!!!!!!!!!!!!!


I though @bsongis had already done this, he has found this issue if I'm not mistaken... on second thought though I think he did it only on master! I have no Windows to test but I can search and cherry pick the changes from @bsongis, what do you prefer @kilrah? (it is actually just a bunch of casts, as suggested by @projectkk2glider)


Actually, on thinking about this again. I'll redo this to implement a sane ordering so that we don't get bit by this somehow in the future.


I'm wondering whether this is the task of the promise or the promisor? What you sometimes want is "resolve the promise with the value of another promise"… that's what it is doing ultimately. So… looks much more like it would be the task of the promisor as only the promisor should alter the promises success and failure.


Βλέπω τελικά τα κατάφερες να ανεβάσεις εικόνες!


s/desser/dessen/ oder nicht?


Seems like that would be more robust. Especially since there is no wildcard or anchor.
Will update.


I think that there has accidentially an ``s`` been deleted: ``If you elect to`` ...


Remove the console log.


Refresh can actually fail if UNIS is down, which means you can't re-enable.  The error should still be failed.  The re-enable service message should be moved down to when that actually happens.


**Code Quality - Critical**

The spacing in your `if` conditional statements are inconsistent. Refer to the others in lines _14_, _28_ and so on.
Make the adjustments to standard.


In general, isn't that function pretty much redundant, when we have a `then` function? Just check if the second parameter of then() is set… or remove the second parameter of then altogether...

I prefer the former though.


Da fragt man sich, warum github so eine Funktionalität nicht implementiert hat (soweit ich weiß). Viel Erfolg :)!


Maybe not, I guess! Waddya think? Kill it?


Inconsistent indentation.


Hey @kurzee!
Could you elaborate a bit? I’m not sure where the concern would be.

The values for `sourceView` and `sourceRect` will be used to calculate the correct location to display the popover in the context of the popover’s UIWindow. This is done via a calculation similar (I’d say identical but I can’t see under the hood) to `UIView convertRect:fromView:`.  We can test a call to `UIView convertRect:fromView:` to compare the result `button.frame`, `button.superview` and `button.bounds`, `button` and see that the result is identical.  I think the important thing to note is the view specified in `sourceView` is the view that gives context to `sourceRect`.

An advantage of passing `button.bounds` and `button` is the popover remains correctly anchored to the button after an orientation change, where as the popover will be misaligned if `button.frame` and `button.superview` are passed.  We *could* update the popover positioning via a delegate method, but I’m not sure there is a need for the extra overhead.

Another reason I like using the view itself is we’re not guaranteed to “own” its superview in all cases (e.g. a UIBarButtonItem with a custom view) and I think its a best practice to avoid relying on a view we don’t own when there is another way.

Is there an advantage to using the superview and frame that I’m overlooking?


No space between `value` and `=`.


`b..jpg` needs renaming.


templates now working correctly.  I was working with a Django 1.7 tutorial while work with Django 1.8.  The templates are now being stored in the app directory rather than the project directory. This is better for coupling and I assume that is why Django made that change.


Don't worry about GridFS--it's only needed for files > 16MB and we should enforce a file size limit of 1-2MB. Also we will always want to load the whole file and not just part of it.


Δυστυχώς στο άλλο repo (testtest) δεν μπορώ. Αυτή η σελίδα είναι απο generator του github που έχει έτοιμα πρότυπα.....Υποπτεύομαι ότι φταίει που δεν εχει την μορφή username.github.io...θα δώ


I think this will break the infoboxes used to present album descriptions. Like if the window is resized, then the dropdown will be too long or too short.
InfoBox info: https://github.com/owncloud/gallery/wiki/Gallery-configuration#album-presentation


It's not slimroms, atm I'm working on top of the aosp source ;)


You need to know if the service was not found or if UNIS is simply not reachable.  You don't want to call setup_service if UNIS is not reachable, it'll just generate more errors.


check!


This is great! Waiting for this for years.D


**Code Quality - Nitpick**

You can return the `.test` function call as it returns the pertinent boolean, as opposed to creating a conditional block.
Hence:
```
String.prototype.hasVowels = function() {
  return /[aeiou]/g.test(this);
};
```

This refactor can be applied to line _62_ as well.


Looks like the user will only be able to update settings OR manage their phone number in a single page load - just flagging to make sure that's intentional.


@lirainer Thanks for catching this.


Technically it's right. You do not use the same type.


Correct.
So I just need this special configuration to avoid cache coherency problems on 2-3G.

Sent by MailWise<http://www.mail-wise.com/installation/4> – Your emails, with style.:)

-------- Original Message --------
From: Vineet Gupta <notifications@github.com>
Sent: Friday, October 9, 2015 04:36 PM
To: EZchip/linux <linux@noreply.github.com>
Subject: Re: [linux] ARC: [plat-eznps] Disable D$ for kernel (7870ee3)
CC: Noam Camus <noamc@ezchip.com>

so looking at prev email of yours
"These huge pages also got bit for cache yes/no and the fast path applications need it enabled.
The fast path application do not worry about coherency since they are organized in a way that their data is private."

—
Reply to this email directly or view it on GitHub<https://github.com/EZchip/linux/commit/7870ee377b255e6f3c4067e1b35d7d522194381a#commitcomment-13686520>.


sure! let's remove all tests :) more seriously, we've changed how tests get run on branch-7 onwards and don't need a git subproject and such commits anymore, but they are here to stay on branch-6-4 unless someone wants to backport those changes.


Awww...mein Tamagotchi ist nie den "guten Pfad" gegangen, man musste auch nur einmal was verpassen und schon war alle Mühe umsonst. Ich hoffe euer Tamagotchi ist etwas nachsichtiger mit seinen Haltern!


I'm having some issue with this part - for me it works only when changed to following:
```
if args[:hypervisor_host] != ''
    host  = client.hosts.find { |host| host.name == args[:hypervisor_host] }
    logger.info "create_vm_from_builtin: host : #{ host.name }"
elsif
    host  = client.hosts.first
    logger.info "create_vm_from_builtin: host : #{ host.name }"
end
```
or even move logger.info outside if


s/aquiriert/akquiriert/


Notice how you're not using `i` any more? :) This should be outside the loop, you don't need the loop and you don't need `expected = np.copy(input)`.


Also, not gonna hold you to it, 'cause I think this is how we normally do things (including right below here), but I think on successful form submission, you generally want to redirect (rather than return the `get` method), so a reload won't resubmit the form.


Ok. Now what? :)

(I get notifications for "@dennis" - and I assume its not me you want to check this.)


Hmm... it should be fine. The callback block will call `setCurrentFilterIndex` and finish execution before calls to `viewWillAppear` or `viewDidAppear` are dispatched.  But now that you've pointed it out its tweaking my OCD a bit cos it *looks* out of order. Let's switch the order of the calls.


I have reverted the commits. I should have discussed the issue with everybody before making the changes. The issue that I encountered is that dsge_likelihood.m, dsge_var_likelihood.m and non_linear_dsge_likelihood.m don't return the outputs in the same order. This is a problem as we want to write generic code for several part of estimations. I think that they should be rewritten using a uniform order for the output arguments and that using functions should be corrected. Using grep, my impression was that the number of calls to these functions was not large and I changed the calls in the present commits. Maybe I didn't see of the calls in the subdirectories.
However @JohannesPfeifer raises another important point dealing with commits in upstanding pull-requests. So we should deal with relevent upstanding pull-requests before dealing with the present issue.


Addresses https://github.com/nest/nest-simulator/pull/118#issuecomment-146710461


Hello,

The ZX Gesture is released since several month now (I bought mine in july 2015) but it seems the gesture detection is still in beta, not really supported. I'm confused because it's the goal of the "ZX Distance and Gesture Sensor"

I just have 2 small questions:
- I suppose Sparkfun will finalize this library when the new firmware will come but have you an idea when it will come?
- Also, as I buy an early version, will it be possible to "flash" the new firmware to my current sensor or I need to buy a new one?

Thanks,
Jerome


@tbonfort good to know it was changed on 7.0
I understand your wink for the backport :-)))


At least remove the default values.


Thanks for the clarification. If I understand correctly how it will work, there should not be any issue. Instead of creating multiple versions of multiple transients, there will be multiple versions of a single transient (one version for each combination of elements). Since all the product prices in each transient are generated within the same context (i.e. same currency, country, tax settings), then they will be consistent within the same transient.


:+1: Minimal API is preferable.


So much faster now !!  Thank you :)


I think this should have its own class.


I can't understand how you ended up to the src folder???
the `package.json` points veryone to the dist files and that's the only officially supported way to import localforage.

Anyway, the `ParseError: 'import' and 'export' may only appear at the top level` can be solved if you use `"loose": "all"` on your babel configuration as we do on [.babelrc](https://github.com/mozilla/localForage/blob/master/.babelrc).


Just want to confirm before I make the trip to campus that we are still meeting at  3:30ish in front of the library, Does this work for everyone?
@ecomitz
@MatthewM39
@MilesBenetDyson


Yes, let's. We still need to check that the key is present, though (to fail safe if the ENV variable is unset).


нахождение моды массива.
где то косяк


Thanks @JohannesPfeifer this is a mistake indeed.


> Can we just change that to be H2?

This doesn't change the fact that this particular tutorial is incredibly long. Additionally, I don't think it makes sense to try to fight the semantics of markdown just to support the OA.

If we just split the tutorial across a number of pages, it will both solve this problem and be easier for our students to use.


OK!


@maackle: resurrected in 99190fb.


I think it is indeed very insightful.

Our discussion last night was mainly about the maintenance scripts remove
alive BLiPP services records, and the running instances need to fix it by
themselves. It solved this problem, but I should have further considered
more complete use cases.

So, what would be a proper behaviour if UNIS is down? kill itself? keep
refreshing and re-enable itself until the connection is re-established?

On Fri, Oct 9, 2015 at 10:43 AM, Ezra <notifications@github.com> wrote:

> You need to know if the service was not found or if UNIS is simply not
> reachable. You don't want to call setup_service if UNIS is not reachable,
> it'll just generate more errors.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/periscope-ps/blipp/commit/d95504cfe71d4c546e84f4d7e9906d50f1216f99#commitcomment-13688045>
> .
>


Is it possible to pass the result of `link_to` to a traslation interpolation?


@brylie I meant that we need a task on https://github.com/brylie/meteor-api-umbrella, because we are not getting an error, but a string.


LGTM


After pulling this, I found this error in the log:

2015-10-09 16:30:24.745305 [ERR] mod_lua.cpp:203 ...pp/xml_handler/resources/scripts/directory/directory.lua:102: attempt to index global 'xml_handler' (a nil value)
stack traceback:
...pp/xml_handler/resources/scripts/directory/directory.lua:102: in main chunk
[C]: in function 'dofile'
/usr/local/freeswitch/scripts/app/xml_handler/index.lua:96: in main chunk
/usr/local/freeswitch/scripts/app.lua:48: in main chunk
2015-10-09 16:30:24.745305 [ERR] mod_lua.cpp:270 LUA script parse/execute error!
2015-10-09 16:32:57.265345 [ERR] mod_lua.cpp:203 ...pp/xml_handler/resources/scripts/directory/directory.lua:102: attempt to index global 'xml_handler' (a nil value)

I'm not an expert, but it looks like the xml_handler is not inizialized, is this possible? Thnaks.


Okay. Figuring out how documenting uploading works in general then.


Seems the valid_package_name! call could be done after the if/else


Φτιάξε ένα issue στο testtest να το ψάξουμε εκεί.


Test test


Test


Fabrizio, I don't understand the reason for modifying the MPTCP_SUB_LEN_ADD_ADDR4_ALIGN from 20 bytes to 16 bytes.

In mptcp_options_write you have the following sequence:

if (unlikely(OPTION_ADD_ADDR & opts->mptcp_options)) {
		struct mp_add_addr *mpadd = (struct mp_add_addr *)ptr;

		mpadd->kind = TCPOPT_MPTCP;
		if (opts->add_addr_v4) {
			mpadd->len = MPTCP_SUB_LEN_ADD_ADDR4;
			mpadd->sub = MPTCP_SUB_ADD_ADDR;
			mpadd->ipver = 4;
			mpadd->addr_id = opts->add_addr4.addr_id;
			mpadd->u.v4.addr = opts->add_addr4.addr;
			ptr += MPTCP_SUB_LEN_ADD_ADDR4_ALIGN >> 2;

Initially, the ptr field point to the beginning of the mp_add_addr structure and after several operations the ptr is advanced with MPTCP_SUB_LEN_ADD_ADDR4_ALIGN bytes.  In our case, let's assume that we receive an IPV4 packet with HMAC and port set and valid. This packet option would have a size of 18 bytes and yet the pointer is moved by the current value of the MPTCP_SUB_LEN_ADD_ADDR4_ALIGN which is 16 bytes. Do I miss something?

Also, wouldn't you consider it better placing the HMAC field outside of the union since it is not IPV4/IPV6 dependent?


test


Yes, just make the resulting string as `.html_safe`


А что случилось с провайдером?
Вроде как он работает, и сайт http://cab.stsats.ru/ по прежнему пускает на вход.

Да, и потом, зачем вы вписываете себя автором провайдера, автором которого, вы не являетесь?


LGTM!


Thanks, it works and now I can use bootstrap


Test.


Yes, that what blipp did before.  It keeps trying and writes "refresh failed", but now you added setup_service, which is what you want to do if the service doesn't exist, but not if blipp can't reach UNIS.  The solution is to know if UNIS is unreachable or if the status was "404", and only re-add the service if "404".


Adicionado todo o corpo totalmente funcional do Hist03


오.... 메로나


Я бы это вынес в отдельный метод вроде `_setupPresenceListener` и вызывал бы его [https://github.com/shriming/chat/blob/89d9726d4c926612c8e7f5a3d557543fc81de89d/frontend/common.blocks/list/list.js#L17](на 17 строчке)


The email address on your website doesn't work (bounces). Can you get in touch via nickshanks@nickshanks.com ?


@rhempel MotorHelper: the usual method to do this is build a superclass for the overlapping functions and subclasses for specific methods. This clashes with the structure of the generated code, though. So I'm afraid some repetition will occur.


Giving the nod to Alkalili I see


What an idiot!


index


Remove `static`, use `CustomFog.instance.colorRed`


@nl_0 that's what I was thinking.

On Thu, Oct 8, 2015, 14:49 nl 0 <notifications@github.com> wrote:

> @bloo <https://github.com/bloo> that's cool. btw we can factor out
> reusable api client and publish it as a separate npm package, so that our
> front-ends and cli clients could depend on it
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BellhopApp/npm-bellhop-api-client/commit/3a01c08f15ad4708d3201476b2ed7d3d5304e660#commitcomment-13670661>
> .
>


You need enfroce FusionPBX recreate `config.lua` file.
I think it possible by Upgrade->Default settings.
But also you can just go to Domains -> <ANY DOMAIN>
And just press save.
Or add
```Lua
--set xml_handler\n";
	xml_handler = {}
	xml_handler["fs_path"] = false
```
To `config.lua` by hand.
Or ...

`config.lua` usially placed in `FS/scripts/resources` directory


@psynaptic I'm working on `commerce_paypoint_advanced` module in different branch which works with different PayPoint [API](https://developer.paypoint.com/payments/docs/) as mentioned in [here](https://www.drupal.org/node/2581111). Is there any chance that you can assign me the maintenance permission for the module on [d.o.](https://www.drupal.org/project/commerce_paypoint)?


Openpli 8.6.0 z 25.06.2014
08.10.2015 11:23 PM "herpoi" <notifications@github.com> napisał(a):

> Picony w NumberZap działają jedynie na Graterlia, bo tam jest dodana
> poprawka umożliwiająca wyświetlanie logo na tym ekranie.
> Dodatkowo w aktualnych OpenPLi nie ma problemów z przeźroczystym tłem przy
> skalowaniu pikon.
>
> czw., 8.10.2015 o 22:27 użytkownik herpoi2006@gmail.com <
> herpoi2006@gmail.com> napisał:
>
> > Jaki soft wgrany?
> >
> > czw., 8 paź 2015, 21:03 mihail1 użytkownik <notifications@github.com>
> > napisał:
> >
> >> Witam
> >> No i przy przełączaniu kanału gdy ustawie środek z bigpicon jest tylko
> >> cyfra kanału nazwa żółtymi literami i tak gdzie ma być ikona jest puste
> >> pole:(
> >> Sorki za moją nie wiedze ale dopiero raczkuje w temacie.
> >>
> >> Pozdrawiam
> >> Michał
> >> 08.10.2015 8:55 PM "Michal Kawaler" <michal.kawaler@gmail.com>
> >> napisał(a):
> >>
> >> > A widzsz tego nie wiedziałem:/ a ja durny że starych picon nazwy
> >> > kopiowałem:d tak poradził em sobie właśnie z tym i wrzuciłem duże do
> >> picon,
> >> > tylko gdy chciałbym ustawić coś z klasycznymi piconami są one wtedy na
> >> > sztywno przeźroczyste.
> >> >
> >> > Wygląda to mniej więcej jak w załączniku.
> >> > 08.10.2015 8:22 PM "herpoi" <notifications@github.com> napisał(a):
> >> >
> >> >> Jak to ręcznie przypisywać kanały do ikon? Przecież OpenPli już od
> >> niemal
> >> >> roku rozpoznaje pikony o nazwach kanałów.
> >> >> InfinityHD już od dość dawna widzi tylko pikony w
> >> >> /usr/share/enigma2/picon/. Wrzuć tam po prostu duże pikony zamiast
> >> małych
> >> >> i
> >> >> będzie po sprawie.
> >> >>
> >> >> W dniu czwartek, 8 października 2015 mihail1 <
> notifications@github.com
> >> >
> >> >> napisał(a):
> >> >>
> >> >> > Robisz super picony;) szkoda, że się nie znam na tym i muszę
> ręcznie
> >> >> > przypisywać kanały do ikon.
> >> >> > W Twojej wersji infinityHD mój nBox nie widzi picon w katalogu
> >> >> > /usr/share/enigma2/BigPicon/picon/ widzi tylko z tego
> >> >> > /usr/share/enigma2/picon/. W związku z tym jak chce gdzieś duże
> >> picony
> >> >> to
> >> >> > dekoder skaluje te małe standardowe i wszystko wygląda jak jeden
> >> wielki
> >> >> > piksel
> >> >> >
> >> >> > —
> >> >> > Reply to this email directly or view it on GitHub
> >> >> > <
> >> >>
> >>
> https://github.com/herpoi/BigPicon/commit/e58d0950ea9f1920648864c73cd69cb59310e8e2#commitcomment-13668443
> >> >> >
> >> >> > .
> >> >> >
> >> >>
> >> >> —
> >> >> Reply to this email directly or view it on GitHub
> >> >> <
> >>
> https://github.com/herpoi/BigPicon/commit/e58d0950ea9f1920648864c73cd69cb59310e8e2#commitcomment-13670139
> >> >
> >> >> .
> >> >>
> >> >
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/herpoi/BigPicon/commit/e58d0950ea9f1920648864c73cd69cb59310e8e2#commitcomment-13670945
> >
> >> .
> >>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/herpoi/BigPicon/commit/e58d0950ea9f1920648864c73cd69cb59310e8e2#commitcomment-13673987>
> .
>


The correct usage is "elect", as in "elect to use". The correct usage of "select" is "If you select the new file locking mechanism...".


Can "comm" have spaces in then causing some issue with awk filtering? If this is an issue, I would suggest putting stat field first, and comm as the last field.


You should always use one `main.c`!


LGTM!


LGtM!


Looks good otherwise!


The importance of single `main.c` is to ensure both implementations being buildable and comparable.


you should not escape in calling code, but in the methods itself which build a css-selector out of this parameter


Yes.


@vilchik-elena okay!


Why there is no permission level on fog command?


@pynicolas :)


should we search for exact process name?


the comments describe the obvious, so please delete it


`@SideOnly(Side.CLIENT)`


done.


same here, delete the comment


I find this really useful, so I followed the gist linked to [here](http://stackoverflow.com/questions/19781163/refactoring-a-large-routes-rb-file-in-rails-4/19781204#19781204) to get the basic functionality working.

The problem I'm running into now is the lack of autoloader support. I see in this commit that a bunch of integration points had to be made to autoload the files in `config/routes/*.rb`. Does Rails 4 have some cool API I don't know about that will allow me to affect the same changes without monkeypatching?

Thanks!


@tchaikov I cleaned this up, and fixed the v6 case too (which was totally broken.. reqid is at the very end!).

This'll need to go through some upgrade tests before merging, but see if it looks right first?


Thanks for the explanation, Ezra.

On Fri, Oct 9, 2015 at 10:57 AM, Ezra <notifications@github.com> wrote:

> Yes, that what blipp did before. It keeps trying and writes "refresh
> failed", but now you added setup_service, which is what you want to do if
> the service doesn't exist, but not if blipp can't reach UNIS. The solution
> is to know if UNIS is unreachable or if the status was "404", and only
> re-add the service if "404".
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/periscope-ps/blipp/commit/d95504cfe71d4c546e84f4d7e9906d50f1216f99#commitcomment-13688403>
> .
>


You should make something to make the server control the client's fog.


My bad!


Don't you notice that I use the command handler on the client side to register the command? This mod *shall* only affect client side.


Will do


There should be nothing on the server side.


Sorry, I was misunderstand the pervious advise. I will revise it.


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


Protocol が Factory になるのは1つの方法としてありだと思うのだけど，使い方としては，
Protocol を保持しておく意味は特にないから，
```MachineBase machine = (new Ev3Protocol(new BluetoothCommunicator())).createMachine();```
ってなるんだよね？


The compiler can inline it if he wants.
Reusing code increases maintainability, that means if you find a bug on `loadb`, it'll also be fixed for `loadtqb`. It's easy to forget that another code is similar and has probably the same bug.


Awesome, thank you for your effort. Hope it will be released soon!


@randaz81 What is the rationale for this change? I actually find the single file import feature to be much more useful then the mass import.


@Pipsqueake to prevent this from potentially getting muddier in the future I'd like to create a new CALLEXPORT routine to handle running this manually rather than adding it at the top of the program.  Not that it doesn't work, it just makes it harder to test things out by having more than one thing going on at the top of the program.  So instead of having EXPORT registered to 20b56 you could have it registered to the new program number and then call EXPORT like you are doing now.  Also when adding a new run we'll want to add that to the SETUP.INF so that the runs get registered properly when this gets deployed.  Can look at the existing SETUP.INF file and ADD EXPORT to the list and report number.


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


DTM!


For testing purposes, will not be included in final release


@lindamartin we don't need to extend CheckTest


I don't understand please put more context here


改错了吧，把 let 替换为 const 了。


@jkotas Maybe I'm missing something but what's not well about this code on 32 bit? Sure, long is emulated with 2 32 bit registers but that code is likely better than the code necessary to read two ints and combine them.

But this reminds me of another issue - is there any guarantee that the first element of a byte array is 4/8 byte aligned on all 32/64 bit platforms?


Can you change the unversioned file? I think the numbered versions shouldn't change (branching wasn't invented back in 2009 :)).


I found better and standard solution for this purpose.

java.util.Objects#requireNonNull(T, java.lang.String);


方法としては，MachineBaseがProtocolBaseを取るんじゃなくて，Communicator自体を取るというのもあるのかなと思います
MachineBaseは使えないわけだし


This is an estimated guess. Opening the official TerrariaServer in the assembly browser is showing the same code as what we currently have. There's no logic to move the NPC when it can't hit the player. But when executing the official TerrariaServer the NPC is being moved. The teleport I've used here is comparative to what I'm visually seeing in the official and is not by any means an exact reproduction.


@traversaro We just removed the use of `ResourceFinder` for finding files as it was at the beginning. Now all the paths have to be relative.
The reason for this is that we want to replace the tags to import files (`<devices>`, `<params>` and `<actions>`) and replace them with `<xi:include>`, in order to be able to import the xml files from other tools, and the `ResourceFinder` makes it impossible.


`local.properties` doesn't support `~`.


yeah, probably c/p'd a button


Is it getting phased out in 1.9?

In any case, super clean code. You clearly had a much better idea of what you wanted here - I basically futzed around with the request.POST querydict until I found all the pieces I was looking for. This is much nicer!


> MachineBase machine = new Ev3Machine(new BluetoothCommunicator());

ってして，Ev3MachineがEv3Protocolを使うのは前提としちゃうとか


I think that, semantically, what we want to see here is whether the value is equal to the default specified at phonemetadata.proto, not whether it's empty (the default happens to be empty in this case). So I would prefer
```
!currentMetadata.nationalPrefix.equals("");
```

For more background on this, see my comments at https://github.com/AnastasiaSulyagina/libphonenumber/commit/80300165248486185571e2e8ef6efbb560602f58; specifically, according to https://android.googlesource.com/platform/external/protobuf/+/master/java/README.txt - "Nano version",
```
How fields with defaults are serialized has changed. Because we don't
keep "has" state, any field equal to its default is assumed to be not
set and therefore is not serialized. Consider the situation where we
change the default value of a field. Senders compiled against an older
version of the proto continue to match against the old default, and
don't send values to the receiver even though the receiver assumes the
new default value. Therefore, think carefully about the implications
of changing the default value. Alternatively, turn on accessors and
enjoy the benefit of the explicit has() checks.
```


还是不满意，判断条件过多。
后续继续简化，争取语句简洁，效率提升。


Sehr interessante Idee. Gibt es dafür auch eine Ingame-Erklärung? Ist das Notizbuch verflucht oder so? :)


FWIW I'm planning to write tests to ensure these defaults never change.


Yes, we plan to update the library when new firmware is released. For the time being, I2C definitely works. If you are able to use I2C with your microcontroller, then I highly recommend you use that to detect gestures.

The firmware is created and maintained by XYZ Interactive (http://www.gesturesense.com/xyz/). I don't think we (SparkFun) will be able to release the firmware. I recommend contacting XYZ Interactive if you have questions or concerns about the firmware.


> it only formats a zoned time.

But dealing with a time zone to me also seems to imply that you deal with things like daylight saving time, right? Otherwise you'd need 2 different time formatting functions, one that deals just with zones (but when are you going to use that? Half of the year it's answer would be wrong in most parts of the world) and one that deals with DST but that means it needs to know the date.

If I look at the APIs for date/time formatting in Java 7, the new API in Java 8 and JODA there doesn't seem to be anything dealing with *just* time, they all seem to take sources that could include both. And then you either have date + TZ info and you can adjust for DST or you don't and you just deal with what you have.

Btw, this is the new formatter used by Java 8: https://docs.oracle.com/javase/8/docs/api/java/time/format/DateTimeFormatter.html

It seems in Java 8 there's something that can hold "time parts" and then you can ask if it represents a time or a date or both.


@aerych I thought about this more last night and I think I may have confused myself with older issues of popovers in iOS6 and 7. If I remember correctly, there used to be issues with using source frames and views that were too small as bounds, but I that may been specific to those projects I was working on. Please disregard.


Can you maybe push some commits that put this in a try/catch? and attempt to import using urllib2 for anyone running python2.7?

Thanks for your consideration.


Hi Antony,
When I was trying to plot 3 contour lines by setting "contours = 0.68, 0.95, 0.99" in the *ini file, however, the plotted figure always shows 2 contours.  Then I found from the doc that this can be done by adding "g.settings.num_plot_contours = 3" in the python script generated after calling GetDist.py.

I think it's would be better that GetDist.py can also write "g.settings.num_plot_contours = 3" into the plotting scripts, so I read the source files again, and found that it might be better to remove the options "num_contours" and "contour1, contour2, ..." , leaving only the option "contours", and the number of contour levels can be obtained from len(self.contours) and be written into those plotting scripts.


0 shouldn't show up empty so you need an explicit `is None` here.


It worked, thanks!

Running "Advanced -> Upgrade -> App Defaults"

solved it, no more errors in the log.


How should this break?
`min-height: calc( 100vh - 120px );` does almost the same calculation, except that it subtracts `120px` instead of `200px`.
It looks like this on a small screen:
![image](https://cloud.githubusercontent.com/assets/1267827/10397817/eccdcb20-6ea9-11e5-83af-1dd8a49f1a1e.png)


FYI, instead of checking if 'readCelsius()' is NaN, you could just call 'readError()'. This would be somewhat clearer.
https://github.com/adafruit/Adafruit-MAX31855-library/blob/master/Adafruit_MAX31855.cpp#L118


@hinerm Would it be ok to remove this `'disp` command?
It clutters the command prompt.


Are you sure? It doesn't currently really seem to be trending: http://w3techs.com/technologies/details/pl-php/5/all


I will take a look this weekend.


a dict would be nicer:

```python
mapping = {None: '',
           False: _('No'),
           True: _('Yes')}
return mapping[registration_data.data]
```


This is a fix I made after pushing my answers to a question regarding Schema and how to add to a remote repo.


イメージとしては Protocol のほうが general な気がする (Ev3Protocol が SuperEv3Machine? とかにも使える) のでそちらの方が良さそうですね


@lindamartin It seems to me that both the old and the new implementation only care about the first opening tag. Is that the expected behavior?


@aerych I also believe it would make the call prior to the appearance calls, but I would say it's not guaranteed as it relies on the underlying implementation of `dismissViewControllerAnimated` to dispatch the appearance transitions on the next run loop, which may or may not be the case in this version or the next.


@drdanz Ah, basically then the changes in the `yarp-config` utility are induced by the change in the robotInterface... Actually I was referring to the single file import of configuration files that we have in robot directories that are not used by the robotInterface (see https://github.com/robotology/yarp/issues/419) but I don't see a way of supporting both use cases without being too error-prone, so I guess we'll just have to adapt.


again, please compare with empty string (nitpicky I know)


:+1: Cheers!


master to dev


As a result no one would add a mod which block their sight to their client/modpack.


Mind to reduce the number of test records here? 5 or 10 should be enough.


Linking to the conversation at https://github.com/wordpress-mobile/WordPress-iOS/commit/3b4790a2d497cbf7c9e3c738f50b5602576c6523#commitcomment-13677535 for posterity.


I need to fix Double Fault issues


I looked at this a bit and the problem is that the periscope_client and http interface return None for both connection exceptions and 404.  This will require a somewhat extensive reworking of the error handling within blipp to fix correctly.  I think it should be done but in a separate development branch.

Please pull your last commit out of master and starting a branch to work on this.  We need to the master branch to remain clean for packaging.


Just the idea of a basic function


wildchar?


remove


This one looks like it would be a clean merge.


What


nein, Fingerprint hat einen anderen Effekt wie minsecs und ist dafür nicht brauchbar.
Ein z.B. minsecs = 300 bedeuted, daß 5 Minuten lang keine Werte ins Reading und Filelog geschrieben werden. Auch nicht wenn die Werte verschieden sind.

Gruß Ralf


incorrect copyright year


missing slot on jpeg


This looks clean, we could use this line.


From documentation:
"The argument passed should be an absolute filename or wildcard expression."


missing slot on libpng


Nice find; thanks


1.修改url 正则表达式，以便正确获取图片.
2. 修改部分中文编码为utf-8


Undefined is a bit of a scary word because we rely on it. We actually want to make sure that iteration under modification never throws exceptions or otherwise breaks the structure. But we won't guarantee that everything makes it. Maybe another test for this?


じゃぁ Machine がそれぞれProtocolを保持する感じですね〜
MachineBase から変更しなくては


Looks good, and "nothing bad"™ happened in all the compiler, language module and sdk tests.


this looks like you should/could use `make_wrapper` from eutils.eclass


hey , which version are you working with?


Checked.
> Look good :+1:


Good catch. Yeah that line is only needed if your feature will use Chrome extension background scripts. Which I have not gotten to in the document.


Starting with this line, I think this is where the merge conflict happens.


The unversioned file seems to be quite different from 0.4, I can't find anything resembling that paragraph in it. I'm also not clear on which html file is generated from the unversioned xml file.


Yup, confirming that the forms are mutually exclusive right now - all we're doing here is just porting over the phone number management from the edit mobile worker page, and so we ended up with the same behavior here as on that page.

Good point about page refreshes - I hadn't thought about that. I was thinking returning get() would reduce the page load time, but it seems like in this case there's a valid reason for using redirect, so I updated it now to do so.


Por favor borra ficheros que no son de los ejercicios, como el .ds_store que aparece en este commit.


Sorry @thomasjo, this was replaced with a timeout in the main process in 4863b71fd71df1793364343de37c2fb3a45ca270. Thanks for having a look!


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


Thanks for this, and apologies for not catching it before committing your earlier change!


Style: Can you use t.equal(expr, 45) here? It makes the default error mesages easier to read.


why not use EAPI=5?


Looks reasonable.

Prefer `if cflags is not None` btw, for consistency with the rest of my code.


\o/ ah good catch :+1:


I'm less certain about this change.  The big use case that I can see is _if_ we wanted to build a cloudprovider outside of AWS.  The most likely use case for that would be if we built a better installation program that reused the cloud provider.   Now I'm not sure whether we would do this, and indeed if we did whether we would use the configuration mechanism

Another use case that people have proposed in the past is putting an AWS id & secret in here (instead of using IAM instance roles), although I think we opted to go with env-vars instead.

My inclination is that we should separate this one out as it is less self-evident than the others.  Also because GCE still has a configuration section - I'd be much more comfortable removing both at the same time...


This should fix issues people were having with the timestamp being off by several hours, yes?


Do I gather correctly from this check-in that the purpose of this repo is a IIIF Presentation application rather than just providing an API for working with presentation metadata?  Curious because of the addition of the Spring stuff.  Thanks!


@lindamartin LGTM!


Ah tu veux le www toi :yum:


Fog *is* used to block sight in order to rise the difficulty. I don't care who will use it.


@Kevindeving any solutions or ideas??:)


Removed it alltogether now.


A great simplification - thanks!


Heh, yeah I ran them all and nothing seemed to have changed, but who knows what kind of corner cases I missed. Especially the `ant nodetest` doesn't really return something that's easy to "parse", all the failure counts are mingled in with the output. I still wish it was possible to run them as JUnit tests in the IDE so you'd have a much nicer view of what works and what fails.


Yeah, this one keeps coming and going because I keep trying to switch node names to be instance ids (instead of private DNS names).  For now, it is unused & inefficient, so I agree we should remove it.


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


I think synchronized(JSONKeyPathsByPropertyKeyByClass) instead of synchronized(self) would be more suitable here


**Code Quality - Nitpick**

You created some variables in lines _6_, _14_, _23_, _31_, _38_ and _67_. You can pass those values directly into the functions or return statements they are used it.

There's no need to created a variable when it's going to be used once.


Small fsck-up on my side, sorry for that.


I organized the tutorial into logical milestones by page, where the end of each page has a working result for the student to test.

It may be that some can be broken down and should be - but I have absolutely no time to commit to that. As it is now, these changes break the tutorials, so should be reverted @alengel


Well, that would be difficult because `OAuthApplicationPresenter` does not represent a single application but all of them. That's why I'm redering a single application as JSON here to be passed into the subviews.


@lindamartin LGTM


Added Amazon AMI 64bit HVM EBS linux kernel configuration file


THE DRAMA WE'VE ALL BEEN WAITING FOR


I'd move the knowledge of the regexp and the position in here, so that you have something like:

uas = {'edge': (/Edge\/(\d+).(\d+)$/, 2),
           'chrome': (/Chrom(e|ium) (\d+)/, 1),
           ....}

and then the call could be extractVersion(navigator.uastring, 'Chrome'), and you don't have to reproduce the regexps and the positions in the tests.

That way, knowledge of UA string formats is limited to the extractVersion function.
The disadvantage is that knowledge of UA string parsing is in a different place than the knowledge about the rest of the browser-specific parameters.

Does it matter?


NOTE: Only tested on Mac- before merging- someone should quickly test on windows using the following link- https://rawgit.com/ericrius1/hifi/targetIntoEntitySwitch/unpublishedScripts/immediateClientReset.js


Agreed!  Would be nice if go had assert statements, but not sure I would put an assert here anyway!


One alternative I thought of - would extension methods be useful here? An extension method of ToChannelCredentials could be good:

var creds = (await GoogleCredential.GetApplicationDefaultAsync()).ToChannelCredentials();

Just a thought. It's a shame about the predence of the await...

I quite like the idea of GoogleGrpcCredentials.CreateDefault, mind you...


I probably should have committed earlier.


@lindamartin I would rename it to hasModifier


尽管有结果


对方感受到


If we wanted to avoid the whole "User gets a friend request", "User accepts/denied a friend request" stories, we could just just keep it simple like "User adds friend" and "User deletes friend" and then friendships can either be one-way or mutual. When I used to use Hamachi (a social VPN tool) that is how they did it.

Later on if we decide that we don't like it we could always change it in V2 and add those screens.

![Friend list for Chris](https://cloud.githubusercontent.com/assets/3138467/10398271/24ed5bea-6e7a-11e5-8b1f-3bde34ae77d2.png) ![Friend list for Pat](https://cloud.githubusercontent.com/assets/3138467/10398203/a85e03e0-6e79-11e5-826f-8446cb7267fc.png) ![Friend list for Sam](https://cloud.githubusercontent.com/assets/3138467/10398204/aa9cbcaa-6e79-11e5-8a7a-08ab32ca4444.png)

![Friendship Table](https://cloud.githubusercontent.com/assets/3138467/10398207/aea3c7c6-6e79-11e5-96d0-f5e29a58a517.png)

We could do stuff like hide online status for people that don't have you as a friend on their list, we wouldn't need to store "is accepted" because that would just be whether or not that other user has you on their list.

Not really sure what we would ever do with the 'isFamily' Boolean, but we were told to store that.  Do we need to have the other person confirm if that's true or not?


@lindamartin Why did not you use this method in rule implementation?


大师


Sounds like `OAuthApplicationPresenter` should be called `OAuthApplicationsPresenter`.


Would be nice to see a marker of "why not just disabled? Because fallback without JS"


Why submit instead of button tag?


Je t'avouerai que le www est le dernier de mes soucis...
Je comprends juste pas comment on peut mettre un lien CNAME vers xenomorphales.github.io/xenomorphales quand justement le lien CNAME n'accepte que les nom de serveurs et pas les "sous dossiers"


ASD


@lindamartin Except adding unused method (`hasModifiers`) everything looks good :)


LGTM


test comment)))


@traversaro The ability to import and export the xml from and to a database is a very high priority task at the moment, so I think that the possibility to import single files is not that important.
Anyway, this is actually something that makes sense and that we did not think about (I completely forgot that the robot folder does not contain only robotInterface files). @randaz81 perhaps we could consider moving all robotInterface files in a subdirectory, reverting this change and modifying the command to import a file or a subdirectory from the robot directory, but not a file in a subdirectory?

I.e., assuming that robotInterface is a subdirectory, this should work

```
yarp-config robot --import theRobot robotInterface
```

This will fail
```
yarp-config robot --import theRobot robotInterface/icub_all.xml
```


here is patched kernel for mako


@lindamartin LGTM


Test comment)


Right, the combination of having a zombie process with a name starting with "ganesha" at the time we're trying to restart ..... being something else but one of processes/threads in question is very unlikely.
Since a false hit won't have any bad side effect, I'd rather go with the current version and see which information we'll collect.


Use `@Instance(Constants.MODID)`


LGTM!


Sorry for my lack of response here, I got sucked into thesis stuff!

I tried to find a way of filtering the annotation, but all I could find was something like this:

http://timmyomahony.com/blog/filtering-annotations-django/

which seemed quite involved at the time. Then I stumbled across using this case and where. I did just try filtering the annotation, using say a .filter, but could not get it to work whatever way I tried. Would there have been a better way?


I guess static block is better.


Updating the reference to `MANUALS_TO_TOPICS` in the error message is proving tricky to implement, as I can't seem to access instance methods/attrs. Spoke to h-lame and will try making a custom validator to deal with this


WARNING: EVERYTHING IS WORKING IN PROGRESS, EVERYTHING IS NOT EXPECTED TO WORK PERFECTLY. DO NOT TRACK IT--- EVERYTHING WILL BE FINE *FINALLY*. I DON'T WANT TO SEE SUCH A MOD DIE IN THE PAST HISTORY.


thanks for the hints.
i still see one problem: is such handler's job like `tini` will at some point be the default for every Docker container; how would you remove the handler from the image and maintain backward-compatibility for older Docker versions?

from a user's perspective the latter is unnecessarily redundant:

> At present, this works docker run jupyter/notebook or docker run jupyter/notebook jupyter notebook <ARGS...>.

the charme of having the necessary options included in the entrypoint, is also that it's very easy to override in derived images.


-> #47


Remove the tab here or in main mod class.


Die erste Beschreibung, bei der ich wirklich lachen musste...obwohl's ja eigentlich eher zum Weinen ist.

Also das will ich wirklich spielen!


```
case 1:
                    // fall though  , is it violation?
default:
                    // it should be the same as for "case 1 :"
```


I consider it part of the engine contract that this will never be NULL. Again, if it ever is, something has gone seriously wrong.


Are those just copy-pasta from original frogcraft, or you fixed compile errors?


ok


osal lib! I hate auto correction.


os.walk should iterate over all files including sub folders.

Secondly, welllllllllllll that's true... will change it :-)


This allows you to view the change-log on start up, just after the screen has been unlocked.


This is not used by anyone. INVALID COMMENT


Google Style Guide "[dictates](https://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Function_Parameter_Ordering)" that input/output parameters should be pointers instead of const-ref. However, here it makes things harder to read, IMHO:
```C++
CheckForBrokenHardlinks(*entry); // de-reference to pass as const-ref
MaskFileHardlinks(entry); // might change entry
```
Practically everywhere in this file `entry` is a const-ref, except when it's not....


Directories such as /usr/bin are as of El Capitan completely blocked and even with commands like "sudo" and "su" you cannot manipulate them any more, and all stuff Mac OS doesn't recognise there is automatically deleted. I always use a special script to quickly run and update this repository and it was located there. I needed to come up with an alternate solution and I had to test if it worked the way it should. The fact that this commit appeared here on github is the solid evidence that this is the case.


Ahh right ENTER/LEAVE is for the stack frame.


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


I was hoping for an earlier time..because I honestly do not feel like waiting until 330, and I'm done with all my classes for today


this comment does not make this line of code any easier to understand :-1:


@vilchik-elena indeed, maybe with an "s" at the end ? hasModifier*s* ?


@Kevindeving any solutions or ideas?


So DMD is still generating antiqued ENTER/LEAVE


Generally I don't like magic numbers, but I'm not sure it really would add to the readability that much?


@liach Again, *this is a rewrite, based on old code from M3gaFra3k, using the original texture, due to the huge amount of incompatible code from 1.6.4.* PLEASE DO TO TRY TO TRACK ME!


@lindamartin I think you should check that `elseClause.is(ELSE_CLAUSE)`. See, it should not be issues here
```
if ($condition1):

else:                     // OK
 if ($condition2){}
 foo();
endif;
```


I would definitely introduce an inline function for this macro. In that way, you could return a value directly without passing around this result variable in the middle.


return EXIT_FAILURE;


Line too long


I see, interesting. I'll go and reorganize that separately, LGTM for the moment.


I would arrange the branches in another order, like so:
if (rc == 0) ...
else if (rc < 0 && errno == ENOENT) ...
else return false

The condition would itself document that the directory doesn't exist, and an error would still be returned for the impossible case of rc>0.


rc is too short for name and not descriptive enough.


Thanks, I forgot about it


void setup_caps(void);
etc. :)


I release these code just to show that I, as will as this project, are still alive.


If you think errno is not portable enough for our purposes, could you consider introducing an abstraction for error text? It's not important if you think it would be difficult.


@lindamartin Fix FP, otherwise LG!


Why add the extra braces?


Schöne Sache, könnte mir vorstellen hier mitzuarbeiten. Es ließe sich ja eigentlich auch ohne weiteres hin und wieder auch ein Bild oder ähnliches einbinden.


Right, I get your point. However it would only be used for the async posix stuff so the risk is quite small, whilst the cost of losing OS-X is quite large. I think it is worth defining it.


`HashFunc` should be marked as "static" since it is used internally.


Steps 5 and 6 seem to overlap "User registers for an event", the description for this use case was "User searches for an event through the website".


@vilchik-elena because I needed the var tree node to report the issue on. [VarKeywordUsageCheck.java#L48](See https://github.com/lindamartin/sonar-php/blob/9a82e1fc36334ad2a1e4705a4ea4d58b81369c8d/php-checks/src/main/java/org/sonar/php/checks/VarKeywordUsageCheck.java#L48)


To Chef-Essentials-Participant-Guide-v1.0.0.pdf


@drdanz no need to do all this extra work, all people working with local robot files should just know this and be extremely careful, but the database of robotInterface configuration files is definitely the top priority.
I am a bit worried of robots that need to have local modifications and so don't get proper updates in the configuration files, but I guess this is part of the tradeoff.


It is not elegant because hash functions refer to uncertain variable `method`, that causes runtime overhead.


Instead, you should write specific hash functions with the help of macros.


@mstembera Can you please take a look at this https://groups.google.com/forum/?fromgroups=#!topic/fishcooking/UXwGwZt4KmQ


@lindamartin why not just `CheckUtils.asString(tree)`?


@vilchik-elena indeed!


@lindamartin LGTM


What is the error message ? You can run the script from another path.


Ohhh i think I see what you mean now. by *most recent phase of the target contest* in the first comment you meant the most recent phase that this proposal is in. I understood it as the most recent phase of the contest, in which case the proposal would become hidden when it doesn't move to the next phase. Most recent phase of the proposal makes more sense, I will implement it like that :) Thanks for clarifying!


5255-EMS-dual-Temp-Node-Setpoints (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (1903 of 1903 tests passed)

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


ты видишь, что код дублируется?


Is your comparison approach compatible with the original?


num_contours was just there for consistency with old ini files.

The "contours" values are also also used for the marginalized limits (e.g. output .margestats values), and one may not want to plot all the contours calculated in the .margestats file, hence the separate num_plot_contours option.

Could add a separate option to getdist. But personally I don't use GetDist program for plotting any more - usually better to either use the getdist gui for a quick look (and to export starter script), or directly write a plot script.


@vilchik-elena Because as discussed with @pynicolas, this method is not 100% reliable and it is a little controversial.


I am confused how the OR logic works: The `find_candidate_transaction_for_refund ` will raise an exception if no transaction of that type can be found so either there is something for ':authorise' or will raise an exception?


Args, the wildcard stuff for this is something we probably should not have merged...

Anyway, i think we are good nonetheless... The parser for the unit file option EnvironmentFile= verifies absolute paths too, and I think that's the right thing to do, as the absolute path check does little else then check that the path begins with a slash, and that's a good thing for checking in globs, too. Hence, your patch is already right in this area.


Hi @dennwc, Why treat reflect.Int as reflect.Int64 and not according the size?


нафиг сколько новых строк? Есть отличная штука в шторме - reformat code. используй ее ПОСТОЯННО!


Is the `nil` case a valid case, and so what transaction are we looking for in the `find_candidate_transaction_for_refund` below?


@lindamartin Indeed. Why then did you implemented this method?


Why don't you use more generic and effective name? The pair of `init_phone_data` or `release_phone_data` is reasonable.


@romani
Before my changes:

case 1:
                    // violation (should have indentation level as next statement)
default:
                    // no violation

Lets discuss in skype.


Turns out this was neede


Why?


Attempting to create an App to login guests


@lindamartin But still you are using it


Yeah, this is really, really broken right now.

On Thu, Oct 8, 2015 at 6:17 PM, Ivan Reese <notifications@github.com> wrote:

> I think this commit broke the project — the components/qneh.coffee was
> removed, and there's no trace of a components/q-n-eh.coffee to replace it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cdig/q-n-eh/commit/86c216bf7237ee59bd71335e9d621a4b168523da#commitcomment-13676401>
> .
>

--
Robyn Hoffman
Designer/Animator
CD Industrial Group Inc.
···········································
robyn@cdiginc.com <robyn@carldyke.com>
Ph: 1·866·628·3224
www.cdiginc.com <http://www.carldyke.com/>


For example:
```go
const intSize = 32 << (^uint(0) >> 63)
......
case reflect.Int:
   if intSize == 32 {
       ftype = INTEGER
   } else {
       ftype = LONG
   }
```


Agreed, I couldn't think how you would set a #define that would add any clarity to what the type represents, I just am not keen on magic numbers either. I'm happy for you to leave it as a magic number in this case!


Didn't know you could do this. Cool.


Are we sure that we want 8 and 9 to be two steps? That would be inferring that we are doing something asynchronously and we need to deal with events/listeners/callbacks. So far everything has been synchronous (I think)


I've created a PR for this change: https://github.com/Alfresco/Aikau/pull/603


Does it make sense to move the global block up?


The pattern I've been using is to put *Optional* after the type **String**.

Also, I thought we decided to call this `data_dir` instead of `output_root_dir`.


I'm not worthy. :+1:


:hankey:
That was designed this way on purpose. If you link the lines to the refund, the quantity in the refund will be subtracted from the invoiced quantity on the sale order, and the line will become invoiceable again.
For example, you order 5 products:
- you confirm the SO --> qty_invoiced = 0 --> SO line is blue
- you invoice 5 --> qty_invoiced = 5 --> SO line is black
- you refund 5 --> qty_invoiced = 0 --> SO line is blue again

This is misleading for the user, and moreover if you need to invoice other SO lines later, these ones will be included in the invoice. More generally, in case of exception in a SO (refund, return of product), the user decides what needs to be done.

See commit 2d5776d248920267bcf6d002e4420dd3d6a8282e for example. The functionality was removed on purpose to prevent an automatic decrease of the delivered quantity.


@NadjaW you forgot to remove the routes.


Typo in "Examples"


400? For authentication?


These look great! =D


done


This method is only needed in the test layer.


can you ammend a little change here?

if (brokerSecurity == null) {
   brokerSecurity = "the-default-one"
}

you could maybe get the method asking for the property method and that would take care of the default...

small thing only.. just to be safe because of the maven plugins.


Not sure about this one. Maybe it's better to make the argument optional="True", add a min="1" to the parameter and ask here #if $input_options.skip: ?

Any opinions?


just do a test


> Sure, long is emulated with 2 32 bit registers

Depends on platform. E.g. loading of `double` is a single instruction on ARM, and it faults for misaligned address, on some ARM versions at least IIRC. Also, there are large performance cliffs between aligned and misaligned access on some CPUs.

We do have defines (`ALIGN_ACCESS` or `WIN64` - it really just means 64-bit platform) that could be used to tweak this further - but I do not think that forking the code further is good idea here...

>  byte array is 4/8 byte aligned on all 32/64 bit platforms

It is not documented anywhere, but I think it safe assumption to make for any reasonable CLR implementation.


Can you reproduce this failure? I can add new revisions. I infact did some of this for warp just the other day http://matrix.hackage.haskell.org/package/warp


good job!


and GetDiagnosticsForSemanticModel is defined only in language specific Compilation types.


Where did you get issue with BuildListener? I need know for test case.


**Code Quality - Nitpick**

From the use case of `res`, it's a callback. `res` is used to describe a result or a response as the name implies, not a callback, as it does here.

You should have your callback as `cb` or `callback`, literally.


Scaling Archer


We need tests :stuck_out_tongue_closed_eyes:


Didn't realize that. Lets not translate these items for now. On next
review, I'll do more testing to see if a mechanism for translation is
warranted. TODO

On Fri, Oct 9, 2015 at 5:28 AM, Ray Magauran <notifications@github.com>
wrote:

> working on these...
>
>    1. The text in the divs with class=kb are the abbreviated/shorthand
>    names for the fields themselves, referenced in my_base_js.js, and used in
>    the Keyboard/Shorthand Entry feature.
>
> RAD
>
> where RAD is the shorthand for the database field RADNEXA in the table
> form_eye_mag.
>
> An entry "RAD.prolapsed lac gland;" into the Keyboard Entry textarea will
> automatically place "prolapsed lacrimal gland" in the RADNEXA input
> location on the form (and in the database).
>
> If these "kb divs" get translated here, then they must also be changed in
> the javascript file, where RAD is linked to RADNEXA, etc for all.
>
> xlt/a are php functions. How do I do the translations over there so
> translating here will not break the functionality of this feature, or do I
> leave it alone?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bradymiller/openemr/commit/87c95fa272aa1b4169a78009b727c1fc777d662b#commitcomment-13685276>
> .
>


Nope, no exception should be risen anymore because a transaction must exist already (enforced by KB state machine). If there is really a bug (data missing), the layer above will throw something.


**Code Organization - Awesome**

Your folder structure, file naming and overall code organisation is really top notch.
Good job.


`transaction_type` is always `nil` in practice: when asking for a refund, the plugin doesn't know if the payment was initiated by an authorization or a purchase. That's why we need to check for both.


What does "nb" here mean? The name isn't very descriptive IMO.


Regarding the MPTCP_SUB_LEN_ADD_ADDR4_ALIGN: before introducing the HMAC entry, that value was set to 8. Following you reasoning, with a valid port the value should have been 10. This happens because the alignment value doesn't keep into consideration the optional port. I suppose that if the port has to be advertised, the code should explicitly advance the alignment value by two. I couldn't find any portion of code actually allowing for the port to be passed along, thus I couldn't verify my theory.
Before this commit, the value was mistakenly set to 20. I realized that four padding bytes were automatically appended after the _addr_ entry and right before the _mac_ entry in the finalized packet. This was caused by some optimization performed with _unions_ and _struct_ in C. Changing the _mac_ type from __u64 to an array of __u8 prevents the padding bytes to be appended and everything is "correctly packed" allowing to set  MPTCP_SUB_LEN_ADD_ADDR4_ALIGN to 16.

Regarding the position of HMAC: I agree on placing it outside, but there are some problematics about that. First of all, I would avoid to place it before the _v4_ struct, because I think that would break retro-compatibility with MPTCP version 0: the first bytes of the HMAC would be considered as the IP address. A possible solution in this case would be to have a different (but similar) _mp_add_addr2_ struct and use one or the other the  according to the version established during the initial handshake, but I would prefer to maintain a single struct and try to append the new HMAC at the end. Placing the HMAC at the end would cause other problems though: the memory occupied by a union is always large enough to hold the largest member of the union, in our case _v6_ struct. This would cause dirty bytes to be appended before reaching the actual HMAC (when composing an ADD_ADDR advertising an IPv4). Moreover, from higher prospective there is another problem placing the HMAC at the end of the current struct: the port would be always in between the advertised address and the HMAC value, without the possibility to use the effective packet's length to understand if the port is valid when the packet is processed by the receiving side (this is how it works now).

I would like to add that I just started developing all these considerations and they are just a work in progress. Any help on this matter and corrections about my current reasonings would be appreciated.


Hi,
I'm testing this on an AOD file with a 7_4_7 release and I see that the assertion fails. If I add the quantity "out.hfHadronFraction" to the sum (therefore falling back on the assertion intended for other releases) it correctly sums up to 1.0
Might be that the assertion logic got reverted between the different releases?


meh, I tried it once, it's not really powerful :P


@vilchik-elena indeed good catch!


@pynicolas indeed thanks!


Ok....so here I improved the uptime() function by extracting the number of min from the result variable. Then i calculated the values of min, hours, days and years using nested if statements. For the formatting part, I used a bunch of if, else if statements to assign the proper format to the uptimeData string that i will send out as a push notification.

One thing that I need to address at a later date, is to work on formatting the string better. I'm thinking of formatting it with printf() style format...well see..for now I need to bounce of to my real job ;( LOL.

Here's how the notification looks like now:

![img_0077](https://cloud.githubusercontent.com/assets/13931323/10399119/bda34fe4-6e7e-11e5-83d2-2c6d7e083364.jpg)


`Today Bluetooth low energy can be found in many cool applications, it's used to send some data, works as pay terminals we can use it to work with beacons.`

Sounding weird for me, I would suggest to rephrase that for something like:
`Today Bluetooth Low Energy can be found in many cool applications, it can be used from simple data exchange to payment terminals and the more popular usage with iBeacons.`


Thanks!


fgfgfgfg


It means similar to "choose" or "decide" - I [choose|decide|elect] to take the high road to Scotland.


when you wake up @staticfloat, we need to rebuild 0.4.0 binaries for win32 with this change, otherwise they fail `Base.runtests("core")` - probably also linux32


5255-EMS-dual-Temp-Node-Setpoints (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1909 of 1909 tests passed)

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


This looks quite dirty..


lel


good job!


cat so cute. dog too. 🐶


Move this test to the `invalid` block instead of just deleting it?


"Float32Array" -> "Float32Arrays", plural.


@vilchik-elena because I guess I have not thought that far when I implemented it and then I left it because I thought it would be useful for some other checks. You think I should remove it ?


typo: `through`

`But what if we want to build something funny with it, like a simple game turn based game? Imagine you don't need to go through this long setup, waiting for server players to be ready etc.`


Hi, why do we not have test case for instance when promise is rejected?


Added a reset to the `foundJSXKey` state in order to fix regressions introduced by adding support for the export and module pattern edge cases.


You should git ignore the __node\_modules__ directory so it's not staged.


I built


Don't you need xor too?


Good catch, @aks-!


Added in support for the module pattern (including function expressions). This change introduced a regression with the `factory` function declaration introduced in the classic and modern tests that I'm trying to debug now.


5255-EMS-dual-Temp-Node-Setpoints (EnergyArchmage) - 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)


I'm a little torn on this one, since it should already be reported by `no-unresolved`... but considering the implementation burden to tell the difference, probably simpler to enforce `no-require` here too.

So this test should also be moved to the `invalid` block, I think. (and commented better / changed to `./missing-module`)


Why the next block?  Why not the current block?  This would require looking ahead which is complicated if there are audioparam automations.  (Biquads are supposed to be a-rate now.)


Very good catch!


LGTM


As you can see, it's very simple, one send and the other receives the method as delegate.
As both receive and send arguments, we can get the command used in our game to recognize the packet type and the data which will come along with this command.


`add(a, b)`


:trollface:


Is the problem that it makes "require"ing more difficult? If so, I agree that this could be improved


Plus attempted to set up the submodule. I don't think it's working right…


Why not NaN to make it clearer that something unexpected happened?  Or just wrap frequency value around?


shouldn't it be 403-access denied?


So siehts wohl aus. Ich muss mir aber mal was einrichten dass ich ne Mail von travis kriege, weil ich habs erst stunden später gemerkt dass nix deployed wurde.... :/


You should simplify it like this
```PHP
foreach ($this->getColumns($table) as $column) {
    if ($column['autoincrement']) {
        return $column['name'];
    }
}
```


I'm sorry but it doesn't work for me: noofslots_total is always 0..


If you changed to `let!` like I mentioned above, you would no longer have less than 6 videos here. Basically the same thing as your comment above. I guess this is okay, I have just never seen it done this way...I prefer creating the needed objects in the spec when I will need different objects.


This doesn't do much. Better use just `absint( $value )` which calls `abs( intval( $value ) )` already.


You don't need to specify XHTML tags separately. `<br>` and `<br />` represent the same tag.


This still does not solve it as result is a ```dict```, but the following does (defaulting explicitly to ```None``` here):

```
item = result.get('item', None)
```


I'd change this to the following for readability.

```php
$str =  ( count( $this->allowed_tags ) > 0 ) ?  __( 'Feel free to use ', $this->textdomain ) : '';
```

That makes it clear you're assigning a variable here.


@ahdinosaur thanks for doing all this cleanup!


Same here.


I also modified a bit the output of serial monitor, used for debugging.

Here's a screenshot:

<img width="1128" alt="screen shot 2015-10-09 at 12 22 51 pm" src="https://cloud.githubusercontent.com/assets/13931323/10399422/a83846ee-6e80-11e5-9464-e0f49ef7d733.png">


Maybee
```PHP
foreach ($this->structure['columns'][$table] as $columnMeta) {
    if ($columnMeta['name'] === $this->getPrimaryAutoincrementKey()) {
 				return isset($columnMeta['vendor']['sequence']) ? $columnMeta['vendor']['sequence'] : NULL;
    }
}
```


Ah looks great


The option value needs to be escaped here, to prevent XSS injection.

```php
<?php echo  esc_attr( $option['value'] ); ?>
```


I am not sure i follow the intention here? mind explaining?


@floydnoel this change it may impact you.


Don't ever throw a raw Exception class, that's super wrong because it can only be handled by a catchall. In this case you should throw ComandLineException


Does this prevent a crash? If so, could you comment with steps to reproduce?


@robsyed, @dHofstede, @Steve-Bulgin and I talked for a while about whether or not we wanted to have Visitor & Member events for search, we decided that we did because the alternate flows were getting confusing. Not sure if we want to merge all three into one or not.


The fix would be to patch `src/gx_head/wscript` section after `if task.env["HAVE_GPERF"]` to add a parameter `--hash-function-name=jsonrpc_hash` or similar to the call to `gperf`. This way you would avoid the collision of the default name `hash` with `std::hash`.

My python skills are not sufficient to get that patch right, so I'll leave it to you to implement :)


woohoo


test


Yep, three weeks and a month, I have a plan for December.


As mentioned before, please use the more concise `std::move(*(item.zone()))` since std::unique_ptr already has operator*() built in (no need for get()).


For musl you can use [this patch](http://sprunge.us/dVXe) to overcome the differences in `strerror_r()`.


'rename-index', 'remove-old-build-index'

seem to be rebase artifacts? They are no longer tasks I think


These are `std::atomic` types.


I think that you have this conversely, you should use
```PHP
$this->context->getConnection()->getSupplementalDriver()->delimite($primarySequenceName)
```
When `$primarySequenceName` is not null


fixed


done


To make it easier to reference, consider linking this repository to Zenodo and making a Github release. That would result in a permanent DOI that people can use rather than a URL.


@ahdinosaur thanks much for reminding us of this!  I had forgotten we had not resolved that issue a while back, and had made my own assumptions, not a good practice.

Let's continue to work this out in the issue, or here is fine too.

It seems like either Holodex or NRP can translate from whatever we decide to our code, I think all the information is included in either case?

So the question is what is best for the vocab?  And in my mind, a secondary part of that is what is most standard?  (I know that is a balancing act.)


Where in here are <memory> types or functions used? I can't see a shared_ptr or unique_ptr in this file, is there something else that I've overlooked?


Hello I am encountering the same issue with libcares when running
on a windows machine.

http://c-ares.haxx.se/mail/c-ares-archive-2013-04/0045.shtml

Your post on the libcares mailing list was very helpful and I am using the
same fix you made in git commit f6cdd7d404d44eb2d6e8866e85ae651b89fc769e

https://github.com/piscisaureus/cares/commit/f6cdd7d404d44eb2d6e8866e85ae651b89fc769e

However, I think there is a bug in your fix above. When doing strncmp
with txtaddr, txtaddr is still set to its old value from the previous
iteration of the for loop.

You should move the strncmp to after the call to ares_init_ntop. That is move
code block between lines 1015-1019 to line 1026.


When changing a heading the link needs to change to, we could make the link #register-for-an-event-optional-registration or we could keep it as register-for-an-event and put the "optional registration" part just in the name inside the table and try to keep the headings simple.

Hovering over the link next to the heading shows you what the heading link will be. It usually removes spaces and special chars in favor of dashes.

![image](https://cloud.githubusercontent.com/assets/3138467/10399573/72ec9c3c-6e81-11e5-83cd-0124a450940b.png)


lgtm


Забыл реализовать editPoint, вроде не критично, но для обновления фигуры понадобится.


I'm just curious if we need to handle anything when the backgroundTaskExpired before compacting is done?


I'm not sure whether you are happy with how we are logging this exception by sending it through `child_pipe`, or whether this was missed and should be logged via `LOG.exception()`.


test


Haha


"Something something 16 bit something something".


:D


winnar


This is the wrong fix for the SPIE issue. The "series" field was actually correct. It's the "booktitle" field that was wrong. ADS always messes up SPIE entries and I'm continually having to ask for them to be fixed up. The book title should be the specific conference track such as "Software and Cyberinfrastructure" or whatever.


Let's update the comment to reflect the new name we will use (once we decide it on our IRC discussion).


Try to add disable_ht40=1 in your wpa_mesh.conf.


Please change it to:
```Go
const intSize = unsafe.Sizeof(int(0))
```
This variant is more readable.


You are write. I will rename it


LOL


Yes, this is necessary so the mouse doesn't get stuck on the edges


5255-EMS-dual-Temp-Node-Setpoints (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (1909 of 1909 tests passed)

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


I believe that singleStep support needs to be put back for debugging with GDB


hey a comment


why `event` @bsudekum ?


You should store this token in your ENV var instead of your code :)
Heroku let you do this easily.


Again, singleStep support should be put back in order to support debugging with GDB


same here


Al final lo hice con un do -while, y las lecturas con stdin y stdout, en vez de argumentos de entrada


Refactored to use the already calculated sizes


Fixed by: https://github.com/dart-lang/dart_enhancement_proposals/commit/a6437e3b6256be11fbb5394494831b7105aaaa3a


done


Just a question, how come we are hard setting instead of using the variables that were in place?


I think I put code in that used the original request to supply missing data for a redirect.  (i.e. the response generates the new request)  That's probably the way to go rather than saving the entire history as the requests are associated with responses.


I don't think we want to use the Window's surface, because we need to create a surface that matches the optimal format of textures


Wold it make sense to add an assert about this assumption?


solid Honda.


`BPP`s and `BFPP`s should normally be `static` - not currently an issue here but just in case someone later adds `@Autowired` deps or `@Value` to this `@Configuration class.


Part of the reason is the key isn't used on heroku, only locally, but yeah, at some point I'll pull it out.


:D :D


Can we use relative paths in a cabal file?


Finde den Commit jetzt eher kontraproduktiv. :-1:


The resolution will not change if you use SDL_WINDOW_FULLSCREEN_DESKTOP, and the other option when not fullscreen should be SDL_WINDOW_RESIZABLE


Yup that's cool!


Hmm I can't seem to find the commit in master, so yes if you see it I'd appreciate a reference!


Maybe you can also test with a non copyable struct (as a struct containing a unique_ptr for example)


This is definitely the preferred way, albeit maybe a bit less obvious.


93653630-MultipleYearRunPeriodForTabularReports (JasonGlazer) - x86_64-MacOS-10.9-clang: Build Failed

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


hey a comment


Or even

    return target.tier==='Exotic'?5:4


Fixed.


just as written as inline doc in the fix. You really dont want to clutter your log with stacktraces when it stacktraces to stdout anyway. People would like to see a clean log with liquibase error messages and not a stack trace at this point.


I'm not sure this comment belongs here?


Sure, I think we are settled on "eos-chrome-plugin-updater" now, right?


I never say no to an assert. :)


Exceeded! :+1:


I wonder if it's worth checking for existing PAs with those keys rather than unconditionally replacing them?

That way, someone could provide their own implementation by explicitly declaring the FB.


Which pem files do we need?
All I see is tests related data and it seems that we exclude tests.

```
[bitmask_client/] ➜ find -iname "*.pem"
./src/leap/bitmask/services/eip/tests/wrongcert.pem
./src/leap/bitmask/crypto/tests/wrongcert.pem
./src/leap/bitmask/crypto/tests/openvpn.pem
```


@Pipsqueake just noticed that if you export and SUBLWATCHER isn't running it doesn't start it up.  In this case I think we should fire it up.