labofsahil/github-event-dataset-2012 huggingface dataset:
Right now I don't have the time to do it right now, but if you read the post in the sencha forums, you should have all the info you need.
Also if you make an example, make a pull request with it and I'll include it.
The basics are: configure the ext loader, include the css and js of downloadify, configure the button's properties.
```haskell
λ> getSimilarArtists "b25b959554ed76058ac220b7b2e0a026" "smpcln" "ingolfr"
["Death in June","\208\147\209\128\208\176\208\182\208\180\208\176\208\189\209\129\208\186\208\176\209\143 \208\158\208\177\208\190\209\128\208\190\208\189\208\176","Burzum","Current 93","Peste Noire","Taake","\208\149\208\179\208\190\209\128 \208\155\208\181\209\130\208\190\208\178","\208\149\208\179\208\190\209\128 \208\184 \208\158\208\191\208\184\208\183\208\180\208\181\208\189\208\181\208\178\209\136\208\184\208\181","Death in June & Boyd Rice","The Legendary Pink Dots"]
it :: [String]
```
Это утф8 судя по всему. Но на каком уровне делать отображалку кириллицы?
`import Prelude hiding (compare)`
@uwej711 The BlockServiceManager is not loading Blocks, but BlockServices. To do so it needs to get an actual block. That's why I created our own twig-extension, that loads a block. Compare https://github.com/sonata-project/SonataBlockBundle/blob/master/Twig/Extension/BlockExtension.php#L119 and https://github.com/symfony-cmf/symfony-cmf/blob/block_bundle/src/Symfony/Cmf/Bundle/BlockBundle/Twig/Extension/BlockExtension.php#L56.
But you are right, if we could change the BlockServiceManager to get a Service injected that takes care of loading actual blocks from block-names, we could share the twig-extension (also it would be cleaner, because it's not really the task of the twig-extension to load blocks). Shall I create a pull request on the BlockServiceManager?
On the other hand, cmf-users might get confused by the name they have to use to render in a template (sonata_render_block). Is it possible to create an alias for a twig-extension?
sure will keep you posted..
@dntndus9611 메뉴를 잘 찾아보시면 Edit this file이 있을겁니다.
사실은 Git 클라이언트를 사용하는 법도 알았으면 하지만, 웹 인터페이스에서 전부 해결할 수 있어요.
Au jours d'aujourd'hui c'est 242 en fait (et 313 signatures au total).
Okay. Is 9.81f (gravitational acceleration) okay, too?
thanks, i'll have a look at that sometime, i can't get this one to work. we're using a zte prebuilt wifi module without any source code, there are speed problems with the new one that we need to use since the latest zte kernel source update.
I don't like this `@`.
You should put the whole form instantiation logic in `get()` and put the form instance in `self.filter_form` instead of a local variable - that way the form will be accessible in other methods. In `get_contex_datat()` you can just set `ctx['search_date_form'] = self.filter_form` and keep it simple.

After you can just move the logic below into `get_queryset()` and access the form by referring to `self.filter_form` once again.



Additionally, it is good to maintain a single way for naming variables, stick with either "search_date_form" or "filter_form" but not both.
> (.*[%]) est diffrent de .*[%]
>
() permet juste de capturer le rsultat c'est tout.
Si on ne met pas les () sur cette partie l (dont on n'a pas besoin de le
capturer), il faut juste changer l'index de l'lment a rcuprer.
A priori la seul chose qu'on veut rcuprer (si a matche), c'est le 04 ou
4, pour savoir a combien de # a correspond. C'est pour a que j'avais
chang a comme a.


> car du coup dans match[1].second tu te retrouve avec 04d.dpx et non pas
> avec %04d.dpx...
> Aprs pour supporter le %5d, je pense que je dois modifier ca aprs:

try
> {
> matched = match[1].second;
> matched.erase( 2 , matched.size()-2);
> patternWidth = boost::lexical_cast<int>( matched );
> }
> catch( ...)
> {
> matched = match[1].second;
> matched.erase( 1 , matched.size()-1);
> patternWidth = boost::lexical_cast<int>( matched );
> }
> une autre ide ?
>
A priori sans changer la regex tu ne peux plus grer a aprs car de toute
faon a ne matchera pas. Et donc tu ne rentreras mme pas dans ce code !
fixes 23295491
This plugin makes a distinction between friendship and followership. Similarly, it could differentiate between group membership and followership. A user could follow a group without joining it (provided the group is open). 
I suppose the main purpose of 'following' is to get an aggregated river, so populating the 'following' feed with both users and groups being followed would make sense.
todo: revert this commit.
Maybe reset this to fix reboot problems when sleeping...
united pubmeder with thepaperlink, for Apple Safari
Why is this blank?
@lukas-vlcek Ha! :), some load balancers had problems with changing values for GET on `/`, though not sure why they don't allow for HEAD on `/`.
We should rather be receiving the cluster id (vds group id) as argument and fetching all hosts that belong to this cluster.
The new option allows to align popup menu to align its bottom to the hovered menu item instead of top (the default behavior).
It's also worth noting that setting those variables to empty strings will break compatibility with todo.sh.

Might be worth opening a feature request on todo.sh if you need to share config files.
roflamo
i cant add any governors.SPLESH is helping me out,even he is facing problems-
http://forum.xda-developers.com/showpost.php?p=21091262&postcount=53
or its just that i m unlucky :(
See http://codereview.musicbrainz.org/r/1714/
Two separate atomics aren't atomic in aggregate.

I understand that you only set one at a time inside the code that you intend to be concurrent, but given that this test is supposed to be verifying the safety of that concurrent code, I'd prefer the following:

• Guard both with the same monitor, or control access via a synchronized block on the test object (same thing, really). Given that they're guarded, they no longer need to be atomic.
• Provide a synchronized assertion that tests — and sets, if necessary — both values.
• Don't access them directly; only through a synchronized method or a synchronized block on the correct monitor.
It's kind of confusing to have both App config together with GetOptions. 
I think you can either use one of them and have --help for how to use it.
This need to be tagged with hw01. use `git tag hw01` then `git push --tags`
Normal qu'il y'ai toujours les urls en dur, j'ai pas eu le temps de les fixer =p
trinity cause that error ofc linux...
tc fixed linux build check that commit
NewVisiolution soy yo. Eso me pasa por hacer commits desde el curro xD
Exchange from this version is broken. I recommend swapping Exchange for new version from my testing.
This doesn't seem right to me. What happens if the user puts in a 3 digit version (1.2.3). This code will chop off the last segment and make it 1.2?  Are you sure you have the right logic here? This seems like something we would do if the user puts in a 4 digit version, since the version attribute in the blackberry-tablet.xml file needs to be no more than 3 digits.
New audio microphone doesn't work because of the missing flag
BOARD_USES_QCOM_AUDIO_SPEECH := true
minor thing - capitals for constants
The correct place to put this listener is inside Agent#setupEvents in `lib/hub/agent.js`. When you listen against that method's `self.socketEmitter`, you're listening on an [EventYoshi](https://github.com/fent/node-eventyoshi#readme) that has only the current socket EventEmitter attached. (The socket is attached to the `socketEmitter` by AgentManager#connectAgent.) This prevents leaks of sockets as agents connect and disconnect frequently, and it prevents breakage as I merge this into the sockjs branch.

I'm taking care of it, so just FYI.
this should typically take up one line according to our conventions
Found this:
https://wiki.jasig.org/display/CAS/Code+Conventions
oh look!
here it is! :)
no, but `addChild` is a spin-CASing bastard and we don’t _need_ that volatile write to ruin everyone’s day ;-)
oh how beautiful this will look once we move to a more recent scala-library
Make this so that it cannot use a different version than the camel version we use (use a val)
ok, no search bar for local docs, good
yes, that is something to be aware of all the time when working with Member
I'll add a comment
yup, good one!
this was previously `private[akka]`
not possible any more?
perhaps add INTERNAL API comment?
Hmm, which problem exactly is not solved? If my proposal solves the issue at hand and is a tad less invasive than yours then for 2.0.x I’d say we go with the smaller fix. @henrikengstrom should confirm that this works before we go forward with this back-port.
I think it was right before. You added an extra indentation level.
you can move this to the FilterChain class, and mark it internal

internal static bool IsNullOrEmpty(FilterChain ....
Basically groups could be used by people who work on the actual data and want a set of data to work with.  At hackdays people normally start by putting relevant datasets into a group.  My mind was on the datastore stuff at this point and it would be good in an idealistic future where certain groups can have their own views of data.
Surely this causes a SyntaxError to not have at least a ```pass``` ?
Probably, do we have an tests for it?
so this branch is called id_cache, but it actually gets rid of id caching??  The idea here is to write it to the cache for both lookups regardless of which one originated the fetch.  Might not show up in tests, but this looks to me like a massive deoptimization.
Not removed by me but the method do not exist anymore in Cluster...
Then implement it and post a PR for review.
R has "CRAN" which is a big repository of the platform-specific package builds + documentation for just about every R package out there.  It's not pretty, but It's where the "action" is.
http://cran.r-project.org/

This is where most people will install from, as well as find and read the documentation. 

My thought was that I would link from the readme to the CRAN page for the package, once it's uploaded.

Does that make sense?
I just tried it out and it does work like a charm. Sorry for opening and closing the issue, I'm a noob at pull requests.
This method should be on JavaPlugin
The bracket should be on its own line.
It is not being used any more. I will remove it.
we'll need to fix those by hand
I guess this could be refactored to:

``` js
return this.get(component + "Path") + "/" + component + "-" + base + this.get("suffix");
```

Think thats better?
What's this all about??
You're absolutely right. I've fixed this below.
Would it be possible to add the `{` `}` around the if please?
Please control your white space.
It's fairly easy. `NSAffineTransform` provides a way to get and set its underlying structure, which uses fields similar to `CGAffineTransform`.
Would you mind reformatting the headers? It goes like so:

```
@interface PREFIXClassName : SuperClassName <ProtocolName, ProtocolsSeperatedByComma>

// All user interface properties should be nonatomic and must have association strength
// (such as assign, unsafe_unretained, or strong) whether or not readonly or readwrite.
@property (nonatomic, assign, readonly) id someObject;

// Use the `instancetype` keyword to tell the compiler that you return an object of the same type as yourself.
+ (instancetype)factoryMethod;
+ (void)otherClassMethod;

- (id)initWithSomeObject:(id)object;
- (void)someMethodThatCannotBeProperty;

@end
```
Nit pick: I usually prefer `function Deferred() {}` so that objects have a name in the console later on.
Don't delete these because we use them in some tests and benchmarks.
K tomuto zrejmemchybi test (nebo jsem slepy?)
The logic here is backwards, the ordereddict module is only needed in python 2.6 and in python 2.7 we need to import OrderedDict from collections.

Not sure when the OrderedDict is used though.
What do you think about exclude_start in ruby's range?

/cc @matz  @nobu @ko1 @knu @drbrain @nurse
Why return true if... instead of just controller=~.... OR path=~.... I think that's clearer and no return needed
We can't mix CoffeeScript into the HTML so here should be JavaScript
use `t.references` or `t.belongs_to`
I just tested this out, can confirm that reload is not needed, it can also be removed from the `$ rake routes` task as well, I can save that for another PR, or add on another commit here if you like
Thanks Adriaan!
`ScalaReflectionException` indicates an expected exception in Scala reflection. This one is a failure that should never happen. It's not fatal enough to warrant itself an `abort` or to be an `Error`, so it's like it is. Though I agree, the code should be deduplicated here.
That'sa bit problematic:  I want to give back a string for html, and need a test for comment to html generation (to avoid unused method). I currently don't see a way to achieve it without testing html output.What'sthe expected problem with that?
I thought about moving this out into its own method but decided against it because I am lazy.
I guess I will have to check whether the base is of type Add. Then this bug will be fixed I guess.
A test should be added to show why this method was changed.
This is what confused me.
Please see:
http://reviews.sympy.org/pullrequest/1133
you should do something with homAC, shouldn't you? ^^
Well, obviously it doesn't. :) What I meant was that the wording here should be changed to make that clear.
This is actually a bit more difficult. AugeasWrapper class inherits from FileAgent and the LENSES_DIR is used in several classes (Adjtime, SshConfig and later maybe more) that also inherit from FileAgent. So, maybe I could put it to FileAgent class instead.
I suggest here should a log message level warning: <code>Yii::log('...',CLogger::LEVEL_WARNING,'system.utils.CFileHelper')</code>.
Somebody certainly did. Fixed in 8b2dbbc
I think the correct fix here is just `char string[256];`. This will be allocated on the stack each call (so it won't be shared), but it won't cause a leak.
YUCK!
sort please
yuk, but ok...
...with it stuffed down there I'd still need to do something like have an animate flag along to condition off to specify the as_filetype (assuming that we don't bust off an entire separate method like animateMe). Flag arguments are ugly and immediately complicate the signature of the function, and isn't useful anyway since currently animate me is the only place where the as_filetype/animate option needs to be given.
Lots of extra copies of objspace, potentially refactor later to use a st_table <objspace*, mark_queue_node_t*> and remove that pointer from mark_queue_node_t to save space?
Why is this migration even needed? I don't think we had image_answer in questions already.
dbcommand
You shouldn't need to modify `MT_TO_KUMA_LOCALE_MAP`. It represents locales we supported in MindTouch, for redirects on legacy URLs in the wild and within docs. Since this is a new locale, we never supported it there. So, we should never need to modify this map ever again, the fewer additions the better.
here you can create a new var like x like this: x.type(). Just a shortcut.
Because, in the `string` argument to `RegexpCache.get`, meta-characters are escaped .. whereas here we need them to retain their special meaning.  `prefix` and `suffix` are *not* subject to escaping of meta-characters.

Separately, I'm working on #705 for which there is a similar need to cache regular expressions with prefixes and suffixes.  In that case, the prefixes and suffixes are things like `"\\b"` to match word boundaries.
See whether you can get rid of the globals in this class, otherwise get them all here and assign them to class properties so that the other global statements in this class can be removed.
Reason for newline?
We probably need this functionality in other tests, too, so you should make it more generally available to functional test cases.
Or change this to config directory.
You're right. What do you think about 64? That should be enough.
wouldn't it be more suitable to define access control specifiers at the time of refactoring and eliminating code duplication? 
Either way you say.
timestamp of startup and gateway IP too? I dunno it seems possible to have a pid and version collision in the case of two machines.
[ On two lines ;). ]
Put private data members at the bottom of the class declaration.
Having VT100 terminal colors worries me.  There are times when we *shouldn't* be showing colors (such as logging in Manhattan). I'd prefer no colors, to naive color implementation that doesn't take into consideration the execution environment.

Also, calling this out using colors assumes that that is what the user wishes to look at.  Right now, that makes sense since we're focusing on performance. However, later when our mindset is different, the colors won't be appropriate.
wont' pass jslint with the multiple var statements.
can this go below the public methods please?
style: dont leave blank links
Does print card need access to all of these views?
can't have a domain in skyphp!
I think from discussion that depends was slow the first time. I think that the recursion could be a the cause. If you wish, you could try to replace the recursion by a loop. I think it will be faster in python. But profile it before, I could be wrong.
I'd replace ``that there is`` by ``when there is`` on this line.
Should we be concerned about these voltdb-install.py changes riding along with this seemingly unrelated changeset?
``get_next_free_ip``
Instead of "else: if ..:" use "elsif ... :"
Deride all you want, a single comma in that context does not show intent clearly enough. A pair of parentheses does just that. If you'd rather see `[value[0] for value in values if value[0]]` it's fine be me too.
I'm just a C# programmer but I think we should use

```c++
else if(GetPlayer()->GetTeam() == TEAM_HORDE)
```

instead of a simple

```c++
else
```

imho:
If a third faction would be added, this wouldn't work well. We should change this before it becomes a problem.
What does goclus have to do with it?
"that pain" - what pain?
Is this change really "required" (this is more a question for my own info) ?
Just so we're on the same page: being hlint clean is a non-goal for Cabal. Any changes that are made just to make hlint (which is really just an embodiment of Neil's coding style) happy will introduce lots of noise in development. Don't get me wrong, hlint is an excellent source of suggestions for changes (which the many good changes in your patches show), but they're not all good.

To be specific, replacing all `Foo _ _` pattern matches in Cabal with `Foo {}` is nothing I want to do.
`if` statement here, please.
I also think the error message would be more readable laid out over multiple lines, like match failure errors. Something like this:

```
error: import loop:
    gnu.hurd
    gnu.hird
```
looks like an accidental `|` -> `when`
Why change the content here?
Instead of having the ticket number in the variable name, usually we mention the ticket in a short comment. So you can just pick a short name or even reuse `i` if it isn't used in the code path.
thanks for your patience, here is the new pull request:

https://github.com/jquery/jquery/pull/1005
Hmm, it's qualified.  In turn, it is not relocatable since it's schema-qualified.
Apart from adding API call, removing this value is main reason for the changes. Totally unneeded once you get rid of slowing down/speeding up with powers of 2.
If xcode-select is not configured properly, we detect it somewhere else and ask the user to fix it, don't we?
puts_columns just does string compare, so the items itself and the list of to-color-in-green items, both, need to have the same string. What would be coo but not yet possible: 
```ruby
outdated = Formula.outdated.map{ |f| "#{f.name} #{f.versions}" }
installed = Formula.installed.map{ |f| "#{f.name} #{f.versions}" }
puts_columns installed, :colors=>{:green=>outdated}
```
Eclipse kommt damit zumindest klar.
And this one could just become if [ ! "${cksum}" ] ...
Please add anchor_tag.short_description for admin list_display
Looks like a merge artifact.
The parameter is now called `message`.
why do you think this would fail with Parameterized tests?

> Can we just create a filter that takes in a description, and returns true if it matches one of the desired method patterns? 

no for two reasons
  1. you cannot put regular expression with () into Description, same issue as #327
  2. if a filter is created and used by the runner, it is not possible to inform the user about wrong patterns e.g. "MyClass#mthed.* has not matched any methods"
Any reason to not lock at current version (0.1.1)?

https://npmjs.org/package/murmurhash3

I've been running 0.1.0 locally without any problems.
This could be named more generically. It won't be restricted to Wiley in MAP and it could be used for other products or things. Perhaps 'product-phone' and 'product-tablet'?
I checked File::fnmatch, the function it does better, but I'm not sure is it okay to use it here? For me the regex is more general in terms of for string matching. (I'm new in ruby world.)
Can use @memoized for this pattern https://github.com/dimagi/dimagi-utils/blob/master/dimagi/utils/decorators/memoized.py
Why use `Class.cast()` rather than a normal cast?
All this class used old style documentation !
I propose to get rid of this one.
This is nasty – there is a WrapAsScala convertor to treat ArrayList (a Collection/Iterable) cleanly and then pass it back to JsArray as a Scala List
There's a missing period here after "date".
With the change you made here https://github.com/mozilla/butter/pull/938/files#L1R81, this check might be unnecessary. I'd rather have this function fail as an obvious sign that we call `close` way too much than it half-fail silently.
Might be better to do 

`requestString = options.search && "http://search.twitter.com/search.json?q=" || "http://whatevertheotheroneis"``
 at the top
To keep style consistent, all of these `if` blocks should start `if ( ... ) {` (note the space after the `if`).
You don't need to say `undefined`, actually. `return` will just make it `undefined` if you're not returning any data.
Maybe just remove blob and add nothing?
oh cool, that makes perfect sense.
Missing label tag.
I'd say we either start a separate CONTRIBUTORS file or start a wiki page.
Wrong path.

For now it is `http://localhost/CoinCod/` but in production it will be `http://coincod.com/`
& `echo $_SERVER['HTTP_HOST'];` with `/CoinCod/` will give `http://coincod.com/CoinCod/`
zipe_safe is used by setuptools. If set to True it will generate a zipped egg archive at install time. Sometime it sucks so I'm used to set it to False to avoid some problems.
This should have a jira to track this new feature. Please make one to get it in release notes.
C语言中的位操作的非是~符号，!是逻辑非操作：
ss_srv->flag &= ~NGX_HTTP_SESSION_STICKY_INDIRECT;
This should not be the default behavior, i.e. it should be '0' by default.
我知道你的意思，最好能直接忽略，方便维护者，因为有时是打包的时候，可能所有的的包都会加上这个选项，如果碰到FreeBSD7 以后，就只能改编译选项了。
No worries.  Easy chgs
I think that this would be better if it is just:

```
results = np.ma.masked_where(results.imag, results, copy=False)
```

However, it is not important for the moment.
Filenames should be in code tags.
We can wait and see what others think.
in allowedFields
You could also use `~0` to specify "set all bits". It's less controversial. Though I think (unsigned) -1 is correct. As in, converting from signed to unsigned never results in undefined behavior, don't try it the other way, though...
HEREDOC somehow?
please remove the ``_``. It was part of the doctrine1 CS (used for the early development of doctrine2 too) but not of the current ones anymore (new code does not use it anymore, and private methods are renamed when the code is modified in existing classes.
indentation, and print (see below)
i only see 2 spaces, maybe they're tabs?
Also, do these need to be ordered?
Actually, I'll recomment this and deploy testkit in our Nexus. The reason is that maven is definitely  lost on some people that are unable to configure their entreprise repository, so we limit what they have to configure to the strict minimum.

For the record, Akka 2.1 will be deployed in central.
update 3: I just realized this is still inside of the phpcr class, so I guess that's why you insist on having a parent. Disregard my last two comments...
This is a nice simplification
This is a problem - if you are using an API that is for TFS 2010 - put that in [GitTfs.Vs2010/TfsHelper.Vs2010.cs](https://github.com/git-tfs/git-tfs/blob/master/GitTfs.Vs2010/TfsHelper.Vs2010.cs) file.
None of the objects we put through `LazyEvaluate` by default should be non-ascii byte strings. If the user cares enough to configure it to do something else, and that includes non-ascii characters, they should use unicode.

In any case, they could directly provide a non-ascii byte string (i.e. not using LazyEvaluate), which would break in the same way.
I don't see where this function is used.
Am I missing smth ?
No need for this if statement.  just use `f['kernel_id'] = km.kernel_for_notebook(nid)`.  Then use null in js instead of undefined.
Can this really be constant? How does this effect localization?
the start of the week is a weird thing in the US its Sunday for example there is some ISO that states
generally its Monday I can't remember which =/ I was having this debate with a colleague who lives in the UK.
By the looks of it, I think a single `memcmp` over the whole struct would be faster -- regardless of whether there are unset fields or not.
It actually hits somewhere in the first stage, while missing items are being created. And yeah, I've seen the progress jump back from 50% to 33%. Is this better or worse than having an instantaneous jump from 66% to 100%? 

We could also weight the third stage to be 10% of the total and always include it. It's only creating a relatively small number of directories, and I doubt we'll be implementing anything that includes fetching and checking out the submodules.
i think this should be in a presenter / printer class.

eg.

```
class MapPrinter
  ...
end

```
these, not those
If it's not planned for the fist iteration, please remove those. We'll have all the time to add them if this has to change.
These trailing commas were adding a new undefined array element that was throwing errors in IE.
Remove that `#` there and the `# recommended: ` above? Is that what you are asking? I am game for that.
Please confirm that the Web_Service class can be used >1 time to request data from the middleware. If so, you're good here. If not, you will want to instantiate this each time you need it.

Also, if you do end up keeping the instantiation here, please fix Branch_Model::_getValues() to use the class-level property.
Why the hell are there unit tests in a file that should contain a middleware service???
could you change this? (to answer your question: yes. how? that's still being figured out by Identity.)
No a co si teda tu funkci definovat predtim a mit ji pojmenovanou? A do
setTimeout jen strcit jeji jmeno.

Dne 18. srpna 2012 17:05 Jiří Maršík <notifications@github.com> napsal(a):

> In coffee/recorder.coffee:
>
> >
> >    $('#playButton').click ->
> >      myPlaybackMirror.focus()
> >      $.each recordingTracks, (name, track) ->
> >        $.map track, (event) ->
> >          setTimeout ->
> > -          playbook[name] event.value
> > -                       , if name == 'evaluatedCode'
> > -                           $('#turtleSpace').get 0
> > -                         else
> > -                           myPlaybackMirror
> > -        , event.time
> > +            playbook[name] event.value,
> > +                           { codeMirror: myPlaybackMirror
> > +                             turtleDiv: $('#turtleSpace').get 0 }
> > +          , event.time
>
> Tak ten setTimeout není moje funkce, leda že bych si udělal alias s
> přehozenejma parametrama.
>
> Jinak jako ne, tohle z toho nemá vyjít, playbackMirror není funkce. Grrr
> CoffeeScript! Dám na začátek řádku 82 čárku, líp jsme to tady nevymysleli
> (když píšeš v CoffeeScriptu objekt a máš ho ve složenicích, tak musíš buď
> dávat čárky na začátky řádků anebo srovnávat ty nový property names s
> otevírající složenicí).
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/progma/recorder/pull/3/files#r1408886>.
>
>
Yeah they're non-final so that we can load them in the background thread if you're executing things asynchronously. We need to know whether or not the method is a synchronous one or not regardless. Perhaps we just check the synchronous boolean like before and then pass it down so all these values can be computed in the constructor. Will think on it a bit.
The description does not seem to be properly aligned.
this one should probably extend from the ConsoleInitEvent (which should then be renamed to ConsoleEvent) to give access to the input and output too
should be an `isser` method as it returns a boolean
workflowId should not be parsed for any meaningful data since should be considered an opaque value. There's no contract for it's contents and implementers of StatsReadService and StatsWriteService are free to implement it as they see fit for their environment.
since this parameter expansion technique is currently unsafe; can we drop this feature alltogether until we figure out a better way?  we could simply require the user to provide fully expanded parameter lists for now?
What happened to us using a jQuery library?
This is side effect of parsing JSON. You can't put `nil` to `NSArray` or `NSDictionary` - so it uses `[NSNull null]`. Problem is when you do:
```
utc = [NSNull null];
[utc intValue]; // Error - there is no intValue method for NSNull
```
So you have to make sure that NSString is NSString and not NSNull.
To highlight the fact that this is not yet ready for use, and the semantics or parameters may change, you could write ```~lifecycle: []``` instead of using ```~in_product_since```. This indicates that the call is not yet in any release and would render it as a "prototype" in the API docs.
Adding bangs to user-defined methods is not something I've ever seen.

I find it confusing with exception-raising use of bang (and the "not" meaning in front of variables).

Now that I type that, the bang operator is already overloaded, meaning-wise, which is why we wouldn't use it in method names.

You can do `self.update_attributes!(thing: value)` to call a `save!` at the end:
http://apidock.com/rails/ActiveRecord/Persistence/update_attributes!
I would also assert that the `User.find_by_email` was not called. Basically the inverse of lines 10,11.
Unless there's a compelling reason to keep it, I'd remove the twiddle waka.
The grammar of the RFC is confusing. I see you implemented "All users who are not invisible and not sharing a channel", while I read it as "All users who except those invisible and not sharing a channel" (See the difference?). I don't actually know which interpretation is correct :(
I think so.  Below the this_program_path() function returns a std::string.

This has been like that for some time, I think some other rearrangement of headers (or maybe inclusion in a new place) made me realize that <string> wasn't included here as it probably should have been, we'd been coasting based on the coincidence that most files that included sysutil.h happened to also include <string>.
Hmmmm. I can't remember this functionality being intentional (the "Does not reset data limits." part). Is that correct? If so, did you try removing that particular section and seeing what happens?
Better would be subclassing TreeDecorator with each one having a method that takes coordinates, a chunk, and a random to generate each tree type, then a generic ChoiceDecorator which would choose a tree type from those available in the constructor for the biome.
Because I dropped the .py file from that location - and its discouraged because vundle, pathogen, VAM (I'm the author of this) are better alternatives compared to putting everything into ~/.vim which makes it hard to get rid of plugins later - as well as debugging (disabling plugins temporarily)
VAM is the only one supporting rtp subdirectories, see json file.
Yes - I know its the way Vim help recommends doing things .. but I'm pretty sure it will get into your way more often than not. If you feel differently readd that line.
makes sure that HREF and FORM urls are handled the same, first language THEN pages_root
So you mean that editor widgets should intercept the contextMenuEvent and deal with it before it arrives to the area pane, right? It does sound reasonable... I'm having a problem with an editor which displays an old component which displays context menus by intercepting the right-click-up event (it's a chaco tool...). It never gets the right click event, the context menu wins. Any suggestion?
Yes, this method should be working on the model, not the operation. And the model should be read using jndiNameAttributeDefinition.resolveModelAttribute(context, model). The isDefined() check should be made on the node returned from that call.

What's "jndiNameAttributeDefinition"? A final field set in a new protected constructor, passed in by the subclass constructors. It could be a common AD shared between the classes as well if that's cleaner.

protected abstract Service<KeyGeneratorFactory> getKeyGeneratorFactory(final ModelNode operation) should be changed to take the model instead of the operation (or both if the operation is somehow needed). And any reads from the model should be via AttributeDefinition.resolveModelAttribute().
THis should really be injected with 

    @ContainerResource
    private ManagementClient managementClient;

And then use the client management address and port (which will format the address for ip6)
I know the existing tests don't currently reflect this, but the standard is actually double quotes for strings.
Is this our new coding standard?
You don't need bind.
Please localize this string.
Nit: no needs for the extra parenthesis around the inline if
Anybody could tell me who would execute this shell script(`tools/ci/unit/*.sh`)? And then I could pass the variable `$(GAIA_ALL_APP_SRCDIRS)` to it exactly.
`function md_handleEvent`
This is useless to PDF.js, which is an inline activity frame.
two (aside)
You’re right, an `<li class="description">` element would be better here. And that should be applied to the whole Settings app, since all paragraph-in-list occurrences are related to this `description` class.

I’ll see if I can send a PR to @vingtetun tonight.
@vingtetun Oup! here it is again!
@ian-liu Sorry for the misleading of deviceStorage, I didn't know we can get a storage type of "sdcard" and access any file on the sdcard. That will be great to remove the switch handling of contentType.
And if we can successfully get the file from the deviceStorage, it's also possible to get file.type, which should be the mimeType of the transferred file.
Maybe when the contentType is empty, we can use file.type instead.
This seems less than ideal. What if the script is run more than once against more than one pe version (without cleanup)? The version should probably be captured after the curl and then used in the install call here.
you could write this as:

f1.andThen(f2)
1. Should you pull commutativity into the pseudoring as well? That's what I'd assume, which agrees with http://en.wikipedia.org/wiki/Pseudo-ring. (And, practically, it seems like the existence of a one is the only possibly tricky thing anyways.)
2. I think zeroAnnihilates already follows from the rest of the pseudoring properties.
You don't need the `else` here.
I don't think that it's a good idea to thread Sburb exactly like an actionQueue. Then it would have to have the same functions/variables, which doesn't seem very reasonable to me. Otherwise there would always need to be a check if the returned value is Sburb or an actual actionQueue.
There's lots of direct quartz usage here.  I'd like to see quartz contained wholly within the scheduler package so that we don't let it sprawl out everywhere in case we ever need/want to replace it.
Given that the code you wrote below already handles "None" for this setting, maybe using None as the marker to skip this would be better than a magical value?
I mean that what is written to the cache has been successfully parsed as json already, so it should be safe to re-parse it, and should not blow up.
Not sure I'm following you. What do you mean? I'm not using any textarea here. Selection Listener are not working on text fields, as mentioned on the `selection.js`. That's why `onSelect` is used as well (see the test below).
Side-note: Very interesting way of creating  a `document` I wonder if this technique can be used to provide `DOM` api in the module context.
I've modified test according to this comment, but for some unknown reason, the cross domain xhr test doesn't work.
It also maybe worth adapting [ES.next private names shim](https://github.com/gozala/name) I wrote until we get an actual one
once [bug 645416](https://bugzilla.mozilla.org/show_bug.cgi?id=645416) is fixed.
Can EJS throw an exception or anything that would bring the server down in development mode?
cleaning up copy paste errors in comments
It's inefficient to do this with n+1 queries, where n is the number of batches. It can be done with a single query - just fetch all perftest datasets and loop over them to build up your dictionary.
nit: for the new code please immediately use the triple operators. Others we can update later.
I actually removed that before committing because I wasn't sure about the proper way to do it.
Should I use php_error_docref with E_WARNING ?
makes a more significant different on the multi-proc version, INAL but I'm guessing it's because there's less contention for I/O.
Is this magic state method really better than just passing the parameters to the block?
Unfortunately this is used indiscriminately and I don't have the time to untangle that right now. For the moment it's better that it doesn't break.
`arity` has some subtle edge cases you have to be aware of:

``` ruby
def foo(a, b, *c)
end
method(:foo).arity # => -3
```

``` ruby
def foo(a, b, c=3, d=4)
end
method(:foo).arity # => -3
```

The arity is returned as `-3` for both methods but they will fail when called with 1 argument.
it's not the only one, I will provide a commit that will limit the scope
Deleting this line makes it so that the slider doesn't update to the new value. I think this line should be placed after the sanity-checking of input, right?
useless define ( if mvidner confirm it )
The setServer method adds cookies to the header object, so it was being done twice.
When you change hash generation in `Bundle` please add: `length=40`
@cordoval https://github.com/cordoval/KnpBundles/commit/0eae336e888b3a4ecb3436d5a3775e69099ae4f6
whats mean _best_ mb bundles with much srores?
What is this doing? Please add some docs and tests so we are sure 1. to understand what's going on (at a high level) and that we're not breaking anything when doing modifications.
Why import these inside the local_render_to_response function rather than at the top of the module?
Doesn't seem to work for me, still shows a thumbnail.
Please use ``$req->query-get()`` instead. It is more efficient and more precise
I agree; that is a legacy method that I did not dare touch without a thorough search
Regex issue here, I believe
Doesn't help, in ezp-next context the path is still invalid (ref: /vendor/ezsystems/ezpublish/)
As discussed, if you want a better name for magic (for some _crazy_ reason :p), go with facetField
Could you also remove the 2 commented lines (other one is in Makefile)
add newline before `##`
We support ppn=All on our clusters atm, 
this might be a bit more practical, but only usable for our purposes,
although I think this class is pretty constrained to our environment anyway...
Noted.  I'll do my best to conform to your preferred style.
@kitcambridge: I think you're too quick to call bikeshedding. A line that was unrelated to the pull request's advertised feature was changed. It was an improper change, as both @Raynos and I showed that consistency is more important than any premature optimisation concerns. Discussion over. Now  we're just bikeshedding about what qualifies as bikeshedding.
Did you mean to use `is_element_visible` here?
this should be `move_to_element` not `mouse_over`
Can we get a test for this?
Is it "profile-pic" or "profile-photo"? The bug says one thing but this says another.
I don't think that you should bother fixing this now, but I don't like all of the selectors hardcoded deep in the code. As many as possible should be declared up front as static configuration.

It *might* be okay for sub-selectors to be declared in code rather than configuration:

```js
var $loggedIn = $('.logged-in');
/*... */
$loggedIn.show()
$loggedIn.find('.next').click('showBadges');
$loggedIn.find('.email').text(Session.currentUser);
/*...*/
```

But since we are moving away from this, don't fix that now. Just keep it in mind when you write the next versiiiiiiiiiionnnnnnn
Make this an if block please
`client.storageUrl` is undefined. I'll look into it a bit more. Also see some comments I should have removed. I clean it up a bit later.
This is reverting our changes twice now, please do not merge this.
Right, but the heading itself isn't a navigation item. Heading tags get the class heading, and then we modify it with something else when necessary (e.g. landmark if it's just there for screen readers).
Comment is useless, if you have the data might as well make it easier to access.
с названиями что делать будем?
Yep. Reusing notification IDs can also be problematic if an application is buggy and uses an ID that has already been freed/reused. I'll make it a growing number but the notifications.contains(...) check still needs to be there in the theoretic case that the ID loops back to 0. In reality no one will ever have 2^32-1 notifications though, but still.
is succes a typo?
Can we get a version of this patch which follows a naming convention consistent with the other functions in the library?
Oh, bummer. I'll fix this after I pull it in tonight. I'll take care of the rest. Thanks, @AlisterH!
Thank you :)  I can't believe `lein` didn't complain about this...
Maybe this makes more sense in one of the query namespaces? Nick's newer query code has some abstracted out query functions in a shared namespace; that may be a better place for this.
You moved this docstring away from ``.scene`` so that has to be moved back. I'm not sure what ``ikscene`` and ``ipscenes`` are. Can they get better names and docstrings?
Please initiate in the same order as the signature.
The `hasAccess` check should probably happen inside of `getLatestRevision`. Also, can `getLatestRevision` take care of mapping the `uri` to the download link? Seems like something that could be better be done in the API.
A **principal id** must be provided
Is there a rule of thumb in deciding where it should go that I've missed? It seems to me that adding it to the URL would make more sense.
A good rule could be that only parameters that need to be persisted go in the body, navigational parameters like commentId/replyTo/contentId in the URL. What do you think?
OK, I'll add Python(x,y).

I see Sage as kind of its own thing - although it includes most of the key packages (except no pandas, older versions of IPython & Scipy), the user experience is very different from these distributions. I certainly hope we'll keep a dialogue going with Sage, but I don't think we should present it to newcomers as a way to install the Scipy stack. Sage seems to be promoting itself in its own field quite successfully.
How about something like:

```ruby
  data_keys = headers - rows.first['Inputs'].keys
```

The header array contains just the data fields. Then the fields for programmatic use would be ignored automatically.

The last part where I subtract the keys of columns with input fields is not safe. This implementation would fail if an input field would just appear in the first row.
Retain the spacing here.
It's good that you are checking for moderator permissions first :)
yes I agree, I find it confusing. I am not entirely sure what the use case of the <code>req.accepts()</code> would be if it always returns true (or at least for 99% of clients). I understand that from the framework point it matters, but APIs are designed for user code not system code.

There can be an internal interface to check implicit accept, or an optional flag in the interface called implicit set to false by default.
а зачем нулем? главное, что там не 1, верно?
Is the `.` at the end here correct?
minor thing here. i like error messages to be as user-friendly as possible, so i'd suggest you phrase it "unable to update" instead of "unable to put." that way it uses the same wording as the function name and abstracts away the concept of a PUT to the user.
I can continue to work on this pull request adding commits. I'm suggesting to merging it sooner then later cause the current unit tests wont work if you try calling any of the chrome.* apis.
Ok for the new line at end.
In other AUTHORS files like [AUTHORS of dart-synonym](https://github.com/dart-lang/dart-synonym/blob/master/AUTHORS) or [AUTHORS of bleeding_edge](https://github.com/dart-lang/bleeding_edge/blob/master/AUTHORS), there is an empty line between `Google Inc.` and other authors. Shall I really eliminate this empty line ?
Commands: None? Shouldn't this be required?
Strange, I updated the InlineData content to the following

```
[InlineData("http://github.com/libgit2/TestGitRepository")]
[InlineData("https://github.com/libgit2/TestGitRepository")]
[InlineData("git://github.com/libgit2/TestGitRepository.git")]
```

Here's the result of the test run

```
------ Test started: Assembly: LibGit2Sharp.Tests.dll ------

Test 'LibGit2Sharp.Tests.RepositoryFixture.CanFetchFromEmptyRepository(url: "git://github.com/libgit2/TestGitRepository.git")' failed:
	Could not find the reference refs/tags/annotated_tag in the list of expected reference updates.
	RepositoryFixture.cs(546,0): at LibGit2Sharp.Tests.RepositoryFixture.ExpectedFetchState.RemoteUpdateTipsHandler(String referenceName, ObjectId oldId, ObjectId newId)
	RemoteCallbacks.cs(123,0): at LibGit2Sharp.RemoteCallbacks.GitUpdateTipsHandler(IntPtr str, GitOid& oldId, GitOid& newId, IntPtr data)
	at LibGit2Sharp.Core.NativeMethods.git_remote_update_tips(RemoteSafeHandle remote)
	Core\Proxy.cs(1101,0): at LibGit2Sharp.Core.Proxy.git_remote_update_tips(RemoteSafeHandle remote)
	Repository.cs(630,0): at LibGit2Sharp.Repository.FetchInternal(RemoteSafeHandle remoteHandle, Int64& bytes, GitIndexerStats& gitIndexerStats, Nullable`1 callbacks)
	Repository.cs(450,0): at LibGit2Sharp.Repository.Fetch(Remote remote, FetchProgress progress, RemoteCallbacks callbacks)
	RepositoryFixture.cs(105,0): at LibGit2Sharp.Tests.RepositoryFixture.CanFetchFromEmptyRepository(String url)

2 passed, 1 failed, 0 skipped, took 7,32 seconds (xUnit.net 1.9.0 build 1566).

```
I'm not a huge fan of the inline rescue being used here.
Could we use a `where()` to avoid raising an exception and then use`try()` against the possible nil object, something like:

    new_label_field_name = self.entries_custom_fields.where(:_id =>value).first.try(:name)
これなんぞ！
I feels weird to skip the 13 to 20hz band. That should be beta too.

also how about theta?

what if the epochs are too small for low frequencies? would it throw a warning? just trying the anticipate mistakes a user would do...
looks like this file slipped through the cracks
fix indentation of the try block
Just as a side note: this can be a simpler 1L if you are creating the file for the first time with the uid.
style: wrap mod_* in double backticks ``
so the build is broken right now?

can you send a separated PR to fix it?
,该模块尚处于试验阶段，其指令在未来可能会发生改变

这句我们已经提bug了，不用翻译了。
That's correct. Monitor has the same interface.
Make WW active initially, so it would become inactive when scrolled down.
Open ticket and remove fixme
You are right, however, from pep 353

>Conceptually, Py_intptr_t and Py_ssize_t are different things: Py_intptr_t needs to be the same size as void*, and Py_ssize_t the same size as size_t. These could differ, e.g. on machines where pointers have segment and offset. On current flat-address space machines, there is no difference, so for all practical purposes, Py_intptr_t would have worked as well

Maybe we should make that official for Python > 2.4?.