labofsahil/github-event-dataset-2012 huggingface dataset:
Here is a suggestion. You can use the `str_pad` function from `stringr`. So you would do something like `str_pad('Appendix', getOption('width'), 'both', "=")`
That was from before namespaces and should obviously be replaced by a namespace.
Well, autoloading would be a separate module, obviously. We could use it to setup autoloading from outside the module, though, no? Will talk to you about that next week.
You sure this isn't just when you start MacVim through zsh via `mvim`? It tends to do that.
This broke some things that depended on errors being raised early, suggest we put this on top: https://github.com/jacknagel/homebrew/commit/5c5e31bdf2c3d2f76b18fc688dcdb373842c01df
["Keyword argument queries -- in filter(), etc. -- are "AND"ed together. If you need to execute more complex queries (for example, queries with OR statements), you can use Q objects."](https://docs.djangoproject.com/en/dev/topics/db/queries/#complex-lookups-with-q-objects)
Give it a dedicated .springrc that looks in the current directory, then let the user decide what should go inside it if they **want** alternative behaviour.
Yes, cool!

How do you like Brendon's approach?

Cheers,
Henning
It would be better to use an array comprehension instead of `meshgrid`.
You really don't want to call pipe() here if you're not going to use the resulting fildes, else we leak them because we don't try to close() or fdopen() them before returning since we have nowhere to return the fdop() result.  You can either make sure we close these before returning if we're not outputting them, or better, do like the original code and open the pipes only if we need them.
Ah, ok. I would still have that as a separate commit because the intent
wasn't clear
(at least to me)

On Fri, Jan 13, 2012 at 10:47 PM, hautreux <
reply@reply.github.com
> wrote:

> It is not just a cleanup as the intent is to be sure that in case of an IO
> setup error, SLURM_SUCCESS is still returned (to prevent the node from
> being set to drain). I needed to ensure the return of rc instead of
> SLURM_ERROR in the fail* label and as a result had to do some modification
> to set rc to SLURM_ERROR where necessary.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/hautreux/slurm/commit/632de91534a723f4d84d59ab3fc5676e88f0f944#commitcomment-866397
>
Then leave this and I check this tomorrow. 

----- (Sent via mobile phone) -----

Am 12.01.2012 um 21:18 schrieb phaidon<reply@reply.github.com>:

> If I use your suggestion the text which should be under the box is on the right side of the box.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/phaidon/Wikula/commit/7bc9db668ef89f508413bd600db781faa9bf31d5#commitcomment-862248
@Th0ar 不过通知不一样的话貌似model也要变。。那`one_to_org`什么的再说吧，先不写了
Thanks, I guess I'll find the other commits and merge those too.
Imagine that you start writing. I get the notification that you began to write. But you removes everything and turns to write. I turn to receive notification that you began to write.

I do not know to what extent, this is good. Perhaps the best, would notify the user starts to write forever and only after an event in the conversation.
crush in map->GetUnit()... as i think, may be only hardware.
Whatever passes the tests on all versions of Plone that we're trying to support. :)
Move the block to its own method and put it at the beginning like is done elsewhere for consistency.
I want these a lot, the most annoying part is actually thinking about the correct width. I'd really like to keep these. Especially bits. I really want to see what bits something has quite often (with correct leading zeros).
about dcmgr/config/db/migrations/0001_v1110_origin.rb, please discuss how to add an new column on an existed table.
i think it is better to create other file, like 0002_add_instance_nic_id.rb, because we can upgrade to latest schema from old schema for a service which already started. we should prepare the migration path.
I can fix that, I merged it in.
I Config RealmZone = 8 English , did mean i need to use witch one en or us ?
possible fixed in 1663.
@schmoozerd, this crush not related to multithreading, only to inattentiveness and lack of testing under heavy load.
PS cyberium new patch has no sence in R2 - different checks "right to enter" systems.
test this. https://github.com/mangosR2/mangos/commit/5e2a66298db24c2aa0719b6817cbe391821bac5d
need players feedback.
:heart: :heart: :heart:
Why do you need allow_input_downcast=True?
Exactly what I need, thanks for this commit
Better to refactor matchWord and reuse here.
But... isn't a qemu-img raw image similar to one created with dd?
Thanks.  relativenumber is a new option, so there needs to be a guard on the setlocal for backward compatibility, but it's pretty straightforward so I'll put it in.
I agree with @lukeman https much better. using https in http site is okay. but using http in https is not secure.
Meh. This seems to break vintage ex.
All files start with an empty line before the licence boilerplate. May I remove them?
Could you publish the update to npm?  Cheers.
If you could, I would <3 you forever
@kkfnui

Thx, the '\t' should not be there. 

I will delete it.
@lloyd - I'd say - yes, but would it be better to keep knowledge of the static middleware out of here and have the resource path be part of the configuration object?
Valeu Shadow!!
Maybe a case insensitive re search here, just for the sake of security?
event works
Ehm, yes there are.
Also just FYI, the way this patch is written won't actually correct the IncompleteRead as subsequent reads will simply return an empty result (if not an exception) because the file descriptor has already been closed. What would need to happen would be to reissue the entire request which my suggestion to move this error handling code will inevitably do.
Well, I can make it `Base < Syncer::Base` for clarity :)
here we can use, menuitem.data() instead of using contact_menu reference.
But without talking of inheritance. I mean for example Qt has for determinated widgets an userData property where you can store what you want.
I should grep :P
beginning to grasp the attr, text, xla and xlt ; bear with me :D
is it needed if add, 

top.restoreSession before : $("#addDR").submit();
The test that identified the logical error should probably be added, too. Also, `dumb` is sometimes faster than `smart`, so if a change is going to be made we should make sure that our assumptions about the smarter being faster are correct. In this case they aren't:

(tot is your current function)

```python
>>> t=time();jnk=[tot(i) for i in xrange(1,1000)];print time()-t
1.98099994659
>>> t=time();jnk=[tot(i) for i in xrange(1,1000)];print time()-t
1.02399992943
>>> t=time();jnk=[totient(i) for i in xrange(1,1000)];print time()-t
0.0240001678467
>>> t=time();jnk=[totient_(i) for i in xrange(1,1000)];print time()-t
2.27099990845
```
Matching "#\d+" seems overeager. In my case, we often discuss Zendesk tickets by number as well. It would be nice to listen for "issue #?\d+" instead.
你这写法碉堡了……
Тут и дальше всего лишь форматирование кода. У меня так NetBeans настроен по Alt+Shift+F. Настраивал в соответствии с Symfony 2 Coding Standards из документации.
Vadim,
I hit upon this bug while using the dpb -J 1 switch and it exposed the make install where it did not install files in the /etc/.... folders. I hope you checked that the configuration files is really installed in /etc/.... locations. If it is installed then sorry for the noise.
Hah, that's what I was thinking. It looked like it was copied from Java or something.
Thanks, I've added it manually because I've made a lot of changes and I didn't want it to be added automatically
this gives me a syntax error.  please provide tests or at the very least _make sure it parses_
Please publish this as a gem once it's working. Not all of our backends can support git-based gem deps. I won't be able to merge until it's a properly published gem.
the requestCapture update was just moving the div tag which was on the end of line 84 to a line of its own.
Why did you change from @Resource to @Autowired?
I personally rather @Resource over @Autowired since the former is JAVA-standard vs Spring standard.
any way of keep using @Resource?
I'll keep it in mind, even though I initially rejected that due to it complicating inheritance. This bit of code is going to change quite a lot, soon, anyway.
That would be a really good idea for consistency. Fonts are assets just like textures and models so this makes sense. Since I'm already using require.js it is a matter of 4 lines of code to load the font.
I had to undo this commit - not stopping event propagation here causes a mousedown-mouseup sequence on mobile devices after the touchdown-touchup sequence.
The error message should contain something about the expected parameters. Think of the similar Java error that lists both the expected and the actual parameter list. Sometimes it is needed.
loadFiles(qstringlist) should set this->Implementation->FileNames.
not loadFiles() shouldn't.
You can ignore the counter cache directions. I removed them in a subsequent commit as it seems there are some bugs with counter_cache and rails 3 so I'm showing you how to manually implement it in the HW.
As I suggested above, this could have been made clearer with the same code...

```ruby
r = [s.created_at.strftime('%Y-%m-%d')]
r += data_required[params[:report]].map {|which| s.send(which)}
```
Wat? I removed the db ones o.O Ah, well i'll fix that, i suppose x)
Looks good. Pretty nice that we have a crude oil EF in FuelType!
I cherry-picked this commit, so you will need to reset to master.
`var elems = (content.cheerio) ? content : parse.eval(content);` is a better way to write it.

And I couldn't use an object deep clone, as I didn't want to follow and all the parent links, etc.   Converting to html and then back to a cheerio is a non-optimal way to do it, the right way is to write a recursive cheerio object clone that only goes "down" through children properties.  But I wasn't familiar enough with the code base to write this function, and using cheerio --> html --> cheerio was quick and dirty.
I think we should encourage cimports from Cython's "stdlib" also in our test code, i.e. change this to

from cpython.whatever cimport PyBUF_C_CONTIGUOUS
Thanks. Important Fix!
don't actually know if the json stuff is working, didn't test it - but it's useless until the game object can be jsonified
Thanks ekeeke, this will definitely save a lot of time in the future. Also, thanks for pointing Maister in the right direction with regards to SSNES Wii.
nope. just random stuff i saw when reversing client
I'm not sure.  Yes if this is the only active sort  in the graph, and if no other cpu intensive activities are going on.  I'm not convinced that is likely to be true for roxie.
Also, the benefit of the parallel sorting only tends to come in for large sorts, and roxie tends to be sorting small quantities of data.  
There should probably be a hint which could be used to enable the parallel sorting.
Error occurs:
03-05 16:20:04.311: E/AndroidRuntime(6632): java.lang.RuntimeException: Unable to start activity ComponentInfo{com.imps/com.imps.activities.IMPSContainer}: android.database.sqlite.SQLiteException: no such table: recent: , while compiling: SELECT * FROM recent ORDER BY time
03-05 16:20:04.311: E/AndroidRuntime(6632): Caused by: android.database.sqlite.SQLiteException: no such table: recent: , while compiling: SELECT * FROM recent ORDER BY time
Move into bounds class as getCenter()
Hi Tair,

Thanks for the pointer on the windows postgres user, I have put back a fix for that into master, so if you are happy to drop the 's from around the cd, I will merge in your changes!

Thanks,
Tom
If you remove line 9, what happens here?
@leak 
this commit will propably not fix : https://github.com/TrinityCore/TrinityCore/issues/4415 right??
maneiras que -> maneiras em que
pottyParm

I like it!

:clap:
Yes, this patch is worth a new gem release
Thank you thank you thank you!

Just for future reference: In addition to the complaint regarding #attributes_for and nil values ([here](https://github.com/thoughtbot/factory_girl/commit/a883315bc4775f5b9996a378d6c4ebe0ebf44a5e#commitcomment-878888)), replacing #build_stubbed with #build was unsatisfactory because the later does not set the "id" attribute, so resulting objects are useless for url_for helpers.
Auto-generated, will change.
Sorry for the coding style, i'm always chocked to see that in templates html indentation level is mixed with ruby indentation one (that are totally different, so no need to include spare spaces from the ruby indentation level in the html produced).
But I'll respect you're habits and code the Redmine way.
Since `34` ends with an arrow, the next line is part of a function — and CoffeeScript uses significant whitespace (so it has to be indented 2 spaces).
Thank you a lot again, very usefull informations:( that nvram.txt was something i wasn't aware of ).... I'm testing them as soon as possible.

Nice job on the iconia, you and astrasikov form a super kernel team :)

p.s.
are you testing the hardfloat builts? it seems that there's some power in A9 that wasn't unleashed yet
What context is this generating an error for you?
Ouch, will look at diffs more carefully next time.
Please use the self.timeout here instead of the hardcoded 10 seconds. Also, you're passing selenium to the lambda so just use `s.title`
Unfortunately, this doesn't actually fix the issue, it merely prevents the error from being generated. The options object is still undefined until after this part of the script is evaluated (unless options are defined in the). For now, removing the "options.clickBlock &&" part of the IF check will allow the code to operate as expected (blocking context clicks on media links), and removing or commenting the code will disable it.
yes i know, but with all these crappy ruby paths .... under ubuntu vm as a root it wont find the plugins/ directory cause it searches in the var/lib/ruby-rvm/ .... and not in my home directory ... maybe i find another method to figure that out ...
Heh yea, you're totally right, simply forgot in this case, I also forgot to bump the API level. I assume this broke adventure, sorry :(
3.4.0 introduced the first version of it. 3.5.0pr1 contains the latest which is actually powering Loader.
what about making this backward compatible by writing code like:
```var Script = require("vm").Script```?
AFAIK this works both in 0.4.x and 0.6.x.
Note that Titanium.File rather than Titanium.Filesystem.File is being used in this file.
Just use view string from settings
Just curious, which ruby interpreters like reject! but not select! ? Seems weird.
issue http://jira.grails.org/browse/GRAILSPLUGINS-2968
Just a quick note. Using this syntax is only valid in ruby 1.9.3 (not 1.9.2)
More information on [stackoverflow](http://stackoverflow.com/questions/198460/how-to-get-a-random-number-in-ruby/2773866#2773866)

When you `rvm install 1.9.3` and receive a configure error, try the following:
`rvm install 1.9.3 --with-gcc=clang`

Then do this: `rvm use 1.9.3`
and `rake setup:dev` in order to setup everything
Please insert a Whitespace between "*" and Text
You don't have to put the files you modified in the commit message
Why `1.0` and not just `1`?
shared_ptr в новом стандарте есть, лично я его использую
перезагрузит>Ь<ся
you need those other 2 patches to get use of this.
one in framework base, other in dalvik!
seems, wrong change at this line
Ganz großes Kino!
Actually, final functions will always have a vtable slot in D.
I understand your concern with regards to nested objects.  This approach supports nested objects, lists, etc. with the major caveat being that the same tag name must have the same "type" in the entire document being parsed.  So I cannot have a tag named "Value" that contains a string in one part of the document and then a complex type in another context.  All of the beanstalk API calls have a unique type per tag, so this case does not arise.  

The obvious solution to this is to declare the tags using blocks:

```ruby
tag "SomeObject" do
  tag "foo" :string
  tag "NestedObject" :list do
     tag "foo" :datetime  # In this context, foo is parsed as date time
  end
end
```

This is a little more complex to implement, so I decided to skip for now.  If you would like to adopt this parsing approach in other parts of fog, I'd be happy to implement it.

Finally, on performance, one would have to benchmark to know for sure, but I think the overhead here is negligible given that the bulk of the time on any request is spent on the actual request round-trip, and not request marshaling.  I think this is a worthwhile tradeoff for writing less code and easier maintenance.
You should be able to get the value just by using the Route53 web console to ALIAS to an ELB in that region. In fact, it probably doesn't even have to be a real ELB, you can paste the name of a fake ELB with the proper DNS suffix for that region and the console will tell you the Hosted Zone ID before even trying to save the record.  

Note that it's the same Hosted Zone ID for all ELBs in a region, for all users.  


On Wednesday, November 14, 2012 at 7:59 AM, Wesley Beary wrote:

> In lib/fog/aws/requests/dns/change_resource_record_sets.rb:
> > @@ -198,6 +198,7 @@ def self.elb_hosted_zone_mapping > @elb_hosted_zone_mapping ||= { > "ap-northeast-1" => "Z2YN17T5R711GT", > "ap-southeast-1" => "Z1WI8VXHPB1R38", > + "ap-southeast-2" => "Z1WI8VXHPB1R38", # TODO Needs updated value  
> Maybe if the value is incorrect we should just leave it out.
> @bgentry (https://github.com/bgentry) - I think you got these values via our TAM, any chance we can get this one also?
>  
> —
> Reply to this email directly or view it on GitHub (https://github.com/fog/fog/pull/1271/files#r2125585).  
>  
>  
>
This line doesn't add much, yet does mutate the global namespace. It adds nothing, since the `Queue()` call in the next line will do the same to find the current connection if none is explicitly provided. `use_connection` should be avoided in scripts altogether. Instead `push_connection` or `pop_connection` should be used.

What I would like better is to pass in an optional connection to the constructor for explicitness.
> Could you point me to the right direction?

Did you see the links I included in my comment above?

> I'm basically changing a sleep to be longer if the cassette *does not*
exist (it makes an API call out and waits some time if the thing it's
trying to get isn't ready). The tests are much longer without being able to
check.

Honestly, for your use case, it sounds like using a hook might be better:

``` ruby
VCR.configure do |vcr|
  vcr.after_http_request(:real?) do |request|
    sleep 2
  end
end
```
Upon further reflection, should we just make this return Collections.emptyCollection() ?
I believe this was already taken care of with Nebula. I submitted thinking he was going to leave this suggestion on a villavu thread, however he migrated to git and submitted himself.  Sorry I don't know how to remove pull requests (via online navigation - [from the computer + files I believe it's tons easier] on github & processing commands for repo's/pull requests/etc..) as it would help clear a lot of clutter I seem to bring to these attempted changes >.<
you can drop such braces in matches
Make this a 1-liner and move it into the if-clause below (since it isn't used in this scope.
Ouch, undocumented behavior that an empty whitelist makes ALL unknown addresses whitelisted, undocumented surprising behavior.
commented tests are not nice.

Please add tests for looking up threadActors (using system.actorFor()) and examining their path, printing them, etc.
No, i was only thinking that you should use copy internally, since it's a case class. You need this for java.

25 sep 2012 kl. 21:25 skrev Raymond Roestenburg <notifications@github.com>:

> In akka-camel/src/main/scala/akka/camel/CamelMessage.scala:
> 
> >     */
> > -  def withoutHeader(headerName: String): CamelMessage = copy(this.body, this.headers - headerName)
> > -
> > -  def copyContentTo(to: JCamelMessage): Unit = {
> > -    to.setBody(this.body)
> > -    for ((name, value) ← this.headers) to.getHeaders.put(name, value.asInstanceOf[AnyRef])
> > -  }
> > +  def withBody(body: Any): CamelMessage = CamelMessage(body, this.headers)
> you mean that I should take it out and get people to use copy?
> I like that, I've actually used copy a couple of times in tests instead of these methods, cause the semantics are clear already instead
> of some weird named 'withBody' etc. let me know.
> 
> Maybe all this with... stuff should be removed, ar least for Scala and leave things there for Java, and also leave in the methods for Scala that do something different than case class copy style
> 
> —
> Reply to this email directly or view it on GitHub.
>
it depends on the circumstances whether that is relevant, but introducing a name like that in the for-comprehension leads to allocating an extra tuple to be passed into the yield part, IIRC (if this is a hot spot, you should consult the byte-code)
Ok. Now I understand.
why parens here and not below?
The alternate solution is to allow to set nr-of-instances = resizer and then it uses the resizer. Thoughts?
This part of the code is not used within the Akka Logging framework – it's a kind of adapter to use the Akka Logging API in non-Akka code (I'd like to have a single logging API in Scala)
I think there is a race here. If first leave runs first and that is completed on third before this registerListener happens you will never get the notification. need some more barrier.
But on the other hand it’s not as if calling this constructor would immediately end the universe, either. It’s more meant to give a hint that the origin of these exceptions is within Akka.
I'd say 'healthy *nodes*' or 'member nodes' in the cluster not 'systems'
this doesnt look right, your removing all the new stuff
This is main cause of 
java.lang.NullPointerException
	at org.jooq.impl.TableImpl.toSQL(TableImpl.java:131)

Reasons - the same as above
I think that something can *be at* the next level, but you *take it to the
next level*. But yeah - let's ask the native English speakers.


On Thu, Dec 13, 2012 at 11:34 AM, Eleftherios Garyfallidis <
notifications@github.com> wrote:

> In doc/devel/intro.rst:
>
> > @@ -2,14 +2,13 @@
> >   Introduction
> >  ==============
> >
> > -dipy_ is in development mode at the moment.
> > -
> > -Please do join in the fun.
> > +Dipy_ is always seeking courageous scientists who want to take dMRI
> > +analysis at the next level. If you share the same vision and you are willing to share your code please let us know we will be happy to help.
>
> Google thinks otherwise "at the next level" 422M, "to the next level" 70M.
> Somehow, to seems nicer. This is what I initially wrote and then changed it
> to at. No worries, will ask Ian or Matthew who are native english speakers.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/nipy/dipy/pull/103/files#r2409455>.
>
>
yes, that would indeed be a lot nicer. tbh, i'm not sure whether any add-on is actually using this right now. haven't updated that recently, so things might be missing that are available in guilib. just fixed it up when it failed to compile by implementing the changes when merging it in. but i haven't used or tested any of the gui stuff in here.
extradata check not needed.
Where are you using shoulda?
why not just call this "coerce"?
Not sure if a check needs to be done if the annotation exists before doing that
Ok done !
Will throw an exception. My next task is to review error handling. Doing it in the next pull request...
Yes Sir
docs needed
These use cases do not miss. If you have a look at how it works behind the scene, you will see that we generate a "private" template; so everything that works with normal templates also work with these "special" templates.
This should be prefixed with `tui_` to avoid future potential naming conflicts.
Could you please add inline documentation as for the other methods? Thanks!
there are other places where we get a list of version, they should be extracted to a method more generic and re-used.
Are you sure that this works as a way of passing a default? If not, maybe you can just bite the bullet and ask for a keyClass and valueClass as before.
Wrong coding standard here.
Or just not allow nulls in there at all :-)
Hmm, is that settled yet? What do you think would be a good timeline? CC @njsmith @teoliphant @certik @rgommers
Gotcha.  Sorry, I got distracted before finishing the review last night... let me see if I can do a better job now.
Nvm. the python documentation says that it can be discontinuous in that case.
Adam I think you are right about that.  Changing to Z_TYPE_P(tmp) ==
IS_STRING seems like a better way to avoid the problem.

I will ask for help on IRC for how to submit a better fix.

Thanks,
Brandon Kirsch


On Wed, Sep 12, 2012 at 11:13 PM, Adam Harvey <notifications@github.com>wrote:

> In ext/odbc/php_odbc.c:
>
> > @@ -1765,7 +1765,7 @@ static void php_odbc_fetch_hash(INTERNAL_FUNCTION_PARAMETERS, int result_type)
> >  		if (result_type & ODBC_NUM) {
> >  			zend_hash_index_update(Z_ARRVAL_P(return_value), i, &tmp, sizeof(zval *), NULL);
> >  		} else {
> > -			if (!*(result->values[i].name)) {
> > +			if (!*(result->values[i].name) && Z_TYPE_P(tmp) != IS_NULL) {
>
> Should this be Z_TYPE_P(tmp) == IS_STRING, rather than not IS_NULL? From
> context, it's obvious that the only two possible zval types for tmp are
> IS_STRING and IS_NULL, but this strikes me as a potential gotcha if the
> function is changed down the track to return more native PHP types for
> numeric values.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/php/php-src/pull/193/files#r1594144>.
>
>
Another spacing issue.
Why toString()?
Are there some case where `locked` is not defined?
Sorry about that, wasn't meant to be in here.  I thought that was a useful feature at first, but didn't realize that old versions of assets might still be requested by cached pages, etc. I've since solved this problem within Capistrano, since invalidation depends on knowledge of previous releases: https://github.com/capistrano/capistrano/pull/281

So I've removed this 'cleaning' feature from the Rails 4 PR, as well as my gem, but forgot to remove it from this Rails 3 branch.
fix punctuation please :).
I like the idea of easy to read code. How about this?

    if token.present? && protect_against_forgery?
Not sure I follow. 

`*` is zero or more repetitions, which still makes the space optional, right?
Maybe it makes sense to marge into 3_2 as well? Then the defined? check would make more sense.
as below...
Cannot call `abort` here - this location is outside the cake.
Perhaps we can improve the wording on this one? To me (maybe only me!) it's not clear exactly what you mean when you say:

> To provide your own string interpolator, **create an implicit class against this one**

What you mean seems much more obvious if you say:

> To provide your own string interpolator, **create an implicit class which adds a method to `StringContext`**. For example:

But again- maybe that's just me...
L63 needs spaces around the ``=`` (PEP 8). L64 needs to get rid of white space.

You should figure out how to run the tests, the white space should cause a failure.
This should probably be uncommented, with d looping up to the last denominator for which cos(pi/d) automatically expand (10 or whatever it ends up being).
Correct all the spaces, please (i.e., one space after comma, minus sign, etc.).
Could you comment or change the variable names? It took me some time to understand that `lfree` are free variable in the limits and `efree` are free variables in the expressions.
If I remember, once you said there were some problems in circular import. Is it one of them?
How about rewriting this as
```Python
for i, arg in enumerate(expr.args):
    if isinstance(arg, B):
        first, b, tail = expr.args[:i], expr.args[i], expr.args[i+1:]
        return B(*[A(*(first + (arg,) + tail)) for arg in b.args])
```
?
is one -> if one
Missing spaces around `-`.
`expand_mul` will exactly help.

I was hoping that Poly embedding will remove `expand` at all.
No, I don't think we have anything that can represent them convincingly - in particular, ImmutableMatrix doesn't work either. This would be a good reason to add it.
Not good when the tag is completely empty, it sometimes causes problems with layout and in some cases with validation. Desirable way to add a method CHtml::tag this functional. Basically add either a space   or a comment <!--Tag-->
We need to add an explicit dependency on clojure.tools.macro. It isn't in the project.clj, and I don't even know where it is coming from.
Doesn't show solution if you're not logged in?  Is this solution behavior really what we want?
or mongod
only use the glossary reference the first time you use a term in a section.
it's also not really deciding?
open question: should we have included content here or not?
This looks like it should be `Project:About` - no leading '/' or `en` in the slug
Don't know if it is required or not, but I have always seen adding full ternary condition in phpBB, even if results are boolean values:

`($mode == 'searchuser' || ($mode == '' && $submit)) ? true : false,`
Function signature would be changed to phpbb_plupload $plupload = null so it stays optional
Please explain what happened here. How are permissions affecting time display?
This has been renamed to phpbb_notification_manager
No, it is correct. See http://www.w3schools.com/cssref/pr_dim_line-height.asp
Indeed, please just call it camel case.
I can't think of any files other than attachments in a vanilla phpBB. MODs/Extensions like a gallery might add files that can't be referred to as attachments?
I don't understand the comment here. Maybe it should just be removed.
guess you should change this->width, this->height here.
Today, with mustache, we can add "mojito-partial-addon" and call for ac.partial.compiler('foo') to get the stringify version of the template, and pass it to the client side if we want to use that there.

With handlebars, we should be able to do the exact same. Here is an example:

https://github.com/caridy/mojito-handlebars-demo/blob/master/mojits/HelloHandlebars/controller.server.js

The difference between mustache and handlebars at this point is that mustache produce a string that has to be processed at the client side by the mustache engine, while handlebars produce a Javascript code that represents a function, and by calling that function, it will return the rendered template.

But from the mojito point of view, they are the same thing, so, ac.partial should behave the same.
Yes, that's the problem with async, it's a virus :smile: -- if a low-level function is async that very often causes higher-level functions to need to be async as well.

I think the use of `readFileSync()` in a codepath that happens during runtime might have significant performance implications, especially with higher levels of concurrency.  Yeah, I do think that means that `compiler()` and `precompile()` should by async.  (And yeah, that's outside the scope of this pull request.)

I'm not especially worried about multiple reads of the same file happening at the same time.  Yeah, we should probably optimize this, but also very often the operating system filesystem has caching built in, and might also be doing collapsing of multiple calls.  (Similarly, we should be doing same-request collapsing in .client. so that we're not making multiple HTTP requests for the same template at the same time.  It's more important here in fact, because the Y.io library is very likely *not* doing same-request collapsing.)
if 'debug' is false, logLevel is irrelevant. In this particular case, the translation from "log" to yui->config will be having debug: false, without logLevel.
probably want to change this to "3.0 (native)" to be proper
Слишком длинная строка
В общем я поговорил с Садистом про неактивные баннеры. Не нужно их удалять. Они должны помечаться как-нибудь, чтобы было понятно что они неактивные. Плюс к этому, когда редактируешь какие баннеры показывать у конкетного баннерного места, должно быть видно что такой-то баннер хоть и включен на поках, не будет показываться из-за истекших просмотров. Не нужно объединять состояния удаления баннера и истечения просмотров.
Also incomplete
Should these be properly named, like "Find All References" like some of the value above, or just the key?
I dont know why I filter by boolean >_>
Got that
i think this check and call to ```DestroyKeMapButton``` is unnecessary, if you're going to overwrite ```keyMap.keyName```'s value a few lines below anyway.
add the next onload here as an if statement, and take out your 600 ms buffer.
This is normally done as ```import gribapi"
Displaying "memset_0=False" seems confusing and unnecessary for me. I think we should only add "{memset_0}" or "{memset_0=True}" when self.memset_0 is True, and keep only "GpuAlloc" when it is False.
does not work in a way I expected
column_count == 1. No need to loop.
m_dummy is only used in one function -- replace it with a local variable.
m_tuple is set but never used -- remove it.
This should be accomplished by the removed fetchAndResetBaseline() call above. No?
I'm not sure this change is equivalent ... because I can't figure out the old code.  I think I've convinced my self the comment about "continue until..." was misleading.   However, the old code added all inExprs to out (wonderful naming). The new code adds adds AND sub-trees to in and non-AND sub-trees to out, and always returns out.
http://pypi.python.org/pypi/dj.choices
Should the "all" target include deps as well?

all: deps compile
Azok a szóközök kellenek oda...
Didn't mean to include this.
`return None`
What does `ios` stand for?
@rwldrn I updated the unit tests to conform to the style guide.
How many iterations would you make through here, and would the starting value get a lot better after dozens of iterations? If the infinite loop is the exception then a small failsafe loop counter might be easier to deal with and a heck of a lot less mysterious.
@jaubourg, can you tell me why we `.resolveWith( deferred )` but `.notifyWith( promise )`? Standardizing on one could really simplify the synchronization code.
It looks like you have converted the code from being something that tries to support multiple data types to a function that only supports int64.  Given that this is the only interface created in the SQL layer I don't see any compelling reasons against this.  But it seems strange to be calling functions such as get_fn_expr_argtype in some places that make no assumptions about received datatype, and then to assume that the datatype is int64 in other places.
creiterion -> criterion
Commented out because...?

Would it make sense to have a compile option to enable heavier weight sanity checks?
This is a GP backend-specific question. Hitoshi/Caleb, could you please clarify?
For opening {, fixed. But for indenting, why do not we used tab? GPDB requires indenting with tab. Madlib requires indenting with 4 spaces. One group, two standard, is there any good reason?
I wonder if materializing the training source with unnest would be more efficient if you use it multiple times.
I don't think there is a way to make this work with PostgreSQL. The transition state is of type bytea, so the final function cannot return anytype...
Apply_tolerance is now being called in check_expectation, but is instead modifying a querydict. It makes more sense to do so instead of modifiying the original query_string.
Unwanted whitespace change here
These 4 lines can just be

```rb
bin.install 'tal'
man1.install 'tal.1'
```
Is head a better name? I considered it, but figured that since head is a moving target it is an incorrect name pretty quickly. head_when_built would be better. Though maybe my understanding of what "head" is is wrong.
The first edit needed - this stops the page class from trying to compare things like 'index.html' with 'index'
Why is there a new line? - This looks ugly.
A supprimer , c'est du debug ;)
This doesn't stop looking after it finds a match, so it's possible to route the same keyword to multiple applications. Is this desired behaviour?
I guess we might as well just construct the result tuples in the correct order straight away?
Return a short link to the submission.
Формально <a href="http://www.gramota.ru/spravka/rules/?rub=zap&text=19_26">правила</a> допускают оба варианта, и положение запятой определяется смыслом предложения.

Посмотрел примеры, когда сложный союз предлагается разделить на две части запятой, там ярко выражен акцент на причинно-следственно связи, и при произношении хочется выделить интонацией первую часть союза и сделать паузу перед второй:
> Не вспомнил он этого намерения только потому, что на набережной и возле самой воды было слишком много народа.
> Всю свою теорию Раскольников построил исключительно для того, чтобы оправдать в собственных глазах мысль о быстрой и легкой наживе.

В нашем случае союз не играет столь важной роли, и я бы руководствовался общим правилом:
> Когда придаточное предложение стоит после главного, запятая при сложных подчинительных союзах (потому что, ввиду того что, благодаря тому что и т. п.), как и при простых союзах, ставится один раз – перед союзом.

Вообще текущий вариант перегружен каскадом придаточных предложений, следующих за главным. Предлагаю избавиться от запятых, упростив его так:
> При выходе из строя серверного диска можно использовать любой из клонов на любом из клиентов для восстановления сервера в то состояние, в котором он находился в момент клонирования (...)
I'm not exactly sure why, but this fails to do what I intended it to do. The idea is that if someone copy-pastes a single identifier, but accidentally grabs an empty line preceding or following (or both) the identifier, we clean it up and keep everything in single line mode. This is also supposed to remove any trailing and leading spaces/tabs. While the newline characters are removed, the leading and trailing spaces remain.

For example, pasting the following string into the textbox:
```

      1234567890  

```

should only display `1234567890`, but I am also seeing the spaces before and after the numbers.
Huh, I guess it can. I could've sworn I got compiler errors other times I've tried to use scoped mixins.
good catch. i missed this one
Nit: this comment and the ones below it should have the "*"s indented one more space, like the other comments in this file.
Typo: "focusable"
Yeah, just thinking about typing perf. :)
With Glenn's work for JS inline editors we can. I was keeping this around to get the context around the regex results. I'll remove it.
why is this needed? .$enable does the same https://github.com/ajaxorg/cloud9/blob/master/plugins-client/cloud9.core/www/core/ext.js#L247
It is funny note as we don't support setting it during creation time
We said ... not do this in `get` method.
Reordered fields and added new line.
Organize imports. Import only modules.
Some of the keys here are wrong. The only underscored one is update_type -- the rest are camel case (yay consistency!): https://github.com/mozilla/balrog/blob/master/auslib/db.py#L598
Needs space padding before and after the round brackets
Add some spacing around the arg here, please.
Is there an issue somehow with Mongo and IE9?
Add a "Popcorn Maker" string to this warning.
Yea, guess we'll see when I rebase
space after <code>if</code> and before opening brace
ok - trying. textwrangler doesn't seem to do this in the same way that whatever whomever authored this does.
Ok, could do that too
OK, that's awesome.  Maybe leave a comment to that effect above this line.
Actually, if this is declared properly for body, all of these font-family declarations can be removed.
I'm not sure which is best but I kinda like these things in the settings.py files
Then what's the point of rest of the contents, in the `<body>`?
the Publisher and one for the response port. Thus, large installations may, upon
There are no zip artifacts for these guys, these aren't official deployed artifacts by the tomcat group, these are things that have been deployed in rso's thirdparty repository.

https://repository.sonatype.org/content/repositories/third-party/org/apache/tomcat/apache-tomcat/
Could be "Successfully upvoted post" to be consistent with the method name
beshe ke esme famile adamo khob ba horofe kochik nanvisid khub onam yek jor tohine  :D
You're right, I missed something. I'm sure line 82 can be done with C++ strings too, but I'd say just leave it like this.
I was getting "access to undefined reference" errors earlier. I can't reproduce now. Either something changed, or the new type methods are masking it. I can try taking it back out.
Can parent_repo be nulll ??
Is this displaying the system group itself or the history of events for that system group?
This is the convention we've followed for what looks like all of the glue layer methods that talk to a bakend engine.  Look at just about anything in  /app/models/glue/   and you'll see the same.  I'm not 100% sure if its needed or not, but i think we'd want to do a full overview of the app and change it everywhere in the glue layer if its not.  I think the right thing to do is follow the convention to keep the code base consistent.
removed dir path output
I don't know if matador wants to add this shim.  We need a better way of sharing code between the client and the server.

Since the client's not using this code yet, we can punt on it and just make it standard node-ish
I like it being explicit in this case. Thanks for the recommendation nonetheless!
@davidfowl As before - I haven't touched routing at all
I'm not sure I understand the point of "levels" - are they just constants to simplify setting up a debug/info/warn/error heirarchy, or is there some meaning to the numbers themselves?
`table.meta` is an OrderedDict, so `update` should do the trick.
This is the hard-limit for outbound connections, right? It's possible to disable them with -maxconnections=0 or negative vaules.
But that line could as well look like this, right?

int nMaxOutboundConnections = min(MAX_OUTBOUND_CONNECTIONS, (int)GetArg("-maxconnections", MAX_OUTBOUND_CONNECTIONS));
Perhaps you're using spaces? CakePHP uses tabs for indentation.
That's now in the common bit here - https://github.com/davidfowl/JabbR/pull/490/files#diff-3.

The thinking is that download parameter can be applied to any api call, so I wanted to move it out of any specific controllers, to keep them cleaner.
I'm a little anti-comments. I should probably get over it, but this is a good example of an unecessary comment. Specs already have a nice string name. If your name isn't good enough that you feel the need to add a comment, try improving the name instead. Perhaps it should be: it 'assigns correct reply_to'.
If you're hiding this using jquery then we don't need to add the hidden class.
the other thing I don't like about using action_pipe here, is that it's possible that the action pipe is None. This means that the snapshot() method can't be used for debugging images without a pipe. None of the widgets require action pipe for anything besides sending messages, so one could conceivably grab a snapshot() of the widget tree to persist the ui state to disk, and restore it at a later time.
All of this refreshing code can be replaced with a single `Signal`.
Won't this disable ALL swipes?
Nitpicking: the last bit is the hex digest of a hash, so it should only be `a-f\d`. But that's not really going to make any difference.
Ah crap, I will fix that...

On Sun, Aug 5, 2012 at 12:01 PM, Bussonnier Matthias
<reply@reply.github.com>
wrote:
>> +      }
>> +     ],
>> +     "prompt_number": 2
>> +    },
>> +    {
>> +     "cell_type": "markdown",
>> +     "metadata": {},
>> +     "source": [
>> +      "Or these URLS can be relative to the Notebooks working directory:"
>> +     ]
>> +    },
>> +    {
>> +     "cell_type": "code",
>> +     "collapsed": false,
>> +     "input": [
>> +      "Mesh('files/data/smallsurface.vtk', opacity=0.7, magicmode=True, bgcolor='#FFF')"
>
> why `data/?`  I does not work with data for me.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ipython/ipython/pull/2258/files#r1308450



-- 
Brian E. Granger
Cal Poly State University, San Luis Obispo
bgranger@calpoly.edu and ellisonbg@gmail.com
Those would be good 'why not' reasons. ;)

Let me think about it tomorrow and get back to you. I just know that having a setup method really bothers me.
was this line removed on purpose?
「開発者としてサインイン」とかにしましょうか><
:+1: Let me know if you need a hand with that.
typo- auto-generated
Good catch, thanks! Will fix in the next push
Hurray for unit tests! \o/
Should I also fix the other similar calls on lines 520, 595, 638, 668, 681, 696, 705, 731, 749, 768, 813, & 829 ?
Sweet. I remember hearing that a while ago but I think that's when they introduced `.live`.
os.environ.get('RACKSPACE_SERVICENET) != None ?
ça ne fonctionne pas: dans la version actuelle de webassets il faut que le répertoire avec les fichiers générés soit dans le même répertoire que les sources (le répertoire `c2cstats/static/`)
I agree. Out of scope for this but we should fix in a followup.
should also be `AcceptHeaderItem[]`
Yes, I just fixed that.
Yes, you can skip the test in `setUpBeforeClass`.

[markTestSkipped](https://github.com/sebastianbergmann/phpunit/blob/3.6/PHPUnit/Framework/Assert.php#L2594) is a static method.
@stloyd: Thanks, I didn't do a full review. I've just seen that also `addViolationAtSubPath()` is deprecated, but have missed completely the use `addViolationAt()` instead statement...
Here is a diagram of the time it takes to match each route 50K times:

![diagram](https://imageshack.us/a/img708/96/outvt.png)
The args not needed, good catch. Will remove.
No, it's cosmetic change. User can enter any country in editable checkbox.
alan inja bayad to khoroji beshe `div.backimg div.slideshow`
ke mondam chera nemishe
vaghean emrooz alaki hodode 1.5 vaghtam gerefte shod sare select kardane in stylus
不仅是本科生的论文要求，研究生的也要求是表内文字为5号宋体。
r.c() and r.begin() starts at the came position. But this has to be documented (see #398)
To avoid a warning, we can move "return OUTSIDE;" outside the if statement.

"/home/travis/builds/DGtal-team/DGtal/src/DGtal/shapes/EuclideanShapesDecorator.h:163:5: warning: 
      control may reach end of non-void function [-Wreturn-type]
    }"
Die urspruengliche Landingapge gibt es nun nicht mehr.
```self.origin``` is essential, used in ```self.retrieve`` to flip the image.
No, this was a mistake on my part. It is right to be player.
the original exception shows the detail log for debugging,  the HectorTransportException is thrown for hector client to catch it and align with the existing hector code style.
div?
Почему не в один partial: ```_form.html.erb```?
No prints in the test code, please
@Sunight @mockee MBProgressHUD的简单用法就是在合适的时机show和hide就好了，后面定制起来其实很简单的。应该先用原生的`UIActivityIndicatorView `来做这件事，然后再看第三方库，不过这个真的挺简单好用就先算啦，后面还会有用到`UIActivityIndicatorView`的很多地方，扣细节时`UIActivityIndicatorView `还是挺常用的。
I'd put a space after `landlord:`.
I think we need a type field to be able to connect our record classes to the actual NDEF records so the class on itself is not sufficient.
However we could define a table with the types and assign them the record classes, similar as it was before ( type => XXXRecord.class)

Let me think about it a little bit more ;)
Can we make this exception derive from RuntimeException?
IllegalArgumentException has its own meaning.
Implemented _moveToTop in 3cbca20
No we  don't use the values in the monitor to determine which hyperparameters to select as part of the cross-validation algorithm.
See previous comment. Most observers in Magento 2 return the observer instance, so I thought that might be the new recommended practice.
а нельзя этот HTMLPathPrefix как-нибудь запихать в класс тега через Autowired или что-то такое? Или оно спрингом совсем не управляется?
long story short:

```
@import url(homescreen.en-US.properties)
```

is enough for the `locales.ini` file, and one `homescreen.en-US.properties` file is all we need to get the localization process started.
please move this section above the `Improve Firefox OS` section