labofsahil/github-event-dataset-2012 huggingface dataset:
It would help performance in the server itself
I remove code and rebase commit.
「grails run-app」のデフォルトであるdev環境では、Grailsアーテファクト(GSPやコントローラ)のソースファイルの変更を監視していて、変更されたらリコンパイル・リロードが行なわれるが、prod環境では一切そういうことをしないから、更新検出のオーバーヘッドが無くなる、ということかな?
boolean isWifiPreferred = (prefs.getString("pref_net", "0").equals("0")) ? true : false;

is the same as:

boolean isWifiPreferred = (prefs.getString("pref_net", "0").equals("0"));
Yes. Testing the result of the possible configure options is fine. BTW lars do you know with there is no installation of -dev includes for addons builders? You keep duplicating XBMC code (atomic stuff was the last example)
Oh dear...

```
$ python plot_equalize.py 
Segmentation fault
```

```
Program received signal SIGSEGV, Segmentation fault.
0x00007fffe8361830 in MakeHistogram (pLookupTable=0x7fffffffb270, 
    uiNrGreylevels=256, pulHistogram=0x26f43d0, uiSizeY=64, uiSizeX=64, 
    uiXRes=512, pImage=0x202876260) at skimage/exposure/clahe.h:241
241		while (pImage < pImagePointer) pulHistogram[pLookupTable[*pImage++]]++;
(gdb) bt
#0  0x00007fffe8361830 in MakeHistogram (pLookupTable=0x7fffffffb270, 
    uiNrGreylevels=256, pulHistogram=0x26f43d0, uiSizeY=64, uiSizeX=64, 
    uiXRes=512, pImage=0x202876260) at skimage/exposure/clahe.h:241
#1  CLAHE (pImage=0x2875e60, uiXRes=512, uiYRes=<optimized out>, Min=0, 
    Max=4095, uiNrX=8, uiNrY=8, uiNrBins=256, fCliplimit=<optimized out>)
    at skimage/exposure/clahe.h:130
#2  0x00007fffe83626e8 in __pyx_pf_7skimage_8exposure_10_adapthist__adapthist (
    __pyx_v_clip_limit=<float at remote 0x90fa70>, __pyx_v_nbins=256, 
    __pyx_v_ny=8, __pyx_v_nx=<unknown at remote 0x100>, __pyx_v_max=
    <unknown at remote 0x2160fff>, __pyx_v_min=0, 
    __pyx_v_image=<optimized out>, __pyx_self=<optimized out>)
    at skimage/exposure/_adapthist.c:1328
#3  __pyx_pw_7skimage_8exposure_10_adapthist_1_adapthist (
    __pyx_self=<optimized out>, __pyx_args=<optimized out>, 
    __pyx_kwds=<optimized out>) at skimage/exposure/_adapthist.c:1148
```
dependence must not be bounded by 0 and 1
Could you seed the rng inside the test to make the test outcome independent of the test ordering? For instance, add the following line at the beginning of the test:

```python
rng = np.random.RandomState(0)
```
Just out of general interest: Did you profile this and it made a difference?
@ndawe can you rename ``learn_rate`` to ``learning_rate`` - we recently changed the name in the ``gradient_boosting`` module to be consistent with other estimators.
I'll make it LastSuccessEndTime
@WalterBright This test case should reproduce the std.format bug.
OK. WIll do.
Language features are supposed to go through valid -> spec change -> warning -> deprecated -> error -> removed.  That was the idea, anyway.
Looks to be an extra space here... was that intentional?
Is this really necessary? I created a project (which uses tibbapp, not tibbtest) and it ran fine.
I'm thinking of making a specialized stream class for serializing to the editdata - doing things this way is still a little stupid but it is more convenient than before...
Can you eventually add support like doctrine/DoctrineModule#88 here?
No rollback()? 

Looks like (confirmed with Renato) there's no mechanism to rollback a removed super..
If there were attributes in the list, wouldn't you get them rather than the last dataset?
or even actualRows
Agreed that the single if-clause would be a bit better, in part just because it would make the diff shorter. Not a big deal though.
```unless user_id```:question:
Won't be needed if `line_items.create` were `line_items.build`. Saving the order would save the nested line item.
This fails for me with the .h.  Removing it gets things passing again.  As we don't include the .h elsewhere, I'm going to remove this.  Let me know if that causes any issues on OS X.
probably worth doing something like:

```js
var engine = requires.dust;

if (!engine) {
  engine = requires.dust = require('dust');
  engine.onLoad = ...
}
```

so it's not re-assigned each time
No empty line after the docstring please.
Ok sure thing! I will amend it now.
The loader required that I `composer install`. Maybe the loader should get refactored then?
Well, the cause is "allocating twice the DMA buffer", but what is the result? I suggest to add something about the MBP too.
Maybe it's a naming thing then. If ``SENDGRID_CREATE_MISSING_EMAIL_MESSAGES`` existed before we added the newsletter-specific handling then it sounds more general. To me ``SENDGRID_CREATE_MISSING_EMAIL_MESSAGES`` should apply to newsletter and non-newsletter events.

The name ``SENDGRID_CREATE_EVENTS_AND_EMAILS_FOR_NEWSLETTERS`` sounds to me like it will create the missing emails for newsletters only.

In either case, appending ``_FROM_EVENTS`` to the name is more clear all around.
In java you need to use equals instead of == if I remember correctly.
remove all Thread.sleeps!
@dkubb here's where I 're-write' left and right. This was the only way I could come up with to get correct operands that don't include key attributes.
Why change uid to 1? That means that the test will always run on the same user, so the test is in different conditions when it's run on a non-blank db.
Turned on SublimeLinter. There were spaces instead of tabs.
I get a 403 from this:

```
$ vagrant box add oneiric64 https://github.com/shaftoe/vagrant-ubuntu-oneiric64/raw/master/oneiric64.box
[vagrant] Downloading with Vagrant::Downloaders::HTTP...
[vagrant] Downloading box: https://github.com/shaftoe/vagrant-ubuntu-oneiric64/raw/master/oneiric64.box
[vagrant] Downloading box: https://raw.github.com/shaftoe/vagrant-ubuntu-oneiric64/master/oneiric64.box
Bad status code: 403

Please verify that the box exists and is accessible. Also verify that
this computer is properly connected to the internet.
```
Move the declaration to the top of the function.
Dette vil ikke fungere når du kommer til den 8. og 9. i hver måned, eller til august og september. 

read-string tolker tall som begynner med 0 som oktaler, så (read-string "08") vil gi java.lang.NumberFormatException.
```OptAddress``` please
I switched it to "Appears" anyway.
Maybe: "Yield images of the gaussian pyramid formed by the input image."
##### `numpy` casting change

fixed with:

```python
    a *= dtype((2**m - 1) / (2**n - 1))
```
Please update this to match the doc. I'll merge and create a new release once that is done. Thanks!
i don't know if i understood right, but if i try to set in my define_model the attr to attribute, it will raise something like ActiveRecord::DangerousAttributeError

is that what you recommend me to do?

    @model = define_model(:example_foo, :attribute => :boolean) do
        validates_inclusion_of :attribute, :in => [true,false]
      end.new(:attribute=>true)
no phpunit tests
Hmmm. Thus far I have essentially been mimicking the existing PHP Twig code. And in my FormExtension, I want access to the environment like this:

Set it. I don't want to do this in a constructor to a) mimic the existing PHP code and b) because I am imaging (in the near future) a very cool Symfony2Js that uses a container instead of global JavaScript variables (e.g. `Twig`)

```js
/**
 * @param {twig.Environment} environment
 */
twig.FormExtension.prototype.initRuntime = function(environment) {
    this.env_ = environment;
};
```

Use it as expected:

```js
/**
 * @param {twig.FormView} view
 * @param {string} section
 * @param {Object=} opt_variables
 * @return {string}
 */
twig.FormExtension.prototype.render = function(view, section, opt_variables) {
    opt_variables = opt_variables || {};

    var mainTemplate = section in ["widget", "row"];

    if (mainTemplate && view.isRendered()) {
        return "";
    }

    if (null === this.template) {
        this.template = this.resources[0];

        if (!(this.template instanceof twig.Template)) {
            this.template = this.env_.createTemplate(goog.getObjectByName(this.template));
        }
    }
    // ...
};
```

Let me know what you think because at this stage I am still very much decomposing both Twig and TwigJs and I've not had a rea use case for this.
`y` does not refer to “yank” here. The vim-surround maps `ys` to one of its commands. It will only ever mean “yank” if you wait too long (more than one second) before typing the following `s` (depending on the settings of `timeout`, `ttimeout`, `timeoutlen`, and `ttimeoutlen`).

The documentation suggests “you surround” as a mnemonic for `ys`.
is this what we really want though? if the width is 2 and the outlineWidth is 2, don't we want to be able to pick the entire line width which is 4?
Hi Daniel, wdyt about extract it to another class responsible to handle HTTP responses?
there should be a space before the bracket
useless function `group-by-count`, with just have to use the `frequencies` function
all these these methods to the same, is this intentional? if yes should probably only implement once + delegate.
Defining locals sounds like a good idea, but be cautious of the naming. When merging, Timsort deals with three runs at a time at the end of the stack. So when naming these locals, be sure it reflects:
Run1 = stackLen - 3
Run2 = stackLen - 2
Run3 = stackLen - 1

As for stateful process, I'm not sure exactly what you're proposing, but it doesn't sound good. I wrote my code for maximum efficiency. That meant exclusively using static functions with references, but I used references as minimally as possible. This can be seen in mergeLo and mergeHi, which *return* the new value of minGallop, since accessing this value through a reference would be detrimental to performance.

I don't know what effect it would have if you wrote it as a class, or stored the "state" in a struct. But I originally tried using nested functions, and the performance was awful.
`foreach (i; 0 .. range.length)`
> If some struct store its pointer somewhere. And when (I hope) GC will become "moving", such struct will just add a check if it is allocated in GC heap and if not, still consider itself unmovable.

I just had a discussion about this in another pull, and the conclusion (according to me), is as stated by TDPL 7.1.3.5:

> Because of postblit, D’s approach to copy elision is simple and largely automated.
First off, D objects must be relocatable, that is, location-independent: an object can be
moved around memory by using rawmemory move without its integrity being affected.

So, unless Andrei disproves this , EVERYTHING in D is movable.

I'm not sure how a moving GC will cope with this, but that is the current state of the language.
return a ref T
Good idea, I'll add a few more tests for other predicates.
Is this oK ?
Nevermind me, its early in the morning. Looks good.
This will never be called. Was the intention to fail the test or raise the exception?
<tfoot>
This is so that we can do this:

	Knockout.Computed(() => {
		TwitterApi.GetTweetsForUsers<object[]>(
			self.EditingList.UserNames.Value,
            result => {
                self.CurrentTweets.Value = result;
            });
	});

It is an action based on any changes to any of the used properties.
А это updated_at
Support tabs pinning.
не уничтожится. У одного автора может быть несколько твитов, например.
Публичные не-метаметоды с большой буквы.
Хотя этот бы я слотом сделал, пожалуй.
Hmm, I'm a bit torn on this. On the one hand, good to be safe. On the other hand, Case Conductor is clear about its requirement for MySQL 5.1+ with InnoDB, and we aren't generally trying to make it database-agnostic (especially not to databases without transaction support); lots of other parts of Case Conductor will break if you try to use it with a non-transactional DB. So it feels a bit weird to make a little gesture towards supporting such databases in this one case, when we don't really support such databases. On the whole, I think I'd lean towards removing this bit, unless you have a good reason for it I haven't thought of.
I've modified script a little bit and now it should work in IE6 and IE7, but with this changes we loose function code formatting.
Please remove this whitespace change
As I said before,  let's decide how transport client and settings will interact with each other when we'll get settings reworked.

Fow now my understanding is following:
1) Client should be instantiated with an auth provider instance provided - agree.
2) Client should have its own settings too - at least endpoint url.

From the other hand, possible, IYandexApiClient may should be authentication agnostic: only specific implementations should assume usage closely together with an authentication provider.

Let's make final decision during the second turn.
Removed breadcrumbs from ::buildQuery
spring:message?
better to use startsWith()
In other words we are not talking about some claim check or outbound adapter. . . just a MS
I think we can kill 2 birds with one stone (make the class package protected while allowing early migration to the 3.0 structure) - take a look at

https://github.com/garyrussell/spring-integration/commit/75b584770ce9504e842e5cf3a242266e4b2ae3b3
i have to describe the lifecycle:
- manager instance is created
- local attributes are applied
- init method is called (from this point on, the jndi binding is available)
- start method is called

the problem is that through the lifecycle it is not possible to assign the global config first and then override the attributes with the local config. you have to first set the global config, then you have to override it with the local config and then you can set the attributes if a value is present.
Yes it's easy to do without adding complexity.
The date is not correct anymore, and the block must be re ordered
You can set and get the value via:
`view.settings().set('sidebar_results', True)`
`view.settings().get('sidebar_results')`
On 03/01/2012 01:57 PM, Diego Zamboni wrote:
> Ah - I had never seen that bundle in the stdlib.
> 
> I just checked, and it seems that set_variable_values2() was
> created as a workaround to a bug that existed at the time, but has
> since been fixed. You should use set_variable_values() now, which
> does not need the filename as an argument. In fact
> set_variable_values2() will shortly be removed from the stdlib.
> 
> If you could please make this change, and I will merge the pull
> request.

Well that bug still bites me, which is why I am still using it. I
stick with the CFEngine provided RPMs and there are no updated RPMs
available beyond 3.2.1.

I suppose I could pull set_variable_values2 into the bundle and name
it so it wont conflict.

-- 
Nick Anderson <nick@cmdln.org>
Please this line separate it in various lines
You could also just do an else after the if (i == 0) instead of using a continue statement. Continue is kinda like goto, and generally frowned upon unless there's a very good reason.
Locally you need to start running mongo. On Cloud Foundry just do

``` bash
vmc create-service mongodb mongo-asms
vmc bind-service mongo-asms node-express-start # Or whatever the name is of your app

```
You can combine both of these `before` blocks into the higher-level `describe` block:

```ruby
describe '#activated?' do
  subject { GoogleAuthenticatorRails::Session::Base }
  before  { GoogleAuthenticatorRails::Session::Base.controller = controller }

  context 'controller present' do
    let(:controller) { MockController.new }
    its(:activated?) { should be true }
  end

  context 'controller missing' do
    let(:controller) { nil }
    its(:activated?) { should be false }
  end
end
```
And perhaps add some negative tests to the above too:

```ruby
specify { GoogleAuthenticatorRails::valid?(code, secret.reverse).should be_false }
specify { GoogleAuthenticatorRails::valid?(code * 2, secret).should be_false }
```
Since we are pointing to the list of supporting, could we link to :bioformats:`File Formats <formats>`?
it is a bad idea as ``.`` has a special meaning in jQuery events.

Listening to ``sonata.collection-item-added`` means listening to ``sonata`` but with a way to remove only the listeners marked as ``.collection-item-added`` if we want.
This settings/debug pattern is something not in this gem currently. Please remove them.
This will not work because you do not have a patient identifier type in the system yet for "CCC Number".  You will have to create that first, and save it in order for the MohCacheUtils class to find it.
Hmm, now that you mention it its probably best not to.
why? You can have several constructors
You are right. My bad!
And add tests for the cases that raise expected exceptions with `assert_raises`.
Cosmetics: insert a new line here.
Could we get the password fields moved underneath the username fields, please?  :)
you don't need the ``else`` at all as the ``if`` returns. Simply use

```php
<?php
if (isset($this->filters[$filter]['options'][$key])) {
    return $this->filters[$filter]['options'][$key];
}

return $default;
```
the role="option" in li is missing
The implementation comes from [this `define_method` block](https://github.com/rails/rails/blob/ad46884af567d6f8d6d8d777f372c39e81a560ba/actionpack/lib/action_dispatch/testing/integration.rb#L321-L331).

I doubt we'll need to integration test OPTIONS requests anywhere else; CORS is the only common use of it I know of. So I'd prefer to make the motivation for it more obvious by keeping it close to its use.
Pass in RequestContext(request) to render_to_response and use request.path instead?
Same story, relative paths MUST have current dir.
We could check if class_exists first. Thoughts?
Eep! Right you are.
Should be the same description as for `setProperty` except with an addendum concerning whatever it is that fires.  Otherwise people are wondering about the difference between those two methods.
Why was this change made? The Protocol is for a view that does not have pages, but is a single continuous area. That is, for scroll views, not scrollable views.
Don't rely on ShrinkWrap View type, check archive.name ending instead. the Type could be anything, the archive.name is what the container understand.

related, https://issues.jboss.org/browse/SHRINKWRAP-402 https://issues.jboss.org/browse/ARQ-891
"application"
I'd prefer to have these event classes as just simple data containers with as little logic as possible.
So move this into the for example the GerritTrigger class or if you find some better candidate for handling the logic around the data in this event.

Also the lack of javadoc tells me that you haven't run checkstyle on you'r commits yet ;)
The current checkstyle settings in the pom only works in maven2, but I have a fix for maven3 in the works.
`$user` is not necessary
spaces replaced by one tab
could you use a 2-space indentation here as well? I'm talking about indentation for the generated output.
@croby Should be ok now.
I don't think a foreach is what you want here.

Looks like a reduce is what you need since you are converting a list of options into a single value you want to act on.
Woops nvm
I think the proper way to call the fail callback is to pass error code (e.g. -1) as the first parameter, and error message as the second parameter.
My proposal in bug 7417 would fix the version system without causing the complexity problems. In fact it would reduce complexity.
I really don't like obj.related.lead_art in python code. It's not a technical reason, but it is the reason that the syntax is that way.