labofsahil/github-event-dataset-2012 huggingface dataset:
same as above - scala offers a very concise syntax for functional applications
Are you sure ``$propertyPath`` has the correct type as "object<PHPUnit_Framework_MockObject_MockObject>" is not a sub-type of "string|object<Symfony\Component\Form\Util\PropertyPath>"?
Is it possible to add a ``@param`` annotation for parameter ``$dateTimeFormat`` which defines a more specific range of types; something like ``string|array``, or ``null|MyObject``? Please also check line 59.
no return. So it means that we'll be using the blocking way. Sounds right. Is it indeed intentional?
This should be passed at class creation, not here – we shouldn't assume that `$_SERVER` is defined here.
I'm not sure if @fabpot will merge this PR, but your package definition is wrong. Twig does not require 5.3
Добавить к методам контроллера префиксы "Action" и убрать суффиксы "post"
Исправить отступы
It should be fine because after `validdateField` is called, we do this check:

    if ($valid instanceof Exception)

Since JException extends Exception we should be fine.  The extra arguments in JException will certainly be lost but I think we can cope with that.
Nitpick: greater than. Também acontece mais à frente no diff.

Eu corrijo isto num commit, don't worry. :)
The shebang needs to be the first line (above the copyright header)
raw не поможет, это как раз отключает фильтрацию
:-) Nice way to get rid of name and make translations for yourself easier. ;-)
Why do we have this variable? Why not simply call `$('#search_users').val()` in `redrawUserList`?
Have you checked [this example](http://api.met.no/weatherapi/locationforecast/1.8/?lat=46.17;lon=14.96) to see which all data is available?
So, what now. Does it work or not in Chrome?
Furthermore, it is not a constant. So it should not be an all upper-case variable name.
*Klikni*
Hm, unnecessary change?
For non-anonymous functions, style is that there is no space before `(`.
Isn't this `True` by default? http://readthedocs.org/docs/mongoengine-odm/en/latest/apireference.html
Why not? It should be there at least for checking if the post empty.  Other sites (facebook) trim posts before saving to db. But I guess we should decide about this in backend and not javascript?
Sorry, I'm not qualified to review this PR, but please make sure you import any from theano.gof.python25 to remain python 2.4-compatible.
This is good. Should make one for `logout` as well.
$targetProperty = $this->getProperty()->getTargetProperty();
if (in_array($targetModel->get($targetProperty), $localModel['values'])) {
void UpdateAI(const uint32 diff) - > void UpdateAI(uint32 const diff)
this is not error!
Nevermind, cleared on IRC :) Installation path is predictable and the PR is valid
line 5 should be git push origin feature_branch_name
I'd put these two function bodies on the same line as their arrows.
This mentions "this variable declaration" but I'm not clear which variable declaration it's referring to?
two arguments is iffy. e.g. if you just want to enable -nestedLoading, you also need to specify 'org.apache.hadoop.mapreduce.input.TextInputForamt'.
we can move input format also to arguments (-inputFormat). '-inputFormat' will be rarely used.
FYI blindly copying without understanding what code does is not the way to contribute - it creates unintended side effects and makes the code harder to maintain.
I think these should go in nonparametric/api.py
Why is the value `nil` here, and why lock the returned trie?
--[no]wait  - ждет в случае fromurl (по умолчанию: да)

--[no]info - выдает инфу по файлу (по умолчанию: iff есть приватный ключ)

--[no]store - сторит загруженный файл (по умолчанию - нет)

--url - выдает cdn-урл, заодно подразумевает --store
this syntax is not supported in PHP 5.3.x
Done. Nice.
I always have trouble with class names that don't pluralize nicely.  I ran across this with Matrix in my TMV documentation all the time.  But I like the "RealGalaxy objects" solution.
I'll fix that in the OpticalPSF class where this was inherited from too...
So according to Inspect Element, the icon for icon-book and icon-file are 33px by 16px, while icon-check is 17px by 13px. Perhaps the extra horizontal pixels in the image files accounts for the weird pushing to the left I'm seeing in Chromium/Linux, and perhaps it explains something I've long wondered about, which is why the icons get styled to have a left-margin that's negative.  

I don't quite understand how we're delivering these. They just get put in the class on the ```<em>```. Do we have styling that attaches an image, or are these the renderings of unicode characters in a particular font, or what? The answer probably determines what response makes sense.
You should make sure this is doing integer division -- if donation_amount is a float, then I don't think it will do integer division
Treating a timeout of -1 as "forever" seems strange: it's not like any of the java standard library APIs, I believe.

Also, the newer APIs treat zero as zero (e.g. CoutdownLatch.await). But 0==forever matches Object.wait and the javadoc so no strong feelings.
We have added the method "get_help_page" to achieve this.
If errors are propagated to the client, this message is pretty meaningless. I think we need to have a ```defaultMessage``` property on the prototype. That way it could also be changed at a later time
Heh heh, copy and paste makes fools of us all :-)
`create_user` can be True, False, or a string. True means that we should automatically create users if they don't exist after we verify their assertion. False means we don't and fail authentication. A string means the same as True, but instead of using the built-in user creation function we load the string as an import path to a callable and use that function to create the user.

This is for things like people who want to create users but keep them inactive until approval, or for sites that don't use the default Django user model, etc. etc.

The outer if checks whether we're creating a user or not, and the inner if handles how we're creating that user.
Fixed here and in new PR #242
okay, makes sense. Should we just *remove* the open stuff from py3compat, then?

Seems like we should either:

a) remove open from py3compat
b) on py2, use `open = io.open`, rather than the custom class we have currently
Old habits die hard I guess
I am basing this on what is being done to list the post in news.html.
nit: remove this leftover
According to the documentation at php.net the default value should be `'nocache'`: both `0` or `false` are not valid values.
http://fr.php.net/manual/en/session.configuration.php#ini.session.cache-limiter
{{ grid.hash }}_ajax : Il n'y a rien qui se sert de ça après, n'y a-t-il pas une erreur?
Hi my friend, please replace the typo 'enpoint' by 'endpoint'
I'll remove the lightness + green check then
Лучше не использовать `\` я бы даже сказал предельно крайне не желательно использовать. Проще всего вынести этот шаблон в HTML (в скратый элемент) и не мучаться так
If credits is an integer, this will wind up resulting in an integer.  That might not be what is expected.  For example, if credits is 50 cents, This will return 0, not 0.50.

I kind of suspect this method might be presentation-related in its original context and not really the responsibility of  Promotion.
I talked to Keith and it basically only testing that Galaxy Client can set the url correctly, headers, etc.  Since it's only a unit test it doesn't test galaxy.  It doesn't even seem like it gets back anything to test for expectations. 

On the human side, I am pasting the URL into the browser to get the json response.  I would argue that issues could get flushed out in the cukes, and that mechanically the urls are at least constructed to a spec.
Для уникальности ты используешь equals (toLowerCase().equals() в случае -i). А можно было бы использовать компаратор, который ты используешь в сортировке. Меньше кода, код, в общем случае, более корректный, т.к. при проверке на равенство используется тот же алгоритм, что и при сортировке (да и быстрее это, использовать компаратор, чем приводить строку к нижнему регистру).
Что-то мало знаков препинания.
Слишком большая область видимости переменной. И у остальных тоже.
Здесь используешь конструктор с int, а в мёржере?
Такие циклы по code conventions недопустимы. Фигурные скобки обязательны.
Is there a reason this isn't on 8080?
Yeha I am getting a failure at this step too.
The screenshot suggests that the page has not finished loading when this step starts.
-1

We use `precomputed` everywhere else in the scikit...
Just a stupid idea: could it be faster to write this out? Fancy indexing is usually quite slow, right?
Is it ok if i fix this in my next PR/in a separate commit in this one? Going to add some stuff to the identification handlers, and i'd hate to go through all the work of recreating the PR for some spaces :/
> @foo int a;

This is a irrelevant. That is an issue that UDA should interpret given function, or not.

I think that the parenthesis-less UDA should work syntactically as like the parenthesis omission in template instantiation.

```d
Template Templ(alias a) { ... }
alias X1 = Templ!(a => a);   // OK
alias X2 = Templ!a => a;     // NG
@(a => a) void foo1() {} // OK
@a => a void foo() {}    // should be NG, and it is consistent with an illegal in X2
```
So i found out that the die is from older code.  you likely need to pull changes from the macadamian upstream as the code your commits shows as removed here is the newer code.  I'd say pull from the macadamian fork (should be upstream if followed the setup procedure) and push a new commit to do the name changes.
I''m okay with that, but I still think we should make this clear so people understand.
Very interesting. So in this case you use both the grid approach to quickly identify the area of interest, and also the distance filter to provide the actual radius for improved precision.

Is there no use case for a box query, to perform the double range query but skip the filter? I'm thinking about two cases:
1- when the box is good enough, and the filter too slow to execute
2- when you are actually interested in a rectangular shape area, for example to show all matches on the map being shown on screen (and the screen/window is usually rectangular)
ditto, see above.
There is a separate issue open regarding support for allocations over 4gb - we might want to change it then.

Though having it return memsize_t makes errors such as multiplying this value by another integer, and overflowing, a bit less likely I guess
should be setown to avoid a memory leak.
Result of pipe->run should be checked here
> Could just use errorMessage(errors) and add "," when not last.

That'd require a condition inside the loop, and as confusing as removing it afterwards.
Hmmm, I think something got lost during a merge, recipientList should have been used in the replacements below. I'll double check my git history. May need you to pull again and reapply once I get it figured out.
Oops. I read it wrong. Anyway, I would still drop the formatting. That's the templates' job. It's part of presentation and shouldn't be done in the models. Not even in views actually. In templates is where that belongs.
Shouldn't this line be:

    <th><%= t('static_content.title_table') %></th>
This would cause the `check_volume_pricing` callback to be called twice during an update.
http://symfony.com/doc/current/contributing/code/standards.html
Appelle le ```title:``` ce field
defvar を使うと、どちらかのテストで指定した変数の値が使われなくなってしまいます。 let で指定するのが良いんじゃないでしょうか。
this looks like it should be in the actual glance module.
So far this looks mighty convenient for us Rails-without-CRUD-DB types.
@shacker when looking at [__the file__](https://github.com/shacker/calcentral/blob/d5d58adf2880b7a6cc3bbd0abe93562fb4bef01e/public/dummy/tasks.json) on GitHub it doesn't look right either
`(not (get authos :death-year))` would do the same.
How about `form` or `quotedForm` instead? (Since `quote` can be applied directly to a form that's not a list too.)
@wicz if you haven't already, you should check out the [Railscast episode about Draper](http://railscasts.com/episodes/286-draper)
Shouldn't his be 0644, rather than 0755, since we don't want to set permissions on all files to "executable"?
Both execs will always execute. Puppet should not execute commands if the system is in the correct state.
This should have a munge that converts a single string to an array, so that policies with one tag don't need to be defined with a single-element array.
Maybe move this comment down to the second loop?
I do not know why capistrano is not loaded while rvm-capistrano is, the `if` should be on line 142 to avoid executing this code when outside of capistrano executable (like in passenger).
missing `{% trans %}`
addExtensionToMimeType
As far as I know the only place where subregions are used is in one of the sandbox examples, and I agree it feels out of place now.
Me and @shunter want back and forth on this for a little while.  See issue #86.  Essentially the method you describe creates circular dependency issues.  I'm open for debate on the subject (which is why I opened the issue).  I think the most important thing is that it's consistent across the code base.
The `CompositePrimitive` doesn't return anything from `update`. Adding a `CompositePrimitive` to another would never be rendered. We could have it return something that is not `undefined`.
Needs to be specified as typographical points (font sizing only). Note that it is (technically) a bug on iOS that we do not support the `pt` specification with font sizes.
The syntax is incorrect. Should use ":" instead of "=".
Ahh, you're right. I think it's time to drink my coffee :)

shared_ptr: yes c++11 has shared_ptr, but also has unique_ptr which is non-copyable, but moveable. In its move-constructor and move-assignment moves the ownership of the pointer.
Ah, okay. If it seems to work without it I can just remove it then.
nice - did not know that!

2012/4/13 denlab <
reply@reply.github.com
>

> >
> >  (fact "group-by-count"
> >    (group-by-count ["a" "a" "b" "a"]) => {"a" 3
> >                                           "b" 1})
> >
> >  (defn make-query
> > -  [hashtag pagenumber] (doto (Query. (str "#" hashtag))
> > -                         (.setRpp 100)
> > -                         (.setPage pagenumber)))
> > +  "Build the query for a hashtag and a page number"
> > +  [hashtag pagenumber]
> > +  (doto (Query. (str "#" hashtag))
> > +    (.setRpp 100)
> > +    (.setPage pagenumber)))
> > +
> > +(fact "make-query" ;; java not easy to test
>
> you could use: `bean`, so that the test would be re-written as:
>
> ```clj
> (fact "make-query"
>    (bean (make-query "test" 10)) => {:page 10, :rpp 100, :query "#test"})
> ```
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/clojure-consulting/twitalyse/pull/5/files#r682833
>
Move this comment onto the next line.
```python
'ext1' in [x['name'] for x in eb.exts]
```

or (if you don't want to hardcode `'name'`

```python
'ext1' in [y for x in eb.exts for y in x.values()]
```
Let's not use the word virtual :)
Это для того, чтобы у чуваков была возможность убрать с элемента все ивенты виджета. Может даже `uploadcare-dialog-open.uploadcare-widget` или что-то такое.

В смысле, добавлять только при подписывании? А что это даст, если триггерилось событие без тегов??
Since these attributes are to be accessed publicly.. why store them in a dictionary and not directly in the object? looks to me that you are doing java here. :p
Не думаю что это должно быть здесь.
>Technically, array ops will trigger an opAssign, whereas the mem[move|copy] will bypass it.


Yup, I meant a cast to ubyte/byte[] first then copy.
can you add a TODO to use something more efficient than this :)
we're not really looking for Play 2.0 here, but rather any Play project. it should probably be finer grained in case this come before the play buildpack in slugc. also, might want to make it more future-proof for 2.1, 2.2, etc.
deleted due to misspelling on a spreadsheet
Why are you adding a row when you handle a deletion?

If we reconciled to an existing deleted row, there's nothing to do.

If we reconciled to an existing non-deleted row, just delete it from the database. We can worry about tracking deletions for multiple sync destinations in the future, if that ever becomes necessary. We shouldn't need to ever add anything to the deleted items table.
Named variables for these things, please:

```
String authority = "mockmyid.com";
String whatever = username + "...";
return factory.cre…(…, authority, …);
```

Also, constants!
What's the intent/impact of this method?
This should be removed since autoload is being required at the head of this file.
seems like this will break Redhat. Why even bother with making this an OS specific param?
Não precisava ter colocado esse embedded, para fazer o relacionamento coordenador e viceCoordenador.
Veja http://grails.org/doc/latest/guide/GORM.html#gormComposition
That's great! I think there are other places in the code where we do the same thing, which can go away now as well.
I'd suggest only allowing edits for 10 minutes or something, so they can quickly correct a mistake or something, but not forever.

Doesn't sound very "mongo" to me...hmmmm...will require more thought and research methinks....
ça va être à garder. parce que pour l'instant on utilise pas l'overview tree... mais on va en avoir besoin. Sauf que tu l'as pas mis à la bonne place. C'est une composante du configurateur aussi... donc framework web.
I would suggest to use the json api for frodo instead.
For Frodo all direct sql access will be deleted anyway. Only reason why it's not deleted yet is because Eden don't supports the required functions.

So, if you have the time I would appreciate that :) don't waste your time on a deprecated end.
There are so much details in regular expressions. Thanks for the clarification!
I keep saying I can accommodate your preferences... I appear to be lying... sorry!

Stupid tabs, it helps if your IDE tells you it doesn't have write permissions to the preferences file.
The only thing that /should/ be in there is the encounter binding.  The templates that you call should not depend on there being an "encounters" list.  The "patient" is not strictly needed, because it can come from encounter.getPatient(), but its nice to have as a direct binding too.

Make sure you have the latest from upstream master because some of the templates were updated to be more correct today.
Just changed it to read 'py.test' , choosing readability
Since we are no longer using this `cookbook_file` should we remove it from the files directory?
This would necessitate extending the BaseController class which we are not doing, could you revert this change?
This makes sense; in your tests, you may want to avoid setups like this that could be "wishy washy" --- instead, you could prefer to set the pre-conditions to be absolute and then make your assertion.
What is output used for?
Apply PSR-1, so brace on next line. Please use here underscore separation too, so $allow_linker
Please use `message` as the field name too.
The paths are wrong, but OMERO_TEMPDIR is also in effect for Windows
Ok, It's weird, I don't get notification for this project when there is a new comment.
same for all other events
The idea of renaming `_set_version` to `_get_git_version` was that it could be used here, so it is more clear what is going on here:

```python
with open('content/gitversion.txt', 'w') as f:
  f.write(get_git_version())
```
Yes I agree that might help even more for particularly low rank matrices X. Not sure how it would work in general. I didn't realise sklearn had an implementation of that randomised low random SVD algorithm - very nice! Are there any benchmarks comparing fast_svd to other SVD approaches?
Currently, captions are disabled by setting the currentTrack index to -1, i.e. when the user selects "Turn Captions OFF" from the menu, currentTrack is set to -1.

As it happens, Alex just spoke to James, and it seems James now want the system to retain memory of what the most recently selected caption was, and to separate selection of language from showing and hiding, so this will have to change. I'll have to check with James on that, to be sure of exactly what he wants.
Add a depends on at least python version 2.5 note in readme?
It's a better idea to use this import where necessary, so you don't inadvertently need to make headers public when they otherwise wouldn't be required to be.  I'd move this to the .m file, unless you anticipate making CDVPlugin+Extras.h itself public.
@fabi1cazenave as we've discussed, its not practical to do styling entirely with role attributes alone.
Fixed in 056b2cfc41edb7f57fb3af40534434242cd58104
para que va este comentario
Please remove all the blank lines.
This looks like a bug, should be
  final boolean doAuth = (_authDescriptor != null ? true : false);

I'll change it this way.
lets remove this file
utf8 is required to handle accented characters correctly.
Can we combine these two if statements?
breaks on windows machine, so this will have to be changed for sure
rename to m_mgr to show it's a member of a class
I'm not sure if I agree with this section.  I think the intentions are good, but it has a substantial chance of being turned into something negative.  For example, say you make the ticket price $10 since that's a pretty low number.  We all assume that everyone has that $10 to spare for a single event (which is probably true for the most part), but what about people who go to an event every weekend or more?  That small amount of cash adds up for someone with no income like a student.  Also, most events that charge require a credit card and not everyone has one of those either (believe it or not).

I think a better alternative is to generate excitement through some sort of challenge.  Ecommerce hack day did a great job with this recently in NYC and they ended up with a reasonably low dropout rate.
Some other plugins are also adding a Twitter field to user profiles.
If it isn't there already, it will be after we add it.