labofsahil/github-event-dataset-2012 huggingface dataset:
Bedrock is a new version of mozilla.com written on top of Playdoh, Mozilla's Django skeleton project. https://www-dev.allizom.org/b/ is the Django part of the website. Removing the /b tests the old code base which is still on stage
Just the IF statement or the whole get_profile and saving part?
Might as well move the track._natives._setup && check into the if ( isUpdate ) {
I'm confused how not having the above doesn't trigger a JSLint error when running ```make check```.....
Same as for the other plugins. Make sure there's (at least) one zero termination in the peeked string.
I want it above that specific field in that box - what would the notice syntax be?
Sorry, I was checking REE.
Seems there are some status not implemented ?? check out [here](https://github.com/tpope/vim-fugitive/issues/206#issuecomment-6262802)
try template.start() catch { case NonFatal(e) => context.stop(); throw e }

This has the advantage of not having to look up which exceptions are caught and what else happens.
This is unnecessary and unrelated
See comment about me also not knowing enough about how to do this, above :)
I'd like to see this logged to watchdog instead of to the screen.
yup, sure it is. Atm the jcr event has the Throwable cause, but i'll update the pull request and add the extra information from the work item, as you suggested.
`Current Song` is never set anywhere in the 'correct' path.
What happened here? Why is :null => false needed?
This file seems to have an odd encoding. Not sure if I should change it..
@corranwebster A block's codestring is readonly, so this would not update it
Unnecessary changes, imo.
yes, this sentence only check if the string exports.favicon start with "http", so with "https" works too
Instead of checking table has options and then iterate them, you should directly iterate them as if there are no options, they will have no iterations.
I only explicitly return in ruby when I need to exit out early from a function. Please make that change and I'll merge this in.
Easy Bergin.  I respectfully disagree.  Code shouldn't smell, but I wouldn't write 1000 lines of code without throwing in a few here and there.
DIE ALREADY `app/views/main/index.html.haml`
you beauty!
This function calls down to all of the above functions. As it is an exported function I'd rather be complete.
Testing the added manager is probably not necessary, as we already test the creator.
Ignore this, created is a variable, not a string.
Or, can the download stream from the response be returned?
Is this because oae-rest cannot have a logger, in order to not have any dependencies? In that case, I think it's sufficient to just pass the error into the callback and let the client deal with the error.
It's not something I feel particularly strongly about, but for some reason I prefer the exception (TypeError) you get passing a non-parseable object into `Integer()` than when calling `something.to_i` in which there is no to_i method (NoMethodError).

```
1.9.3p286 :005 > {foo: "bar"}.to_i
NoMethodError: undefined method `to_i' for {:foo=>"bar"}:Hash
1.9.3p286 :006 > Integer({foo: "bar"})
TypeError: can't convert Hash into Integer
1.9.3p286 :007 > Object.new.to_i
NoMethodError: undefined method `to_i' for #<Object:0x007fdb13746a80>
1.9.3p286 :008 > Integer(Object.new)
TypeError: can't convert Object into Integer
```
mmm.. something smells wrong.  As a rule I always ensure font sizes are always no bigger than the parent element.

Maybe we need to reassess our font-sizes elsewhere.

We seem to be using a mixture of pixels, ems and %s - we should probably go with font-size: 100% on the body tag and then define all other fonts in ems. Thoughts?

Should also probably adjust #monument-detail .monument-sub-details font size instead...
Thoughts?
unrelated whitespace
1) again when after you compile all you manipulate really is List<Object> and Java automatically do the cast if your code specify explicitly a Type. In Velocity where you can't explicitely specify a Type you will just get some odd result when you have mixed type but it will "work"
2) No, all we (actually all java-diff-utils) use basically is Object#Equals so any type would do even different type.
Waarom deze line in deze branch?
You're using `substr()` at line 818 to assign this one and `substr()` will never return NULL, so this condition will always be satisfied.
Either use `empty()` or remove the whole `if` condition.
Вообще, сказано. ;)
было. @freetonik об этом неоднократно говорил в комментариях
@Senior-Student у меня там "не" потерялось. Я не совсем правильно выразился изначально, вот. То есть сначала ищем место, а потом обнаруживаем, что у нас автомобиль больше, чем предназначено для места? =) Ну тут две точки зрения, которые имеют право на жизнь.
Would be good to avoid private access across classes.  When I was looking at DepManager it wasn't clear how it'd be used.  You could perhaps expose `depManager.getKeys()` for example.
```js
return {
  exist: true
}
```
It looks like there are some artifacts from the merge on the master.

[don@seneca swapmeet]$ git reset --hard HEAD@{1}
HEAD is now at f206589 Merge pull request #16 from andrewberls/sj_cleanup
[don@seneca swapmeet]$ git status
# On branch master
nothing to commit (working directory clean)
[don@seneca swapmeet]$ grep -r HEAD *
app/views/offers/_bid.html.erb:<<<<<<< HEAD


Also in the _bid partial, there are:

<%= render :partial => "users/karma", :locals => { :user => bid.user } %>
    <%= render 'karma', user: bid.user, byline: false %>

Which one is right?


On Sun, Nov 11, 2012 at 12:59 PM, sarahjones <notifications@github.com>wrote:

> In app/assets/stylesheets/messages.css.scss:
>
> > @@ -0,0 +1,26 @@
> > +// Place all the styles related to the Messages controller here.
> > +// They will automatically be included in application.css.
> > +// You can use Sass (SCSS) here: http://sass-lang.com/
> > +
> > +#messages {
> > +  @extend %whitebox;
> > +  @include clearfix;
>
> Okay, I just copied and pasted what was used elsewhere.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/andrewberls/swapmeet/pull/17/files#r2093212>.
>
>
Caching as well?
@jamill Is this what you had in mind?
same here. What happened to have to change this?
plot_channels is a weird name ... 

maybe

types = sorted(scalings.keys())
test_ica_raw.fif is too big. Can you use a mock info structure. It's an overkill to add 9MB or files to the git repo for a layout test
no prob, I will do it later today
I'd prefer Raw here too
The&lt;space&gt;&lt;space&gt;internet  -> single space
you&lt;space&gt;&lt;space&gt;already  -> single space

corporations—think -> space before em dash (although since it's an in-sentence aside, arguably the em dash should not be used here, it should just be a full stop =)
I was looking up various libraries and nobody seemed to have refuted this guy - <http://stackoverflow.com/questions/7588581/what-nodejs-library-is-most-like-jquerys-deferreds>

> Q()'s interface is just damned confusing

Although to be fair, they all implement the same proposal so it should be generally easy to make the switch.
I agree this is a hack and this is not readable, but this is a known workaround for 2+3 metaclass support. 

There is a more readable alternative in six library: http://packages.python.org/six/#six.with_metaclass but it doesn't work with factory_boy as-is because of advanced metaclass magic in BaseFactoryMetaclass.
For consistency, this should you modify the current instance and return it, rather than generating a new one.
Extra newline; please remove.
Should be Option(...) and not Some(...)
Is this thread safe?
use filterNot
I meant  use_default
yes...and as with sa, so with s: put the pieces in a list then join when ready
Pero extiende de BackBone.Model, verdad?
Ese último no lo se. En general diría que esta bien y que lo dejemos así, pero si algún día nos da por hacer correr las pruebas en el entorno web, reconsideremos esto.
Yeah, I've been kind of surprised by that: One of the features boasted by Clojure is its extremely compact syntax.  So I've been surprised by some of the longer length symbol names, which seems counter to the terse philosophy.  Don't get me wrong, I don't want to sacrifice readability for convenience, but when there's little chance of misinterpretation (e.g. prods vs. products), it seemed a reasonable thing to do.

I can certainly change it for consistency with the rest of the code if you feel there's value.  And I'll definitely add an example to the doc.
try to refrain from using "p" or "x" as variable names in ruby -- "plant_names" would be easier to read in a couple of months.
Here one can use the logging module instead of print().
I don't like two potential calls to super here, it's confusing. Can we get away with always calling super first? eg:

```ruby
def initialize(pattern, match_payload)
  super
  return if pattern.size.zero?
  ...
end
```
I guess, all needed libraries should install now
plz do sir
the date should be updated
This enum should use the same type as the `GIT_RESET` constants if we're going to have a one-to-one mapping.
@pelson Under what circumstances would this case occur?
This is one complicated namespace. If you aren't already working on it elsewhere, it would be great to see some serious examples of how to use each of these functions. I like how this is done in the [thrift namespace](https://github.com/reddmetrics/forma-clj/blob/a50f892568ecbe37756f68f3982b1c205e4945b4/src/clj/forma/thrift.clj)
probably best to read up a bit on protocols and leave this out for now. You can have a function do different things based on what comes into it (just like you can write function that do different things based on how many args come in.)
This should be changed to fire the click event on the anchor tag. This would ensure that any events that are fired by clicking would also be fired by this method.
No, your commits will appear here
I know that you adepted this from the other class and it might be annoying, but please again here, use SGVector stuff. This do_free flag is horrible :D
Shame on me :D Copy-pasting spotted. Fixing these comments.
Should this be CurrentMember.Favourites.max(LastEdited)?
Could remove this commented step if we copy directly to `.env`
`relocator.scrollFunction` doesn't depend on context, so this could be:

```js
codeMirrorScrollArea.on("scroll", relocator.scrollFunction);
```
It shoudn't be in commit.
can you send these from views.py instead of rendering in the template like this? e.g.

pass in profile = pageuser.userprofile, mentor = profile.mentor.userprofile, added_by = profile.added_by.userprofile

then you can refer to them in the templates like profile.display_name, etc.
it would be great as it is generally an error
ENGLISH IS FULL OF BULLSHIT FILLER WORDS AND IT BOTHERS ME.
If you followed the flow of the code, key here is the object of key: value pairs you just set, so you can use it just as any other assignment in javascript, e g: a = b = { x: 1, y: 2 }; sets both a and b to the given object. If you insert a $.cookie call anywhere in that assignment chain, this makes it still work, which is expected, intuitive and useful behaviour.
### -> ##
```javascript
if (arguments[0].id && model.id === model.cid) {
model._deferredRequest[1].set({id:arguments[0].id}, {silent:true})
}
```
Smart. esse kwargs ainda n~ao funciona, mas eu quero implementar no lado dos bancos tb pra permitir passar os parametros todos por kwargs.
I think I slightly prefer having a `var code_name_map` (or whatever), and test with `in`. Also, please add `{}`s.
Would this be better as "Merges a new element into a source <field> element."?  Just going by the description in the next method (mergeNodes).
Best to use ofClear() here
Move this to utility folder please.
That breaks if {% static %} returns something which already includes a domain.  It's not perfect, but it's easily fixed if users set STATIC_URL to a full HTTP path.
"modificaciones" ?
oh, in that case clearly you're right!  I'll patch this up ASAP.
1. I'd make the limit overridable from environ.

2. Could the output be that big that it is a problem? If so does this change fix thr problem of having the entire output stored in memory in the "output" variable?

3. I don't think the and/or construct is extremely readable here. :)
Also, it might be better to make this `protected`, since I only anticipate it being used by subclasses.
this is up to the checker. i specifically did not want to require adding logic to the model.
Right I'll move back the functions and will then have a walk in the sun with the family #niceweatherhereyay
I guess we could use `$baseLineHeight` here?
Why `automatic-width` here?
`if (!selected)`?
No, every user action should be a POST request. Not GET. We should not do anything over GET. Make a form.
Start with capital letter for class names.
Давай вернем прверки на BelongsToProperty и если нет то исключение
Causes a Crash: See @Dekadencee crashlog.

If (Player* player = unit->ToPlayer())
    player->SendAutoRepeatCancel(unit);
This link is going to be stale, but isn't currently. We can update it when the link actually changes.
you should skip this if we're running over windows as it seems to be explicitely not compatible with it (you are making calls directly to linux binaries)
Why this function is removed? Is it used at any other place?
We already have scalable? defined on other classes - that's what you should be using.
when you say 'We have a patched ssh running' are you talking about OpenShift hosted service or about RHEL?
don't touch this file
endog in the class docstring said 1d, I guess this is an exception and requires 2d
Also, because dygraph.js occurs prior to dygraph-utils.js, so it's not defined early enough.
Use space after function arguments `,`.
Is a subclass using the context?  Why are we caching it?
I know, but I preferred here to keep the URL on a single line and respect the 100 char, I think this is more important.  I've indented all other lines, and pushed, but I think it looks worse!
In SBProfile, we have (now after my changes for #141) a fairly consistent scheme of using a stepk equal to pi/R where R is the radius that encloses 99.5% of the flux.  You might want to do something similar here.  I don't know how 10*fwhm compares to what you would get with this choice.  It might be similar.
Minor update: I fixed some of the tests which didn't specify a radius and relied on the defaults in the C++, and which failed once I changed to require a radius.  Two things remain:

1) the aforementioned handling of Sersic etc. that only have one radius specification, and a default.  (I am going to a talk in 10 min, won't have time before that.)
2) the fact that, as I just realized, the C++ and python behavior are now different - the C++ still has default args, so if someone calls without arguments when they are writing C++ code, there's no problem.  This seems bad; however, perhaps could become a new issue for cleanup next week, rather than something to settle now?
Can write simply:
```
Shear s(g1,g2)
```
Is it on purpose that you delete this file?
Confirm that we can satisfy dependency for this with ```apt-get install python-numpy``` on util machines
Hello Rene,
please remove this too.
This existing code for getting a JMX_PORT looks strange/wrong. I presume it should be using the new port lookup mechanism to confirm it's getting a port that is really available from the port range.
Why did you remove version here? Are you sure, everything will work with mongoid 1.x?
Sure, I'll fix it and send another pull-req.
I don't run my tests with rake - slower and clunkier CLI interface

I use:

    ruby test/test_whatever.rb

Because of this I don't have the load path set up automatically
`isReady()` does not follow the underscore separated naming convention.
Do you mean `binding.namespace`? There is no `client.namespace`.
should we really "force" ``dcterms:title`` in this class? maybe we should just delegate this to the DI extension?
My thinking was that you can add a GTalk buddy to your Jabber account and a generic Jabber buddy to your GTalk account.
Will this generate a notice is flashes['new'][$type] is not yet defined?  I seem to recall it doing so for me in the past.
Erm... this means the longer you are logged out, the longer it takes to wake up.  No.
Используй полные имена - `button` вот butt это немного другое :) Аналогично с arr.
Боремся с copy-paste: выносим 4 одинаковые строчки в один метод.
Не надо кидать NPE -- для этого есть IllegalArgumentException. Плюс тут непонятно, какой именно параметр null.
Should you be destroying the session here? The only methods that use this in the BaseFacebook class are _rest and throwAPIException. this can cause problems if you don't leave it up to the user of the class to kill the session. Makes sense?
I assume you will take care of this later?
Done. Submitted in #145
You repeat `str(current_time).split('.')[0]` several times. It's probably worth storing the result as a variable and using that.
Actually, bidpom defaults to new user, which is the sensible default I was alluding to. I still not clear why you're setting it at all, or to 'returning' when you're using a MockUser. I also still don't see MockUser being used anywhere in this pull. I don't see why using a MockUser should differ from using a string.
I missed that. Thank you.
I would just call this `fileExists:`
What is the * syntax?
Is ``mask.dtype==np.bool`` valid? ``np.issubdtype(np.bool, np.int) == False``
This code was untested and also unreachable afaik.
How should the grid-search be done without a score function any way?
And same remark about the link to the PDF :)
The lines above and below should extend to cover the whole width of the title, elsewhere this is not a valid rest title.
According to the api doc getLeft gets the value of the left property, and if the left value is modified it should return the current left value, not the one which was set first time.(tested with android as well)
The same for other getters
I'm not asking for documentation, i'm asking you to try it out and report your findings.
do we?
Al these is controlled by the user controller!!!
You need to avoid this happening, and it could, e.g. some other thread(s) have consumed the memory and there's no longer enough for the original size.

Unfortunately that means you need to allocate the new size, _before_ freeing old, i.e .so you don't have to reallocate the old size again.
Or, as long as there's only priority callback (i.e. sort) and no locking is required (because you call removeRowBuffer), I think you're safe to call resizeRow() on index here instead.

@ghalliday - do you agree?
I don't think that test is correct if you had a multi-byte escape sequence.  (However bizarre that case is)
Is it guaranteed that this file is a super file?  From a quick scan of the code cloneSuperFile may be called on non super files as well.  (clone parameter is set to true)
Checking for null after you have already used it?
on écrit pas `fainéant` ?
As noted above, there is no ZF namespace anymore.
I understand that HTML/XHTML is a valid response format from a RESTful web service, I'm just not sure I understand why you need an aspect to do this and why you don't, for example, create a HTML equivalent of the MappingJacksonJsonView?

Also how does the above aspect fit into your typical Spring MVC Restful web service?

None of this is clear from the code you submitted.

Lastly, FormBuilderAspect doesn't have any unit tests.
i'd prefer an enum, I think
I think res should be check against nil before using it here.
I don't agree.
It does work with SYSTEM, at least in my test case.
this should maybe be changed to a warning.
Indeed better.
I'm not as worried about 0 being a magic number when you just want the first item from a list.

I am worried about anything that smells of relying on the order of a ``dict``. However as I commented below, I'm not even sure what this try/except block does.
Not related to #350.  This is related to WebGL, not worth trying to compare right now, believe me I tried.
Here is the [code](https://github.com/AnalyticalGraphicsInc/cesium/blob/polygonidl/Source/Core/Plane.js).  It is a signed distance along the vector from the origin.
I don't know the name for the convention, but I like to assign to all `this` properties together at the end (or somewhere) of the constructor function.  As @mramato says, assign the property to `undefined` if the object will use it later, but you don't know the value.  Don't let the JavaScript engine dynamically add the object later.
The bounding sphere is changed based on the camera position. It estimates the size of a pixel at the distance from the camera to the sphere and updates the radius base on that pixel size and max size billboard. This way billboards that have a constant size no matter the camera position aren't culled when they shouldn't be.
The terrain branch will expose the terrain provider architecture.  In imagery_layers, it's private.
Please use the new styling of BS4: http://www.crummy.com/software/BeautifulSoup/bs4/doc/#method-names
I know, I'm annoying ;)
Btw you can use the `generate.bat` script to generate the files.
It's failing for me in Ubuntu 10.04 64 because it's calling the 32 version.  Reason is that if I go into python and look at `platform.architecture()` i get:

      >>> platform.architecture()
     ('64bit', 'ELF')

It looks like we might need to check for tuples here.
What happens if this is called and the android SDK is not installed (or not found) on the system?
Should we use commas instead of a bunch of 'and' ?
Spacing is off here
Так получится немного короче
```javascript
$.getJSON('/dz-7-jquery/current-event.json')
```

```javascript
$.getJSON("example.json")
.success(function() { alert("success"); })
.error(function() { alert("error"); })
.complete(function() { alert("complete"); });
```
http://api.jquery.com/jQuery.getJSON/

Старайся использовать относительные пути `/dz-7-jquery/current-event.json`, на практике часто бывает, что скрипт нужно запускать на разных доменах.
With the proposed class design this wouldn't be needed anymore.
@whiskybar there's been some other commits since this, see https://github.com/ripienaar/graphite-graph-dsl/blob/master/lib/graphite_graph.rb#L342-343
What needs to be done? Empty TODO
'pred' is better than 'less'?
In std.algorithm, pred is common name for such prediction.
> Anyhow, static T i; should be equivalent to static T i = T.init; ...

No. The first one will fail for `@disabled this()`.

> ... Better yet, let's make this static immutable T i; so as to avoid thread-local storage.

Same problem with `@disabled this()` for `immutable T i;` And [Issue 8902](http://d.puremagic.com/issues/show_bug.cgi?id=8902) for `static immutable S s = S.init;`.
Ok. I'll amend it.
ok, that's a much better solution. only (minor) problem is that all classes have to be listed since lint doesn't accept recursive ignores (example here for reference: http://stackoverflow.com/questions/13516327/writing-custom-lint-xml)
Personally I tend to use empty lines generously, but I'll leave them out in such places. I rembember this most of the time :)
変数名がそのままだよ！
transformedParameters
Why is this error being suppressed?
This test was failing for me before even branching. For some user, LastWriteTimeUtc was returning my CST -0600 time, not GMT time.
perhaps limit the scope of the event handler?

    $('.bigTimeGraph').on('mouseover', 'circle', function() { ...

>For best performance, attach delegated events at a document location as close as possible to the target elements. Avoid excessive use of document or document.body for delegated events on large documents." 

Source: http://api.jquery.com/on/
Please delete the css for `.noticeContainer` if it's not being used any more.
Opportunity to capture update, create, destroy with one method.
Let's wrap the entire essentials section in:
```
<% wat_presenter.render do %>
  BOOYAH
<% end %>
```
Sounds fine to leave it be, a clarified comment will suffice I think.
Typo: `excution`.
I used PHPStorm, and it automatically generated that -- sucks because it didn't notice the use statements. I imagine this would also effect code completion for other IDEs, but I can short name the ObjectManager, and the FormException, but I have to put in a use stateument to shorten the EntityLoaderInterface for it to be resolve correctly.

I'm not sure what the style guide is here, should I short name everything, and put in the appropriate use statements? Or only shorten the items that have use statements and remain resolvable?
Hmm, you're right, will look into that.
This should probably be host['user'] || "root", as the Windows user is different.
What does `get_as_dict` add here over just accessing `result.num_cases`, `result.num_suites`, and `result.warnings` directly?
Sure thing, added an ID to the span and updated the selector.
Any reason for the `<!---->` here?
Can this be an `img` instead of a `div` with a background image?
Then please open http://www.geocaching.com/map/ and your modification will not work. The old beta-map was at http://www.geocaching.com/map/beta/default.aspx
@elemoine: Yes now I am sure. I reversed the two commits and it **works**, but still do not know why it fails.
Most likely everything related to TypeInfo in any way should probably be immutable. However, given that druntime is nowhere near const/immutable-friendly at the moment, making them just const seems like the safer route (because the compiler can't optimize const in any sensible fashion that would break druntime code). Eventually, we can change all of these to be immutable (shouldn't break too much code).
(Fix Submitted)
QUERY: Can we not just pass the value instead on constructing an object and then streaming the value in?
Already defined in inglenook::core::exceptions
Yes; and I commented in the other commit. Open another JIRA.
These two lines read a bit like Dr. Seuss :)
I'd rather you use any Base64 decoder already available in core.
Not sure if puppet cares but facter returns Debian
かっこつけてください
I like that you're sending in the movies here and injecting this dependency.
Let's move this section to right below the people section
Fair enough I think, I'll change it
I'll do some general proofreading on this, but I think it'll be easier to that by updating the file once you've landed this version and posting a diff.
This is third-party code, right?  It should be in a ThirdParty directory like we do in Cesium.
"if ((blinkCount & 1) != 0) {" would be more efficient here.
Then it's not using armstrong.apps.articles.
yeah, indeed. It will allow them to be pruned if they are skipped
Any reason to have both a query string and explicit parameters instead of just sticking with a single way of specifying options?
Missing radix parameter - желает получить систему счисления которую будет использовать для парсинга строки. Можно просто писать `+ temp[i]` см https://speakerdeck.com/u/azproduction/p/2-liektsiia?slide=13
This indentation is a bit strange.
Would making tfidf's parameters properties on `Vectorizer` do the trick? There's only four of them...
don't need this print in real code - either wrap it with a check that `settings.DEBUG == True`, or use logging, or get rid of it
Actually, it's not. It's an `Iterable[java.lang.String]` - a "Java String" not a "Scala string". These are two separate types that aren't converted automatically.

And I'm down for loosing the option.
we appear to have an old tab/space issue here
here the message can probably show something like : 
contact1, contact & 'x' others or
contact1 & contact2
пробелу
has a default sensor *of the given type*
log where used
Hmm, so they would want to talk to the native version X while using the js version Y? Should we allow that?
The version above is meant to be a guard against out of sync js - so that we can at least log or even reject the call.
this got moved to the async task.
This one is fun, there are 2 correct ways to spell `Cancellation`/`Cancelation`.

Should check what the .NET framework naming guidelines say. I honestly don't remember. Windows phone got this wrong and so they have both in the API..
I would prefer this be a string constant.
The code here in the else clause is basically the same as the code in the if. Can we maybe wrap [ data[dataKey] ] as an array when it's just an object and go through the same code flow?
`mkdir /var/logs/instagram/` and place your instagram output in a well-defined file there, such as `instagram.txt`
Dialog seems like a good idea to me.  We can switch it when we get the toaster.
The private constructor is invoked within the static method GetServiceProvider()
this map should be a typedef, much easier to read
By the way, are versions 3.3 and 3.4 actually supported?
i guess we should also delete the unneeded template files from ./Resources/views/CRUD/
s/declaring/defining/