I think that our recent changes to be python 3.4 compatible means that we are no longer 2.6 compatible
if so, then we can assume that OrderedDict is in collections.


Oh great! That's amazing of you. I have gone through the solution index and it extremely helpful. Thank you so much. May I respectfully ask if there similar index page for  Elements of Statistical Learning Hastie 2nd -Springer-V-ESLII?


Note: Guzzle supports resources that are objects implementing __toString, iterators, and callables. Looks like the Guzzle streams docblock needs to be updated too.


I don't understand what you mean by a restriction anymore


Bon, j'ai corrigé, je ne comprends pas ce qu'il s'est passé, tu ne me rends quasimment rien ..?
C'est dommage, vraiment.

On a passé du temps sur ce projet, je ne comprends pas qu'au final ça donne seulement ça.
Qu'est-ce que tu fais pendant mes cours ? :/

Bref, le récap :
Utilisation de git : 3/5
Qualité du code : 1,5/5
Professionnalisme, Rendu Visuel : 0/5
Investissement perso : 1/5
Total : 5,5/20


I thought l2j was all about moving these instances to DP?


hey @dharmamike - this isn't major, but here we use template_id, but in the serializer we use job_template_id. I'm building the UI to the current spec, but we may want to consider making them the same. Although, making them different actually marginally helps me out in one spot.


C'est un peu juste, le rendu de ton site.
Ce que tu as fait est correct, mais ça manque vraiment de travail, c'est très dommage.

Essaye de t'investir plus sur le prochain projet stp.
Le récapitulatif de ta note :
Utilisation de git : 2/5
Qualité du code : 3/5
Professionnalisme, Rendu Visuel : 1/5
Investissement perso : 2/5
Total : 8 / 20


This one is special


It would be great if you can share your experience!
I am opening a new issue to assemble the suggestions and track contributions of more guides https://github.com/tqchen/xgboost/issues/159

Thanks!


Presuming these test are pretty fast, this doesn't hit SQL at all does it?


Il faut faire attention, et toujours vérifier. Gaffe.
Sinon, de manière générale, ce que tu as fait est OK, mais dans l'ensemble je pense que ça manque un peu d'investissement. Il y a de bonnes idées visuelles, mais ça manque de finition, surtout pour un projet sur lequel on a passé aussi longtemps.

Le résumé de ta note :
Utilisation de git : 2,5/5
Qualité du code : 3/5
Professionnalisme, Rendu Visuel : 3/5
Investissement perso : 2,5/5
Total : 11/20


The ones found in comments, etc, yes (hopefully, never foolproof).


From testing and digging around in the code I am pretty sure that this exception is never fired (never say never, I know, that's why I put the asserts there ;); that's why I made the decision to not bother with the user readable exception anymore. My long term plan with this is to replace it entirely with some enum or something as you mentioned, but I kept that for a later stage. The code really shows its age and the fact that a lot of people have worked on it.

BTW: I am also removing all the headers, in order to make your name go away from all that pretty code ;)


By restrictions I mean the restrictions which I've enlisted in the list. In general it's restrictions with regards to the context-free grammar, which I believe has no scoping resolutions needed (I don't count modifiers, like public/protected, as scoping). Scoping I believe includes context-sensitive analysis. If you want to talk it out more in detail we can talk later if you want.


Globalement, C'est pas mal, on voit que le sujet t'intéresse, dommage pour certaines erreurs (et ces histoires d'images qui ne s'affichent pas).

Continue comme ça, et essaye de trouver un bon sujet, qui te motive, pour le prochain projet.

Le résumé de ta note :

Utilisation de git : 3/5
Qualité du code : 3,5/5
Professionnalisme, Rendu Visuel : 3,5/5
Investissement perso : 4/5

14/20


Bon, dans l'ensemble c'est vraiment pas mal, je suis assez content. Le rendu est propre.

Petit récap :

Utilisation de git : 4/5
Qualité du code : 3/5
Professionnalisme, Rendu Visuel : 4/5
Investissement perso : 4/5

15/20


The one associated with a GitHub commit, for example, lives on githubusercontent.com, and is actually being declared by GitHub as their Open Graph image for one, so those would still pass.


I don't understand why you are pushing against having a symbol table. We need one. lol. Even if we don't need it for weeding - which is why I said move the symbol table generation to the parser class. We need a symbol table.


Falscher @consti - aber viel Erfolg beim Projekt!
Liebe Grüße aus Bangkok! :)


You can just tag it here right, in github v1.0.0-RC2? I think we are broken b/c the -RC1 is assumed ahead of a @dev version especially if you have prefer-stable: true in your composer configs.


hmm. I tried including the package at top level `dependencies` node and it seemed to work - I mean build.cmd completed successfully without build errors. Though I did not see any other sample doing this.


I'm not sure I can think of a great use case for actually selecting a gravatar, but it might be nice if, when there are only gravatars, don't add any featured image. If they select the switch image icon, then seeing and selecting avatars isn't terrible. Then again, maybe not.


It's just a matter of priority you know. I know we need it, but I just want to emphasize that we don't need it now technically, and if we get to a position that is pressed in time we can delay the symbol table for later.

That's really what I want to emphasize on lol


@sebastianthomas in 0.8, all layers must inherit from `L.Layer` class.


Meh. We'll play with that stuff when we have less pressing matters, like TinyMCE. :)


> You can just tag it here right, in github v1.0.0-RC2?

No, that doesn't work. Packagist literally totally ignores the tag. Composer reads tags from packagist unless you use the "repositories" thing.


Deal.


Great, thanks Dave. I'll proceed to the next step.


I'm curious, why did you add this?


Good catch.


yep! thanks, fixed, finally.

Is it just me who has this problems? :) In any case, really appreciate you taking the time to look at this. I will wait for the new API to proceed.


# hello-android Jonathan Torres  Marisol Flores Melissa Alejandro  Osvaldo Vazquez


gfgfgf


This annoying warning is showing up everywhere. What is needed in order to remove this? is there an open issue?


Okay. I'll remove it then lol.


LinkedLists only have one Next node.  If there were multiple "Next" nodes, it would be a tree instead of a list.


@gwake1 What is the question?


86121710_WindowFrameDivider (macumber) - i386-Windows-7-VisualStudio-12: OK (2172 of 2190 tests passed)

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


The problem isn't only this commit but all commits of your sources! Alls need to have the right author.


This is a comment, cvt5175


Yeah, Graham by we coul use https://github.com/GrahamCampbell/Laravel-Flysystem as a CVS source, which would be great !
Because repositories are taken into consideration before packagist ;-)


Hi Craig


> Because repositories are taken into consideration before packagist ;-)

Yeh.


And...you can study git commands to change  author without cherry-pick


Should we use a regular expression for these clip values?

We should account for all these different values that could be passed in/set by the browser:
```rect(0px 0px 0px 0px);```
```rect(0px, 0px, 0px, 0px);```
```rect(0px,0px,0px,0px);```
```rect(0 0 0 0);```
```rect(0, 0, 0, 0);```


what did checkpatch whine about?


asd


Ajax changes to incorporate jQuery can be added to these.


:D danke!
lg :)

(@ConstTheFirst)


Need to add a constructor that takes an inner exception, good idea/catch!


Hello Carl.

> These are great footprints but you had forgotten the courtyard. I
> added them for you.

Uuuuh! I am getting old. Thank you!

> Next time, if you would like to create a pull request, it will be
> easier for another librarian to double-check your work.

Yes, you are right! I think, i could add the PDF/PS dokuments at
weekend. Then i will it do that way.

> Inversely and if you wish, as a librarian yourself you can give us a
> hand for approving other's contributions.

Of course, yes. But therefore i have to test with github with my own
repository. There are some things which are not really clear for me.

With best regards: Bernd Wiebus alias dl1eic


I think this is actually an issue with the grammar then. I believe that it should be obvious based on the number of non-terminal and terminal tokens in the rule. So rather than PackageDeclaration* it should be:
CompilationUnit -> PackageDeclarations ImportDeclarations TypeDeclarations
PackageDeclarations -> PackageDeclaration PackageDeclarations | PackageDeclaration
PackageDeclaration -> package Name;
etc. for import and type.
package declaration's parent will always be packageDeclarations
packageDeclarations' parent will always be compilation unit
and repeat for import and type

or is this incorrect?

and I finally understand what you mean by toParse string. And sorry - I agree with you. Just didn't understand what you were trying to say until now.


Je n'étais pas convaincu par l'intérêt de gérer de manière particulière les évènements de l'ordonnanceur. Je n'ai remarqué aucun changement fonctionnel. Moins on a de lignes de code à maintenir et mieux on se porte.


It's for a checkbox for additional settings.


so i think you will want to redirect back to your post that the comment was created on.

hint - do a rake routes you will need the topic and the post to get the post you just commented on.

let me know if that helps


should we just rename the 'appSources' sourceset to be named 'scala' here?


Related to previous...checkbox for additional settings.


Could we add a comment to give example of each case especially the one from ES6?


From this point on (Factory::create) the InputParameters will be referenced from the objects. Until this point the InputParameters are generic, so referencing is useless. It may be possible to have Factory::create have a reference passed in. There really should be two spots where InputParameters are created: (1) Factory::getValidParams and (2) in the InputParameterWarehouse.

What if the InputParameterWarehouse has a secondary storage of generic InputParameters that only creates one object for a Kernel, etc... Factory::getValidParams will cause the creation of this generic object and return a reference if it already exists. Then all the other pass by value stuff could probably be cleaned up to references. Then, when the object is created in Factory::create we copy the generic and store it for the specific object, as it is here.


This line is redundant. `CRVNetworkManager.h` is part of umbrella header already imported in [AppDelegate](https://github.com/netguru/carrierwave-ios/blob/master/Carrierwave%20Demo/Source%20Files/CRVAppDelegate.h)


Ha ok oui c'est vrai que ça paraît logique !
C'est cool je suis content même si c'est sur que j'aurai pu rajouter
quelque détails :)

Le 29 janvier 2015 12:20, Clément Oriol <notifications@github.com> a écrit :

> Bon, dans l'ensemble c'est pas mal. Y'a de l'investissement, de bonnes
> idées, c'est assez chouette.
> Visuellement ça manque encore un peu de finition et de soin, mais ca
> viendra.
>
> Quel dommage que tu aies laissé les typo de base, et pas intégré une typo
> depuis google font, ça aurait vraiment eu plus de gueule.
>
> Bon boulot en tout cas,
> Utilisation de git : 3,5/5
> Qualité du code : 3/5
> Professionnalisme, Rendu Visuel : 3,5/5
> Investissement perso : 4/5
>
> 14/20
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/baptistelassus/repotest/commit/f23934a8d9ed44085c1ea4e447782f63c7e707f6#commitcomment-9496488>
> .
>


space after `*`


I dont know how to GitHub...


here also


Shouldn't it be ````id="min"````


r+


Hm, I initially folded over `Option<PlatformRequirement>` but then changed to `PlatformRequirement`. I believe you are correct, however! I will try to write a test to ensure that this does not happen and will fix.


Shouldn't it be ````id="max"````


I had to have the PackageDeclarations* since it signifies that it may derive to an empty word (epsilon in this case). By the way the LALRgrammar.txt is a grammar to be read by us humans, the actual input to the parse table generator is in JLALRinput.txt, which is in my latest PR. I think it might be easier if we talk over this directly


in our current implementation assets go into an assets jar and conf goes into  the 'normal' jar. so we handle 'conf' and 'public' differently. I wonder if it makes sense to have one jvm resourceset that contains both. we could have an additional asset resources sourceset.


tab this over one


Oops, fixed!


saw approval from jdm
at https://github.com/Ms2ger/servo/commit/d752cdc44f4e0bb924538ee4ae758f71236b4483


merging Ms2ger/servo/snake-utils = d752cdc4 into auto


Ms2ger/servo/snake-utils = d752cdc4 merged ok, testing candidate = d026457f


This is not safe for Windows. Better to use an `isAbsolute(path)` utility method.


Please create separate class for whitelist. `CRVNetworkManager` shouldn't take responsibility for checking/saving/parsing whitelist.


@magedmakled


Yay! :+1:


Actually we do not use doxygen style documentation in implementation file. If something needs to be explain (is not clear) please use just `//` comments


Oh this actually got mixed up as well, this commit wasn't supposed to be part of this PR, it's actually #1243 (will fix over there though)


I have no idea what I am doing


Thanks! :)


87219700_ReplaceSimSettingLineEdits (evanweaver) - x86_64-MacOS-10.9-clang: OK (2170 of 2190 tests passed)

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


maybe `whitelistURL` ?


The feature of 13754bb800e5793a7686571de4cede7da9e24a8d was untested..


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


I think it is better to throw a real error here:
```javascript
throw new Error('resolvePath("' + path + '") failed. Target file does not exist. Started search from directory "' + from + '"');
```


At some point we should update the README.md to document the internal API for the `Dependency` type.


Ah, this is the reference issue you were talking about. Good catch!


For my information, what's the difference with = 0?


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=1d6cb9a44f76408eb7a10ca5a62a922a
 plone-5.0-python-2.7 [SUCCESS]


lz4 already supports some kind of multi-threaded'ism doesn't it? What is the particular goal of this project, compared to that? Just wondering, thanks!


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


@yellowled That would be the polyfill for progress, You might want to embed it differently, but I tested it in IE9 and it worked.

Missing now would be the activity indicator, but before I build a solution there I wanted to hear whether you want to use font awesome instead. Because that wouldn't work in IE9 either afair, but would need another fallback.

I'd otherwise probably include https://github.com/padolsey/sonic.js, which worked fine in IE. But am open for suggestions. gif fallback?


Please change formatting back. Supposes that clang format made a nice mess here.


@tarlog looks great!
I'd change ```forbidden`` to ```excluded```


also this dictionary formatting :smile:


This needs to be tested by everybody to make sure there is no problem


`dot` at the end.


I don't think this is a good change.  If a JVM exits due to an uncaught exception, it exits with exit code 0, which can lead to all sorts of problems when systems expect failing processes to exit with non-zero exit codes.


accepeted ? => accepted ?


Thanks for this pull request. We will review and merge shortly.


How does this assert not crash for an assignment pattern?


This isn't correct. There are only certain valid things you can put in here. Sensiolabs insight details them.


Yeah ...


```
The recommended notation for the most common licenses is (alphabetical):

Apache-2.0
BSD-2-Clause
BSD-3-Clause
BSD-4-Clause
GPL-2.0
GPL-2.0+
GPL-3.0
GPL-3.0+
LGPL-2.1
LGPL-2.1+
LGPL-3.0
LGPL-3.0+
MIT
```


`BSD-3-Clause` is what we need.


Please keep consistency in method naming across project: `withCompletion` should be replaced with `completion` just like in `UIView` animation method:
`+ (void)animateWithDuration:animations: completion:`


implemented nameLabel functionality


Please put it back


the pg flag name to flag should go into a generic helper in osd_types


don't we want to look at all flag names in teh vector<> and | them together?


nice try


:) @mnemonyc  thanks for advice man


That's fair, I'm just trying to figure out why the example projects are failing.


First change to the git hub project.


it seems more efficient o pass the Formatter* and ostringstream* to this function and have it dump directly instead of constructing the intermediate pg_map (which could be really big!)


I've created an issue for it too - #246


@Ben-Speakman looks like because of the L5 something changed in service providers. I need more time to investigate it.


Agreed, and stopping `System.exit` in a testing context is valid.  I just think we'll want a different technique.  Maybe installing a `SecurityManager`.


Shouldn't this be part of the src/platform/ hierarchy, and be guarded by QTOX_PLATFORM_EXT (possibly as well as the UNITY guard)?


Oops... left a debug statement in there... will take out in next commit.


Is this the commit where they changed the previous history and forced a push to Github, breaking the commit history and attempting to alter the previous history?

This commit does not retroactively change the source code of the previous versions to anything but GPL3.

AKA, any "Chinese" looking to "steal" your work are free to do so, because you cannot change the terms of your agreement at a later time.

Anyone who wants to use versions of this software prior to this version as GPL3 are welcome to do so.


A complete description of the projects' license, as well as license
+notices for 3rd party code incorporated into this project, is available in the repository

Should this be changed to "are" available?


I like this approach. May I use this same language for my Sass meet up? Changing the contact person to me, of course. :)


This is used by `float_of_string` so would be nice to have in the runtime if possible (and Magnus will be happy then :p)


Persist on the `temporaryContext` here before you persist the `mainContext` in `CDMDataManager` below. You should, by best practice definition, save your changes on your child context (in this case the `temporaryContext`) before saving the parent context. Saving the child context writes your changes to the parent context and then saving the parent context will write the changes to persistence.


IOException is the only declared exception here, no need to catch Exception


Is that really graings with two g-s?


here also clang-format made a bad work.


These should be: Added after 4.7.1 :)


General feedback: This looks great, may try to switch to active voice when possible to make the content a little more "personable"


Oh, I didn't know that :)


I'm pretty impressed that the positional argument parsing will actually work here and let you do

 ceph pg dump active
 ceph pg dump peering down
 ceph pg dump osd.123
 ceph pg dump 123

but I wonder if it might be simpler to just create a new command for this.  The 'pg dump' dumps everything related to PGMonitor (or a subset), which is osd, pool, and pg stats.. and we're interested in here is specific PGs and filtering those.   What about

 ceph pg ls [poolid] [osd.NNN] [state1 ...]

It's not that different but quicker to type and slightly cleaner usage.  The only part I don't really like is that you have to use a numeric pool id and not a pool name.  (If you could use pool-name we couldn't disambiguate it from a pg state name.)  But we're looking at PG ids anyway so it's probably okay... what do you think?


Nit:  Do you have "strip whitespace" turned on?

![screen shot 2015-01-29 at 11 01 48 am](https://cloud.githubusercontent.com/assets/3331678/5964112/5f67ec2c-a7a6-11e4-8f2d-a169acdb2a01.png)


Gives the following failure for me:
```
======================================================================
FAIL: test_linear (implicitfunction.NLPtests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/janderss/dev/casadi/test/python/implicitfunction.py", line 67, in test_linear
    self.checkfunction(solver,refsol,digits=10)
  File "/Users/janderss/dev/casadi/test/python/helpers.py", line 330, in checkfunction
    self.checkarray(trial.getOutput(i),solution.getOutput(i),"",digits=digits,failmessage=failmessage+": "+message)
  File "/Users/janderss/dev/casadi/test/python/helpers.py", line 211, in checkarray
    self.assertAlmostEqual(zt[i,j],zr[i,j],digits,LazyString('"In %s: %s evaluation error.\n %s <->\n %s\n [digits=%d] at elem(%d,%d): " % (name,failmessage,str(zt),str(zr), digits, i,j, )'))
  File "/Users/janderss/dev/casadi/test/python/helpers.py", line 164, in assertAlmostEqual
    unittest.TestCase.assertAlmostEqual(self,first/n,second/n,places=places,msg=msg + "  scaled by %d" %n)
AssertionError: In : : output(1) evaluation error.
 [0.696923076923076867, 0.430769230769230738] <->
 [0, 0]
 [digits=10] at elem(0,0):  6.9692307692307687e-01 <-> 0.0000000000000000e+00  scaled by 1

```


Maybe we can come up with more transaction-y sounding?


What happens if there are multiple touches?


Woops, migration fail :)


Nice work.  Looks good.


I though about it, but it seems like it would add more complexity, and doesn't really fit how the system tray is implemented.
We'd have a unity/ and a qt/ platform ? Or a unity/ platform and the rest inline ?
Or maybe dump the whole file under platform/generic/ ?

What precise hierarchy would you suggest ? I'm a bit worried that it'd make the code even harder to read.


86121710_WindowFrameDivider (macumber) - Win64-Windows-7-VisualStudio-12: OK (2177 of 2190 tests passed)

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


One other thing is that the osd filtering only filters on primary-ness and not whether the OSD stores the pg.  I think both will be useful.  What about

 ceph pg ls-by-primary osd.NNN [poolid] [state1 ...]
 ceph pg ls-by-osd osd.NNN [poolid] [state1 ...]
 ceph pg ls-by-pool poolidorname [state1 ...]
 ceph pg ls [poolid] [state1 ...]

?


I don't like the extra CSS reference and JS reference. However, I'd have to check out the polyfill myself to see if the JS exits in browsers that don't need it (like for example the details polyfill does).

I'm fine with using the PNG, but it's correct that it won't work in IE < 10, either, because those don't support CSS animation. However, I think it's perfectly okay to just drop the indicator for IE < 10. It is, after all, more or less a visual enhancement. IE 9 is a dying breed anyway, and IE 8 user will just have to make do with the polyfilled `<progress>` element. I would not want to add another JS asset or a gif just for that.


Thanks, @arowla!


missing copyright


Or, perhaps we build a set<pg_t> and look up each item in the pg_map hash.  That way we get a sorted result...


if this class was in a separate commit, it could have been cherry-picked into other branches and be used right away


missing copyright?


@thallgren is this a mistake? parse_string's 3rd argument is extra_data (a hash), so I think recurse_guard is being added to the hash rather than passed as the 4th argument.


Good line reduction, one comment though:

I personally like adding parentheses around the selector in the ternary statement, e.g. replacing:

```c
pos.lnum = lnum != MAXLNUM ? lnum : curwin->w_cursor.lnum;
```

with

```c
pos.lnum = (lnum != MAXLNUM) ? lnum : curwin->w_cursor.lnum;
```

I'm not a C genious, and my brain appreciates a bit of help in grouping sub-expressions.


@cirosantilli Could you explain this? This invocation adds 10+ seconds to *every* rspec run, even when the shell is already setup from previous invocations.


I dunno it could maybe use some content or something


oops good point. I think it's either there is any touch (true) or there are none (false). I need to add check that all touches are gone for ontouchend.


On my machine (amd64) the worst is smarteiffel doc; it takes around 2.5GB.


Thanks @dgutov, I've addressed both your concerns.

I've had this sitting around in my `.emacs` for a while now and I think it's working well enough to the point where others might enjoy it.  Would you be interested in adopting this package over at the `company-mode` organization eventually?

The final fix, for the colors defined in `pos-tip` that didn't take was this: `(x-gtk-use-system-tooltips nil)`.  On my linux box it would delegate to `gtk` and I would have to fiddle with a `gtkrc` file to get proper colors.  Not entirely sure why this is default.


bizarre..


I believe that's what I'm doing :S. Saving first to the child and then calling `persist` on the manager, which will save the parent. I rewrote it to make it clearer, though.


On Thu, Jan 29, 2015 at 3:24 PM, Keyan <notifications@github.com> wrote:
>
> Please remove VKL and REIZFramework. They are basically empty.

Done. Thanks for letting me know.
SVAKSHA ॥  http://about.me/svaksha  ॥


This is creating conflicts with another feature... so this shall be removed as well...


BTW this is uncharted territory for Composer. Everything should work perfectly in theory, but feel free to bug me if any problems come up.


I don't think that this is in the total diff, I was messing around with stuff at this point.


should not the SQLITE_OPEN_NOMUTEX flag be applied for older sqlite3 versions between 3006018 and 3007015  as it used to work before this change ?


looks good.


I was using kg because I was thinking about this as building up for eventual harvest totals. Harvest totals seemed like they'd tend to end up in kg range more than g range. I had 4.9kg of sweet potatoes last year. 4900g is the same number, it just feels more awkward (like reporting 176oz). But then I guess that's a display / number format thing.


Should this also change?  "Latest stable release: 0.2.0".


Sorry, but we are blocked currently in our project if we cannot have a new tag soon. Should i just fork your code?


[The documentation still mentions full join](http://www.querydsl.com/static/querydsl/3.6.0/reference/html/ch02.html#d0e273).
So could you update that as well?


Ship it!


Cool, thanks.


You'd usually capture the `error` return here to see why the creation failed (permission denied, etc).


Yes! Of course! It was taken from http://confcodeofconduct.com/ – they use "branded T-shirts" to identify organisers but I can't be sure I'll be wearing something Sass related in April :)


87219700_ReplaceSimSettingLineEdits (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2167 of 2190 tests passed)

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


`log.Fatalf()` will accomplish the same thing in one line, without the need for the additional `os.Exit()`.


Esta linea se implmento para agregar usuarios a la base de datos


> Sorry, but we are blocked currently in our project if we cannot have a new tag soon. Should i just fork your code?

Please go and spam the packagist guys until they respond to me. This is totally out of my hands.


Any specific reason why you opted for https://github.com/jonstipe/progress-polyfill instead of https://github.com/LeaVerou/HTML5-Progress-polyfill?


See the discussion at https://github.com/npm/npm/pull/2873#issuecomment-61255834 and https://github.com/npm/npm-registry-client/issues/82. It seems right that npm now follows the convention of `curl`, `wget`, and others by reading `PROXY`, `HTTPS_PROXY`, and `NO_PROXY` from the environment. It also turns out that `request` was moving in the direction of implementing this stuff directly, so it made sense to push _all_ of it into `request`, instead of some in npm and some in `request`. This feature is hard to get exactly right, because it seems like whatever we do, we end up breaking things for thousands of users behind enterprise firewalls.

I need to look at the proxy stuff anyway (because 4701a29bcda41bc14aa91f361dd0d576e24677d7 broke `--ca` / `--cafile` / `--strict-ssl`), so I'll see if I can figure out a new way to support command-line overriding of `--no-proxy`.


rax retry tests


no but you can create a tag in Git, you can bypass packagist altogether is what we're saying


Tested both for inv and hrm. Tests are working fine now(before inv didnt worked)


D’accord, j’essaierais de faire mieux la prochaine fois ! J’ai eu des points en moins pour le footer du coup ?
> Le 29 janv. 2015 à 19:24, Clément Oriol <notifications@github.com> a écrit :
>
> Il faut faire attention, et toujours vérifier. Gaffe.
> Sinon, de manière générale, ce que tu as fait est OK, mais dans l'ensemble je pense que ça manque un peu d'investissement. Il y a de bonnes idées visuelles, mais ça manque de finition, surtout pour un projet sur lequel on a passé aussi longtemps.
>
> Le résumé de ta note :
> Utilisation de git : 2,5/5
> Qualité du code : 3/5
> Professionnalisme, Rendu Visuel : 3/5
> Investissement perso : 2,5/5
> Total : 11/20
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/Manoush/projet-test-cesacom/commit/addf574d741044e0101e745765e6e769c3434a29#commitcomment-9503951>.
>


I prefer having limit = { region = AUS_729 } because I don´t want to give to Venice Gorizia at this point.


@wibrahim yes, that would be reasonable. Are you seeing that `SQLITE_OPEN_NOMUTEX` helps performance on a certain platform with ancient sqlite3?


You eliminated the option for Lombardy, but maybe is better.


We solve this problem temporarily by set `process.env.NO_PROXY='*'`


http://cdn0.sbnation.com/imported_assets/1590893/BmcF0wm_medium.gif

This entire conversation - I thought tokens was children. Oops.


super tiny nitpick: this is causing a warning in minall (Throw expression is not a minErr instance)


@kevdowney the fastway is probaly to fork this repo, create this tag on your repo and add https://github.com/GrahamCampbell/ as a repo
@GrahamCampbell true, this has also put my team in a bad bad real mess this afternoon - we will definitely get a Packagist proxy


@frowing This is what I mean:

```
NSManagedObjectContext *temporaryContext = [self temporaryContextWithParent:CDMDataManager.sharedManager.mainContext];
[sirenModel managedObjectInContext:temporaryContext error:error];
[temporaryContext save:error];
if (!error)
{
    [CDMDataManager.sharedManager persist:YES];
}
```

Notice you call save on the `temporaryContext` first and then you call `persist:` on the shared `CDMDataManager` instance. Temporary writes to parent (main in this case) then the `persist:` call will write to persistence.

Additionally: You probably want to check for errors by dereferencing it first (this is an NSError address). So:
```
if ( !*error ) {
    ....
}
```


Bonsoir Clément,
Oui je peux tout t'expliquer lundi en face, est-ce possible?

Margaux

Le 29 janvier 2015 18:55, Clément Oriol <notifications@github.com> a écrit :

> Bon, je suis embêté sur la correction de ton CV.
> Tu as utilisé un thème tout fait, ce qui, même si ce n'est pas une
> mauvaise idée, est vraiment hors-sujet par rapport à ce que je demandais.
> L'intérêt, c'est d'apprendre, peu à peu, à pouvoir réaliser quelque chose
> de cette qualité toute seule.
>
> En plus, comme tu as très mal utilisé git (seulement des commits à la
> fin), je ne peux même pas avoir exactement comment tu t'es débrouillée sur
> la modification du template et d'ou ça vient. De plus, ta note est censée
> aussi représenter ton travail du semestre en cours, et là je n'ai rien pour
> l'évaluer ..?
>
> Bref, ce serait bien qu'on en parle.
> Tu peux m'expliquer tout ça (honnêtement) stp ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MargauxLgs/CV/commit/c30dffcf4779d5e5826a2404ae8024348a62ff3c#commitcomment-9503419>
> .
>


awesome idea :)


87219700_ReplaceSimSettingLineEdits (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: Build Failed

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


```json
    "require-dev": {
		"graham-campbell/flysystem": "~1.0"
    },
    "repositories": [
		{
			"type": "vcs",
			"url": "https://github.com/yourname/Laravel-Flysystem"
		}
    ],
```


all tests pass:
success: http://build.servo.org/builders/linux1/builds/629
success: http://build.servo.org/builders/linux2/builds/628
success: http://build.servo.org/builders/android/builds/664
success: http://build.servo.org/builders/mac1/builds/845
success: http://build.servo.org/builders/mac2/builds/845


I'm not seeing it, I only see the following:

```
build/angular.js:4146: WARNING - Throw expression is not a minErr instance.
          throw err;
          ^

build/angular.js:9813: WARNING - Throw expression is not a minErr instance.
            throw e;
            ^

build/angular.js:11315: WARNING - Throw expression is not a minErr instance.
        throw e;
        ^

build/angular.js:14495: WARNING - Throw expression is not a minErr instance.
            throw e;
            ^
```


Received this update but Error remains the same. Still so after manual restart. I do not use premium.


> Would you be interested in adopting this package over at the company-mode organization eventually?

That's not out of the question, but I've got other stuff in my queue now, I think more important than spending time fiddling with tooltip behaviors. And that's bound to happen were I to make up maintenance.

> The final fix, for the colors defined in pos-tip that didn't take, was this

That's great. On my system, the behavior was clearly buggy: the colors were displayed fine the first time the tooltip was used, but after I `Alt-Tab` from the Emacs window and then return, it switched to the GTK colors.


Very nice ebay, many thanks for all your contributions!!!


It is ok to do that but in fact currently the id of the container is not used because we refer them always with name, but of course I prefer to have this piece of code so if we need in future it already works in this case.


:tada: :balloon: :+1: :rocket:


fast-forwarding master to auto = d026457f


:+1: Definitely run_once shouldn't be there.


@malavsky why you pass `$doctrine` here? We already have doctrine object from `__construct` .


I can't remember it's been so long, give it a try :)


True dat - I'm just being paranoid. :-p


I'm not aware of any such index page for ESL. You'll have to Google it.


Instance created


Please change the commit message to a better one and then reinitiate the pull request


There should be a comma and the end of this line


"A mass of chains and steel" sounds too much like the chain golem to me. Does Automatos look like a giant man?


Guys, I'm actually going to just tag the `v1.0.0` release now anyway. If packagist doesn't work, then that's up to them to fix. This should mean you don't need to fork this repo though. :)


Austria should start with all Lombardy expect Milan occupied, or Sardinia will win easily.


I did not notice any huge performance improvement, I am actually using sqlite 3.8 anyways. I was just was wondering about the logic before and after this change


Wrapping this now prevents fatal aborts of calling sub and logs the fault.


Lewd como sempre.


@ulferts if we want to let themes override these values then we should append `!default` to the variable declarations.


Cool. Done.


I see, so you're basically searching for a sub-string. I don't think this is a big enough worry and the test stays simple (i.e., doesn't require forking based on platform).


'pre-push'


arg no -- I fixed the typo on the remote machine but didn't fix this version.  I'll repush this to a new commit.


@myabc thanks for letting me know!


Wouldn't it be more simple and less memory intensive, to check in the relation's object instead of loading related objects. What i mean is if you have a Book and Authors and they are related by many-to-many then you have a cross-reference BookAuthor, which is a relation that contains BookId and AuthorID. And then when adding back reference load all Author->getBookAuthors() and check if Book is in that collection, instead of loading all Author->getBooks() like it is now on every add. You can image loading a 1000 Books on call to doAddAuthor


I'm going to delete this branch since I don't need to install Pundit (yet)

I'm going to use Devise's  `before_action :authenticate_user!` instead


Looks like someone accidentally deleted some native classes... I'll fix this ASAP.


@ulferts what was the reason for moving the `@import` here?


See 183ecfdc9f3bb76582f177dcb1b96649bab47458 instead.


@mihailik  Yes, you have an accurate explanation of the usage of the DocumentRegistry.

> If so, here's an extra question. As I have two separate LSHosts, one for tsc.js another for typescriptServices.js — should those hosts return the very same instance when asked for the same file path (say 'src/compiler/parser.ts')?

If by same "instance" you mean same instance of IScriptSnapshot, then yes. ScriptSnapshot is just to get text, and changes from a previous snapshots, and sharing that is perfectly fine.


:+1: :smile: Been waiting for this for a few years. Thanks guys!


Could you link to Tornado and capitalize it?


It helps pushmasters ~~to~~ conduct deployments by providing a centralized view of


86605678-CustomMeterCrash (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1589 of 1589 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)


push requests from engineers and any information they need from reviews, tests


Could you enumerate this fully or maybe scratch the etc?


86605678-CustomMeterCrash (JasonGlazer) - 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)


Committed the fix to change to map to Byte.class instead of String.class


Push Request - A branch that ~~is~~ can be included in a push


> pool-conduit 0.1.2.3, which was causing an error.

What error was it causing?


With the previous change, the LS will be doing resolution, so the host does not have to. this means that the host has no way of knowing the "complete" list of files that the LS considers the "context".

I debated just exposing a list of file names, but we do expose an API to get SoruceFile, so the program did not feel too much. it also allows for extending the LS by adding new LS features that are not covered by the base LS, all you need is an updated typeChecker, and you can get that from the program.

One thing to note, is that a program is immutable. once there is a change, an old Program instance is obsolete. you will need to requrey for the new Program instance.


Journal usually comes last but thats a mega-nit


I just got Automatos in a game! Unfortunately, he's already malfunctioning. I didn't even hit him and he's playing the easter egg.


I'll try to understand what it actually does....


I personally wasn't doing any ng-animate stuff, but we were debugging it. I'll definitely check those out though. Bourbon has come up, but for whatever reason the design team is doing it by hand.


I'm not going to cry about whitespace changes being mixed in here....its just as easy to review as without.


Hm, I'll change it to Pushmanager has been integrated with:

and then take out the etc. We technically support any via url linking, but that's not easy to explain this this kind of list.


Verify and confirm verification sounds redundant. Could you rephrase?


pushmanater -> pushmaster (?)


That should just be verify the branch in the staging area.


Asking for trouble on Windows


Oops! He obviously needs a fourth pass. ;-)


Also, if you hit him hard after he's out of actions, you get a runtime problem.


@timowest Sorry to bring such an old commit to life, but since `PathMetadata ` is effectively not generic anymore, isn't it better to remove the type parameter?
That way, all raw type and unchecked compiler warnings with respect to `PathMetadata` will be eliminated.


Why `beast::Stoppable`? This is a bit odd. The server is only known by its Stoppable interface?


After investigating I'm pretty sure releasedEvent was intended - so I'll close the issue.


"Pushmanager is two services" -> "Pushmanager is comprised of two services"

api -> API
tornado -> Tornado


MMMhhh, no se, eh, no me acaba de convencer...


@derekcicerone, I have removed getScriptIsOpen as it was not used. and undefined is an acceptable value to getScriptVersion().
The way I have it here is to check on getScriptSnapshot() value, if undefined assume the file is missing, if exists call getScriptVersion().


False.


Copypasta


<head>
 HELLO
</Head>


The windows version could be executed unconditionally, but not the unix one. vim_tempname()  would return a different path at every call.


That's right. The temporary variable *librados_path* should be removed as well.


Also, where did this issue come up? I feel like tokens that are whitespace is a bug from a tokenizer...


How does this work if someone is not in the lobby, but dead and stays in the respawn menu just to farm money?


I think it's from icu_tokenizer without length_filter.


Well, what about this line comment?


oh, so blank filter can be accomplished with length_filter...


which ones do you think may have been better unfixed?


@jrieken I hear you. I think once async/await support is in it is going to be much easier to add that. the main problem really is that nothing is asynchronous now, and i would expect adding async resolution will need to turn other things into async as well.


jad4d
fevfrdew
fe
c
fcrd


Thanks, that clears it up for me.


Oh wait, I'm misunderstanding things now. I though it was meant as a pipe name generating function, to be used every time a new pipe was needed. That's what I get for not reading the comments properly. Disregard me.

By the way, I think this:

```c
static char servername[ADDRESS_MAX_SIZE] = "\0";
```

wants to be:

```c
static char servername[ADDRESS_MAX_SIZE] = "";
```


Evil browser detection. ;)

Should be switched to feature detection.


This looks ok I guess


The current contents of src/platform/ is timer.h and various timer_{platform}.cpp files.

In this merge, you have systray.cpp, systray.h, and systray_private.h. In my opinion, the latter filename is very confusing, and doesn't in any way indicate what its contents are, especially in relation to the other two files.

It would be much clearer I think to have a src/platform/systray.h, which the rest of qTox uses, and then a systray_unity.cpp implementation and a systray_qt.cpp implementation (which are chosen between at compile time, same as timer_*.cpp). The systray_private.h and systray.cpp files are littered with #ifdef ENABLE_UNITY_BACKEND conditionals, whereas the whole point of the platform hierarchy is to just have one conditional for each implementation, whereas currently there's one conditional for each function with as many branches as implementations. Adding a third implementation would be nearly impossible, whereas with the platform design it's just adding a third file.

So I would argue that it would be significantly cleaner code :) (also no need for a union either...)

It's of course possible that I completely misunderstand what's going on here, but the code *looks* like a spaghetti of choosing two different implementations. Is the unity version interacting with the Qt version in some way?

Also, fuck Canonical lol :P


Have to be in the header?

Just asking..


This link appears to be broken.


An error message in [Graphics], really ? We have [Error] for that... :/


Thanks for the contribution.

It's always nice to see a pull request together with an issue report!

Unfortunately the enhancement request (i769) has been denied (see the issue page for more information on that).

Because we love pull requests, for future reference, we'd have needed a few more things before we could have accepted it:

* Update the tests (or write a few new ones for new features).
* Both an eclipse and a javac implementation (this pull request only includes an update to the eclipse impl).
* Review the documentation, particularly the relevant 'features' page, or write a new one. The website is in the github repo under /website.
* Add your name to the /AUTHORS file, which also serves as indication that you accept lombok's open source license.
* (optional) Add a line to the changelog at /doc/changelog.md


This is great .. thanks!  This port really makes things work well since it makes good use of each connection.  I'll start putting it to the test tomorrow...


@thongsav-usgs , you were right! Thanks for catching that.


rebasing went surprisingly smoothly after I squashed the whole branch into one commit.


86605678-CustomMeterCrash (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (1595 of 1595 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)


Здесь нет рекурсии, просто **bAllowToComment** переопределяется и имеет новый смысл в шаблоне *comments/comment.tree.tpl* нежели тот который у нёё при передачи из экшена

новое значение
**bAllowToComment=** - определяет, выводить надпись *"Автор статьи запретил добавлять комментарии"* или нет

передаваемое из экшена
**$bAllowToComment** - Переменная, которая хранит *true* если пользователю разрешено комментировать топик и *false* - в противном случае (вне зависимости от того, разрешил это автор топика  или нет, т.е. его потенциальное право на комментирование топиков в блоге).

**$oTopic->getForbidComment()** - разрешил или нет автор комментировать именно этот топик

**E::IsUser()** - Авторизован пользователь или нет

Таким образом получим что надпись *"Автор статьи запретил добавлять комментарии"* НЕ выводится если:
*(E::IsUser() && !$oTopic->getForbidComment() && $bAllowToComment)* - пользователь авторизован, ему разрешено комментировать топик и автор НЕ запрещал это делать

или

Пользователь НЕ авторизован и автор НЕ запрещал комментирование. В таком случае сама форма не выведется а выведется надпись "Автоизуйтесь если хотите оставить комментарий"


Since config is an object, this will actually change the `config.serve_static_assets` value.  I don't know if we want to be so obtrusive as to change a non `config.font_assets` configuration.


This can not be changed since rosdistro is not available in all cases when this module is being imported.


Yeah, I already noticed I forgot a check there.


err yeah, this should say in `pre-push`, I'll update


Yepp, it would.


Invasive, not obtrusive...


This should be `{{ edx_notes_api_home }}/venvs/{{ edx_notes_api_service_name }}/bin/python`


Oh ok, well that's pretty strange. Normally I would put that in a different module and conditionally import the whole module. But I can just revert this one.


Hm, it's a graphics related error though .. but yeah, I even forgot we had [Error] :P


86605678-CustomMeterCrash (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1595 of 1595 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)


Back seems unused


= 0 means pure virtual, which means the function has to be overridden in subclasses. I changed it to have a default implementation here and to use that, as the modes except game, editor and console just had the functions copy-pasted anyway.


That makes sense. It doesn't seem that `config.serve_static_assets` is available at this point. I just tried to start passenger and got this message

```
undefined method `serve_static_assets' for #<Rails::Railtie::Configuration:0x007f54c90efa18> (NoMethodError)
```


I've applied the correct fix over on my repo btw.

https://github.com/GrahamCampbell/Laravel-Flysystem/commit/d405468c5ffe2c214c741ec77f92e62facd0ae15


@jcfr I'm not sure what you mean by "will not impact the fuzzy compare".


1.0.0 is tagged on my repo now.


This class is not referenced anywhere else, so the import could be removed.


Each clade emits observations of 0 or 1 at particular points in time. These are dated virus isolates that either belong to the clade or don't belong to the clade. If you look at a clade that eventually fixes, it will look something like:

0,0,0,0,0,0,0,1,0,1,0,1,0,1,1,1,0,1,0,1,1,1,1,1,1,1,1,1,1,1,1

as it goes from 0% to 100% frequency in the population. This SMC method is meant to estimate the smoothed trajectory through time.


Oh yeah, flipping the comparison direction for the second case makes a lot of sense to reuse the same logic using a function for it.


almost there ;) should be `rom/sql/rake_task` otherwise it looks like part of rom "core" and it's sql-specific


Brilliant


the rest of the file uses spaces before and after the "|" sign, this breaks consistency


Okay. I think we should keep it the way it is then: The assumption is that if you care about performance you'll be using recent sqlite3.


While reading the individual arguments is easier it uses a lot of vertical space. Not sure if that is a good trade off.


https://github.com/rails/rails/commit/3c10043bc27

Applying this change may affected to before v4.2.0, I think.
(But test with ar4.1 or ar4.0 works.)
I think need to cut out v42 over version class.


It's cool man. Just check your translations before making pull requests.


Greetings! I am a little confused regarding this change to browserSync. My dev apache service runs within a Docker on my local system. Having the current port set as :3000 is going to cause some issues...

I'll explore [browsersync.io](http://www.browsersync.io/) but I mostly wanted to provide feedback.


saw approval from mbrubeck
at https://github.com/pcwalton/servo/commit/111b5116582d553e454765386f715e46becfc267


merging pcwalton/servo/text-align-justify = 111b5116 into auto


pcwalton/servo/text-align-justify = 111b5116 merged ok, testing candidate = 2b8f8839


may become 1.6.2 I mean


*sigh*


app-engine-java is a part of pkg-java.


Nice file 10/10


With updates LGTM


Using Markov Chains


It doesn't seems so:
```
| Not Installed | gcloud app Java Extensions                    | app-engine-java            |  89.0 MB |
```

After:
```
gcloud components update pkg-java
```


fcd


1.添加标签
2.添加白点.
3.跳转按钮的位置.


This has been fixed in a different way. The branch can go.


There is a bug here related to splitting on the endline character. I believe there is a different function that will split on whitespace as the default.


good job :)


Thanks Joe, it worked perfectly. I also have another quick question, if you don’t mind. I tested the JTAGulator out on a ZTE phone and found some obvious test points. When I execute the ID code and bypass scan, it gives me about 6 different results of what the pinouts could be. How do you figure out which one is the true pinout config?

From: Joe Grand [mailto:notifications@github.com]
Sent: Monday, January 26, 2015 6:35 PM
To: grandideastudio/jtagulator
Cc: Krypt1c
Subject: Re: [jtagulator] Updated for FW version 1.2.1 (a023bd5)

Here's a video about how to load new firmware onto the device: http://youtu.be/xlXwy-weG1M

—
Reply to this email directly or view it on GitHub <https://github.com/grandideastudio/jtagulator/commit/a023bd52f3554f78566bd8b4f8f10aaeab8cac87#commitcomment-9451530> .Image removed by sender.

  _____

No virus found in this message.
Checked by AVG - www.avg.com
Version: 2015.0.5645 / Virus Database: 4273/8997 - Release Date: 01/25/15


seems to allow `TrueClass` which is not legit, should either be false or a valid parameter to expire_metadata, e.g. `\d+[hmd]` or someting, yea?


op is working as designed.  We are not using it correctly.

Basically the right fix is to introduce 'fst' and 'snd' routines in Maple which evaluate as expected on Pair, and otherwise return unevaluated.  Other code will take care of pushing these into if_.


Russell,

Thanks for the resources! I'll definitely look into SVG as a possible
solution.

On 4:15pm, Sat, Jan 24, 2015 rbvea <notifications@github.com> wrote:

> Looks like you're using font-awesome! May I suggest using svg icons?
>
> Some resources:
>
>    - http://css-tricks.com/svg-sprites-use-better-icon-fonts/
>    - http://caniuse.com/#feat=svg
>    - http://evil-icons.io/
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/evantbyrne/evanbyrne-2015/commit/f5359d6977cb71b39db9b3e3c6fd8789430d0c0a#commitcomment-9429563>
> .
>


Probably. I was following the existing pattern from the :priority attr. Would it be better to be a String only with a validator, and default to nil?


kind of: string/validator/default nil sounds :+1: to me


Should this be left in?


its not intuitive. its misleading. I am closing the pull request. but remove brackets.


Do you want to make another v1.0.0-RC2 tag to include this fix? I am unable to get the latest version via composer updates no matter what I specify


Hah!  so true.  We should probably will add a link to the new package.


"jQuery-fu" might be confusing for newbs. But perhaps I'm wrong, since most JavaScript people know about jQuery. (Thus, no edit required.)


Good catch!


You run your local dev server in docker. I run my local dev server in vagrant. There are probably poor souls running their local dev servers in MAMP or something. We all access our local servers by typing `config.devUrl` into our browsers. My `config.devUrl` for my vagrant box is "example.dev". Browsersync [proxies](https://en.wikipedia.org/wiki/Proxy_server) your local server and makes it available at http://localhost:3000 and injects styles and such as changes are made.


The dplyr internal are so, so much easier to read. Pipelining is such a good thing.


I agree with you!


can you print out `path.Join("/api/v0/hub", tc.path)` for each case and see what's the output, 404 is most likely to happen because path is not matched.


sdfasfdsf

From: Allan [mailto:notifications@github.com]
Sent: Thursday, January 29, 2015 10:16
To: Allan-Mao/GitTestProject
Cc: Mao, Allan
Subject: Re: [GitTestProject] Create forknewfile.txt (7a578fc)

test1

—
Reply to this email directly or view it on GitHub<https://github.com/Allan-Mao/GitTestProject/commit/7a578fc8315ff0f43d3ba8ea31a6cd72a5470dc7#commitcomment-9491000>.


In the following section (line 525) you put the `+ New` link in code formatting. But as I see it, it should be in quotation marks ("+ New") like the other references you make to UI text.


This is the information I was looking for. I'll play with the conf and continue my readings.


This will only work for modules, I think. One needs to read `inspect.getsourcefile()` and check if any of the first two lines match the regex.


if any one help me i will pay for that :)


Hrm, how does this happen? A comment would be nice.


fix #17


"databases, tables, *and servers*"


I removed the code tags around `Future` to be consistent with your other uses of it. I also noticed it's one of the only functions that's capitalized. I'm not a developer, so maybe this will make sense to JS and Meteor vets, but I wasn't sure if you were being consistent with what functions and thingers you were capitalizing. (For example, you capitalized Future pretty consistently, but never "underscore".


제가 다시 클린에서 시작해보고 말씀드릴께요 ^^


I changed all instances of "login" and "setup" that were verbs to their proper forms: "log in" and "set up". Login and setup are technically only supposed to be used as nouns.


In the squashed version this shouldn't be necessary. Just nuke the files that are white-outed.


@culotso Check out my latest commits on the branch lz4-new, and see if this setup runs any better. This setup should be identical to the one in the study I linked to before. Still waiting to get my G2 fixed, so I can't test it, but this should theoretically run better.


> you replaced magic numbers with more magic numbers, and this function seems to have lost its genericity.
That's true. But it makes this class convenient so I did. Otherwise we have to calculate hexagon proportion(itemSize, spacing and gap) by ourselves.
May be I should've kept gap property public to allow customization.


I think we will also need to talk about technical and non-technical use of numbers. (5 versus five.) On themeteorchef.com, the What We'll Be Learning section before the article starts uses numerals "parts 1 of 2", but at the end of the article you refer to "part two". Another thing we should consider for the style guide.


Oh shoot, duh. More logic needed to, that source file could be a zipfile or something right?


I added a link to the Spacebars page on GitHub. I apologize if I was out of line here.


Bold super-emphasis here. Is there any other way you will be using boldface within recipes? Just wondering about that, re: style guide.


I think that conditional will always be true


Steve,

  I decided to be lazy and tap into your expertise as I await your comments on my pull request.

  If you read my initial pull request comment you will understand what I referring to.

  Thanks for the encouragement!!

Eric

Sent from my iPhone

> On Jan 28, 2015, at 8:51 PM, Steve Iannopollo <notifications@github.com> wrote:
>
> Nice refactoring!
>
> —
> Reply to this email directly or view it on GitHub.
>


Note that UUIDs generated with the code before this commit are broken and won't compare as equal to other UUIDs with the same string.


It's `inspect.findsource`, we want here for getting the full filename


that


some tests failed:
failure: http://build.servo.org/builders/mac1/builds/838


Target-specific questions are better suited to ask via http://www.grandideastudio.com/contact/ <http://www.grandideastudio.com/contact/>

On some occasions, you'll receive multiple possible pinouts. This usually happens in a case where the JTAGulator is connected to some other type of (unsupported) interface or maybe not all of the necessary JTAG pins are connected or maybe there are multiple TDO "breakouts" along the JTAG chain.

If you think you're on all of the right pins, you'll need to manually verify by using the Get Device ID and Test BYPASS commands for each given pin permutation. You can search resulting IDCODEs online (look inside BSDL files or it might be mentioned on a forum if it's a common/popular chip/architecture) to see if they are actually valid. If you don't want to do that, you could just hook up your JTAG hardware and try each of the resulting pinouts to see if any of them work.

Remember, the JTAGulator is a tool to -assist- in finding on-chip debugging interfaces. It will help you slog through the possible thousands (or more) permutations and return what it thinks are possible valid pinouts, but you'll usually still have to supplement that with some manual work after the fact.

Joe

> On Jan 28, 2015, at 6:07 PM, Krypt1c <notifications@github.com> wrote:
>
> Thanks Joe, it worked perfectly. I also have another quick question, if you don’t mind. I tested the JTAGulator out on a ZTE phone and found some obvious test points. When I execute the ID code and bypass scan, it gives me about 6 different results of what the pinouts could be. How do you figure out which one is the true pinout config?
>
>
>
> From: Joe Grand [mailto:notifications@github.com]
> Sent: Monday, January 26, 2015 6:35 PM
> To: grandideastudio/jtagulator
> Cc: Krypt1c
> Subject: Re: [jtagulator] Updated for FW version 1.2.1 (a023bd5)
>
>
>
> Here's a video about how to load new firmware onto the device: http://youtu.be/xlXwy-weG1M
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/grandideastudio/jtagulator/commit/a023bd52f3554f78566bd8b4f8f10aaeab8cac87#commitcomment-9451530> .Image removed by sender.
>
> _____
>
> No virus found in this message.
> Checked by AVG - www.avg.com
> Version: 2015.0.5645 / Virus Database: 4273/8997 - Release Date: 01/25/15
> —
> Reply to this email directly or view it on GitHub <https://github.com/grandideastudio/jtagulator/commit/a023bd52f3554f78566bd8b4f8f10aaeab8cac87#commitcomment-9490935>.
>


Where did you get your .gitignore file list?


yes, change to refptr.


got it.


まだ動作などはちゃんとは見てないのですが、それでも単純に見てforブロックの先頭で毎回
を
入れているので、continueするときは必ず
を入れないとブロック先頭の
との対応が合わな
くなっちゃうように見えるのですが、そんなことは無いですか?


nice


lgtm :+1:


Better name it DeadlockError then?


It's needed by WebCLMemoryUtil.


Not related to the immediate mode patch. Was working on pushing video update via ast_control_vidupdate as well.


nvm, added a test that check that all component are installed.


:D


It's also needed by WebCLMemoryUtil.


update_sqlite_coverage (mbadams5) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


yes, will remove it. thanks


呵呵呵呵


apply korean patches. (fonts, addons, etc.) I forked xbmc to learn.
Thank you for your commits. I didn't success to build before your commits.


test time for comment


오홍~


entries/jQuery.get.xml，line 7：The type option will automatically be set to GET


I'm reconsidering deprecating this--that's a lot of calls that would require changing. And we would still have a lot of `jsonBodyGenerator()` calls that wouldn't match.

Instead we could just change this to `setBodyGenerator(BodySource)`.


Final Patch to correct the reload issue. Currently being tested by Jean-Francois.


update_sqlite_coverage (mbadams5) - x86_64-MacOS-10.9-clang: OK (1565 of 1604 tests passed)

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


_**What the heck is this?**_


Typo ?!!!


I think **async/await** pattern might be useful here


hey Git


you better would have merged 'modernize' into 'master', than 'master' into modernize, i think.
so, what is the branch we should pull from to work with our local clones?
which branch can/should we use NOW on the web server?


`new` needs abstraction to `Robot` class


Looks good to me.
Can you do separate commits for each of the other files that need to be changed?

Thanks


Thanks.  I was afraid of that, too.


Ok I added it to the library. Now when I attempt to upload it I get:

Arduino: 1.0.6 (Windows 7), Board: "Arduino Duemilanove w/ ATmega328"
rainbow.ino: In function 'void __vector_9()':
rainbow:46: error: 'class Rainbowduino' has no member named 'draw'
rainbow.ino: In function 'void reset()':
rainbow:58: error: 'class Rainbowduino' has no member named 'reset'
rainbow.ino: In function 'void loop()':
rainbow:71: error: 'class Rainbowduino' has no member named 'next_frame'
rainbow.ino: In function 'void check_serial()':
rainbow:93: error: 'class Rainbowduino' has no member named
'get_current_frame_nr'
rainbow:97: error: 'class Rainbowduino' has no member named
'get_current_frame_nr'
rainbow:101: error: 'class Rainbowduino' has no member named
'get_current_frame_nr'
rainbow:104: error: 'class Rainbowduino' has no member named
'set_current_frame_nr'
rainbow:105: error: 'class Rainbowduino' has no member named
'get_current_frame_nr'
rainbow:108: error: 'class Rainbowduino' has no member named
'get_current_frame_nr'
rainbow:123: error: 'NUM_ROWS' was not declared in this scope
rainbow:125: error: 'class Rainbowduino' has no member named 'set_frame_row'
rainbow:132: error: 'class Rainbowduino' has no member named 'get_frame_row'
rainbow:136: error: 'class Rainbowduino' has no member named
'get_num_frames'
rainbow:140: error: 'class Rainbowduino' has no member named
'get_num_frames'
rainbow:144: error: 'class Rainbowduino' has no member named
'get_num_frames'
rainbow.ino: In function 'void save_to_eeprom(word)':
rainbow:190: error: 'class Rainbowduino' has no member named
'get_num_frames'
rainbow:193: error: 'NUM_ROWS' was not declared in this scope
rainbow:194: error: 'class Rainbowduino' has no member named 'get_frame_row'
rainbow.ino: In function 'void load_from_eeprom(word)':
rainbow:203: error: 'NUM_ROWS' was not declared in this scope
rainbow:204: error: 'class Rainbowduino' has no member named 'set_frame_row'

On Wed, Jan 28, 2015 at 8:11 AM, Alex Nas <alex.j.nas@gmail.com> wrote:

> Ah thanks!
> On Jan 28, 2015 7:18 AM, "Tobias Bielohlawek" <notifications@github.com>
> wrote:
>
>> its in a submodule: https://github.com/rngtng/rainbow.ino pls check this
>> out too..
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/rngtng/rainbowduino/commit/e6ea49ecbf2c47e226fe845184182fbb598b6c54#commitcomment-9477596>
>> .
>>
>


Looks good, nice job. Any issues along the way?


Here is my progress, but for some reason we passing this cmd list to cmd.run, it doesn't work.

https://github.com/russellballestrini/salt/commit/bc517c0e1c73a8f6f19fae8da2aa6255b7b5090c


What did you use to write this? Also I will give you credit, but your repo name should follow the same format as stated in the homework. Any issues while doing the homework?


It is "canola" not "canols"


remember to `.enable` the pid on init (just do it in the constructor, it's close enough to init that it doesn't matter)


What was the reason for adding dataset and experiment here? Nothing seems to have been updated:

    INFO:encoded.commands.update_keys_links:Collection /datasets: Updated 0 of 186 (errors 0)
    INFO:encoded.commands.update_keys_links:Collection /experiments: Updated 0 of 7435 (errors 0)


You're right of course... I hope you'll get to it before I do...


Looks good. Any issues along the way?


Danny, why did this appear to come from Praj? Really confused.

In any case, you should remove the whole section on "Deep Linking”.

David

> On Jan 28, 2015, at 6:07 PM, Prajwal Tuladhar <notifications@github.com> wrote:
>
>
>
> —
> Reply to this email directly or view it on GitHub.
>


建議修正成「因為怕 CEO 不知道什麼時候會來 26 樓，然後很不巧地就經過會議室」。
1. 據說老闆的辦公室通常會設在最高樓層，而如果是大型共用會議室，會在老闆辦公室的下一層樓，如此老闆開會只需最後一分鐘再輕鬆「下」樓
2. “walk by” 應該是「走過、經過」的意思


I think **async await** should be helpful here. Check out this [async await gist] (https://gist.github.com/dac9d06f76964411a763.git)


Permalink for comments on what to fix:

https://www.reddit.com/r/netsec/comments/2ts3qm/get_local_and_public_ip_addresses_in_javascript/co23btd

https://news.ycombinator.com/item?id=8951145

Thanks all!


I did a `git pull` inside the submodule and committed it from the main starcheat repo.


I don't see where you pushed to a new branch, did you make these changes right into master? You need to create a develop branch to make all of your changes in, then open pull requests to merge your changes into master as the instructions from Lesson 2 describe.


I'd prefer to see an actual option here--something more like:

```js
var freeTime = confirm("Do you have some spare time?");
if (freeTime) {
//...
```


LOL oh my god dude...  First of all, 0% of that code is similar to mine.  Second, this is literally how binary.mk works.  You know.. as per Google's use of it too?  Jesus christ I cannot believe for one second you think you're that special.  Or did they get that idea from you lol.  As far as acting like I have a shitty attitude towards others i'm pretty sure you're the one treating someone with ~4 years of experience in Android like they can't have ideas without you or apparently even read source code released from google.  You had an idea?  Congratulations, I'm sure you're very proud of yourself, but there's a reason I (we actually) haven't looked at your source for cherry-picks in years.

BTW, does kang mean putting your name on a toolchain and acting like a single amateur could put it together better than an entire team of legitimate professionals?  If so i think i got the gist.


For example if you have an update form and a list of documents, and you can click on a document to change the `doc` for the form reactively (like http://autoform.meteor.com/updateaf). Before this change, the value from the previous doc would remain selected. So this fixes that bug.


bro @hariawan module `countries-in-the-world` di update bro. soalnya di method `getCountryByCode` itu returnnya bukan langsung `country name` tapi object `country`


you're correct, it was! :)


rax retry tests


I'm just setting about building the latest BlissPop, so this suits me fine. What's up with the 'arm-eabi-4.10-linaro' though? Why is it that no one is updating that 4.10, but 4.9's kept up to date by everyone and their mom? Curious.


Is it worth reusing value.abs() call ?


Dat tracking


Was that GitHub's doing or ember cli?


86605678-CustomMeterCrash (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (1595 of 1595 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)


Same as for wrapping function arguments. While reading the individual arguments is easier it uses a lot of vertical space. Not sure if that is a good trade off.


That would indeed be good. Should do this when its "finished" and nothing moves around anymore.


Well, I just copied the licence file from cloudpipe, so neither I suppose!


This is all static data. You should just have the method return the the constant.


a separate issue would be good, please -- it's not a small function to add in


@skypanther  this is a good commit. it would be good if you could work with Tony L. and submit a similar PR to appc-cli/appc-install.


You're right, it does look like spaghetti now that I look at it again.

But the current timer in platform/ works differently. There's one declaration, and one of 3 possible definition is picked at compile time via the #ifdefs.

For the systray, the decision is made at runtime. If we're on a Unity desktop, use the Unity appindicator, otherwise use the Qt QSystemTrayIcon. We have to dispatch at runtime.

The #ifdef mess can't really be removed right now, otherwise you'd have to pull in libappindicator, libgtk2.0 and libdbussomething even if you don't care about Unity (Windows, Mac, *BSD, GNOME, KDE, etc users).

There could be a way to hide the platform specific code in platform/, but there'd still be runtime dispatch, it would just be an extra layer of abstraction. It would essentially move each branch of each if/else if/else into it's own function in platform/
I guess that could work, but I'm not sure if it's more readable

>Adding a third implementation would be nearly impossible, whereas with the platform design it's just adding a third file.

You'd have to add a new else if to every function you want to implement for that new backend. You have to implement those functions anyways, and the if/else are needed anyways because of runtime dispatch, so I can't really think of anything simpler.


Thanks. This fork will be deleted any second now.


@naucoin Nevermind, I just noticed that the precision was set on the `warnings` stream. You can discard my comment.

Looks good to integrate. :+1:


qDebug() adds the space for us


Seemed it was incompatible with the yesod version


I just declared the method "const" locally and it still compiled, from this I derive that this method does not affect the "this" object and hence it should be safe to not call it.


Instead of calling `save` it's faster to call `.valid?` in this case, that way no IO has to occur.


We store an int in line 229, why is this a long later casted to an int? Maybe this would not work on 32 bits?


oy


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=ce9c48060ad641d0b8f916790422812d
 plone-5.0-python-2.7 [SUCCESS]


sched_setscheduler is a syscall that allows an user changes a process schedule parameters like police and priority. This commit allow a modification on sched_reset_on_fork when nothing else changes.


It would be nice if you could check the whole patch for use of 4 space indent for logic blocks and 8 space indent or any other reasonable non 4 space indent for line breaks. Thanks


What is this comment supposed to mean?


<iframe width="560" height="315" src="//www.youtube.com/embed/7F2F1iGOw4s" frameborder="0" allowfullscreen></iframe>


We can simplify this, since `__self__` was added in 2.6. :smiley:


It's more difficult to test static stuff. Use the singleton pattern instead


Sorry, this piece snuck in here. This is for fixing the build on NetBSD by defining the network API to use.


@smfn3321 Do you think its okay to remove that? It makes some things easier because we don't have this damn unpicklable stdout thing.


Just a question, would this interfere somehow with multiple nvim's running alongside each other?


Looks good.


This method is nested pretty deeply. Additionally I would make this method just know how to send a single email, then put the logic for sending bulk emails elsewhere.


make it final and assign it from a constructor. Will remove alot of code.


Yes, I knew about pure virtual functions. I was talking about "override". But I've asked google and seen that it allows to explicitly tell we are overriding a function. If it was me, I would allow overriding only when explicitly written (ie when using override keyword like it is done with c#). But I've seen it is a c++11 keyword. I will have a try with MSVS to check if all we have lately added still compiles...
Thanks for the info anyway.


@vncatalin ok, ne mai gandim dupa ce terminam aici.

Poate la tine apare logo-ul, dar ar trebui sa functioneze exact la fel cum era inainte ( acelasi nume etc), ca sa mearga la update.

Daca vrei sa testezi iei ultima vers de aici : https://github.com/Codeinwp/bookrev/releases, pui logo in footer, faci update la noile fisiere si vezi daca mai ramane :)

Ar trebui schimbat si text-domainul daca nu ai facut deja + regenerat fisierul .po


Executors.newScheduledThreadPool(1, new ThreadFactory() {
            public Thread newThread(Runnable r) {
                Thread t = new Thread(r, "gethostname-thread");
                t.setDaemon(true);
                return t;
            }
        });


I'm sure doing such a lookup is faster than the algorithm below, but it should do the same in less code. But that's just personal preference.

if (tile_split >= 0 && tile_split < 7)
  return 2^(6+tile_split);
else
  return 1024;


Nice cleanups


@micnic seems like the email you committed by isn't registered with github.


Is moving this call up from line 244 necessary?


scheduleWithFixedDelay


Thanks for the tip! Updated the code: https://github.com/discourse/discourse/pull/3150/files


Don't pair indention changes with normal commits


86605678-CustomMeterCrash (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (599 of 599 tests passed)

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


Not sure why the TI build failed when I only changed the README.


start the timer only if envHostname was not set. where did this method go?


I think we should apply tileset in MapGenerator class as any tileset can be applied to any algorithm. So this code will be shared between all algorithms, and should go to base class. So ApplyTileset () call should go after Generate () call


Warum zeigt github mir hier kein diff an?


Done (see https://github.com/eugeneia/snabbswitch/compare/virtualopensystems:straightline...straightline#diff-5eefe53b964d7d533db36a9635317575R81 )


Looks good.


Issue is fixed. It was to do with

    transform: scale(-1, 1);

on that icon. Not sure why it was added removing that fixed it.


Enable that a task go to the head of the run queue when sched_setscheduler dequeues its priority.


You're not using $wpdb->prepare() properly. Check the docs on it. IT requires the values to be passed as parameters.


I think this was intended
widgetSize = widgetSize.expandedTo(QApplication::globalStrut());
do we use a global strut?


=D geil!!


partially fixes #39


@JulianFernando . Tanto o setInteger, quanto o setParameter funcionam para consulta. Por que você achou necessário especificar o tipo de parâmetro no caso de um inteiro?


It still needs to be updated to support v1.1, but I think we'll want to add http://labs.data.gov/dashboard/datagov/csv_to_json back since several people were using it, whereas http://project-open-data.github.com/catalog-generator/ was just broken.

I know this is a small edit, but let's try to stick to PR's as much as possible.


See https://github.com/mirage/mirage-platform/issues/118


Thank U. Ha Ha Ha!!


Calling this function is only useful if you do something with the return value since it's `const`, cf. <http://qt-project.org/doc/qt-4.8/qsize.html#expandedTo>.


You probably did the right thing.  I don't personally like to initialize variables to 0 when declaring them, to fix "uninitialized variable used" warnings.  It could potentially add extra instructions, so I was trying to think of a way to adjust the functions themselves to make them see that `VECTOR Result` was in fact getting initialized, without explicitly initializing it in the declaration, but never got to a simple way to do it.  (One of my ideas was to make Result a global variable, but this too has certain perf. detachments and namespace collision vulnerability.)  So maybe what you did was best, as interpreters don't necessarily have performance as a priority anyway.

The other thing I was unsure of was turning off all the remaining nameless struct/union warnings.  Again, it's possible that may have been best.  It boils down to a difference of opinion on portability / whether or not it could be desirable in the distant future for anyone to work on Project64 on other operating systems without MSVC.  Technically speaking the use of struct bitfields within a union is undefined behavior in the C language anyway, but it is a lot more intuitive that you used that method to begin with so I think it's a mixed blessing.


> I would allow overriding only when explicitly written (ie when using override keyword like it is done with c#)
Indeed. The override keyword in C++ is mostly a safety guard against forgetting to declare something virtual, or mistyping etc.


No worries! It's a minor annoyance, and easily fixed.

On Thu, Jan 29, 2015 at 5:22 AM, Philip <notifications@github.com> wrote:

> sorry, tab spacing again, will fix.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/pchi17/ruby-exercises/commit/fbb29df949369ec2883441d0a5ae0fbc7fadecbd#commitcomment-9495592


There might be a future case where that symmetry is necessary, but for now it seems a bit extraneous.

On Wed, Jan 28, 2015 at 6:22 PM, Philip <notifications@github.com> wrote:

> Right, I don't need the second print_space(), I was only thinking of symmetry.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/pchi17/ruby-exercises/commit/d7c37ab28d62f26d94b683dea82a4f16595c8b1a#commitcomment-9489234


Sim setting fc in msim phones pls some one pls link me to the fix pls


did someone say 'skinny views'?


I know that Go automatically dereferences pointers as needed, but this line, where you're using the '*' and the previous test where you didn't looked funny to me.  What's considered best practice for Go programming in general, and do we want to enforce a specific style ("always", "never") for this?


I would recommend doing the same as the FAQ.  See https://github.com/SpongePowered/SpongeDocs/tree/master/source/images
https://github.com/SpongePowered/SpongeDocs/commit/edc630d8e1cb726b8f98c8d7ec7ecb3a06732e74


@nimaai returning a Hash was the simplest way to do it and it's not reused at all. feel free to improve.


LGTM


Ah, I thought they would be smarter and more restrictive ;) nevermind then.


Can you guarantee only this one will be special? And not every next one aswell?


Ótimo s´ø a parte do BSON e JSON que não eh beem assim, já que o BSON é uma forma interna do MongoDB trabalhar com o JSON, mas no caso apresentado os 2 são JSONs ;)


<h1> Good bye, world! </h1>

That's more fitting.


it's *(scanMeta.OldestIntentNanos), OldestIntentNanos is int*. I don't think you have much choice here.


I was gonna say... the SVG just contains the PNG elements of all the gradients inside it.


fixed :)


> The override keyword in C++ is mostly a safety guard against forgetting to declare something virtual, or mistyping etc.

ANd the most important, IMHO, is that if you change the base class (removing a virtual function), you will get errors telling you that it was overriden (in most of the cases, the overriden function would be useless).


ES6 shouldn't be default here.. We want to be similar to tsc behavior in which case the default is es3


YAY!!!!!!!!!


@codygman - Here's the error that the tests were spitting out prior to this change:

```
ERROR:root:cabal: Error: some packages failed to install:
bench-0.0.0 depends on pool-conduit-0.1.2.3 which failed to install.
pool-conduit-0.1.2.3 failed during the building phase. The exception was:
ExitFailure 1
```

Source: [an older Travis run that ran on master](https://travis-ci.org/TechEmpower/FrameworkBenchmarks/jobs/47840059)


86956490-SPMMagic (kbenne) - x86_64-MacOS-10.9-clang: OK (2168 of 2191 tests passed)

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


It is inhuman how big community is behind Stockfish, even if you have a good idea to create a new chess engine, if it not open source, then basically you have no chance to create a beast like this one.


Haha


87219700_ReplaceSimSettingLineEdits (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2182 of 2190 tests passed)

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


I would rather keep these modifiers for clarity.


I'm going to merge and test your changes soon, but it's been so long since I've done Pebble dev that I have to setup the SDK again, and I don't really want to use Cloud Pebble. Also, I'm sure you're already doing this, but make sure to thoroughly test your code before pushing to github and requesting pulls, it'll make it much more smooth.


Are you sure about that ? IIRC, some test were done and the correct framerate was achieved when the value was 2.0 and not 1.0 (and if we keep 1.0, we should remove the comment)


Sorry, I don't remember very well.

As mentioned at https://github.com/gitlabhq/gitlabhq/pull/7823, I think before this PR gitlab shell was always removed and recloned and I stopped that from happening, first checking if it is already present and avoiding the clone then, which is what takes the large amount of time.

I don't remember why there was this unless check before.

> Also isn't this rake task run as part of the one-time dev:setup task and shouldn't need to be run again, much less every time?

I think I didn't touch this. It was like that to ensure that the shell is always on the right version corresponding to gitlab.


There's no need to add SkillService as a dependency here.


I think this is an involved process, but the doc is clear!


I had the idea the other day we should review pass_gen.pl... I'll create an issue right now instead of writing things here.


This change could probably be done on the main branch, as to not muck up the layout-experiments branch too much.

This also doesn't really matter in the long run.


Can we make BlockAccessDummy static?


:+1: :heart_eyes:


Yea probably.


should be 0.7.1-rc1, yes?


Sollte das nicht eher `${...}` sein? Also das Dollar-Zeichen vor der geschweiften Klammer.


Hier ebenso.


This commit will only be available in the 1.4.7 update. There's an issue #24 where this is discussed.


Hilarious. Instead of saying that the value must be representable in the new type you chose a description that is machine dependent and might or might not imply that `-1i64 as i8` panics.


hm, you changed only one line, and don't added new variable 'hasMarkedText'? Where is full commit?


Reflaction in performance dependent parts should be cached.
Maybe general utility class?


i'm planning to refactor later, right now most of the c++ backend is mixed with the rust generator, its just a fast way to get it working, and trying to keep both backends following the same rules.


@jaredly is there a specific reason why this couldn't be in the account config with a UI?

Also, why are there default `appId` and `appSecret` values here?


Hier bitte keine Reporte schreiben, das müllt mir das Filesystem voll, wenn ich die Tests ausführe (und hilft dem Test nicht).


Matt can you tell me what is the goal of calling this wrap function instead of directly doing new BukkitPlayer?


Neither gives the right result for me, but using 2.0 made my fps lower even when not hitting the limit. I read on the ogre forums that timeSinceLastFrame isn't actually the time since the last frame, http://www.ogre3d.org/forums/viewtopic.php?t=32206&view=previous
so I don't know if we should be using it to limit the framerate at all.


:+1:  Sweet


86956490-SPMMagic (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2068 of 2191 tests passed)

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


Das ist ein ziemlich grober Test, und ich fürchte, der wird gelegentlich fehlschlagen, weil die Einheit den Kampf überlebt, oder irgendwas anderes schiefgeht. Zumal der Test auch nicht das testet, was Du willst - das die Skelett-Einheit angreift - sondern nur einen Nebeneffekt davon (dass ihr Ziel 0 Leute hat). Das ist die Sorte Test wo später jemand drauf guckt, und sich fragt, was der Autor eigentlich wollte, und ob wir den Test überhaupt brauchen. BEsonders wenn er fehlerhaft ist.

Gibt es keine Möglichkeit, das in einen einfacheren Test zu giessen? Also z.B. in C die plan_monsters() Funktion aufrufen, und hinterher den Befehl des Skelettes checken? Das halte ich für einen besseren Test.


@marcelb thanks for cutting the release. do you think we could do another? :-)  we added MongoDB 3.0 compatibility just after you cut the release. i'd love to get the 3.0 compatible version out since MongoDB 3.0 will be released very soon

https://github.com/devbliss/mongojack/commit/cdb4d61648fc2cfeae8df558fafc8cca2c3dd65e
https://github.com/devbliss/mongojack/commit/774780cd4bc4cb8ec8f3e1e6e27dff7daec32257


That's all we use from it externally.

The original `WSDoor` was simply Stoppable with nothing else on top of it!


I would have left the checklist here, it doesn't hurt, no?


"a peasant" klingt schöner als "one peasant", finde ich.


lol @ title


Holy hell, this is exciting!


This is great!


wanna change the naming early, rather than we being stuck with another horrible naming....


There's a missing underscore in the PyBytesFromString (should be PyBytes_FromString), but more importantly, the result of a logical OR between the two resulting pointers is of type int, and I'm pretty sure the result is either 0 or 1. The logical OR on C doesn't work like in Python.


Already created a PR #1325.


Can we use underscore_names for these methods instead of camelCase?


86956490-SPMMagic (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: Build Failed

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


nitpick: vielleicht peasant_growth_factor() in eine Variable ausserhalb der Schleife speichern, damit es in dieser Funktion nicht n-mal aufgerufen wird?


At this point, you are tying the auditing of certain actions to the actual implementations. This is not a separation of concerns. If I swap out the JDBC implementation, I lose auditing.

A better way may be to do this
https://github.com/cloudfoundry/uaa/blob/master/uaa/src/main/webapp/WEB-INF/spring/scim-endpoints.xml#L40-L57


I'm skeptical, because the purpose of the businesses method is to implement the 'adapter' duck type. I'm thinking that the filename should stay hidden inside it, especially, even the fact that there is a file, that it's csv, etc.


O


Good point about the side effects. Yeah, I'll remove that. FYI, it's within an `if` because the zip gem hasn't been updated since July 2014 :-( but the repo master has this new method for disabling warnings. So, I couldn't be sure if the version in use supports this.


Since commenting for readme.md is not appropriate place for talking about your question, I've moved this to the issue #29.


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=a15430b4d9e74efbaa72ca097b3a1506
 plone-5.0-python-2.7 [SUCCESS]


OK, I will change the colours then as it is model specific as well.


This patch adjust the behavior if a task boosted by Priority inheritance has its parameters changed by setscheduler() call.


if ```foundation``` is imported after ```settings```, the mixins employed in ```settings``` will not work as they are defined too late. As I wanted to use ```rem-calc``` I had to move the import statement.


The two `%i`s are inside a formatted block and need to be escaped as `%%i`. Really, this is bad code and it will keep breaking. See #882.


Diese Art Satzfragmente sind es, die mir hier Bauchschmerzen machen. Es ist generell besser, immer ganze Sätze zu übersetzen, weil solche Fragmente sich zu sehr auf die Grammatik verlassen.


an even power


@Narretz could you show me the command you're running to get the warning?


Did you add the other commands as asked?


Let's fail out here.


Also removed the space before the colon in all the dictionary mock-ups


Use either @CompileStatic or @TypeChecked


No, this isn't right. It's true that the first function call does indeed return a generic InputParameters object but it is an instance and can be held onto from that point on. You should be able to hand that instance to the parser to be filled in, or to the Action system to be filled in, but you should not need to create it again. If you are calling a copy constructor, you already have a live instance so why would you build another?

Now to the practical part... There is an issue with getting the InputParameters object off the stack and into the heap. If the end user is creating a local InputParameters object it may be necessary to copy construct it once in the heap, but even that could be fixed if we also wanted to change the return type of validParams...  We probably shouldn't go there just yet.


The same check for the number of faces? (if not an attack vector, then at least for symmetry)


I wonder who will be the first to override the downloaded stubs manually with access to each of the WireMocks :)


:+1: screw camping migrations


"there's a giant buffet of soggy wet dicks somewhere else you can go eat" https://www.youtube.com/watch?v=ogQJb2wXt9w&t=0m10s


At this point I would again fail.


if it's a constant then it should be public static


hmm.. I don't like this over current just because we can have a common place to parse fromKeyword and finish the ImportClause node.


if that's a constant then it should be public static


Link, if you're going to include one, should be to Amazon or GoodReads or something, not to a Wikipedia page *about* the book — unless the Wikipedia page itself contains the strategy you're using.

Also check the licensing in the book you're using, please, to make sure that we're allowed to include code from it. As I mentioned previously, Numerical Recipes does not allow this — and it's reasonable to expect that other textbooks might also not allow it.


No... it seems not.. sorry... :o/


lgtm


Can we merge the two checks?


Some adjusts on task priority when calls setscheduler .


@dylandechant https://www.namecheap.com/domains/registration/results.aspx?domain=bigthingspoppin.ceo


Perhaps worth updating https://github.com/jspm/jspm-cli/blob/e0077138bddacf31464d31e82d3786a60207ce01/lib/bundle.js#L156 to reflect whether sourcemap was fudged


Ooh, this whole function is actually in a subshell, so you could even drop this bit if you wanted (we subshell so we can `cd` without harm).


Yes, that looks like a mistake.


start auto fixing...


Wouldn't `return 1` be the same thing as this `false` line?


it will return all the records and error ErrSnapshotMismatch


No, moved to `experimental` and removed this test from `test-cmd.go` (we don't test experimental commands there).


It's for Euphoria's prebuilt bootanimation. No need to worry about it.


Did this marker change upstream?


No, first we have to check we have enough data to read numIndices and only after that we can check that we have enough data for those indices.


There is no local 'autoconfig.h', missing this file causes it to fail to build.


Good catch - It is really unintuitive and error-prone at this point that `End` points to the last element and not the element beyond.


Backslash still legal in *nix paths, perhaps too boisterous to ditch the OS check?


This entire change is due to Traceur altering its handling which broke the source maps on update. sourceRoot is now the folder of the source and the sources are always the direct names with Traceur regardless, so that these normalize to `path/to/file/@traceur...`.


I have plans to catch the user so we don't need to keep creating BukkitPlayer objects. I will explain in more detail in skype.


I didn't know - then we have a lot of other places to do this in jspm as well.


Good point, contributions welcome :)


This video wrapper should be likely deleted, because it is empty and there's no purpose to keep it.


I would personally use windsor m8 xcx


it uses cmoney, which resets to the starting value when the player actually spawns (playersetup)


I think this comparison was just wrong before, right?


Also, this isn't indented correctly. There should be one space before starting the `*` on all lines but the first `/*`.


also it has a max of 3 times multiplying after which it resets to 1x again for those that hide and go afk


:zap: nice


Thanks!!


Nice job finding titleize.


Oh yes indeed the issue seems to come from jaxb itself.
But there still an issue with namespace when generating two xsd inside the same directory


This is cool and a good idea. Can you think of any reason why these wouldn't be available to `path.zsh` scripts? I'm trying the following in my `java/path.zsh` script:

```
export PATH="$JAVA_HOME/bin:$PATH"
```
after defining `JAVA_HOME` in `.env-vars` but it never seems to be set at that point. Is this a timing issue perhaps?


Typo: masturbation


Since `End = P + pBuffer.size() - 1;`, why would an access at `P[ofs]` crash if `P + ofs == End`?


Looks great, dehacked


r+


Great.  A more typical Ruby style would skip the curly braces – `where(published: true)` – but this is valid, too.


86956490-SPMMagic (kbenne) - i386-Windows-7-VisualStudio-12: OK (2175 of 2191 tests passed)

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


shouldn't this be groestlcoinj?


A super minor thing, but one that will help you later, is to make sure all your left edges line up.  All the indents should be multiples of two spaces.


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


aha


Thanks!


Corrects the early console kernel messages (allow the kernel to print messages very early in the bootup process) on Spark architecture.


эффективнее использовать directBuffer


:thumbsup: right on!


Commenting... :P


+# TraceService
+
Данный модуль реализует Паттерн Уроборос

Кольцо Дизраптора для каждого потока с возможностью потерь


Let's try it!


Why use the name 'options' here? This looks more like a 'lookup_context' or perhaps just 'context'.


Could you put the info@ronin-labs.io email instead?


Creates the CONFIG_EARLY_PRINTK option for the early console kernel messages (allow the kernel to print messages very early in the bootup process) on Spark architecture.


Thank you for this pull request. I have pushed my own fix to version 0.9.12. The Makefile requires PackageInfo.cfg in two ways: (1) as parameters for making the license and (2) as a Makefile include. The second precludes sections, so I had to keep support for a sectionless PackageInfo.cfg. I did use ConfigParser however. I'll use this fix on my other packages that have this problem.


Doubt an issue in practice. A bit like overloading case (say `README` and `readme` files in same repo) - legal to git but then unable to cleanly checkout on windows. Filesystem and routing complications for the slash mean verboten in any real JS filename.


в случае большого количества строк в файле они могут не поместиться в память jvm


I assume, that the button input field itself won't be submitted to the server in case it is disabled. Therefore the server doesn't know which button has been pressed and will simply refresh the page.

If you disable the button after submit, everything should be fine.


Awesome logo : )


Sorry, i didn't know that i can't edit the commit to the "test-push" afterwards.

I use the "Changing Lac" check (in the if/else part, when "Lac has NOT changed) to insert data to the table, because that happens quite often and so it is easy to check if the values are written correctly into the table.
I documented this in the code (See AIMSICDDbAdapter.java [line518](https://github.com/SecUpwN/Android-IMSI-Catcher-Detector/commit/548a11e1335796fdb37e4841da3447d667d92bf1#diff-afcaea7464548d33f335e3340f0dbd49R518) )

Before we can copy this to the parts in the code where a positive detection occurs, we need to insert the values. Timestamp, lat, lan and accur(rancy) won't work with "cell.getLat(), cell.getLon(), cell.getAccuracy()". **I tried a lot of things but - as the code of the remote repository is changing so fast, i didn't want to wait any longer before i can to kick my code in the waste basket, because it wouldn't fit in any more.
I hope there is no new code missing. I merge it manually but couldn't push it afterwards. Can't tell how much i fxxx xxx and so on... so i keep checking the code...


I should note that this seems to work ok in a different file:
```
alias m='mysql -u$MYSQL_USER -p$MYSQL_PASSWORD'
```
where `MYSQL_USER` and `MYSQL_PASSWORD` are also in `.env-vars`. Seems like it's got to be some kind of loading race condition.


Indeed, copy-n-paste.


r+