labofsahil/github-event-dataset-2012 huggingface dataset:
For your changes and the new files you created, please use the unmodified license header found in other source files. I will make sure that you get attributed as a committer to musicbrainz-data (changelog + a new section in the readme called 'contributors').
You probably want to initialize this to `[]` so that the return type is consistent whether it succeeded or not (i.e. so the caller doesn't need to whether the return value is set).
The resize use case is easy and and passing the last arguments is not a problem techincally, but think about a method responding to some IO ( e.g. keyboard and mouse ) where multiple keys can be pushed and released, or the mouse moved and clicked, ... between two actual call of the method: You can not pass the last argument in this case. You need to use a state variable.
No space between type and *.
This should be `const std::vector<Light> &GetLights() const`
You need "m_randomPortEnabled = false;" here probably.
This should be 1.3.0.
I also like the Survival abstraction but maybe not from a user perspective. I think it could make sense to have a SurvivalData class or something of the sort similar to what we have to TimeSeries. It's been pretty useful there. Will think some more. I need to play around and get back into this, but I think it looks like a pretty good start and not far from being polished.
Thanks Mike, I'm sorry!
I see.  That makes sense.
this lines is not necessary
*ByKey fields are transient, so don't need to call fromPersistedMap. The postDeserialize should have dealt with that.
il faut bien que quelqu'un fasse la distribution non? meme si le modele de distribution doit changer
Imo this explicit type check is wrong and should be replaced with a type cast.
Why would the string "1" be wrong?
Can you move all of this into an external stylesheet in `styles/whatever.css`?
@mathiasbynens yeah, you're right. I guess I'm experimenting with using threequals for `undefined` comparisons and what not. :)
Additionally, the actual lines in question here don't necessarily make sense for the Windows build. The `preparation` software is force-included by omnibus-ruby as a build dependency for every every software configuration, and while that has served us well on the Unix-like builds, we may need to add some functionality to the Omnibus Software DSL to define a set of directories that should be present for each piece of software.
I wish! Unfortunately `detect` returns the element that matched, and not the result of the block.
idea was wrt is build automatically a separate step might be confusing
If you keep the scope, move this to Line 2
all if statements are bracketed
The comments out lines up there could go... Actually, I was briefly considering putting isAlnum etc. into namespace Common. IMHO there are some good reasons for doing that, but the drawback is that one has to add the Common:: prefix everywhere this is used. Anyway, I'll be happy to change it to use the namespace if you prefer it, but only if the patch has any chances of being merged :).
I would've but I needed to take some stuff out of the template so its a little bit custom actually, me and @christianchristensen were discussing creating a new public repo for xsl templates for sportsxml, I'm gonna create that and then we can use them as a library.
I only tested it on a couple of requirements files yesterday, its possible I missed something...
Same situation as for the bootloader.
I assume that whenever you deleted/renamed a method you verified it wasn't being called elsewhere (pretty sure is true for this one on malawi/tz end)
А что нужно использовать взамен?
В таких случаях неправильно делать try..catch. Надо решать с помощью правильных условий.
The issue is whether you clean up just the things that you "dirtied" in the test, or go to the effort of assembling the "allFeatures" record which in the end will clean up more than it needs
and locators should pretty much never be in tests, just in page objects.
`if (!($fileToInclude = $annotation->getFileToInclude())) {` 
or
`if (false === ($fileToInclude = $annotation->getFileToInclude())) {` 
or
`if (empty($fileToInclude = $annotation->getFileToInclude())) {` 
look cleaner to me
Reverted all changes to release notes.
I would rather says something like "there is a probabilistic interpretation of nn"
Do you need to specify "eigen_solver='auto'", it seems to me that it is the default argument.
`s/bounds_n/ngram_range/`
Indeed there is already a smoke test for cloning.

https://github.com/scikit-learn/scikit-learn/blob/master/sklearn/tests/test_common.py#L55
Investigate reserved word clashing with "and" and "or" fns.

_and
_or
On 10/09/2012 07:21, serge-sans-paille wrote:
>
> In pythran/pythonic++/pythonic++.h:
>
> > @@ -13,7 +14,7 @@
> >  #include "core/tuple.h"
> >  #include "core/intrinsics.h"
> >  #include "core/io.h"
> > -#include "core/generator.h"
> > +#include "core/exceptions.h"
>
> Why #including exceptions twice ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/serge-sans-paille/pythran/pull/12/files#r1563252>.
>
Oups
Hmmmm, I am not super sure I understand what exactly you're asking. Would something like this work?

```ruby
dogs = [Dogs.new('Bingo'), Dogs.new('Doodle')]

5.times do
  dogs.each { |dog| dog.pet if rand > 0.5 }
end

dogs = dogs.sort { |dog| dog.petted }

puts "#{dog.name} was petted most often and is quite a happy chap"

dogs.each do |dog|
  puts "#{dog.name} was petted #{dog.petted} times"
end
```

Probably not ... I'll give another comment further below.
Wow, thank you for all the details and guidance! I will try this.  Wheeeee *makes a handstand*
It would be great to get this in.

Death to autogenerated page variants!
What if I have a single cache, but I'm download a file of the same name and extension from two differernt paths? Won't they overwrite eachother?
What about the case where I have no filename in the URL, but it is still actually a file download?
Why do we care about the extension? I'd rather us hash the URL and use that as the cached file name.
```
Exemple d'utilisation
----------------------------
```
I'll keep the output format as it is now, we can change it later when we have no more problems in the code.

I think the annotation syntax has to change, but the curly brackets won't solve the problems I'm thinking about... Or maybe they will...
I think  you missed one comment from the previous commit
I think this should be:

meta->querySerializedMeta()->getFixedSize();

and then no need for the adjustment in the lines below.
This is unsafe (rogue path intercept) and unstable (new versions, OS-specific).

I'd have used a library (such as http://libgit2.github.com/)
Index should be using rowManager memory also, not heap.
I added the MIT license verbage.  Do I need to resubmit a pull request?

- kevin

On Thu, Apr 5, 2012 at 1:49 PM, Cody Cutrer <
reply@reply.github.com
> wrote:

> > @@ -0,0 +1,490 @@
> > +using System;
> > +using System.Collections;
> > +using System.Globalization;
> > +using System.Text;
> > +
> > +/*
> > + * This class was obtained from
> http://techblog.procurios.nl/k/n618/news/view/14605/14863/How-do-I-write-my-own-parser-for-JSON.html
> > + *
>
> please insert a copy of the MIT license here
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/instructure/canvas-contrib/pull/3/files#r652284
>
A bit fragile. I will try to solve the reported bug differently.
yes, because of <code>before { subject.visit("/hello/world?success=true") }</code> at https://github.com/joshuanapoli/capybara-webkit/blob/9cbf7d2de0a7d16e5c5dc299cace630199f6e22d/spec/driver_spec.rb#L6
Should this really be a fatal error, or just a warning? Deleting a key that doesn't exist doesn't strike me as a problem.
`\n` will not be expanded with single quotes.
Thanks for noting this, you could make it a one-liner with:

    kwargs['tmin'] = kwargs.pop('tstart')

but it's fine either way
please replace this xpath with a css path.
I agree, @shacker, this use of dummy struck me as odd when I looked at it earlier but I was too shy to type anything. :)
The double negative of the unread property in this case is a little awkward. Maybe this would be a better ux:

    ===> 
      PUT /notifications/summaries/:id
      content-type: app/v3
    
      {read: true}
    
    <----
      200 OK
Destructurointia voi tehdä myös kerralla enemmän, ja vieläpä parametrilistassa.

~~~clojure
(defn contains-point? [[left bottom] [right top]] [px py]]
  (and (<= bottom py top)
      (<= left px right)))
~~~
revise this: bad example should be a succession of self.method definitions
why did you split the authtoken config from the keystone config?
mention that it's the SSL version.
This looks the same as the data blob in ```check()```.  Put it in a method or an instance variable so you don't have to copy-paste.
wbem requires the user be running as admin, which severely limits exploitability.
It doesn't raise an exception it will display the 404 page.
Docstring please :)
Why don't you do the removal in place? There's a lot string copying going on now.
No, i think it should just append the URL (might look a tad messy?)
Should we use a more recent user agent? Such as `Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:10.0.1) Gecko/20100101 Firefox/10.0.1`
I started to make this change, but it affects to much code to do in this pull request.  I'll make a note of it and make the necessary changes later on in a future pull.
There's no reason for `transform` to be put into a variable here.  Just pass `this.getInverseTransform()` directly to the function.
passed that file through a tab-to-space converter. Nothing of consequence changed.
请把sound和music合并到同一个数据结构里面去……这2个都是关于声音的
Do we also need any npm packages?
You probably don't need to create a new variable focusable when you're only using it once.
Qualified name as in the name can have a prefix
The only way the directory could not exist is if someone manually deletes the configs directory from the SDK dir either distributed or pulled down from the repo.  It's such a hard fail case that I don't think it matters if we handle it gracefully - kinda like how we don't check that the build script is present before we invoke it.
Why do we download and unpack AS here? For testing? The setup and description so far assumes a local AS instance. I think we should stick to that, otherwise it is not clear what's going on.
What happens when the clone for a repo is running and a new build is triggered that will try to pull the same one?
rename to `attrs` - plural
This URL conversion was also moved to Docs::content() library method
I think we should leave it here, because in fact we are testing the extensions and also their sorting.
This isn't trying to check whether the div is open or closed but its trying to check whether the js has been loaded.
This should be in your local gitignore.
It appears that we can assume that `core.bitop.bsr` is defined:

http://forum.dlang.org/post/CAEnAdhYY=c5yfVG45fd=ObPJ1urH0DNS7Z7y2hhzv4jgQAXwFw@mail.gmail.com
Example:

....
`$(D dchar)-qualified`
Why not test if `channelUrl == null` here, rather than below, and don't add the `currentStage` instance variable at all?
maybe put this in a submethod? would have the advantage to make code more readable since it gives you the advantage of  putting names on things.
description of the properties

#
# blabla
#
property...
Will be added : )
If it's just a simple msi to install, I don't mind, but it's the whole platform sdk, then it'd be nice to be in puppetwinbuilder
I think this whole for loop could be written in a nicer way.  This works for the moment however.
What happens on fail? i.e., someone passes in bad ```specialty``` param
Makes sense. I was trying to avoid naming conflict between my class and System.Net.Http.HttpContent. I'll fix that.
I don't want to throw ArgumentNullException for something which is not null, and throwing a more generic ArgumentException requires specifying an error message, not an atribute name. 

Handling empty strings and adding meaningful error messages is on my TODO list.
xmlWriter simply doesn't have the objectSerialize method. It is our XmlSerializer's feature.
Will this break if it's not an openstruct?
Actually, can it be double quotes?
would it be possible to abstract the root of the path so that it would not have to be changed in every place?
Please put Helios Technologies Ltd.
email contact@heliostech.fr
There should be 3 more arguments. The `EventDispatcherInterface` interface expects `$eventName, $class, $format, $event`.
I just did it like it was in all the other cmf bundles.
So if we change it, then we should do it in all bundles.
Can we align => with 'Symfony' ?
To increase lisibility
Why get rid of tools? That's currently needed cos RhqPluginXmlGenerator is there and that's what's used to generate the RHQ xml config.
`<nag mode>`
Is there a reason this isn't following a normal accessor naming structure? setExceptionOnBadDataType(), getExceptionOnBadDataType().
I've seen this appear quite often in the framework, and think its a bit short sighted to only implement setters,
and assume introspection is never required. We had the same case for FormField->describe() (now changed to setDescription/getDescription()),
and a bunch of setters on Requirements and SSViewer classes. Sometimes even worse: Without arguments,
so its a "set and forget", which rarely makes sense.
`</nag mode>`
Should be trace level
QUERY: Based on comments, how does this function reset the log_client data buffer?
The NPE is actually happening with CCF (very intermittently), somewhere in the LinkList ( I can't remember) , the reason why i didn't care is the same reason we swallow exception in 'deleteDestinationIfTemporary'. It doesn't matter what happened unless it returns true; I'll change it to Throwable.
These should be changed to just `data/DoctrineMongoODMModule/Proxy` and  `data/DoctrineMongoODMModule/Hydrator`, as of the application CWD.
wtf?

src/test/resources é para leitura apenas.
I think `name` was passed here to make `log_info` output which connection was used. Did you consider this not useful?
OK. We have a problem here. All text displayed to the user should go through localization layer provided by Django. For example, in different language decimal point is differently displayed. So, if we try to make this library independent of Django, then we have a problem. Maybe we could make a configuration switch where you can provide localization functions, otherwise they are by default identity functions or something.

But in any case, any user displayed string should be formatted localized. And this is such case. I would like that we start to think about this from the very beginning.
Nope, it gets an arg count error. I'm not sure how to support both off the top of my head.
OH, I just see that we didn't take care of the `lifetime` and `locale` parameters.
This could just return (InputFormat) PigContext.resolveClassName(inputFormatName).newInstance();
resolveClassName throws only IOException.
Yes, the mapper is thread safe. I have tested the console under heavy load with a big JSON object to (de)serialize.

I'm rewriting the little program I wrote for this so that it doesn't use anything linkedin specific, and will send it to you for review. The program will do this:
- be able to generate a huge JSON model and upload it to one's dev console
- be able to perform N waves of M simultaneous 'status' requests to the console and show how much time it took to perform each wave

I've tested this by hammering the console with waves of up to 200 simultaneous requests.
If dataMax has a value, then this is not required.
When I see `TestCase`, I usually assume it's Django's stock `TestCase`. I think a different name helps.
I have added https://hibernate.onjira.com/browse/BVAL-317
How does this work with determining group sequences and determining whether a requested list of groups contains circular loops. Just thinking out loud.
I'm 99% sure this will hit the disk.
True. Adding an extra hyperparameter is not user friendly. But that might make it possible to scale to very large n_samples dataset.
rename to TalkToNotificationManager
Activating the transaction had other undesirable side effects (like adding actions), and since the return was via ```setActive(false)```, not ```commit()```, all changes would be discarded. Cache and activation are two completely different things altogether.
It seems like anything can change according to the spec, but it doesn't explicitly say either way.
One or more

Sorry, on a mobile so cant fix for you
> equal to or greater than
Seems to not need to, been testing fine.  Its also been this way since first ios6 fixes (see earlier use of  __restoreThenRemoveBackupLocations).
It was the userdefaults setting that needed to come before webview init.

Still, I think I may move this around just to simplify a bit.
yeah, log.warn -- but it's already doing that, below.
It is realized as http-servlet by default.
All we have is host  ? ec = 301 & startUrl=xyz, I'm not sure we can know the host for sure (when people have custom domain or branded login or sso etc).
I like the even more complex shortening is moved to the helper but I dislike the key parameters are moved as well. It's view's core business to determine how many character long summaries to show. So the refactored helper shortened_summary() should take the three params from index.html.haml instead of "hard-coding" them. Perhaps nice middle-ground can be carved from initialized parameters, which lets override the lenght later on. Even so, I'ld like to see index.html.haml to keep defining the length (270, 230) and the end string.
`object['string']`
ummm, I wish had a clear answer
w3c spec doesn't specify
but I think its a goog feature

so I am saying case insensitive is a better but we should probably open an issue for the other features that don't do that right now.
hpp extension as well
mising->missing
Decorator?