labofsahil/github-event-dataset-2012 huggingface dataset:
What I had in mind:

        if isinstance(key, str):
            name = (key if epochs.name == 'Unknown' else 'epochs_%s' % key)
            key = (self.events[0:, 2] == self.event_id[key])
        else:
            name = self.name

        epochs.events = np.atleast_2d(self.events[key])
        epochs.name = name
        if self.preload:
            select = key if isinstance(key, slice) else np.atleast_1d(key)
            epochs._data = epochs._data[select]

        return epochs

you can get rid of ._get_events()
Ahh overlooked this one. Thanks for reiterating.

2012/10/29 Alexandre Gramfort <notifications@github.com>

> In mne/artifacts/ica.py:
>
> > @@ -420,3 +660,118 @@ def _pick_sources(self, sources, include, exclude):
> >          out = np.dot(sources[restore_idx].T, mixing).T
> >
>
> don't compare strings with "is"
>
> what happens if len(pick) > 1 ?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/165/files#r1965910>.
>
>
This will switch back once [CHEF-3305](https://github.com/opscode/chef/pull/337) is merged.
context = api;
I think there are a bunch of sections above that we probably don't need to get things running :)
dunno, but even if we add a warning, we should put it at the end of the list, not as the first found one
@WouterJ I have fixed the first by rebasing against latest 2.0

I will open a separate PR for the second as its not relevant to the rest of this change
https://github.com/jboner/akka/pull/332
Ты и так в этом неймспейсе, можно `qobject_cast<IAccount*>`
We should aim to use a parent pom for both archetype for dependencies so that we have only one place to update them
I didn't see that, thanks! ;)
Force sending of the document
Perhaps it's more efficient to do:
```
var scope = 'g';
if (!caseSensitive)
  scope += 'i';
// or
var scope = caseSensitive && 'g' || 'gi';
```
Rather than duplicate a more complex line of logic.
For the tick spacing of 2' and formatter of ``dd:mm``:

```
ax.set_xticks_spacing(2./60.)
ax. set_xticklabels_format('dd:mm')
```

And for the example of 0.001 degrees and dd:mm:ss, I am indeed saying that an exception is raised (although I would be very open to a better solution if you have one!).

Regarding units, I completely agree, and here we can use the units framework from astropy internally, and users can either give a string, or a units object:

```
ax.set_xticks_spacing(2, unit='arcmin')

from astropy import units as u
ax.set_xticks_spacing(2, unit=u.degree)
```
the same could be written as:

named_scope :not_in_project, lambda { |project| {:conditions => "id NOT IN (select m.user_id FROM members as m where m.project_id = #{project.id})"}}
Same great Instance taste! Half the attributes!
Is ManagerInstance.instance a singleton?
Here and below, I would prefer using parentheses rather than the trailing backslash.
Should the `For artists...` part start on a new line?
this locator is not on the home page
I think you accidentally your redirect.
Well, pure is set to an unmodified bit of data that we were passed. In that sense, pure isn't "modified" in the first case. Does that help explain things? I'd be happy to hear of a different way to phrase it that avoided the confusion, but at the moment it seems reasonably clear to me.
Why this if? (IMHO the Control.Panel can not trigger it before being on the map)
Absolutely. Fixed.
The refactor depends on the contents of propertiesMap to obtain parameters from the request object, including the remote user ID. ClientUtils.escapeQueryChars munges that user ID and makes it unusable by Authorizable.find(...). I have removed the ClientUtils.escapeQueryChars here and all unit and integration tests pass again. But I am concerned that this may leave a problem lurking. Mightn't it be better to use ClientUtils.escapeQueryChars at the point when these parameters are added to the SolrQuery object itself, thus preserving any incoming request params intact?
Nah, leave it as-is for 1.4. In 1.5 the only change I made to the routing package was to move it out of activity API and into Impl. I'll make a note to refactor it for 1.5.
again + vs. + '' +
should this take an `OAuthVersion`?
@ayakushev99 should this also have a ```\s*``` after the equals?
Ok, I can look at adding that.

On 23/12/2012, at 11:06 AM, Sven Fuchs <notifications@github.com> wrote:

> In lib/travis/model/artifact/log.rb:
> 
> >  class Artifact::Log < Artifact
> > +  # TODO remove this once we know aggregation works fine and the worker passes a :final flag
> currently it's based on content introspection, i.e. matching that we've received a log line saying "Done. Build script exited with:"
> 
> it'd be better to pass an explicit flag along with the last log message from the worker, that could be a normal log message with an empty string but a flag like :final => true
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
also, does `vanity_metric_counts.create` actually create a `VanityMetricCount` that is associated with the current `VanityExperiment`?  I think it should, but I'm not sure.
I think even changing "strained for" to "did" would make it less awkward.
up to here, and that's it for Pinch.js
Pshhhhhh

Sent from my Android, please excuse any typos or mistakes.
On Jul 24, 2012 1:27 PM, "Gur Dotan" <
reply@reply.github.com>
wrote:

> >
> > -      return this;
> > +        while (event = events.shift()) {
> > +          list = calls[event] || (calls[event] = []);
> > +          list.push(callback, context);
> > +        }
> > +
> > +        return this;
> > +      }, this);
> > +
> > +      if (_.isObject(events)) {
> > +        // First argument is a map of event names => handlers
> > +        // Second argument is a context object
> > +        context = callback;
> > +        var $this = this;
>
> Good catch.  This variable is redundant - I removed it in the next commit.
>  Thanks!
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/documentcloud/backbone/pull/1456/files#r1223619
>
Oh, I thought it was relevant to the fix for IE -- my mistake.
For combining several assignments in one var statement, the standard in this project is

var
  x = 1,
  y = 2
  ...
  z = 53
;
This is the most dead simple implementation to use the new API so we can deprecate the old one. It's not particular nice - I agree. But I have us rather think out carefully how we set up the password hashing than do a quick shot we later regret.

Also I'm still not sure if the current JCryptPassword interface is the ideal thing. If next year the new "supercrypt" algorithm should be used to do some flaw in blowfish a platform consumer has to inject a new object and define a new constant for it which the consumer needs configure into JUser or some other place. Also he probably needs to duplicate at least the verify and detect parts from JCryptPasswordSimple (or possibly inherit from it). That said I haven't come up with a better idea ;)
Can you explain how this works?

For the `json` function (listed in the same file) I used the more verbose:

```bash
if [ -p /dev/stdin ]; then
	# piping, e.g. `echo '{"foo":42}' | json`
else
	# argument, e.g. `json '{"foo":42}'`
fi
```

Is yours fully equivalent or is there a difference? (/cc @janmoesen, who taught me the above trick.)
Why the alias here?
Why are you stopping propagation?
Use [global .gitignore](http://help.github.com/ignore-files/) for this.
You should also display error message! Read [here](https://docs.djangoproject.com/en/dev/ref/contrib/messages/).
Please style it in the same way as we have it the rest of the file. Indentation and `)` at the end, for example.
Import only modules!!!
I don't understand. You want this style to be reusable? Or why *box*?
O no. You will do an external file for JavaScript.
Should I wait that jshrew accepts your pull request or just copy from your repository instead (by copying I mean adding to requirements your link) ?
How could we get into raisePropertyChanging but changingSubject is NULL?
Using double here doesn't make much sense, since only integers get added up.
it might be a little hard to read, but we can condense this to

```python
venue.contact_name = venue.contact.get('name', getattr(user, 'name', None))
```
No space between the ! and $this.
Remember Arrays sind schneller!
Let's avoid the `alias` thing here
also one user can have connect with several openid providers. that's should be keep in mind.
Sounds great. Thanks! I've fixed the other things you pointed out.
OK. ignore me. I just think the header processing stuff should be put in the header filter function.
Good morning,

> +  if (hour == 12 && pm[meridian]);
> +  else if (hour == 12) hour = '00';
> +  else if (pm[meridian]) hour = parseInt(hour) + 12;

-If it is meridian and 15PM, will it increment 12?

Hours has a range of 1-12.

-Is this if necessary? it seems to be doing nothing.

Starting from the end, If hour is PM, hour + 12 (1=13, 2=14...). If
hour is 12 and not PM is 00. And default havent update.


> +		var pm = [];
> +		pm["PM"] = true;
> +		pm["pm"] = true;
> +		pm["p.m."] = true;
> +		pm["P.M."] = true;

-Do these formats exist p.m. and P.M.?

Everything is possible in spanish.


Excuse my english, bye.



2012/12/21 Pablo Cantero <notifications@github.com>

> In jquery.dateFormat-1.0.js:
>
> > @@ -52,6 +58,10 @@
> >                      hour = values3[0];
> >                      minute = values3[1];
> >                      second = values3[2];
> > +
> > +                    if (hour == 12 && pm[meridian]);
>
> Is this if necessary? it seems to be doing nothing.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/phstc/jquery-dateFormat/pull/31/files#r2484758>.
>
>
"set to 0"  (equal sign reminds me of assignment to variable ``set``)
I took those two lines out.  We changed the default solver settings so that thigns work most of the time.  No solver is perfect.
I don't know the reason for the transpose yet, but it's a bit unexpected given our conventions.
(need to check across models)
version 0.2.maddog?
Thanks.
 I can actually see my last commit on my fork reverting that code down here in this pull request thread. 
I guess it's now part of the pull request then.
why 003 (isnt this the usb bus number which will change with which port you plug it into) ?
I see, I see. Confusing! but acceptable for now. I'll take this in.
Commented out??
Come to think of it, this might not be bad policy for all of our checks against user-generated strings in the config file.  We don't need to be heavy handed about enforcing CamelCase for everything.
Agree, and `RuntimeError` might work.  Alternatively, we could leave out the `**kwargs` in the `__init__` definition line and let the default message be given for this problem?

P.S. `NameError` seems to be mostly used when using a reference that isn't bound to any object, which isn't quite what is going on here.  e.g. for a `NameError` is if the reference `bar` is completely undefined in the following:
```
foo = bar
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
NameError: name 'bar' is not defined
```
And, finally, `_chi_squared`.
changed to nil instead of -1
better i think not to change behaviour, even if slightly, if we have a 
clean migration path to the better way!

--a


On 10/10/2012 12:22, Aled Sage wrote:
>
> In usage/test-support/src/main/java/brooklyn/test/TestUtils.groovy:
>
> > @@ -319,6 +321,9 @@ public class TestUtils {
> >          }
> >      }
> >
> > +    /** @deprecated since 0.4.0 use HttpTestUtils.assertUrlEventuallyHasText or HttpTestUtils.assertUrlHasText
> > +     * (NB: this method has "eventually" logic, with default timeout of 30s, despite the absence of that in the name) */
> > +    @Deprecated
>
> Can we delete impl, to instead delegate to new method?
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/brooklyncentral/brooklyn/pull/348/files#r1805465>.
>
Sure, why wouldn't I?
Is this called often? Would it make sense to maintain the "top flow" as we go?
Why do we want to use a `phpunit` database?
the code was using a strict comparison previously
Instead of adding 1 listener per element, what about adding 1 listener on the main container and use event.target to know which network is selected?
I know this was copied from the example, but i can't quite understand why this is needed.  Have you tried removing it?
be careful here. The vendor folder is missing in the exclude list so it will try to generate the code coverage for all vendors. You should fix the phpunit config
contrived example is contrived :)

We discussed on irc, customizing the nested collection w/ generics marshalling would be less contrived/more real-world usage for it
Да, можно
Починил только для ```@Collect```
Напиши тест, в котором будет не int возвращаемый тип, а Integer.
メソッド呼び出しが深過ぎるので何とかしたい...!!
@stof yeah it's right. I was just viewing that set/getVar was going to set/get.
全てをテスト -> 全てのテスト
の表現の方が適切なような。
Ran into this again today. It seems like merging this fix will allow the default template to be useable.
I would rather return a triplet `(X, y, qid)` if `qid` was present in the file. This way, this is both backward compatible and convenient.
should probably return ``self``.
I am not sure I am parsing this sentence correctly. Maybe it can be reformulated to be easier to understand?
basis_indices_ would be better.
для того, щоб юзера можна було використовувати в інших фікстурах, так сказано в документації Symfony
It would be actually nice to "keep" some README / README.markdown :) See yast-devel :)
What you're looking at is for custom fonts.  Can you install custom fonts on the BB?  If you can find the answer through investigation or documentation, then ask suavek.

The Font doc still needs to be updated to reflect the work of this PR.  
- fontFamily: Do you know which fonts are supported currently?  
- fontSize: what unit does the BB use?  should the unit be appended to the number?
- fontStyle: does italic work?  currently it says only supported for iphone ipad
- fontWeight: do bold and semibold work?

If you've tested your work you should already know most of these answers
Please add an assert too and get rid of the `if (owner_)` checks
I didn't change it for this pass because it would involve major refactoring but I may do it in the next pass. Personally I don't like the name "proxy" because it is not strictly a proxy but I may change it for consistency.
Both get mapped to the same place in Dali, and since ```thordata::regress``` is a relative path, it makes more sense. Changing it to ```thordata/regress``` also works, if you think it's best...
Well, until there is a change to the serialization format for dictionaries, this code is correct...
standard 
```java
response.sendRedirect2(req.getContextPath()+'/' + getParent().getUrl());
```
would be IMHO more appropriate
ok, but i had copied it from another place of this ex.
This is an artifact from when I refactored the code, if its r.body, '' or None does not matter at all in this case.  Unless I did think of something clever which I've now forgotten...
spaces after comma here too
a nitpick, but can we make line 123 isOnShowPage method or something? It is a little bit confusing...
Okay, I removed it.
I see.. makes sense!
A little bigger description would be useful.
I've lately been naming implementations ``_regionprops``, to make sure that we have a clear distinction between the implementation and the function name.  Not very important, though.
Could you realign the indentation for the wrapped text?
The deviation from POM is already existent, the proposal would bring it more inline imo. I am fine with the base method going into base.py or page.py for this project but it is likely to be refactored in the future. I'm wary of over-investing in an inefficient or problematic solution, and the new helper method doesn't actually solve any problems, it just adds a layer of abstraction and removes a few lines of (admittedly duplicated) code.
Also I just extended this test with what was already there, if we make these types of changes then we need to do it for the other elements in there, like 'ips' which also have sub elements.

On Mar 8, 2012, at 10:12 PM, Tim Simpson wrote:

>> @@ -62,7 +62,8 @@ def test_mgmt_instance_index_fields_present(self):
>>                 'deleted_at',
>>                 'deleted',
>>                 'flavorid',
>> -                'ips'
>> +                'ips',
>> +                'volumes'
> 
> For volumes we need to validate that "mountpoint" and "size" is found in each element of the list (actually I'd love it if we tested all of the values using regex's at least but since this test was already there I won't hold up the pull request for it).
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/rackspace/reddwarf/pull/416/files#r537693
@mrmattf had originally made a bunch of changes that I then reverted after he moved to a new branch.  Nothing should have actually changed in Polygon I believe.  I did a diff against master and they are identical.
We should be using regular expression literals instead of `new RegExp` all over the code for better performance.  See: https://developer.mozilla.org/en/JavaScript/Guide/Regular_Expressions Don't worry about this for now, I'll open a different pull request, I just wanted to mention it.
Extra white-space?
I don't know that it matters anymore performance-wise, but we usually replace `/ 2.0` with `* 0.5` (well, it still matters in hardware I believe, but  I'm sure a compiler can make the trivial transformation).
If this is a tool - and not part of Cesium proper - I don't think we need to put it in the main LICENSE.md.  The main LICENSE.md should answer the question "can I redistribute this?", "what licenses does this code use?", etc.  Does this proxy really fall into Cesium proper?  I don't think so.

A separate LICENSE.md in the proxy directory is OK I suppose - I just don't want to confuse users will lots of different license files.  We already have the copyright.js file in addition to LICENSE.md that I mentioned earlier.  Most users don't even know what the different licenses mean - they just want to know if they can use it.  Maybe we should make a LICENSE.md file that says "Yes."

One benefit of the LICENSE.md for the proxy directory is that we can explicitly use the Apache License for this.  I'll leave it up to you.
We may want to make it clear that the "clearing" of the field occurs when the view gains focus, not when the user starts typing in the field.
Again, can't really follow this one. Does this suggest a bug in user code or in Ti?
This may be dead code or at least not very interesting code...

If we're using this actively, I'd tag all these DEBUG_IMAGE_CACHE lines with a common prefix like "ImageCache:" so I could filter them out of the log easily. But that's just me.
Hi Bryan,

I think to your first comment, we ought not to mention this. The notion of top-level controls is somewhat synthetic anyway. The key words are "not intended", rather than "not allowed". A window, _in the Ti API_, can't be added to an arbitrary view or another window. In some implementation of the Ti API, this might be possible. But this is an implementation detail, and not part of the contract of View.add.
Identity matrix.
Negative. It's set to the default font only on creation, not on cell reuse. What you'd see if you don't change the font back to systemFontOfSize is that, in a table view with rows of varying fonts, the ones without a font property will somehow pick up font styles of other rows as you scroll and the cells get reused.
Every child has the same width in vertical layout
Drop this check so that an empty field will still update the DB
I personally prefer to avoid inline assigments as I think it is easy to miss these during debug sessions but do not consider this blocking for the merge
Christi,
     There should be an upright (roman)  d in the integral.
Please see: http://en.wikibooks.org/wiki/LaTeX/Mathematics
I'm still looking through your other corrections.
Thank you
-Katharine


On Sat, Feb 25, 2012 at 7:12 AM, Christi Rockwell <
reply@reply.github.com
> wrote:

> > +                             } else {
> > +                                     return ans1Denexp1;
> > +                             }
> > +                     })()</var>
> > +
> > +             </div>
> > +
> > +
> > +
> > +             <div class="problems">
> > +                     <div>
> > +                             <div class="problem"> Indefinite Integrals
> of Binomial polynomials </div>
> > +
> > +                             <p class="question">Evaluate the following
> indefinite integral: </p>
> > +                             <div><code>\displaystyle{\int{<var> A
> </var> <var>v</var>^ {<var>n</var>} + {<var> D </var>}}}\, {\mathrm{
> d}<var>v</var>}</code></div>
> > +                             <!--<p class="solution"
> data-type="regex"><var>nest</var></p>-->
>
> The 'd' in the integral shouldn't be in \mathrm
>
> See: http://mathworld.wolfram.com/Integral.html for how it is typically
> formatted
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Khan/khan-exercises/pull/14610/files#r488598
>



-- 
Katharine J. Hudson
Learning Center Math Specialist
*The Haverford School*
450 W. Lancaster Ave
Haverford, PA 19041

610-642-3020 x1345
khudson@haverford.org
So we leave only hash styled Refinery::Core.config.site_name ?
Wait, why is this? The values are not distinguishable anyway, so just accept.
I think we should go with the more restrictive but simpler `static assert(is(typeof(r[0 .. $] == R));`
Maybe use `isBoolean`?
No need to extend with CommitFu::FlogCommit unless you are measuring complexity
"... return a..." isn't appropriate for the comments, since that suggests a method return value instead of explaining that the request itself will be handled by the method instead of returning an event.

Wording updated to something that I think addresses both
Shouldn't it be called *login_template_name* to keep the naming consistent?
If you move the `cursor = queryHelper…` outside the `try`, you can rely on the behavior of `safeQuery` and your `throws` clause and get rid of this condition.
maybe using .getModel() instead of .model
I was not sure, I though that it may be needed if the `view_response_listener` was disabled. Not sure if it is needed though.
Yes that's true.
But this line is not under the `fos_user` element. 
`services` is a part of the [symfony/dependency-injection container](http://symfony.com/doc/current/book/service_container.html#creating-configuring-services-in-the-container). 

```yaml
# app/config.yml
# .. some configuration

fos_user:
    db_driver: orm # other valid values are 'mongodb', 'couchdb' and 'propel'
    firewall_name: main
    user_class: Acme\UserBundle\Entity\User

services:
    my_canonicalizer:
        class: Acme\UserBundle\Util\CustomCanonicalizer
        public: false
````
Because we need to do format for the date time, the current writeOptionalElementNS doesn't support customer transformation during writing.
Anywhere #method_missing is defined, #respond_to? should follow :)
No need to wrap it in a block.

```
<%= image_tag(profile.image) %>
```
forgot about that,
hm i am trying to understand your concern.... you want to move this composer install section into travis? and the purpose is so that other people could just run this part? i think you want the opposite... not sure I am following...
This should use the timezone that the search is in. Maybe `Time.zone`.
Узнай. Позаботься о потомках.
Because there are no wildcard characters in character groups.

> I assume that the RegEx-parser escapes the dot also in the character classes, but it's handled as dot also without escaping.

http://www.regular-expressions.info/charclass.html
ahh, great point.  OK.  I created a cache object for the users that handles all this now.
Can we combine all these in a shorthand property?
This check for response was correct. If something fails, an error will occur inside here. Let's catch it somehow (either with the if again, or a try/catch) and use the callback to generate a safe error.
yep.  I actually made cases optional too, since there's really no reason someone couldn't just import a bunch of suites instead.
@deadalnix My apologies. You are correct.
Opps, my bad!
Mind-mess-up...was introduced with 2.1. Deprecating is correct of course...
It simply means that its should not fail
The bootstrapped configuration as described in JIRA would fail. There is really nothing to assert.
you should use ``&&`` instead of nesting 2 ``if``
Why is this not available for operational scripts?  This is an input and should always be there.  If it isn't then there may be an other issue that we need to investigate.
Hm, it should be probably wlanslovenija here?
Thanks, done. Didn't seem to react to "note" in preview, but I fixed the code parts with code-block bash parts while I was at it. How is it now? Should I change something else?
возможно, стоит перевести так: "Есть две основных, наиболее полезных, реализации интерфейса"
this is wrong. First, ``master-dev`` does not exist anymore, and second, you should not depend on it (it would imply that you can guarantee the compatibility with any future state of the master branch)
not exactly. this filters referrers by the referencing property name. filter is optional.
@rande fixed.
already working on adding more tests and breaking them down more.
The third party project already contained that folder. I've deleted it and the upgrade-related files you note below.
enabled=False?
N/m, I was blind and didn't see it.
this url need to be fixed with the --prefix
Removes 2 unneeded spaces .
Whenever you can leave the assignments out of the conditional expression, it's better.
fixed, get_answers removed
where does INTERVAL_GRACE come from?
if you make this a singleton, all these checks wont be required
> I was going to just fallback to returning absolute path if given or current dir + relative given.

I considered that (as I said in commit comment ^).
But - a common use now is, for creating a local log,
e.g. logs/mylog.log
The result (of makeAbsolutePath) is logged at some point for user information.
Other scenarios also, where FQ path is logged/kept prior to it existing.

So I'll leave it as it is.
^^ ?
договорились psd
life cycle -> lifecycle
PD.
Hurr? Why are we fetching headers twice?
Look at BaseActivityInstrumentationTestCase.java (in RestExplorerTest), in the setup method it does:
                eq = new EventsListenerQueue();
		// Wait for app initialization to complete
		if (ForceApp.APP == null) {
			eq.waitForEvent(EventType.AppCreateComplete, 5000);
		}

Precisely for that purpose, so test wait for app create to complete. And therefore we don't need the special code in ForceApp and we can actually test encryption properly.
No need to keep the old code in with a comment, version control is great :)
Same as:
~~~clojure
(my-map rest seqs)
~~~
JNEXT_PLUGINS it is!
the { should be on the same line as the if statement, same with the following if statement