labofsahil/github-event-dataset-2012 huggingface dataset:
suggestion: write a new function:

Dygraph.shapeFunction_ = function(numSides) {
  return function(g, name, ctx, cx, cy, color, radius) {
    ctx.lineWidth = 1;
    ctx.strokeStyle = color;
    new Dygraph.RegularShape_(3).draw(ctx, cx, cy, radius);
  };
};

Then do:

TRIANGLE: Dygraph.shapeFunction_(3),
SQURE: Dygraph.shapeFunction_(4),
...
You tried to sneak this one by me you, huh? You POS!!!
For real, tho, this is fine, but why not stick with the default 5000 that foreman does?
I was only thinking that if `preferredType` got set to the empty string, would anything break? If so, I suggest doing `var preferredType = $(this.el).data("format") || "defaultType"`. Also, if empty string is a legit input for type, then no problem!
<headdesk> You're absolutely right.  Fixed now, and I put shears up to 0.3 per component to expose other possible errors that might not be obvious with tiny shears.
Do you really feel that no comment is necessary here either?
And here I got you covered :)
It looks like you have tabs here. Indentation is messed up.
Should this really be returning a map? The key is the same as was passed in to the path, so seems unnecessary. But if this is convention...
typo: MY_BROOKLYN_RAW_DATA_ATTRIBTE
Declare `selector` to be `objc.SEL` instead of casting.
Furthermore, where is `languages.español` defined?
Yes, we don't need an init method here.
It is covered with another test:
```javascript
      it("should send the sideloads in AJAX fetches", function() {
        expect(Em.Resource.ajax).toHaveBeenCalledWith({
          url: "/foods/1",
          resource: food,
          operation: 'read',
          data: {include: "burger,smoothie"}
        });
      });
````
цього теж не потрібно. я тобе вже казав, що page_range пагінатора це не кількість елментів на сторінці, а кількість сторінок в блозі пагінаторі
oxid should observe adminErrors, instead of pushing errors to an explicit shop system.
It wouldn't hurt.  I omitted since we don't build it but I guess better safe than sorry.  Will fix.
No no, this API is designed to replicate the old functionality (no arrays).  All this API is already deprecated.
thanks for the suggestion Jeff but in this case state doesn't matter. its grabbing the touch variable from the preferences and its state isn't as important as long as its in some form of true. good catch on the spelling error though, Conan the grammarian strikes again.
What about:

top = topview.view
top.subviews.find { |v| v.tag == tag } or top
Uninitialized isn't something the user needs a warning about.  I end up with 5 notifications all the time for no reason.  If it's a problem on the flight side then some warning/error should be set (e.g. if the GPS is meant to be running then System Configuration will fire).
yes, just a minute
space after the comma in the range
Любой запрос проходит через эту функцию, и если сюда повесить synchronized сюда, вроде должна появиться Thread-safety...
Это copy-paste
я когда заливал, то все было норм..
Suggestion: Better stick this in a css file, don't you think
I thought we wanted to use a different field to hold the fields that uniquely identify the object. For instance, the reaction object will be uniquely identified by the type and id. But I think we should leave this for now and update things when we need it in the reaction app.
Please never include images in a table; it doesn't work with PDF output.
This final comma breaks puppet 2.6 builds.
Just rename to ``denoise_tv`` and I think we're ready to merge!  Another job well done, thanks @ahojnnes
That's perfectly fine.  Our only guarantee to the user is that we'll take any input format, and return whatever is best for the implementation.  Users who want to make sure they get a certain format should do ``img_as_float`` etc.
Is there a need for a line return?
As far as I can see, the class also supports multi-class classification. I think this should also be mentioned here.
Do you think it should be public?
> Would this mean that there is one instance per score? I.e. this namespace
> contains an object called AUCScore?

Yes, or rather auc_score (object, no class, thus no CamelCase)

> How would you handle the case where there are additional parameters, such as
> fbeta or precision@k?

Indeed, with additional parameters, having only one global instance of
the scorer will be a problem. One way to solve it would be to teach the
user to use the ScorerDecorator, that could provide support for
additional arguments. I could do something like
<pre>
  GridSearchCV(..., score_type=AsScorer(fbeta_score,
                                               args=dict(beta=3)))
</pre>
(I just decided that I really didn't like the ScorerDecorator name :} ).

By the way, generating classes the way you do it will pose problems, as
they won't pickle (they are defined in a closure).

I am going to have to run to bed right now, I fear. Will keep on thinking
tomorrow.
Can you add a comment here stating what are the specific properties of the data matrix that is interesting here.

Also, ideally, if you could engineer a smaller data matrix that reproduces the problem, it would be great. The reason being that it would make the test easier to read, and thus the problem easier to understand, if ever this test is to break in the long run.
@vene could you compare if indexing via ``.data`` is faster?
On Thu, Sep 13, 2012 at 07:09:46AM -0700, Varoquaux wrote:
>    I'd love to find a better name than J. Any ideas ?

I don't know: I don't know or understand the algorithm.
Weird, maybe there is some difference in types between ``np.npy_intp`` and ``int`` between numpy 1.5 and 1.6?  I dunno, but you should probably remove the commented out lines below...?
This should be public final.

Package private cannot be accessed across module boundaries, so unless you install the DUP that sets this in the web module this won't be accessible.
Should we include empty specs?
> I don't think this should compile. It doesn't make sense to have a manifest constant which is initialized to void.

NO, the variables `cx` and `ix` are declared as *runtime variable*, NEVER as *manifest constant*.
I think such "implicit manifest constant" enforces special treatment to much meta-programming code, and the worst design in current D implementation.
Again, the two variables lives in *runtime*, then void initializer should be accepted. If we really want to manifest constant, we can use `enum` for the purpose. There is no reason to enforce manifest constant. For that point, I think Walter and you are completely incorrect.
Wouldnt' that conflict with other Knp\ namespaced packages?
This seems like it would be useful for other things too. please put it (and the Q_DECLARE_METATYPE) in its own header.
I wouldn't have the constant check.
You do need a check for ds->isPure() through if you don't have this test.
Yes, it prevents it allocating 4/8 elements when only 1 is likely to be used.
We'll end up with a long list of booleans... It's more effective to create bitfields here.
I don't check numCommitted outside the block, because putRow, can change # committed rows on antoer thread.
I'll try and reword.
Is this going to return NULL if the form field is disabled/empty?
`tmpl(options, fn)` would be fine here (didn't notice that I did this as well)
I'm going to kill this pull request, modify the licensing and references to Kevin Schaul and start again.
Ok ba on le laisse là alors, mais je ne sais pas trop à quoi ça sert ce truc.
@ctweney might be good to wrap with a nullToEmpty to guard against null entries in the postgres custom user entries, since table def doesn't enforce not nulls.
`(== (height rectangle) (width rectangle))` is sufficient, no `if` is needed.
Why linear retry
it seemed a useful place to capture the logic, happy to implement a better idea
This is a hack that I have reverted in d6f226d
Copy-paste comment
true or false?
no parens neeed. I also think the following is better, but just my opinion

if dcerpc.last_response and dcerpc.last_response.stub_data
Also, your comment made me notice that in test_random_walker.py, I did something evil, that is using random numbers without fixing the seed. We should instead use a random number generator with a fixed seed (as in http://scikits-image.org/docs/dev/auto_examples/plot_random_walker_segmentation.html, but with a fixed seed). My bad... This explains why the tests fail from time to time.

And I think the value of beta is a bit too large (the default value of beta has changed since 0.6), and this is why the tests may fail. I hope this is solved with a smaller value of beta and a fixed seed! 

Thanks for helping to improve the code!
hand.npz is simply a set of coordinates that I entered myself... you can actually just unpack the data into a list, and use it directly (it's not big at all).  Or make up any other set of coordinates that illustrate the principle...
actually this test (originally lines 32-34) is not useful any more, since we do not try to return an array with the same dtype as the input array. I would just remove these lines, and test instead that the returned dtype is float.
About 3 seconds when running against the preview db locally.
Can I no longer set the type through the `Response` constructor? I'd still like `new Request(array('type' => 'json'))` and `new Request(array('type' => 'application/json'))` to both work.
we have a little ajax here too... we should wait for the filter to take effect and then go to the next step
Did I write this comment a long time ago?

If the server provides credit, we should certainly use it.  If not, we should be able to find the credit line on the website, which looks like what we are using now.
I'd rather see this API use the promise system, which provides both callback and errbacks, both of which can be chained.
Is a developer going to need more context to make sense of this? What is it that it already has one of? property? Member? beer?
'height' should be capitalized here.
Remove extra new lines please =)
Shouldn't you return if you're invoking the callback?
split this onto mutiple lines 79 characters long
comments are flush to the margin, not indented.
Triple equals in JS is best practice, even though in this specific case you're likely to be okay.
Just put 3.2
There is such a function already.
Go with `$(LREF functionAttributes)`; if later on it's unneeded it will be equivalent to `$D`
What about 

T1 foo();
T2 bar();

fn(foo(), bar());
fn(bar(),foo());

?
@jabley: it’s mentioned in the Versioning section of the aforementioned Gem guidelines.
oops - I commented on the commit rather than on the pull request. Here is the same comment here:

can we move this to be inside the `if (... needs identifier) { ... }`
This isn't as clear to me, maybe it should be something like `Use the COMPONENTS list? to find other catkin packages`?
Is this `console.log` necessary?
Yes, that looks okay.
Probably it is no longer needed, but that is outside the scope of this pull request.
It currently complains about many things, but not about this one. I have decided to instantiate the object myself instead of letting users create their own dictionaries because I wanted to control which string comparer is used for keys. With my design keys are case-insensitive, and users cannot change that.
I am beginning to wonder what value this **if** clause adds. Why not just JSON encode everything?
Please replace `selenium-java` with `selenium-support` only
(comment)  I just wanted to footnote that this view (graph.js) needs some attention. It is  big, and complex.  It would be great to split it up into smaller controllers and smaller views.  Every time we work on it, we put more complex logic into the view.

We can't  address this now, but think we should foot-note this for a future refactor.  ( cc @rklancer )
Leftover stacktrace debugging ?
Good question.  I think I've been presuming that the most common scenario would be that PuppetDB and postgres would be on the same machine, but that would be a separate machine from the puppet master.  Not entirely sure if that assumption is valid but that was how I'd chosen the defaults before.
da $data ein Array von AccountsReceivable ist, wird das kaum klappen...
du musst aus dem AccountsReceivable $row auch ein array machen.

zb. array(
  "order_id" => $row->getOrder()->getId(),
   "usw" => asdlfjk
);
Yep. And there was also a typo - should be fixed and i added an integration test for this
Sorry for late reply.
In [Twitter Bootstrap](http://twitter.github.com/bootstrap/), navbar has ```z-index: 1030``` and drop shadow has ```z-index: 1040```. This is between them.
I guess they are. Will check back and when this differs on other platforms too I'll add AnyX86/Error branches.
I am not convinced this is correct - canAdd() in SimpleMessageGroupStore uses the sequenceSize to detect duplicates., as well as sequence numbers > sequence size.

While these decisions should not be tied to the SequenceSizeReleaseStrategy, it is possible that someone has implemented a custom message store that uses the canAdd() method using some other algorithm to reject messages. This change would break any such implementation.
Input[type=submit] still needs these rules.
````
typedef = {
    'int': (int, random.randint, '0,100'),
    'float': (float, random.uniform, '0,100'),
    'enum': (str, lambda *x: random.choice(x), 'a,b,c'),
}
```
Please move demo page to `/`.
It's because I didn't knew how to call it :)
I propose `SnijureContext` as it may hold other stuff than a context (?)
No, I think that code there looks fine, it's just that there's no real point in using ```.toString()``` on the column name there.
Use fully qualified paths for mocked class and it will work.
Also while you're in here can you move `2012-6-8-notifier.markdown` -> `2011-2-6-notifier.markdown`
it was there before and I'll put it back.
:disappointed: but there's not really a graceful way to do that.
Test config got cleaned up this PR, messing anymore with this would've pushed it back too far.
the keys in `config.jelly` don't use the .exe suffix so this should be fine
Why `EmbeddedDocumentField` has `resource_uri` in the first place? Is not `EmbeddedDocumentField` just "abstract" (in the sense that it is stored embeded in the parent document and does not have its ID)?
What does this file do, for what is it needed?
This was just a reminder, you prolly did it.
Now that it's merged, it should be very easy to improve on locally
On Mon, Feb 20, 2012 at 08:51:43AM -0800, Marc Remy wrote:
> >      ensure  => present,
> > -    require => Package["libapache2-svn"],
> > +    require => Package[ $pkglist ],
> 
> I made a successful test with 3 modules on rhel6 with puppet-0.25.5-1.el6.noarch
> Something changes in next version ?

Oops, you're right. This syntax seems valid since ...0.24.6.

apt-get update mfournier  :-)
This right here is the greatest function name in all of history.  Worthy of the Legion of the Bouncy Castle.
Ah! Why didn't u say before ? ;)

ok, right, unlockProperties, with state retry, unlocks by changeMode(RTM_NONE.
I only saw the changeMode below, the old way.

Yes, that's what I was missing.
To clarify the example code. If you like you can write it in a line, but for me it's a bit confusing.
Now that including the arguments names have the same intention.
this is the wrong way to do this. You could have several parameters in a value. what you need to do it ``$container->getParameterBag()->resolveValue($storageDefinition->getClass())`` (which works in any case)
Not on method parameters.
Done. Meant to do that earlier but just forgot.
can you file a RC ticket for this?
Функция для пометки узлов на удаление. Помечает все узлы в дереве. И ставит таймер на удаление помеченных узлов.
I'm hesitant to rely on any 1.x version of `armstrong.apps.content` with out some testing of the possible consequences when running a `pip install armstrong` with those dependencies explicitly stated.  I have a feeling they could possibly get out of control should a `*.*.content==1.3.0` that's picked up by `*.*.articles` with `armstrong` having a `1.2.x` dependency.
Avoid global state. Could you please use a local variable instead a global? Just create it in the `Extension` class and pass into `Indicator` constructor.
nonon. i ment.

I ment if i manually open up a new browser, etc (or even use command line .. like `curl` ..) and manually enter in the callback url .. with no data, etc. what would the value be? an empty instance? `null`?
this has to be PHPCR\NodeInterface (and then you do not need to use the mock builder).

btw, you should use \\ for backslash in string. \ is a special character even in strings with single quotes.