labofsahil/github-event-dataset-2012 huggingface dataset:
I changed it so the converted file is cached and served if a request comes in to transform the file the same way, to the same size.
I would move all of this formatting out into its own function
I think this comes in handy when the code that calls this method is not ARC
If there are no responses, the Answer field in the Assignment will be omitted. The Assignment should never be nil. I'm basing this off of: http://docs.amazonwebservices.com/AWSMechTurk/2007-06-21/AWSMechanicalTurkRequester/ApiReference_GetAssignmentsForHITOperation.html

Can you clarify what it means to have multiple judgements on the task? Does this mean that multiple MTurkers answer a specified question? Or multiple questions in a Task?
Yes, you also need to add import from future. See how we do it elsewhere.
I really wanted to fix this so I tried what you suggested. But Json serializer don't know how to serialize objects from MongoDB.  If I do internal request to get this post as json (as you described) then I can't send it to push server because the argument in method send_updates accepts only object.
Then do it. :-)
I kept this one (#map-layers) because I used it in css to style the div.
OK, this works, but it is semantically misuse if `.is` function. `.is` function should determine if some predicate holds for `.post` or not. If you want to iterate over it and do something, then use `.each`. (And then you can also break out of the loop, once you find it, now that you continue searching.

But inside of the `.each` loop, you should do an `.is` call to see if comment is maybe already shown bellow the post. Our PUSH method which we are using can introduce duplicate pushes to the client, so you have to check for duplication. This is why post code has `.is` here.
So what is the error you are getting?

(I updated documentation, if you have time, you can test if instructions there work.)
We put space before `TODO` keyword.
OK. This error message is completely and really completely comprehensible. :-)

BTW, messages is not useful to use if you are displaying a form. Much better is to simply set some flag and then based on that flag in the view template display the message.

messages are used when doing redirects and you want to display something on the target page.

I would all this code move to form validation. And this could be simply a form validation error. So you check all this things in the form and if there is no value for choices, you display a validation error and ask user to select the value. If value is already there, you process the form.
Yes, everything is working OK with .NET 3.5. But as I've mentioned in Pull Request comment - my own project is .NET 4.0. So it uses NSubstitute from NuGet - also .NET 4.0. So I forced to use .NET 4.0 version of NSubstitute AND all libraries that uses it - ALSO. If I will not do it - I'll get weird assembly linking issues. That is the reason I leave it as is. So I asked AutoFixture owner to build AutoFixture.NSubstitute NuGet packages for each platform - net35 and net40. So everyone could be satisfied :)
I'll fix this myself. Merged into master, thanks!
not understand
Ah, danke! :)
:cyclone: drehen tut es.
Hmm, stimmt, das wär noch gut. Ich denke was auch noch gut wäre, ist ein Link auf CreateBrewery aus dem Menu der BreweryListe...
According to Carmen, you could say "Neunte Geburtstag" (using "Zweite/Dritte/Veirte/..." etc for number), or even just "9. Geburtstag" (yes, that is a dot after the number). However, Carmen says they are both quite 'choppy' and short.
Good idea :) For now let's ship this ;)
If you can read through it and eventually fix what you notice I'll merge this evening
Do you think we should throw an exception or rather use the `$this->authenticateResultInfo['code']`? (return early, decision up to you imo)
this message is wrong too, it just means the format isn't correct, we can't assume it's a single value.
А Int точно может быть нормальным полем в пакете? 
deserialize возвращает просто число, без хвоста
If you want the really C++ way, it would be:

    std::transform(cameraName.begin(), cameraName.end(), s.begin(), (int (*)(int))std::toupper);
This reads really oddly. SpawnMissile takes a already-existing missile?
You should not add indentation here.
Why not `r = self.est_model.resid`?
If I restructure directories a bit as I suggested [here](https://github.com/twitter/twitter-cldr-rb/pull/14#issuecomment-5188319) this file will live under `lib/twitter_cldr`.
CoffeeScript automatically returns the result of the last expression as a function's return value, the same way as Ruby does. Just saying ;)
I don't know how to format text in comments :(
Can we get rid of this LOG statement?
Good spot, change pushed.
I like how you've neatened this up.
Hmmm, I'm not sure about this one.  I *do* agree that the kwarg list is rather busy, but... for example, I could easily imagine that if I want a deterministic script (so that others can compare with the exact same shear field) then it would be nice to just specify a seed rather than having to make a deviate that I'll never otherwise use except as an input to this function.  Then again, it's just one necessary line to make the deviate, and then it simplifies the code in the lensing engine.  So I don't know, I'm on the fence here.
I was wondering about that.  I read up and it looks like those two are interchangeable.  But if DoesNotExist is more standard than I should change to that.
I don't understand why the button inside an anchor tag...
can't we just have `<a class="btn btn-mini">` ?  Is the button better html, in your opinion?
change to 100 millis so it runs more quickly ?
data.paintRectStruct = contentRect;
    data.paintRect = &data.paintRectStruct;
Blah.
Good point. It should be using the dbname() function.

In a authenticated environment there are no guarantees that we have access to that databse, so it needs to be configured in mongo-test-cfg.inc
This will work, but you don't need the brackets
You could use IEnumerable.Zip here instead to get a new sequence of tuples of xVersion, yVersion.
Wouldn't z/ been a simpler fix?
two reasons to that:

1. The docstrings are split after a certain number of chars, so this doesn't really help me when running the tests
2. It's actually shadowing the dotted name of the test for me, so it makes it really hard to see what is failing when it happens.
Should you use this syntax?

```php
<?php

if (version_compare(phpversion(), '5.4.0', '<')) {

} else {

}
```
The types in rectangle.h weren't actually modified. I converted them to qint and quint for consistency with Point and Size.
Изо всех. Каждому отдаю по List'y из toMerge, в который они уже записывают данные и сортируют их
У тебя название интерфейса не пишется.
In fact, this is done from submit_app, so doesn't need to be included here
Need Info: Is there a reason why this commented out?
:doc:`bio-formats-xuvtools`, which uses...
Not necessarily to be addressed in this PR but could you improve the narrative doc to give the definition of the deviance loss (e.g. math formula) and some of why it's a good loss for GBRT models (vs other classification losses)?
Could you indent your continuation lines, in the 2 above statements. Thanks.
The text did show up for me but I changed the sizes a bit, so it should look better now.
Thanks. That works :) I guess that trick can also be applied to the inversion of "inv".
again, either use plural, an indefinite article or reformulate the sentence.
I'm not sure about this change.. Arguments are considered the query params and the URI parts (/products/{identifier}). As I understand from current implementation, you can omit to define the type of URI parts into API DOC.. For this reason, the RestRoutingArgs object can receive on constructor a list of `lang.reflect.Parameter` possibly combined with numbers (the matching URI parts). I think an example is more useful:

````php

/**
 * Handle products collection with filtering possibilities
 *
 */
#[@webmethod(verb= 'GET', path= "/products/{productid}"] 
  // because the API reference to {productid} is missing, the RestRoutingArgs will receive 
  // an array with a string  ({productid}), not an object..

````

I agree with you regarding the ternaries,  but usually I hate having the `else` statement inside my code :). I will change it if you want
can we use reverse in js?
I saw you like otimizations then I decided to use a boolean variable to prevent the hash key lookup during rendering =)
How about a comment explaining what this means? // was generated by the compiler, wasn't present in source code
This seems wrong.  getNativeHandle needs to be implemented in each subclass, I don't think the container_ from NCO is the right handle to return here.
Will crash here if no arguments, need to check against length first
Ok, ignore then
Yep, there is a total of 32 entries but each page displays 5 of them.
why you capitalize each letter of the name?...escaped name is supposed to be all downcase and without especial characteres. Take a look on students model in linku app.
indentation is really off here.  I am assuming this is a tabs-to-spaces issue?  This repo follows the [PSR-2](https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md) guidelines
Exactly
On Nov 19, 2012 1:48 PM, "Hardy Ferentschik" <notifications@github.com>
wrote:

> In
> hibernate-search-engine/src/main/java/org/hibernate/search/engine/spi/DocumentBuilderIndexedEntity.java:
>
> > @@ -253,7 +253,13 @@ protected void checkDocumentId(XProperty member, PropertiesMetadata propertiesMe
> >  			}
> >  			if ( isRoot ) {
> >  				if ( explicitDocumentId ) {
> > -					throw new SearchException( "More than one @DocumentId specified on entity " + getBeanClass().getName() );
> > +					if ( idAnnotation instanceof DocumentId ) {
>
> So the issues is in which order @Id <https://github.com/Id> and
> @DocumentId are discovered. For some reason with our JVMs we always get the
> @Id <https://github.com/Id> first and then the @DocumentId. If it happens
> the other way around there was a wrong exception thrown!?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/hibernate/hibernate-search/pull/353/files#r2166277>.
>
>
> The old command dfuserver stop is designed to do the latter - it's designed to be called by the init system when you call (for example) service hpcc-init stop in order to ensure that the dfuserver is terminated cleanly. It probably should have added the !STOP job at the head rather than the tal of the job queue, but other than that it seems basically sound.

Not sure what stopping the process mean in that context, but issuing "dfuserver STOP=1" does not stop the process or the queue from running, accepting jobs and executing them for several minutes (before I killed it) on an empty queue.

> The code should live inside eclwatch, 

*eclwatch* is the third place that has been recommended to me. The other two were dfuplus and daliadmin. I personally think that all three look odd.

> and should look very very much like the code that pauses and resumes the thor and hthor queues - simply manipulating the wujobq so that (when paused) jobs are not passed to any process waiting on that queue until the queue is resumed.

The code is similar to those two (from which I based to make own), but still has a bit of the idiosyncrasies of dfuserver in it. How I do it (queue->pause()) seems correct, the problem is where I'm doing and which queue I'm really getting with that...
Memory leak - you need to be calling queryType()
Right. Having wait=0 make it actually wait indefinitely but asynchronously seems more hidden to me....
I misread.  The row isn't always serialized.
However if it was - or you had the compare functions - you could read direct.  However it may create a giant memory block which could be bad.
One part me breaking up the code into smaller chunks and one part me thinking someone might want to be able to change it in the future, say if some other moving part is using `.libraries`.
This is also incorrect. It should be: ``settings.ACCOUNT_TIMEZONE_CHOICES``
I think this does the same and is'nt as ugly ;-)
Rails.application.routes.url_helpers.spree_path

maybe someone can shorten it up?
мені більше подобалось
```$this->getSession()->wait($time_wait);```

чому додали проміжний метод?
Looks like there is a missing example here
Has this file actually been modified?
Thanks, fixed on 2f128d5f1555bd9a9f68d9d3e186281f539d48d0.
Why `()` for `defconst`?
Yup - it's ok because the assertion is in the block.  So, rufus-scheduler fires the events on every machine, but only the master processes them.  That way when a machine becomes master, no changes to the rufus-scheduler state are needed.
Make this a warning?
there's a discrepancy between the time formats, which caused issues between Python and SPARQL, which lead to this approach.  I'd have to dig back through notes to find out exactly what.

could this be a todo, rather than changed now?
Turbo FTP Server 1.30.823 PORT Overflow
Maybe use separate indexes for the `X` and `Y` equations:
```python
j_xeqn = 0
j_yeqn = u/2
```
(and increment both in the inner loop and use for indexing `A` below.)
There's no reason we can't make that work, is there?
Is ``float`` guaranteed to be ``np.float64`` on all systems? It is on mine, but I just want to check that's always the case.
Why not just make this a one-liner?
Fix ie6 and ie7 Pop-up Error like this:
"
Syntax error,unrecognized expression:8000
jquery-1.7.1.min.js			line:3
code:0					Character:14659
"
yeah .. use PHPCR interfaces only .. aka `` PHPCR\SessionInterface`` in this case
just be glad that commit comments don't show up in a red font.
I had the same thought about passing the array, and agree that it will be much better.

@bagnell I think this is a good change to make now, and shouldn't cause a problem when we move from a command list to a command tree; we'll go from an array to an array of trees.
I agree completely.  It would be even cooler if we could handle repeatable intervals, for example, a sweeping sensor that has the same orientation looping over and over.  In my viewer branch I started a static Iso8601 class that will eventually handle the other interval formats, and when we add support I plan on updating the above code.
`Posix`, not `posix`.
This block of code is rather dense. Consider extracting some well named methods for each stage of the process.
remvoe selse
This method shouldn't be needed follow other mentioned changes
D'oh. My bad. Fixing.
This is removed b/c we already check javaObject in line 180. javaObject COULD be null, and we don't want to crash the application if this is the case.
Date.toString() probably isn't a good value to use.

Shouldn't we expose this as a new property ("timezone") in the data object the developer passes to us?
If it isn't provided fallback to the default timezone:

    String timezone = data.optString("timezone", TimeZone.getDefault().getDisplayName());
`horizontalWrap` property
These three variables are a bit pointless. Should just build one string/regex or the other. Vars should use camelcase, not underscores.
Need better hints.  This hint is an example, but is rather vague why it is being given.  Without understanding what Modus Ponens is I could see a student thinking that you just got lazy with building the hints and added something about pie because you were hungry while programming.

For that matter, I took courses in logic at the graduate level and was taken aback by the hint on first read.
кажется, что да, так как иначе у получателя есть сомнения --- нужно ли ему удалять этот указатель.
Oh — I thought this was JSON stuff based on what @parndt said.
Could we use `let` for this?
How does this even compile? `n` is `size_t`, not a range. If you're going to be asserting here, you need to assert that `r` is empty, not that `n` is empty. Also, I'd advise checking whether `hasLength!Range` and asserting with `length` first if it's `true` and forgo this `assert` entirely in that case. There's no need to take the performance hit here if `hasLength!Range` is `true`, even in non-release mode.
s/ForwardRange/$(D ForwardRange)/
I'll wait on using logger given that it is not part of the sdk yet (it is only in the dart_lang repo). 

Meanwhile, added stack trace to error message.
minor nit: we try to keep lines under 80 columns. This might not be the case after the other changes, but just so that you know :)
this is fine grained in the detect script.
This is the relevant change; the others are mainly PEP8 changes absorbed after a rebase.
Let's add a finalizer somewhere in this chain that will flush and close the output stream. And a test that drops one of these objects, GCs, and makes sure that the file gets written.
Filenames? Wat?  Better javadoc, plz. And this should be in SyncConfiguration, and renamed to start PREF_, with all the other PREF_* keys.

This pref seems to have a life-cycle different than all the other prefs, pretty much.  Please document it, with particular reference to PREF_ENABLED_ENGINE_NAMES.  How is your pref different?  And maybe consider a name like PREF_ENGINES_TO_SELECT_NEXT_SYNC or similar.  Basically, yell out what happens on an ordinary sync, on a sync when you add an engine, and on a sync when you delete an engine.
dont start them here
this line looks a bit suspicious, the values of `@shard` and `@on_slave` are changing a lot in runtime...
No it doesn't. I'll look at adding it to the zip archive so we don't
need to worry about the path to the windows ask.
same for this class
These names are similar to those used in HttpContent and Stream classes; they all have CopyToAsync, but no CopyFromAsync.
Missing a quote here.
#Set-Location "$srcAssetsDir\Templates  removed from this line.
"trustees" is misspelled here.
no. it's always better to explicitly support this. makes it clear that it's intended to be configurable
Well that was a major bug. Would of made life interesting.
Was лишнее, убери.
perhaps it's worth mentioning that you also switched to normal/2pc? Also the log should be moved to the Log interface.
Oh, you're right. Withdrawn. :-) But doesn't this defeat the other places in this PSR where it says "have one rule for everything", like indentation and line breaks?
Although this is a migration script, is there no way to achieve this without using `eval`?
Should we remove this line entirely?
ah, i get it. 

it makes sense indeed.
The only reason I am hesitant to do this is I was under the impression the GPL had to be the first thing in the file. Which seems to be the way of most people from a quick google (https://www.google.co.uk/#hl=en&tbo=d&sclient=psy-ab&q=gpl+license+filetype:h+%22pragma+once%22&oq=gpl+license+filetype:h+%22pragma+once%22&gs_l=serp.3...3424.5439.0.5599.7.6.0.0.0.0.231.603.5j0j1.6.0.les%3Bcrnk_timediscounta..0.0...1c.1.L_gKs73DKJo&pbx=1&bav=on.2,or.r_gc.r_pw.r_cp.r_qf.&fp=903017e9244fcea3&bpcl=38897761&biw=1906&bih=921)

Give me a few to just quickly see if its a non-issue.
Why not call here just `TransactionSynchronizationManager.clear()` ?
Also have you tested it without `TransactionSynchronizationManager.initSynchronization();` or it can't happen at all, because in the real life *synchronization* is raised only through `PlatformTransactionManager`?
Falta la apertura de exclamación
This no longer has `!`.
@apex13 Sure, that's indeed the way to do it, it's just that it's more prone to other layout issues (things not showing up completely)
Do we still need these colors in here? Won't they be overridden by the prettyprint?
We should check the whole commit for this.

I also saw that we're using `prettyprint grey` in some cases, can't we remove the `grey` class in there?
It does. See http://twitter.github.com/bootstrap/base-css.html#icons
Using CDATA here seems like overkill (and is a change from what ```extraOptions``` was before). What was the thinking behind doing so?
100 correct
zi mai specific, `example: (999 ...`
Thats why I'd just remove them for now :)
move javadocs for this and `eligible()` from subclass to here
Change to getById instead of clearing out button bar
Won't the code at line 86 return (0) if count(//docblock/tag[@name='deprecated']) is 0?
Whoops, you're right -- I think this should exclude self._message_log.
.. and we do depend on Rails, right?

From a `rails console`:

```
1.9.3p194 :001 > Hash.new.respond_to? :as_json
 => true
```
:( The solution that you are proposing is much less readable. I discourage it.
I agree that `TfidfVectorizer` (to stay consistent with `TfidfTransformer`) would be a better name. Would this API change be the right moment to rename the `Vectorizer`?
This option is going to be useful pretty soon - I would like to propose (in another pull request) moving user image to the left and making it much bigger (256x256 looks nice).

Also this could include removing Paperclip styles (sizes) and going with a simple config-file based definition of :small and :large for thumbnail sizes.
It wasn't overkill, now it is. ;)
That class has now gone.
require => falseつけたのは最新版入れた？
You should convert this into a "cleaner" java object structure.
@rwmtse Can we combine this code to make it cleaner? Maybe something like;
    rawImages.forEach(function (img) {
        if (img.text && img.text["@"] && img.text["@"]["xml:lang"]) {
            var lang = img.text["@"]["xml:lang"];
            widgetConfig[key]["image"][lang].push(sanitize(img.text["#"]).trim());
        } else {
            widgetConfig[key]["image"]["default"].push(sanitize(img).trim());
        }
    });
What do you think?
I initially separated the if statements so that i fire the webworksready event once and only once in pre-BB10 webworks SDKs (was firing 3x before the move). However, i like your idea of avoiding the XHR call all together if blackberry exists. @rwmtse Thoughts?
Style nitpick: blank line before 'describe' please
just to make sure - the one with keys describing the sessions, right?