labofsahil/github-event-dataset-2012 huggingface dataset:
Strange behaviour was noticed by me. Without this line, after updating the data (by clicking on the chart), `@hilight` method was called and the hover was displayed in upper-left corner. With this line, everything works as it should.
If I HAVE TO make every class that I touch perfect, I won't do any refactoring at all. Come one, one thing at a time. This is already a big PR. And you don't want it to block other evolutions, like the removal of Peer classes :blink:
As above, I'm not sure we should mention other ways of submitting code.
1) sometime ago i compiled some win32 sources on linux, under a wine emulation layer.
'Windows.h' will work both in native windows and under wine.
'windows.h' will work only in native windows.

can anyone verify this? just for my interest

2) those two macros windowsx.h macros are needed for my implementation to work. if we dont use those macros users with multiple monitors might have troubles when modifying windows. are you sure its not delivered in a gcc/mingw environment?
Np, unfortunately I have been out of development on rubinius for long enough, I'm not quite sure if there are other reasons this PR is on hold, so I would bug some in the irc channel about merging it, or seeing what else is an issue.
All this can be rolled up a few lines of program code that allows for additional cmd states.
Did you test this against a version that caches? Otherwise, every time you test a given class, it will have to do the reflection, which is going to be slow.

Honestly, I'm not 100% convinced we should abstract this into a static method call; the handful of places it was used before were each optimized for the specific use cases, and you'll notice they differ in approach based on context.
In this context, it's just a test note - all other escaping tests define immune chars so I wanted to explain the omission here.
excessive white spae
excessive white space
Missing docstring for function
Why do we need a copy of mercurial outside of the virtualenv?
marked as ``EXTRA_LAZY``
meta \w include underscores
Done. LDAP password no longer stored.
could you remove this macro please.
Yeah a bug occurs when you try to unset the first element in a foreach but you're right, it's better to keep both test.
Needed for the new layout of the packet. It's listed like that now: §1\047\012w42b\0Wiki.vg Minecraft Server\05\010 (protocol version 47, minecraft version 12w42b, motd 'Wiki.vg Minecraft Server', 5 users, 10 slots)
remove default html comments
It would be more efficient to just create a new instance here.
the proper one would be ``{@inheritDoc}`` here.
Im not sure I understand. The logic here is `if ($referenceable) $metadata->setReferenceable(true)`. For versionable it is the same but I set the value of $versionable, which is a mutable string (simple, full)
Oops, left over from a previous commit that I ended up reverting. Good catch.
I'm wondering if we should actually send the scope to all the other providers too. If that's true, this method doesn't need to be overridden does it?

I'll check if the other providers accept the scope too, but I believe they do (it's an optional parameter according to the spec).
You do use a web.xml so this plugin config can be removed...
Can this be removed now, as Errai is in central...
Believe me, I tried. Short of writing custom SQL, I couldn't find a way to batch-add an item to a related set for all users at once. If you can find one, let me know, I'd gladly prefer it over this.

The production site has about 33k users, which I think this should be able to handle as a one-time thing.
Zacc,  this marker is for pages that end up having  /b in the url. I will update this file  with  the working markers
Filed [follow-up](https://bugzilla.mozilla.org/show_bug.cgi?id=762234).
Django's
Can you give the audio/video elements an id, save it to `options` and use `getElementById()` instead of `getElementsByTagName`
We should see if Paul has got this bug fixed yet
too late to nit :-) I would've suggested putting each action on it's own line for readability.
The same comment copy/pasted almost verbatim the fourth time in the same file.
why this move?
The first line looped through each item as an array. The group gets passed into the SQL to the DB. This shaved 30seconds off the query for me.
Why did you do this?  The whole point of it was to allow `AddItemByDefindex` to add two items with the same Defindex without moving them instead.
I was debating that (that's what was in the `""` I forgot to remove). I was also trying to keep it from getting out of hand with stuff like this:

>  Scratchpad not available | Debug mode | Problem permalink: some-long-description #123

Are we even interested in supporting a non-debug mode?
See my comment at the top, I must have missed these.
masquerading as a man with a reason, my charade is the event of the season, and if I claim to be a wise man it surly means that I don't know
Please document.
I don't think this was meant to be included :)
Are these changes also done in the production repository? (e.g. I don't see ITIN appearing here)
Also here, no need to do any transformation. I'd just expect an `ArrayConfiguration` to be passed in instead of doing this inside the command itself
Why are you not setting this in the `initialize` method?
Можно все-таки называть, чтобы было понятно читателю. Хоть класс и спрятан, но его можно назвать TarjanRunner
Uh, I guess I can't easily change that. If I add the spaces, toHtml will render
```(xml)
<div class="vcard">
   <p><div class="fn">Joe Doe</div><br></br>
   <div class="org">The Example Company</div><br></br>
   <div class="tel">604-555-1234</div><br></br>
   <a href="http://example.com/">http://example.com/</a></p>
</div>
```
and even so if a add a `rep(' ')` before the `accept("<div")`. So fixing this test would need (a lot of) (hard) debugging work. And it's not completely obvious what the desired outcome should actually be...
If you really need to control those behaviours for testing, wouldn't it make more sense to allow a trivial code object to replace them, rather than exposing these internal implementation details?
So it seems like for some of our docs we're documenting both v1 and v2, and for others just v2... is that accurate/calculated?
Cool, haven't run `lein docs`, will give that a whirl.  And the italics sounds reasonable.  Was there anything else you meant here by "The comment lines aren't correct markdown"?
Should be : E-mail adres (see : http://www.vandale.nl/opzoeken?pattern=Emailadres&lang=nn )
This is not the comment id. It's a utility for sorting. The comment should be updated.
The managers are not passed in to the create.
Wil `Row[]` be valid since it's not a type?
no, initially I though each subclass would implement the isValidProposal differently and forget to remove this.
What happens if ``running := false`` but ``dispose`` was not called? The ``eventActor`` may prevent GC of ``ScalaJdiEventDispatcher`` instances, no?
25 seconds timeout... (just making sure it wasn't an oversight)
Are there any threading requirements? `typeTree.symbol` may be unsafe.. If so, please add a warning in the scaladoc.
OK, last push on this - if you insist, I'll make those require_once() even though I disagree. The difference between include() and require_once() is making sure the file exists and hasn't already been included. If the library name isn't already in the property, we know the file hasn't been included yet. If we found the file in the current directory, we already know it exists. These are steps we have to take in our loading process. So, why have PHP repeat those same checks? It's redundant and inefficient. As I said above, if you insist on that redundancy, I'll make the change and shut up about it.
Missed the last one here
We do not plan to identify this vocabulary as part of this work.  I would invite you to open an issue to discuss this once we have closed this issue.  Thank you for your attention and input on this issue.  :-)
здесь будет сравнение именно объектов, а не ссылок на объекты?
Имхо, лучше обращаться к переменным через свойства, все таки ООП как никак... self.dna
Опять же имхо, [self.dna objectAtIndex:dna_i] куда читабельнее [[self dna] objectAtIndex:dna_i], нет вложенности скобок...
Даже если ты переопределяешь метод init, не обязательно его указывать в h-файле. Его объявление уже имеется в классе NSObject, о которого ты наследуешься. Это не ошибка, а просто лишний код.
Вообще, по условию задачи, методу mutate нужно передавать параметр типа int. =)
в Final 3 исправил на isEqaulTo:(id)
Иначе чем что? По заданию кол-во символов == 100, значит совпадает с процентом.
"Правила" ООП здесь совершенно нипричём.
Да, возможно не вчитался в ТЗ, спасибо.


Sun, 04 Nov 2012 11:02:56 -0800 от s1dney <notifications@github.com>:
>	
>
>
	
	
>
		
		
			
>
>In DNAProject/DNAProject/DNAProject/Cell.m:
>> +                [_array addObject:@"T"];
> +            } else if(r>=50 && r<75) {
> +                [_array addObject:@"G"];
> +            } else if(r>=75 && r<100) {
> +                [_array addObject:@"C"];
> +            }
> +        }
> +    }
> +    return self;
> +}
> +
> +-(int) hammingDistance:(Cell *)cell {
> +    int result = 0;
> +    if(cell.array.count <= ARRAY_COUNT){
> +        for (int i=0; i<ARRAY_COUNT; i++) {
> +            if(self.array[i]==cell.array[i]) {
>
>дистанция увеличивается, если array[i] НЕ совпадают
>
соответственно должно быть '!=' вместо '=='
>
              —
>
              Reply to this email directly or view it on GitHub.
Нет никакой надобности в создании этих iVar'ов, где необходимо нужно создавать локальные переменные.
May be, I dunno...
return ((direction + facing - 2) & Direction.MASK) + 2;
1.0, woo
"Sisepõlemismootor"
It is really only a Cordova 2.0 thing.  If you have another platform loaded this isn't an issue.
Removed:

https://github.com/worldofchris/cucumber-js/commit/b827efbf17fd6cd2aaf285622c074ce43ed03906


On 24 Jul 2012, at 23:05, Julien Biezemans wrote:

>> @@ -109,6 +115,10 @@ describe("Cucumber.Cli.ArgumentParser", function () {
>>       var shortenedOptionDefinitions = argumentParser.getShortenedOptionDefinitions();
>>       expect(shortenedOptionDefinitions[aliasName]).toEqual(aliasValue);
>>     });
>> +
>> +    it("defines an alias to --output as -o", function() {
> 
> This is empty.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/cucumber/cucumber-js/pull/79/files#r1230307
thats what I was thinking, still new to using the js interop stuff. Wanted to make sure I'm not passing back a `Proxy` object.
hmm the comment describes the opposite what the code is doing?
Where shall I add it. In the doc string? In each topo functions doc string?


On Fri, Nov 2, 2012 at 6:59 PM, Alexandre Gramfort <notifications@github.com
> wrote:

> In mne/viz.py:
>
> > @@ -20,17 +20,22 @@
> >  from .fiff.proj import make_projector, activate_proj
> >
> >
> > +def _clean_names(names):
>
> can you just add a note why we do this?
>
> otherwise +1 for merge
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/172/files#r2019230>.
>
>
That's really cool. would love it.
1) Ok. I have to take closer look to the mne.baseline
2) I thought the scope of the function would not modify power if power was bound to "tfr". But might not a copy incur memory costs?
this is technically teaching people something wrong - you use the h1 element, which has two tags: an opening tag, and a closing tag. Neither of these are involved in the actual text, the text goes in between the two.
Remove //NEW
Why are you removing this code?
I generally prefer the matching bracket to be on the same column. This helps a lot for finding where a block starts/ends. 
If possible, do not move "{" to the end of the previous line.
coffeescript does implicit return, so you don't need the return here.
Ok, thanks for explaining. So wouldn't this revert the outline normalization done by `normalize.css` on flexslider only, if flexslider is used together with normalize.css?
Trošku mě mate to <code>Tag</code> proč je tam velké písmeno?
so: every module that would like to use this abstract class has to do:
1. create a issue in zikula core project
2. zikula core extends DiscriminatorMap
3. zikula releases a new version after ? monthes
4. module can release its new version too

Do I understand that correctly?
`html_attributes[:selected] = 'selected'` ?
This is the only thing itching me at the moment.

This code above: ```relayout_decl (t);``` guarantees that declsize == typesize by zero'ing out DECL_SIZE (t) and re-evaluating it.  So that leaves the check for declsize < initsize, which can be changed to typesize < initsize.

```
if (int_size_in_bytes (TREE_TYPE (t))
    < int_size_in_bytes (TREE_TYPE (DECL_INITIAL (t))))
  {
    internal_error ( ... );
  }
```
```#120``` references an old ticketing system, so that part of the comment may be removed in the future.
It does make more sense your way, I'm going to look into that
Won't cut it, sorry. The profiles will break... hard..
s/String/#to_s. Symbols are valid.
@deprecated merged with initWithLayer -> @deprecated merged with initWithLayers
Again, loving the indents
Tässä riittää vain tarkistaa ovatko molemmat sekvenssit tyhjät ja jos ovat palauttaa `true`. Muut haarat hoitavat loput.
You can ignore my comment, leave it as it is.
Doesn't this mean that segments were lost? i.e. there's not a single segment to provide data?
If so perhaps add a warn?
Leading space? You mean the space I added there to match with just about all the other docstrings?  Note that I just noticed that I also should add a space after "array" on that line as well.
In another PR (I can't remember which one) you just added an extra `from matplotlib.blah import` statement at the beginning of the wrapped line. There was also a PR that used `(` and `)` to encase the imported items so they would wrap well. Either of those approaches trumps this one, I think.
This is going to conflict with my changes to boilerplate which were put on master a month or so ago. Is there a reason for basing this against v1.1.x? Can this particular change be avoided to the v1.1.x line?
These should line up with the literals on the line above.
The wait_for_element_not_displayed() cannot receive the HTMLElement obj as parameter, so using wait_for_condition() to replace sleep().
É possível utilizar o pacote lmodern para o uso da fonte Latin Modern em vez de Computer Modern super. Eu pessoalmente prefiro usar Latin Modern, mas mative a sua configuração. Há uma certa discussão sobre isso [aqui](http://tex.stackexchange.com/questions/1390/latin-modern-vs-cm-super).
`*request*` is defined in `response`. This can't work, right?
say this in the method name
This looks a little over-simplified. I would write something like

```php
    if (($info = $this->getHeaderInfo($i)) && $this->createTicket($info)) {
```
typo here. Missing ``s``
Dois-je utiliser le mot **namespace** ou plutot **espace de nom**?
Should I use the word **espace de nom** rather than **namespace**?
Whoops! ;)
Not a gaussian kernel ;)
Use checkNotNull
like this ;)
Jo pozor, vsechno jinak. Oznacme A' jako doplnek A, C jako count a a L jako last_relative_time a carkou doplnek.

Kdyz timer pretece, tak kolik ubehlo? C + (L' + 1) ... tj, C plus tolik, kolik chybelo do nuly (protoze L' + L + 1 = MAX + 1 = 0)
No a to je C - L, protoze to je vzorecek, jak odecitat v binarnim kodu pres doplnek (doplnek plus jedna a secist).

Zaver je, ze vzdalenosti se daj pocitat stejne, i kdyz mam preteceni.

Ale radsi si to necham od tebe jeste odsouhlasit :)
Diese Notation empfand ich als besonders verwirrend. Vielleicht sollten wir noch einen deutlicheren Hinweis  geben?
You fascist!
And requiring it here makes it clear that is what they need to do. If you just call handle() all they know is they need to know handle, they know nothing of the interface. This is just fine as it is now.
Are you sure ``new \Zend\Validator\Between()`` has the correct type as "object<Zend\Validator\Between>" is not a sub-type of "object<Zend\Validator\ValidatorInterface>"?
Are you sure ``array($this, 'filterTimingByCategory')`` has the correct type as "array" is not a sub-type of "object<callback>|null"?
Are you sure ``$output`` has the correct type as ``*`` is not a sub-type of ``string``?
Are you sure that the method "getForm()" exists?
I think `Unit` has a method for that test. If not it should.
How about calling this directory "jekyll" or "markdown"? I suspect we'll have other static-y content that isn't "docs" before too long.
var on these ones too :D
Cool.

@guedes You forgot to add `word` and can you please format so both expressions start at the same column?
you have an extra star here. And ``@var object`` is not precise. It should be ``@var Twig_Extensions_Extension_Cache_CacheInterface``
Subset Vietnamese?
Helluva selector. Do you need the `ol` in there?
Yeah, I typoed that.  Fixed now.
Ok, created https://jira.sio2project.mimuw.edu.pl/browse/SIO-1128
It is a leftover from an earlier version, good catch
I originally didn't have it, and it wasn't working, so I added it.  I just removed it and it works now... my guess is my initial <=> iterations had some bugs.  Thanks!
Indent one to less, to the left.
Can you move this to some top-level constant, and then use the same dict to update the `default_error_messages` in `form_fields.py`?

(Do you remember me saying that I do not want to see copy-paste?)
I just thought it was better to send these two variables like this, because I use both of them twice. So sending 3 variables instead of two seems wrong :)
Check other templates. Every Django template tag or HTML element we indent once.

```
{% block content %}
    <div class="registration">
        ...
```
we need source for displaying it
`;` at the end?
Why is one function written in camel case and other with underscores?
`response_type=token` is missing?
This part can stay here. So when comment is created, author is automatically added to the list of subscribers. This is something which is part of the resource-handling, yes.
All these strings should be constants.
Is there a way to use #defines for this so we don't have to manually update version numbers?
should this always be done? i.e. isn't this breaking stuff for people how would use __mysql_server in a generic situation?
Good catch. I meant to take that out.

On Aug 23, 2012, at 7:38 AM, Dan Crosta <notifications@github.com> wrote:

In meetups/forms.py:

>
> +    capacity = IntegerField('Maximum Capacity',
> +        [validators.NumberRange(min=0)], default=0)

Probably shouldn't have a default, right? I think we'll only end up
querying by capacity for claimed venues.

—
Reply to this email directly or view it on
GitHub<https://github.com/NYCPython/wheretomeetup/pull/22/files#r1443978>.
Are these working credentials we can ship with the library?
Delegate to resource?
this seems unused
This line and line 76 were always setting configfile to nil, no matter what was passed in as the :configfile argument on line 23. My changes make it so the configfile variable isn't wiped out if you set :use_config_file to true. There are definitely nicer ways of handling this logic, but I didn't want to make a lot of changes to the file.
> Often the JVM nowadays defaults to use IPv6 network stack

Nowadays the JVM often defaults to use IPv6 network stack
We don't log anything in this sub... can get rid of the logger.
If you REALLY want to keep it, s/pf::iptables/__PACKAGE__
I actually think of it as "the Kiji URI of an HBase instance", to indicate that no Kiji instance name or table name is specified. Hence the name hbaseURI.
Breaking whith space is still not defined. Can add a parenthetical comment like
>  (breaking in the sense of sperating the headers, which is \r\n)
"Virtual" function? Would put everything on the same line like usual.
l  (small L) is not a good variable name

txdat[l, :]  is this a 1 or an l
And it does explicitly state f(X_i). D'oh. Sorry for the noise.
Dont wrap  if not over 100 characters
This sets a unique ID on the incoming requerst so it can be tracked later in log files.
This was originally set (and uncommented) in __init__ => when you suggested moving to the app style you moved it quite rightly in here.  

I moved it back after reviewing the falsk docs.  Now the flask docs have changed again (Rel 0.9) your suggested approach seems to be the supported approach.  So its back in views (else the g object will not exist during initialisation)

It probably had a comment line once upon a time.

On a seperate note, the JanRain OpenID people seem to have fixed their server since I last looked as well.
I just noticed this in my recent edit.  I almost fixed it there but restrained myself so as not to confuse the issue. (I'm not as good as you at doing atomic commits!)

Part of the point of using the pimpl idiom (although not the main reason we're using it) is so that the implementation details can be in a different .h file than the main class.  Then if you decide to change aspects of the implementation, you don't need to recompile all the files that include CorrelatedNoise.h.  Only CorrelatedNoise.cpp would include CorrelatedNoiseImpl.h, so only that would be recompiled.  You can look at the various *Impl.h files to see how I did this for the other classes, and I would recommend doing it here too.

Like I said, it's not the main reason we use pimpl, but it's enough of an ancillary benefit that I think it's worth doing.
In the end I commented each instance (not just once per file), because there's no reason to believe that people are going to read sequentially through the files.  They might open the file up to search for a particular function and be confused by the cond/endcond that appears there (and this is particularly true since the cond/endcond stuff tends to be in long files with lots of classes/functions).

On Sep 14, 2012, at 2:51 PM, Mike Jarvis wrote:

> In include/galsim/FFT.h:
> 
> > @@ -51,6 +51,7 @@
> >  
> >  namespace galsim {
> >  
> > +    //! @cond
> This is a bit cryptic when seen out of context. Perhaps a comment each time (or once in each file) you use this saying that the code between @cond and @endcond are excluded from the doxygen output.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
> 


-------------------------------

Rachel Mandelbaum
rmandelb@andrew.cmu.edu
http://www.andrew.cmu.edu/~rmandelb/
Made this specific change here:

https://github.com/GalSim-developers/GalSim/commit/5e8c1737df1c64d1db4052b1e9c480ba0377159d

As for the more general rule, yes I agree that probably makes sense.  I've got to get on a bus now, but I'll make the push when I get into the office... Thanks!
That merely gets it to use the faster downloader since I couldn't get options working on it
I will add a few methods to my provider. Some of the attributes are required only for certain actions.
Agree with Minh. We need consistent internal storage and cents is probably best
Get rid of the example routes
our StatusListener is gone :(
static import of "of" is confusing I think. Other method names are intuitive what they'll do, but "of" is not without the classname as well.
please put commas at the end.
C''est pas un peu fort de juger des projets comme gunicorn et Graphite en disant `it's bad` ? Faudrait ptet plutot tourner ça dans la forme interrgoative genre: Ils doivent avoir de bonnes raisons de faire ça mais j'aimerais les connaitres/ quelles sont - elles ... un truc commeça
That's not the correct signature for this function: https://developer.apple.com/library/mac/#documentation/graphicsimaging/reference/Quartz_Services_Ref/Reference/reference.html
are those /> really necessary? i thought the magic of html5 was you didn't have to close stuff anymore

:trollface:
This was always in the 0.6 -0.9-ish range, so I just bumped it up for now.  My machine is dragging from the deals suite anyway, so likely related.
Yes you're 100% right. It just creates a new File instance. Disclosure, I did verify it by looking at the actual [Play API for getFile](https://github.com/playframework/Play20/blob/master/framework/src/play/src/main/scala/play/api/Application.scala#L286). :) 

I couldn't find any info that getFile() does a stat or any sort of IO when it is created.
I think there's a typo: `hieirarchy` => `hierarchy` :)
I didn't know you could do this — amazing.
+1. I will test and merge if all goes well once I'm on the ground.

On Fri, Aug 24, 2012 at 4:43 PM, Ewoud Kohl van Wijngaarden <
notifications@github.com> wrote:

> In templates/server/puppet.conf.erb:
>
> > @@ -4,6 +4,7 @@
> >      reports        = foreman
> >      external_nodes = /etc/puppet/node.rb
> >      node_terminus  = exec
> > +    ca             = <%= scope.lookupvar("puppet::server::ca") %>
>
> That sounds like a good thing to support as well and I think this change
> only makes it easier.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/theforeman/puppet-puppet/pull/9/files#r1454099>.
>
>
I don't see a big deal for the .jay files (in xp.public, there's just src/main/jay for this).

For the .ini files the question is: how will the classes find it
a) when in a non-maven setup
b) when in a maven setup

?
ANother option on this whole thing is to allow BanditAlgos to provide lists of configs and let the bandit algo sort out how it will compute them.
i might just not be familiar on how to get objects from the html document except to store them in a string, int or other primitive. is there anyway we can pull those variables into the code in the event_create method in the Events class without them actually being an instance variable in the class? 

i did it this way b/c i was basing this off of how other people were passing objects from html to java
missing space after the first `,`
у тебя тут 2 пробела, а везде 4
String.split?
"this" is a reserved word in c++.
s/login/log in (sorry, English major -- noun vs. verb thing!)

and while you're here, s/paypal/PayPal, because I'm pedantic
Looks like you messed a merge
要に「Grailsプロジェクトのルートで」と補足をつけたのは何か思いがあるのかな？
Grailsコマンドは、ここまでのリファレンスの中で何度も実行しているだろうし、若干説明が冗長な気がしました。
One thing to note is that I had to wrap the movie drawing code in an `if (movie.isLoaded()) { ... }` statement, to give the file time to download during the first few cycles.

Do you think it would be better to just create a separate example for URL loading?
I would say CHEF-2748 is right on the money.  I guess it is a bug in Chef, as one expects the ability to access data structures in a consistent manner.
I like driveLeft and executeLeft. Make that change real quick and I'll merge this in.
In the issue rather than on the wiki.
connectivity matrix
Ok, I'll change it to tell the user to use ``svm.SVC``.
n_jobs is not documented
https://github.com/sebastianbergmann/phpunit/blob/master/phpunit.php#L40 shows the right way to perform this check.
я б лишив як було, але 
'wallpaper[categories][]' => $category1->getId()
'wallpaper[categories][]' => $category2->getId()
або 
'wallpaper[categories]' => array(...)
не працювало, вирішив робити як в документації
Random spacing change? :)
unintentional reversions
It's a bug. It's good you fixed it. Just don't put it in a comment please. How would the codebase look like if we always put a comment that something was wrong in here?;)
use forward declarations for those you can. Add appropriate headers in cpp
Weird. I'll take a look at it in the next release.
Switched it so if the _cacheSchema is nil that when we access it, we will create the default one.
There is a risk of confusion in the given exemples 4 termsOfuse : 

Why :

"termsOfUse": [
 	  {
                  "locale": "ar_SA",
                  "terms": "ar_SA" /* Check this */
           },
 	  {
                  "locale": "de_DE",
                  "terms": "Ich bin ein Berliner" /* What is the correct example ? */
           },
]
@matthuhiggins Ah I am not very familiar with active_support, taking a look now, will send commit soon :)
rigid mind ! ;-P
This should be looking at the annotation type, and generating the annotation string depending on the type.  It is even possible to have two of the same annotation on the same expression (not particularly desirable much of the time)
Yes, but it doesn't call init on hash tables 0..numHashTables so they will stay at their previous (potentially large) size.
Indentation doesn't match the rest of the function - although to be fair the existing code is also inconsistent.
You haven't addressed this comment as far as I can see.
I think this is going to be unsafe when the ulimit for file handles has been raised (as we do by default). Would need I think to use the XFD_ macros as defined in jsocket
Is there anyway to typedef this struct? For instance,

typedef struct tempodb_config {
    char *access_key;
    char *access_secret;
    int sock;
} tempodb_config;
4.5 seconds is a pretty low default.
You wrote C instead of G. ProbeAllGpus "false" will make it disappear in nvidia-xconfig if I understand ftp://download.nvidia.com/XFree86/Linux-x86_64/304.48/README/xconfigoptions.html correctly. Is it really necessary?
i see here also the missing of break could it be also due to the php5.4 decision?
actually, open() returns a ref that you can call close() on. My intention was to mimic what window.open returns, which you can call close() on
Isn't that in the old version?  I don't refer to field_values[field] anymore in the new version.
i assume the dot here is a typo?
contexts.reserve(numbers.length())
Get the inboxes of all enabled accounts in the store.
this doesn't appear to ever be set
Most likely not, as that would eliminate the {'limit: 200} hash record.
Hi Yoav, the Session Title and Abstract is merely a place saver... The real title should go here.
For example, in this case:


    Building a Web Infrastructure for 10m Users
    ==================================
    Over the past 4 years....


Anyway, I'm going to merge that in but I'm just saying for future generations....
Yes, you're right, will push the fix shortly.
yeah, ignore these =)

I was confusing @ and $ for some reason.
Ups. Do you mean X.times do ... end block? :) Yes, this is my mistake. Need to initialize :app_env outside of cycles.
header dige lazem nabood
yeki azina kafiye ke 2vomi behtare
* `header nav`
* `nav.menu`
I would have never in a million years have even noticed they were in the same order let alone think it an intentional and relevant convention.  I'm not sure how it really helps anything.  Maybe that's just me though.  I can put it back if you really want.
We need to clearly document that a call to `addImages` or `addImage` could cause a texture atlas recreation, which invalidates the texture coordinates.
Applies throughout.
Should these have a `@type`?
As I said, I don't think checking the event target is the best way either. My reason is that I am not sure if such comparison is sufficient in all cases. Being not able to devise a better way, I pass payload instead of target alone to allow checking other data.

I think your time could be better used in proposing a better way and try to implement it. There is no need to argue about how `target` is passed in when you don't like total idea of checking the event target.
Fare enough. As I said before, I don't insist having such a method, as long as you understand what this method does, and you don't like the idea of assuming such a naming convention. After all, it is just a helper method, and it can be replaced with several calls to self.bind.
#782 to track that
Comments for `add:` for module docs now reference the wrong method. This needs to be moved.
better to check against void 0
take a look at http://developer.android.com/reference/android/graphics/drawable/DrawableContainer.html#getCurrent()

This might provide what you need.  Given that this code runs when tabs are changed we should try and be as efficient as possible to avoid any noticeable delays.
please use IPA header from setup repository
As with the "checked attribute, the "name" attribute is a part of the variable TPL_INPUT, which will be moved into the template.
Podrías remover ésto? Este tipo de configuraciones es mejor tenerlas en un gitconfig global propio.
Я видел, чуваки так делают и не стращаются.
config.h shouldn't be in public headers
In this case, using ```bool``` instead of ```auto``` is clear for me.
But it is subjective.
I'm not asking it again. Judging by the discussion, the `= T.init` should be there.
I think people, when writing their own predicate-based code, will not go with string lambdas since the new lambdas have so much going for them (use of locals, unsurprising syntax). So the scenario in which people choose a string lambda for their own functions and THEN use that string lambda with commonPrefix is pretty much low on my priority list.
Ah right, that's actually fairly tricky.
I wonder if this should be made working through SFINAE, as the compiler might know that instantiation failed because you tried to call an unsafe template argument in a safe function. I might file an enhancement request for dmd.
You could probably also rely on `@safe` inference for this function if you can omit the explicitness, i.e. `@safe` should propagate from the template argument to the function.
Why are you assigning to the same variable twice. Also, is there any way we can avoid going after a 0th index of an array here?
`true` to `TRUE` ... makes it more readable.
Well, it might be if the surrounding code doesn't catch exceptions properly, but it's damn useful when things aren't behaving as you expect!

I supposed it could be an optional setting to print the connection details when the DatabaseException is converted to a string?
Haha not sure if you're serious or joking
ok fixed... the rtrim functions I just copied it how it was done with $system_path....
these tests are broken on 5.4
http://travis-ci.org/#!/FriendsOfSymfony/FOSRestBundle/jobs/1834684
Except for the salt, you can use the `defaultValue` attribute in the `schema.xml`.
you should remove the extra empty line
I am still trying to figure out how that ended up in the pull request and my file copy, sorry I guess it would have been my mistake when I was debugging the code and adding lines to figure the data structures. Wont happen again
Message should contain  : date - time, log level, Logger name
Not sure I follow this one. Why is it only safe if B is not connected?
See: http://nhforge.org/doc/nh/en/index.html#manipulatingdata-querying
This is changed now.
Right, so we probably want this (taken from another place in ModelAdmin.php):

$validator = ($this->currentRecord->hasMethod('getCMSValidator')) ? $this->currentRecord->getCMSValidator() : new RequiredFields();

Thoughts?
OK)
Why are you opening the file in append mode - the 'a' option? It should be 'w' so the contents are overwritten.
hasText?
If you making this bean BeanFactoryAware than what's the purpose of this constructor? I see your comment, but it doesn't feel like Spring and by simply removing it you can eliminate about 20 lines of code and comments above ;)
Remember we want to stay consistent with what we already have as well as common patterns and styles in Spring, so I'd simply remove that
Whist the name amuses me, I think that should probably be `$execute_hooks` or similar, and a similar `$register_hooks` could also be provided (see the other note further down)
What do you mean? It iterated in the array order. But I find it much more readable and useful where speed is not important.
I changed it to >=1.4.1. I did not test it on earlier versions of Django. Do you think it's safe to lover the version?
тут перенос строки добавить, слишком длинная
The exception should be kept and the log entry should be removed.
i am going to use new use keyword on each line as this way is used in all other files.
This certainly gets the job done... Notes:

* The Game knows a bit too much about the dealer's hand --- going in and grabbing cards.first.suit is a sign that it knows too much
* Instead, you could change the dealer_hand to be its own object, say `DealerHand` .. and maybe change it's initialize method to instantly convert the first card to a HoleCard.
Who will be hacking on this idea during the Hackergarten?
Hack idea developers?
Thucydides and Drone Hackers/Coder/Developers at Devoxx 2012.

List of developers @ event

Then could have a sentence listing people available online/afterwards if applicable
readFileSync doesn't take a third argument, meaning your error callback will never be called.
why do you need to save escaped name?
There's a surplus tick following omero//log
Use ```R.string``` for this and the one below.
``null`` is the default so you can simply omit it
Yes it was exactly that,
Their was a tabulation, and I have changed it in 2 spaces.

I will try to use only two space in the future.
{前要有一个空格
is this really necessary? If it is please explain why?
Mmm.. good question. I don't really know. But to make sure we could assign the value to a variable, and then create the instance.
Fair enough, but I still think it's misleading.  I wouldn't take django examples as gospel.  It's pretty minor though; I don't really care.
```php
->select('Id')
->count()
```
Pushed a removal of that dd.

On 2012-11-09, at 3:39 PM, Jonathan Green <notifications@github.com> wrote:

> In islandora_pdf.module:
> 
> > @@ -104,6 +104,7 @@ function islandora_pdf_islandora_required_objects() {
> >   */
> >  function islandora_pdf_islandora_sp_pdf_islandora_view_object($object, $page_number, $page_size) {
> >    $output = theme('islandora_pdf', array('islandora_object' => $object));
> > +  dd($output, "PDF VIEW OBJECT RESULTS");
> looks like a dd slipped in there.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
I don't get why the runtime notification appear here ?
I think this line is not necessary, because $textarea.val() is also copied to $origin in `grow` function at line 93.
I checked it working without this line on several browsers.
me refiero a esa linia, la de "Forum['Forum52'] = " ";" no hay ningun texto traducido
WHy are you updating the framework hash?
Yes, added validation
how will you pass memory data to lint and jasmine for linting and testing?
Code zegt iets anders