labofsahil/github-event-dataset-2012 huggingface dataset:
use `typeof projection !== 'undefined' ? projection : new EquidistantCylindricalProjection();` instead
May want a comma before the 'and' to keep it consistent with other entries =)
Why return "medium" here rather than "unknown" or some value that is representative of the value not being available?  If the value is unavailable for some reason on a large device then returning "medium" (as in this case) would be a bug.
No I meant values for verticalAlign. You should be reusing the TEXT_VERTICAL_ALIGNMENT_TOP, TEXT_VERTICAL_ALIGNMENT_BOTTOM, TEXT_VERTICAL_ALIGNMENT_CENTER properties.
The contains method in ArrayList is O(n). The efficiency is equivalent to this.
I think so. No strong opinion. I think our other samples use run(). Check the samples in the main repo.

I'm more concerned with following convention here.
Your point (offline) of not knowing what is setting `self.exts` in between its initialisation and here is valid, so leave the check there.
I like how you've got support for decorators being passed in, either at construction time or during an API call. I'm not sure we quite need that flexibility yet, though, I've got a YAGNI feeling, and think maybe the MetricsService class should just hard code the metlog decorators that we want to use.
Remove the extra space after the `parseInt` function call.
I'd like to see this in a module helper, like `redis_supports_virtual_memory?`.

And for parsing version, you can do something like:

    major, minor, patch = *options[:version].split('.')

Also, I think you will want to use `version`, instead of options[:version]. That way, it takes into account the default set in `moonshine/redis.rb`
I think that @sdeleuze used the same log level as in [Spring's DafaultHandlerExceptionResolver](https://github.com/SpringSource/spring-framework/blob/master/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/support/DefaultHandlerExceptionResolver.java#L152).
Urgh, it only works when converting to a string type though... this doesn't work:

```join([[1], [3]], filter!"true"(2));```

I'll add a fix and more unittests.
0-dimensionality is something special imo:

```D
alias byte[0] T1;
alias byte T2;
static assert(T1.sizeof == 0);
static assert(T2.sizeof == 1);
```
Yes, or else we can do `$('#top-bar a[href^=#]')`
Think it would be much if eclipse injected a sentinel of some sort saying the process is launched from eclipse ?
Mention duration.
Nah, this is writing to the local DB only.  As soon as the command hits the DB, we're silent (for good and bad). `notifyAndFinish` is only every called once per SendTab, so I'm not worried about multiple toasts.
for reading the code it will definitely help if you pack all those if things into mehtods like

if (edbObjectContainsSubModel(object)) // if this is what contains propertyName+".0.key" means :-)
same rather large instantiation everywhere
Thanks for the submission, but could you please modify it to use the symfony2 coding standards. Thanks

```
use FOS\RestBundle\Request\ParamFetcher;
use FOS\RestBundle\Controller\Annotations\RequestParam;
```
and btw i haven't seen such syntax "@AcmeHelloBundle\Controller\UsersController" anywhere, only 
"@LiipHelloBundle/Controller/ExtraController.php" for file
"@SensioBlogBundle/Controller" - whole directory
Don't leave these comments in.
This has been done for consistency with System.Net.Http.HttpResponseMessage class, which exposes both StatusCode and IsSuccessStatusCode. I was using this property with the .Net response class, and adding it here let me minimize changes.
* ロジックとコメントがマッチしていません
context "given multiple root nodes"
А вдруг кому-то еще будет важно, откуда? Продумывай заранее систему хуков.
Can't quiet follow what is happening here - can you explain?
It's named that way because the D version identifier in the spec is X86_64.
QUERY: Can we const ref: message, log_namespace and event_type?
I understand what you are saying (re; Resource), but I was using existin TxSynch conventions...

    TransactionSynchronizationManager.bindResource()

However, will change to FileMessageHolder
I agree with Amol's changes, except

s/where as,/whereas/  (one word, no comma)

also put empasis tags around Inbound and Outbound

The JPA _Inbound_ adapter ... whereas the JPA _Outbound_ adapter...

(by the way, be consistent; in both cases either '...Channel Adapter...' or '... adapter ...')
I would call this "prepareEvaluationContext" ("determine" sounds like it is always choosing between existing alternatives)
we inline the else's with the closing braces. E.g. '} else {'
return levelMap;
Implemented Nick's suggestion.
Interesting - may I ask what bug is it? I have been using set_variable_values() for a while and I don't think I've encountered any strange behavior.
Code is ok
i'm not sure about `integer` as user id type, you shouldn't force developres into it if not necessery
We talked about this; leaving unchanged.
@slavcodev, а в этом случае, кстати, не получится проверять атрибут на безопасность. С 1.1.12 атрибут `CFileValidator::$safe` равно false. Тут только разве что сценариями и делать.
!error
If you tested if(!sig) on line 36 where you return early ... why you need to retest here again?

This code won't be executed.
Sure, do it the easy way :)  Sorry I didn't notice that.  I changed that and squashed my new commit in.
I like that this is on one page, but I don't know about the button. What do you think of adding a tabs? We already have jQuery UI on some pages, so we could use http://jqueryui.com/demos/tabs/.
do they need to build 0.6.10 specifically? 0.6.21 should work and as well for them as it works for me on mac, also you need a period at the send of the sentence
No, you could add `is_grappelli_installed` to the context and then add some if statements in the template. I just thought it was going to be cleaner this way.
It's done :-)
there is leave it as is :)
Si, tenes razon, ahi lo arme! Fijate que te parece.
One additional thing you're going to want to do in here is toggle the visibility of the button.  We won't want to show a "Forgot Passcode" button when creating a new passcode.  I'd say set the visibility to NO by default (in loadView), and toggle it to YES if self.mode != SFPasscodeControllerModeCreate.
it should be removed from the non-profiled part.
but it is as it was in the original pom.xml
Indeed, but this was a duplicate of
https://github.com/Sanne/hibernate-search/blob/d44326a6d5bb21de5a8c67d39156851e1bf96b6b/pom.xml#L670
I thought about that... 

But there could be abnormal cases (truncated file for some error in the disk), and maybe it'd be more beneficial to receive a proper error message later, than garbage on the input...

Though, either way is fine for me. If you think it'll never happen (errors being caught before), I'll remove.
should this reference $asacct.DEFAULT_TIMEOUT_MS, or some other constant?
I'm going to rip these out just as soon as I have all the tests passing. I'll reimplement the async code when we decide on a library at a later date
You typoed it; you used "label = interface.pop['label']" instead of "label = interface.pop('label')"—square brackets ("[]") instead of parentheses ("()").
On creation? Or every time someone sets a property on the proxy?
Since this is `deleteFile` i don't think the directories comment is needed.
This property is not on the scale event in iOS.
availble on IOS but is incorrect. Make this android only while we fix this bug on IOS
What's the difference between your Initialize and a new .ctor?
Since the build has finished it could be that you can no longer write to the build log.
jasne ze ano :) prave preto som tak apeloval, aby sme posielali PR a mergovali az ked dalsi dvaja developeri budu suhlasit, aby sa na nic nezabudlo ;)
LOL woops I can't do pull request -___-


On Thu, Nov 8, 2012 at 6:51 PM, Seshadri Mahalingam <
notifications@github.com> wrote:

> In logrotate.conf:
>
> > @@ -0,0 +1,17 @@
> > +cat logrotate.conf
>
> wat
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/berkeley-food-recommendations/Realtime-Demo/pull/3/files#r2079031>.
>
>
Oops, sorry about that.  Fixed.
We could have left it, but we wouldn't get to customize the title, I can reset it if we like. Perhaps this is why Igor is having problems, but webworks seems to work for me.
'dragEnd' is deprecated. We should use 'dragend' instead.
rename this to stringToInteger or stringToInt?
It does work on IE now. I tested.  I removed the file in this pull.