labofsahil/github-event-dataset-2012 huggingface dataset:
Tu vas me faire le plaisir de faire qqch de joli  la home'n'go dans la
topbar de la version web :p

Le script dans le lien, avec un popover bootstrap "glissez ce lien dans
votre barre de favoris" :)

Regarde comment c'est fait sur home'n'go ^^

On Mon, Jan 16, 2012 at 10:01 PM, Yoann <
reply@reply.github.com
> wrote:

> C'est le code qu'il faut insrer dans le lien que tu glisses / dposes
> dans ta barre de bookmark =p
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/yoann-losbar/BalloonGithubIssues/commit/b1e79bb2b3601b94571d67974a34595e6a8523be#commitcomment-872119
>



-- 

Guillaume
copy settings.java from cm9 source into framework/base/core/java/android/provider/here
Jep :)

This way you got a very simple plugin. Another thing they might mention is the missing localization ;)
Sometimes I purposefully leave the old CSS in there (including commenting it out), as it helps to make future merges from upstream theme changes a bit easier.
Not sure why these are in such an odd format. I think we can do without the IEEE80211_ prefix on these lines.
understood. thanks for the hit. i'll give the check_openmanage a try.
Yes please reimplement this i saw it requested in the thread and power menu settings in liquid control :)
1. You never know if it might be compiled using `-b`
2. It is best practice to use `@` only as a prefix as in `@method`, if you want to refer to `this` alone, just use it.
You're welcome !
this is not for I/O sorry about that not sure how this commit got in there...
Missing a comma at the end of this line?

Also, what does adding these different power supply types do?
This function has been deprecated. A bool attribute 'undeditable' has been added to the control_group function.
a cleaner option that's working and passing tests is to execute the proxy setup in a sitecustomize.py (instead of a pth file)
this allows the sys.path to fully build before pkg_resources is imported.
I think that's the winner.
re_RGB and re_Hex were in Y.Dom. They are only added to Y.Color for back compat
What is this doing? Comment would be helpful here.
fixed in 4e08f6f
More important than the highlight, it doesn't limit the search to the exact version. For example "2.0" will also find "2.0-RC" and "2.0.1"
Yep. I wrote that up in the ticket. You didn't see that? The Gossip does not help the user much. We should do the diff and publish actual changes.
Ah, didn't know. Nice. Thanks.
I'm not a huge fan of adding dependencies (even for tests), can we avoid these?
well, that was done by Roland in the original fix for 2.0.4, and I can agree that the sample is too long to be included here. It's included in the linked `fault-tolerance-sample`
This is the part where we do the simple, straightforward way of invoking functions just by name.
I'm keeping this to remind me I need to do something about making this a pack and having codenames assigned by packs.
These sorts of headers are italicized everywhere else in this page, don't forget to add it here.
I went pretty all out on the transclude include change, text and code, make sure it is right.
this one is hard...shouldn't we stop protocol first to avoid receiving new call...but then we can still execute actions...I think protocol should have start/stopServer and start/StopClient so that we stop server, then services, then client.
Still confused why logging is being configured here.
I should have been clearer: ``JOB_FLOW`` and ``JOB`` should say they only fire if the job has not completed when cleanup runs.
I'll update the README tmw, no problem.

I *would* suggest changing the name as well, I meant to bring that up.  bigml-r?
This may be unnecessary, but ok
We could post new copies to maven central with other group ids. Dojo in particular looks like it won't be released again, not sure about YUI.
Actually I'm using a patched version of YUI, so it is out of discussion.  
Regarding your suggestion - it is a good idea. The only problem is that I wouldn't like to become the owner of dojo at this moment (it would require extra effort and time). But anyone else contribution is welcome and appreciated.
It'd be better to use service instead of system
you mean `LX-2` here...
Let me take a wild guess: retain cycle. I'm obviously incorrect. =facedesk= (I'll go fix that now...)
This is horrifically unsafe, because it depends on the exact representation of `CGFloat` values. Each member of the structure should be encoded independently.
This is rather wired, would it not be an idea to prevent `process.nextTick` after the `exit`event has emitted.
I think you could get away not having them, but it  is standard. See http://en.wikipedia.org/wiki/32-bit
thanks!  i changed it to use Feed:: instead of self::  please let me know if there's anything else.
Also, it might be a good idea to decouple the overrideMimeType call from setting the Accept header, now that we can support APIs for both. Speaking of which, we need a `xhr.header(name, value)` method.
[code]
if (out.hasArray()) {
    process(encrypt, in, out);
} else {
    try {
       ...
    }
}
[/code]
@njsmith: Good point.  I think either version could produce that sort of output, with about the same amount of code.
здесь и ниже больше 120 символов
не очевидно, что этот метод делает, нужен комментарий
Where this is used?
Oh, sorry! Fixing.
I think it'd be possible to put these system calls together with `;`, something like:

```console
`bundle exec rails g model user;
 bundle exec rake db:migrate;
 bundle exec rails g observer user`
```

And perhaps would be possible to use `Dir.chdir` only once, and execute the tests inside it, wdyt?
Here you could use `map.with_index` instead.
You probably want to do `if options.key?(:default)` and also ensure a value can be set to false.
Perhaps best to annotate the return types, e.g:

```scala
implicit val stringFromRepr: FromRepr[String] { type A = Char} = new FromRepr[String] {
```
Can you put these 3 lines in a single expression? Then the ``del IN1`` statement can be removed too.
please remove ``NumPy`` and ``Python``.
BL -> bodylist

using readable words instead of abbreviations makes it much easier for the end user to understand
`rm_id` removes all identities unless there are only identities in the args. In that case it leaves one. The idea is 

    X op Identity -> X

but

    Identity -> Identity

> `And(0, 0, 0)` should end up as `0` and not `Add(0)`. 

`rm_id` is often paired up with an `unpack` which turns `Add(x)` into `x` for any `x`. I thought it made sense to separate out the extraction behavior. In general, as Sergiu points out, each rule tries to be atomic. It tries to do no more than one thing.
The ones in generators.py are only meant to output all the group elements in a list
It doesn't dispatch any event. Moreover, it's mil*l*iseconds.
Need code example here.
The gin.ginjector cannot be removed, it's used as a return type for the delayed bind registery when it comes to generate the proxies.
Removed, b/c build work with different configuration. :)
documentation as in something in the docstring? afaik testify doesn't have docs outside of the readme.

setup is a general purpose way of setting up the world where let and let_before are specifically meant for setting up test *data*. if we (testify) were to prescribe how to write tests, i think we'd say always use let and let_before unless you need to do more complicated setup. i'm not sure all of that would be appropriate for the docstring for let_before :)
Ok, nimmst du die Umbenennung auch gleich in den Pull Request?
Looks like a mistake, please revert this part.
if you wrote this class then have @author yourself
But the for-each would use the same iterator(), right?
Shouldn't these be >= ?
$parse_post_flags
Then again there's nothing wrong with being explicit I would say. A use statement is kind of useless if you only refer to something once.
Annoying as it is, does it cause any problems? Again, what filenames will be used by extensions implementing new avatar types?
If $user is optional, it should not be taken from global but left null. If it is required, the parameter $user should be required (and argument order changed).
I read this line about a dozen times before I spotted the difference between it and 88.

self._connectionStream:getActiveTimeSyncClient()
                                      vs.
self._connectionStream:setActiveTimeSyncClient()
Functia spell_amount returneaza un array care contine o singura valoare. O sa modific functia sa returneze acea valoare si nu un array
propun să scoți deasupra partea de „dacă nu există, creează” (atât pt folder, cât și pt fișier), iar apoi să existe doar o dată logica de scriere. De asemenea, sunt cam lungi liniile.
Using the classic sinatra style like this works great if you only intend the server to be runnable stand-alone like this.  But I think there's great potential for this to be mountable :).  That's the way the resque server is, and we mount it into the CMoz rails app and the silo rack app.  To switch to that, here's what you'd need to do:

* Move this into lib/qless so that it can be required from a consuming app using `require "qless/server"`.
* Change the sinatra require below to `require "sinatra/base"`.
* Subclass `sinatra/base` and move most of this code into the subclass.

The resque server is a good example of this:

https://github.com/defunkt/resque/blob/master/lib/resque/server.rb

And you can also check out any of the silo sinatra apps:

https://github.com/seomoz/silo/blob/master/lib/apps/event_feed.rb
https://github.com/seomoz/silo/blob/master/lib/apps/collector.rb
See other comment about `cmp()`.
I will add (and review all other methods for the same omission).
Yes, formatting is important.
Many Many Additions
Please don't refer to the split package as `split_0_1_4_3 ` here. If the packgage needs a specific version of 'split', then that version should be passed to the expression as an argument called `split` in the 'haskell-packages.nix' file.

Generally speaking, I wonder whether it would be possible to patch `urlencoded`'s Cabal file to accept `split-2.0`. I'm pretty sure the package doesn't actually need the old version, its dependencies are simply inaccurate. Maybe the upstream author can be persuaded to release an update?
Let's just delete the function instead (#112).
add ```and numpy.ma.count_masked(array) != 0``` and we are good to go.
You don't need a separate build step. ```python setup.py install``` should suffice.
nit-pick: PEP8 suggests that using parentheses to split lines is preferable to a backslash (http://www.python.org/dev/peps/pep-0008/#maximum-line-length) - in this case, as you already have the parentheses, you could just remove the backslash.
Op done with the instruction -> Ops created with the instructions
DO assert here that (activeNumSearchKeys > 0) or DON'T assert this later.
Do we really want a WARN message at 100k txn/s? This will obliterate useful logging.
I didn't have any specific method(s) in mind - maybe the mix and intersect methods?
This is a style issue and not a pull-stopper
That's what I was concerned about - it's quite a big job, and there's no way it's going to happen in the next week.

I'm starting to be less sure this patch needs to be in 1.5 - after all, without the Field API changes (which I could feasibly do in the next week but might be a squeeze) 1.5 won't be directly usable by migrations, so perhaps it's best to wait until the 1.5 branch is forked off and then give 1.6 all of the schema alteration, field API changes and migrations-running code?
I think it would be nice to use a random high port here, to prevent 'address already in use' errors etc.
![http://i0.kym-cdn.com/photos/images/newsfeed/000/085/444/1282786204310.jpg?1318992465]
Actually, it's impossible that this is effecting, right? I think this is OK, and perhaps the ticket I was thinking of was mentioning that this was OK.
This function should be a utility function. However, the name of this function is not good (we can use array_search), and the implementation can be described as follows.

```
CREATE OR REPLACE FUNCTION array_search(find ANYELEMENT, arr ANYARRAY)
RETURNS BOOLEAN AS $$
	SELECT count(*) = 1 
	FROM (
		SELECT unnest($2) as elem
		) t 
	WHERE $1 IS NOT DISTINCT FROM elem
$$ LANGUAGE sql STABLE;

```
If we change control file, we should change madlib--{version}.sql file respectively.
I'm not familiar with doxygen rules, but it seems other stuff like regress has @ internal instead of this style.
No effect at all from this change.
Hmm.. not sure why I don't check if value > 5 here instead of making an else-clause.
should be spaces on either side of the `+` here (and in subsequent cases)
`args << "--with..."`
On 28 March 2012 16:49, Misty De Meo
<reply@reply.github.com>
wrote:
>>    homepage 'http://www.openssl.org'
>> -  url 'http://www.openssl.org/source/openssl-0.9.8s.tar.gz'
>> -  sha1 'a7410b0991f37e125bf835dfd1315822fca64d56'
>> +  sha1 'a6476d33fd38c2e7dfb438d1e3be178cc242c907'
>>
>>    keg_only :provided_by_osx,
>>      "The OpenSSL provided by Leopard (0.9.7) is too old for some software."
>
> I guess this needs to be updated now!

I'm not familiar with apple support policy, bump it to Lion Snow Leopard then?

A+
Dave
This is the standard style for raise simple string exceptions
Why is this 400 now?
"transport, application worker or dispatcher" -- two "or"s looks clunky.
The idea was actually that this wouldn't get called externally. I haven't built the wrapper around it yet, but I'm about to start on that.
Another "verify" method that I think more properly should exist entirely inside a Cucumber step definition. Please confirm with CVDL.
Can we just list the included and excluded categories?  The main time this is used is when the framework has to inform the user that, unfortunately, the filter excludes all of their tests.  In that situation, more information is definitely better than less.
The reason why I used IllegalStateException is to be consistent. An IllegalStateException is also thrown if for some reason a file could not be created. (line 64)
My hypothesis is that they essentially are POJO's, but because they're created with the same constructor, the VM should create a shadow class for all of the objects to reference internally. Not sure if it will be a drastic enough change to actually even blip Watson, but I'll check it out. Would be good to have some proof of that one way or the other. They still hang out on the stack though, they just have a prototype that isn't Object.prototype.
Would it be okay to have a module like `utils/Resizer` and scan for resizer classes on initialization to add the necessary elements as well as configure the behavior?

It could work also on the sidebar...
Agree. Will update.
It looks like the "clike" mode provides specialized functionality for several of these languages (see the bottom of clike.js).  I _think_ if we returned one of those mimetype strings here, CodeMirror would then use the language-specific settings.  E.g. for Java we'd return "text/x-java" here so that the Java-specific set of keywords are used.
Fixed. I also changed "MenuItems are views of Commands and control a MenuItem's ..." to "MenuItems are views of Commands which control a MenuItem's ..."
Yep, will rename
Yep, been in there for a bit-- removed.
why a list? ordering is not guaranteed
Reverted? I thought it was.

On Tue, Dec 11, 2012 at 10:54 AM, Jeff Ciesielski
<notifications@github.com>wrote:

> In lib/stm32/f1/Makefile:
>
> > @@ -23,7 +23,7 @@ PREFIX		?= arm-none-eabi
> >  #PREFIX		?= arm-elf
> >  CC		= $(PREFIX)-gcc
> >  AR		= $(PREFIX)-ar
> > -CFLAGS		= -Os -g -Wall -Wextra -I../../../include -fno-common \
> > +CFLAGS		= -O0 -g -Wall -Wextra -I../../../include -fno-common \
>
> Is this required or can it be reverted?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/iCD/libopencm3/pull/7/files#r2382693>.
>
>
Aha, I see. That does indeed complicate things. @dpp does this look reasonable to you then?
I wanted to do the same way it is done on wlan homepage.  Now it is fixed.
`next_url` is not defined here anymore, look the scope where `next_url` is defined.
Don't import like this. What if there is some Python package/module in Python path with name `account`? Use absolute/relative imports. Or `from piplmesh.account import models`.

Also order this import bellow. First come system modules, then Django, then other libraries, then our imports.
if `options.name` is ever undefined we'll have some problems below. Perhaps this fn should not execute at all if there's no options or options.name value.
Can you use `classList` instead?
This will deselect everything even if the track event can't be pasted, i.e. if `start > _currentMedia.duration`, no?
codes above 0x10000 must be split into high and low surrogate
Jumped from a <h2> to an <h5> - if done for styling reasons then maybe attached the heading styles to a class that can be used on any heading/piece of text to take on that particular colour/size
这里不应该返回 rs.PutFileRet，而是 CallRet。
Try to keep lines under 100 characters for readability
Indent this, gets hard to read otherwise
recently I had to do a double check to find out if something was a local record, not sure if the same applies here as well: https://github.com/GeoNode/geonode/commit/fad1a4d9f935504c11f184d6cd6367b11e35f400
Why this constraint? Is there any added value in it? I mean, I could be able to create multiple "x" -> "y" combinations, if I want. It only takes resources IMHO.
So it can be removed now?
You'll never know! :smile: I think I'd probably need to write a one for Orchard, although I'm not sure. Better to have it.
miss the doc for a public method
We need get the count for the init process to ensure `IndexedDBStorage.prototype.isEmpty` function(line 1717) returns the right value. It won't take much time to get the count value, so I keep it.
I thought about downcasing the `attr` in `__getattr__` when deciding whether to delegate to the format or scale changing code.  I wasn't sure if this was a good convenience or if people would be confused.
Is there a reason for using ``dl`` instead of ``download`` in some places? Why not consistently use ``download``? (these are not functions that users will use all the time, so I don't think the length of the function name is an issue)
As mentioned above, the `conesearch.py` module docstring duplicates some of this information.   I think all of the "selecting and managing databases" information should be here, with `conesearch` just linking to this.
My understanding is that a factory class returns a class, not an object instance (as is the case for `Coordinates`).

http://en.wikibooks.org/wiki/Python_Programming/MetaClasses#Class_Factories

Or is there a subtle difference between a "class factory" and a "factory class"?
No, they are not running the full validation as it would make it far too slow. They are simply skipping unknown keys
Should we do an if (fClient) return;
__attribute__ is gcc only, we should try to stay compiler-independent as much as possible.
That one can go, too.
isnt the input always a string? maybe `$nextValue !== '0'` might be cleaner here maybe?
1.2.0? You were a visionaire hehe, just select a correct version and year please :P
If the fonts we're using in c2cgeoportal do, let's go with this character.
tahoma,arial,helvetica,sans-serif;


On Tue, Dec 18, 2012 at 10:47 AM, ochriste <notifications@github.com> wrote:

> In core/src/script/CGXP/plugins/GetFeature.js:
>
> > +                }, this);
> > +            }
> > +        }, this);
> > +
> > +        return {
> > +            internalLayers: internalLayers,
> > +            externalLayers: externalLayers,
> > +            unqueriedLayers: unqueriedLayers
> > +        };
> > +    },
> > +
> > +    getMessage: function() {
> > +        var tpl = new Ext.Template(this.queryResultMessage);
> > +        var key = 'CTRL';
> > +        if (Ext.isMac) {
> > +            key = 'META';
>
> ⌘ is not defined in all font, any other suggestion ? also I dont
> understand what you mean by "This also must be taken into account in the
> control. This doesn't seem to be the case currently." can you explain ?
> … <#13bad6989fe71ffa_>
>  In core/src/script/CGXP/plugins/GetFeature.js: > + }, this); > + } > + },
> this); > + > + return { > + internalLayers: internalLayers, > +
> externalLayers: externalLayers, > + unqueriedLayers: unqueriedLayers > + };
> > + }, > + > + getMessage: function() { > + var tpl = new
> Ext.Template(this.queryResultMessage); > + var key = 'CTRL'; > + if
> (Ext.isMac) { > + key = 'META'; META is not understandable for many people.
> Please use "⌘" instead. This also must be taken into account in the
> control. This doesn't seem to be the case currently. — Reply to this email
> directly or view it on GitHub <
> https://github.com/camptocamp/cgxp/pull/308/files#r2446004>.
>  -- Camptocamp SA Oliver Christen PSE A CH-1015 Lausanne
> www.camptocamp.com +41 21 619 10 23 (direct) +41 21 619 10 10 (centrale) +41
> 21 619 10 00 (fax)
>
> —
>
> Reply to this email directly or view it on GitHub<https://github.com/camptocamp/cgxp/pull/308/files#r2446504>.
>
>
>


-- 
Pierre GIRAUD
Géomaticien, Analyste

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac, Cedex

Tel : 00 33 4 79 44 44 93
Mail : pierre.giraud@camptocamp.com
http://www.camptocamp.com
You mean unflip.
You can also enforce the ClassMetadata in $class.
I'm merging, but create another one adding this.
Slightly faster, and a PHP bug with the ``getName()`` call.
This needs a function to adapt the traitsui font to an Enaml font which is expected by the backends.
Might well be :)  Have extracted it out into the stylesheet.
How about `setdiff(names(op.devtools), names(op))`
Nope, this merge didn't quite go right.
This reminds me: I've also got a similar function in interactiveshell, `find_user_code`. It might be possible to combine them:

https://github.com/ipython/ipython/blob/master/IPython/core/interactiveshell.py#L2738
Yes, that would make sense. Although I've just been through and checked,
and it seems the places where we use NamedTemporaryFile don't actually need
to re-open the file. Perhaps it's not as useless as I thought ;-).
credential yes, but not identity and private key. added checks accordingly
I was thinking about using `HTML::entities()` would be fine. What's the difference?
>> @@ -486,9 +486,14 @@ int git_path_cmp(
>>  /* Taken from git.git */
>>  GIT_INLINE(int) is_dot_or_dotdot(const char *name)
>>  {
>> +#ifdef __amigaos4__
>> +/* This is irrelevant on AmigaOS */
>
> Even if this is irrelevant, is there a need to #ifdef it out or can we
> just leave it? I don't think git is going to let you add a file named
> "." or ".." so having those files show up in `direach()` or `dirload()`
> call is just going to cause issues.

I meant to remove that #ifdef... absolutely, it doesn't need to be there.
应该这样写:
if (status) { // 需要先判断 status 的是否正确获取到了
    s_free(buddy->status); // 调用 s_free() 的时候不用判断NULL
    buddy->status = s_strdup(status);
}
Yes for ``import views``, no for ``from views import *`` :)
That's what my comment was about, to pull that code out into a decorator or something.

I don't think you're supposed to handle form stuff in a middleware, since it requires the template context and such. Hm, I'll chat with Winston and see if he's cool if we just redirect the user to the /newsletter page or something.
Yes, that's solely to get the wiki syntax.
This looks weird. aaq_step3 and aaq_step4 call aaq the same way. Is that right?
(nit) The """ should be on the next line.
The topic icons are uploaded and managed through the admin panel which is why we have `topic.image_url`. I assume you are setting them as CSS backgrounds (I haven't looked further down), but that won't work with allowing the admins to manage images in themselves.
This looks strangely familar to my hacks on the middlelware_app where I tried to make the old external classes work like there was a `self.config`. I can't remember the exact details any more.
Oh! ContainerAware fixes all my problems dd16a91b6e8ffbddef5443207618e459daf91030 .
:) краще silver
The idea would be to issue a `fread($handle, 50000)` and loop if not at eof - you should really be careful with the filesize cache, got trapped a few times !
this is wrong IMO. If you only use the HttpKernel instead of the Kernel, you don't need the bundles (see Silex for instance)
Please inline those 2 functions.
too much indentation. It should be 4 more spaces, not 22
Isnt this wrong? Command could possibly be called from a webserver instead of the command line.
This looks weird as two replacement patterns  (first and third) are the same ```'\\"'```.

(I was curious about this pull as PoFileDumper uses (undocumented)
```php
    private function escape($str)
    {
        return addcslashes($str, "\0..\37\42\134");
    }
```
)
and the problem space seems similar right?
first commit は1つ前のコミットメッセージだよね。

Gemfile の更新は最近個人的には同時でもいい気はしてるんだけど単なる手抜きかもしれぬ。
Yep. This definitely is better.
I'm actually not sure what the point of the support for a callback is.  What is the use case here?  It seems to me that one could just call the callback outright, instead of passing to logly to execute.
just switch this to check the button state and drop the property.
As long as the button state is being set correctly this should be fine.
Should this be `build_stubbed_list`?
Yes, that done.
srs => SRS
Missing Visibility keyword.
Yes. I meant something like opening get_dispatch_list.
If so, you could say addrs = sorted(a.ip for a in self._addresses if a.version == 4)
Instead of doing it this way, what I meant was something like:

    if (module != null) unloadModuleLocal(module);
This should be done as part of Framework.init(). The above is equivalent to Framework.start() and should not be done as part of FrameworkFactory.newFramework()
>> Most EAP4/5 customers have jar for the valve, asking them to repackage the jar in a module won't be nice.

Jar files on filesystem won't work in domain mode.
Well, maybe I'm not fully doing things right :)
I simply need to wait until service svc is active. My understanding was that you could do this with an SVN like I'm doing here, however when I use the default settleTime (100ms) it pretty much always times out with the message: 
  "JBAS014750: Operation handler failed to complete"
Is there another way I can increase the timeout? Or should I be doing things differently?
It's part of my last commit.
I think that we really want to wait on the container state monitor here, or at the very least give this a much larger timeout. Although this will always be very quick for small deployments, there are some things that can slow un-deployment down considerably (e.g. a slow @PreDestroy method, as even if the deployment does not complete singleton EJB's can still be created).
Sorry .... I also can't run the patch against my MySQL.  Error:


ERROR 1064 (42000) at line 7: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'DELIMITER//
CREATE FUNCTION `get_event_rating`(event_id INT) RETURNS int(11)
  R' at line 1
Good question. It was a anchor since the filaments group first selectmenu script.

I assume changing this will lead to different behavior in screenreaders.
I don't remember the exact reason we used `&#160;` but I seem to remember there being a reason. I can say that it's more portable than `&nbsp;` as numeric values will work without additional definitions for named entities.
Definitely, let's call `UssdManager.init()` directly at the end of the file and load ussd.js before dialer.js, both are defered and the message handler won't be triggered before dialer.js is loaded.
It will be catched by the <tt>try ... catch</tt> in lines 128 ... 135 triggering the red flash of the display.
I'm thinking I'll write a simple indexeddb utility that defines an API very similar to localStorage, except async.  If I do that, then you could use that utility here.  See https://github.com/mozilla-b2g/gaia/issues/4076
I find this hard to read. Seems inconsistent /w line [265](https://github.com/mozilla-b2g/gaia/pull/1597/files#L6R264)
Can't it just be created in 'Started' so you don't have to call Start everywhere you create one? Also not super clear what 'Start' means. 'StartDisplayingProgress'?
I don't think this instruction is useful. The point is that installing git-core works sometimes when installing git doesn't. What error is ok?
We don't want this expectation (or the others like it above) because we don't want to say that the ordering is alphabetical; just that it is consistent.
I don't get what you're clarifying here. `<C-X>H` does general HTML completion, not HTML tag closing. And the addendum to `<C-X>/` makes no sense.
Add in your e-mail address.
What are the units on this? Milliseconds? Seconds?
This block can be simplified by using the `?` operator: `return (condition ? 1 : 2) + bla`
This is occi specific. it should be flavour independent, so either DefaultSumitPools has both (Amazon and Occi) or it has none.
`fn()` is fine
I think ``cache`` should default to ``False`` for consistency with ``get_readable_fileobj``
error message should also include a reference to the codebases.
Should be `if (` (missing a space)
good call. :thumbsup:
REDIS_DEF_PASSWD should come after REDIS_DEF_HOST.
Thanks guys. I've updated it with "install" rather than "update". Not sure how to change to the `which` check, as my bash scripting isn't that good :-) . Thanks for the code review!
apologize if I was a little too abrupt, what i meant is the purpose of this PR is to fix weird issue on Windows for ppl running php from `Git Bash` or `Cygwin`, what I think is that we should instead try to fix this weird issue for everyone using PHP on Windows instead and this is only my opinion
Agreed, just following the quick&fast dark way :) I'll create getters/setters. 
Basically, I may need to replace one or more of them
Why are there two entries for webhook?
Are the schema.org features worthy of being in a separate test sourcefile?
Note to self: Here `TrailWithPackage` is unnecessary and being removed. Not related to model refactoring.
goto error on it's own line
Use `target="_new"` for external help links, since the Back button commonly messes up Jenkins configuration screens.
I think you have a typo here. The in the other files the count() is in the second condition and not wrapping the whole thing.
ACK re: moving the requires into register.

re: stub vs rufus,
afaict stub::interval doesn't provide the nice thread management that rufus does, so it would be very difficult for me to replace rufus at this point due to my lack of ruby thread knowledge.
Actually, you can also use $(function() {...}); as a synonym. Lots of forms arrive at the same place.
I use TextMate.  Vim and Sublime Text  are also popular.  I've also used RubyMine, which was pretty good.  You have to pay for RubyMine and TextMate.
I'm a little curious what you think about this pattern, dropping in a new "page" attribute and using the casper object as the mechanism to order test actions.
Didn't you meant `returned promise` instead of `returned function`?
As-is this imports the NetUtil object possibly polluting the global scope too. Should pass {} as the second argument to avoid that
This import is no longer necessary please remove.
Oh I see, even once #2455 is merged, the compiled function will still contain a call to `partial`.
@lloyd - both password_reset_status and user_creation_status return "mustAuth", they are both polling functions as well, which is why I am wondering here.
I have renamed almost all of the variables in BorisRelativistic and
deleted one square root. Thanks!

I have one question: if I click on "Merge pull request" at the end of
the page (green button), will then the new commits that have been done
after the pull request be merged with the ones from before the pull
request?

2012/5/19 aipp <reply@reply.github.com>:
>> @@ -0,0 +1,19 @@
>> +package org.openpixi.pixi.physics.solver.relativistic;
>> +
>> +import org.openpixi.pixi.physics.ConstantsSI;
>> +
>> +public class SolverRelativistic {
>> +
>> +     protected double c;
>> +
>> +     public SolverRelativistic(double c) {
>> +             this.c = c;
>> +     }
>> +
>> +     /**gamma = sqrt[1 + (u / c)^2]*/
>
> I think there was confusion, because you called your variable "v". I would suggest to call all helper variables "u" instead of "v" if they mean "u".
> For example, also in Boris, use
> double ux = getVx()
> instead of
> double vx = getVx()
>
> Also, in calculateGamma(), first you calculate the square root, then you calculate the square of u (I'm using "u" here instead of "v"):
> double u = Math.sqrt(ux * ux + uy * uy);
> return Math.sqrt(1 + (u / c) * (u / c));
>
> You need one square root less if you write directly
> return Math.sqrt(1 + (ux * ux + uy * uy) / (c * c));
>
> The comment of calculateGamma could explain more, for example:
> "gamma = sqrt[1 + (u / c)^2]. This expression is obtained by solving gamma = 1/sqrt(1-(v/c)^2) with v = u/gamma for gamma."
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/openpixi/openpixi/pull/19/files#r849643
I prefer 'greater than 10.7' because I'm HOPING Apple doesn't change the Password format again, and that > 10.7 will actually be true :)
.exempt makes you exempt from being banned
.exempt.override allows you to override that exemption when banning other players. This perms check should be completely removed.
There is a typo here: s/pkgnam/pkgname/
Yup, everything Mocha-related gets moved to `_mocha`. So this fixes broken code like

```js
beforeEach(function () {
    this.timeout = setTimeout(doStuff, 1000);
});
```

which is just one of many instances where mocha has reserved common words for its own use and gets confused when you try to use them yourself.
We should speak to @weierophinney about that, but following the new conventions of ZF 2, I think those validators should be moved to Zend\File namespace (for instance, all the i18n view helpers were moved to I18n namespace, same for Form view helpers).
Same question as I have on the other PR -- it seems to me like this could lead to unexpected side effects. Can you detail how you would deal with those situations?
The things we do for lack of complex types :)
Geht das nicht auch über annotations?!?
I'd have this call get_simple_player_stats to get all the stats that method handles (kills, assists, etc).  No duped code that way.
I'll make a note of that and report it upstream.  Essentially, in their Makefiles, they completely ignore `CPPFLAGS` and everything is in `CFLAGS`
Hmmm, I think maybe this could check virsh.VIRSH_EXEC instead.  Advantage to instantiating however is it allows class to alter logic w/o affecting unittests.
You should indeed use applier here.
Extra space before semicolon.
I find this new form harder to read. Maybe we should make another paragraph? @justinmayer is the documentation chief. Any idea?
Docstring, describe types and such of arguments?
drop space in "num= -1" ?
Testcase provided by user failed with ictce:

```
Traceback (most recent call last):
  File "<stdin>", line 173, in <module>
  File "/apps/gent/gengar/harpertown/software/DOLFIN/1.0.0-ictce-4.0.6-Python-2.7.3/lib/python2.7/site-packages/dolfin/fem/solving.py", line 254, in solve
    return cpp.solve(*args)
  File "/apps/gent/gengar/harpertown/software/DOLFIN/1.0.0-ictce-4.0.6-Python-2.7.3/lib/python2.7/site-packages/dolfin/cpp.py", line 20142, in solve
    return _cpp.solve(*args)
RuntimeError: 

*** -------------------------------------------------------------------------
*** DOLFIN encountered an error. If you are not able to resolve this issue
*** using the information listed below, you can ask for help at
***
***     https://answers.launchpad.net/dolfin
***
*** Remember to include the error message listed below and, if possible,
*** include a *minimal* running example to reproduce the error.
***
*** -------------------------------------------------------------------------
*** Error:  Unable to factorize matrix with UMFPACK LU solver (symbolic).
*** Reason: UMFPACK has not been installed.
*** Where:  This error was encountered inside UmfpackLUSolver.cpp.
*** -------------------------------------------------------------------------
```
we still want a toString() in this class
Yes. The alternative would be to write new python everytime we want to change the HMAC_KEYs which seems lame.
Why do we need this?
https://docs.djangoproject.com/en/dev/topics/http/views/#the-403-http-forbidden-view
`request.user` should always exist, and if it's an instance of `AnonymousUser` then `is_authenticated()` always returns False. Food for thought.
I don't understand what this variable is used for. Could you explain with an example of code would work incorrectly without `loopedPlay`?
Please remove the end of line white space  here
Since this has nothing to do with the re-ordering of start and end arrays, it shouldn't be moved.
`return args;`
Transform Data Processes will use a TransformAlgorithm (TransformFunction) to actually perform the data transformation. Like this.
You don't have the easiest coding style, but I've done my best :)
does it make sense to move it into a common helper method?

also, not 100% sure, but does it make sense to move it to the model (as both User.current and Settings are avail)
Handy tip, thanks
no reason, just left-over from old code. removed.


On Mon, Dec 10, 2012 at 9:23 PM, Aurimas Vinckevicius <
notifications@github.com> wrote:

> In OECD.js:
>
> > +	var articles = new Array();
> > +	if(detectWeb(doc, url) == "multiple") {
> > +		var items = {};
> > +		var titles = doc.evaluate('//td[@class="expand metadata"]/strong/a', doc, null, XPathResult.ANY_TYPE, null);
> > +		var title;
> > +		while (title = titles.iterateNext()) {
> > +			items[title.href] = title.textContent;
> > +		}
> > +		Zotero.selectItems(items, function (items) {
> > +			if (!items) {
> > +				return true;
> > +			}
> > +			for (var i in items) {
> > +				articles.push(i);
> > +			}
> > +			Zotero.Utilities.processDocuments(articles, scrape, function () {});
>
> any reason for the no-op done handler?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/zotero/translators/pull/467/files#r2373997>.
>
>



-- 
Sebastian Karcher
Ph.D. Candidate
Department of Political Science
Northwestern University
Actually, I'm not really sure about how base works and should be used. I suggest you get some comments from Espen on this particular point.
Apparently random Level import that was not the right one.
About the alignment, I'm unsure... it is just personal taste, I experimented and left it to see if it would generate any comment.

I transformed the list to code to leave the emphasis only to the 3 questions and because it fitted better with the other sections.

What do you suggest... to revert to the original?
This used the non-cached version before, might want to test if the cached version is good enough.
This method is a bit unclear and needs documentation.
Comment below logic, again
It's hard to understand what's going on because this chain spans hundreds of lines. I'd like to see the chain on a single line:

```coffeescript
vows.describe('Gatherer API').addBatch(cards).addBatch(sets).export(module)
```
I put this in to give a cleaner response for bad padnames, it's not 100% required, it would just make the padnames like `/p/asdf/ghjk` return a `Such a padname is forbidden` instead of a `Cannot GET /p/asdf/ghjk` - Just a little nicer looking.
говорят в while можно условие писать.
The DiagInfo pointer is passed in only to set this one integer value. It would be simpler to return an integer value here.
I still think this should iterate over pages, not paths.
Also, if the recursive approach is not correctly handling multiple levels of delegation, that indicates a missing testcase
This is the only difference.... So I don't think we should add another file that does the same excpet that it removes a rotation of -90 deg around the x-axis....
Please review this sentence and make the necessary changes.
Should be documented
I added a field for the appid.
Ah ah ah :)
Should keep the spacing here. `($token, $_GET['sid'], TRUE);` is correct, not `($token,$_GET['sid'],TRUE);`
ok so apparently not :-)
Same thing applies here and to the rest of the changes (except the `sinatra` one, of course).
Again, copied from YahooProvider.
That line used in 4 places just in this patch, with difference in only one word - mech name.
Add spaces around the assignment operator and the method name needs to be fixed as well. :)
AND -> &&
Seems to be missing a word in the first sentence.
Выбирать нужно случайные n ячеек, а не последовательно первые n
Было бы не плохо вынести размер массива DNA в константу.
Это какая IDE такой файлик рисует?
No, this directory can currently be browsed. But it might be a good idea to conceal these files somewhere there, where most users will not be tempted to look...
Can we also take the same approach I proposed for initializer (isolate in a private method)?
This turns this method into a creator of definitions, compatible with SOLID principles by having a single responsibility. =)
This was to bring it up to parity with `IScheduler` in Rx land. I can see how it'd be useful, though I haven't encountered a situation yet. Why would it be dangerous?
I've used Transmission before, but had no idea there was an HTTP interface. Could you include a Notes section with how to find the info to put in here?
Are these whitespace changes? If so, what did you convert from and to?
I will expose this as a new property. 

I was not aware of local tracking tracking branches ... I covered this behavior in a test, In this case, there is no "Remote" for the local machine that I could see, so this would return "null"... Maybe we should add a property indicating that this is a "local tracking" branch?
ok, I rewrote it to the extent that this makes sense. Part of the code is dictated by the urllib API, so it cannot be expressed differently.
good :)

On 21.10.2012, at 19:54, Alexandre Gramfort <notifications@github.com> wrote:

> In mne/fiff/raw.py:
> 
> >          indices = []
> >          for time in args:
> >              ind = int(time * self.info['sfreq'])
> >              indices.append(ind)
> >          return indices
> >  
> > +    def index_to_time(self, *args):
> Exactly !
> —
> Reply to this email directly or view it on GitHub.
>
ok this is wrong.

there is an inconsistency here.

do you agree?
2012/10/28 Alexandre Gramfort <notifications@github.com>

> In mne/artifacts/ica.py:
>
> > @@ -230,6 +248,8 @@ def get_sources_epochs(self, epochs, sort_func=stats.skew):
> >          sort_func : function
> >              function used for sorting the sources. It should take an
> >              array and an axis argument.
> > +        collapse : boolean
> > +            If true, epochs and time slices will be collapsed.
>
> collapse is not standard numpy naming.
>

I know, it's just that it seems natural to me, also had it in the nitime
epochs exporter and it's shorter than concatenate.
But I'm not passionate about this, we can change this If you suggest to.

> Do you mean concatenate?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/165/files#r1960925>.
>
>
I don't really understand the TODO side of this comment. What are you referring to?
"up skilling", while I understand what that is, probably isn't a real term. :grinning: Maybe a better phrase?
Those routes are to test each reserved words are correctly escaped.
I could use a different route for each action. But the purpose here is to test the `routes_routing.scala` is generated correctly. You  wouldn't do this in a real project.

Generating a different function for each route would be okay, but will introduce a lot of noise. 
Still if this is better for you, i can introduce a different function for each route.
What is this spy for?
Use the same API doc syntax as I showed in the previous comment.
模块是一个过滤模块，它可以在回复正文前后添加内容。
Actually no, not in my tests, since we're setting the "key_name" (1st param). But I'll look into it.
Pas de prob
Merci Alex
À+

Sent from my iPhone

Stéphane Fréchette
mobile: +1 (819) 319-2935
e-mail: stephanefrechette@me.com
.tel: http://stephanefrechette.tel
GatineauOuverte.org - Initiative citoyenne, données ouvertes


On 2012-04-04, at 6:44 PM, Alexandre Potvin Latreille<reply@reply.github.com> wrote:

>> @@ -185,7 +185,7 @@ App = (function () {
>>             '<div class="info-window-content">',
>>                 '<b>', data.name, '</b><br>',
>>                 data.address, '<br>',
>> -                '<b>', localeData.labels.installations, '</b>',
>> +                '<b1>', localeData.labels.installations, '</b1>',
> 
> Je n'aurai pas le temps ce soir Steph, mais je devrais pouvoir faire ça demain soir! D'ici là, je vais ouvrir des issues sur GitHub et peut-être que quelqu'un le fera...
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/GatineauOuverte/parcs/pull/18/files#r648060
so each time a user clicks, the widget gets set back to 'none' mode -- how does a user switch it back to goal mode?
oh right, I it's ```sort_by(&:ranking)``` .... either way, that's active support magic that calls :ranking on each object in the array, then sorts by it.
Why this data[0] ?

Only 
$this->ormMetadata = array_keys($this->data);
its 3.7.0.0.a9
Touché, I guess I just figured that ng-transclude was not actually a
directive but a subset.
On Oct 31, 2012 7:46 PM, "Andy Joslin" <notifications@github.com> wrote:

> In modules/directives/input/input.js:
>
> > +
> > +          element.html(response);
> > +
> > +          var inputEl = element.find('[ng-transclude]');
> > +          angular.forEach(tAttrs, function (value, key) {
> > +            if (key.charAt(0) !== '$' && ['src','uiInput'].indexOf(key) === -1) {
> > +              inputEl.attr(snake_case(key, '-'), value);
> > +            }
> > +          });
> > +
> > +          //prepare validation messages
> > +          input.validation = angular.extend({}, uiConfig.input.validation, scope.$eval(tAttrs.validation));
> > +
> > +          //expose model to a field's template
> > +          scope.$input = input;
> > +          $compile(element.contents())(scope);
>
> I think $compile errors because of the ng-transcludes. It doesn't know
> what to do. You'll have to manually transclude the elements with
> ng-transclude attribute, then remove it before $compiling.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/angular-ui/angular-ui/pull/191/files#r2001035>.
>
>
The name may be close to a complete sentence, but I have no idea what this class does.  Javadoc?
disagree with how we store this. 

It would be stored as "cbetta" as that is the username "@cbetta" is a stylistic version to signify it is a twitter name which already know from the field name.
Duplicate of chart1 with no additional functionality. Removed it.
I put it as TRACE because it allows me to manage the cache configuration in a symmetrical fashion, which is easier to maintain.  Plus the code skips replicating to itself anyway.
What's the leading 'r' mean?
Agreed it's not very well documented, but this looks fine.  As a sanity check, I would run the `test_image` unit tests with the PGF backend and see that the results match the PDF backend.
Yes. Thanks Eric.
Someone with more knowledge of the mpl documentation setup should look at this carefully. Was there a reason why ```rc``` was not just set as an alias to ```matplotlib.rc``` in the first place?
Ah, of course. I am not with it today. I didn't mean to do that at all. Your solution was the one I actually meant to suggest.

Thanks for pointing out that critical error.
So far we've found that Gosched() can help improve performance by putting it before a channel select{} clause, but it's not generally necessary. I think we want to stay away from explicit yields unless we know there's a reason to use one.
A question based off an observation, I'm just curious -- what's the difference between using <code>self.apps.kill_all()</code> vs. <code>self.apps.kill(self.app)</code>. Is there a reason to use <code>kill_all()</code> in this situation?
Isn't this throw exceptions when we run the `syncdb` again?
Для чего?