labofsahil/github-event-dataset-2012 huggingface dataset:
I think  we won't find any answer in the git doc :-) But we should at least try to figure out some answers, and then express them as  unit tests.
So I can't get it below 3MB, not even with 5 ICA sources and 3 timeslices... Would that be ok?
Same here, except with max().
On 11.10.2012, at 17:39, Alexandre Gramfort <notifications@github.com> wrote:

> In mne/fiff/evoked.py:
> 
> > @@ -314,6 +314,18 @@ def plot(self, picks=None, unit=True, show=True,
> >          plot_evoked(self, picks=picks, unit=unit, show=show,
> >                      ylim=ylim, proj=proj, xlim=xlim)
> >  
> > +    def to_nitime(self, picks=None):
> > +        """ Export Evoded object to NiTime
> > +        """
> > +        from nitime import TimeSeries
> > +
> > +        if picks is None:
> > +            picks = np.arange(self.data.shape[0])
> > +
> > +        evoked_ts = TimeSeries(self.data[picks], sampling_rate=self.info['sfreq'])
> and raw.copy(preload=fname) would preload in a new memmap or if preload=True (in RAM) just do a deepcopy of self

why not just preload if not allready and then deepcopy?
we then could omit the keyword?

i can look into this.

wdyt, single PR or as byproduct of ica?

> — 
> Reply to this email directly or view it on GitHub.
>
Same as above:
- Add logging here.
- Use status code 500 (internal server error).
- Separate the message into more than one line.
Use the EmailValidator class from apache commons instead.
-too

just say complex
You should replace the last `.` with `;`
国语通俗语序与英语经常相反
没有指定日志格式
Creo que este campo no es necesario teniendo un tipo VacationList que se encarga de eso.
No idea what has changed here, you've done a bunch of alterations to the file that suggest you've reformatted it before appending your style.
I thought, it would be too lengthy, if that's not a concern , would follow the same convention used in the code base.
Why not caching this output ?
I don't understand why these were changed to be framework-relative, when this file is already in AppKit/. In every other AppKit file we use local references.
Nice catch. I missed this one. It's not wrong, per se, but doubles up what we are already testing through the generated queries.
The osapi relies on modules in the gridcentric.nova.extension. In particular the gridcentric.nova.extension.api
Devise doesn't rely on this.
UIに出力する文字列はi18n対応したいので、役の定数は文字列ではなく数値の方がよいのではないでしょうか。
文字列を返す機能は別途実装する事が望ましいと思います。
定数にはfinalを付けてください。
I don't agree with this one.
does this need indentation?
There is a round-trip with tracker to get this information (confirmed with dbus-monitor). Since you're only setting one dimension of sourceSize (so aspect ratio while loading isn't a problem), won't implicitWidth/implicitHeight be fine?
no, just leftover from bende's two camera stuff. shouldn't be there apparently.
@gsilver can we do without this?
@apex13 We should probably use `|safeoutput` in this line and the one above for the titles. - We do this for XSS reasons
I don't think window.location.redurl will actually exist.
Here should be 
            if (mostRecentRepos.Count > 0)
                _NO_TRANSLATE_Workingdir.Text = mostRecentRepos[0].Caption;
           else
               _NO_TRANSLATE_Workingdir.Text = Settings.WorkingDir;

This was introduced in 3ae1cf67015783108349ef7826b85d0abd074801 lost while merge to master.
Removed copy paste.
Ok,
I m going to change that too.
mappau

-----Original Message-----
From: Jonathan Hoffmann <notifications@github.com>
Date: Wed, 5 Sep 2012 12:13:56 
To: blackberry/NDK-Samples<NDK-Samples@noreply.github.com>
Reply-To: blackberry/NDK-Samples <reply@reply.github.com>
Cc: Andre Pura<andre.pura@mappau.com>
Subject: Re: [NDK-Samples] Sample added: Multitouch (#36)

> +
> +    dialog_destroy(alert_dialog);
> +    alert_dialog = 0;
> +}
> +
> +void show_alert()
> +{
> +
> +    if (dialog_create_alert(&alert_dialog) != BPS_SUCCESS) {
> +        fprintf(stderr, "Failed to create alert dialog.");
> +        return;
> +    }
> +    dialog_set_size(alert_dialog,DIALOG_SIZE_FULL);
> +
> +	if (dialog_set_alert_message_text(alert_dialog,
> +			"Download our brand new application Compass: \n\nThis is the first magnetic compass written natively for the BlackBerry� PlayBook. It does not use GPS (no movement necessary to get a correct direction), instead the Magnetometer is used.Keep the PlayBook out of range of any metallic object, as this can interfere with the compass!\nTry it!\nFeatures:\n\n- Real magnetic compass (no GPS and/or movement needed)\n- Stunning graphics\n- Swipe down for easy to understand calibration") != BPS_SUCCESS) {

We can't accept samples with advertisements.

---
Reply to this email directly or view it on GitHub:
https://github.com/blackberry/NDK-Samples/pull/36/files#r1536915
Why do we need to register this additional listener? I would expect that ``ScriptEditor.handlePreferenceStoreChanged`` is called every time a property is changed in the editor's preference store.
why not returning $this for chaining?
Falta indendar =P
Why `horoscope_instance`? And not just simply `horoscope`?
Aha, `filter` tag with `wrap` argument. This would do the trick.
Unnecessary space at the beginning. Please clean all such things.
We will automatically change that username to something else. Like we could add the sequence number at the end.
on my to. Mongodb get's really nervous =)
Maybe we shouldn't really store all data to make things faster?
so you typedef the shortest-name-length type but not the std::vector<std::vector<uint32>> huh?
So are we being picky for a reason? In the Unit() constructor alone there are 12 "m_" variables and 1 "i_".
This might be simpler if it read:

request.GET.get('week_start', "")
self.week_start = week_start if week-start else default_week

or something similar.

This isn't a hard requirement, but just thought on how we might simplify saying "If week start is empty, use default week"
Please remove unnecessary view.
Will this have an effect on the URL's (/ting/object/775100:24702146) ? If so I think we should revert this change.
syntax error -- unnecessary `)`
Jar is default packaging; can be left out if you like.
don't think you need curly braces here
added a test for that in another commit.
Is this needed? The default should work...
I don't really understand the advantage of this. Why is it better to upsert instead of insert? In my opinion this is dangerous since we haven't defined a uniqueness constraint yet.

Little side note, the `datastore_create` only creates a dataset if it doesn't exist. Otherwise it will alter the table and insert the data. It's not realy clear in the docs but you can find more here: http://docs.ckan.org/en/latest/datastore-api.html#ckanext.datastore.logic.action.datastore_create.
`transport_type` might be more accurate.

https://homes.oslo.osa/rchlodnicki/docs/master/index.html?service=./protofiles/resourcemanager/ResourceManager.1.3.proto
`window.event_handlers` instead.
Agreed.
This class name sound ( and should be ) very generic. However I see it only used in the resource_detail view, to highlight a specific line when viewing a text/* resource from another service ( e.g. Documents )
Yeah. Moment of confusion.
Good point! I've taken a different approach, preserving most of the existing code, so this becomes a moot point.
Oh, this isn't a subclass of `NSWindow`, is it? :-) So looking at it further, it goes `QSDockingWindow` → `QSBorderlessWindow` → `QSWindow` → `NSPanel`. `canBecomeKeyWindow` is only implemented in `QSWindow` and it returns `YES`. So that seems to take care of it.
What do you think of writing it that way ? I think it limits the patch size while keeping the behavior :)

``` ruby

if ARGV.empty?
  puts config
  exit
end

path_handlers = ...
```
This ``i=0;  i+=1`` is a bit un-Pythonic. You can get ``i`` from the for-loop like so:  ``for i, X_j in enumerate(LOO):``.
Yes, that's a great idea, thanks.
Warn seems like an excessively high level.  Recommend info, or lower.  This is happening inside a loop.
No need for more, I'm very happy with the upgrade.
Do you want me to change it? :)

I'm a very defensive programmer usually, but having some parts of the system programmed using this style, and others not, perhaps is not the best way forward.
just spacing changed in this file
Only on new async flow branches, which should be much less than scope enters/leaves (where the top flow could be maintained). I removed the pooling of flows too because I feel like the extra logic may be more expensive than just newing things up. It'll be easier to add back now that things are super simple if we find it being a problem.
Bleh - messed up the POM change somehow. I'll fix it.
On Apr 24, 2012 4:14 PM, "Adrian Cole" <
reply@reply.github.com>
wrote:

> > @@ -20,8 +20,8 @@
> >          <jclouds.test.provider>stub</jclouds.test.provider>
> >          <jclouds.test.identity>identity</jclouds.test.identity>
> >          <jclouds.test.credential>credential</jclouds.test.credential>
> > -        <jclouds.version>1.5.0-alpha.5</jclouds.version>
> > -        <guava.version>12.0-rc2</guava.version>
> > +        <jclouds.version>1.5.0-alpha.4</jclouds.version> <!-- note that
> this needs to get bumped up to whatever the next release is -->
>
> I think this needs to be alpha.5, rc2, as viewableAs isn't in alpha.4.
>
> we can block on this PR until central syncs up..
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jenkinsci/jclouds-plugin/pull/26/files#r729831
>
Can we change the user name and passwords to "admin":"admin" and "user":"user", the "primary" to "primary.local" (and "secondary1.local" etc...) and use "phpunit" and "phpunit-auth" for a database without auth, and one with?
Actually, isn't this this issue: https://jira.mongodb.org/browse/PHP-585  ? Same for all the other 1 -> %d ones.
Quando realmente for deletar uma aeronave a gente pergunta pro usuario se realmente quer fazer isso e avisa que todos voos relacionados a ele serao deletados tambem. Se sim, a gente faz um delete em cascata. Isto é, deleta todos os voos primeiro (delete voos where aeronave = xxx) e depois executa este delete.
For this return press you can import 'selenium.common.webdriver.keys' and then `send_keys(Keys.RETURN)` much cooler :)
Adicionei essa feature no TK314, acredito que depois de aprovado o TK 314 este pull não realizará merge automático.
The `# Fails` addendum is confusing.

Does it mean the assertion fails currently?
If so, is it *supposed* to fail, or is this something that should be fixed?

Ditto below.
This one isn't optional so you should put in ``setup.py`` - no need to have it in the docs.
Yeah, but if you tell a QPixmap to load an invalid url, what happes---is it null, or is it still what it used to be before the load? If the former, we can still skip the success var :)
as above, rename it to features.json
OK ... when you say, the caller, you mean, the caller of new_trial_docs?
Why POST and GET?
Will pull in fix. For some reason doing pull upstream master gave me this and some weird text.
BUG!  ; after if statement on this line.
Can this be ActiveRecord too please?
Quite right, I was a bit lazy there, I admit..
main function for shifting
Think you meant glance above, not keystone :)
oh, I see. Maybe ~>0.5.0 then?
ok so you added the logic for these. Remember you need to rename the function at the very least.
This will work, though I don't personally like modifying input parameters like that, which is why I preferred ```opts = opts.merge!(:filter => "in_flight")```

Kind of a best practice thing so there's not unexpected side-effects.  

However, this will work without any problems since we know exactly how it's used.
isWhitespace это проверяет
Ошибка в условии.
CASE_SENSITIVE (слова надо разделять _)
CASE_INSENSITIVE
I think my argument was perfectly cogent - would you like to dispute part of it, rather than merely repeating the question?

Please only ask questions where they are productive, or move a debate forwards!
Dense random projections {-1 / sqrt(n_components), 1/ sqrt(n_components), } is a valid random projection and share the same property as sparse one (see thm of 1.1 in the Achtloptas paper).  Ping Li shows that it remains valid for a wide range of `s = 1 / density`, see for instance fig 1 of the Ping Li paper.

Here we are talking of the case where `n_features` < 9 which in my opinion is not high dimensional. And if you make some dimension reduction, you will have  `n_components` < `n_features` < 9. So you have a random matrix of size  `n_components`  *   `n_features` < 81

Maybe, I read the Ping Li recommandation in a too strict way.  As @amueller said, it is more an edge case. Feel free to revert to the previous behavior.
Could you please find a more explicit name (longer) for this variable.
np imported by unused
hi @GaelVaroquaux : could you please take a look at this function and see how it might relate to:

1. https://github.com/briancheung/scikit-learn/blob/f5d219e828a0892642b895238bcdd6364eec60e1/sklearn/cluster/spectral.py#L125

2. https://github.com/briancheung/scikit-learn/blob/f5d219e828a0892642b895238bcdd6364eec60e1/sklearn/cluster/spectral.py#L136

without writing out the math, it's not immediately obvious if there is some direct relation or these things are calling two similar but not identical things - diffusion maps.
Sounds good. I'll expose it in `RidgeCV` only when `cv=None` and document appropriately.
Probably because `fit_predict` doesn't make sense, except when doing [transductive learning](https://en.wikipedia.org/wiki/Transduction_(machine_learning)).
hm ok, why not.
No argument there.
I thought it would be possible without a helper function. But what Gael did there is basically write down this function.
So sorry for the noise and +1 for including it :)
I would be nice to maybe also match substrings, as was done there.
There's not actually anything special about it -- it's the labels that matter. I'll do a test to verify, then reduce this matrix's size.
fixed in 01acd4b19945acc679ad17c67bc4273dc8ac8eab
See DialogProvider and how is it used. If the dialog is correctly handled from Actvitiy, than there is not a problem with it when rotating the screen and in other lifecycle of activity.
should set state before firing the event
Should return this as a value. The ThrowException schedules an exception so the code doesn't halt when this is called.

e.g.

...
else
{
    return ThrowException(String::New(Native::Msg::Expected_argument_of_type_object_or_external));
}
> Since hide and show are not setter or getter methods they need to be implemented in TiUIBase

Can you explain this thought please?

Also is there a reason for show not to be here next to hide?
We're already using it in an app like this - I'm not sure if it's semantically correct, but it does work in our case
I'll take a look at how FWS was implemented, but I believe it's one of those Exception driven parts of the code (kinda dirty, I know).
Behave as different user then authentication
I haven't determined if there are ORM specific use-cases/performance improvements that's why they're in there.
Don't know why I changed it, changing back.
Some compilers might give a warning about "flags & x" implicitly converting unsigned to boolean.  Should probably have != 0
Not sure what you mean (unless you mean that the commented-out line should be removed, in which case I agree)
I'll make this [0-9]+ to capture one or more digits.
Ah, that's a good point. But... I don't think it's worth maintaining support for older versions of homebrew. The cookbook updates homebrew on every run, so unless you're running from a fork or using the provider but not the default recipe, it won't be a problem.
C'est bof c'écrire unless teacher.user.nil? En plus tu le multiplie à chaque attribut. Englobe le tableau avec if teacher.user
Je t'avais fait une remarque sur ce genre de nom de  classe dans la PR précédente.
They can't use `saveProfile` because it's private.
Was more to harmonize the init Deferred call, but that's was mentioned above.
Please add a check for this in ClassPagesServiceTest.
Yep they ran fine
What I originally was looking for is a good place for GetTagHeads (https://github.com/feinstaub/gitextensions/commit/a13507551349c9a487f43b6a8fad6c0315353a89#L0R2371).
This will make the Repo displays larger.  Right now they are at 450 to keep consistency with GitHub's website.  Using 504px also widens the extension.  The 450 can be kept and the .popup can be reduces to get ride of the GAP.  I think you will like these results.
```mapping_p_o[label]``` again
Should these be Rex::Socket methods?
Not sure why we did this for some examples in the past.  Let's remove so we don't get flooded by output when building the docs.
I don't understand why you would want sigma to be an adjustable parameter: if you want a wider Gaussian you can just change block_size. And neglecting the contribution of pixels for which the value of the Gaussian kernel is < 0.01 seems pretty reasonable to me.
Why have you put this line inside the block?
You're right. It apparently failed yesterday.
I think the -* is wrong, because that would allow any number of dashes.  -? maybe?
I have no idea how the JavaScript engines works, but is setting length to zero better than assigning `[]` to create a new array.  The goal is to ensure that there are no outstanding references to the vertex array so the GC can clean them up.
Not all browsers do Error() the same way.  I wasn't sure if checking "typeof Error" was good enough.  Perhaps it is.  But if you fail in a routine that's supposed to identify failure, you've hidden all problems everywhere.
Is this static class the best design?  Instead, why not have static creation functions on 'Cartesian3' and `Spherical` like `JulianDate.fromIso8601`?
Please use the 'extra_score' to key in the release item to add extra scoring
This probably should be wrapped into a transaction.
Insert newline before override
No function calls, can't spare the overhead.
That exception does not necessarily mean the file does not exist. The message we concat onto the message should
have more details on the exact reason.
Not a property on iOS.
Android only, not Android and Mobile Web.
afaik, tabgroups may not be closed on Android without closing the app.
These AWS keys are no longer valid :)
PHPCR-ODM is always used, regardless of the PHPCR implementation, so this note applies to all 3 (and should actually go to where we talk about adding the phpcr-odm and not to the PHPCR transports.
Need to give more detail and a final answer hint.

For more detail, I'd suggest showing what you need to multiply the term by and the steps to get it.

For example when the integral has:
11/(5/4+1)s^(5/4+1)+13/(0+1)s^(0+1)+ Constant, talk about how the 5/4 as an exponent means you need to multiply the term by the inverse of 5/4 + 1 or the inverse of 9/4 or 4/9.

For the term in the integral that is a constant (and becomes the term with the variable to the first power), I'd give the rule about the integral of c dx is cx.

Also, give the rule about the integration of sums.
Please don't put the code of the Console component in this repo. there is no valid reason to copy-paste it in this library
Yeah... You''re right. I think I got that last if-else upside down. We'll my code was suggestion/inspiration. I'm sure you'll get it done better than I did.
let's rename this file so that it is more easy to discover that it is a script, maybe call it
tools/run-tool
or 
tools/runtool.sh
?
This is just adding the comma although it counts as a deletion
Can this me changed to a multi-line quote format?
Thats the wrong place to put this, it should be calling doAdd() directly. As the addform was never rendered, no CSRF SecurityID was created, which is a problem for the submission (it'll fail). Because we use one global security token, I'd recommend that you get if from another already rendered form (batch actions?).
This should be protected, so it can be extended.
As you always inject the handler in this listener in the container, this ``if`` will **always** return ``true``.
The only reason for this ``if`` is that you added ``= null`` in the constructor to allow someone to use the class without an handler.
it is there for custom logic. You may want to redirect to a public profile needing the user id for instance
Also need the OpenMRS license. (I'm not going to repeat this comment for every file, but it applies.)
I think people will always want to see hte list when they use this agent from 'mco rpc filemgr...' so we should have here:

<pre>
display :always
</pre>

like in the status action
This will expose a global variable.

Are there no other plug-ins that do a similar thing?
e.g. https://github.com/filamentgroup/jQuery-Equal-Heights/blob/master/jQuery.equalHeights.js
But that's ambiguous because that's compile time, while this is runtime (and independent of the bitness of the process), no?
Why won't it be enough to use default serialization's methods and remove 'transient' modificator from 'elements' property? 
Is it realy this class should be `Serializable` if it is used as package-protected only in the `AbstractDispatcher`?
I understand that this wrapper should follow for all List implementation convention, but we have to be consecutive and just write our KISS framework.
Isn't it? ;-)
I am not sure I understand why slowing down the SA poller will fix this; Mark's occurrence failed with

java.lang.AssertionError: expected:<1> but was:<4>

It's also not clear to me how the channel/store could ever get 4 messages in it for this test, regardless of the timing.
Per discussion in JIRA - there is __no__ reply message in this case.

If we agree this attribute is useful (e.g. for the scoped @ bean as discussed in JIRA), we should remove this last sentence.

If everyone agrees, I can just remove it during the merge.

Personally, I don't think it hurts anything to merge this, even though the usage might be very limited.
we got a mess in context.xml, when user leave this default value 
https://github.com/rightscale/rightscale_cookbooks/blob/master/cookbooks/app_tomcat/templates/default/context_xml.erb#L29
it is not adopted for "jdbc/ConnDB"

set_unless will never work here because we always have default value in the attribute
Was a ticket created for adding the server name to the log file?
This is a breaking change. I'd rather prefer that the mongoid_colored_logger support both 2.x an 3.x versions.
This is probably incorrect, as this refers to ZF1 ViewRenderer and Layout.
This line should reference a commit, too. L299 can change easily ;-). Or remove the comment altogether.
Part of this is due to not having shapefiles for each shape type that are small enough to include for testing.
I could not get node to recognise zombie.js without this line (it sucks, and no response from #travis yet)
Do we really need to add libs-snapshot-local?
I presume M2 is in the releases section of ccweb (unfortunately it's not on developers yet)

But presumably no harm adding the snapshot repo...
I have corrected this problem in new commit. Please review it.
I know this was also in vanilla code but isn't the for loop
``for (int i = 0; i < 1; i++)``
completely unnecessary?
Can you make this static or its own class?
Idem ici. Utiliser un callback after_update pour cela.

Utiliser un where qui check le nom de la locale genre :en plutot que l'id.
Aria roles should **NOT** be replaced by arbitrary class names.

As discussed in the meeting we had with the UX and FE teams and Andreas, that’s precisely what should be avoided if we want these components to be reusable.
tests should not depend on external resources!
again, create issue and add id here
This sentence is pretty generic.
I don't get the second part "works on data with related structure"
It is another small nit, but consistency in XML comments is good... so please end with a period here: 'Inner HTTP handler.'
Couldn't hurt to have them be enumerable.
I would love to move an abstraction of this into the web platform
already exists
commands could change, should't they be connected to the Karaf version somehow?
here we overwrite the setter methods for position and move to implement the double buffers
TODO lol