labofsahil/github-event-dataset-2012 huggingface dataset:
I used our default Gravatar
For every template tag rendering we will be doing a request to foursquare.com?
Empty line between code and imports.
No, `source_url` shoud point to the website, not feed URL. So people will click on this link to see who provides us. Not machines.
Why put this into template and not into `browserid.js`?
BTW, there is semi-standard way to define inputs and such in docstrings, but OK.
add in private:
EEEEK

```c++
if (me->GetCharmer() && me->GetCharmer()->GetTypeID() == TYPEID_PLAYER)
    me->GetCharmer()->ToPlayer()->SetClientControl(me, 1);
```
It looks good otherwise, i will pull it as soon as you change that ;P
Einen von beiden brauchts, sonst motz Monsieur le Compilateur, Konstruktoren werden glaub nicht verrerbt...
Evtl. auf den Zählendenm Progress Dialog ändern.
Imo, `AuthenticationService` (your implementation) should be left out. Instead, we can just create an instance of `Zend\Authentication\AuthenticationService`
@ajpiano I think it wouldn't hurt to include a note about it :)
Это среда генерирует.
Alternatively, put the method in a separate class in the Internal namespace. That has the added benefit that we can more easily add to it, but does make the code non-local.
Yeap, this comment is redundant anyway.
Didnt see NetworkManager installed on non-systemd system.
If systemd is not present it falls back to chkconfig + service <name> operation
`10px 0 5px`
Formatting: please put the pipe at the end of the first line here. Also, would ExtMethod be better as a "CI ByteString"? I don't have any strong feelings one way or the other, just something to think about.
space after not (brackets are not necessary but don't hurt)
thanks for the hint :-)
This is a constant in NhincConstants
I need to delete these... for direct these are stored somewhere else.
Just a query: is it quicker / otherwise better to do `(1./6.)*`... rather than ...`/6.`?  It seems like the latter might be more secure against rounding error, but I can imagine division being slower...
can do this via a map
```
video_list = map(lambda v: v.id, list(videos))
```
but your version is equivalent
Ah, I misread the !findUrlMappings. You're right about that one.
Not convinced by this call to connect(), but happy enough to leave it in.
It means that for some methods you can "get away with" not calling calling connect explicitly, but for other methods then you'll get an exception unless you've called connect explicitly. I'd prefer consistency, where we always explicitly call connect (or second best where it automatically connects for all operations without exceptions).
Florian, launch file arguments are there so this should be fixed... And I've also fixed loading of node parameters using the private node handle...
You can remove this and the bit in the page object, it's not relevant now that this uses browserID
(Per discussion in IRC,) we've identified the `forSetting` function as vestigial code and will be removing it.
Sure I'll amend an re-push
"self.innerHTML" should be replaced with "str" over here, as well
My bad, seems it was only incorrect on a few of the member variables.
confirmed - inclusion of bootstrap-file in shop-installation under /modules/function.php
One thing to keep in mind is that all the datetimes in the db are stored in the same timezone as `settings.TIME_ZONE`. Not sure if this is making a different assumption below? But we should definitely have a test passing in a datetime and make sure it remains the same if the timezone is the default one.
Do you have a suggestion?  Please remove the comment :D open an issue if you think something should be changed.
Right now, I'm looking at this one of two ways:

1.  Allow the JobLoader to deal with this as an optional, first class component (your second option).  Populate it when it's there...don't when it's not.
2.  Require it as a first class component and treat it just like the JobRegistry.

I'm leaning heavily towards option 1 here since it is not something that everyone will need.  What do you think?
Be aware that `$blnSent` would only show the status of the last email sent. However, we need to remove this anyway ;-)
Is city and state backwards here?
Название директории должно соответствовать названию пакета.
BigInteger? 2 млрд. слов - это уже памяти больше 10 Гб.
Не закрываешь файл.
I don't think the flakiness you observe is due to this pull.
*if* the tests are regularly creating new apps, this should work.
in a stricter world, the test would create the app as a precondition and delete it to clean-up, but this is not necessarily a stricter world.
yeah, I'd rather not duplicate the Selenium API, but I also don't feel strongly either way here
Add a docstring - what are acceptable <code>payment_type</code> values?
you don't need to step through the lists, you can just compare them as-is. and you don't need to put the statement inside [] either, since you don't need to save the results of an Assert in a list.
リンク部分は追記を忘れてしまっていました。なので、指摘頂いた形で修正したいと思います。
Tell me more: what should I see here.
??

Please remove those empty lines.
Why commenting this out? Isn't it useful to expose the C API to other cython modules that would cimport this pxd?
is that on purpose?
just a suggestion - perhaps add a link from `Bag of Words` to, say, : http://scikit-learn.org/dev/modules/feature_extraction.html#text-feature-extraction
well, I did a lot of that in the other tuts - linking to wikipedia pages or scikit-learn pages. :)
again maybe atleast2d or array2d
can you put a comment to explain why this class is skipped? it this a TODO?
I will change the dialogs I have created, however there are dialogs created in this way throughout the entire application. I do not intend to try and change them all
you should revert this
You should add a source file name as an argument so an exception will return a call stack if it crashes.

Example:
Handle<Script> compiledScript = Script::Compile(String::New(buffer.c_str()), String::New(fullPath.c_str()));
Is that really the correct return code?  it doesn't seem to match the condition
fix indents, same for the rest of the file
Aaah, I see now, I'll update the diff in a minute.
would be nice to have test with real vcr cassete to replay ( to check if it's really set current user as users in headers instead this in authorization )

not sure if there are test credentials there to connect to zendesk ( we shouldn't use ours)
+1 are you going to do that? So we don't need to add the -orm dependency to engine?
Should default be undefined and Eclccserver auto determine based on # cores?
does cache?  the lock does.
I think this is wrong - it needs to assign the same min/max for all splits, the clear will append NULL for splits>1
You have inverted the test.  It would be worth trying to create a test that this would trigger.
As I say, you can't rely on parsing the log paths to establish what the component name was.
Ah, there you go. That's how the sub get's NULL (as you said before).

I'd rather not touch this code now, and only refactor this once I get to change it to IArrayOf<>
No  idea how this happened but I am getting
fatal: reference is not a tree: a57795255065fd1a6cdd6e5f25fb93cf4a1bd9f9

when running fatal: reference is not a tree: a57795255065fd1a6cdd6e5f25fb93cf4a1bd9f9
r- 

There has a be a better way. I can think of one possibility. Hang on...
Does the .get() method here not return None by default? If so you can drop the "None" parameter.
Instead of feeding to MapReduce, use `Client#get_index`.
On your example the key -200 will be converted to an integer value by PHP, or has it changed since PHP 5.3.3 ?
array(4) {
  [0]=>
  string(1) "a"
  [" 1"]=>
  string(1) "b"
  [-200]=>
  string(1) "c"
  ["1000.9999"]=>
  string(1) "d"
}
Filter, again.
Will calling `bound_column.render` pass the correct value for `self` by doing this?
change "#" to "number"
braces go on the same line as foreach
anchor is a resource, you don't need to include class to use the anchor resource.
"typical CSRF" doubled.
No, you were right the first time. The idea is that the machine, whatever it is on the domain, would get the names of the logged in users from machines it has local admin on and then query a domain controller on the network as to what groups they are in. It does the gpresults to pull the DC that applied the GPO to that particular machine, my line of thought was that it would be a DC the workstation/server normally talked with so it might not trigger any alerts. I added a status line to show the DC it grabbed to query against if the setting is left blank as a sanity check. I reran this last night on my test environment, which was 2003 boxes and it worked fine. I'm wondering if I messed something up in the regex that checks the sysinfo on line 61 which appends a /R to the command which is required of 2008 and up. What OS is the DC you are using?
The import suggestions call returns a field `decl-as: 'class ` with a ' to begin the string. This was causing the read_form function to fail.
We can do that the only downside is that I have to change the jasmine.yml and backbone-support.js to make sure observer.js is loaded first. If that sounds better to you than have the _.extend in observer.js I'll make the changes.
Or keep it `required`? That would allow people to go back to destroy-then-create if they really want to..?
The Eclipse formatter inserted an extra tab. It does that when continuing a conditional on another line. I guess to distinguish between the conditional test and the conditional body.
`BillboardCollectionSpec` only writes to a 1x1 fbo for testing, so I had to set the `scalingFactor` to `1` so that texture reads would be accurate and the specs would pass.
Object.freeze does still have a performance impact on all reads of all properties of the frozen object.  It depends on how much code we expect to grab references to these instances and pass them around and perform math on them.  

The most performant option would be to take out the freezes, but to remember that they should be there, so we can re-introduce them when V8 gets its act together.

Relevant Chromium bug: http://code.google.com/p/chromium/issues/detail?id=115960
Jasmine has a built-in matcher for reference equality:

`expect(returnedResult).toBe(left);`
Is this example complete?  Looks like billboards.
Credit goes to @emackey who did this for cesium.agi.com first, I just copied his.
Don't remove it - and don't construct a texture atlas directly.  This create function mirrors the create function for other textures and all WebGL resources like 2D textures and cube maps hence it's location in Context.js and that `TextureAtlas` was flagged as `@internalConstructor`.
@bagnell can you take a look at this and make sure it is reasonable.  If Frank wrote it, I expect it to be solid even if I don't recognize it.
No `USE_TI_FACEBOOK` guard.
Android supports this property, the default is undefined.
Since we only need to force the tab indicator to be selected when the app is launched, i think it's better not to add this to changeActiveTab.
Good catch on the bug. It slipped my testing somehow.
Does node.js really not have a way to keep a file stream open, and every time we get a log message we'd append to it? That seems implausible, and better than opening/appending/closing a file every time we get a log event for I/O.
Same as above. Maintain array of KeyManagers
"windows" should be singular.
I think I'm missing something... firePropertyChanged() calls modelListener.propertyChanged().  I don't see where we call setProperty() on the krollObject from there.
Лучше сделать какой-нибудь main.js Стараются не мешать HTML JS CSS между собой
not sure about this one. @lsmith77  we said we will target symfony 2.2 for the first stable release of the cmf. on the other hand, we should try to stay compatible with 2.1 unless we have compelling reasons not to.
For some reason it didn't flush for me yesterday but it does now, I've removed it now.
that reminds me, can you add a TODO to clean this test up? it should be using the polyfill scripts, not hardcoding the component construction map
move to `rmpi.py` module
TODO: Comment this out for now.
Add reference to issue please
Well, we are now no longer testing against it and phpBB 3.1 is 5.3.2, I think I'm certainly leaning more and more towards requiring 5.3.2 (or another 5.3 release).
Ah, of course! Sorry about that. Will merge now.
"to a FIF file"
this also conflict as it is needed in the other PR (even worse: git will not conflict, but it will remove it when merging and break things)
Can you put the header above the [TestClass] attribute? That would be consistent with my code and with the coding style you're using with test methods.
Fixed in this commit: https://github.com/releod/woople-theme/commit/4ad82a7a22d4a7658e0517bbb88bcab5a2538730
This change will leave the ";" on the end of the session ID, where the prior code will strip it off.  Is this ok?
Same line as closing brace.
You don't need the empty array anymore. jQuery has supported `$()` for an empty set since 1.4.
if we don't need this prefix anymore, we can probably remove routeNamePrefix completely.
(I think they're importers, because they actually perform data import. A map is just a data container, it doesn't do anything with its data.)
Can you remove these whitespace changes?
Actually, I'm not so sure. In my usage of libffi in a VM interpreter, I found that unwinding through C frames by throwing D exceptions works perfectly fine.

But maybe I'm misunderstanding?
Because this is a template funciton. Similars (`reserve`, `assumeSafeAppend`, `_ArrayEq`) are also implemented in `object_.d`.
Look at the AbstractCorrelatingMessageHandlerParser (line 50) and then injectPropertyWithAdapter(..) method (line 81)
I think you'll be able to figure it out from there, otherwise ping me on Skype tomorrow
functionally very similar...
If data-linenum is not specified, line numbers shouldn't be generated. If its present, but has no value, default to 1. If present, and has a value, use that.
should we have a try catch here and return null if we get an exception, since IslandoraXacml will toss an exception (an XacmlException i believe) if it is unable to parse the policy.
FYI: "<<" is referred to as the "shovel" operator.
switch from # to /
This doesn't seem to work since the Ajaxifying of the lhnav.

1. http://localhost:3000/sdk/widgetdesignandstyle
2. Click on `Back-end feed and UI API documentation`

The content isn't pretty printed.
What is this for? javaDoc missing?
I would think this would just be a thin wrapper for protobuf.ExtensionRegistry
MAy be it should be here.
I'll remove it for now. The default sorter is already configured in the app config, it's not necessarily to do the same with javascript.
This was not supposed to be committed. How do I get it out again?
Sounds good.

Actually, I think I just figured out how to do this.  It looks like in JS its possible to define a function within the local scope as a function but not inline in the actual function call, I'll update to use that approach and update.
I'd use some more speaking suffix for file name. I used .tpl before, but that's tautology too. Maybe the best choise is something like .email
Ok should we just tell them to install node and not reference a version? Since this is the node used during runtime anyways (wont run into npm issues etc).
Does this also (unfortunately) need http://ccweb.cloudsoftcorp.com/maven/libs-release-local?

Even better, let's get M2 onto releng2 (i.e. the developers machine).
Should probably also run this through htmlentities, come to think of it; I'll do that now.
Probably ObtainAuthToken
without this this field in syslogs appears as 'undefined'. This allows for more meaningful logs.
parentesho zekr kon
shayad man bekham too projeyi ke azin estefade mikonam ye div ba in class besazam
Scratch that. Loosing the `Option` isn't an option. Using `+` with just a string results in `Iterable[Char]` for some obscene reason.
if the user was not logged in, that could be nice to redirect him to the page he wanted to access before being redirected.

try something like
==>

if !user_signed_in?
   session[:translator_return_to] = request.url
   redirect_to new_translator_session_path, :alert => exception.message
end
Will fix ;-)
It looks suspicious to me because floating point expressions are not common subexpressioned.
Oh right...silly JavaScript and your royally screwed up scoping rules.
Sitemaps are limited to 50K lines so the trick is to link the different sitemaps themselves
e.g. Google : http://www.google.com/hostednews/sitemap_index.xml
this was already merged, remove it, please, and do `pull` to match latest changes.
Remove unnecessary testCompile dependencies
Does this need to be out of process?

Out of process is significantly worse on memory
Can you clarify these two lines throughout?  Without context, they don't really make much sense.