labofsahil/github-event-dataset-2012 huggingface dataset:
Well... If we want to use the cache system of AA, we have to rewrite all the parsing mechanism of CodeModel. I wanted to subclass JCodeModel and wrap ref method but JCodeModel is final. If you have a better idea...
Why not use ```user```?
The phpdoc way to document a fluent interface is ``@return self``, not ``@return $this``
```None``` is redundant. Same below.
`self.process_media_content` is gone along with the Picasa feed functionality, as it did not support hash digests. You should no longer be calling it.
Also did not know this was a package.. slick!
Seems like this line is used over and over wihtout any changes, lets make it into a method just to clean it up.
```ruby
matcher = get_matcher(:attr)
````
I guess this test can even completely disappear as in case of 'inrepo', the branch is taken from the changeset itself later on ...
Doh, you're right! Nevermind me :-)
Sorry, that is residues from previus version. 10x for attention.
Would be nice to make the install command a code block, just to make it more visible.
Does this have to be called manually to load the user's profile? Remember that when you login the profile is sent back in the response so you can just pull it out of there and deserialize it.
I choose the array of double for the estimated probability, because #estimated probability is equal to #class.
Could you explain more on this
Use login manager. Can this run in production?
@christianv Ah, roger that.
I think there's a double tab here.
You can add a TODO, even if you don't address this in this pull request
No double quotes here. Is this intentional?
*cough*, shouldn't have made me look :-)

your indentation is off here
I was pondering on using the other attributes to exclude any obselete accounts. But I finally tought it was best to register any cpassword, just in case.
So a structure would have been useful for better validation.
Hm, imho the way I did it is more readable/understandable in the least squares context. Probably agaon a matter of taste... ;-)
@nerdsRob might it be that the delegate is already called by the things happening in the init?
what does ? mean, if it's not used or invalid we can delete it
We should be using the Color object here and everywhere else we use colors.
I'm pretty sure we use curly braces even for single line blocks.  If enough people feel otherwise, we could loosen it up.
`hours, minutes, seconds, milliseconds` are plural here, but inside `GregorianDate` they are singular, `hour, minute, second, millisecond`.  We should pick one and be consistent with `year(s) month(s) day(s)`.
Why not provide sensible defaults like we do for `Cartesian3`?  Perhaps all zeros.  Otherwise we are forcing code like `result = new BoundingRectangle(1.0, 1.0, 1.0, 1.0);` in `expand` below.

Why can't `width` and `height` be zero?
Do we want to mention anything about improved performance for the `LabelCollection`?  We did the first time we added optimizations.
The guideline is to document all exceptions a function can throw even if some exceptions are thrown by functions deeper in the call stack.  I agree that the situation kinda sucks.  @kring any ideas?  Just skip the validation, and change the doc to expect "a validated extent?"
You're right, that would hit two birds with one stone.  However, I think we need to be careful insofar as there will be cases where we want a "just this one thing" deletion, and not a full cascaded deletion.  If the cascading is done DB-side, we may not be able to control that.  This will take some thought.  I think this is okay to merge, and we can refactor later, but let's explore that idea about a transaction here.
app.js / package.json
Jon513, If you could submit a pull request with that change I'd be happy to merge it in!
This was a merge conflict error. Will take care of it.
Default is true. Fixed.
Thank you for your diligence in removing TODOs that are no long TODOs :)
Your comment states that it defaults to the OS' temp folder but override it with /tmpa here; should line 22 be removed?
@DiegoLopesLima o ";" não é pra vc que esta programando certo é para pessoas que não estão, ou que esqueceram do seu ";"  Muitas vezes trabalhamos com vários códigos de terceiros, nesse caso, plugins, código antigo, código de outro programador, etc.

E caso de problema exatamente nisso, você não irá saber onde está o erro já que ele pode apontar o erro como no final do arquivo ou simplesmente encarar o seu plugin como parametro para outra função.
NVM, I must be tired given that line above
`auto p = typeid(T).init().ptr`
Unfortunately this is a bit confusing...

    LaggedFibonacciEngine!(stuff) x; // not seeded
    auto y = LaggedFibonacciEngine!(stuff)(); // seeded

Not sure whether this idiom will make it or not in the long run.
I meant the asXML() is the one that creates the tag I believe.
isStyle?
@sigmundch -- I dunno if we really need both kinds of tests for this? Seems like one of them would be enough. Maybe just an emitter_test since those are faster to run

@sqs -- thanks for fixing this :). It will be great to have data- working!
what exactly is this commented block for? Either put more information about the code line commented or remove it pls
``cliquer un lien`` is really weird. ``cliquer sur un lien`` is better IMO
Add:
```
input {
  margin-right: 0;
}
```
I support proper indentation! Awesome!
I woud feel a lot better about it if it were inside a if $config_cluster statement and if it took a backup first :)
We can remove the `q` parameter here I believe
Can we convert this use to multiple lines?
should this be options.id, if id is supposed to be the element id?
Rather than importing all these models separately, the pattern I'm using everywhere now is `from cc import model` and then all the models are available as e.g. `model.Product`, `model.Tag`, etc.
> As the comment says

Comments for strange looking code should be in the source code, not the pull comment.
more efficent:
 if (is.null(activity))
   return(NULL)
propertyValue(activity, "used")
See comment in 2610; need to decide if we still support JDK 5
We could easily detect whether an explicit scheduler was set by overriding the setter, setting a boolean and calling super.set...().

That way, we can throw the configuration error if trying to configure the global scheduler; not sure how we'd detect two or more delayers getting the same explicit TS, though, short of some ugly BFPP code looking for TSs that are injected into multiple Delayers.
Good point. It fixed the issue, but doesn't mean that it is the *right* fix. I'll dig deeper to see what other side-effects forking has.

Let's hold of on merging until I can answer this question.
well, an interface but this is also handled by ``:class:`` ^^
Too many gems! I thought the bootstrap-sass-rails or whatever was the right one. Oh well! Thanks to both of you. :)

On Aug 17, 2012, at 3:20 PM, Steven Harman <notifications@github.com> wrote:

> In lib/highgroove_command.rb:
> 
> > @@ -20,6 +20,8 @@ def new(name)
> >        append_to_file "Gemfile" do
> >          <<-EOF
> >  gem 'slim'
> > +gem 'slim-rails'
> > +gem 'twitter-bootstrap-rails'
> Actually, that's NOT the one you want. You want bootstrap-sass
> 
> —
> Reply to this email directly or view it on GitHub.
>
> It's not about errors, it's optimization.

I am saying that it is error prone. Doing timeouts manually can led you do pile of things executing at the same time, overriding state of each other (no locks in JavaScript), thus things breaking apart randomly. Also that arbitrary 40 there is a sign of this. Because, what if it gets more than 40 ms, then it breaks again to the user and it requests data twice. You need more concrete solutions if you really want to prevent duplicate downloading.

So yes, try queues. Or jQuery "futures" concept also looks interesting. You could define dataset to be future and others then block until it is available. Something like this. But have not used them much.

The issue with queues is that they are really queues, so you cannot remove one entry in the middle of the queue, you can clear the whole or pop one out. But you can store somewhere else some flags which tell if data is already available. (But make sure you don't use `fx` queue. Use your own.)

I use `.data()` on DOM objects. So if data is already there, then don't download it second time, for example.
@maks3w You don't use "an" in front of words beginning with "u" typically.
Well the idea is that an incomplete ReportDto will get passed in and then this class will finish it off, so I think reportBase makes that a little more obvious.
Best if use `$response->isOk()`
It was fixed in the other noted classes. I just missed it here and in MultiPointMShape. I've changed them to GetMeasure(index).
Same mutability issue here, too.  I'm going to stop commenting on it now.
Use a descriptive name here (this was a change from earlier versions of Mokol)
Thanks. Eliminated.
If I make it a property will it get nil-ed out?
Why have you removed support for the PHP magic functions?
It is not. Keep the text version, so that the client that only supports text won't fail.
Could you factorize a bit the code above ? Or at least put all the checking logic within a separated function ?
Love this decorator!  Wish we'd done it ages ago.
Could you please add this guy ( @ornicar ) to authors, probably at first place :) as first and main developer.
This method is pretty complex. Could you move a couple of these statements into well-named methods? E.G. 

```ruby
def set_attribute
  unless subject.send(@attribute.to_s)
    attribute_value = @test_value || "some_value"
    subject.send("#{@attribute}=", attribute_value)
  end
end
```
Android doesn't support this property right now
`disassociate`
no space before ?
Therefore always imagine you're part of a huger manual; which means: this part/section explains
Should we take the opportunity to adopt a standards-compliant versioning policy, like OSGi?
Wouldn't this generate the same error?
Why not keep return getImageUrl(DEFAULT_ICON_NAME); ? as it won't contain rootUrl
Minor issue with indention. Pretty sure this will cause a lint error.
Expect what?
The expression (== 0 (mod n divisor)) returns a boolean by it self.