labofsahil/github-event-dataset-2012 huggingface dataset:
Yes.

I moved shortcut key code from scriptconsole.jelly to hudson-behaviour.js in [1], within 1.451.
and scriptler uses the code in hudson-behaviour.js.
so this pull request depends on 1.451.

[1] https://github.com/jenkinsci/jenkins/pull/364
Not new code, but should this only increment rec if (row) ? (e.g., if grouped)
@nashby thanks, you're right, hadn't picked up on this as I was just working off the existing specs (which mock the validators and therefore don't capture that behaviour).
You didn't understood my doubt :P
Structure is fine.. Which way would you do this (it does the same thing)

https://github.com/Fredi/WowPacketParser/commit/9f535fb98b7e6de1719397120281ec03bcd4f2db (Using ReadValue and just naming "Unk 1" and "Unk 2")

https://gist.github.com/1605882 (Naming accordingly for each version)
Comments Deleted.
tai ihan vaan ´if @ratings´
Stuff like this can be done with a one-liner using Guava:

com.google.common.io.Files.toString(new File(pathFile), CharSet.forName("UTF-8"));

Using Guava also avoids the file descriptor leak that your code has (if an IOException is thrown in read() then you never close the file, thus resulting in a file descriptor being leaked).
I would have to do what you would do, just use some google-fu and find out. Once you find your answer, please share here and I can post to the README to help others.
The getter should match the instance variable name (`Command#stderr` => `@stderr`).
Frédéric,

I have not visited this project in a while. I wrote it to handle some specific problems that we were having, and unfortunately haven't had time to generalize it for more use cases. I'll try and find some time in the next week or 2 to take a look at it, but in the mean time, feel free to submit patches.

Thank you,

Luke
Checked scope. Reviewed.
Glad to hear it, thanks

On Mon, Feb 20, 2012 at 2:35 PM, Jake McGinty <
reply@reply.github.com
> wrote:

> Sorry for the late reply, but yeah it's no problem. None of us care about
> a beautiful revision history, just making cool things :).
>
> --
> Jake McGinty
> ACM@UIUC Chair
>
>
> On Friday, February 17, 2012 at 8:09 PM, Joe Betz wrote:
>
> > I apologize for my previous three commits, the first was accidental and
> the last two were attempts to undo what it changed.
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/acm-uiuc/twitter-music-pd/commit/5f67e043c46d4f0ddcf9c14a965f4617b6eb3b59#commitcomment-980522
> >
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/acm-uiuc/twitter-music-pd/commit/5f67e043c46d4f0ddcf9c14a965f4617b6eb3b59#commitcomment-987243
>
for pep8 style guide class names must be capitalized and variables must not be camelcase (e.g.: not loadUrl but load_url)
Almost .. this throws the following error

    [javac] Compiling 18 source files to /home/sicsr/Downloads/WikipediaMobile/bin/classes
    [javac] WikipediaApp.java:8: cannot find symbol
    [javac] symbol  : class GeoName
    [javac] location: class org.wikipedia.WikipediaApp
    [javac] 	public ArrayList<GeoName> geonames;
    [javac] 	                 ^
    [javac] SearchSuggestionsProvider.java:41: cannot find symbol
    [javac] symbol  : variable HttpApi
    [javac] location: class org.wikipedia.SearchSuggestionsProvider
    [javac] 			String content = HttpApi.getContent("http://" + lang + ".wikipedia.org/w/api.php?action=opensearch&limit=10&namespace=0&format=json&search=" + URLEncoder.encode(query, "UTF-8"));
    [javac] 			                 ^
    [javac] 2 errors
The third from last number on each line is the divider between the cpu clock & ahb clock, minus 1. So speeds from 787mhz to 844 are still wrong.

{ 1, 787200, ACPU_PLL_2, 2, 0, 262400, 2, 7, 262400 },
{ 1, 806400, ACPU_PLL_2, 2, 0, 268800, 2, 7, 268800 },
{ 1, 825600, ACPU_PLL_2, 2, 0, 275200, 2, 7, 275200 },
{ 1, 844800, ACPU_PLL_2, 2, 0, 281600, 2, 7, 281600 },

fixes it.

You might be able to get higher speeds if you change the msm7x2x_clock_data.max_axi_khz = 422400; line in board-zte-skate.c, but it might be too fast with the higher cpu speeds in the skate.
The same : I will use Element el_wrapper = writer.element("div", "class", className, "id", "clerrors_"+formId);
Este commit ferrou o master
I've asked this on http://superuser.com/q/383355/47108, and BACON and PowerXPress seems synonymous.
Do people do this a lot?  My impression was that nobody really uses anonymous accounts
v0.4.1 is released! :)

Thanks again for your help.

-Spencer
I'm sorry. After cleaning cache it works properly.
YES : That's the thing in being a non-native speaker ...
@TomGiordano 
I know this is completely off topic but I didn't know where else to contact you. I was wondering whether you could help me in creating a libcamera source (for gingerbread). 

The reason for this is that I was working on camera in ICS when the camera was announced as partially wokring on the Blade. So I took the camera.msm7x27.so file from the Blade ICS and put it ont the Skate. The lib was detected and it tried to load libcamera.so. But libcamera.so threw an error. But I'd encountered the error before on GB so I did what I did for CM7 and I compied libcameraservice and libcamera_client. But when I rebooted it went into a boot loop complaining about the two replaced libs. So I've hit a brick wall. What I do know is that the libcamera I'm using is proprietary ZTE whereas the Blade is source built. So I'm pretty sure that's the problem. I'm asking you because you designed the libcamera for the Blade and I was wondering whether you could help me adapt it for the Skate?
Pues como prefiráis, si queréis volver a cambiarlo ya ves que no es mucho esfuerzo xD Por mi sin problema ;)
Very cool.  Not sure how you initiated this thread, was it part of the commit option?

-----Original Message-----
From: Matthew Moses [mailto:reply@reply.github.com] 
Sent: Thursday, January 05, 2012 10:37 AM
To: Rob Jones
Subject: Re: [fun-with-egit] Test Commit (c0d6efd)

The initial comment was created through GitHub's interface. Follow the links in the email to see the comment stream. You will notice that replying to the email also updates the comment stream.

On Thu, Jan 5, 2012, at 07:29, robjonesjr wrote:
> Matthew,
> 
> Was this a test?  Or, are you testing a feature?  I noticed this 
> e-mail was sent via GitHub too.
> 
> Rob
> 
> -----Original Message-----
> From: Matthew Moses [mailto:reply@reply.github.com]
> Sent: Thursday, January 05, 2012 9:57 AM
> To: Rob Jones
> Subject: Re: [fun-with-egit] Test Commit (c0d6efd)
> 
> (Just testing the line-by-line commenting in GitHub) Should the output 
> generated here include punctuation? Maybe something like this:
> 
> ```java
> System.out.println("Yet another line!"); // Note the exclamation point.
> ```
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/salamandertech/fun-with-egit/commit/c0d6efd774dad47
> 3899505dc50f94ed27d0add41#commitcomment-840552
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/salamandertech/fun-with-egit/commit/c0d6efd774dad47
> 3899505dc50f94ed27d0add41#commitcomment-840635
> 
--
  Matthew Moses
  mlmoses@fastmail.fm

---
Reply to this email directly or view it on GitHub:
https://github.com/salamandertech/fun-with-egit/commit/c0d6efd774dad473899505dc50f94ed27d0add41#commitcomment-840658
I've just noticed, that there is a 'Quick fix' for this error :-)
what about improvements applied to previous version?

http://forum.pfsense.org/index.php/topic,43188.0.html
maybe also mention [rbenv](https://github.com/sstephenson/rbenv) since some folks like SH and WF use it instead of rvm?
you can use pathSeparator from module System.FilePath
I made a pull for a proper fix to this in
https://github.com/D-Programming-Language/druntime/pull/112.
#1  0x0000000000a1c66d in Unit::_UnapplyAura (this=0x7fffc3fab000, i=..., removeMode=AURA_REMOVE_BY_DEFAULT) at /home/ACORE3/source/src/server/game/Entities/Unit/Unit.cpp:3355
#2  0x0000000000cd44b3 in SmartAI::RemoveAuras (this=0x7fffc3ebb040) at /home/ACORE3/source/src/server/game/AI/SmartScripts/SmartAI.cpp:440
#3  0x0000000000cd4995 in SmartAI::EnterEvadeMode (this=0x7ffff5f8e860) at /home/ACORE3/source/src/server/game/AI/SmartScripts/SmartAI.cpp:461
#4  0x0000000000a29670 in Creature::SelectVictim (this=0x7fffc3fab000) at /home/ACORE3/source/src/server/game/Entities/Unit/Unit.cpp:13126
#5  0x0000000000cd5985 in CreatureAI::UpdateVictim (this=0x7fffc3ebb040, diff=100) at /home/ACORE3/source/src/server/game/AI/CreatureAIImpl.h:566
#6  SmartAI::UpdateAI (this=0x7fffc3ebb040, diff=100) at /home/ACORE3/source/src/server/game/AI/SmartScripts/SmartAI.cpp:373
#7  0x000000000092d8f5 in Creature::Update (this=0x7fffc3fab000, diff=100) at /home/ACORE3/source/src/server/game/Entities/Creature/Creature.cpp:523
#8  0x0000000000b2c73d in Trinity::ObjectUpdater::Visit (this=<value optimized out>, obj=<value optimized out>, gridVisitor=<value optimized out>, worldVisitor=<value optimized out>)
I think characters_database.sql is outdated, becouse when I aplly it, I cannot start server, becouse of characters database SQL errors (like missing columns etc.), thanks for fix ;-)
Serverside is not executed as root. File.read won't work.
i used 
repo init -u git://github.com/tpruvot/android.git -b ics

repo sync 
Does it correct?

many thanks
another editor backup file
If "Symbolic link support" and "support for supplying a password directly" were separate commits, that would make my life easier and the history clearer.
A Map<Command, String> won't help because the value of this mapping contains more than a String (in fact has 3 strings and other useful stuff) and I don't want to create a wrapper to model the value, when the Property class does exactly what I need and it's semanticall and technically correct to have 
this new property.
But anyway why are you soo against it, what serious/ valid reasons do you see not to accept it into the main repo?
* double-click on caption opens textEditor, good so far.
* double-click on image thumbnail opens full-size image (`this`) in dialog box, good but, it works for the wrong reason.

Passing `this` to dialog exposes the dom element to unwanted repositioning. But, by coincidence, the double-click also brings up the caption textEditor which rebuilds the whole story item when finished. In testing, this accidental repair was hidden behind the pop-up window. How convenient.

I've made several attempts to do it right but none of them worked as expected. Answering `false` in the right place does prevent the textEditor from opening But I can't seem to keep the dialog box from messing up the dom for the story item, even when I duplicate the dom construction from the bind: above rather than using `this`. The original dom is still corrupted.

Any suggestions? I'm stumped.
APP_THIS_WAS_SECRET
Don't apologize! Thanks for the contribution; this is quite a clean pull request and we appreciate the effort.
It would be much more straightforward if you got the parser from `utils` and then added your own arguments rather than the other way round.
No. The dims have to be the same in all cases.  The matrix is a map between two functionspaces and the dims are the dimensions of data in each of those spaces. Unfortunately OP2 considers data dimension to be part of the Dat rather than the Set so we can't actually enforce this at sparsity construction time.
better ? :)
I'd move down the "gracefulStop(actorRef,"
this could in principle break source compat (while keeping binary compat); hence it could not be done in 2.1.x (x > 0)
Should FiniteDuration not be documented if it surfaces user APIs?
Hmmm, aren't you required to implement onReceive?
Roland implicitly suggested withDefault instead of map(_ -> 0).toMap
We'll make them package protected
This shouldn't be added into akka-remote, it should be a sub-module to akka-remote so people don't need the extra dependencies if they don't use rcl
I disagree, if you want to copy&paste from the ScalaDoc it's way easier if the name is contained within the description.
This seems very, complicated, simplify?
ouch, good one!
I don't really like methods with innocent sounding names like "getX" that do a whole lot more than just getting something.
Any suggestions as how to improve on this?
Why are those delay parameters not `Duration`?
Shouldn't they also be dilated?
this pretty much is a constant, so it should be declared like one:

    final val DefaultReplicas = 10

(observe the missing type ascription, which is part of the deal)
The timeout can be considered after the call returns. It will not fail fast of current call, but the failure counter is increased.
Using Future with Await here is not an option, since it will consume additional thread resource.
We could always just inline it?
And this should be 'FieldNames.member.%d'
Is this motivated by version-incompatibilities in the repository metadata for git and svn, e.g. Leopard's svn from Xcode being too old?

If so, rsync wouldn't be subject to this issue, and there is _no_ newer cvs executable than what comes with Xcode on 10.5.  Plus,  selfupdate-rsync now _only_ uses rsync from Apple or from us, and selfupdate-cvs now _only_ uses cvs from Apple.  

On the other hand, if this were analogous to the Bzip2Path option, where there are alternative executables with the same functionality that provide some sort of performance boost (along the lines of pbzip2's SMP threading to speed unpacking), then having the ability to override the default executable and use such an alternative would make sense.  

In the case of Bzip2Path, there is logic to deal with exactly the issue of having the field set in fink.conf but not present on the system:

		$bzip2 = $config->param_default("Bzip2path", 'bzip2');
		$bzip2 = 'bzip2' unless (-x $bzip2);

so I wouldn't say this is a show-stopper (I haven't looked into the branch yet to see whether a similar fallback mechanism is implemented there).
```
References
---------------
```
Should this be `_is_sysadmin()`? It looks like a helper for is_authorized() below, it is called directly in some other places like controllers, action functions, validators, plugins.py, model_dictize.py,  but maybe it shouldn't be? (Or if they have to at least it can be clear that they're calling something private).
Good docs change to indicate it's related to bind, I was confused above in the method that calls into this that just mentioned artifacts.
You should use the permission constants here, just on the off chance that they change.
Maybe "container" is a better word than "conduit" here? And maybe it would read easier if simplified to "Repositories are the container into which content is drawn and by which..."
I think you could change the code circa line 152 to loop over the extra frameworks paths (instead of trying a single one), and expand the extra paths to include potential name variations, e.g.
suffix = name.indexOf(".framework") == -1 ? name + ".framework/" + name : name;
for each:
  "~/Library/Frameworks/" + name,
  "/Library/Frameworks/" + name,
  "/System/Library/Frameworks/" + name;
  try library lookup and break when found

This allows specification by framework name, or explicit framework path to a particular version.
You could put the tracer param on another line
I had that originally but then when I ran the tests, persistent job flows would be terminated right after they were created, so I took it out.
I have no way of knowing the service name with only the OpalConfigurationExtension...
currentRetries ?
No sender?
how would the ``even.twig`` look like in such case ?
It's gonna be set either way when `_updateKnob` is called, so there's not much point right now. I'd rather use `CGRectZero`.
Lets try to keep our method names non-underscored. That's reserved by Apple.
But that again might take longer... the real implication here is that this `sharedGraphicsRenderer` is "class-pervasive", which means there's only one for all sliders on screen. That's why we need to set these properties each time. Each slider might be differently rendered, but only one cell is taking care of them (I guess AppKit developers who decided on the `NSCell` segregation are quite happy....?)
Y no linux, eh?
Define special enum with two values to use it in only one sitaution...  Looks redundand for me. 
May be just bool, whisout default argument?
Fixed that and other issues in next commit.
Sorry. I understand. :D
this shouldn't be deleted.
Is this github or is the indention wrong?
That should go to the top.
All right. I have updated it accordingly.
`ThreadSafe::Cache#[]` does a volatile read, therefore see the "Fixing Double-Checked Locking using Volatile" in the article you linked.
I thought there is `:force => true` option supported by ActiveSupport::CacheStore to do this. I believe it is supported at this layer as well.
Typo: `minimum`
I don't see this table being used anywhere else, is it really necessary?
That's surprising and unfortunate. I'll try to remember to add comments to remind people.
What kind of exception?
I'm glad to see the reuse effort. But, I see no reason we shouldn't handle the jar universally, just as we do with .flags and .check files.  Also, this trait is documented to be about putting a single jar on the classpath, but then it proceeds to place the scala library/reflect/compiler jars on as well.  Finally, we have way too much partest behavior hidden in system properties already, which is a complete PITA when trying to reproduce a partest result from outside of partest.  I don't see that we need separate system properties for all those jars.  How about we just pass a classpath.
HAS_SENDFILE_H_
Add spaces around `=`.
Well, it's an outdated standard that causes a lot of confusion, IMO. And I don't think it makes any measurable difference to performance on any real benchmark (i.e. excluding specially constructed micro-benchmarks).
I think we shouldn't be close-minded about what Trace can handle. I would suggest to allow other classes to define `_eval_trace` if it makes sense for them, instead of restricting what kind of input Trace can have. This means removing the check on `mat.is_Matrix` here. If `mat._eval_trace()` is defined, we return that, if not, return an instance of `Trace`, as below. Then anybody including the quantum folks will be able to use Trace without touching its internals, by defining `_eval_trace` for their own classes. What do you think?
pep8: space around `=` and space after comma.
I think the Float below is better. If division hasn't been imported, 77/3 -> 25 and if it has, it will just be the Float `25.[6]`.
This is the only way I knew how to create the iterator. I'd be happy to replace it with something simpler. Please post any recommendations you have.
And these.
The first time that I read this I was a bit confused, because I expected the "classical" `_eval_expand_*` methods to be defined as well (which now is obvious for me that it is not necessary).

I was wondering why non-Basic args work with this. It is because of this magic here. Any `expand` method must be explicitly defined, otherwise it is just a no-op by default.

Can you add something like this to this docstring (and the docstring of expand, maybe). "If you have not defined explicitly an `_eval_expand_*` method for your class **AND** you have created your class by subclassing `Expr`, your class is guaranteed to work with `expand` even if it contains non-`Basic` `args`. This is due to defaulting to no-op for any undefined `_eval_expand_*` methods of the `Expr` class."
Abbreviation representation of month (see http://www.unicode.org/reports/tr35/#Date_Format_Patterns)
Move up with rest of imports?
maybe…I didn't want to get into the whole mega/mibi thing.
(it is 1024*1024 bytes, not 1000*1000 so is theoretically mibi)
omit the final example.
s/nod/node/?
Also keep a username => user_id map so calling this again does not require a query.
I don't believe php has "integer".
How about -1 for consistency with all other find function in existence?
Added a function to flush the user data
Who -> that or which.
See the edit please.
Poate ar merge făcut un API precum: `Formula.get('%s-infraction' % rule[0], 'general-infraction')`, la fel ca la dicționare, pe care să-l folosim și în partea cealaltă?
It's common to deploy the same sensu config that covers the settings required by all the components. You can also drop json config snippets into conf.d/ and they are merged in, commonly done by puppet users.
I didn't even realise this syntax was possible, but of course it makes sense that it is!
I agree, so long as travis is running the full test suite (func as well as unit).
Вот так перезаписывать тип переменной плохо. Был массив, стала строка. Да и разве при join без ',' получится парвильный sql?
Missing XML documentation
should still check if these are valid, I think. They just aren't required.
so we were having trouble calling this.loginContinue inside the success function within saveBasicAuthHeader so we decided to move it to the global. the global file is kind of like our Util
For completeness, nice to have *test_bad_projection()* too.
A couple of indent tidy-ups needed.
If you mean the import, I would rather not as it is really not supposed to be used anywhere else, and is a dirty implementation detail (until somebody comes up with a better solution).

Actually I could store this stuff in theano.config.gpuarray if I was confident this will not break in the future.
You mean by `process_stream`?
Почему бы не использовать [q-http](https://github.com/kriskowal/q-http)?
you could move it outside of this class.
What warning?
gcc 4.7.2 does not generate any warning without this useless assignment.
Cabal tries to support as many old GHCs as we can. At least back to 6.12.3.
Unfortunately, yes, even IE10 can properly clone checked elements :-(
Yes. It's called "indirect eval"... it's really a thing. http://es5.github.com/#x10.4.2, step 1
Iteration may be 0 in the beginning and match no rows.
I added a reference to the Matlab function used to check the values of the multinomial logistic regression.  I also added a short matlab code segment that could be used to reproduce those values.
Don't include this, as this is a generated file.
I borrowed this name from wikipedia.
Then the probability density function for  is given by
http://en.wikipedia.org/wiki/F-distribution
_isatty() instead?  we already use it in util/log.cpp
Ah, I didn't like this player death implementation. I still need/plan to  do one myself though. So remove this bit and its DoPlayerDeath() func.
Mhm, but that would require the audio actor to have a reference to the backend, breaking the layer separation.

I'll move the caps to the Spotify backend as well. That's a good idea.
Ah, that needs to be updated in the Xcode 4.3 era.
find_domain should throw if it doesn't find the domain like ActiveResource/ActiveRecord.  Generally, if you expect the response to always succeed, and it's a core operation like "retrieve by id", you want to throw if you can't find it.  This then means that a bunch of duplicate error checking can be returned.
what is the reasoning behind this change?
the domain shouldn't be hardcoded but obtained from Sites.objects.get_current_site()
fix notice in strict mode (only variable should be passed by reference ...)
Whoopsie, somehow compose has gotten replaced with kwargs. Did feel like this looked odd. Will change it back.
^To easily replace those objects with Comments, use the comments_flat property instead.
Yes, the rule is we always use prototype-based classes. But we don't write module.exports = new Translation() as we've discussed about singletons.
We use a static get() method.
documentation is missing a ``@name`` doctag for the ``StackTraceFormatter`` class
Ahhh, I think we should be fine without it.  Seems pretty obvious...
The string `"Reddit compat"` is written out twice in this code. If you wanted to change the name it'd require changing in two places.
Okay -- this block convinces me that `LockFailedError` ought to subclass `UserError`.
hmmmm does it mean i name the class wrong ?
Me bummer ! ;)
I don't really have any idea, but I assume this fixed a bug at the time. You can try disabling it.
Should this really be just ignored?  Would we not want to notify the user?
I commented code out because it can be intensive in my mind.

If its acceptable just remove commenting on it.
:squirrel: is the official GitHub sarcasm smiley.
nitpick: our code style is to put linebreaks between style blocks.
This also looks cut and pasted. I'm not sure you need it in this case, though? The `tapped` class was for applying styles while something was mid-tap, but I've found it better to apply it to `:active` instead, so this  may not be needed.
if you'll notice, the code that actually installs the plugin -- line #73 for example -- is within a callback function that gets passed to PhoneGap.addConstructor().  since we shouldn't assume that a callback will be invoked immediately we can't set a var initially.
As above, please follow file conventions: `eltDialog`, `eltMessage`, ...
If the shortcut is the same for both platforms, you can just put "F2" alone (see examples elsewhere in this file).
Change to 15. We'll need to refactor this in the future so we don't have to update each translation for every sprint.
Also, two little nits here: "effected" -> "affected"; and there's an extra space after "a" on the first line.
Typo - extra ']' at the end of the line.
Typo: missing ")"
It's actually a style tag, not a script tag...
This is the same as the previous cases's assertion.  Should be checking command17 instead...
How would this not match `if(err) log(err)`? (that has `err` as an argument too, right?). Note that this of course very much based on common coding patterns, not an absolute solution to the problem.
is handling the error.type in the template like this really better than using messages?
Can avoid writing error message twice by having `assert interval > 0` within the `try:` and then catching `AssertionError` as well.
magic numbers like this are hard to understand
Actually, I might change this error to be a little different. Maybe something like `"Invalid mutation of reference #{it.value}"`.
@mgencur correct, but we always keep XSD's defauts and model in sync.

@tristantarrant sounds good ;-) thanks
Ok. This is indeed low priority.
Yeah, `merge_base` is kind of the odd-one out. I think it fits here just fine.
*Show* -> *Shows*
Please make an empty line before this line.
I like what you attempted to do here, but it should probably look like this:

```javascript
var script;
EditorHelper( butter );
script = document.createElement( "script" );
script.src = "...";
...
```
oh oh! you can use `originalEvent.dragging` here now, right?!
Don't change this alignment
please use ! whenever possible!!
this is very cool :)
Looks like the "fr" in "from" got cut off here.
I worry that this wouldn't be quite sufficient, as someone could search for something like "foo"  and this would match on "foo bar".  You can't really split on open paren's either, as we allow '(' & ')'  in system names.  You would almost need to match on the last open paren and trim everything from the space before that. 

I wonder if there's a way to send down 3 different values (The label, the id, and a matching value).
In general, in javascript, would recommend using the === and !== operators over == and !=.
Can you nest all of this test into another 'describe' with the method ur testing? These tests aren't really for the whole model.
installBoilerStrap

advancedReqResMiddleware.

Dunno.. ick
Same level for everything looks like a bug
That passes the tests and is more readable. The extra dependency on numy is not an issue inside astropy, so I'll change it to this version.
Will do.  I'm surprised I wrote that because I always use newformat.  Except there, I guess.
I think client core is missleading here.
Don't know how that happened
Yeah I wasn't too sure about which exception to use so just went with what was suggest on the ticket. I am fine with NotFoundExpection.
It is also unclear why github is rendering these bullets with the bar to the left like they were in a piece of quoted text.
again, indentation issues for lines 99-104.
I don't think we need to provide this helper.
Could you add a comment stating that this is not meant to be an annotation?
You mean not return them in the same dict at all? Could - but these are all part of the character sheet, so not sure why not keep them in the same data structure?
indeed sounds right. mapper->prepareObject $parent should be optional. so the loop above would only try to find a parent but then try to create an object with null parent if none found.

then the phpcr mapper will throw an exception if it does not get any parent.
The installer was having trouble due to not having a service container handy. This if-block is a fallback to allow a Fork install to continue.
I think you note that this requires git installed, and there should be a specific error if it is not
The same function can be imported from `IPython.lib.inputhook`.
Let's do this differently.

Add self.file_to_run as a traitlet, and add this to the end of parse_command_line:

```python
if self.extra_args:
    self.file_to_run = os.path.abspath(self.extra_args[0])
    self.config.NotebookManager.notebook_dir = os.path.dirname(self.file_to_run)
```

Then the block above in init_configurables need not change at all.
I would set this to start before other commands so that we reduce the  total nodes at runtime.
^^ I think you should consider changing your shiny new laptop.
Your TAB key is dead ;p
Please wrap this into `{% if request.locale == 'en-US' %}` and remove the l10n call of the alt attribute.
I'd make this parameter non-optional and move this value out into urls.py to make it easier to see that each page uses a different template.
A comment explaining this logic would be nice (what a download_link_direct = False does, for example).
Does this take a User or a Profile? Using `user` here suggests it takes a User, but the tests show it taking a Profile. Further, it looks like the tests create the question like this:

    question(creator=prof.user ...)

but then call `user_num_questions` with the profile and then `user_num_questions` does a filter on the user (which is really a profile). That seems wrong.

@rlr Does that sound wrong or am I confused?
ugh, sorry, I didn't mean to leave that in there.  I had to do that to get my test to run at all since my manually set up test database doesn't have a name beginning with test.  Apparently, I'm to lazy to rename my database...
Weird, I don't remember rebasing that file. I'll push a commit to keep the original comment in the new pull request.
Nope. Must have creeped in from merging the API History code. I'll remove.
I think that this sync functionality probably belongs in the model rather than in the controller.

Also - is it called from anywhere yet?
did you want the `:ID` in all caps?
Oops, copy/pasted the error, will change it for next commit
I'm not 100% happy with this solution. Probably a "constant" in server_conf would be better than a string. Or maybe somebody has a better idea.
or for this for example : `rtrim(sys_get_temp_dir(), DIRECTORY_SEPARATOR)`but if you start to use it everytime you have to deal with paths, it becomes crazy
Better yet, it should read:

```php
$emptyData = function (Options $options) {
    $class = $options['data_class'];

    if (null !== $class) {
	return function (FormInterface $form) use ($class) {
	    if ($form->isEmpty() && !$form->isRequired()) {
		return null;
	    }

	    return new $class();
	};
    }

    return function (FormInterface $form) {
	if ($form->hasChildren()) {
	    return array();
	} else {
	    return '';
	}
    };
};
```
what if it is a variable containing an array ? Will it be a Twig_Node_Expression_Array ?
I think it would be better to use a global variable rather than a reference to `$this`.

``` php
$constraintsByGroup = array();

array_walk($constraints, function($constraint, $k) use ($constraintsByGroup) {
    foreach($constraint->groups as $group) {
        $constraintsByGroup[$group][] = $constraint;
    }
});

$this->constraints = $constraints;
$this->constraintsByGroup = $constraintsByGroup;
```
This can be moved after `if` below.
It should be `$controllerResult`
let's not hijack this global. `tempfile.mkstemp` and similar functions accept a 'prefix' argument.
it might make sense to name "valueOf" -> "apply".

by convention in scala, maps use that for their keys, then you can access columns thus:

  val row: Row
  row("theColumn")
Is this necessary? This `before_filter` is already in `ApplicationController` [link](https://github.com/yammer/sched.do/blob/master/app/controllers/application_controller.rb#L4), from which we are inheriting in line 1 (and all controllers).
Perhaps this could be moved to the app model? `exec app.upload(path)`
The next step is to add image testing support for other filetypes. Once I have done that I think it is worth then feeding this back into the matplotlib testing code so that other projects can benefit from this change. At the moment, I do agree that it is very strange behaviour. Maybe it would be better to pass the extension through as an argument to the ```expected_path``` and ```result_path``` methods... (I will do that)
Doesn't affect the test - the test only looks for `@name=viewport`. The value is irrelevant.
We shouldn't merge code which references development branches in master.
We use a lot of function calls here to hide stuff (which may be the right way) and then render content... how many fields are left, would it be faster to just render the last field (or fields) ?
If you have `selection.@each`, it also covers `selection` itself. So you can just reference `selection.@each` here.
This was recently debated actually. The most important reason that I know of is to make the scope clear, but that would apply to many var statements at the top of the function as well.

As for grunt,  you can run grunt or run the unit tests, both will lint the source files. It sounds like your editor isn't reading in our .jshintrc file.
agreed, the only integers you ever want to see in code are 0 and 1.  This would go well in config variables
Yeah, according to Sergi's commenta at https://bugzilla.mozilla.org/show_bug.cgi?id=806368 and the emails I received from him, he always mentioned 300 for light and 400 for regular ;-)
Remove useless `default: break;`
Not really your bad but the usual guideline for if/else (and so for if/else if/else) statement is to use brackets for each block. Only simple |if| does not need brackets.
Would you change the name to screenshotDB or something more specific?
Can we rename itemLogEl to something less redundant? Like |logItem| ?
I think `inputParser.importDate` is more what your looking for here. (as on line 357 you basically extract the same information out of the date object?)
I don't see the new code for this feature. Are you sure you rebased the latest before creating your code?
What I meant is that I'm not sure if this overflow should happen at the end of the row or in the place where the image would be.
you are right. The local proposal has not been implemented yet
getAllContact function has been removed. There is only one getContactData function for query and update the contact cache.
Out of curiosity, why these 2 lines are enclosed with { } ? Is there something special about this syntax I'm not aware of?
``ApplyToImplicitArgs`` is not a case class, but if pattern matching only on ``Apply`` is enough, another level match can be removed
one-liner?
Special quoting, the standard one with single quote didn't work for some cases
Why include a stop command if it doesn't work? Also, the comment is misspelled.
Include the reason why.
Wouldn't hurt to add a print line to the end if the build completes successfully. Not really required, just could make it easier to see at a glance if things worked or not.
you should send a PR adding the composer support. It would make things easier
This should have the same effect as line 157. Why is this needed here?
I said to do that in the actual change room commands. This command, while called with them, is not fundamentally part of them.
This call got moved to the end of `updateDisplay()` since we want to do it as late as possible (after the gutter has been relaid out, etc.)
It's not fully pretty, I think it should be better to have just one "substep" class in the same way there's a "step" class, used both for identification of the substeps and to manage them. Later you could be able to add several classes for the effects.

For example, I would do "substep show text_to_red" and "substep show text_to_green", being "substep" an identifier for the substep, "show" an effect to show the item, and "text_to_red" and "text_to_green" color effects. This would allow to being more simple and versatile and also some basic effects like "show" could be collected on a library and also could be used both for steps and substeps.
typo in comment 's/no/on/' towards the end
Nope because the whitelist contains - that are escaped by preg_quote, it's a regex whitelist so you should know that and not pass garbage in it. I'll add a docblock to clarify.
oh sorry it wasnt about strict per-se, more of just a global, THIS FILE :)
Please add a new line with just your name on it for copyright on 2012.
It'd be helpful if the error message included the name of the filter that raised it.  Same with line 43.
I think this line got quite long. Maybe it could be written like this to improve readability:
```
<body class="site <?php
echo $option . ' view-' . $view
	. ($layout ? ' layout-' . $layout : ' no-layout')
	. ($task ? ' task-' . $task : ' no-task')
	. ($itemid ? ' itemid-' . $itemid : '')
	. ($this->params->get('fluidContainer') ? ' fluid' : '');
?>">
```
this should be experimental for at least one version of logstash
Yay tests! :)
eval(target.classify).find(id)
> I'm fine renaming this to manifest?
> But I don't think it really worth modifying the final name of harness-options.json file in xpi.

That's a different discussion, and I'd like to learn you're argument, but let's defer it for the future.
> If we care only about path, we should just expect the path instead of the module.

No that's intentionally abstracted away! Most of a time it will be passed a `module` from module scope and what is used by this functions should be irrelevant to a consumer as we may in a future use `module.uri` instead of `module.path` or we might add something else to the `module` and use that instead. This way changes to the consumer won't be required, while if API was designed around `module.path` we would not have a way to make changes without breaking consumer code.
already exists as `domain` a couple lines down. If you want this name for consistancy, just change the `domain` and usage in the template.
actually, in this case, the pushlogs are compared to the project test run data.  so we do need the project param.  Where we don't need it is for pushlog/branches and pushlog/db_size.  So I'll change those two.
@daniel-pittman Not that this is authoritative, but from wine, it appears the file being replaced needs to be writable: http://source.winehq.org/git/wine.git/blob/c41a1a29b538127ce66f7fdbdb818b613b767f27:/dlls/kernel32/file.c
Fixed in https://github.com/puppetlabs/puppet/pull/623
If the code is says the same as the comment, moreover using less symbols, then I don't see the comment's value, apart from having to change modify something in two places if there is a need to change anything.

Comments should *add meaningful* information to the code, otherwise they are just noise (of which most code already has more than enough).
I don't like that this takes away the static typing,

    _.each(@[@"foo", @"bar", @"baz"], ^(id key, id obj) {
        // ...
    });

will - unlike JavaScript - give you a nasty runtime crash. I think we should use static types throughout Underscore.m.
Have you considered e.g. `_.arrayEach` ?
Just to confuse things further: even though having `stub_const` return or yield the replaced value is useful, it still feels awkward to me. At first glance, I'd expect a method named `stub_const` to return the `stub` (in which case @justinko's consistency argument would make more sense to me).

Also, I can't think of a good use case for this. When would you ever want a handle on the original value? Even if you could argue it, why not just leave it to the user to assign the real value to a variable?

```ruby
real_thing = THING
stub_const("THING", fake_thing)
# etc
```
We usually prefer the be_kind_of matcher here, also see for example spec/ruby/core/time/at_spec.rb
I'll look into it. See how it works out.
This was useful to know.  Thanks @EvanDotPro for helping me find this.
Nope. No longer used.
Our thinking was that it's easy just to s/Interface/Trait/ and that it's not really "Providing" anything - the trait just makes implements the Awareness....

So these traits make things that are aware of things actually aware of them...

s/Trait/Skynet/
You should also add `js` to repo...
I would recommend using the filter instead of the tag, so that the auto-escaping is applied on the translated string being output.
maybe move this below the line below to make it consistent
superstition? i tested this code and it worked fine.
you dont have to get all schoolmaster on me with this one, i know its fishy, but the dependencies of the PersistentCollection are borked. Target for refactoring in 3.0 i guess.
for the child/children annotation, this is kind of pointless (and depending on order of operations could produce false errors). children are automatically removed. i wonder if we should somehow throw an error if i try to remove a document that still has children but does not cascade delete on a children annotation. (an ORM would do that, if children would be some sort of foreign key)

as children is a potentially large number of nodes (they in turn could want to cascade deleting their children and so on) we could handle that case specifically and not schedule a remove. but on the other hand you won't get doctrine events then. so maybe the thing if you really want to delete a large branch would be to go through PHPCR directly, in the same spirit as you could do a large operation on a SQL db through doctrine dbal if the ORM is too slow and not needed.
oops ?
Use a join.
fix this todo
you can just add the option  `default="simple"` to add_option
These are starting to get long. At some point, we might want to replace this with lookups into an array.
part 1-multiple exammarks for multiple grades
tiny note! the period in (www.) needs to be escaped, otherwise it means "any character" -- so (www\.) :) 

THEN cake? ;)
Sounds good. I'll break it out in a separate patch, soon.
We probably need to escape these strings, yeah?
can we turn off oauth by default? it seems like the defaults are way to easy to guess.
@ohadlevy, @witlessbird .  We were using Rabl 0.7.6 which didn't have condition :if, :unless.  This was released in version 0.7.7 (on Nov 26), but there was an issue with the root json object randomly not appearing.  I don't see the root addressed in the change log.  It it's fixed in the latest version 0.7.9, then we can upgrade the gem and the code.
Hate to be picky, but can we change the text to: 
"When enabled, the 'Object Details' tab will be visible to users with the correct permissions when viewing an object in the repository"

Via David.
This section below is worng, please change to something like:

G::LoadClass('pmScript');
    $oPMScript = new PMScript();
    $oPMScript->setFields($aFields['APP_DATA']);
    $oPMScript->setScript($trigger['TRI_WEBBOT']);
    $oPMScript->execute();

$aFields['APP_DATA'] = array_merge($aFields['APP_DATA'], $oPMScript->aFields);
$caseInstance->updateCase($aFields['APP_UID'], $aFields); // to save change in the case variables
To avoid the argh of spawning and adding a dependency on the cli tools, use something like g_app_info_launch_default_for_uri ().  Passing a launch context will also give you a busy cursor, which g_spawn_* won't do.
probably that type on the list can go away
слишком много ответственности на функции. ОНа и used заполняет и reachList. Список можно заполнить в отдельной функции. Нужно минимизировать ответственность функции, чтобы ее было легче понять и у них было меньше параметров, а так же можно было ускорять или улучшать отдельную часть, а не большую функцию целиком
Having the full_name in the DB will certainly be useful in the future, but I can't help thinking that for the purposes of checking a repo already exists in bulk a LINDEX against a Redis list would be the simplest and fastest.  Saying that, there's probably a large gap between us, and when the load is sufficient that we need to consider performance this closely.
It seems to more precisely express the intent to me. The first thing must be the string query, and all the others must be singular parameter values. Technically the string query is also a singular value, but it's a different kind of thing from the parameters.
This is TextEdit for now, just to demo the functionality. It can be changed to whatever program you want to send system events to, e.g. Team Fortress 2.
Perhaps reduce the indentation a bit here ?
If we're ignoring then I think this error should be changed to a warn.
201's are for new resources. I can make this a 200 with no body?
I think I understand what the _extra property is trying to do. I think it's adding items onto the searchdocument that can be used to process the result. However, the search feeds are returning a `extra: null` property for groups and content, which shouldn't be returned to the UI.
typo s/NoTimouts/NoTimeouts
It's more to know to know that a classfile was found, and if a hyperlink is not found, it's not the SourceLocator anymore, but the presentation compiler that messed up. So the file name is enough.
I'm not positive, I suck at git.
is nbsp needed?
I'm not getting that error... it's working lovely for me. Let's merge and see what happens. ;)
Great, i use this coding standard, too. ;)
Ahhh, found it. Nice:

"Adds or updates the number for name in the debug menu on the right. E.g. if you want to display the player's x velocity for each frame, do the following in your update() method:"

```js
// assuming #player# is an entity
ig.show( 'x vel', player.vel.x.round() ); // round before output
```
As this file is no longer referenced, Visual Studio might complain about it not existing. 

If you delete it from VS and do `CTRL+SHIFT+S` to save all changes, this change will also disappear.
From testing, this doesn't actually affect the updater behaviour, but changes UpdateState.
You need to assign the key. Moreover, you should just replace lines 107-108 with this:

    $key = config_item('encryption_key');

... and leave the rest as it is.
No, but you do need to push your local changes to your fork on GitHub in order for the pull request to be updated.
Так вы же используете в этом месте не как логирование, а как получение даных DNA для обработки. Это очень плохо, и повлияет на оценку,  так делать нельзя.

ЗЫ: вы акуратней отправляйте с почты, посмотрите, вы цитатой заполнили всю страницу :)
Пофиксил в https://github.com/volt0/osx-project-1/commit/54e0c91df05e97991fc485ea1afbf55435b58707
Да, главным образом я про геттер. Обнаружил забавную возможность - изменить массив, получив его геттером.

Можете проверить сами, сделать что-то вроде [[myCell DNA] removeAllObjects] не позволит, однако следующий "финт" очень даже пройдет:

```
id tmp = [myCell DNA];
[tmp removeAllObjects];
```

Всё, массив удален, а класс об этом и не ведает.
Do we really need to define this constant twice?  Could this be public, on either the controller (here) or on a service interface?
I don't think you have to do this explicit initialization any longer
ol semantics?
Nope, because that's not true :)
the brackets usually denoate a pattern. Like search <query> - search something for <query>. Just say standup for the command.
This and all other occurrences in this test case should rather be `LookupTest.superuser_login`. Aside from that, why not rather use an editor login to make sure that not only superusers can use the lookups?
I would write eeg=False explicitly and include misc channels too.
you should get rid of the have_isotrak even below and always write the dig points.
I initially thought so too but I tested that it still runs fine if the constraint doesn't exisit