labofsahil/github-event-dataset-2012 huggingface dataset:
Will fix.

Normally I do that to create better exception messages instead of one hugh try/catch list.
I wonder if MutableMap can just be represented and deserialized as a default Map.
I view MutableMap as really useful for its fluent builder, but not much else. Once it's instantiated then one can/should  treat it just like a normal map.
Why static? Is it only possible to talk to one instance at a time?
I guess this is "safe" anyway.
to be left destitute, as consumers get their work for free POINT.
原本的 code 有 parse HTML HEAD 裡 favicon 的 url ，你要不要先看有沒有這個 URL ，如果有，優先抓該 URL ，如果沒有 parse 到任何 favicon url 在用這個 default 值
Why is this subclassed from Commentable? Can someone comment on a comment?
dateAdded
Is this really the Who?  It's more of the Where?
Should these all be 2 space tabs?
*facepalm*
if we use torso as baseicon we have individual_limbs=1, in that case if(!individual_limbs) fails and does not remove anything at all.
And if we use any other baseicon (which HAVE ALL LIMBS) we just remove destroyed ones while someone would come and draw a husk torso and a fat torso.
By the way, if you're husked - you are lying most of the time, so no holes through you when arm is masked out.
exemplar is the a array of the type being deserialized from json
klass is type of objects that will come over the pipe.
What exemplar and klass means?
`for (var i = 0; i < thList.length; i++) {`
what is the 2nd param for?
readLine может возвращать null
Подчёркиваний не должно быть в названии переменной.
А если extension == null?

Теряешь исключение e передавая e.getCause()
@ctweney Liberal refactoring of the fakeable proxy, feel free to coment
PEP8 issue: test_statistics_page.py:74:1: W391 blank line at end of file
No worries - I should be apologizing for not elaborating. The term <code>value</code> doesn't feel very descriptive to me - for example does this method accept a <code>integer</code> as a value or a <code>list</code> of values?

The method appears accept a string of text.
Extra whitespace must be removed
Move this outside of the class
Wow, I'm really jumping the gun here. Ignore the last message please ... I'm actually going to delete all of these.
This line seems to be incorrect - it prevents ics/steve/*.ics, ics/paul/A1A2_gradient.ics, and ics/paul/GCI/simulated time resolved.ics from opening.

Could you please have a look at fixing that, and make sure that then all of the sample ICS files that we have are readable?  Instructions for how to test that are here: http://trac.openmicroscopy.org.uk/ome/wiki/BioFormats-CommitTesting
This URL is wrong, the repo has been renamed.
I wanted to keep one test function for one property to have a clear structure (property == unit for me here).
git checkout master
    git pull upstream master
    git co your-branch
    git rebase master

There is possibly a conflict in the __init__.py file...
`f` => `pred` or `predictions`
``pl.subplots_adjust(wspace=0.4)`` prevents overlapping y-axis labels
It is not clear to me what this comment line means, would you mind explaining?
It seems ``eigen_tol`` is not used yet (currently it is also ``eig_tol`` in ``SpectralClustering``). I'd be in favour of renaming to ``eigen_tol`` to be somewhat consistent in naming.
checked -> check ?
In scikit-learn, we are used to put a trailing underscore to all fitted attributes (`self.W_`).
I will issue a warning if everyone agrees on this renaming. My original concern is that the original `mode` is a little misleading so I change it to `eig_solver`
mhh, I am not sure I understand. Yes, I tested with p in the global namespace but I also tried to assert these expectations in the ---> class JukeboxTest < Test::Unit::TestCase 
where things ended up too long because I guess Ich verstehe es doch nicht so ganz. But nevermind, I kriegs noch raus. :)
I do not think it hurts anything to show some real url.
This was the old avoid ping pong (it didn't play well with computed properties) that was replaced by the suspendObserver calls in https://github.com/kselden/ember.js/blob/8975853944ffbc973b61a8ac11b0ed214b8482e1/packages/ember-metal/lib/binding.js#L533-554

I meant to remove it not just comment it out.
I didn't mean delete the entire TEMP. Create some GUID name folder whenever you need to do TEMP work. In there you can have arbitrary structure. But when you're done, you can safely delete the entire GUID folder without affecting anything else or leaving any garbage.
Memory layout rules are part of the platform ABI's.  Are they similar enough across all the supported platforms to use LP64 here?
If `prot() == 0`, `Pprotectionnames[prot()] == NULL`. And then, The NULL check in `property` will eliminate this `if` branching.
fix order for the changes in this file.
Do we really require a comment and is that always sent from shopify?
Ok pour la doc trésorerie mais au moins faire les autres documents en attendant.
Pour la route au lieu de faire ton gros 'if' ici, essaie de faire un switch dans une fonction de Doc qui renverra la bonne route en fonction du type de document.
L'identifiant du document (ou élément) placé en quarantaine.
et la sauvegarder pour prendre en charge de nouvelles variables
delete this commented gems!
These values are in uppercase in avoinministerio, and AFAIK validation is case sensitive.
This either wasn't tested, or the function is never used.

You need to call getdata().  refdata() clears the result, so this condition would always be true.
Don't need to check row is non null.
I think you mean row->getOperator()  != no_createrow
I'm not a fan of this technique at all... but it seems this whole file is implemented this way.
Currently, the instance directory and the common run directory is hidden and can be only changed manually. However, I agree with your comments that this code belongs to the component.
`&` --> `&amp;` here.
breaking before also does not help
@shacker Good catch, I'll remove it
Where is the 'db' variable defined?
Starting at 60 seems an odd way to do it.  I would start at zero and use ```(i % 60) == 0``` in the if condition below.
Not sure if @jlee-r7 has something to say about code reuse or would like to something like moving it to some mixin... just warning him :)
`(width, height)` switched in docstring for `image` and `segment_mask`. Same goes for quickshift and slic docstrings. I'd vote for removing the shape spec from the docstring (i.e. just specifying `ndarray`) for all these docstrings. Since `slic` requires 3-channel images, it might make sense to keep the shape spec in that docstring, but I think it'd be better to just specify `Input color image.` or `Input RGB image.` on the description line for `slic`.
jump directly to `ll. 455, 693` - other diffs originate from file reverting
I'm not sure what you mean.  If I remove the value_outside_of_array call then Im' just passing in that it disallows_value_of the entire array which it shouldn't be doing.
`if( !Popcorn.vimeo ){` => `if ( !Popcorn.vimeo ) {`
Actually, these should be `czm_` (eventually) and documented since you are treating them as public functions.  This doesn't need to happen right away, even before going into master.
I'll let @shunter take this one.
Minor optimization thought, why not check values most likely to be different first, like text and position.  This way the comparison can short-circuit as soon as possible.
`renderState` is a bad name for this since `renderState` is already used in Cesium - and computer graphics in general - for the depth test, stencil test, blend mode, etc.  Ultimately, `update` will probably return something called a `drawTree` - dare I describe it as a mini-scene-graph.
I think you might want to be clearer here, since it's a little ambiguous whether you're checking the equality or asserting it. What I think you mean is that this line *sets* the variable `num` to be equal to the value returned by `rollDie()`. Just make sure you're explicit. Compare:

```js
// *checks* whether num is equal to the value of rollDie()
num == rollDie();
```

to

```js
// sets num equal to the value of rollDie()
num = rollDie();
```
Would leave this method declaration/definition in place even if we do not currently use it. Will be useful if and when we write an animation engine that can handle animating multiple views together.
Should check if(tagEng+1 < html.length()) before calling substring().
Yea I did last night :)
We should just have the config file as a module that we can require(). That way we don't have to do all this sync file reading and evaling.
If that's the case, then I would like text to be labeled as iOS only...I'm going to remove it from Mobile Web because we had to do an ugly hack that violates some JavaScript best practices to implement it.
Тип неконсистентрый `{Number|Date} reminder`, а фактически может быть Function. Лучше лишний раз функции не порождать.
Source formatting note: new line needed here as well (and after line 311) because the difference in the method chain.
why begin and rescue here? Can't we just use the second form on all rubies?
This is important because people may use the conversion to bool in tests.
Are you sure this is »forgotten, useless code«? I didn't read the `Array` source too closely, but it seems like this checks if the passed range actually belongs to this instance. `enforce(_data)` might really be unnecessary, though.
Stylistic nitpick: Omit the parens in `!()`.
The support of opApply is not documented, so I'll straight up deprecate it then. I'm not sure an outright delete is called for, in case somebody accidently (or purposefully) called it.
this module has a bunch of styles, sigh
Version numbers should only be specified in the root `pom.xml`.
s/Get/Got/
Please file a bug to be able to do this via UPDATE on the ContentProvider. No sense doing two hits on the DB in separate transactions.
keep set methods together
Bad link, I see those lines right here: https://github.com/y-trudeau/resource-agents-prm/blob/master/heartbeat/mysql#L582
why configuring the profiler bundle here ? This file is about LiipImagineBundle
I believe we can delete Devise::PathChecker from the repository.
When you first put the name of the class in a parameter at the top of this file, and then you use that variable, then it's possible to use your own classes in your project so you can use some other class for the "security.acl.permission.map" service. this can also be done for other services. see https://github.com/stof/FOSUserBundle/blob/master/Resources/config/security.xml
Please don't bring unrelated dependencies
If I change the naming convention of the bundle, that seems like a lot more effort than your solution here. I guess we should have a change log to note things like this.
It seems like you could just store dataoffset() here instead of extracting the actual pointer.
http://www.freedesktop.org/wiki/Software/PulseAudio/Documentation/Developer/Clients/DBus/ConnectingToServer implies that it should be on the session bus, with a possible override for PULSE_DBUS_SERVER, which i don't think anyone in our case is ever really going to tinker with
correct me if i'm wrong, but should the `opacity: $opacity / 100` already solve this
Блин, когда уже люди прекратят юзать `""` вместо `QString ()`?
The issue here is that I need to append values to an initially-false variable:

    $list: false;
    append($list, foo, comma);

This results in:

    $list: false, foo;

I expect:

    $list: foo;

The only workaround I've come up with is to set the initial $list value to an unquoted string. Unfortunately, this adds an empty space to the beginning of the value and fails during testing:

    $list: unquote("");
    append($list, foo, comma);

Result:

    $list:  foo;

Expected (sans space):

    $list: foo;

Any ideas?
Thanks for merging these in!
This module ought to have a docstring, right between the license block and the first import.
:hand:

See my [comment](https://github.com/php-fig/fig-standards/pull/16/files#r532945).

Especially note the position of the comma, too.

```
$object
    ->foo('value1', 'value2')   
    ->bar(42, 88)
    ->baz()
;
```

Or even better (see my [comment](https://github.com/php-fig/fig-standards/pull/16/files#r532981), can't seem to link to a line of your code):

```
$object
    ->foo(
        'value1', 
        'value2'
    )
    ->bar(
        42, 
        88
    )
    ->baz()
;
```
```
$id = $this->getRequest()
    ->getParam('id', 'foo');
```

I know it's not a fluent interface, yet it could solve the "weird" (imho) wrap problem combined with the 75 chars line break.
Can you break these callbacks into separate functions?
I think we probably want to separate out the setup from the actual showing, so you only have to set up once and then you can show/hide at will.
Is there a reason to use the dot notation one time and declaring it within the object the other time?

Can we do this instead:

```javascript
            var newRow = {
                'id': sakai.api.Util.generateWidgetId(),
                'template':  'row',
                'sakai': sakai,
                'columns': [
                    {
                        'width': 1,
                        'elements': [
                            {
                                'dummytext': true
                            }
                        ]
                    }
                ]
            };
```
We should be using `on` instead of `live` in this file.
Oh, now I see what you're getting at. Yes, I'll remove it.
This should use the TemplateFunctions.

The call is just: ${ func.callTemplate("Default Patient Identifier") }
?? cachedSessionView is always null if not CCF - why not test for null here? Or, initialize it with an empty map and overwrite if CCF found in onInit() ??

Just seems odd to catch an NPE rather than testing for a null reference.
Yeah, ironically you can see how in one component connection-factory *WAS* required. I had to fix the schema, but anyway, i'll fix this.
Ah, forgot to change the whole function to spaces as I've been doing. Adding a commit in a second.
Looking at your code and what fills this field, i think name is a misnomer. It should probably be
``HttpException(406, 'Export format not allowed')``
then I have to fix it in DoctrineBundle as well :)
IIRC this depends of the region used, isn't?
Is chunk a buffer (I haven't ran it yet). If so, should the encoding be 'utf8'? I'm assuming toString just defaults to utf8 here (checked, yes).

Still, I'm a little unclear on how this line converts an ISO-8859-1 or UTF-8 encoded format correctly. Can you elaborate a little. Thanks!
If you call something "Sorted" and don't actually sort, it's rather misleading. You are doing nothing enforce the use of mongoengine.SortedListField with your EmbeddedSortedListField. So if it doesn't sort and isn't specifically tied to a SortedListField, the "Sorted" part of the name is rather meaningless.

EmbeddedListField doesn't work very well. If I switch from EmbeddedSortedListField to EmbeddedListField, I get errors. Maybe you could do some more testing. Your test app doesn't use EmbeddedListField at all, and only tests creation, not updates or deletion. A PUT on a document in an embedded document list copies the document instead of modifying it.
А якщо контексти додавати через useContext?
http://www.craftitonline.com/2011/07/behat-mink-new-use-of-aliased-subcontexts-complex-step-chaining/
Sounds good. What was the reason for the passivation capable requirement?
It does. But it at least stops the site from crashing until we have a good
strategy for dealing with that.
On Oct 2, 2012 4:01 AM, "Haak Saxberg" <notifications@github.com> wrote:

> In DjangoProject/HvZ/forms.py:
>
> > @@ -95,6 +96,13 @@ class EatForm(forms.Form):
> >          help_text="If you got a particularly epic meal, go ahead and describe it here",
> >      )
> >
> > +class UserEmailField(forms.EmailField):
> > +    """Checks that no user has its email address."""
> > +    def validate(self, email):
> > +        super(UserEmailField, self).validate(email)
> > +        if User.objects.filter(username=email).exists():
> > +            raise forms.ValidationError("An account with that email already exists.")
>
> Don't we want extant users to be able to register, though?
>
> I think this prevents that from happening...
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Claremont-HvZ-Web-Team/claremontHvZ/pull/61/files#r1737454>.
>
>
Could use a better description of what the params mean
is this required right now?
I was just planning on doing this properly after feature freeze, but I can make a sort-of-working version for now if you think that's better.
а зачем тут cast нужен?
b/c its calling the function that is throwing an exception
These two descriptions are redundant, can you remove the first?

"This is the parent class for objects that can be created in Javascript and can also be returned from Java to JavaScript without the need for explicit creation."
"A proxy is a dynamic object that can be created or queried by the user through a module or another proxy's API."
${spring.integration.version}
@rande not sure if i'm missing something here, but that's actually how it works right now. checkout my current configuration: https://gist.github.com/3979600. 

If you don't specify a queue parameter to the handler command, the `AMQPBackendDispatcher` will return the default queue. You only get an exception if you specify a queue that does not exist.

If you produce a message which does not map to a queue, the `AMQPBackendDispatcher` will also choose the default queue.
Should these hardcoded strings be replaced with I18n strings instead, to start with?
Функция для удаления узлов, помеченных на удаление.
it works on the CI server, is it not using the correct Node?
Do you need to add new unit test cases for parsing <rim:permission system="true">xyz_permission</rim:permission>? And there should also be a test for parsing <rim:permission> which does not have the system attribute?
Spec doesn't specify.
Yea. If there is an error receiving the query targets or there are no targets available to share I would imagine that there shouldn't be a Share Link menu item.
It's unnecessary case. The `metadata.json` strongly restrict shell version. So this case will never been reached.
looks good, but then pls adheret to the license definition and include
their licenses into our packages

On Tue, Feb 28, 2012 at 09:55, pgrm
<reply@reply.github.com>
wrote:
>> @@ -0,0 +1,5 @@
>> +﻿<?xml version="1.0" encoding="utf-8"?>
>> +<packages>
>> +  <package id="DotNetZip" version="1.9.1.8" />
>> +  <package id="HtmlAgilityPack" version="1.4.3" />
>> +</packages>
>
> both are Ms-PL (http://htmlagilitypack.codeplex.com/license and http://dotnetzip.codeplex.com/license)
>
> i guess the important parts from the licence for us are in §3
>
> (C) If you distribute any portion of the software, you must retain all copyright, patent, trademark, and attribution notices that are present in the software.
>
> (D) If you distribute any portion of the software in source code form, you may do so only under this license by including a complete copy of this license with your distribution. If you distribute any portion of the software in compiled or object code form, you may only do so under a license that complies with this license.
>
> (E) The software is licensed "as-is." You bear the risk of using it. The contributors give no express warranties, guarantees or conditions. You may have additional consumer rights under your local laws which this license cannot change. To the extent permitted under your local laws, the contributors exclude the implied warranties of merchantability, fitness for a particular purpose and non-infringement.
>
>
> @D - according to http://www.apache.org/legal/resolved.html - we could also stick to the ASL 2.0
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/openengsb/loom-csharp-visualstudio/pull/9/files#r495012
@fightmaster: thanks. FYI: as it stands right now this PR is not going to be merged in - the remark you made and others I have pointed out are preventing it from being accepted/merged...
Let's have a default value in case configSettings aren't provided as expected
consider using a template