labofsahil/github-event-dataset-2012 huggingface dataset:
But the name throws me off. What is *anything*? in what context? If this is supposed to be exposed to a user so that they can map this to something, "NextANything" is not meaningful, it is confusing. Next line? Next slide? Next line and then slide? 

The problem here is that you don't want to abstract it this way, the user needs to know that it is doing the next line(s) or next slide.

If I am mapping this to move forward with the next lines I would not expect to suddenly find myself on the next slide.
CS, имена переменных начинаются с  маленькой буквы
we should actually just change userdict for anons so its like {'is_authenticated': False} and that's it
Does Java allow you to specify "throws" even if it doesn't?
this helps the make a bit more
https://github.com/joomla/joomla-platform/pull/1519
I was thinking the same thing. I was not sure if there already was a good place to put it or I would have to create one.
Right, we don't
u_norm, du_norm
Where is this `TcpInputReceiverBehavior` defined?
Please move this to another Python module.
I know. Thanks.
MongoEngine already defines [signals for documents](http://mongoengine-odm.readthedocs.org/en/latest/guide/signals.html). Maybe this is what you are searching for here?
Oh, why load? jQuery proposes `ready`, which is before `load`. (Load means all dependencies are loaded, ready is when DOM is ready, which is much sooner.
`id` will not work. Use something else.
Index. I changed columnId to columnIndex.
One space too much, still.
*Returns provider's language.*
Is this ok ? 

#: .\account\templates\user\login.html.py:54
#, python-format
msgid "If you don't have an account <a href=\" %(the_url)s \"> click here to register.</a>"
msgstr ""
This is really no good and I did not understand the reason behind adding your nick to name space. Even more file location is not okay.
Ok, @Sharpie should I change it to `ENV.llvm if MacOS.lion?` ?
Don't we have this already?
Can you comment or remove that version variable assignment since it's not needed in this function anymore.
: instead ;
this is only used in the one place?
`\s` accepts a lot more than `' '`... are you sure you want that?
If you want, you can wrap this logic in a method assertSomething related to measure scale.
Should be gte excluding current.
oh, it is checked in walkPath
Yes, that was another case I had been thinking of. I am in favour of renaming linkedct to clinicaltrials or something similar for consistency with the principle. Does Miriam/Identifiers.org have a namespace assigned for either clinicaltrials.gov or linkedct.org?
Perhaps an ignorant question, but driven by curiosity - why did you chose to do an inner-class here when it is the only substance in <code>class BD():</code>?
Right on. Thanks for writing a repro case.
This looks a bit like a hack.
`contains` works here too.
Oops, never mind, I just realised that you're turning all slashes into underscores, precisely to avoid this problem :-) Not sure I like that way to turn URLs into paths, I have to think about it a bit more...
seems we leak pool_ here as well?
You can call this "planet", no need to save letters :)
Changed if/else to assign result to monitor_master
I think you could change dict into an `NSMutableDictionary` and add `-setObject:forKey:` calls for `uti` and `extension` instead. The others should have valid defaults since they're passed to `NSNumber`/`NSValue`.
I pushed a first try here:

https://github.com/VirgileFritsch/statsmodels/commit/1a1a14cb46db40621bd0cd5179c7a17b6ecfd7b6

As said in the commit description, the example I provided does not really show that there is an interest in having a large sample version yet. I will keep on trying because 1. my code may not be optimized for speed and 2. maybe I was just not able to construct the "good" case for "LTS large sample"  to beat "simple LTS".
Don't need a copy here.
if it's relevant (doesn't look performance critical): avoid copy by resetting changed element to original (done this way in numdiff)

xi = exog0[:,i]
..... loop
exog0[:,i] = xi   #reset
Never hardcode year. Make it programmatic, so that it updates automatically next year. ;-) But in our case this is unnecessary. Such line does not give any additional content to the site (copyright holds automatically, you do not need to write such lines to have copyright).
Why exposing the locks here? As I see, no implementor of RevertableConfiguration uses the lock, meaning, the locks being here is wrong decision.
Is there a reason you need to FORCE a configuration update with that true parameter?
This was removed by a previous pull request and shouldn't be included here.
P.S. I should say, in case of people not being able to read my mind, that the image above consists of a 4x4 array of ones in an image, placed directly into the `CorrFunc` as the underlying discrete correlation function, and then drawn.  Because I currently enforce 2-fold symmetry via (e.g. in a pseudo-code representation of `xValue`):
```
if y < 1:
   return xValue(x, y)
else:
   return xValue(-x, -y)
```
this means that the actual information we do have in the upper left quadrant is ignored.  I'll fix this using the glue-at-init strategy described above to reduce the amount of case switching that occurs in calls to `xValue` (which are at the heart of `draw`...)
I'm fine with that.  But definitely needs it own issue.
Since this is the first appearance of CCDNoise, perhaps it's worth explaining what's going on here: i.e., that Poisson noise is added with a default gain of 1 unless explicitly specified otherwise, that getting noise corresponding to sky + object requires us to explicitly add in the sky background as we do a few lines up, and that Gaussian read noise is automatically zero unless explicitly specified otherwise?
the way cookbook_file works it it looks for a file named "cli53" in the files/default subdirectory of this cookbook.
I think that it would be clearer with :
Assert that the actual object is <b>lenient</b> equals to given one by only comparing actual and other on the given "accepted" fields.

We should add an example here because it is not easy to understand what method does.
At least I think that is what this is for, not sure it is ever used.
I think what you want is `erlang:error(badarg)`.
Is this interface used? Does anything implement it?
i guess this is getting better with use of lock, and probably not too hard to fix (e.g. wrap getAllMachines ImmutableList.copyOf(machines) in a lock) ... in any case good to note it now
The return type of objectForInfoDictionaryKey() is objc.id. It is mistake?
Request parameter name is not set?(value annotation parameter)
Where does **B** figure in here?  This doesn't look right to me.
For the C-tors that don't have ISerializer and IRequestBalancer param, specify in the C#-doc which default serializers and balancer you'll use by default. The API user cannot guess that.
an integer is perfectly valid too?
We magically translate that into w=N
Seems like this method actually belongs to FeatureConfig class, since the Config module might eventually contain other classes or modules with different behavior (e.g. a ComplexConfig that allows for values other than true/false, or default values).

But if someone specifically needs a FeatureConfig, then he can FeatureConfig.read('my_file.yml')
{ should go on new line and missing spaces around variables :)
SOunds good .. the only I think I want to understand from you before being ready to test is what you want me to instead of checkpoint ....
Oh, I used django's `SortedDict` so I think it should be fine.
i removed it but now there is a main.css and a checkin.css. Are we still removing them or are we putting the css files back into the stylesheets folder?
But non one can comment on a comment. This should not inherit from Commentable i think.
Where have you found 'already existing' ones? There are Pump-Vents already, not Plain-Vents...
we are now using sha1 or sha256 instead of md5.
Надо печать человеческий usage.
<code>s/email/email_address/</code>?
what about other traversable classes ?
Should this be 4.4-DEV?
``X_``
@hab278 do you remember what this refers to?
s/Manhout/Mahout
This generates a warning
No need in this declaration.
I admire your attention to obscure platforms, but don't feel obligated to write an fgetln. I'm totally fine with no OpenSolaris support.
That's one of the input files, I think, not a job temporary...
The directory is named as such and it would be up to the doc team if they would like to change them.
very minor point, but I'm not keen on the //"HpccInternal" comment
if we decide to change the definition in the called routine, comments like this will likely go out of sync.
should only walk the children if body == expr;
Yes, just use `file_date = datetime.datetime.utcnow()`
"[...] try again later [comma] of course."
I've searched for it, and it's not documented anywhere. I don't think so, however I'm not sure.
I think this was supposed to be``repourl``?  I'll fix that up.
I think this added complexity and memory for caching is not worth it.
@christianv That looks like a stray trailing html comment.
`(= name (:name x))` already returns a boolean
You could also just return `(= (width rectangle) (height rectangle))`.
as said i think this should be a class property and i think it would make sense to key it as follows:
    $supportedHeaders = array('etag' => true, 'last_modified' => true, 'max_age' => true, 's_maxage' => true, 'private' => true, 'public' => true);

then the filtering could be changed to:
    $controls = array_intersect_key($options['controls'], $supportedHeaders);
    $extraControls = array_diff_ukey($options['controls'], $supportedHeaders, function($key1, $key2) {
        if ($key1 == $key2) {
            return 0;
        }
        if ($key1 > $key2) {
            return 1;
        }
        return -1;
    });

with the current code i saw that the filtering wasn't working properly if i use for example:
    $options = array('controls' => array('must_revalidate' => true));
We do want to e-mail support; e-mailing developers directly is not really an appropriate way to manage this issue (or other issues, for that matter).
Rails can't mass-assign other attributes, when adding currency from admin page, so all attributes should be accessible
this could potentially be simplified as:

  device_config_hash = {$name => {PARAMS}}

  create_resources("swfit::storage""${fstype}", $device_config_hash)
Now that this is pure Python, (I think) preallocating the array is unnecessary. You can just append `thresh_image =` to each of the filter calls below (and remove `output=thresh_image`).
This is already tested by the `map_coordinates` function.
The problem is that different situations demand different timeout values. In this case, we need the timeout to be fast enough that it doesn't irritate the user when they're waiting for completion results, that's something we can nail down ahead of time (probably closer to 100 or 200 ms). 
In any case, this is something that will need some playing with.
As above, should follow the standard layout for readmes.
I think this needs a “with” between “it” and “brackets” so that it reads

> … surround it with brackets by typing …
From an API standpoint, it probably makes sense to have an addImage(image) as well, because a lot of the time, people may only be adding a single image so this lets you avoid lots of pointless array allocations.  You can then put most of your code in addImage and have addImages just iterate and call it (unless there is significant optimization to dealing with them as a group).
`context` no longer appears to be used, it should probably be removed from the param list and all code that passes it in.
I prefer your solution because it is concise.  From a performance point of view, I expect them to be identical if the compiler does any optimizations at all.
I see the `_textureAtlas` check in `update` now.
why would you need to check zoom < minimumZoomScale in maximumZoomScale setter ?
Do not see code where width is undefined and can not be calculated from pins (follow auto behavior) or auto where the default auto behavior if fill. Are these flags set up somewhere else?
we shouldn't really require the flag (even as part of a pair argument) to remove the dialog.  Only the dialog should really be needed.
why newValue is ignored ?
You can set properties on this object, but cannot invoke any Activity methods on it.
In this case, Timer is compiling down to window.setTimeout in the browser.

The benefit is a simpler "surface area" to teach developers. Another benefit might be "new Timer()" looks better than "window.setTimeout". A developer that is new to web programming (aka our audience) doesn't know what window is, but could infer what "new Timer()" is.
Can you get rid of this commented out code, or is it somehow magical?
This is the crap I changed.
Basic auth is a pain. Anyways, I did this last week and dug this out for you. Mix in to your package object.

```````````````
trait Strings {
  implicit def string2SplitAtFirst(s: String) = new {
    def splitAtFirst(regex: String): (String, String) = s.split(regex) match {
      case Array(_) => (s -> "")
      case Array(prefix, rest) => (prefix -> rest)
      case splits =>
        splits(0) -> s.replaceFirst(splits(0), "").replaceFirst(regex, "")
    }
  }

  implicit def string2kv(s: String) = new {
    def kv(delimiter: String): (String, String) = {
      val (k, v) = s.splitAtFirst(delimiter)
      k.trim -> v.trim
    }
  }
}
```````````````
And yes, nil means it will be left out entirely.
можно сразу сконструировать копию new vector<T>(neighbours).
Don't free it. it's const.
anyway, even if it weren't const, avoid side effect in functions.
If `R` has `@disable this();`, `R r;` is not allowed.
To solve the problem, I had changed to `R r = void;`, but it has still problem if `R` is const  or immutable type.
(`const`/`immutable` range is almost meaningless, but it is syntactically allowed). Then, use `R.init` is better.
Filter accepts only template predicates now.
`popFront` is _not_ supposed to be a property. The fact that `isBidirectionalRange` on a type which has `popFront` as a property would compile with `-property` is a bug.
There are tabs in the comments. Please replace them with spaces.
Wait. This is the unittest block for verifying the example (note the comment). It should include _exactly_ what's in the example in the documentation and _only_ what's in the example. Any further unit tests need to go in a separate unittest block. And don't put all of this in the example as your fix. That would be _way_ too much.

But it looks like you did this with several unittest blocks. Please fix this and then rebase the code so that there aren't so many commits for these changes.
juast => just
Nah OK, I misinterpreted.
The extension that we need to deactivate is on this dependency. I put a note about it.
s/There/This/ I think
All of these should move back to the clients repository file, IMO.
because stringstream + unsigned char has problem.. overflow or something I think

http://stackoverflow.com/questions/3356661/getting-a-byte-value-using-stringstream
I'll leave it up to release engineering, but yeah I agree.

The point is that we _could_ if we want to.
Additionally, the correct format for your repo URL is `https://github.com/hdemirchian/CSSFormat` (no `.git` on the end).
It seems like this should happen dynamically whenever a collection is upserted, it seems dangerous to depend on it existing at load-time? Also hard for local dev, and/or when collections are installed dynamically someday?
You are not relying on the view response listener here as you are using the ViewHandler explicitly (you cannot rely on the ViewResponseListener btw)
if you use GraphDatabaseAPI here as literal you don't have to do Class.forName in other places
Nit: The preferred number of characters per line is `80-85` this could ease and enhance the code readability.
Maybe a better variable naming other than method ? seems confusing with _method ?
Same for above
InlineModelAdmin is not used in this file. Why this import?
drop the `"."` , please stop using this hack, it's not needed (if either files or dirs are non-empty, it's fine)
Закомментированный код надо выпиливать
could you use ``&&`` instead of nested if ?
The JMSSerializerExtensionTest is just doing some basic tests to ensure that all the different classes are wired correctly together by the DI container. Can you move these more fine-grained tests to the BaseSerializerTest?
Surrey, actually I never used xml configuration in such a way, do you know how it should look like?
"[...] accidental injection of *escape* characters"
s/escape/extra/ ?
Seems to be fixed in my local branch. I'll keep an eye out for this.
Can remove the `type` attribute here
Moved breadcrumb rendering function here. Quite long, but lots of comments and whitespaces. One loop for filters and one for the query
Either...

A boolean flag; indicates

or

A boolean flag indicating



select operation ~~gives~~ is expected to return a single result
Hey Chris,

Are you still working on this module actively?  I'm at a different company (MaestroDev) and I'd like to help you wrap this up if you're not done yet.  We are using postgresql 9.0 on CentOS.

Etienne

On Jun 15, 2012, at 5:39 PM, Chris Price <reply@reply.github.com> wrote:

>>   # We must set a "listen_addresses" line in the postgresql.conf if we
>>   #  want to allow any connections from remote hosts.
>> -  file_line { 'postgresql.conf':
>> -    path        => $postgresql_conf_path,
>> -    match       => '^listen_addresses\s*=.*$',
>> -    line        => "listen_addresses = '${listen_addresses}'",
>> -    notify      => Service['postgresqld'],
>> -  }
> 
> If we do templates for postgres.conf, then I like the idea of having a separate template for postgres 8 vs postgres 9.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/cprice-puppet/puppet-postgresql/pull/2/files#r996699
String test with eql? safer than ==
Coding Java without it feels like using chopsticks without thumbs to me :D
"для помещения данных строки в целевой объект" , вообще слово строка довольно двусмысленно, тип данных и набор данных, возможно стоит использовать слово ряд?
что-нибудь вроде "для помещения данных ряда в целевой объект"
This is an example of an early return:

```
def average(objects)
  return 0 if objects.length == 0
  objects.inject(:+) / objects.length
end
```

The return causes an early exit from the method. In Ruby, you do not need to add a "return" at the end of a method. It's implicity and you don't want to break this convention.
Looking at it now, I don't thing the target directory should be removed to avoid overwriting existing files.
Это в стиле кода фреймворка: https://github.com/yiisoft/yii/wiki/Core-framework-code-style, пункт 4.5. Но вообще, нормально и так и так. Просто должно быть в пределах рецепта единообразно.
By the way, i just realize that i am forcing sonata_type_boolean on a boolean field (doctrine) instance of the default on
same as before ;)
Can you please move this logic to the "raise_error_if_needed" method? I want this method to be responsible for decisions on when to raise an error and when not.
you do not need this property here, because it is provided by BaseOpenmrsObject.
can you fix the formatting here?
Unfortunately, we have to keep these lines.
As the motion files are distributed inside the OS applications directory, trying to write in this file can imply some issues.
If I remember well it was problematic on Windows mainly.

Could you revert this modification and ask Chase to modify the Framework accordingly?
I preferred using the __setattr__ method of Response, because that its more or less the *official* accessor. 
https://docs.djangoproject.com/en/dev/ref/request-response/#setting-headers

Thats the thing that wont change. The _headers attribute.. who knows.  And because django and django rest framework are totally different *products* i'd prefere using this *official way* - using the
```
for key, value in headers.iteritems(): set those
```
huh, that's a weird way to write this, creating a list then throwing it away, just for the side effect. I think you should use

    for p in patches:
        p.start()
Note include.js is hot-linked here
Better to wrap the table within the form tags.  I don't think it's valid HTML to have a form element within a table element.
This causes compile errors, because you changed the method name to drawOpaque() !
> yes though in Java EE the order will be defined by the salient number and we will effectively be the last unless the user add its own purposely behind.

Can one already read somewhere how this is done?
should have newlines between if's for readability.

what are these special cases?

comments would help.
@hferentschik well there are two styles really within this section, the action style and the feature name style. I tend to prefer the action style, it fits the how-to approach and flow this section has.

I never liked documentations built upon code artifacts (like `@Indexed`):

- JavaDocs are the perfect tool for that
- it gives no clue about how to achieve things in a reference documentation unless 
   you already know what the feature does (which defeats the purpose)
case 'content':
    default:
        $root = apply_filters("adp_root_{$root}", WP_CONTENT_DIR);

Probably a good idea.  Makes it more flexible.
Probably shouldn't be public
Remove the dsd.getUuid() == null in this if clause
Is there a reason we can't just set the paint alpha in TiBackgroundDrawable.setAlpha()?
Shouldn't this line be below the next one so that 'PP' is matched before 'P'?
I guess it's not possible to transfert man hours to days and to months, since everybody will have different preferences so i could define a class ManTime or PersonTime with an int and an Enum (hours, days, weeks, months, years)
Can we rename `MockEnsureClusterURLStage` to `MockStage`?
The factories set attributes including those that are created created after the object has saved.  

The tests in some places require less attributes than those set by the factory.  Rather than manipulate the attributes of an object generated via the factory I decided to explicitly define the request params.  It also makes for easier reading.
I think it's a good idea to do this either way. It's good practice to check that an object is defined before trying to access its properties.
I'm guessing like `if (newElement.is("input")) {`
A space after the if as well.
Masz racje.
.planets already represents the getter method