labofsahil/github-event-dataset-2012 huggingface dataset:
To be consistent, we've been calling internal caches as: ___xxx, see ___defaultcache, or ___hotRodTopologyCache
Descriptions need updating...
Ah Ok, sounds good.
This is a manual stress test, I can see the point here...
`public` modifier not needed on interfaces.
you should add it in the suggestions too
Can you please style this as `obj.sha` instead?
I know this test is disabled, but this isn't semantically equivalent. This should use `STAssertEqualObjects`.
Would the ```.. note::`` directive render things nicer here, or is that too loud?
Is this hard-coding intentional? Is there a reason is doesn't take the default dpi from rcParams?
Thanks for the feedback -- I've amended with a docstring
This will not work:

ERROR | profile_args.setdefault('preferences', { })['extensions.jsbridge.port'] = self.jsbridge_port
ERROR | TypeError
ERROR | :
ERROR | list indices must be integers, not str
Done: see https://github.com/openlayers/openlayers/pull/637
Why is this commented out? If we can rip it, let's rip it. Otherwise just a quick doc explaining (e.g., a workaround? crud?)
Also, the way it's defined it doesn't need "vec" around it
The medium-term plan is to build on the parent class to eliminate the blockages & boilerplate of the current SolrSearchServlet + search template nodes + providers/processors scheme. So I'm trying to keep the template-oriented interfaces out of the base class so as to keep the boundaries a bit clearer.

Since "loadUserProperties" is a nonfunctional pure jump-through-the-template-hoops method which wouldn't exist in a pure servlet world, I'd rather keep it outside. "configureQString" _is_ functional, however, and would be needed in any Solr query builder, and so I put that method in the parent class.

That said, it might work to put an intermediate "template hoops" class between the functional base class and the domain-specific classes -- I'll keep an eye out for that pattern as new subclasses get added.
remove comment please
Yes; the original authors had identical copies of the migration in vanity/lib/generators and vanity/generators. I don't see an immediate use for it but decided to stick to their structure. Should I just remove one?
There isn't any official documentation of it that I could find. However, libraries like sigar, code samples from the msdn blog, and just about every other Windows uptime snippets on the internet that use performance counters instead of GetTickCount, all use 2 for the "System" counter type and 674 for the "System Up Time" counter id.

FWIW, the built-in counter types and values can be seen in: HKLM\SOFTWARE\Microsoft\Windows NT\CurrentVersion\Perflib\<LangID>\Counter
I usually like a struct for this sort of stuff just to keep it more compact. e.g.

    struct ssl_verify_error_map {
        long error;
        char *string
     };

    static struct ssl_verify_error_map error_map[] = {
        {X509_V_ERR_UNABLE_TO_GET_ISSUER_CERT, "UNABLE_TO_GET_ISSUER_CERT"},
        {X509_V_ERR_UNABLE_TO_GET_ISSUER_CERT, "UNABLE_TO_GET_ISSUER_CERT"},
    }
Yes, the data is contained in a single location. Adding/removing options will require just adding/removing a single line rather than duplicating a whole block. It isn't uglier than having a dozen blocks that do al exactly the same thing, but its a lot better to maintain/understand.
$config->get() is an expensive operation which you shouldn't do in a loop (_writeIndex() is called in a loop when retrieving the categories). You should set this already at the beginning of the calling method.
@denniskeefe - Does the group have an email address that we could use?
I added this so people would be forced to read the README for install options
In my opinion, the fact that the user can be trusted or not is independent from the use of the module.
So if we are using clone here and clone uses IO streams... would that not be quite a bit of performance overhead... I assume yes and then I would think that this fix is not worth it in the sense that newInstance is not that often used I assume but the overhead for fixing it this way is quite high.. too high imho
I did notice. ;) Other parts of the code call MakeMultiException and test for ordinality, so I don't think it's confusing.
Parfait, je comprends mieux. Pas de soucis à ce niveau là
You still don't store time when activation key was generated.
Do you maybe have whole this function just because you do not know that you could do `self.layout.get(columns)`?
No empty line. This is the same package.
Initialize? Initialize panels layout?
This code is really necessary? You are just creating a dict from ... dict?
Will be this OK? 
Code: blur(checkPassword).keyup(checkPassword).change(checkPassword);

if ($('#id_password1').val().trim().length > 0 || $('#id_password2').val().trim().length > 0)
But here is OK. Because you want for second `get` to operate on dict.
I hope so. I will not ever check. ;-)
I don't see the reason to use one more global variable. Why we cannot simply `$('#search_users').val()` in the code above when we need its content?
What is the added value of this?
Do you mean "host will provide food" or "it's OK if someone buys food for everyone"?
Die folgenden Methoden kann man eigentlich löschen. Hab ich am Anfang nur als Erinnerungshilfe erstellt.
What is `db_driver`? Does it needed when we has standard user provider support?
- the _toString_ implementation is indented via spaces instead of tabs
- map.size() is missing spaces around it
- the first append is having an initial space, but not on the closing paranthesis
So new objects go on the end rather than the beginning? Does this change need to be documented?
Ok, let's go with PartialDate and see if we can evolve it later. Now that we're changing the API I would also suggest switching from DateTime to LocalDate: http://joda-time.sourceforge.net/apidocs/org/joda/time/LocalDate.html
I did this locally too - Haven't fully switched to VS2012 yet, and until there's a need to use 4.5 features, might as well stay on 4.0.
"until _the promise_ is resolved"
private method doesn't need to validate the input, the validation logics should reside in the public method.
There shouldn't be a t() on ```<h3>```
However missing t() on text.
Extra space before `new Dict()`
Does it still, after these changes?
Maybe if we only change Ship::Refuel() method on line 244 of Ship.cpp by the code in InfoView.cpp of issue #1649 after it gets merged, it will be OK? The Refuel() method of Ship class is probably useful, whether it does the refueling on its own or asks lua to do it.
Yes, the earlier if is reading a plist from disk to get the API keys.
yeah ist just a 'capricho' you know a _psychorigide_ think of mine to have this as the first line don't worry about it ;)
No, it was renamed. This is the old name. Somehow the add got into the last set, but not the delete
We'll go ahead and get them right here (since lightweight isn't in yet).  But for DS we will omit them.
Those are some awesome keytars, where can I get one?

![224081_453380174718095_2116520284_n](https://f.cloud.github.com/assets/102889/8110/98fd05f6-4463-11e2-9765-ab87867b1dae.png)
Yeah, that was my js noob breaking out for a bit.
I'm pretty sure the std:: isn't required, but I guess it doesn't hurt anything.
I think yw should be in group "optional".  It's not required.
This looks like useful new information for me.  I wasn't sure how to make the brief description go on more than one line, so my lines are too long in some places.
also, you're definitely missing code in `edit_exam`, though I think you're aware of this.
That line is not jslint compliant.

Wrong:
obj.get( 0 );

Correct:
obj.get(0);
All looking good except that I don't understand this comment. I don't think anything is being put back; rather the "App" page is being removed.
That was my first preference, but Object not supported for types of annotation attributes:
``Invalid type Object for the annotation attribute SetFromFlag.defaultVal; only primitive type, String, Class, annotation, enumeration are permitted or 1-dimensional arrays thereof``

So I think String is the best we can do.
Yeah - I've seen experienced programmers have trouble with `(a b).c`, but that's a result of `a(b).c` obfuscating what's actually happening. Definitely with you on `(a b).c` being more clear, and I don't remember why I moved away from it.
うーん、こんな感じになっちゃうますねぃ :-<
No route matches [PUT] "/users/2/configs/00002-hoge.en.ja"
This test should be marked with --XFAIL-- referencing the problem (and preferably a bug report :))
That's neat. I added it.
I think you do need it in quotes.
I think this can open Python files, which aren't necessarily UTF-8, although most are. I was preparing code somewhere to sniff the encoding magic comment and open a Python file accordingly, so let's just make a note to hook that up to this when I add that in.
you should remove the leading backslash here too
Can you fix the width?
Remove this line and the later ones.
e.g.:
```Charset UTF8 = Charset.forName("UTF-8");
...
...
new String(responseBody,UTF8);
...
```
I could go either way on this. 

On one hand, they mean the same thing.

On the other hand, HttpException is something we expect our code to throw, vs Exception being a wrapper for any of the nonsense other libraries may throw.  If we remove HttpException, we lose this hint to the user.  Also if an IDE is examining the code it won't know to suggest HttpException.
yeah they are the same method.  one at the instance level and one at the class.  we should probably move this up to the class level but that refactor is not part of this scope.

We can add a comment as a TODO.  good call though.
А почему это неправильно?
Надо экранировать строку, а не кавычки.
How does replace differ from namingExtract? the code appears to be the same, so I'm just wondering whether it's easier to just think of it as phased replacements, with namingExtract = phase 1, and replace = phase 2.  If that is the case then could it be that some people would want a phase 3?

If that's sensible then could we just have nargs='*' and use re.sub(e[n+0],e[n+1],work)  for all n in nargs/2
`Project`クラス群にそれぞれ`#source_directory_path`を定義しておけば、ここの条件分岐は要らなくなると思います！
Ok, so, `input`... then I could change `supportsClass` in the interface to `supports`, and the `FormTypeParser` could check to see if it has a FormType for the given string, whatever it is.
Could you explain?  I'm sure this is correct, I'd just like to understand.
What do you mean by "during this time" ?
Why is this particularly useful for binary classification?
Because the the nonzero values change at each iteration so if I dont copy in the end i would stack 2*n+1 times the same array
No need for whitespace under our styling guidelines [1]


	if(answer) {

[1] https://github.com/wikimedia/WikipediaMobile/blob/master/STYLE_GUIDELINES
This test really belongs in druntime, not dmd.  And tls addresses are not win32 specific.  But nothing is really TESTED here, other than execution.  It could fail in all sorts of ways, and fail to fail.

Would you file a bug report against this test file so that it has a better chance of being fixed for real?  I don't expect you to move it and fix it during the win64 work.
I don't think this is ever used outside of json.c, so it should probably be in there instead.  These are effectively internal functions.
`inout` qualifier is a kind of `const` qualifier, and they are exclusive. So `non-inout` message is a little unnatural to me.
I really hate to see more error gagging, there are many unresolved issues with it. Can't this be done more directly?
As discussed, ip validation removed to allow support hostnames as well, i.e. just pass the param to blackberry-deploy
Les documents indispensables à une étude sont insérés dans la BDD lorsqu'on crée une étude (dont la facture). Ce cas n'arrive donc que si l'on tente de récupérer une facture en jouant avec les paramètres passés en GET : ça paraît plus légitime de mettre un badRequest histoire d'éviter que des petits malins ne s'amusent trop :)
Pour les factures des études pré-Play-Intranet, il y aura sûrement plein de problèmes : à voir si on gère tous les cas dans le code, ou si l'on modifie la BDD en conséquence, au moins pour les études dont la JE est encore responsable (5 ans je crois).
that's an interesting example, but it looks like a word is very expensive for you to write. Could we add a little context, like "For example when results from US, Europ and Asia collide because they share the same latitude, the ..." (what happens ?)
memset is preferable, more portable I think
All the following code can be inside a single if(max).  comma can move inside as well.
>Rows are added to rowsToSort via the append method, which does not take ownership (ref count stays at 1).


Yes, that _is_ taking ownership, the rows from the input activity (via input->nextInGroup()) are added via 'addRow(next)' to rowsToSort.
rowsToSort "owns" those rows.
Best to remove altogether I think.
You also want to cope with unsigned decimals (udecimals).
I suspect the windows compiler (and others) will complain about loss of precision.  You should cast the final result back to the type of maxRowGuess.
We don't need to continue if we have hit the end
where can i find this header file???
Shouldn't that be 
`releaseChannelsContainer.html(crashReportsHTML);`?
I don't feel it's an exception if there are no restart tests, so preferred it when we skipped running them if the path didn't exist. That said, once we're migrated to Mozmill 2.0 we will be dropping this concept of restart tests, so any exception here is temporary. Fine by me either way.
Shouldn't this cast to (bool)? (same for line 269 below)
This can be removed as you already check this few line above.
try catch. in the failure case this will not halt your test.
remove comment ?
Funktion `alive?` voi antaa `filter`:lle parametrina sellaisenaan: `(filter alive? authors)`.
If think it's better to display the title here. Displaying the company name here is not very accurate.
Not sure, it's more about developer stations setup.
res should be checked against nil I think
I imagine you're using send_request_raw to avoid uri encoding, did you test if you can use send_request_cgi? Would be better to use it.
```store_file```, please.
I think it's more consistent with the conversions to other unsigned integers (treating False as 0, and True a 1).
It's not better, just an alternative way of doing things. If it doesn't work though, I've no problem joining and stripping.
please can you add 2 spaces in front of the return statement
well, I know Edgar, but why adding it in the output ?
I thought about doing that.  OK.
Why are we still slicing this?  This is not reliable.  Is the GUID used for cache lookups based on the material's `type`?  If so, just use the entire GUID.  It won't hurt performance I promise.
Is this the assert? Naming here (RuntimeHelper) sounds like a class used during execution, not at test time. Hard to tell what this is actually testing.
ContentProviderOperation has an update operation.  Is there a reason we are not using that instead of deleting then reading?
The root directory is not a good place for this file. I'm not quite sure where and how this is currently used in your code but my guess is that it should be in `db/src/routemap/common` because it should be the DB update function that calls it.
The actual spec for local.redis starts here.
comma never starts a line
important*? imported*? Whichever you meant.
Can you explain why you added these two lines?
I thought about it, but I don't really like it.
Please always put  brackets { }, also for one-liners.
this wont work. A title of undef will result in an undef file.
I just changed updated this in the lib master branch so it reads "Google API project number".
Because there is no `Registrar` object for the overall view. (`/registration/registrars`)
This is not needed anymore:

- the RouteVoter is now enabled in KnpMenuBundle
- your implementation will break the use of Twig in the CLI as you would be getting the request outside the request scope
remove `jboss-logging`
Есть же `clear ()`.
extra space after `(`
We should pull this `$.map` out into its own variable declaration, to make debugging easier and readability better
where is v cleaned up / destroyed?

I think you need to call v.destroy_vector() at the end
you have three states: rollback, commit or nothing. I can create a method like isMarkedForCommit() instead of using the return value
I don't know why you want to remove this. I went to a lot of trouble to find the information in this module. (In fact the value of this module is more in information, than anything else). Really they are comments (and should probably become comments). They actually serve the important purpose, of trying to avoid this module being x86-specific. Once again, druntime is *NOT* intended to be simply be the runtime for DMD.
date filter validate
maybe getIdleCount? (size makes since for the pool itself, but here it's really the number of items in the pool - not checked out).
Not important - we can handle it with the others in 3.0.
s/~~In a sense, a/A/
Thank you, Gary, for all your comments.
About my english: I'm still not sure in its ideality. So, it's wonderful, if you all are correcting me ;-).
About RHRA: I've done it intentionally, you gave a comment: to be or not to be RHRA in this case, or the more in the `<poller>` or AMQP container 'advice-chain'.
However, as I see you start documentation issue about RHRA and I'm looking forward the usage of RHRA!
I'm going to remove this bean from sample. So, it will be saved from 'can of worms' ;-)
Is there's a way to centralize the parameter changes so when u change parameter that affects all subsequent places?
s/the IRC/IRC
s/of via/or via
This might be a dumb question (I did no research :) -- where did $::openstack_version come from?
I added the .gitignore on each directory because I need empty dirs.
Should this class be prefixed with sketchname_bundlename ?
It's highly advisable to put every binding on its own line:
~~~clojure
(loop [a-seq
       sum 0
       amount 0]
  ...)
~~~
i thought of it more as a method of "flagging a package/assembly for use by Fody" rather than defining it as part of the Fody product.
I did not want to overly pollute what the name/branding of the addin
Ok, I will merge the two classes. Nice catch!
I think the CCC Number Module calls this cccgenerator.CCC now ... is that right?  Make sure you double check with the latest code because I think I committed changes that matched what comes from the CCC module.
No problem, I just thought to keep all highlight files grouped together (it files that would go into css).
Could we go back in history and figure out the exact date ? It is in the repository...
Sample or not, should this be dummied, or is it already?
After this, we can create the view, so the end-developer can just use it without calling .as_view()

obtain_auth_token = ObtainAuthToken.as_view()
the code generator just moved it around, it's still here somewhere
reverted it now
And what's wrong with this one?
TIL about `pluck`. Awesome!
please use ``2.0.*`` here
Only flush once
вместо глобалов можно было сначало объявить функцию, а потом ее использовать те  bubbleSort опустить в самый конец скрипта
Why are you converting a perfectly good use of a named route back to the old format?
( String className)  has extra space throughout
Perhaps another constant? `$asacct.DEFAULT_TIMEOUT_MS` is the ActiveSync-specific timeout value.
To save on confusion I wouldn't use 'is' here. I get that None is a singleton but this just doesn't feel right.
Not sure, haven't look into the calculations of horzontal layouts. But that is outside the scope of these bugs
Space needed in `gesturedrag`
This is a bit problematic to implement. The Adminstrator SID contains a domain component which is unique, e.g. `S-1-5-21-887289371-3252031711-946894838-500`. The last component (RID) is always 500, but there could be multiple such accounts, e.g. the local Administrator and the domain Administrator. The other issue is MS "best practice" is to disable the Administrator account. In which case, the "C:\Users\Administrator" directory may not even exist, which could lead to failures if trying to managing ssh keys for the 'Administrator' account.
This is omitted in the setup scripts on purpose.
It's a somewhat unfortunate fact that one cannot use macros as high-order functions. And `and` is a macro in Clojure, so this won't work.
header guard should go before #includes and should have a space between the comments
this was preventing the list to be emptied when the store was emply after filtering
we don't distribute their software in source code form, only in object code form (there are only DLLs), so the licence wouldn't be necessary, but i still can add it to their folders if you want
Maybe throw service unavailable instead of deadlock?
Yes, but I need to check first if they are not needed. Just removed them here because Thomas said so ...
(< 12 age 20) palauttaa sellaisenaan booleanin
Hmm... I'll refactor then.