labofsahil/github-event-dataset-2012 huggingface dataset:
I would prefer if we could simply avoid 'set_cmap': it has nasty side effects. We can use the 'cmap' argument to the different plot function, such as scatter.
AFAIK, cloning is currently not implemented with pickle. Would be better rename this test to avoid any confusion. A test that does real pickling would be nice too but I am afraid that lambdas are not picklable hence the callable kernel would have to be defined in some python class wrapper instead.
I guess you mean something related to a stride issue, like accessing a row in a FORTRAN continuous array.
But I don's see see this issue here, since the loop over the coordinates remains the same, so this issue can only occur for each iteration of the outer loop. 
Also iterations on the coordinates in random order would have the same problem, but this has been mention to have a positive impact on the convergence. 
Is this what you are referring to?
Maybe not. I'm confused. Shouldn't the formular be like that, though?
Plenty of them:

```
git grep warnings | grep test
```
useless comment sir
As a note, we should allow multiple `TEST_OUTPUT` section by the improvement of `d_do_test.d`.

```d
/*
TEST_OUTPUT:
---
Error: 'error' is undefined identifier in foo
---
*/
void foo() {
  return error;
}

/*
TEST_OUTPUT:
---
Error: 'error' is undefined identifier in bar
---
*/
void foo() {
  return error;
}
```

Actual output result will be tested with merged result of all `TEST_OUTPUT` sections with lexical order.
Oooh, there is now a `--default-password ''` option that can be passed so you don't have to be blocked and press return.
this isn't updating prev in the same way as the roxie version does when it transforms a row.

Test cases should try and cover that case as well.
I don't get why is this called ```elseDs```
Kræsjer i prod!
Consider starting with just `setHomePage` and adding more specific stuff like `setDocumentation` and `setChangelog` after you've demonstrated an actual need.
so, locally this is all good now but, how do I get this out of the PR?
How is `signature` not required?
Goes against what I see here: http://socorro.readthedocs.org/en/latest/middleware.html#list-report
These should be in single quotes as per the Puppet Style Guide
Can you enclose type, uri, ... allowPlugins with backticks ?
Like parent and name from newTopWindow.
@gozala, you are right about `tab-browser` it isn't necessary relevant as we really want to drop this one. It just sounds like we let them wander until we come up with nice low-level modules.
That's ok to let some fuzzyness for tabs and so avoid talking about `tab-browser`. But I won't say the same thing about `window-utils` and its `WindowTracker` class which has been proven to be usefull and working good for most addons that used it!

@wbamberg if you want some examples, you can do whatever you like with browser UI by using WindowTracker. Here is a set of real-life examples:

  * Hack existing context menus
https://mxr.mozilla.org/addons/source/307406/resources/jid0-pojopto6sue8jwiqcgvygyaahfm-legacy-link-context-menu-lib/main.js#19
  * Add an icon in the url bar
https://mxr.mozilla.org/addons/source/360550/resources/copytinyurl/lib/main.js#5
https://mxr.mozilla.org/addons/source/349294/resources/nctutab/lib/main.js#197
  * Hack reload button
https://mxr.mozilla.org/addons/source/5200/resources/reloadplus/lib/main.js#128
  * Overload chrome JS methods
https://mxr.mozilla.org/addons/source/302182/resources/clear-the-search-bar/lib/main.js#1
  * Listen to chrome events not already expose via the SDK
https://mxr.mozilla.org/addons/source/335961/resources/jid1-xeaurv8gzibudw-at-jetpack-peek-lib/main.js#145
Лучше путь делать относительный `/dz-6-async/current-event.json`
Does the "alwaysRun" value add performance overhead? In CX - my test class opens the browser once on the first test and then continues to use that open browser for methods inside of the test class.
one used to be able to go thru the meterp route (Probe?) the other not.
I guess PEP8 doesn't really apply to Cython files, but my editor soft-wraps at 80 characters so it's a bit of a visual annoyance.
I don't think `NotImplemented` is an exception. You probably meant `NotImplementedError`, but this exception is meant to signal that a subclass needs to redefine a method in a base class. I think the appropriate exception is `ValueError`.
What do you think about making this call synchronous? That's what we do for completions, so the user would probably expect a blocking call here as well.
PEP8 likes a new line at the end of the file 
Can you add it pls
Can we invert these ifs to reduce nesting?
We can try and justify it all we want, the point is that we have completeness criteria that hasn't been met.  If the team is okay with going against our own guidelines in this case, fine.  Just call it what it is, breaking our own rules.  If @shunter @pjcozzi and @kring are on board merging despite the DOC TBAs still in place, then that's fine with me.  I'm not trying to be a blow hard, I'm just trying to enforce our own guidelines.
So what's your take?  Rename mock elsewhere to fake - or leave it alone because it doesn't really matter?
If it's just a line or two of code, perhaps we should just include it. Like

`menuexample.js`:
~~~
var doClick = function(e) {
    Ti.API.info("Menu item clicked: " + e.source.title);
}
~~~
OK, this is weird. I checked to see whether we had a bug for this on Android, and came across this:

https://jira.appcelerator.org/browse/TIMOB-7967

It appears that textContent is supported on Android, but only for Element and Text nodes (WHY?).

It also appears that since API level 8, Android's DOM implementation supports setTextContent and getTextContent:

http://developer.android.com/reference/org/w3c/dom/Node.html#getTextContent()

Maybe there are bugs in the Android DOM implementation? Or maybe we just didn't use this because our minimum SDK level was 7 when this was originally implemented?
I think this should be "Set of ACL objects" based on other usage.
This statement can throw 
IllegalCharsetNameException - If the given charset name is illegal
UnsupportedCharsetException - If no support for the named charset is available in this instance of the Java virtual machine
See comment on TiLocation.java:48
уменя есть вариант по умалчанию 
'''eventOptions.color = color || "#fff";'''
Should be `if (value === "")`
They are different, one is on fields and other is on availableFields.
Why is it "munging"? :)
Слушай, а почему мы вообще используем чужой jQuery? А если он пропатчен? А если обратная совместимость?
should this newline be here?
CTFE ?
Shouldn't this be: alias std.numeric.CustomFloat!16 HalfFloat; ?
yeah - it is to me too. Initially I was thinking that we don't need it, just use top-level symbols. But it helps make the main page look more similar to a normal component.

'BodyComponent' was a hard-coded name, I changed it to MainComponent and added a binding in the input html (at the top of the body tag)
Кстати, по поводу настроек: там VK_APP_KEY -  я так понимаю 1) не используется, 2) далеко не всем надо. 
Предлагаю сделать приписку типа "Если вы не знаете что такое VK_APP_KEY, то он вам не нужен - пропустите его". А то я сам время потратил и в гугле кого-то встречал, кто требовал втолковать, что же это такое.
We're certain that all the remote engines that need to be wiped have been wiped at this point, right?
why this change? isn't it likely that we would like to add some further denied states in the future?
I disagree ... it is acceptable for this to return nothing 

----- Original Message -----

From: "nrporter" <notifications@github.com> 
To: "plymouthstate/plymouth-webapp" <plymouth-webapp@noreply.github.com> 
Cc: "bscoleman-psu" <bscoleman@plymouth.edu> 
Sent: Wednesday, October 3, 2012 7:59:20 AM 
Subject: Re: [plymouth-webapp] changes for adding new BM Report App (#425) 


In legacy/hub/HUBSQL.class.php: > +    {
> +		  $sql = "SELECT * FROM `bm_rounds` rounds WHERE rounds.bmr_id=? ORDER BY time ASC";
> +      $rs = PSU::db('hub')->Execute($sql, array($data['id']));
> +			$rounds = $rs->GetRows();
> +			$data['rounds'] = $rounds;
> +      return $data;
> +    }
> +
> +	return $null;
> +	}
> +
> +/**
> + * add a Round Entry for Building Manager Report to the MySQL hub database
> + * 
> + */ 
> +	function addBMRRound($now, $data) 
This function should at least retun the result of the query even if we aren't using it. This allows for easier failover testing. 
— 
Reply to this email directly or view it on GitHub .
Please scope to the current form
Nitpicky. I would include these function (rebase, getRevList) in the low-level git functions.
Also, I think getRevList() can now be renamed as get_rev_list() to unify the code styles
When we implement this, how tolerant should we be?
`@param` is missing
We should probably add a doc string for this method, or describe what's going on in comments further down a little bit, definitely not something I understand at a glance anymore.
Do we need this? It seems the renderer side code has to change either way.
The ~ need to be as long as the title.
Well, we got:
1) connection per keyspace(with all it`s resources protected by mutex);
2) x workers for each application(nginx workers);

I dont think there is a need to organize pool of connections within this gem.
When we will reuse astynax we will reuse its connection pool as well.
smae as above
It seems this is under system.

Filesystem paths are not a system property, but a container specific one. Consider moving this under container
Unfortunately not. TinyMCE's support for this is not great at all. The upside is that it doesn't break anything important if it's not executed in time.
Shouldn't be an issue as this won't be exposed to end users.. In this case, I also think we need eval.
Ditto (x2).
This class and its methods all need JavaDoc. We especially need to document the behavior for the case where either channel OR expression is provided.
Your usual "rules" would dictate we rename this obtainReplyDestination :-)
same comment as above for the other parser
but if the race condition occurs, you would be using one instance of a temp reply destination for a single time only, and then you would use another one for all subsequent request/reply operations (starting with the one in the thread that just happened to be between the 2 checks). Does that have any implication on the assumptions made elsewhere - in the temp queue resource caching, etc?
Remove the double extending, continue setting the individual properties.
Images needs a title
Or don't use `update_attributes` at all so you don't have to worry about making it accessible.
Optional: consider splitting the `specify` into a separate `subject` and `it` for better readability.
of course it is. load is called when the spell is cast before targets are selected
mai pune 3 spații la indentare
"JSON schema validation support for Backbone."
also, the type should be org.openmrs.Location ...
Done.

(By the way, your link is behind a paywall)
This change doesn't look good to me, but I don't know why there are hard-coded paths in this script in the first place
Поправь кодировку
Is this used anywhere else? (maybe in plugins)
beshe ke file hat moratab bashe
yeah I see your point, it just looks a little odd, having two url patterns define the same name. I understand that you can use it the way that you have described, but I think that it would be better to be explicit and use different names, it would make it clearer to the casual reader of the code
I reckon you can bin this function, as it doesn't really achieve anything given your implementation of it on lines 40/41
yes ur right ..the point was for the formdata  generator to yield one question and answer one at  a time from the cleaned_data dict  and save or do whatever  its memory efficient  .we can get rid of it.
same comments here as for action menu although I don't understand the difference between action and object menu ....
Ok done!
https://github.com/telefonicaid/Gaia-UI-Building-Blocks/commit/62280f57c6dba8a023e4f7686406a7cbbce6ae07
If someone adds a NULL to the list (via ```addFile(NULL, NULL)```), this pointer will be null.

If it makes sense for that pointer to be null sometimes, I should add a pointer check here. Otherwise, I should add an assert on ```addFile``` to make sure it doesn't happen.
This breaks the type system.
Hooking the main class into `plugins_loaded` on priority 10. This will allow people better control via hooks. It also makes sure, the new `static $instance` only runs once.
You are going to need to make this more realistic.  If I recall the ActiveSync spec correctly, INBOX can be assumed to be 00000000-0000-0000-0000-000000000001 and I am leveraging this in my fix for the whole "no inbox! asplode!" assertion bug.  Lemme know if I am wrong about this, please.
It's perhaps worth noting that globalPoint isn't actually in 2.0, and this documentation will never appear anywhere.

The only thing that matters from a documentation point of view is that the globalPoint properties are in the right place--that is, in the events that listed them for 1.8.x--so we can generate the removed APIs list correctly.
den hist brauchts nicht
Does this mean the factory doesn't always create a new webview?
`url : $sn.fms.url,`
I think the author has since fixed those issues, and has all proper tests for IE. Do you have a Windows machine so you can check? If all goes well, you may want to remove this file.
je trouve que cela fait répétition du mot ainsi.