labofsahil/github-event-dataset-2012 huggingface dataset:
@ColinHebert You cannot install virtualenvwrapper manually by just copying the `virtualenvwrapper.sh` script around.  It needs to properly installed, and provides a standard `setup.py` for this, just like all good Python projects do.  And this `setup.py` file can install the script to a lot of different places, depending on the options given to it and on the Python version used to execute it. On an OS X system, there can at least be three different locations:  

 * `/usr/local/bin/virtualenvwrapper.sh` (global installation into system python)
 * `/usr/local/share/python/virtualenvwrapper.sh` (global installation into homebrew python)
 * `~/Library/Python/2.7/bin/virtualenvwrapper.sh` (per user installation into per user python site, e.g. with `pip install --user virtualenvwrapper`)

There can be more, e.g. a `virtualenvwrapper.sh` installed into a custom build of Python, a fairly common thing on shared hosting environments.  But whereever the script was installed, it is guaranteed to be executable, and hence the `$+commands[virtualenvwrapper.sh] ` will work.

Your suggestion works, too, but required `setopt nullglob`, and boils down to searching the whole `$path` manually instead of relying on the previous `$path` search done by `hash`:

```text
sphinx ❯ which virtualenvwrapper.shhttp://github.github.com/github-flavored-markdown/
/Users/swiesner/Library/Python/2.7/bin/virtualenvwrapper.sh
sphinx ❯ unsetopt nullglob
sphinx ❯ reply=(${^path}/virtualenvwrapper.sh(@,.N))
sphinx ❯ echo $reply

sphinx ❯ setopt nullglob
sphinx ❯ reply=(${^path}/virtualenvwrapper.sh(@,.N))
sphinx ❯ echo $reply
/Users/swiesner/Library/Python/2.7/bin/virtualenvwrapper.sh
```
covariance matrix (reversed)
is this a different sigma from the one in the model._compute_sigma ?
If yes, different name?
Why do we have these changes here? Is this related to the PR? If no can you make a separate PR?
Might want to reference ${project.parent.version} or ${project.version} instead
This line repeats info from 48-54.  I don't think it's worth repeating every time there is a maven command line to tell the user about this option.
typo (AtmophericPSF --> AtmosphericPSF)
Not a huge fan of landing comment out chunks of code. Is there good reason to do this? If so please add a comment to the reason for that here.
Added that to my TODO list
no.  javadoc says no scanning is done.  scan only applies when first 
loaded.
adding a classpath entry is only to support adding entities dynamically.
(if we needed to scan outside of a catalog we could refactor so the scan 
method is accessible.)

On 26/11/2012 12:13, Aled Sage wrote:
>
> In core/src/main/java/brooklyn/catalog/internal/CatalogClasspathDo.java:
>
> > +        return classloader;
> > +    }
> > +
> > +    protected synchronized void loadLocalClassLoader() {
> > +        if (classloaderLoaded) return;
> > +        if (urls==null) return;
> > +        classloader.add(new URLClassLoader(urls));
> > +        classloaderLoaded = true;
> > +        return;
> > +    }
> > +
> > +    /** adds the given URL as something this classloader will load
> > +     * (however no scanning is done) */
> > +    public void addToClasspath(URL u, boolean updateDto) {
> > +        if (updateDto) classpath.entries.add(u.toExternalForm());
> > +        addToClasspath(new URLClassLoader(new URL[] { u }));
>
> Do we need to invalidate the previous scan at all (e.g. isLoaded = false).
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/brooklyncentral/brooklyn/pull/396/files#r2220992>.
>
I will rewrite id to cocoa.id.
add a new line here
The beginning of the sentence is a bit weird, I couldn't understand it fully :(
No, the test you deleted says: `"should not include the sideloads in resourceURL"`
This is not necessary. 
HTML5 allows you to omit the slash.
bleh but I really wanted to learn how to say bloodz
I was a little skeptical of this, but I ran some tests using 32-bit Java 6 on OS X Lion — the client VM averaged 8.006 seconds, the server VM 7.430. Alrighty then!
is it really needed to have 2 levels of array node here, with the first level having only one child ?
please use `com.twitter.finagle` for package namespaces -- for consistency with the rest of finagle.
I don't think you meant to put that there.
Hiérarchise tes traductions et scinde les en plusieurs fichiers.
Here is where you could pass down the configured cache directory.
Исключение в середине кода - и output не будет закрыт.
Символы: http://docs.oracle.com/javase/tutorial/java/data/characters.html

getStackTrace()[0].toString()
Please look into these comments and determine if they mean anything anymore.
in general, I would avoid prefixes like "temp" in such a context.  it's no more temporary than any other variable on the stack.  "message" would have been fine.
why do we implement this 
if we need to return to the previous page we should use the url
Indeed I checked and passing a callable as kernel is just a shorthand for the precomputed kernel.

https://github.com/scikit-learn/scikit-learn/blob/master/sklearn/svm/base.py#L228

Lazy evaluation would be possible using a function pointer but the datastructure conversions for the sparse case would make the code rather complex and probably not very efficient anyway.
Er, my bad, done in f402556.
I prefer the previous version or "This problem is sometimes know as"
Should this constant be exposed here?
To have this render well in rest, you need a '::'.
Right, I'll remove it
A "deployer thread" would be an MSC thread, since all the runtime deployment activity is done in the Service.start(...) methods of the deployment phase services.

I've been chatting with John Bailey: ServiceReferenceObjectFactory is no longer used and should be removed.

A user could deploy multiple applications, and with only 4 MSC threads by default, blocking like this is a concern. 

How hard would it be for a DUP to add  dependency? Even if you can't do the injection and have to do a service lookup like this, at least it's known that the service is available when this code is run.

I admit my concerns are kind of an edge case, but it would be good to know the solution even if we don't implement it until 7.1.2.
Why not make it nullable? int? maxItems = null
Yes, that commit fixes it.
Needs a comment indicating what the patch is for---this allows future maintainers to make informed decisions about whether to pull the patch or leave it in.
need to hide ctors, = operator
Wouldn't it just be simpler to cut the string at each % and then pass that along to the regular sprintf?
je ne comprends pas où cette méthode va te servir, si je veux créer une task je fais juste:
```java
Task t = new Task();
t.save();
```
`Towards the poles with deformation,`
`Towards the poles where the deformation is more significant,`
Yes it is :s

I feel not confortable with this

I am still looking for a good and proper way to deal with it.
I m quite neebie to write such docs :s and so my style is very ... hum ... bad because I try to be to much elliptic in my sentences. Emmanuel told me the exact same thing.
Sorry
firstHte doesn't seem to be cleared/reset if another row is added.  
It should probably loop until i == firstHTe (instead of maxRows) (and rename the variable to nextSpillHTE)
Needs retesting - It would have always failed this test before.  !row->getOp() == false  false == create = false
This is to ignore ```NORMALIZE(ds, LEFT.childDs, transform(COUNTER, LEFT))```, not sure this is the best way, but it works.
Worth referring them to the language reference manual for more details.
Please don't change the indentation of the license headers
It's weird that we'd use on that isn't suited for 80 chars.
You just said the same thing as me with different words. 

" the group was "bootstrapped" by a semi random group of framework representatives at a conference "

So they appointed themselves.
Is there a reason for changing this from a truthiness test?
Wouldn't you want it to default to process.version, rather than 0.0.1?
Use $path[1] here, also !== instead of just !=. Secondly, is it sure that '~' will always be in position 1? Additionally, make an isset($path[1]) call first.
not only Zend\Config\Config objects but any array or Traversable.
Ahh, this is what happens when you copy and paste. Fixed now.
... of the Twitter users ...
We'll have to figure how to handle the initial data with #3.
I think that the numbering thing was for artworks rather than their images wasn't it? Primary image was a different thing for art with multiple images, deciding which was the main one displayed. We just stuck to choosing the first listed image for the time being as a temporary solution, planning to add a database column later on. So I think delete this migration for now, but we will likely revisit it later on
Also, look at the [contraflows/picaroon](https://github.com/contraflows/picaroon) repo for how I did it. The monkey patches in that repo are about to become redundant once a new pull request in sprockets is merged in.
That was my mistake
use id="offer"
no, please link against mlite instead, and use MGConfItem - should be virtually no change apart from the rename

this way we can avoid packaging yet another tiny lib
This currently runs when you set DOMAINS, potentially adding a domain you haven't asked to enumerate to the list. I would imagine the first thing I have done is enumerate the connected domain, I run this a second time to enumerate 2 specific other domains and dont want to repeat previous work.
Yep I agree
PEP 8: line length.
basically this issue is that the default username is a hash of the email. So browserID goes and gets the same anonymised user if somebody creates a new account after having deleted it. We don't want to block them from logging in just because the previously deleted an account. A better solution might be to leave the `is_active = False` but to create a hash of the email + a time stamp or something.
Pagination. :)
move these assignments up top, out of the constructor
I am a bit confused why isn't this business logic?
Why'd you remove these PHP docs?
not sure this applies, want to return v.value if v is non-null, and null otherwise (this is about avoiding the null ref, not about a default alternative to null)
Please put space between the 500, the asterisk, and the 1024.
Not a huge deal, but would be more efficient to just prepend imagePrefix when assigning the background here instead above.
Looking at your changes, url could be null.
It's outside the if block so we can still reset the value even if there is only whitespace in the input field.
This will break the loop on first iteration and it will never check for the rest of elements in data. Because of that, the tests are failing. Check this out http://travis-ci.org/funddy/jstranslations-bundle/builds/2964041
Vou comentar nessa linha aqui somente por preguiça, mas serve para todas as linhas que tenha setter :D

Ao declarar um setter publico, vcs permitem que em qualquer ponto do código os valores sejam alterados. Isso só deve ser feito se fizer sentido na lógica de negócio. Senão, é uma falha grave de encapsulamento. Eu poderia, por exemplo, fazer 
```java
Circulo c = new Circulo(2);
c.getAreaCirculo(); //Imprimiria 4*pi

c.setAreaCirculo(10); //Alterei o circulo de area 4*pi!
```
E por ai vai...cuidado com setters!
Nit: Wouldn't the `on` be needed here?
Included because the NTLMStream class has license information, and was just trying to be consistent.
I always learn stuff from him. One day I'm going to get him to let me watch him while he codes.
use ?: to initialize
Hmm I'm wrong there, nm.
HuHu !
Good comments, will change to "since ..."

As for format, no agreement. I'll be happy to update once there's some sort of convention. I consider this just a first pass.
Ah sorry about that. If you bump the Iterable size generator to `Gen.choose(0, 5000)`, do you see it fail with the original code?
no longer required then
Should this be using dummy-threads still?
true   >>>  TRUE
What is `&:patient_id`?
I would exclude build stats in example easyconfigs. Having them is interesting, but they should be collected in some other way, not in here.
`.apply( this, arguments )`
Не появятся, я уже починил. Но еще н коммитнул. Не надо ассептить этот ревест на данный момент
ok, I created a PR doing this for other filters so please do the change
Why not string.Join?
prepareSite no longer gets called in local testMode?
Don't you only want to print this if there are obs groups?
I like your suggestion.
No, please read Mark's comments in the attached link
Maybe we should do away with the ``_`` underscores to be consistent in the naming.
Removed this one.
everything from this line down is unneeded; all you need to satisfy BaseOpenmrsObject is id and uuid ... then the other properties (user_id and location_id).
please update view to tell user that no versions were found (for some reason or another)
This seems like an extremely heavy handed way to restart the site. Was there any prior discussion on this?
Give me a second.
This for loop is correct but I can be more pythonic like this:

```

for i in j_data['objects']:
    feed = i['profile']['personal_blog_feed'].strip()
    if feed:
        print '[%s]' % feed
        print 'name=' % i['fullname']
        print 'avatar=' % i['profile']['avatar_url']
        print ''
```
Nevermind. I see that it is the actuall response.
It's a trade off.  We could implement a passcode delegate or equivalent that consumers could implement, taking actions synchronously before the passcode is displayed.  It's probably more cumbersome than the notification route, though.
This method is likely to be very expensive - at the very least it will need to get the bounds from the GC object and do an `sql_for_area` filter using those bounds so that we can roughly cut down the matches using a tile index filter.

If this is just to get a description for the location of a changset though then I would suggest using the `describe_location` helper function, which uses Nomination to get a description. We would also need to think about caching the result of that, in the same way the diary pages do.
Extra space before None.
How come this isn't "if(--callbackRequestCount <= 0) {" instead?
Typo? "static?
interface methods and usage would be quite nice here; I know it will come, but just wanted to say it :-)
Sure would be. Looks like PhpStorm didn't recognize that as CSS. I've fixed that in my fork. Will that automatically pull into this PR?
It's a very subtle change I suggested. Rather than calling: `@rating.update_attributes(params[:rating])` do `@rating.update_attribute(:rate, params[:rating][:rate])`

It's a small change, but protects us from exposing more than we need to.
This is quite mind bending. The "classic" solutions is:
~~~clojure
(defn my-map
  ([f x & more]
    (if (some empty? (cons x more))
      '()
      (cons (apply f (my-map first (cons x more)))
            (my-map f (my-map rest (cons x more)))))))
~~~
please, do not add 0.7.1.dev1 in this PR. It will be automatically attached by #100.
I ask again. Why are those classes removed?
This is wrong. Please revert
I'd rather keep this space in there for readability, but can cope without it :)
Replace with getEntityUri
False means the update was a no-opp, not an error.  Maybe log the false case but do not treat as an error.
Can you update the comment to indicate that "More..." is actually a tab container, not a tab.
Class description could be a lot better.
Comment doesn't match comment conventions. Should be @var etc
It isn't always returning true
Hoituu vielä yksinkertaisemmin näin:
~~~clojure
(if x
  true
  false)
~~~
Read the code closer. I keep the parent and delete the children in batches of MAX_KEYS (using the parent to query) until there are no children left, at which point I finally delete the parent.

This is to make the cleanup sane (memory and rpc wise).