labofsahil/github-event-dataset-2012 huggingface dataset:
That crossed my mind too, and I was trying to think whether there were any reasons not to? (cc @eteq)
Careful, you should use https://github.com/zendframework/zf2/blob/master/library/Zend/Authentication/Storage/NonPersistent.php instead. Otherwise you're polluting the test environment :)
This comment doesn't add more information than the code itself.
Thanks; I updated the note.
@mark-hahn: To which issue(s) are you referring? What's necessitating `var x;`?
Remove this string in the middle of the class definition. This should be stated in the documentation.
Also, I consider Capybara 2 support to be a feature, not a bug fix ;)
removed this unused dependency in pull #203.
I've put in a separate bug for this: #84
I am pretty sure it's supposed to be allocated though, lawl (since it's the spawn Queue ;P)
Well, since the ad in kitchensink still points there, I actually did. We should probably move the dummy ad to someplace Toura owns, though :)
I am pretty sure that `border-radius` is one of the css3 properties that doesn't need prefixes anymore.
No, you get as an argument user object, so you change it to `user.email`.
One space between docstring and code.
You can turn the equals around and end up with way less cyclomatic complexity:
````java
if (XOP_NS.equals(namespace) && XOP_INCLUDE_TAG.equals(elemName)) { ....
````
> Hmmmm. I think my preference would be `x` over `r`, indeed because of the use in the relevant Bartelmann paper. But we could always use `xx`, or even what about `u`? It's commonly used as a substiution variable, and we don't use it anywhere else (in Fourier space we use `k`s almost exclusively, except some places buried very deep in the C++ I believe). But if not some alternative, I think `x`.

Well, I guess u has some merits since it's not used elsewhere and we could simply define it carefully in the dox.  People might wonder what it is, but at least (unlike for x and r) they might not be inclined to make some erroneous assumptions about what it means, so they would likely go to the dox and see our definition there.  So I guess I'm inclined for either x or u.

Any thoughts on this, Mike?  I'm happy to make this change plus add some refs.
等号前后留个空格呗。
Professor x =)
With this change, it fails for me when I run inside Eclipse (previously it passed). Is it possible to make it pass both in Eclipse/IDE and in maven?
Perhaps we could have a condition that tried to find the file in the given location, and set it to whichever one it could find?
yes, good point.  is now:  templateOwner
No need for a cast, just make sure `regionToRects` has the correct signature.
s/Firefox/browser
Suggest shortening up the lines so that Teri and myself can read it easier:

    A @ $30 every 1 month

To:

    A @ $30/mo
Let's also use:
mongodb://user:user@primary.local/phpunit-auth", array("replicaSet" => "foobar")
typo: requiters
Removed method instead.
This is fine, but why do:

int success = ...;
qDebug() << .... << success;

instead of:

m_icon.load( iconPath );
qDebug() << ... << m_icon.isNull();

if the icon is null, you know the load failed.
rename here?
@see known protocols on getter
Another style nitpick. I think it's a good practice to always use curl brackets with any if, even it it's a one liner.
Yeah, it's definitely an odd case. IIRC, when testing variations on this, I narrowed it down to an issue in php related to passing a protected (or more narrowly scoped property) as a use parameter of a closure that is passed to `spl_autoload_register`. When any one of those things changed, the container was not cloned. There was an issue in php bug tracker at the time (that I can't seem to find now) where there was a different symptom of most likely the same issue. He was reporting that memory wasn't reclaimed after passing the object to `spl_autoload_register`.
drop me a note after you do it
Can you redo this with a call to apt-add-repository?  That should prevent duplicate entries from showing up.
On thing to note here (not a major issue) 

It's good practice to ask what something _can do_ rather than what something _is_. For instance, if I were to build a proxy object for Cinch::User which provided some extra methods or something, it might not .is_a? the way you think, but it still can fulfill the contract of this method (since it just proxies down to Cinch::User#name, perhaps).

A concrete example might be a multi-name tracker, we might have a model Cinch::MultiUser, which tracks a user who uses multiple names that don't fit some schema, but we want to grant approval to all of them. We might also want to have a Cinch::AuthenticatedUser, which represents a Freenode auth'd user. etc.

Just something to keep in mind.
А что это за название поля не по Code Conventions?
Ещё полезно написать в исключение - у какого класса такое поле не найдено.
config has to be loaded before the rest of the angular slices. There's probably other issues that can occur (beyond just calcentral namespace existence) I'm assuming if config doesn't load before the other pieces of the framework.
Can you find out if this is true? If it's not doing anything useful, let's delete the code.
You don't want it done in the constructors.  Very important values are set  for JavaPlugin's via a .initialize method.  If things are initialized in the constructor, we're creating objects and initializing them with essential traits being null.
Not afaik, you can always try it with parenthesis but I imo less is more. :hamster:
Oups that was not suppose to be there removing it
I only remembered of it after you suggested creating such a function :)
I don't think so. I think each output can have a distinct target classification scheme but the model share intermediate representations. Some kind of multitask classification.
You probably should be using np.arange here, instead of range.
I raised this issue as well, but seems like powermock is some sort of extension of mockito - so not sure.... 

Yeah, integration tests would be optimal. However this is something i need to sort out , since i don't think we really cover that level yet.
Why not use the well-tested smart pointer from std library (shared_ptr)?
done (same as above comment)
Don't create dest file if there isn't a source file.
Ah, yes, I saw that, and was wondering why it would have changed. 

I don't think that it'll have any impact, since the compiler front-end will create the temp and the back-end will remove it anyway.
I don't think it's typically a URL/URN is it?
Since this function is not returning anything other functions need, the return value is just success/failure, it could very well be a boolean.

However, we're throwing exceptions on the errors and ignoring the mistakes, so it could just as well be void.

If you make it boolean, make sure you catch the return values where it's called and do something appropriate. If you make it void, make sure you throw exceptions on the real errors and make sure you catch/ignore them where appropriate.
Sounds like I am not interpreting how and when retry() can happen..
I thought retry() could only happen on 'prepare' failure, before anything has 'run', before anything has changed.
So C lock fail, would be before B::run() (as I understood it)
will cAddSubFileAction rollback, post commit(), if subsequent error?
Refactored to match above code
Typo? `/documentation/releasenotes/1.1,0` -> `/documentation/releasenotes/1.1.0`?
@rhelmer I must be blessed with something different. I was able to write to /usr/local/lib/node_modules as user peterbe. That's what jlongster said was correct too. Perhaps it was when I installed npm globally the first time. It's very confusing. 

Perhaps we should just drop it and just say. "Make sure you have `lessc` on the PATH. See lesscss.org`
im assuming walrus doesn't do any caching internally, so we should cache the compiled function when `options.cache` is true, there should be an example of that here somewhere in the file, if not dont worry about it i'll add that part
no, closing curly bracket.
Author and contributors are different concepts! (I can't think on the Rails project authors list otherwise) :D
I don't mind to keep it, up to you. :)
D'ailleurs border-radius existe déjà dans Compass: http://compass-style.org/reference/compass/css3/border_radius/

```css
@import "compass/css3/border-radius"
```
(Redis will atomically run the Lua code.)
You should put the variables in braces: `${::operatingsystem}` and `${name}`.
This logic will be true on the last page and hence doesn't include the results on the last page.
couldn't think of any nice way to handle trying to render graphs before the API has loaded, you can do a scope.$watch on possedartsStatsModule.value('ready')  right? and do things like disable UI elements until the API has loaded?
Yes but this vprint_status says "Attempting to execute #{command}" which is a little confusing, I think in this step the service is being created (no execution in this moment).
"safer"?! Really, blowing away local changes without any possibility of getting them back is better? I respectfully disagree, sir. By which I mean no effing way is this a good idea.
no, A is removed by shr - giving place for \x00 at the end, making shellcode \x00 free
Resolved in the upcoming fix.
Suggestion:

    Central moments (translation invariant) up to 3rd order.

        mu_ji = sum{ array(x, y) (x - x_c)^j (y - y_c)^i }

    where the sum is over the `x`, `y` coordinates of the region,
    and `xc` and `yc` are the coordinates of the region's centroid.
Since you (I think) use terminal Vim these days, do you still want this map?
We use `instanceof` in a couple of places (DynamicScene Materials comes to mind).  It's not slow enough to be a problem in any perceivable way yet.  I can't imagine why it would be slower than an explicit property check.  Obviously this is just anecdotal, not sure if @shunter has any thoughts.
I actually intended this as a quick fix until I could look into making it stay undefined.  Now that I'm done with the rest of the path stuff, I'll look into it to see what it will take.  If it's not too bad, I'll just do it in place, otherwise I'll open a separate pull later on.  Either way I agree with you.
OK, I will take `OffsetEvent`.
Yeah I know, the first time I use 

data = self.getCache(cache_key, searchUrl)

it's just to see if the site is online really, because the cookiejar is not ready yet and so it won't be able to get search results anyway. Would you like me to change it to this? Does this method make use of the cookiejar btw because else it won't be able to log in. If I can use getCache with the cookiejar it'd actually make use of your cache instead of not really using it (the way it is now)

                f = opener.open('https://www.sceneaccess.eu/login', params)
	        data = f.read()
	        f.close()
	        data = self.getCache(cache_key, searchUrl)
OnIOS currently only supported on Ti.XML.Element
Ha, I suppose having unexplained integer constants is a pretty bad thing. Good call on this one.
Поехала табуляция (табы и пробелы?)
Nor is one necessary
`border: 0` is shorter
And it also means a single `index += n;` *after* the switch. This is also because it means we avoid the explicit 1 < n <= 4 enforce.
I think the bracketed part is worded a bit strangely.
I'm pretty sure you don't need this file.
Changed to @Transactional
Not sure we need to say extension here.
There should be a `settings` variable (e.g. `AJAX_MAX_PER_PAGE`) that sets a sane upper limit. As it stands now, someone could send `items_per_page` equal to `10000` and cause serious issues. I also prefer `offset` and `limit` to pagination like this (I think it's a bit more flexible and maps more directly to the SQL below).
Can you fix this up?

```Drug.all.sort_by do |drug|
  [drug.time_left, drug.alert?]```

Or something of this sort
Partial is rendered and replaced. 08ec53e
Hey. I think that FormatListener isn't apprioprate for this 'job' - it listens for an "Accept" header but in this ParamConverter we need info about "Contet-Type":

1. User is making request with XML content body and headers: Content-Type: application/xml; Accept: application/json
2. Serializer can deserialise it from XML format. (bcs Content-Type is set to app/xml)
3. We are returning some response content
4. Response content is being serialized into JSON format (bcs of Accept header)

if we would use FormatListener for deciding 'from which format we are gonna to deserialize" in (2) we will pass to deserializator JSON format as a base of deserializating.

So the only way to do it is based on the code from BodyListener (https://github.com/FriendsOfSymfony/FOSRestBundle/blob/master/EventListener/BodyListener.php#L53)
This is wrong probably.
Yay! No more saving addresses twice in memory!
You appear to be using this same password for your GMail account. Best not to push it to a public repository.
Woo hoo! So 4.4 is the equivalent of every student getting 20 hours of interested classes (or 13 hours of priority classes), which is both reasonable and pretty impressive.
Parameter was never passed through.  Removed it.  We can add it in a later commit if we need to.
The only issue with animations would be if someone legitimately wanted [0..0.01] opacity. It doesn't sort the underlying issue of the deviation though.

It made sense at the time because I assumed Bootstrap would move to the [0..1] opacity anyway - passing a decimal value [0..100] as a parameter to opacity doesn't even make sense. Obviously I guess that's not going to happen now. Better we move in line with what upstream says, even if - semantically and spec wise - it is wrong.
Это вполне можно приклеить на предыдущую строчку.
Agreed, also changed it in other places in the HTML.
No, tinyMCE is only included on pages that actually allow you to edit sakai docs. On other pages, tinyMCE is not included and not initialized (faster page load).
2000-2012
That is why a check is performed. In the general case, a SignalError is thrown.

As mentioned below, this should definitively evolve into something that check for StackOverflow and allow 3rd party code to hook its logic on a precise memory page.
That's a good point. I guess we would really need a way to prevent the boolean value from being configured if the task scheduler is "shared"/global. Maybe sufficient to log that the boolean would be ignored in that case, but it almost seems like we could throw a configuration error and suggest providing its own scheduler.
```
.setReplyChannel(replyChannel)
```
is redundant. Just declare queue-channel in the config, add 'output-channel' to the transformer element and @Autowire it.
IMHO...
Unnecessary imports.

Please use PyCharm to clean imports.
You are right, but I expect a MvcEvent object here. On the other hand would increase the robustness in a non-standard  ZF2 setup.

I will change it later in a seperate PR, because there are many other things I have to change too.
I don't see an @should for this method ... please add it and link accordingly with @verifies.
remove space!
should not be singleton
We should probably make this a parameter that the user can set in the future
Isn't a global opt out like an event rather than a state?

Meaning if someone opts out, then can opt in at any time and thus the global opt out state might not exist.

Maybe it should not be nullable.  That default false value can make sense.
Display an alert to give a feedback to the user about the completion of the action
On 19 Sep 2012, at 12:00, andrewjpage <notifications@github.com> wrote:
> 
> In sql/VRTrack_schema_20_21_22_23_24.sql:
> 
> > @@ -0,0 +1,28 @@
> > +ALTER TABLE seq_request MODIFY seq_type ENUM(
> 
> I'm afraid I dont know enough about this, but eyeballing our databases the values look okay. I presume you havent noticed any issues with your databases either since updates 20-23 (written by vertres) also reordered the enum?

I did a little test and it seems like MySQL automagically handles the change of order. At least, I hope my test was sufficient. I'm not aware of any issues that arose, but am also not sure if having the wrong seq_type would raise an error anywhere.
I don't really undestand why we have 2 files that will do the same thing (based on their name)
duplicate "and" wording "and and"
There's a slight change I would still do here. data event fires every time the server writes something to the client. We can't assume that it comes in one piece since the encoding is set to "chunked". I would would write this in data handler:
```
var data = "";
response.on("data", function (chunk) {
    data += chunk;
});
```
And add another listener for:
```
request.on("end", function () {
    // Check data variable here
});
````
You like how easy that was, don't you...
If x is not None, we don't ever set team, which we reference when we build EventTeam objects later. We need to do `for index, team in enumerate(teams)`
I tried to test this by changing the URL of MSYS to:

URL_MSYS = "http://ourceforge.net/projects/mingw/files/MSYS/Base/msys-core/msys-%(VERSION)s/MSYS-%(VERSION)s.exe/download" % { 'VERSION' : VERSION_MSYS }

Sadly I get a windows alert box with a warning about an invalid instruction. Do you see the same? Possibly a bug in Python? I'm on Windows XP 32bit..
whoops!  thx.
Copyright year needs to include 2012
please do not use this kind of names (with "laradiodelsur") in CSS clases
On Android, the type of data can only be Titanium.Blob
should be just `*`, there is stuff nested in there
-1 for this change also. IMHO, we should not mix cleanups with this PR.
With this command the ``start.transcode.video`` will be never call, as the first backend ``start.transcode.audio`` will be used.
I have to say that I don't love overriding the rails class for this.  Is there a reason we couldn't just subclass it and point respond_with to our custom responder?  If you convince me, I can merge this in tomorrow, it is otherwise a nice improvement
This should be written in English:

> Load/fetch online users
I believe it is
забыла выпилить, сейчас поправлю
Same move these to the impls.