labofsahil/github-event-dataset-2012 huggingface dataset:
Looking at this file, in cases where the key bindings on mac are different from win/linux, the order is always generic first, followed by platform:mac. These can be specified in either order (and this file just happens to be organized that way), correct?
I doubt `localStorage` is ever null or undefined, it may not be declared, but in that case this will exception. Maybe use `!window.localStorage` instead?
Pushing fix now.
We've gotthis class twice : once in SIF, once in orbisgis-view. The question is : which one will disappear ?
Those two do not got to default messages!
`futureDate` is probably badly named now? Where do you initialize it, BTW? And why do you use it? Simply call `self.error ` when you notice an error condition (it throws an exception).
The point it is that the code is more readable. It is more local. If you assign some variable then use that variable, not use original, because it is easier to follow one variable than two. So once you assign, it is easier to remember that from now on we are dealing with just that one temporary copy and this is it.
I think we should make the default true so 75% of our user base doesn't see a warning instead of their desired web page
No real reason. I was just taught to avoid using flags in uni.
Lots of styling nits in this file, but gonna hold off on those until the release gets landed.
Typo: "make sure ^we have a handle..."
multi-lined object please
ok. comment accepted.
`sorted()` makes a copy already, it doesn't sort in place. So you probably want `s = sorted(s, key=operator.itemgetter(2))`, otherwise that line has no effect...
Instead of default value, I'd just do:

    $foo = $frame->getHeader('foo') ?: 'bar';
@plusjade glad I could help.  I suspect this is related to Issue #79.
Because the offset is used to change the levels of all the headers. Not just the first one.
What will you do if you want to add the last slide and you must use insert function?
ignore this files and create a model named `/app/config/parameters.yml.dist` and add instruction to copy it to `parameters.yml` in README.
Test these paths individually by creating the paths separately
The old env_select is still used.

On Wed, Jul 18, 2012 at 10:17 AM, Brad Buckingham
<reply@reply.github.com>
wrote:
>> @@ -40,6 +40,8 @@ javascripts:
>>      #tipsycustom
>>      - public/javascripts/widgets/tipsy.custom.js
>>      #env_select
>> +    - public/javascripts/widgets/path_selector.js
>> +    #old env_select
>
> Is this still used in the haml?  If not, can we remove it?
> From a quick scan of the haml in my branch i don't see any javascript declarations for it.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Katello/katello/pull/327/files#r1188018
Ok. You mean the whole second parameter to `GetOrAdd` (the LINQ expression), right? No problem - will separate this code in a method called `BuildMethodCacheFor(hub)` (or maybe you have a better name for it?:smile:)
`rawdict.txt` is converted from `PinyinIME\jni\data\rawdict_utf16_65105_freq.txt` of `PinyinIME` repo.  The difference of the two files is that `rawdict.txt` is utf8 encoded while `rawdict_utf16_65105_freq.txt` is utf16 encoded.

For convenient, I perfer to keep one of the two files, as if the schema of the database was changed in the future, we can regenerate from the raw dictionary file.
This works:
```
In [19]: astropy.coordinates.Angle(3, 'deg') * 3
Out[19]: <Angle 9.00000 deg>
```

But this doesn't:
```
In [20]: 3 * astropy.coordinates.Angle(3, 'deg')    
ERROR: TypeError: unsupported operand type(s) for *: 'int' and 'Angle' [IPython.core.interactiveshell]
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-20-09783c72dd3a> in <module>()
----> 1 3 * astropy.coordinates.Angle(3, 'deg')

TypeError: unsupported operand type(s) for *: 'int' and 'Angle'
```
It would be nice if `Number * Angle` were possible, too.
Ah, yes, that's now wrong with the new beahvior - fixed in the version I'm working on.
Yes, I can add support `ndarray` and `complex`, and rename the class to `NumpyOrSetEncoder` (or something like that). But do we need such support for AstroPy? The new class would look like this::

    import numpy as np  # Since Erik B says it is better to import numpy outside...

    class NumpyOrSetEncoder(json.JSONEncoder):
        def default(self, obj):
            if isinstance(obj, (np.ndarray, np.number)):
                return obj.tolist()
            elif isinstance(obj, (complex, np.complex)):
                return [obj.real, obj.imag]
            elif isinstance(obj, set):
                return list(obj)
            return json.JSONEncoder.default(self, obj)
Muscle memory problem - it was supposed to be d's - but changed to distances.  During earl development work for the diversity work I was generating rings and making sure there was one of each distance.  this code could be culled.  I've kept the early work on https://github.com/jonmeredith/ringgen/blob/master/ringgen.erl
Can this really safely be removed? Is this change directly related to coincontrol?
@laanwj See #1569 for a fix for this.
nKeyCreateTime defaults to the current time? Doesn't the break compatibility? (importprivkey currently rescans the entire history).

I'd prefer the current time to be the default (as is implemented here), but I'm afraid it will cause issues.
if defined(WIN32) && defined(QT_GUI)
is equivalent to
if !(!defined(WIN32) || !defined(QT_GUI))
quite confusing ;) thx anyway
https://twitter.com/jabbr/status/207283403274452992
Changing an interface is a BC break
this is an array of string
@joshf Maybe this message should be collected by logger. You can follow Indicator or MessagingMenu to achieve it.
I've thought a bit about the new interface layout, but I'm not sure if I really get it: I mean, there are the type/property/collection definitions that describe the rdf structure, and there are the entity variants that represent the concrete data objects. And then there was the dom structure that was encapsulated in the abstract node class, but now that seems to be merged into the type/entity logic. Design-wise I would find it cleaner if they could stay separate, because they represent the two "endpoints" between which createphp communicates: It turns stored data into rdf-enhanced HTML and rdf arrays into stored data. Both of these conversions are done with the type/entity representation, but the HTML part is not needed for the second function (i.e. the REST service). 

So strictly speaking, a type or an entity doesn't need functions like set_tagname to be useful (they also don't need the attributes functionality since your pull request with the explicit methods like setProperty). It's just the node that needs these, and the concrete implementations for the entity use the node's functions to implement functionality defined in the entity interfaces. But theoretically, it would be possible to implement a node class with no connection to the type/entity model (for rendering plain HTML), or an entity class that doesn't define any rendering (e.g. for handing REST requests) or that outputs to another format like JSON-LD, where the node functions don't really make sense.
Agreed, updating
Actually, now that it's using subprocess, it might be better to make the call with `shell=False` (i.e. the default).  We only used `os.system` because IPython dates back to before subprocess even existed.  But using `shell=True` is slower (especially on Windows), and in this case we don't really need it, so we might as well update the code to current best practices.
But changing the value should recursively call `_notebook_dir_changed`, so it is actually probably okay.
oh, that's how.
done!
You could simply use

```php
$connection = Config::get('session.database_connection');
return new Session\Drivers\Database(Database::connection($connection));
```

Ensuring a string value is not really necessary, as using a faulty connection config should result in errors anyway.
Damn, that's a totally valid point. I didn't think about `vsnprintf` setting errno. Please revert it to the old way (and document accordingly).
Typo: "what those means" -> "what those mean"
Oh, if we're storing the value somewhere, then don't change it. I'll just change my brain.
This one line does all the work here. The rest is just bookkeeping so that the output on the command line is interesting to watch.
After a return, there is no need for the else branching.

```
if ($something) {
    return $something;
}

if ($somethingElse) {
    return $somethingElse;
}

return $default;
```

has always seemed clearer to me.
On Fri, Dec 21, 2012 at 04:15:22PM -0800, Rob Hudson wrote:
>Why the extra `'./'`? Seems like urljoin does the same thing with or without it.

It does if `path` is relative. It doesn't if it starts with 
something like `/` or `http://`.

-- 
Kris Maglione

Simple things should be simple.  Complex things should be possible.
	--Alan Kay
I'm cool with that. I'll update.
We cannot merge until it is perfect.  

--  
Phil Sturgeon


On Monday, 20 August 2012 at 10:42, Phil Sturgeon wrote:

> Right but why is there a list of random categories in there already?  
>  
> --  
> Phil Sturgeon
>  
>  
> On Monday, 20 August 2012 at 10:40, Salmane wrote:
>  
> > In system/cms/modules/blog/config/routes.php:
> > > +<?php defined('BASEPATH') OR exit('No direct script access allowed'); > +$route['blog/rss/all.rss']='blog/rss/index'; > +$route['blog'] = 'blog'; > +$route['blog/positive/(:num)?'] = 'blog/category/positive/$2'; > +$route['blog/positive/(:any)?'] = 'blog/view/$1'; > +$route['blog/positive']= 'blog/category/positive'; > +$route['blog/beautiful/(:num)?'] = 'blog/category/beautiful/$2'; > +$route['blog/beautiful/(:any)?'] = 'blog/view/$1'; > +$route['blog/beautiful']= 'blog/category/beautiful'; > +$route['blog/animals/(:num)?'] = 'blog/category/animals/$2'; > +$route['blog/animals/(:any)?'] = 'blog/view/$1'; > +$route['blog/animals']= 'blog/category/animals'; > +$route['blog/environment/(:num)?'] = 'blog/category/environment/$2'; > +$route['blog/environment/(:any)?'] = 'blog/view/$1'; > +$route['blog/environment']= 'blog/category/environment'; > +$route['blog/food/(:num)?'] = 'blog/category/food/$2';  
> > The initial default router is for blog name of "blog" .. this can be changed to any name or left blank in which case new routing is created dynamically
> >  
> > —
> > Reply to this email directly or view it on GitHub (https://github.com/pyrocms/pyrocms/pull/1707/files#r1412374).  
> >  
> >  
> >  
> >  
>  
>
Can you make sure that you follow the style guide? `if (` and the `{` goes on the next line.
Still, I don't see the point in changing the check to the interface. What's the point?
Oops, sorry :/
I looked at the DomCrowler tests, and the xml documents are defined inline.
I like the idea to make it external however.
Where should I put the file ?
under `Tests/fixtures`, `Tests/Fixtures` ?
@lsmith77 : you're right, should it be ```/** @deprecated ... */``` (as it's a phpdoc annotation) or is it ok in a standard comment?
This should be move to the outer loop
I really preffer to extract this kind of checking to methods to make the code more readable. But I'm reverting to the inline version since the extracted methods seems ankward in this API.
Better error messages for the end user are always a good idea. But changing the way you implement it would be better indeed
I think it would be more elegant to simply split the original if into three if-statements:

```php
Brace in new line. Same below.
Good idea. Pushed.
Just use `if "task_url" in r:`.
Userはじまりなら ```User#has?(topic)```がいいすね。owner?が目的語を取るのは違和感あり
Yes, but you don't know the target file until there's been an initial
parse, which means errors at that stage are not preceded by the thrift
file name in question.

Since the print only happens in verbose mode, I opted for + Compiling
file name (unchanged) when the compile is a no op.

Also I added prepending of file name to the exception println, which
make it weird when the verbose output is

```
+ Compiling foo.thrift
bar.thrift: exception!
```

On Apr 4, 2012, at 5:10 PM, Robey Pointer
<reply@reply.github.com>
wrote:

>> @@ -90,22 +90,35 @@ object Main {
>>     }
>>
>>     for (inputFile <- thriftFiles) {
>> -      val inputFileDir = new File(inputFile).getParent
>> -      val importer = Importer.fileImporter(inputFileDir :: importPaths.toList)
>> -      val parser = new ScroogeParser(importer)
>> -      val doc0 = parser.parseFile(inputFile).mapNamespaces(namespaceMappings.toMap)
>> +      try {
>> +        if (verbose) print("+ Compiling %s".format(inputFile))
>
> we want to avoid printing this unless the file is going to be compiled.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/twitter/scrooge/pull/11/files#r648361
delete this if self.growingTextView appears in the storyboard prototype cell.  Dont need two places to manage.
De ce avem nevoie de profile controller? Nu se poate de făcut toate acestea în user controller?
Oh, if we don't match on hostname then ack
On Nov 12, 2012 10:03 AM, "Eric D Helms" <notifications@github.com> wrote:

> In test/fixtures/vcr_cassettes/consumer.yml:
>
> > @@ -2,7 +2,7 @@
> >  http_interactions:
> >  - request:
> >      method: delete
> > -    uri: https://admin:admin@localhost/pulp/api/v2/consumers/integration_test_consumer/bindings/integration_test_id/677b72d667ceb77612d1/
> > +    uri: https://admin:admin@knifeparty.localdomain/pulp/api/v2/consumers/integration_test_consumer/bindings/integration_test_id/008a725be5d055f6fd0d/
>
> Unfortunately that is my configuration, which I suppose I can fix locally
> just to "clean" this test data up - however, we don't match on hostname
> anymore.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Katello/runcible/pull/22/files#r2098044>.
>
>
I'm keen on an easy way to get at the output of the histogram in text, but I don't think this function belongs in ImageBufAlgo.  At most it should be part of oiiotool.  (Even better, we figure out a way to arrange text output in a more generic way so that this is unnecessary.)
This should be refactored to use ImageBufAlgo::parallel_image, but the use of boost::thread::hardware_concurrency and the need to switch to boost::bind scared me away from making these changes in the first pass.
This is just to check you are connected to the server ?
`json` is also a valid flag rather than `text`
fields are final now?
I would rather just say `graph` instead of `dot graph` here, as it's misleading
I would remove all of the debug print statements.
We wouldn't need a version bump if this was going to be in the same release as all the other changes that went into urn:jboss:domain:1.4. We only need to bump the schema once per Final release/tag. But it looks like we are going to be doing a tag with a more limited set of changes than originally planned, with this feature not one of them, so the schema bump will prove to be needed in the end. :)
Sublime Text 2 has a ton of plugins. One is to run JSHint and look for directory level .jshintrc files. That's what @scottgonzalez and I use.
You can check for exactly 200 here, as everything else is not allowed by XML/RPC.
Не знаю, можно засунуть куда-нибудь вроде PreparedTopic/PreparedComment.

Тут @hizel уже подготовил патч который Template совсем удаляет как класс,
так что в него не надо ничего добавлять

On Sat, Sep 29, 2012 at 11:26 PM, Yury <notifications@github.com> wrote:

> In src/main/java/ru/org/linux/site/Template.java:
>
> > @@ -245,6 +245,20 @@ public String getNick() {
> >        return currentUser.getNick();
> >      }
> >    }
> > +
> > +  public String getRemark(String otherUser) {
> > +    if (!isSessionAuthorized()) {
>
> хм. тут подумать надо, куда сохранять.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/maxcom/lorsource/pull/336/files#r1723788>.
>
>
а тут оно откуда берется? 

Я в прочем не против Configuration в аттрибутах; мне не нравится что путь передается из jsp view в java код реализации. По идее java код и сам должен понимать где это взять, а в JSP этого быть не должно
It's better to check for this in onSliceRequestCompleted.  We can send a zero-length addedItems just to let the user know we are synchronizing, so it could result in an odd situation where we claim a folder has no messages initially, when we just don't have any yet.  Waiting for onSliceRequestCompleted avoids this.  The logic (apart from the !addedItems.length check) should work there fine.  Please fix this in your next PR or in a follow-up to this.  r=asuth to merge a patch with just that change.
I believe the z-index here has been set to make sure it is above the lockscreen. @timdream Can you comment?
Those call name seems a bit excessive, why not simply highlight-red, highlight-green, highlight-blue
add a line-break above
Use addEventListener instead of onload.
is for having a common name for `radios` and `checkboxes` as the final style is applied via `[type="radio"]` or `[type="checkbox"]`
I think so, because i wanted a persistent event. :active is fine when the finger don't move. 
i wanted the .hover to be set even if the finger moved all over the button.
You should centralize `click` event handling in `handleEvent` and dispatch them to function like this with `switch` `case`, instead of abusing `bind()`.

Also try not to pass `evt` object around when you do things like above.
Lockscreen doesn't load panel when change state from triggered to normal state, so I kept this in unloadPanel.
> is that similiarly doing too much?

Yes.  That may explain what @timdream was asking about in bug https://bugzilla.mozilla.org/show_bug.cgi?id=766873.  That would make a lot of sense...

> The only time a second window open did actually trigger was the 4th method of http://www.popuptest.com/goodpopups.html

What do you mean?  All four links on that page should trigger popups, right?  Do you mean, you clicked each of the links twice, and only got a second mozbrowseropenwindow for the fourth one?
Ok, you are right, there is no need at all.
Is is on purpose? The comment says the contrary of what the variable does.
Here, there is no need and I violated my former statement, so I removed it.
Since the value will be null, you can just do away with the defaultValue arg completely.
Is this supposed to return something?  The function name sounds like it does...
Love the note!
yep, see https://github.com/soundcloud/bazooka-hooks/blob/master/server/server.go#L450
there is no simple example.  I had to discover it with a unit test.  I don't think it is worth going into other than as I say, if you want the output aligned, include it in the scan function.

See the unit test and see if you can help me by explaining that weird behavior in a simple way.

Alternatively, what do you think cascading should do when you scan and output more tuples than you put in.  What about the fields you don't mention?  What should cascading put there for the extra tuple values?
Okay, I'll revert.
Oops found a bug already.  `merge` is modifying `attributes` which it can't do.  I'll fix it.
Just use `Djinn.log_run`, and amend it to use `HelperFunctions.shell` instead of backticks to make it mockable.
Nを設定した場合メール設定をスキップするようにしていますが、何も入力せずにリターンキーを押した場合にもスキップするようにした方がいいと思います。
Provides better approach to set Options in field chain itself. no need to make object and access options property.
I prefer my way of code, but thx for your advise
Why not just use kmVec2 final_intersect = {0};
As I know, {0} will initialize all value of struct to zero.
Good point. Fixed at line 144 now.
Actually the `<` should be `<anything-dev` because 2.0.0.0-dev is smaller than 2.0.0.0 for example.
do we need this? I'd like to start as slim as possible and build up what we actually want, not what's cool
Or this…
why is it gone? :P
Please don't change unrelated lines
fixed. Sorry
It's not the same thing. `@request.path = @request.path_info + @request.script_name`.

If i just patch it like before. ( i mean not patch the `@env['SCRIPT_NAME']`

the `@request.path` will be like this `/test.git/test.git/git-receive-pack`
Good catch! Thanks for pointing it out. I also very much like your improved suggestion and will adopt it.
Why use `&lt;` here? Does it render incorrectly on github otherwise?
please see my additions to this pull request
This workaround is very ugly & introduces unnecessary dependency on **Services.jsm**. Please inject `ChromeWorker` in the scope through the [loadSandbox](https://github.com/mozilla/addon-sdk/pull/548/files#L2R195) function instead.
I see why you used camel case and it's convenient, but I still think that consistency is more important. So if you don't mind let's go with either dash-delimited keys or single word names. I think you could just use `xpi`, `template`, `rdf` as names.
BTW: Factory / Service may be manually `unregister`-ed regardless of these options.
can we make a bug for this and add a comment here linking to the bug about this issue.
What you mean exactly? It seems out of the scope of this pull.
Instead of duplicating code, call user.canSetPassword
There is no need for an empty line. Also use const instead of var.
That'd only partially refactor this code, because I have to call new_resource.exporting directly to properly set up these flags.
Fixed in 953d966
`<div />`?
Can't we rather use a constant than typing in 1
Doesn't it make more sense to call apr_sprintf() once and reuse or apr_pstrdup() the value?
You want it in nanoseconds?
Would it be possible to reformat this as a sequence of ifs with early returns?
implementation of stand-alone REST service process
@samiluukkonen Korjataan tähän myös, että käytetään kukarow['taso'] parametriä.
Is this similar to unidecode, which is used in some other places? That takes `u'我爱你'` and returns `u'wo ai ni'` or whatever.
This is wrong. Entities must have an identifier as it is used by the unit of work to store the reference to the entity.
That's a problem of PHP. @lstrojny maybe you could help PHP core to implement a concept that Java has called Interceptors.
That's how Hibernate solves clearly the issue in Java. The get/set property patch may also fix, but it needs approval afaik.
but anyway, the method argument never used the underscore. In the old CS rules inherited from PHP4, the underscore was used to indicated a private property of the class (as there was no visibility in the language). But the argument of a function is not private.
What is `$comparison->getValue()`? I don't see any reference to Comparison construction in this PR. Is it something in Common?
Unfortunately, `module_info(:compile)` is not an ordered list, so we can't use [:options] brackets syntax. :S
Don't all quickstarts require JBoss AS running?
Consistency above all. Coercive comparison is used everywhere else in es5-shim when one operand is a result of the typeof operator. There's no reason that this code should break that consistency.
`self.assertContains` (I forget the order of the arguments) checks for a string in an HTTP response object. You pass it the whole response.
I wonder if calling `db-migrate up` should actually be part of the puppet config instead, and `start-server.sh` should skip doing that? If you ever wanted to launch the app on an older schema or something, this could get in the way. Then again, if you're making schema changes you'd have to keep running `db-migrate` manually which is possibly just as annoying.
Haha thanks, that's some pretty bad engrish right thur.
After a bit more reading, logic is fine, it's just the indentation!
it's just :cake:
findはフィルターで。
コメント残ってる
There's already a `KhanUtil.toRadians()` in `utils/angles.js`. If you want to use it, you can make this module depend on the angles module by adding something to `moduleDependencies` in `khan-exercise.js` Then when an exercise does `data-include="angsegcongruency"`, it will automatically load `angles` first.
Why the different color dots?
right, ``q1.unit`` should return the unit object
i like having these defaults live as constants at the top of the file so it's clear where to edit
neat. didn't know you could use else like that.
Those files are not ignored by the config finder and not hardcoded?
I get what does this function do, but that's not really sexy, especially the `array('prefix' => '', 'limit' => 65535)`. Still I don't have better ideas, and that's kind of a detail, so :+1: for me :)
How about: ``hashlib.getattr(self.algorithm)(salt + password).hexdigest()``, then the SHA512 pw hasher would be as simple as a subclass of this, with only ``algorithm = "sha512"``
it is good you moved this code here - but you have to remove it from Item::remove - otherwise the check is done twice. can you update Item.php and add it to the pull request please?
You're right, I removed it.
`api` is not used in this file. It should be removed.
spelling error: this should read supports

i would call this method `isSupported` and document

    Whether the router supports the thing in $name to generate a route.

    This check does not need to look if the specific instance can be resolved to a route,
    only whether the router can generate routes from objects of this class.
nee. deze zit ook al in de master. dit is de mousetrack van de crosshair die aan je muis plakt.
Не говоря уже о том, что сделано неправильное объявление этого метода, - отсутствует тип возвращаемого значения. 

--
Your sincerely,
Sergey Lebedev.

Twitter: @drasgo
Facebook: s.a.lebedev

-----Original Message-----
From: goodman116 <notifications@github.com>
Date: Sun, 25 Nov 2012 09:44:03 
To: Hexlet/osx-project-1<osx-project-1@noreply.github.com>
Reply-To: Hexlet/osx-project-1 <reply@reply.github.com>
Subject: Re: [osx-project-1] FINAL (#331)

> @@ -0,0 +1,21 @@
> +//
> +//  Cell.h
> +//  DNA
> +//
> +//  Created by phantom on 08.11.12.
> +//
> +//
> +
> +#import <Foundation/Foundation.h>
> +
> +@interface Cell : NSObject
> +
> +@property (readonly) NSMutableArray *DNA;
> +
> +- init;

Указывать init здесь излишне, т.к. он уже объявлен для родительского класса NSObject, а в Cell.m мы просто изменяем процесс инициализации. 

---
Reply to this email directly or view it on GitHub:
https://github.com/Hexlet/osx-project-1/pull/331/files#r2216965
Тут совсем не в универсальности дело. Согласно заданию надо задавать процент элементов, которые должны мутировать, а не количество.
интересно. я сделал бинарный массив и перемешал его. а потом обходил весь что бы понять кого мутировать. в вашем случае быстрее.
Hmm, incompatibility with other plugins...
This still does not need to be its other method.  You at literally repeating the code for factory_girl which makes no sense.
sempre retorne o resultado obtido pela funcao, e nao os parametros de entrada
$postData['id'] -> $model->getId()
Backticks around RACDisposable.
same here : logging or please remove
> 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.

Well, the current unit tests would pass by using the blob id for equality tests.
I think I'm hitting against my limited git knowledge here. I don't even know how I can make it so that a blob can be used in two notes, each one having a different target.
'''python
try:
  user, cmd, who, msg = msg.split(None, 3)
  except ValueError:
    return
  if cmd != 'PRIVMSG':
    return
  if who != '#' + self.config['channel']:
    return
  username = user[1:user.find('!')]
  if who == 'xmppbot':
    self.xmpp.sendmsg('-%s' % msg, username)
    return
  self.xmpp.sendmsg('[%s] %s' % (username, msg[1:]))
'''
应该是这样。who是xmpp的话那么应该是命令，私信发到xmpptalk之后return了（应该就不再执行下面的self.xmpp.sendmsg('[%s] %s' % (username, msg[1:]))这句了吧）如果who不是xmppbot那么按照常规是channel里的信息，转发到xmpptalk群里。
t_typeName = temporal Variable
m_typeName = member Variable
add "titles=dict(grad='Evoked Response (grad)'))"
returned variables are not documented
sure, we're on the same page, just for clarification. 
what about a pointer to $SUBJECT?

On 16.12.2012, at 19:59, Eric89GXL <notifications@github.com> wrote:

> In mne/viz.py:
> 
> > @@ -866,6 +867,79 @@ def plot_cov(cov, info, exclude=[], colorbar=True, proj=False, show_svd=True,
> >          pl.show()
> >  
> >  
> > +def plot_source_estimates(stc, subject_id, surface='inflated', hemi='lh',
> > +                          colormap='hot', time_label='time=%0.2f ms',
> > +                          smoothing_steps=10, fmin=5., fmid=10., fmax=15.,
> > +                          transparent=True, time_viewer=False,
> > +                          subjects_dir=None):
> > +    """Plot SourceEstimates with PySurfer
> > +
> > +    Parameters
> > +    ----------
> > +    stc : SourceEstimates
> > +        The source estimates to plot
> > +    subject_id : str
> Ahh, I see. However, given that we're essentially hiding the pysurfer calls from the user, I think it makes sense for us to use 'subjects' so users of our package have the most consistent usage patterns in our functions. (Imagine if in the future we add functionality from another package that had a different name for 'subjects', users would have to remember a third convention...)
> 
> —
> Reply to this email directly or view it on GitHub.
>
That isn't a problem, the changes never get committed to the in memory DB, whatever is added during one test isn't visible to other tests
The new view code is less defensive than this one.  I can add the exception handling back in to make the alert work in the JavaScript, if you want.
You need to use a double backtick for inline code blocks in Restructed Text.
Please keep this as it is still a place where you can do it. This file still exist in a Symfony project
我们是lang="cn"
The description is probably wrong.
Agreed. It is rather silly to set it conditionally. Anyway, should be good to go. Thanks for the suggestions.
This approach is discouraged, but not explicitly illegal.
that one seems not necessary. Am I wrong?
Code looks good.  Just a small FYI - the Ruby standard for line indentation is two spaces.
please declare used namespaces and refer to the short name in the doc block
I named it this way to maintain consistency with the existing "indexSearch" against a bucket: https://github.com/basho/riak-php-client/pull/19/files#L1R1033
Hmmm2
line to be removed?
Hmmm, 'is' is a parameter, shouldn't it be the responsibility of the caller to close 'is'? Well, the caller, or whoever really opens the input stream. IMO, close calls here should be limited to those resources that are open within this method itself.
private final? 
just to hint the JVM so that it can inline invocations
I was thinking that probably code uses the fluent async().blah() rather than referring directly to the AsyncLoaderConfiguration class, but you are right
This test worries me. The way you have implemented it is about the best that I can imagine for using *actual* timings. 

Based on these concerns (and the ability to test that the functionality that you have fixed continues to remain fixed) it makes me wonder if we can improve the implementation of Artist. I've just posted a message on the matplotlib-devel mailing list on the subject of "Use of meta on Artist". It may be that we stick with your approach (which *does* fix the problem), or that we decide to change the implementation a little.
or Events.jsm
Thanks @malini I wish I'd known about this sooner, I would have implemented it sooner. 
By the name I thought it was something to do with execute_script :)
well, it's okay to me (even though it means the user will be forced to minimize the app to check the time or signal strenght or battery) we just have to decide :+1:
And these extra whitespaces etc... just style things...
Since 'Vendor' is also used in the context of vendor extensions to OpenLayers.Format classes, it would be great if you could come up with a different name. Maybe OpenLayers.XBrowser?
update the patch with commit --amend and i'll merge it in
I had to remove these pre and post conditions in order to allow for screening in hoptree of nil values.
happy to see all that conf stuff whittled down to size!
Thanks - I think I got them all.
ahhh, wait, i get it, sorry, silly me, this isn't an if, else clause :smile:
"else if" -> "elseif"
Sure, I was hesitating on going those directions in both cases.  Thanks for agreeing with my other head.
@cyberalien 
Hum... Ok, makes sense, that's also how it is implemented in viewtopic_body.html.
Expect those changes to be applied into the next commit.
@p 
No. You need to scroll further down. In the bottom div, you have to find tester_134's post.
Note that (IIRC) if his topic was removed it will not appear in the review (I didn't change that part)
Is this sentence missing a period? I'd also suggest writing `make rpm` as inline code and italicizing _a_ and _b_.
Do we want to allows keys starting with numbers? Keys and number values are then not that easy distinguishable. At the moment it will work by position. But in future?
this will enclose all caps abbreviations etc. (i.e. all words with caps in non-initial position) in non-title fields in curly brackets - things like New York, {NY}. My understanding is that bibtex is pretty rigorous in its application of sentence case, so that's what this is for, but I'm not sure that's true and we may not want separate functions for title and non-title fields.
Auf Zeile 61 ist "_GREATER_3" erwähnt, auf Zeile 62 nicht. Vereinheitlichen.
Should this not be `callback('error', response.statusCode)`?
return should be on its own line.
Agree on raising an exception if the parent folder doesn't already exist.
Make key names string.
Why readonly? No, you want to be able to programmaticaly manipulate this fields. You probably just do not want that people create that through API, just read. You limit that by limiting HTTP methods allowed just to GET.
Why removing this TODO?
No need to include commented code.
but i do call startWorker (via startTransport) i just hold off on it until I've setup more on the transport
Aquí el único detalle es que no tenia el usuario **www**.
struct fields are public by default, that line is not needed
Effect 0: Id 6 (SPELL_EFFECT_APPLY_AURA)
BasePoints = 25
Targets (1, 0) (TARGET_UNIT_CASTER, NO_TARGET)
Aura Id 190 (SPELL_AURA_MOD_FACTION_REPUTATION_GAIN), value = 25, misc = 609 (609), miscB = 0, periodic = 0
Player *player -> Player* player
No! you see, this is why it's important we can see what exactly changed.
I remember thinking, "man, this is going to be ugly to write a regtest for."   Still playing around with it.
I'll rewrite pig's parser one of these days. Mean time, I'll change the default alias to... `m`?
Ah, I see you don't have to.
lol. Yep, that is the construct. :-)
Why create a global?
Maybe hold on reviewing this file. I can generate far less markup by flattening the structure. I have no seen any performance issue with the current markup but it doesn't hurt to generate less markup.
`catalog = sorted(load_catalog(), key=lambda x: int(x.bookid))`
 here too, I like that syntax
Very nit-picky comment, but maybe the hy_ isn't needed. Just test_var, test_kurt, etc. I think hypothesis is implied.
да. сейчас нечасто меняется.
ultra-nit: consistent colons, feel free to ignore.
I think that is not OK, because then we would have two search bars (one in header and one default on the result search page).
FWIW, I've created a persistence layer on PyGilt, so a couple of my changes were to allow it to play nicely with SQLAlchemy e.g. converting your object to "new style" Python objects.

https://github.com/doofdoofsf/GiltAlchemy
I do the check like this so that external classes can freely mix setDelegate and setMsg* calls without worrying about a 'correct' ordering.  Everything in this class explodes if delegate is null, so I think it's safe here.
Judging by this tag that I've never seen before and shouldn't have ever existed, [it looks like this was made in notepad,](http://www.ehow.com/how_6771978_create-website-notepad.html) which explains a lot.
He has _zero_ medical licenses ;)
you can delete the locales...they are now in a separate file.
Hello Rene,
there seems to  be something wrong here
This works for me:

```
        Map qpidRuntimeFiles;
        String pluginjar = "src/test/resources/qpid-test-plugin.jar";
        String configfile = "src/test/resources/qpid-test-config.xml";
        if (new File(pluginjar).isFile()) {
            qpidRuntimeFiles = [ ('lib/plugins/sample-plugin.jar'):new File(pluginjar),
                                     ('etc/config.xml'):new File(configfile) ]
        } else {
            qpidRuntimeFiles = [ ('lib/plugins/sample-plugin.jar'):new File('software/messaging/'+pluginjar),
                                     ('etc/config.xml'):new File('software/messaging/'+configfile) ]
        }
```
A  very good remark.

I would like to see that a LocalManagementContext is explicitly configured before starting an application. The LocalManagementContext will be configured with the correct config, and the applications can access this config by calling getConfig.In this ideal world, there is no need for such a 'brooklynProperties' field.

The problem however is that we don't impose such an approach, a user is allowed to make an application and when getManagementContext is called, a LocalManagementContext is created. The question of course is how we can move properties through the application to the LocalManagementContext its constructor.

That is the reason this field exists. So this field only has meaning for Application without an explicitly set ManagementContext but with the need for properties.

It can be used like this:
Map flags = MutableMap.of("brooklyn.properties", properties)
SomeApplication app = new SomeApplication(flags)
app.start(locations).

If you know a better solution, I would be happy to implement it.
Strange indentation. Is you IDE set up to 4 spaces?
yeah, unit tests would be handy, but it's pretty basic stuff i think (apart from parallelisation but we don't need that currently)
Do you think we should have something like /v1/catalog/enrichers ?
The `:locals` (here and in the `new` view) will not be necessary with the `new_record?` solution.
OK, have replicated locally in 1.8.7. Currently investigating.
Pouring all logic into a single method is a pretty bad convention. I think we should work towards cleaning this up for two main reasons:
  1. It's easier to read (which happens 10x more than writing code) when things are split up into smaller, descriptively-named classes/methods
  2. It's easier to write focused tests for small bits of logic

Special exception classes make debugging and handling by users easier.
Seems like a dummy file.
I have cleaned a test folder a bit and will be pushing in changes by today.. Will appreciate a lot if you can move images and css in the new folder.. since most of the images and icon are shared by other webinos page too, i have creates images and css folder inside client folder.. If you have updated css or referencing them in your vehicle app that will be usefil. I will inform you once I pushed changes in. thanks
I prefer this indentation style so much now :-). Much more predictable.
this should simply be ``if ($public) {``
IMO we should still have the long name during selection and for disambiguation during assignment, but on the roster view just use the abbreviation.
I did it this way because when you're _modeling_ an entity, you know if you want to pick a property _buried_ in the json structure response and in my opinion, for an objC dev, should be easy  to use  this entityPropertyWithKeyPath or entityPropertyWithKey. 

Case example:

* [entityProperties addObject:[JSEntityProperty entityPropertyWithKey:@"user" relationClass:[TwitterUser class] propertyType:JSEntityPropertyTypeRelationshipOneToOne]];
**OR**
* [entityProperties addObject:[JSEntityProperty entityPropertyWithKeyPath:@"user.name" propertyType:JSEntityPropertyTypeString]];
Yes thats what i thought as well. I will do it.
can you modify this back to make casing of the properties consistent with the other classes

``` C#
public class User
{
        [DataMember(Name = "RAX-AUTH:defaultRegion")]
        public string DefaultRegion { get; set; }

        [DataMember(Name = "id")]
        public string Id { get; set; }

        [DataMember(Name = "username"]
        public string Username { get; set; }

        [DataMember(Name = "email")]
        public string Email { get; set; }

        [DataMember(Name = "enabled")]
        public bool Enabled { get; set; }
}

```
Кажется, я что-то говорил на лекции, что стартовать поток в конструкторе - это моветон. А ну вдруг исключение произойдёт - поток будет работать со ссылкой на this и никак его не прибьёшь.
Здесь ты закрываешь System.out
Could update this to say the BBC one isn't working and at present recommend using met office?
this variable local and is used in the method where is declared... so it's value will be available only in that method and it is not available in other place.

http://www.saltycrane.com/blog/2008/01/python-variable-scope-notes/#ex1-global-local
this method has no sense here...

if you will add it here some other people will use it and it will create confusion....
Why don't we just use self.selenium.find_element(*locator).click() where it's needed?
Can you please confirm one thing:
You want both cong/test/{user.name} and GEO_CODING_HOME properties out of the project?
(we should remove developer specific dirs from main and from test)
Actually, I think it makes more sense to leave them as floats, to ensure proper spacing of the grids throughout the whole image.  Updated
You should make this more explicit by rewriting the previous doctest example using `n_values="auto"`: the reason is that people scanning the documentation will copy and paste inline examples without reading the surrounding text paragraphs.
As @mblondel commented, this inconsistency stems from SGDClassifier.
I think it is ok like this in the PR, later we can change it everywhere.
Otherwise it is a bit confusing.
Not really important but I would prefer a test that takes a bigger X, does a bincount and compares ratios.
Same for ``test_uniform_strategy``. Though this is complaining on a pretty high level ;)
If I get your point, I can just safely delete the this function and the next function for now as they are not implemented?
Looks good to me. +1 for merge (let's postpone the mixin until more estimators get a `fit_predict` method)
This is still missing.
It looks as if it could be written as
``(contingency[nnz] / contingency_sum) * (np.log(contigency[nnz]) - log(contigency_sum) - np.log(outer[nnz]) + log(pi.sum()) + log(pj.sum()))``
maybe split up in three parts
```python
# normalized contigency
contigency_nm = (contingency[nnz] / contingency_sum) 
# log of normalized contigency
log_contigency_nm = (np.log(contigency[nnz]) - log(contigency_sum)
# negative log of marginal distributions (better name?)
log_outer = - np.log(outer[nnz]) + log(pi.sum()) + log(pj.sum()))
return np.sum(...)
```
It is essentially to be consistent with the doc. But yes, indeed both option are sensible.
I should have improved the doc instead of changing the behavior.
No, it doesn't hurt. I just thought it would be better, nothing more.
Maybe get rid of this tdata.
Use dynamic_cast here and check for null.
`Buffer.toString` is needed by the following code in Anvil:
```
util.js Line 54: var data = e.buffer.toString();
```
Can we make use of std::string rather than using raw char* which is more error prone?
Delete views you are never going to use like:  destroy, update and create one.
Why 'maxRows*2'?
Not sure what you mean by additional steps.  For this commit only 1 new step has been added, the translate.  outputPath is already being translated, and it seems that another translate step is needed to fix the path separators.
yep, possible. So simply fix the phpdoc and I will merge it
You (almost) never have to debug jquery itself. If you have an exception it's most certainly in your own code. And having the non-minified jquery won't help much.

The biggest reason is that the compression (aka minification) of jquery is a concern we don't want to get involved in. The pre-minified from jquery.com is tested and certain to work. If we do it ourselves we just add risk.
А нужна ли вся эта сомнительная конструкция? Во-первых можно убрать много лишнего
```ruby
user = current_user if signed_in?
```
А во-вторых, тут и ```user``` вроде не нужен уже, можно сразу ниже написать
```ruby
@comment.user = current_user if signed_in?
```
I did not want to just put it in system because I thought the shorthand methods maybe too controversial. This way it is in a different namespace and has to be imported separately.
please rename. after we spoke I'd suggest that you break this up into multiple tests.  when one of the assertions fails, the following assertions will be ignored.
@christianv sadly it doesn't look like you can wrap other handlebar statements in a handlebar comment.

This chokes:
>    {{! <td>
>        {{#if waitlist}}
>            Y
>        {{/if}}
>    </td> }}
Nm, i think i see the different now.
Would it? WLT should be calculated from the original team number and ignore all "alter-egos"
`(and (contains-point? outer p1) (contains-point? outer p2))` is enough, no `if` is needed.
Looks like we can use Draper with our `Person` object. Check it out:

```ruby
class PersonDecorator < ApplicationDecorator
  decorates :person, :class => Clubhouse::Client::Person

  def draper_name
    person.name
  end
end
```

```ruby
>> me = PersonDecorator.new(Clubhouse::Client::Person.new("jordanbyron"))
=> #<PersonDecorator:0x00000103e63af8>
>> me.draper_name
=> "Jordan Byron"
```

So let's do that for consistency. It should be a very easy change.
This will match more than just alias, sch as aliasa, aliases, etc. I would actually have the reserved words arrary be an array of string, then below 

if name =~ /^#{regex}$/

The $ is important.
Can't we use the same clean version of psexec we used on https://github.com/rapid7/metasploit-framework/blob/master/modules/auxiliary/admin/smb/psexec_command.rb ??
See [2]_.  Or merge into previous sentence: "image location [2]_."
Make the error explicit, e.g. "Segmentation only knows how to operate on grey-level image, but input array with ndim=%d given.".
Is padding with the last value standard? Would mirroring also work (e.g., `_image[:, origCols:] = image[:, -extraCols:]`)?
Can we just add `Support.Observer` to the extend line in `composite_view.js` instead?
khode file hash koo ?
Since the function starts with an underscore, jsdoc-toolkit will ignore it, so there is no need for the `@private` comment.
there must be a better way to handle this. Using the view to check if this is target of the event handler is awkward.
You should be using UriBuilder instead. String concatenation makes too many assumptions about what the default URL looks like - you're adding a query parameter, but if in the future there's already a query parameter here, you've now got two ? characters in the uri and invalid syntax.
I just realized I misunderstood what I was looking at. You're looking at the first object that was output into the channel, right? In that case, forget everything I said above.
Yeah, it's a little confusing. But the term is used extensively--not just in the docs, but in the Kroll layer. The descriptions of Proxy and Module correspond to the Kroll Proxy and Module definitions. So anyone writing a native Android or iOS module (sometimes called a Titanium Plus module) needs to know what a Proxy and a Module is, and needs to use those terms because that's what the native classes are called.
Stephen - it works on iOS according to my tests.

Something I have just found on iOS, though, is that that Titanium.Database.DB.close() closes the resultset as well as the database. Could you tell me whether this is intended?

Allen - yes, I have already documented that in the last PR.
They're listed in the code, but they don't actually work, due to a bug in Webcolor.m see:

https://jira.appcelerator.org/browse/TIMOB-8678

Try them for yourself...
Should add some doc here that explains what this class does and how it works.
Maybe this should be re-ordered to be more similar to Django style forms?  Put the form_post_data as the first argument (really initial instead of form_post_data)
Don't want Play to be confused about the generic parameter in the Result[A] and perform incorrect escaping. But it's ugly.
that is fine - but you cannot use/set bundle version so users for 3.7.0 cannot install it.