labofsahil/github-event-dataset-2012 huggingface dataset:
So this is what terrorists have been reduced to.  Can I please bring toothpaste in my carry-on bag now?
Forgot to set the color to blue and underlined, to make the "click here" work...
I personally prefer capital E and B since we talk about "E mode power" and "B mode power".  Perhaps we should solicit another opinion, e.g., @barnabytprowe , @reikonakajima ?
Eep!   That was ridiculous.  Will fix (including the single calculation of scale).  Another good catch, thanks.
So, just out of interest and to check I've understood, is the `safe` output (2nd element in the output tuple) designed solely to cover this case?
Doxygen is not detecting the mainpage stuff if I try to use markup to make it miniscule on GitHub.  I give up.
Oops. Will do.
```new_resource.type == "proxy" ? true : false``` can just be ```new_resource.type == "proxy" ```
why to make another loop? You can just fire store() method inside previous loop.
currently we also have AbstractEnricher distinct from AbstractPolicy though personally i don't like that.  my 2p is to merge the not-clealy-independent "enricher" into policy, & supporting AbstractPolicy here is fine.  but wanted to flag it for your opinion.
Should this be in a finally block? Same for insert method?
Yes. And move impl of Exceptions there, with an old deprecated class in brooklyn.util that delegates to `brooklyn.util.exceptions.Exceptions`? And same for `RuntimeInterruptedException`.

I agree "exceptions" rather than "exception"? We currently have "task" singular, and "flags" plural though.
Shouldn't you be using `<h2>` in place of the `<h1>` and `<h3>` in place of the `<h2>`, etc.
Why comment this out? Did you mean to remove it?
On the other hand, people wants a simple way to upgrade to the latest version of Symfony SE, so I don't know what's better. I don't think many people have customized these files, that's why I've opted to override them with the latest version.
It's just an additional option because I'm currently not able to run normal installation, which may happen again in the future (to others as well). 

On 9 Jun 2012, at 22:11, habibvirji<reply@reply.github.com> wrote:

>> @@ -102,6 +102,10 @@ if (cmd === "install") {
>>       });
>>     }
>>   });
>> +} else if (cmd === "wrt") {
> 
> idea was wrt is build automatically a separate step might be confusing 
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/webinos/Webinos-Platform/pull/78/files#r956086
nit: add a blank line
This looks bug-prone to me - I would rather see the caller trim down the tids and leave the original asserts.
Oh >_<
I don't know who Crubier is but if you want to defer this change until after his changes, I'm fine with waiting.
为什么这里不用实例变量@event 而是用event？  这是什么风格？
Coding Guidelines bitte beachten (InstallService, Pascal Case)
is a safe assumption that 100% of fine print contains div tags?
Собственно, это некорректно. Если запускать просто указывая имя файла - программа должна количество слов посчитать.
Is this debug output or intended?  Hard to see from the read
> juse do:

Shouldn't it be "just do" or simply "use"?
Add Karel Zuiderveld and Steven Silvester to the CONTRIBUTORS.txt.
I'll quickly make the changes.
A.T is an attribute of A. Therefore, it isn't safe. i thought it was a whole new matrix (ie a return from a method).

This is why it isn't safe. I've commited a fix.
On Wed, Oct 10, 2012 at 08:33:30AM -0700, Daniel Murphy wrote:
> I suspect this would be much nicer as a switch.

Since this was duplicated a few times anyway, what I did was take the
list from json.c and move it to dsymbol.h so it is accessible globally.

Then all the various repetitions of the protection names started to use
it, following json.c's example.

So now my list is unnecessary too.

(I was planning to do this as a separate pull anyway, but I guess we can
just do it now.)
Moved to ndk and centralized the program names
Is there nothing in userContext, caller, or something in v8 that can tell us which file it is being called from? It could simplify things if we add that functionality.
I wasn't sure what that diff meant, git just seemed to add that itself when I undid my CMake change and put it in a separate branch.
If it really is directly equivalent..
then at this juncture, as you've added the transaction = line, it would be worth that one extra step, to make it explicit you are looking in the transaction for the logical file.
I agree it's not worth changing the backdoor implic use of the transaction via ctx at this point though.
That was my question to you over IM. 

It seems that they are always in order, since the add/del will re-order them every time (part of the loadSubFile's job is to make sure they keep in order, too). So, in theory, this code is safe and fixes our problems. 

There could be some bug we haven't found yet, that will break the order, but I'd rather deal with it once we get there.
Yeah it's a %, I'll change name and I'll change to :
> maxRows / 100 * HTSIZE_LIMIT

as maxRows can't be small enough to be significant
Hope rsync is portable, present in most distros.
From what I read it is.
I really think that this check is a bad solution for JENKINS-12640. I use ant to generate the coverage report xml, so no cobertura mojo was run. Jenkins does not read this xml (although the xml is valid and the path is correct) since v1.4.
This will not work for the ODM as the method name is different. You should use the method of the EventAdapter instead (which is meant to handle this difference)
Elmo uses MPL2 and we use a very different LICENSE 

https://github.com/mozilla/elmo/blob/master/LICENSE
Используется will_paginate, второй наверное забыла удалить. Предполагаю :smile:
See #54 for this too :)
I would say The `contentScriptOptions` is a json that is exposed to content scripts as a read only value under `self.options` property.
This section fail when testing
please don't change file permissions.

If you are on windows (and so cannot handle them on your own), please configure git to ignore permission changes (msysgit does it by default but not all git setup): ``git config --global core.filemode false``
identation is also wrong here
this project uses 4 spaces tab indentation
No need for blank line, is inconsistent with previous half of the example.
Note for get the merger. I've already had the discussion with @christianv with the changes in the backend for /api/canvas/courses. There's an pull request that this is dependent upon https://github.com/ets-berkeley-edu/calcentral/pull/95 for getCanvasCourses to work properly. They can be merged asynchronously but until the other fix is in to allow the passthrough, getCanvasCourses will probably appear broken.
Wouldn't this be more time efficient as a `Set` since you're going to be doing `include?` testing at the end of the method?
typo: "the the"
I was seeing warnings from nosetest. I thought deprecated meant it still exists but it shouldn't be called. I could put the call to homography back in if you like, I guess... but I'm losing my energy for making what was supposed to be a one line change to prevent clipping :/
Shouldn't this be in <plugin> in the main build?
logic was centralized to the event handler.
Wasn't there a discussion about pulling from environment variables? This should check for those environment variables, use them if set, otherwise fall back to the resource.
You left some TODOs in here
This fix will require 2x more memory for the same image. Seems to be a very expensive solution, which will cause memory issues when users attempt to load up high res images. Is this the only solution?
if there's no comment then leave no space between these lines
This causes a rather nasty warning/error:
```
[INFO] --- maven-plugin-plugin:2.9:descriptor (default-descriptor) @ maven-hornetq-plugin ---
[ERROR] 

Artifact Ids of the format maven-___-plugin are reserved for 
plugins in the Group Id org.apache.maven.plugins
Please change your artifactId to the format ___-maven-plugin
In the future this error will break the build.
```

Perhaps we should use a different artifactId?
To do this right I would need to update Stackster::Stack so it can take template_file or template. Right now it only works off of a file. Is that too big a change for the addition of one command?

The alternative is to have the clone command write the template to a temporary file and pass that to Stackster::Stack. Then we have to deal with temp files on the servers, which can get ugly fast.
Type does not match
This hardcoded domain won't work against production.
Looks nice.

I was personally trying to find a more generic "transaction" that could be
re-used for spi/i2c maybe even uart as more and more devices tend to
support several busses and re-using the drivers would be very nice.

(e.g. ublox: uart/i2c/...   many chips with dual i2c/spi interface and the
same driver behind it)

In c++ one would have a

class Transaction;

class SpiTransaction : public Transaction;

How could this be done in C?

On Thursday, March 8, 2012, Felix Ruess wrote:

> > @@ -37,6 +37,17 @@
> >  extern void spi_clear_rx_buf(void);
> >  void spi_rw(struct spi_transaction  * _trans);
> >
> > +#define SPI_TRANSACTION_QUEUE_LEN  8
> > +
> > +struct spi_periph {
>
> This should probably be a part of the arch independent spi.h interface...?
> So we can implement the same for the lpc, see issue #147
>
> @dewagter @gautierhattenberger did you have chance to look at this?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/paparazzi/paparazzi/pull/148/files#r531520
>


-- 
-Christophe
It seems like every plugin that requires files or packages does it in
a less than ideal way. Since you know up front the files and packages
you need to run you can simply define them as class variables and the
superclass will automatically check it for you.

class MyPlugin(Plugin, ...):

    files = ('/path/to/file')
    packages = ('name-of-package')

    def setup(self): ...

The only reason you might want to override checkenabled is if you need
to do something a little more dynamic, like looking for an option to
be passed in.

In any case if you are wanting to call the superclass method its
probably better to use super(MyClass, self).checkenabled() just in
case the hierarchy changes in the future.

On Feb 24, 2012, at 1:29 PM, Adam Stokes
<reply@reply.github.com>
wrote:

>> +## (at your option) any later version.
>> +
>> +## This program is distributed in the hope that it will be useful,
>> +## but WITHOUT ANY WARRANTY; without even the implied warranty of
>> +## MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
>> +## GNU General Public License for more details.
>> +
>> +## You should have received a copy of the GNU General Public License
>> +## along with this program; if not, write to the Free Software
>> +## Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
>> +
>> +from sos.plugins import Plugin, UbuntuPlugin
>> +
>> +class udhcp(Plugin, UbuntuPlugin):
>> +    """DHCP related information
>> +    """
>
> I think I need an example as I dont see this way being used in any other plugin
>
> Thanks
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sosreport/sosreport/pull/34/files#r486732
If you're going to be rewriting these, please remove the unnecessary parens. They shouldn't have been there in the first place (though the examples may predate the ability to leave out the parens when there's only one argument). The same goes for all of the examples that you changed.
Is that only me but prefixing with DL seems unnecessary. 
After all we are inside of the DList.
Good point! Typo on my end.
insert an else here such that `if haystack.empty) return haystack` is only evaluated if you can't compare the lengths.
I asked to create a multi-line file :P
i like the fact that you added this to the doc! thanks!
Может так и лучше. Просто у меня цель стоит допилить то, что мне нужно, не испортив старое. Поэтому я стараюсь сделать так, чтоб если вдруг кто-то обновится, не заметил бы, что что-то поменялось - и не был бы вынужден что-либо переписывать/допиливать
check your formatting; i think we dont want to split here
I just checked how my code behaves, and added a simple check to see if another window is already editing hotkey preferences.  If so, it prevents the second (and third...) window(s) from concurrently editing them.  This is sub-optimal, and doesn't match how `<setting>`s behave, but when I tried a slightly-more-sophisticated technique to synchronize them, I wound up in an infinite loop.  So I'll stick to simple mutual exclusion for now.
[Use tabs for indentation](http://codeigniter.com/user_guide/general/styleguide.html#whitespace). Other lines as well. :)
Could maybe be an Out of range exception? Don't know the best practice.
"register the client the Customer Portal or another subscription management service." should be
""register the client to the Customer Portal or another subscription management service.")
Shouldn't the priority be supported when addign a subscriber here too ? Otherwise, it is inconsistent with the compiler pass.

For subscribers, relying on the registration order is impossible as they can subscribe to several events and may want to use different priorities
Двоеточие на следующей строке.
I think `File.createTempFile` would be better here.
this is WARN so it should be moved to the Log interface.
Hmm. This could result in a product-specific tag being created with the same name as an existing global tag. Does that make sense? I guess if we're going to do it this way, I need to review our tag-handling code to make sure it handles that case reasonably well.
These 4 lines are repeated in each function here. Can we make a util function to handle that for us?
@localheinz :+1: don't.
The rule is duplicated.
We should probably reduce this to one DB query, something like:


```
{% with person.claim_set.all as claims %}

<td>{% for claim in claims %}{{ claim }}<br/>{% endfor %}</td>
<td>{% for claim in claims %}{{ claim }}<br/>{% endfor %}</td>

{% endwith %}
```
how about follow this post http://stackoverflow.com/questions/30571/how-do-i-tell-maven-to-use-the-latest-version-of-a-dependency
to get the latest version?
This is wrong. Please revert this change
Not important, but in future, use 
```sh
php path/to/vendor/ZendFramework/bin/classmap_generator.php
```
No need to do this manually ;)
Will look into this tomorrow morning
"Hinzufügen", "Empfangen", "Löschen " und "Durchlaufen" sind Verben und müssen folglichkleingeschrieben werden.
This will load the CSS files with the same scheme as the page itself, I think you should keep it as https
You just had the routes confused.
indenting is inconsistent with the rest of the file here
Mi se pare ok logica, dar aș prefera să o ținem în cod Python, pe clasa `MarkupGenerator`.
An enhancement would be if this could be passed in via a properties file
This actually has no affect on the solution and is for information purpose (i.e. what tool actually wrote out the sln file).  The solution works in both 2010 and 2012.  It's actually one of my favorite things about 2012 so far and should make migration way less painful than previous visual studio releases.
Remove the resourceName variable pls (unless there's a good reason to keep it)
Why is it removing non word characters? I am not sure it is a bad thing, just wandered if there was a specific reason.
Yes, but pk will not necessary be the same for each subdocument between requests for sorted list? So generated URI will not always point to the same subdocument?
Does it make sense to extend `Record` (in that case, the `elems` field of the struct must be empty, right?)? The cast is also needed for the type constraint (`m: RefinedManifest[_ <: Record]`).
Doesn't this need to be the other copyright header? See the SDK source code.
Actually the would be very useful to have a `NearestNeighbors` implementation that works for high dim sparse data. And it practice, if you only deal with positive weights, it can be implemented as an inverted index. I think it would be great to have such a SparsePositiveNearestNeighbors the implements such an inverted index using numpy arrays or scipy.sparse matrices as backing datastructure. The goal would not be to implement a full featured search engine but just to have a way to perform simple yet exact  and efficient k-NN queries on (very) sparse data.

I also agree that this should not be addressed by this PR :)
Hier bräuchte ich explizit die E-Mail-Adresse deines Gravatar-Accounts. Oder du lässt es leer, dann wird deine Adresse von oben genommen. Werde die Beschreibung gleich noch updaten, dann ist klarer, was damit gemeint ist...
whitespace here and below >>
its fine if its a string, since this int will be converted to string anyways. The problem that caused this was when the user enter "fileType: 2". We recognized that "2" as a Double, so when we convert it to String, it becomes 2.0, which is an invalid fileType.  So a String that is "2.0" would NOT work.
throw a javadoc in here explaining what this is and a brief description of how to override it
Can you use the Guzzle lib to make http request ? I know it is an extra overhead for the current need, but it is better on the long run.
I agree. I am a big fan of asserts myself, and I usually use them for validating input parameters of internal methods. However, I never check such parameters for null values because using such parameters usually results in a NullReferenceException a few lines later.
because then u get the submenuarrow..
yes, it is better :)
Should we denote eval usage with: 

/*jshint evil:true */
eval(evalString)
/*jshint evil:false */

like the browser does?

Could be a good convention.
Useless new line
Consider moving this to a settings.py file
"A priori" ne prend pas d'accent sur le "a".