labofsahil/github-event-dataset-2012 huggingface dataset:
It'd be better if the goToMainMenu method could be implemented only one time instead of once for every activity.
You don't think this would be useful to have as part of the workflow outputs?
These are the most important bits!
You'll need to move these methods elsewhere entirely. We're actively trying to remove pollution of methods on MRI kernel classes.
do we still need these filename checks?
Cool! I love Inno Setup :P
Drawing the grass tile on the bottom of each tile in the map is hardcoded atm. Should be a option when creating the map. (So it can easily be replaced with snow)
Yeah, i got that code from codename_B, and netbeans doesn't fix those when i auto-format :/
When I run this I have one failure left:

Caused by: org.opennaas.core.resources.protocol.ProtocolException: Could not find a session factory associated to the wonesys protocol
	at org.opennaas.core.protocols.sessionmanager.impl.ProtocolManager.getSessionFactory(ProtocolManager.java:136)
	at org.opennaas.core.protocols.sessionmanager.impl.ProtocolSessionManager.createProtocolSession(ProtocolSessionManager.java:135)

This is were a little knowledge about OpenNaaS is required. The protocol session manager has a plugin structure in which manticore bundles can implement protocols which are then discovered by the protocol session manager. What happens here is that you run the test before the bundle implementing the wonesys protocol had a chance to register itself with the protocol session manager.

What you need to do is to add a dependency so that you wait for that bundle to start. So which bundle is it? Well, that's a bit of detective work :-)

If you look at manticore/bundles/net.i2cat.luminis.protocols.wonesys/src/main/resources/OSGI-INF/blueprint/core.xml you see this fragment:

        <service ref="wonesysProtocolSessionFactory" interface="org.opennaas.core.resources.protocol.IProtocolSessionFactory">
                <service-properties>
                        <entry key="protocol" value="wonesys"/>
                        <entry key="protocol.version" value="1.0.0"/>
                </service-properties>
        </service>

This looks like the service the protocol session manager is looking for. So what you need to do is to inject a BlueprintContainer reference for that bundle. This will ensure that the test isn't executed until the bundle started:

@Inject 
@Filter("(osgi.blueprint.container.symbolicname=net.i2cat.luminis.protocols.wonesys)")
BlueprintContainer wonesysProtocolService;

The bundle symbolic name here happens to be the same as the directory name of the bundle. This is usually the case (although it doesn't have to be).

I believe that if you add this dependency the test will pass.
Please split this into `describe '#copy'` and `describe '#move'`. RSpec uses this to generate nicely formatted output, and I think "copy and move" will generate a bit odd output. Here's an example of what the output should be like:

```
ContextIO::Message#copy calls API method with copy options
```

Also, the only similarity I see is the `@message` and the `ArgumentError` check. So not too big a reason to merge them together.
nicely done! that's weird that it  requires both StopIteration and ValueError
what do you need cdefs for ? it breaks my build, besides from being a weird GNU-only header
caribrate?  Did you mean calibrate?
There are some advantages to returning `TRUE` rather than `FALSE`. The main one is that if the caller passed a non-`NULL` value for `error`, you get behaviour consistent with the expected behaviour for functions that use `GError`, i.e. if the function indicates failure via its return value, the error will always be set.

I don't think it makes any difference what we return here, but let's just assume that calling any `eda_config_*` function with a `NULL` configuration results in undefined behaviour. ;-)
Dirk, can you provide me with feedback on this?
Some thoughts
 - I used *Update* for 'C','U','D' operations, but I'm not sure if this covers the 'C','D' operations.
 - I will remove the *Sync BIREL* tomorrow, as I see there is already an interface to *Clearing* defined in your document.
 - *Check mandate* also seems unnecessary
она так и делает
Oh, ok. Then I guess this is fine. =D
In the acceptance tests we do the following when the directory exists:  `git remote rm origin && git remote add origin git://github.com/puppetlabs/puppet.git && git fetch origin && git clean -fdx && git checkout -f <ref>` Im just wondering if we could save ourselves some time, instead of cloning each time?
that loop must be something that was committed since my changes.  look at https://github.com/neonstalwart/knockout/commit/b6fe107601a993ef26970c01a3f58afb2455d80b#diff-29 for my original diff.  

it can be changed.  ...but i'm curious why you want it like that though.  there must be something about it that i'm missing for it to make sense to be declared like this.
Generally properties should not make SQL queries. Or at least that's what @cvan keeps telling me.
How can you do this? Please stop using `std::pair`, it makes the whole code unreadable with all the `first` and `second` of which nobody remembers what they mean after a few days. Also, why is a collection of two values called `AttributeInfoMap`? I find the resulting code confusing since I read stuff like `attributeInfoMap.second`, and find myself wondering how a map can possible have a `second` member.
http://cl.ly/BJRa/pr.png - Is it done using cross-validation? You can make a something like a contour plot, where x is NN, y is ngram, and color is accuracy estimated using 10x10-fold cross-validation.
This breaks the bundle page. : c
rename to ApiKeyPairs
We want to build both qemu and linux kernel at the same time. Won't using linux_kernel build helper prevent that. Because qemu needs to be built using gnu_autotools ? Also can you add a #commented line here which says git_repo_build_helper= linux_kernel. So that it kind of auto document how to get guest kernel built
I think this creates an infinite loop no?
ah yes, thats fine I'll do that - I guess thats pretty similar to the feedbacklink function, which I should've just replicated ;)
It shouldn't actually require this, but we can recommend it for a better user experience...
It would be clients of the lumbar lib (which the CLI is an instance of)
I can think of no way to split the editbar refactoring into separate pull requests without resulting in a broken application.

Exchanging the background images requires a change in the CSS and HTML structure (which is *really* bloated), which again requires to adjust some JS references etc.
The vast majority of changes (in number of lines changed) is in the CSS, which had lots of overspecifications, duplications, etc.

Can you elaborate on the guidelines I should follow to split the pull request?
It's in a PHPDoc block, and its a 3 year old commit.
add option for color of text
trinity means its too hacky and dont accept it
Retriman why you don't Update your core ?
I accept suggestions :)
Initially I thought something around 20 would be good enough to accommodate most queries, but really I don't know and adding 20 more will certainly make no significant difference in memory consumption, so I've picked a rather high.
Wanting to pick a number to avoid most string allocations, we could even pick 1000 but I don't want to waste memory unnecessarily.
don't compiled... please, always check compile before push.
a few too many whitespace changes thrown in here :s
Oh please. What else would you call a person who updates dependencies for 3 weeks without even checking in with core developers. Anyway, it is back in 2d79c76fef, I don't mind having the badge but if it always says "outdated", how much use there is to it.
laziness :)

On Sun, Mar 4, 2012 at 5:03 AM, Felipe Lessa
<reply@reply.github.com>
wrote:
> Why doesn't these data types work with MongoDB?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/yesodweb/persistent/commit/3549fe72dfb9b4931273b49da94627606f785e57#commitcomment-1040071
Ah, you're right!  I did some more tweaks based on that!  Commit: [f37b327](https://github.com/laravel/fluxbb-style/commit/f37b327)
better reformat the readme using markdown:

https://github.com/github/markup#readme
and directive templates
On Sat, Feb 25, 2012 at 10:44 AM, Keshav Kini
<reply@reply.github.com>
wrote:
> Huh. Maybe. I'm using 1.7.9.2. I assume:
>
> * that you have local branches called "github-master" and "master", and that they correspond to whatever they were on sagemath/sagelib when you forked it (or, apparently equivalently, to what they currently are on williamstein/sagelib). Do you have the following output, probably plus a few more lines?
>
> ```
> $ git branch -vv
>  github-master        ecaff3a More corrections to README.rst
>  lfun                 739bcf8 padic_lseries -- working on doctesting padic_elliptic_lseries_fast.pyx
>  master               f583b1b 5.0.beta4

Nope.  That must be the problem.  I have

deep:sage wstein$ git branch -vv
  github-master ecaff3a [origin/github-master] More corrections to README.rst
* lfun          739bcf8 padic_lseries -- working on doctesting
padic_elliptic_lseries_fast.pyx


> ```
>
> * that git isn't choking on the option coming after the argument (because it reads more logically the way I wrote it). Does the following work instead? If so, maybe git learned how to see options after arguments sometime between 1.7.4.4. and 1.7.9.2.
>
> ```
> $ git checkout lfun
> $ git rebase --onto master github-master

deep:sage wstein$ git rebase --onto master github-master
fatal: Needed a single revision

> ```
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/williamstein/sagelib/commit/f4104efd87cb421923f884f8ea5c571fa52d583b#commitcomment-1011154



-- 
William Stein
Professor of Mathematics
University of Washington
http://wstein.org
@Pimbai ctm
This fix solves an error during jqgrid installation.  I t seems though that there is an non closed parenthesis that may cause an error at run time.
You overwrote my change...
I think the oldsk00l sass syntax is being deprecated for scss (long live curly braces, apparently)
As I said in the commit comment, I don't like this either: not clear on a better way to do this. I cut & pasted this process from another part of bndtools itself (in the builder area?)

Open to suggestions....
I think we need to check if (rproc->ops->suspend)
I figured 403 made more semantic sense (since you're forbidden from accessing the resource) but 404 works for me, no worries.

Any thoughts on the back to where you came from versus displaying a 404 page?
You shouldn't need to put in a fixed amount of whitespace.  pre_text will normally include it if necessary.
Also, you might be a primitive inside a group which is also inside a clone... which will throw everything off anyway :-)

Btw. Can we s/group_resource/group/ ?  Likewise for clones.
Oh hey, dood.  No it was just the custom_ignore.  I didn't work toooo hard to figure it out since FuzzyFinder has been working awesome.

But, now that I know to put the configuration before it loads, I may give it another go someday.

Cheers!
**Top_Cat** *looks disappointed*
Space before {

Rather than comment on all of the formatting issues, please look at the style guide above.
looks good to me too
Well, I'll get some fresh data about this one in the following weeks and I'll fix this one too.
Did you find a compelling use case? =)
> First place I saw it was in all the MDN polyfills for array extras (example). It does feel a bit magical, and I'm not wild about that, but it saves me from having to implement the crazy conversion rules in the spec, so it's a dear friend of mine.

I first saw the use of `>>>` aroundish this:
http://www.mail-archive.com/google-caja-discuss@googlegroups.com/msg01606.html back in 09 when I was trying to make methods more spec compliant.
Thanks for testing this, pushing the fix in a few.

On Sun, Feb 19, 2012 at 11:21 AM, Tal Galili <
reply@reply.github.com
> wrote:

> ok,
> I now see that the moment I turned on the "sharing" plugin - the homepage
> got back up.
>
> Please check to see what else is needed so this is fully moved into a
> plugin.
>
> Cheers,
> Tal
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ushahidi/Ushahidi_Web/commit/42104b11e60785b695b24c1801e62d125f8981c1#commitcomment-982260
>



-- 

Too brief? Here's why! http://emailcharter.org
Regards
Linda Kamau
Coder!!
for node.js v-0.6.10 npm v1.1.0-3
In this commit, there is still a sync bug. Assume A and B both opened a page, B closes before A closes the page and after that the anchors are learnt. Then A closes the page. When this happens, A's record will not be correctly diffed/modeled. But this is not a huge issue.
Avoid conceptualizing resources as verbs. Instead of POSTing to `/add-film`, consider PUTting to `/{filmname_year}`, or POSTing to `/` (your root resource).
Apparently the EM code in 4.0.1 was changed to throw an exception if the name is not defined.
Just 1 nit in a couple places: what the indent level (2 instead of 4) - we should put some modelines in there.
Nah, ANOTHER retarded developer, from another plugin, is causing errors in this one! :/
plis fix zone Ala Escarcha, Sindragos not spawning :S
Why not add the delegate overload for the loggers?
wooooooot i got my name on the repo :D I am a developer now <3 haha
+1 Great work as always.
It works =) Thanks!

By the way, if the `getReflectionClass` is used in an other bundle, it will occur a problem as it is not returning a `ReflectionClass`. 
Is there a particular reason for returning a `null` in `StaticReflectionClass::getClass` function?
I fixed this now... can you update and check if it is now fixed?
Was there not a test for this? If not, can you add one?
How hard would it be to address the problems with the FullTermParser?
Hey Ray,

I think that we still want to create a link back to the site template even if "Propagation of changes" is disabled when the site is created. That is, it should still be possible to enable the propagation of changes at any later point of time.

I haven't had time to test this code, but based on the comment "This means merge without creating a link" it seems that won't be the case.
JC,

There is no rational other than this concoction works for us.  I don't know what to do to make this package work in the way you desire.

Hans
I am also a bit worried with the fact that our C is now so different from the libsvm and liblinear habits.... Maybe we should revert C to be unscaled and add an scaled alpha parameter which is None by zero: if None the default C value is used. If alpha is not None it would replace the C value for regularization and would behave as in SGDClassifier, ElasticNet and the other linear models.
Please add comments and document your functions, as it is not clear what this code is doing...
ok sorry
From Andy, thank you. Not sure what a pull request is at the moment, he, he! Figure it's a suggested amendment - to revise the source list.

I suppose I could revise my copy, but I'd rather do it properly. I'm guessing I can 'accept' the request and it will (somehow) synchronise our copies.. I'll have to look into it tomorrow. Cheers. Andy.
In `editorconfig.h` and `editorconfig`, we add a link to the format page (for both html and manpage). Do you think this is good enough?
Sure - I don't have any rejections.
Can't we catch multiple exceptions in one line? (I can do this in python, but maybe not in java...)
I was testing the behavior of PCamera scaling and forgot to remove this line after I was done. It destroys everything!!
It isn't necessary to add this.  The FreeRADIUS-Proxied-To attribute is a hack.  It was done for the "inner-tunnel" server.  It meant that you could use the *same* modules for outer/inner tunnel.

In retrospect, this was a mistake.  We should have had a "files" module, and an "inner files" module.

So there's no need for this here.
same here, PSStackedView uses ARC. Makes no sense.
you need to use auto_connect for it to trigger. while it's waiting to reconnect it will save all incoming queries in the commands list and replay them on connection.
@jeffmccune Note to move elevate into misc directory
There is a new examples folder what will be used for sample servers.
But don't you still need django-stackato for changepassword2 if you want to change the password of the admin user. I know there is a way to do it with manage.py but it seems like using changepassword2 is more convenient.
Redundant cast.
I'd like to lift all of this shared caching layer into an intermediate class between all of ours and SQLiteOpenHelper.
@everzet it depends on the dependency and how its exposed through framework-bundle. but the fact of the matter is .. if we use some class directly we better list it has a dependency.

now where it gets sort of nasty is with type hints, because then you quickly basically have to duplicate the entire dependency list of your dependencies. but i think its still the right thing to do.
There's definitely a lot that can be factored out, but some of the JS is dynamically generated.  That's the reason for this block here.  It allows us to put Javascript in the HEAD that needs to occur after the normal files are included.
Eclipse is wrong :)
I accidentally lost it while doing a rebase. pthulin should be pulling it back in soon, but you can find the original on my branch.. https://github.com/flattr/osd-flattr-chrome-extension/pull/1#issuecomment-3687164
FYI, this has been fixed upstream in https://github.com/shanx/django-maintenancemode.
This works on most qualcomm LGE devices, but not all. The only purpose is to correctly set the baseband string.
Is this just a problem with the test in 1.6, or is it possible that adding patient addresses will fail in 1.6?
Thanks for the idea for the initialize. Kostache/mustache seems very
inefficient when it calls the methods many times just because the method
name is in the template multiple times.

Thanks a bunch

from my android
On Feb 12, 2012 10:49 PM, "Isaiah DeRose-Wilson" <
reply@reply.github.com>
wrote:

> Yes indeed I did. It should still be calling $this->_initialize(); Thank
> you for pointing that out.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/synapsestudios/kohana-projecttemplate/commit/a9a8c8180232a4009d85e5f93a100d5219d26431#commitcomment-958849
>
Note -- I relaxed the dependencies here to work on older GHC's.  At least as far back as 6.12.3.
Key length MUST be 0 in set, add, replace
Makes sense. Will remove it.
Creating an ELB in this region should give you the hosted zone ID.
I agree - I don't think they are used either - let's rename to what we feel is sensible. If it breaks, at least we will understand the code better. We will need to test the whole thing before merging into our main (develop) branch anyway.
Is this documenting a bug or something else?
This is going to come up for group forum replies as well. We need to figure out a better way.
You're right : I will do that.
I believe if you do `del self._lib_handle` you don't need the `if`.
I agree. The old description "Python interface to OP2" wasn't hitting it either I think. What's a good tagline then?
hmmm, don't know. what is the cluster-instance-replacement adding?
parens are not needed here
Probably not very complete, but the start of a set of base views for Json rendering.
dislike!!
this is very important code but hard to read.  we should clean and comment.
I just used the existing flag from the timestamps as a stand in, so we don't have the second flag.
from __future__ import barry_as_FLUFL

In python three will make the <> work in python 3...
Sorry, I'm being rather counter-productive here. On a serious note, wouldn't the following be more pythonic?:
 
    if orientation_dep and orientation_dep != gesture.orientation_dep:
This would likely be better living in config/environment.py and altering the config itself
Do we want to move this to being part of the notifier config either instead or in addition to (allowing it to be overridden)? They probably won't care to change it on a per-event listener basis, but it feels a bit better having it all located in the event listener area. But I'm also not 100% behind that suggestion, so it's your call.
can't some of these things be constants? at the very least _('N/A') is used more than once
Don't recall at this point. That's what I had from before. I suppose it was just in case trailing white space ended up in the path --it could mess up the route matching.
This is the first basic version. I am still working on it at this time and there is less and less duplicate code.
oh wasn't talking about the method but about the const change. but if the method is no longer needed, then that's even better.
looks like this file is a leftover (as mentioned on IRC)
yeah you're totally right - sorry
Sure - will reverse.
I think you mean `UNIT2=`, not `PYTHON=`.
if not find:
Setting the attribute without setting a key caused problems with `get` (because it is key based).  
To reproduce problem:
```
from pyon.util.containers import DictModifier, DotDict
my_mods = DotDict( test='val')
config = DictModifier(cfg, my_mods)
print config.get('test')
```
First shift the between data and the whole match string off the front.  The indexing and ranges are all based on the match array with these removed.
it might be better (i.e. more readable, possibly also JVM-wise) to do

    try X()
    catch { case NonFatal(_) => throw reason }
    throw reason

although it repeats a statement
Sure… but import it in the .m not the .h.
btw, length is uint32_t here... Not sure why you need possibly larger type?
Is there any other way to detect VertX or alternately to patch process.nextTick or the proposed standard setImmediate instead? This line will interfere with static dependency analysis for asynchronous CommonJS module loaders.
I should have mentioned this earlier -- I don't know why, but I get serialization exceptions if I uncomment these assertions.  This is outside the calls to spark, so I am not sure why its getting serialized.  Can you suggest a fix here?  It would be nice if the unit test actually asserted something ...
Don't "oh hell" on me and past a snippet how to do it better.
Right... well it doesn't matter. I have just written it as my own UserStorage anyway.
Should probably use PyArray_TYPE here and in the next case.
Я не разбирался, не могу сказать. Помню, что вылетала ошибка на летней школе, но когда, после какого апдейта - нет. Спрошу Машу - возможно, она мне говорила закомментить.
@Tho85: I am absolutely with you. Why else would you call ``becomes!`` if not using STI?
@al2o3cr gotcha - edification much appreciated. @lexmag indeed - nice work.
Why is that required? Also notice that `keys.flatten` is also called in the following line.
Changed from "No Group" to "Ungrouped".  Does that cause any issues, or just a UI thing?
I've been doing the same thing.  Soon as we a) enforce no trailing whitespace is checked in and b) do one more remove-all-trailing-whitespace commit then the invisible line dancing will come to a close.
Why isn't this just timeLeft < Duration.Zero, which is  the specification? Writing it this way introduces some potential for bugs, now or in the future.
The only difference I see is a few object allocations less. Are there platforms where `nanoTime` is fast enough for this to be relevant (making this a non-system-call has been a goal of optimizations in Windows and Linux, I don't remember how successful)?
Of course the result should be the same, but given the possibility of overflow errors being present or introduced by future changes, there should be (outside the ScalaDoc) a comment which explains why this is equivalent, together with (stress?)-tests checking it. After a quick look, I'd propose:
"This is equivalent to timeLeft < Duration.Zero. Since toNanos is guaranteed to never overflow, and Deadline.time is guaranteed to be a positive Duration, there is no risk of overflow and the results are equivalent."
(Note: I didn't check whether there's any overflow risk for `timeLeft < Duration.Zero`, but in the end that code must work since it just uses the public interface as intended).
I'm not sure what this is doing, but isn't it going to be misled by

  (x)(y, z)  vs.   (x, y)(z)

Those flatten to the same list, and have the same number of sublists, but do not correspond.  Maybe this is handled somewhere else.
Seems like positions for those errors are off.
i think we're in agreement?  If a developer uses custom authentication, then they'll need to provide their own auth scheme...

the IF switch on line 5 actually prevents :email, :password, and :password_confirmation from being attributes of the Spree::User class unless custom authentication is turned off (in which case, SPREE_USER_CLASS will be "Spree::User").  If SPREE_USER_CLASS is set to something else, say, "Person", then we can't assume that :email, :password, and :password_confirmation will be attr's...  (which are default attr's for Spree Devise)
I made that change at put git on my path but it doesn't work:

```
sympy>\Python32\python.exe bin\use2to3
git was not found: make sure it is in your PATH environment variable. Exiting...


sympy>path
PATH=C:\Program Files\Common Files\Microsoft Shared\Windows Live;C:\Windows\syst
em32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v
1.0\;C:\Program Files\Windows Live\Shared;C:\Python26;C:\Users\leslie\AppData\Ro
aming\npm;C:\Program Files\nodejs\;C:\Program Files\QuickTime\QTSystem\;"C:\Prog
ram Files\Git\bin"

sympy>dir "\Program Files"\git\bin\git*
 Volume in drive C has no label.
 Volume Serial Number is 8097-7CB2

 Directory of C:\Program Files\git\bin

10/02/2010  11:07 AM         1,166,848 git.exe
10/02/2010  11:06 AM           322,643 gitk
               2 File(s)      1,489,491 bytes
```
Perhaps this should be renamed diagonal to avoid name confustion with the diag function (which could conceivably be a classmethod)...or perhaps the diag function should be Diag (to indicate that you are creating an object).

I've got to convince myself that this will work when the matrices are of different sizes.
dict comprehension!
Though your new method is working, could you please explain the working of getattr(a,'_ops',0)?
One issue with warnings is that they're only displayed once. So what this really does is warn about the first plot that has complex values, and no others.

As far as I'm concerned, this is the expected behavior of plot(), and is no cause for alarm. Virtually every plotting software will leave the negative half-plane empty when plotting sqrt(x). If you really think its important or useful, perhaps it could be added as an optional keyword argument (I would still consider using something other than warnings, for the reason noted above and the fact that they are a pain to test). We might also investigate visual ways of showing the user that complex numbers were discarded as well (I can't think of any good ones off hand, but perhaps there's a nice way to do it).
done.


On Fri, Nov 16, 2012 at 2:00 PM, Ronan Lamy <notifications@github.com>wrote:

> In sympy/unify/usympy.py:
>
> > @@ -0,0 +1,109 @@
> > +from sympy import Basic, Expr, Tuple, Add, Mul, Pow, FiniteSet
> > +from sympy import Wild as ExprWild
> > +
> > +from core import Compound, Variable
>
> Don't use relative imports, please.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/sympy/sympy/pull/1633/files#r2156211>.
>
>
Does the algorithm have a set guaranteed order?  If not, maybe it would be better to just test sets here.
In that event, I will create a `default.latex.css` theme which has the `gray` background.
I think maybe this actually *should* be `==` in this context, looking at the code
Oh, that's a great call. This is perfect.
Thanks, this should be resolved now.
Pretty much. This is not final word on s/domain/space/ but I needed to keep the patch as small as possible.
This is not production code.
every member of the shard cluster.

The connection pool are just threads on the mongos, that represent open connections to mongod instances.
the density of an urban neighborhood doesn't describe its shape.
La raison pour modifier la classe .row etait pour ajuster les marges.
Bootstrap ont voulu obliger les gens dutiliser Un row suivi d'un span. Le row a une marge -30px et le span une marge+30px. Ensemble elles s'annulent.

J'ai donc enleve ma modification et inclu la combinaison row+span mais ca change les proportions de plein de trucs. Ca complique les marges aussi.
Up until now event names have been nouns, particularly garbage_collection one caught my attention as being very much a noun. This event uses a verb (inject).
Why creating a new instance here, rather then using the service?
You will have to amend the commit that makes this changes to not make them.
Oh, I thought tests already support it. In that case it's fine.
And to clarify, after looking at the code, $row contains the array of data about a given topic. $topic_row contains the template variables filled with data from $row.

I'm not sure if we actually need both, especially since $row is not used at all after the event call.
STD and LTD do not make much sense, also I'd only add one for this
As we have READONLY, you might use READOWN ?
This looks to be a per-row event, therefore it should have "row" or similar in the name, I would imagine.
Did this get fixed / changed in the upstream module?
fixed in 67baa66
`attachActionContextAddons()` is not a public API.  The method should be marked private.
http://nodemanual.org/latest/js_doc/RegExp.html indicates that neither `=` nor `%` are special, so I think the regexes should be `/=/g` and `/%2F/g` instead.
>Ты хранишь количество показов в переменной и записыаешь её в базу, соответственно, когда будет несколько нод, они просто будут перезаписывать значение друг друга.
эм... а откуда возьмется перезапись? апдейты же по очереди выполняются. причем в иннодб же строки по дефолту лочатся? и если запросы к разным строкам то друг другу они не мешают же?

>Да, по каждому промежутку
я имею виду делать 1 запись в таблице и обновлять, т.е. вместо создания 4х сотен строк в таблице за день, делать по 1 строке и обновлять в ней количество просмотров/кликов по интервалу
400 записей на баннер мало? т.е. если будет нужна статистика по рекламе за пару месяцев по десятку баннеров,  нам нужно будет сделать запрос к ~250к строкам? 

-Очень не хочется нагружать ноду логикой
а зачем нодам то их лопатить? я так понимаю, от них требуется только запись в базу просмотров за определенный период?
> Why would $DEVICES_TAR_GZ be empty?
You can make it empty, and then you'll get instructions about how to
recreate it. (Eg if my server goes down)

The problem is that you need to be root to create it, so you can't run
the lines within a build job unless using a VM which would be overkill.
you could write: `Arg.Set verbose`
first, please don't spend time working on CSS right now, since we probably won't be using it.
second, if you must have CSS, please move it to a CSS file in the following path:  /src/resources/css
once we add provider, location, drugpurchaseorder to the java, we need to add them here-- the providerID will become a property, then the provider will become many-to-one with regards to column provider_id
We should link to http://numpy.scipy.org/ . Are you intending to encourage the ```pip``` approach?
文档里这么多空行啊
DisconnectedType()()?
Couldn't this file just be removed?
The C code of GpuFromHostWait and HostFromGpuWait seam very similar. Could we merge them? This is not mandatory that we do this. There is the problem of the make_node
Does write permission imply permission to chmod or should there be error handling around this step?
A name like ``evaluate`` does not need to use the word "formula" twice.
Should we nuke the ID?  Doesn't seem of any value, does it?
I the name of this tool "bloom_scan"?
Для чего здесь `try {} catch {}`, если в `catch` происходит просто `throw` без какой-либо обработки?
The introspection API isn't part of the public API as far as I know. If it is, then skip the rest of this comment...

So, they can be changed without deprecation cycle. We should of course not change the API just for fun, but on the other hand I am very much -1 on treating any internal API like public APIs because somebody might (or is known to) use them. That is a slippery slope which leads to a situation where we can't change anything.
Aren't these names and genus lists unused - you add to them but never read from them?

BTW above in the tests.py add an empty line before the __unicode__ method.
At first sight, I don't understand why you replaced `repr` by `str`, but if you have a good reason go ahead.
Indeed perhaps - I'm still not super familiar with all the different link types in Django
Yeah, no need to distinguish between autoAssignEtc and a transaction by some other means, methinks.
AFAIK, django support user to provide custom form validation (per-field and globally) by specifying (for example) a `clean` method and  `clean_<field>` (for more see https://docs.djangoproject.com/en/dev/ref/forms/validation/). 

Is there a specific reason to override is_valid and implement an additional _add_error method? If yes, can you provide a more informative comment about why we are doing this hack?
Yeah, that's better. I didn't like that slicing business.
Since we'll be merging `sandbox-install` and `install` anyway I decided not to keep it.
The error is the same as in original code : `122: error: no match for ‘operator++’ in ‘++id’`. It doesn't like increment on enums.
I can't imagine this is worth the size increase of defining a method **twice** *and* including a hard-to-compress English phrase in the replaced version when a native "jQuery.getScriptSync is not defined" suffices just fine. And I agree that mangling prefilters is a bigger risk, which is why I switched the code to `dataType: "text"`.
Same comment regarding use as an ordered aggregate.
I thought this and igd were to be integrated into existing logistic.py_in.  Isn't it possible?
`Formula` has a `buildpath` shortcut for this now.
Thanks!

Stylistic nit, but I'd prefer to group fat arrow dependencies into their own groups rather than stick them between standard deps, e.g.

```ruby
depends_on 'cmake' => :build
depends_on 'gnutls' => :recommended
depends_on 'jpeg-turbo'
depends_on 'gettext'
depends_on :x11
```
`system "false"` is the example test that comes with the new formula template. Either replace it with an actual test or remove the `def test` block.
Got it.  Thanks.  Done.
In general, Homebrew formula use this syntax for manual installs:

```rb
prefix.install ['doc', 'contrib']

or

doc.install Dir['doc/*']
prefix.install Dir['contrib/*']
```
I'm not sure which is appropriate, as it's based on whats in those two directories.
If you check out the Formula Cookbook wiki, it lists the helper variables like `doc`
and where those are located.  Man pages go in `man1.install` or similar as you
may know.  Part of the reason for using `something.install` is that it creates any
directories for you akin to `mkdir -p`.

Can I ask what is in `contrib` and whether it's ok if those files are not symlinked into
anywhere the user might expect (like `/usr/local/share` or `/usr/local/etc` for example)?
We have `mktemp` for this; see e.g. the glib formula for an example.
Is the `glibtoolize` call compatible with the version which shipped with older versions of Xcode, or does this only work with the Homebrew glibtool?
This is a good direction - I want us to go further at some point - the current method of creating a new request based on links and payload is horribly repetitive.  All of those default arguments on each call should be moved here.
You don't need this line anymore
Though I wish we could use something cleaner than erb.  ERB's white space handling is really bad.  It doesn't even ignore control tags like if by default.  Since when do I want text indented the same number of spaces as my tag?  I think they do that because tags can have output.  Any better options you know of?
No need to bind `DBResult`, just put the call to equery in the case.
I'd like to but our `ApplicationWorker` doesn't seem to do anything with `FailureMessage`s
"""
Return comments as a tree.

Note that there may be instances of MoreComment objects. To easily remove those objects, use the all_comments property instead.
"""
PEP-8 suggests that tests for non-zero collection size should be ```if found_blklist:```. Scroll down to the "sequences" section.  http://www.python.org/dev/peps/pep-0008/#programming-recommendations
The following section that inserts the note in HTML should be refactored to a different method in the base class (e.g. ``_addNoteToHTML``):

```javascript
Notes.prototype._addNote = function () {
    var note;

    note = new Note();
    // bind event handlers
    this._bind(note);

    // add it to the internal notes list
    this._notes.push(note);

    // insert it in HTML
    this._addNoteToHTML(note);

    // save it in case others get added and this one isn't written
    this._save(note, this._notes.length - 1);
};

// no implementation for base class
Notes.prototype._addNoteToHTML = function (note) {};
```

Now, ``FloatNotes`` would implement its board specific code:

```javascript
FloatNotes.prototype._addNoteToHTML = function (note) {
    this._board.append(note.html());
};
```

And ``FlowLayoutNotes`` would implement its specific flow layout code:

```javascript
FlowLayoutNotes.prototype._addNoteToHTML = function (note) {
    this._layout.add(note.html().prop('outerHTML'), {
        index: this._layout.count() - 1
    });
};
```
s/the standard/standard/
Yuck, Fixed. I'm having a rough time with IntelliJ and swapping between OkHttp (which is 4-space like libcore) and the rest of my world which is 2-space.
Could we switch to http://localhost here, since 0.0.0.0 is technically a misuse of this local address (only permitted as a "source address", not as a target - for loopback you may also use 127.0.0.1) http://en.wikipedia.org/wiki/IPv4
I am going to add:
"Specify HTTP and FTP providers.
The Maven Wagon is a transport abstraction.
For more information see http://maven.apache.org/wagon/index.html
and the guide
http://maven.apache.org/plugins/maven-deploy-plugin/examples/deploy-ssh-external.html"
Thanks for the explanation, but what I see is that even after your change, in the case of methods with names FB and Ea, the return value would be MethodSorter.compareSignatures(left, right), which in fact returns the comparison of the sull signatures.
Before your change, the return value, in my understanding, would have been the same, i.e. MethodSorter.compare(m1.toString(), m2.toString()), which also was a comparison of the methods' full signatures.
What am I missing here ??
This is setting it up as a stub to go forward, since the url and poster will be supplied later.
The documentation states: `{string} oldName The full path of the old name`. It looks like at some point `oldName` and `newName` were only relative.

Changing this would prevent relative renaming if it was indeed intended that way.
Then how does is work on Windows?

You also need to remove the extra space before the first condition.
``brackets`` is unused. remove from jslint options.
(But I think that's actually pretty important, IMO, for trust in our editors.)
``config()`` should only be called once by the initial ``data-main`` module, brackets.js.
ok, merged.
though still think caching `this.getPos()` would be better
having such a thing in one place doesn't affect performance much, but small things do add up. (think of CO2:)
Can we please put a try/catch around this?
Cool trick. Didn't know about `Tempfile`.
Seems fair, just wondering.
doesn't this transform a double back into the same double ?
why "new MIME" ?
inflect?
So i don't need to set up default language choices?
Make a link to Poedit website.
I am confused what are is a problem here? What are you asking here? I do not understand your initial question and also not what are you saying here. Please again try to ask initial question.
Shouldn't this be a part of a special page? Or a header?
*This class defines document type for image posts.*
same deal.
The test title is incorrect here.
spacing needed after "div"
does this throw errors? if so, should it be catch and passed to `callback`?
PDFView.pdfDocument.numPages can be used instead if moved below (also avoids closure)
Yes, it ends up as HTTP 400 Bad Request response.
fyi https://github.com/sonatype/sisu-goodies/blob/master/common/src/main/java/org/sonatype/sisu/goodies/common/SimpleFormat.java
Formatting. See ZRevRangeByScore for the 'standard' multi argument over the limit formatting standards. Apply to other similar methods.
You can avoid using the HashSet with something like
```
  ids.flatMap {
    // return None if we don't want to include the id
    // return Some(id) if we do
  }.toSet
```
I don't think we have a choice in the matter.  Red Hat still owns the copyright up or downstream.  Leave it in.
it (redcarpet) isn't included in the repositories because it isn't listed in the comps file.

ACK from me too
:trollface
Routes are registered once per app hence why this is an edge case problem today, it has nothing to do with the container really. The user is either changing the route or using their own dependency resolver.
nit: If something's called a "wrapper" it can usually be thrown away. Would "frames" or something be a better name for this div?
Take the nav out of the small tag
Replace the sentence starting 'Most of ...' with 'More functionality is available as methods or attributes of the current cosmology object'.
Where does `enotregistered` come from? It might be better for code consistency to use `error` instead (maybe `{error, {enotregistered, App}}`), unless that atom is part of another interface this cache is mimicking.
it'd be nice to have a bit more explanation in this docstring
Extra spaces before `=>`, same issue on next line.
thus, you are indeed making it a reader but without implementing the interface, making the class unusable
I recommend adding user_email and group_name methods to membership.
If I'm not mistaken you add your plugin on "all" fields in backend.js, if you only require the JS file here it will be possible that somewhere else it won't work because the file isn't required.
If this will be part of the core this plugin should be automatically loaded.
Je mag hier rechtstreeks de container aanspreken. Tocheck.
It's not very meaningful unless the users knows what arguments there are, though, and I'm aiming for test_pr to know as little as possible about the specifics of what it's testing. I think I prefer to keep the output cleaner.

I think the important thing is the entry in the Github comment, so that if a PR touches anything that could affect IPython.parallel, reviewers can see that it's got a test run with `--all`.
What's the status of this todo, @jonathan-taylor?  Is this something you can/want to address before merge?
The general goal is to resolve the unnecessary coupling of kernels, notebooks, and cells.  Part of that is using the Kernel *alone* (no notebook, no cells) as a javascript library for remote execution.

I should note that I did a test, and it does seem to work if you call `Kernel.start()`without specifying the notebook-id.
this PoolNodeMetadata stuff is unnecessarily complicated. Also, we don't have a requirement to expose the backend, and in fact in some cases we may not want to.  For example, it will affect serialization of NodeMetadata and is regardless more code to maintain.

just transform to another NodeMetadata.  it can be as simple as NodeMetadataBuilder.fromNodeMetadata(input).group(frontEnd)....build()
If the reference is a direct reference, is it still okay to have it  not found?
icaaq, you can ping Laura Forrest (lforrest on IRC) for any product questions.
Actually, he's right. You're supposed to do all the "import" lines first, then the "from xyz" lines second.

http://mozweb.readthedocs.org/en/latest/coding.html#import-statements
God knows! Maybe someone else who's better at threaded programming can chip in. :)
Yeah, that would require rewriting it in C and writing a bunch of tests. Sounds like work. Maybe I'll email the libfov guy and he can backport it if he wants to.
Absolutely. I didn't even think to remove it.
I'll go back through when I make validator deletion analogous to how we do admin deletion (i.e., delete all you want, unless you only have one left).  I didn't change any behavior here right now, though.
Would BulkInterface be any better? Again, I'm really not that good at naming stuff.
typo: after beeing read
It's one of those PHP oddities -  a static method can be called with $this-> for instanciated objects without generating E_STRICT as opposed to calling a non-static method as static which does generate an E_STRICT.
you broke the indentation. It should stay 4 spaces, not 3
should use error() method to return a http 500 instead.
Style:

```ruby
invitee_attributes: {
  name_or_email: invalid_group_name,
  yammer_group_id: invalid_group_id
},
```

Same below.
non-avatar images are now stored in /path/to/bitbucket/<YEAR>/<MONTH>/
CAdjacency should be a refinement of CUndirectedSimpleLocalGraph
Is this an error in imagebuf.h:

reres (const ImageBuf &spec)

Should it not use ImageSpec instead? I see the implementation only needs
a spec.
nun sollten die sessions bei einem deployment erhalten bleiben und damit user nicht mehr ausgeloggt werden.
It does. I was tempted to remove it completely, but I'm not confident that it would take <25 minutes if I had to find out how to do it again :smile: 

I guess there is always the git history. I'd nuke it if you asked me to...
Missing visibility keyword.
can combine all of these in to row test style (single)
Thanks - good point, I didn't notice we were already using the other pattern. Makes sense that if we decide to change, we should update them all together.
For this, let's do
```python
from akanda import utils
```
And then, in the code itself:
```python
utils.ModelSerializer
```
Rule: prefer explicit language to ensure that syntax highlighting is correct. Don't let it guess.
this is nearly ready but i dont want to turn it on since im not sure what datum will do whenthey ave consultants intheri datum states... this should get added to the datum state + consultant issue
The new restart() method does not have an equivalent semantic to the old "abort(null)" method. The old method would produce an exit code of 1; the new one will produce an exit code of 10.
Done, added them into one statement on line 17.
On 09/07/2012 17:32, albertopq wrote:
>>       customTag = document.getElementById('custom-tag');
>>   
>> +    deleteContactButton.onclick = function deleteClicked(event) {
>> +      var msg = 'Are you sure you want to remove this contact?';
>> +      Permissions.show('', msg, function onAccept() {
> I asked to the homescreen guys, as UX said we should use the same style, and they told me that I needed to use those request.js and request.css files. The main concerns about using confirm were that it doesn't have a title and body params and it is not easy to implement the visuals on them (some icons inside). I would suggest use whatever can be reused in all the apps. @vingtetun what do you think?

I would use window.confirm where I can but I agree that it is pretty 
limited (no title, no way to customize the text of the buttons, ...) So 
let's reuse some code for now and it should probably exists a 'dialog' 
API at some point in the future. I think @fab1ncazenave and @etienne_s 
have some suggestions for request.css/js (can be done in a followup).
`event` -> `evt`
is this.bookmarkEntrySheet.classList.remove('active') necessary here? or we close the inline home and it is no make sense... thanks
current_token should be camelCase too.
nit add a name
A Common way to write this kind of expressions in other files is:

      var e = utils.templates.render(bdayTemplate, {
        i: contact.id,
        bday: bdayString
      });

Also I would suggest to rename 'e' to something more obvious if it is even used after that. 
 i -> id? and bday is an abomination of the Vcard format, birthday sounds so much better...
Too much indexing ;-)
Rewrite this line to 

    `var capitalize = (_currentWord[0] === _currentWord[0].toUpperCase());`

for readability.
I think you may not need this function
I would use a var selector = '.recents-filter-entry > button', as in l60
dcontact -> deviceContact
In fact it should be in ;-) Thanks!