labofsahil/github-event-dataset-2012 huggingface dataset:
Good point. I went back and forth a lot attaching position() combinators and looking at where to put a withValue { } enclosure. To get pinpoint type error reporting, there would need to be more withValue { } blocks around many of the apply methods. Open to direction on how detailed to get here.
much prefer `if (self.selectedIndex = 0)` for the style.
to me it looks wrong to put `if (0 == self.selectedIndex)` with the value you're comparing to on the left side
If `-init` is used, the rest of the split view won't work anyways, so there's probably no point implementing this.
According to [this post](http://ubuntuforums.org/showthread.php?t=1702744) I can actually force it to use a different port, like 80 or something that's more likely to be open. So I think that should probably fix this, right?
после \brief строчку отступи, иначе он все воспримет как краткое описание
Note inclusion of code yet-to-be-merged into master that eliminates the old "ok" bool, displays error messages and properly exits on failure to write.
Slightly more explanation? I had to read the code to understand what this meant.  Maybe say:

If cumulative=0, each bin will contain the count of pixels having values in the range for that bin.
If cumulative=1, a "partial sum" histogram is output, where each bin's count also contains the
sum of all lesser bins.
It is a valid case. It is like placing one grayscale layer on top of the other in Photoshop. Even though they both would have alpha assumed 1, the operation over is non-commutative, and A will cover B partially or completely if they are same size.

Also valid case if A has 1 channel and B has 2 channels, but one is alpha. Hmm, we don't have any alpha assumptions for 2 channel images now, should I add? If the image has 2 channels and no alpha, should I assume channel 1 is alpha? That would be grayscale with alpha.

I vote to add these cases, but not the cases above 4 like 5,6... It makes no sense to assume anything for those.
This shouldn't be in VanillaPlayer,more like SpoutAPI...
some thing as with the previous "result"
Can we require the exact file we need, just like `require 'active_support/core_ext/array/extraction_options'`?
Good catch fixed.
I see the require, but the ArrayController does't not implement the mixin, it's just a miss ?
It is called below ;-)
Perhaps i should clarify that. We already send a restart-required request in case there are failing servers and the rollout-plan allows this. So we just maintain the state until the server reconnects. There is still some work for the :reload operation to clear this state, but i'll do that next.
This also needs a keydown handler, see above. Not necessarily as part of this PR.
I'll do that in another commit.
We should check if the trusted app is being shown before calling `hideTrustedApp`  cause it adds an event listener for a `transitionend` event that might cause troubles in same cases. With the current code, the following steps would end up with a hidden trusted UI and an unusable homescreen:
- Open app A and launch a trusted UI (via mozPay or whatever).
- Click HOME button. The trusted UI and the host app will be hidden.
- Hold HOME button. The task switcher will appear. (At this point the hideTrustedApp is called again and the event listener is added).
- Select the card containing the trusted UI.
- The trusted UI will appear for a moment and will hide itself as soon as the open transition finish, which is not the expected behaviour.
I was not sure here. However we still need a value to put in the checkbox so why not keeping this ?
I just wanted to point out that you can also use this -> [insertAdjacentHTML](https://developer.mozilla.org/en/DOM/Element.insertAdjacentHTML) to append html to the dom.
indeed. removed and pushed.
https://bugzilla.mozilla.org/show_bug.cgi?id=739094
This bug is similar but not 100% the same.
It happens when I use off-line cache (audio are already loaded)
I am not sure if a remote content will cause same problem.
Need to do some experiments.
It's OK on B2G device but freezes on B2G desktop and firefox nightly.
(Happens when seeking too frequently, slowly is OK)
Yep, that's what I understood, but as load is a defined method I was wondering If I was missing something. That's the reason I asked what was checking the condition :)
All those rules seems overspecifi. Once you have an 'id' you don't need to check further.
Use

    if (!origin)
      return;

and save an indent
This module seems a bit unconsistent. Why is there a lot of lazy getters at the beginning of UIManager that calls getElementById and some spreads across the code?
I mean the comment should either say:
```html
  <!--
    - This <style> element is only used for the example code;
    - it is not required for the real use case.
    -->
```
or
```html
  <!--
    - These <style> and <link> elements are only used for the example code;
    - they are not required for the real use case.
    -->
It is more readable, I agree.
replacing `<sup>` by `<span class="favorite">` doesn’t change much the problem: it’s still a very **specific** markup (probably for the Contacts app), whereas we would prefer to keep only **generic** snippets in those building blocks.

My suggestion was rather to remove this part of the code, and the associated style rules.
I wish this live in the platform in a more elegant way (just complaining, there is nothing to do for you here)
Create local variables |var button = this.captureButton| at the beginning of the method and that. Same thing for |var classlist = document.body.classlist|.

Do you really need this._videoCapturing or will it be ok if this is a ui only thing such as checking classlist.contains('capturing') will be enough? This couple with the local variables above will let you get rid of the need for bind in the 2 callbacks.
calc doesn't need to be prefixed anymore.
Also nit but

`"""Ensure members who are owners don't show up twice on project page."""` fits in eight characters and is one sentence (preferred for first line of docstring).
Yeah you are missing my point.

order! with optparse has no ability to tell the difference between a slice command and a switch argument if one is required. If we require that we never use arguments for switches then we are fine (current state). If we don't this looks messy to me.
distinfo can delete. But for ports tree need checksum and code-review by commiter and then added new version to official ports.
Is this file being deleted or is it just empty now?
How was this "fixed"?
Yes it would be nice!
I know this is nothing that belongs to your feature but as we don´t need `offset` any longer you could do me a favour an remove those 3 lines, couldn´t you? :wink:
Nice abstract algebra geekout.
Could use the 'loc' variable you just created ;)
Yea, L411-L415 uses it - Maybe I made have made some invalid assumptions there?
Same note about using a tuple () to split the import across multiple lines.
Fixed. After looking into it, it does do it automatically. I just assumed it didn't when I was coding it originally.
Why not have the app specific BaseView live in /views?

If we're talking sharing between apps (like a modal view, etc.) maybe we add a /assets/components/ directory that would then house the files necessary to make that component work on its own (views, models, templates, css)? Does that make sense?
See comment above about locale/i18n.
first one I agree with, second one I thought was more explicit to wrap parens. Your call though.
it should be ``if ('ext' === $pearChannelName)``
"Fully loaded YET".
How would a driver know that a click has to cause a load?

-----
Ivan De Marino
Coder, Cook, Cyclist, Gamer

Sent while standing on one leg

On 12 Oct 2012, at 17:14, Luke Inman-Semerau <notifications@github.com>
wrote:

In test/java/src/test/java/ghostdriver/FormSubmitTest.java:

> +        // Load Amazon.com
> +        d.get("http://amazon.de");
> +        // amazon search bar
> +        WebElement textField = d.findElement(By.id("twotabsearchtextbox"));
> +
> +        //Submit button on amazon.com
> +        WebElement submitButton = d.findElement(By.className("nav-submit-input"));
> +
> +        // Type beer
> +        textField.sendKeys(strToSearchFor);
> +
> +        //submit the form
> +        submitButton.click();
> +
> +        // Check results contains the term we searched for
> +        assertTrue(d.getTitle().toLowerCase().contains(strToSearchFor.toLowerCase()));

same thing as the other one. Clicking an element has no guarantee on any
driver that the page will be fully loaded.

—
Reply to this email directly or view it on
GitHub<https://github.com/detro/ghostdriver/pull/97/files#r1831027>.
How would you suggest we handle the err? I looked at the console plugin and it's not dealt with there either.
Alright, I will pass it directly to the template.
These may not be as valuable now. I put the refs to storage and window so repeat use of them could be optimized by the JS minifier. Also, is this wrapping constants up in an ```options``` object a convention we've adopted? It definitely made things look more complex when I saw ```options.store``` down in the code. I wondered if ```options``` was some new module or functionality that I had to go through to get things.

Do we think that just ```var```ing some constants at the top of a module is a bad idea? Maybe we could adopt a CONSTANTS casing convention to help?
Transitions should be named for the action performed, not for the states they connect. This should **not** be `waiting_list_to_pending`.
You don't really need this since the test harness will clean it up.
why is there a tray? shouldn't this be names different?
I guess this is an error.
I was thinking it depends on the IDE, several of them identify unused variables, etc. Also  I think in this case the cast may not be needed anyway as it's going into a string.
can we just merge these 6 lines?
The text above this example though claims it is the SDK configuration.
one thing I found the other day was the staticCache middleware.

http://tjholowaychuk.com/post/9682643240/connect-1-7-0-fast-static-file-memory-cache-and-more

not for this pull request, but something to look into adding.  also curious whether it can apply to our views...
this diagram is out of date, it's not a string anymore.
It could be mentioned here that because we're using connection id to claim rows, which will be reused in a later connection at some point, any rows left "in limbo" by a worker that claims them and then dies before processing them should get swept up by the next worker that happens to have the same connection id. This possibility of pre-existing "limbo" rows with the same connection ID also means that it's possible for `claim_objects` to return more rows than the given `limit` - it'll claim "limit" rows but then return any row claimed by that connection ID.
Nit: "noodletalk" not "noodletak" ;-)
missing if (err) { callback(err) } else { activities.push(activity) .... etc

you want to get out of the loop right away if something pukes
Sweet, figured you probably had.
Drop the space at the line's end.
Name the function __createAction, then drop this comment.
I will delete that
not a huge deal for this PR, but we should move that entire query-string thing to utils
this fails if there's no newlines, something like `.split(/\r\n|\r|\n/).length` is probably better
change self:: to static:: too.
:warning: You can't use `$this` inside of a Closure. Tests are failing for this.
parasew had edited this gem last time 3 years ago, while amerine - just 5 month.
What's need for this change?
I like it too. Ideally I'd like a way to also provide 'named' urls (for linkTo, redirects, etc) so that both the url structure and the code structure can be rearranged without needing to change all url references. It's too handy to ignore. Perhaps the array values can be either strings or some NamedRoute object/array. Anyways, seems extensible in that direction with this still as a clear simple option.
Don't think this is needed here.
`$ ->` is enough
We should use http://docs.python.org/library/os.path.html#os.path.join to join strings so that the tests will run on windows
The Docs say that Doctrine is smart enough that it figures out that the entity already has an ID and issues an `UPDATE` query on `$em->flush()`.

See <http://symfony.com/doc/current/book/doctrine.html>
Just need a space between // and the comment
You don't need the `[` and `]`. `dict()` is able to take a generator.
should do the `.strip()` after the `.split(',')` on this line. It's possible to get blank entries this way if g were to contain only spaces.
You don't actually use `i`, or `arr` here, I don't think you should bother declaring them.
We need this or we'll fail because of this change above:

return pluginFn.call( this, definition.call( this, options ), options );
haha nice. I'm guessing I was thinking this was a addEventListener( ) and added a third arguement.
Cool, one less dependency is always nice. I felt dirty with that whole portion of the code, so it's nice to see it go away
This should be done and should be relatively easy. However, you need to provide default mime types..
I choose to use ```AuthFailure``` as the universal error type, it seemed more specific to me.
Authorization mode that*? Sorry, just thought I'd point this out while I was at it.
Should be `== true` ?
I'll need to think about this.
I have no idea if you actually want this, but wouldn't this produce bibtex that looks like `{\\textbackslash}{` for `{` or `{\\textbackslash}]` for `]`, since backslashes are mapped to `{\\textbackslash}` in the second replace?

The first replace does not look correct. `$1` refers to a capture, but there is no capture in the regex. You want `$0`.

similarly, the last replace, you can do `.replace(/[\#\$\%\&\_]/g, "\\$0")` I.e. no capturing parentheses.
I renamed this to "connected" now, because they are the points that are connected to a given point
Nit: put some spaces around the equals, like `FIRST_NUM === ANSWER`
It may be worth it _at some point_ to have a boolean that defines whether there are any connectors; if there are none, 'isLocal' can always return true.

Another option might be to turn DocumentStore into an interface, and to have a LocalDocumentStore (that wraps and delegates to a SchematicDb) and a FederatedDocumentStore (which does what this class currently does).

Perhaps a TODO is sufficient now.
BodyParser is, I believe, used for parsing multi-part form data, which would be necessary to support binary files for get/put, for example.
Done and checked in
We should probably define how we want to define publicly exposed functions.

For now, we've been doing 
`module.exports.getAllTenants = function(callback) {`

We should agree on a convention.
Subscription::isenabled
I think you're right. I was assuming that it was an int, since it referred to frames, but it is a double. And, it's cleared to redefine the typedef than to use %apply.

I'll make that change.
I'm not sure about that change.
I think it would be better, to leave a blank line between the `@param` tags and the `@return` tag.
Вы правы, но я старался писать не гибкий, быстрый или оптимальный код, а хорошо читаемы и понятный, опять же по условиям задания. Можно конечно создать массив с генами и выбирать их по индексу, который рандомна генерируется, но вариант с перечислением, в данной ситуации, мне кажется более логичным.
Кстати, нашёл "Object-C way", вернее, аналогичный способ доступа к внутреннему состоянию объекта в Objective-C. Вместо [someCell->DNA objectAtIndex:i] можно замутить [[someCell valueForKey:@"DNA"] objectAtIndex:i], пускай строка длиннее, но всю идеологию доступа к внутреннему состоянию объектов выдаёт.
Базовый класс ничего не знает про категории, поэтому наивно полагать, что разработчик базового класса подстелит соломку неведомому парню (если тот его не попросит), тем более если ему лично она не нужна. По жизни я бы сделал отдельный класс.
Author says: "Ok, no problem"
ficou bacana :)
ah I remember now, forgot that I added `.paths` to facilitate this stuff. we should just do `.addLookup(path)` where path is array or string and `.paths = this.paths.concat(path)`
Happy to do that.  I was just following the constructor of the gherkin formatter that this wraps.
The purpose is to keep its appearance the same as the original. Originally, there is no placeholder and, because of that, the first item ("Standard: 7 day") is selected. Now that there's a placeholder, I still want the first item to be selected, but now I must select it explicitly.
I think we should eventually refactor ModuleFactory and ModuleUtil so that they're just a single class, and we don't have to worry about things like this.

For now, though, I don't see a problem with this being package-private, since this doesn't allow you to steal a token.
The only thing that's worth considering is whether listing every module in the build file would be a bit tedious and error-prone for bigger projects (vs separating rjs and non rjs processed files via a folder)
sweet pattern! but for 2 cases may be worth just using if else
The only thing I've found is this: http://getcomposer.org/doc/02-libraries.md#light-weight-distribution-packages
The composer link does not explain at all how to retreive the full package.
please use the Sf2 coding standards
Why is it multiplied by 5? Also, this comment can probably be removed.
The uploadFile parameter is qqfile by default.
This is changed by setting the "inputName" option in the view to a different value.

However they must match in the controller and in the view where fileUploader instance is created.
I kinda stole this part from django dynamic scrapers, but I think it could be written in a less confusing way.

TODO: Clean this up.
lol... es-discuss nerd!
I don't see how the path gives any information on whether it's a file or a folder. What do you mean?
What's the Java API?
well this is a legacy code I had no time to beautify so I am leaving this remark "for those who will come after us..." :)
It is internal so users won't see it - resources versus time battle...
The problem with this is that I depend on a specific function being defined in this `.vimrc`. If it were possible to override it, it would break a lot of things. Of course, this kind of "override" is not exactly a part of any public API anyway, but at this point, I'd consider the `.vimrc` file a constant. 

Of course, now that I say that, I wonder why it's not a constant anyway. Gotta think about where to put it, I suppose :).
En principio decidí usar structs para los keys, por ejemplo para donde se indexa por WeekIdx y DayIdx. Luego, no me pareció mal utilizarlo en los componentes de los documentos, ya que por lo general solo tienen sentido dentro del mismo y es raro que anden viajando solos.

Igual es cierto que el OOP en esta area del sistema no es el mejor. Creo que en el próximo sprint podemos refactorizar o rediseñar algunas cosas si tenemos ganas.
But, in answer to your problem of running offline, 
```html
 Grab Google CDN's jQuery, with a protocol relative URL; fall back to local if offline
  <script src="//ajax.googleapis.com/ajax/libs/jquery/1.7.1/jquery.min.js"></script>
  <script>window.jQuery || document.write('<script src="js/libs/jquery-1.7.1.min.js"><\/script>')</script>
```
Fixed in commit.
Could you please add the urls to the references above? So uses could easily download it, or at least the main site.
as task_id is the primary key, there's no reason to limit 1
same here, you should probably better wrap it inside parenthesis
You're right. It's a fourth value. I have hideously embarrassed myself by getting it backwards.
O sa incerc. Thanks for the tip.

On Fri, Mar 30, 2012 at 5:53 PM, Alex Morega <
reply@reply.github.com
> wrote:

> > @@ -15,10 +15,17 @@
> >
> >  {% macro select(form, field) %}
> >    {{ _label(form, field) }}
> > -  {{ form.select.open(field, class="field-%s" % field.name) }}
>
> nu poți să scrii `form.select.open(field, **{'class': "field-%s" %
> field.name, 'data-placeholder': 'Alege o optiune"}) `?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/eaudeweb/natura2000db/pull/1/files#r626339
>
fixed that by marking the tx as rolled back and then doing a check in the STI.
good point, I'll implement it this way.
Manik, why is a separate log necessary? Why not set the message of the thrown exception to this?
yes, i just added that method. I wanted to get the idea right first, I need to fix the cachetest too. Probably I'll have to introduce AssetWithResources interface or something like that.
Still need dot syntax here for `err.code`.
I don't know, I've belted out classics such as `asd`, `asdasd`, `asdasdasd`, `roflcopter`, `rofl` and `copter` in my PhD code. It's not (yet) publicly available.

People are going to hate me.
Machine specific configuration
You copy/pasted this from the test_lockscreen and it's not relevant to this test!
Same as in csc (Next comment)
Same origin policy, otherwise we cannot access the image data.
Why don't we just use the `fl-hidden` for this?
Can you place the `&&` on the previous line? That would make it consistent with what we do in other places.
Missing space here and line above.
For the `clear()` calls below, I've never done this with service deactivation. Did you do this to fix some observed weirdness or as a precaution?
There's no reason for this class to be abstract! All of the calls to it just initialize an anonymous subclass with a blank body. Please make this a full-fledged ActivityRouteImpl class.
Why is the order: 1, 0 2? Why not just do 0, 1, 2?
smame here and everywhere ... do you perhaps use spaces?

Please read src/README.developer - maybe that helps to adjust style.

Thanks
If you do this using xpath instead
```
$xpath = new DOMXPath($xml);
$depcategories = $xpath->query('/UshahidiReports/categories');
$depcustomforms = $xpath->query('/UshahidiReports/custom_forms');
$depreports = $xpath->query('/UshahidiReports/reports');
```
then your only ever going to select the top level categories element
This would get all categories elements
```
$xpath->query('//categories');
```

xpath is full of teh awesome :)
where is this variable used?  did you mean `self.outcome =` ?
I have not tested that. Perhaps it's best not to integrate this change, and I'll see what I can do to make the default database.yml work with the vagrant instance of postgresql. That should be possible in the chef recipes... I believe I just need to add the postgres user in the pg_hba.conf file.
Why does txn.begin/commit/rollback map WT_NOTFOUND to ENOENT?   They aren't looking up an item, I don't understand why they'd need that mapping.
The call to optimize() has now to be done in the objects that call rebuildIndex()
I do not plan to implement reload quite like this, but it was convenient for me to do this as Sniper has no ammo, and thus reloads after every shot.

In essence; Rifleman has two reload animations: Short and Long. RifleFR (Fire+reload), and RifleFS (Firing)
@carhartl ...else if( value=== undefined ) { return null; } perhaps?
I believe performance is not an issue here since the check is not in a loop.
And `typeof a==='object'` has one more char than `$.isPlainObject(a)` oO
not needed!!!
nit: I would probably use the final keyword for both of these fields, but it's not so important in this context.
Just curious, how would this `options` var be used in the View?

Even if you set your own constructor function and call the super-class you still won't have access to this `options` variable.
All the changes for double quotes replacement were moved to 09538aea66365d20f16f8ec3e886c2d6e3a62996.
So, they can be eliminated with `git revert 09538aea66365d20f16f8ec3e886c2d6e3a62996`.

Do you what a separate PR for quotes or just do not want merge those changes to quotes style?
They certainly haven't. But at least Chrome users (biggest group of WebKit users) update very fast and I don't think we should penalize the users of newer browsers.
pls drop the ###### from the EOL
please just use:

`if (xyz)`

[PSR2](https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md#51-if-elseif-else)
I agree, you can change it to just delete. Just make it consistent. :-)
You probably have cyclic imports. ;-)
So is this a right way, displaying info about each node in a for loop?
OK, for now I will leave you to your document schema. We can improve it later.
No change here?
Connected with the `Layout`, yes, but it doesn't support another property connected with an individual `Panel`, regardless of layout. I can take it out if we won't need it, but it will be difficult to re-add later, if we decide to add a property to a panel.
Do you really need `urllib.urlencode(args)`? I think if you just pass `args`, `urlopen` will do the rest.
I cannot agree here with you that `''` is worse for debugging than some text? I think setting it to not be required and using `''` makes things simpler.
(Maybe there are tabs?)
Rewrap.
Instead of `makeArray`, you could simply do `merge([], ...)`. It saves few bytes, and should be faster (less function calls).
`APEIndex` instead of `APE`?
If it's not passed in it gets set to the default so I don't think it matters, i can remove it from the `requires!` if you want.
In mzatko's latest patch, the deployable xml is wrapped around a CDATA: https://github.com/aeolusproject/conductor/pull/270

We may need to discuss this again so everyone is in agreement on the approach.
it could be pretty description as livejournal identity is, and can be just a set of symbols as google provide.
There is! It's even built-in: [email.py](http://docs.python.org/library/email.mime.html)
PigLuceneIndexOutputFormat? Do we need to be so friendly?
Peter has made a good point. Every resource in the DBpedia dump files (http://downloads.dbpedia.org) refers to an article on wikipedia by following the URI pattern: http://dbpedia.org/resource/Name, where "Name" is derived from the URL of the source Wikipedia article eg: http://en.wikipedia.org/wiki/Name.

However, we are RDFizing the DBpedia dump and not wikipedia's. In following the spirit of our other Bio2RDF RDFizers, I think that its best to stay as true to the source's data as possible.  Perhaps a good compromise would be to  add an rdfs:seeAlso from every dbpedia resource that points to the corresponding wikipedia article page. How does that sound?
Comments need some rephrasing. Maybe something along the lines of:

    // Store resolution `value` in `result` as a promise, so that all subsequent listeners can be
    // forwarded to it, whether `value` or not is already a promise.
nitpick: Please use consistent spacing
>> +            exog = dmatrix(self.model._data._orig_exog.design_info.describe(),
>> +                    exog, eval_env=1)

Oh dear, this is very broken, because you're recreating the formula
from scratch. This actually won't always work:
  http://patsy.readthedocs.org/en/v0.1.0/API-reference.html#patsy.DesignInfo.describe
and even if it did, you'd be throwing away stateful transforms etc.
Use .builder:
  http://patsy.readthedocs.org/en/v0.1.0/library-developers.html#predictions

You can pass a .builder to dmatrix instead of using
build_design_matrices directly, I didn't think of that in the docs but
it makes sense. FYI if you do then the eval_env argument will be
ignored (it's already been set by the original call that set up the
model), so no need to specify it here. (And anyway it would be bad if
prediction suddenly switched to using predict's caller's environment
instead of the original environment.)

-N
docstring should go to the class not the ``__init__``
@jwiegley Thanks

@DarwinAwardWinner - is there anything else you would like me to add?
Yea, the API is confusing. I'll ask Casey (the author) about the naming rationale on Monday.

Here's my thinking:

The "sales" endpoint returns list of image URLs and calls this property "image_urls"
It also returns a list of product URLs and calls this property "products".

That's confusing to me.

Your product class has a property which list of image objects called "images". (I like)
It would be nice if we added a list of product objects (called "products")
In the mean time, if we have a list of product urls, I think we should call the property "product_urls".

This makes it a) consistent with the other properties and b) open to add the product list
Also boooo
Are documents and messageId also extracted from the mime?  If so, is there some particular format a client must conform to for the extraction to work correctly?
And never mind about my V->T comment.  I see now that V = Value and A = Arg in the Table class, which is clear in that broader context.
This comment and the one for Nearest only apply when doing FFT convolution.  When doing photon shooting, both of these are quite efficient.
Would the code in V3Publisher work with offeroption lvl discounts?
This is good -- much better to import from settings than database.

In the future consider using this way, it's much cleaner:
    from django.conf import settings
    aws_key=getattr(settings, 'AWS_ACCESS_KEY_ID')
rpm + dpkg would be new commands, not installPackage but e.g. `installFromUrl(...)`  detecting whether it is RPM or DEB etc.
(or perhaps knowing ahead of time?)

revisit when we have a requirement.  removing altogether for now.
Do we really only want to return running apps? What about apps that are starting or stopping?
I'm not sure yet what context this method is being used in though...
If it takes a long time to stop an app, how should the the rest API look for doing that? Poll the server until it ways that the app is no longer running? Or just wait for the server's response and have a timeout?
OK je change
where is template being set?
From my point of view, the SchemaNode deserialize() method does the same thing in the right way.
"missing" case is matched with ```deserialized = attr.deserialize(None)```

Moreover, deserialize() populate the error message with values.
okay, so we still have handlers internally.
The search box locator seems to have changed to (By.CLASS_NAME, 'search-query')
Do you need to keep console.log in?
In the Java suite this method is called `uncheckedSplunkRestart`. I actually like this naming in Python better, so I think I may change the Java SDK's version.
Do a `hasOwnProperty` check here.
why ``title`` and not ``name``?
this return an empty string
k, gonna test that first then and adjust if needed.
OK, I'll leave this alone in this PR.
Да, ты прав. Я забыл про это.
May be I am reading my code wrong, but the game detection happens in MetaEngine::createInstance() method which is called on the engine loading. So the detection will actually take place.
We don't. But each individual is responsible for his/her own codes to fix.
That's what I think.


-- Amin



On Sat, Mar 3, 2012 at 5:04 PM, Geoffrey Parker <
reply@reply.github.com
> wrote:

> > @@ -14,43 +14,43 @@
> >  * news message.
>
> I thought we weren't having people own specific files.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/w7cook/OpenBook/pull/20/files#r515149
>
"SELECT ..." lines up with the open paren of Post.find(
Should probably be  java-7-openjdk-amd64
Revert all the sdisabilities references into proper references of the disabilities var.
I like to use NULL instead of nil when it's a C type, rather than an Obj-C object :-) (even though they're equivalent)
Yeah, we could apply the morphic transformations here to fix that behavior. However, I thought that it might be even better to change how rotation is done for paperjs morphs by using paperjs transformations and rotate the paperjs objects on the canvas itself... for simple usage of lines/curves the current behavior should be OK, though.
Since when do we use typedefs?
I'll add in better comoments and rename some stuff.
Поясни, что значит условие threadsCount == 1 в этом месте?
По идее, это надо было перемещать в пакет chat, а не в рутовый.
Apple says that viewDidLoad will be deprecated soon, and shouldn't be used. So, let's put this stuff in dealloc
I am not a fan of adding another config variable. However, I don't see any way of getting to the source file from within the Annotation. Therefore, I don't see how else to get the path of the included file. Any ideas?
I'll remove the TODO. Thanks!
I've now changed the ring buffer class I was using to a GPL one. The developer of the original ring buffer did not explicitly state any license, so I decided it'd be less risky to switch to an implementation that was definitely GPL.
as_float_array has copy=True as default
If this is just for the score function I think it's ok to introduce it in the existing mixins (although some score functions might not take sample weights into account when overridden in a sublclass). Are their other methods from the mixins that are impacted by sample weights?
Yes, it is.
I didn't know this was there. Why is ``fit_transform`` in ``TransformerMixin`` but ``fit_predict`` is not in ``ClassifierMixin`` and ``RegressorMixin``.
`alpha` is not explicit to me.
I hate to say, but I don't think that this reference is a really good reference to explain the discretization scheme implemented by this function.
? дизайн і верстка це прерогатива дизайнера і верстальщика. якщо будуть проблеми з відступами, то дизайнер на це вкаже і верстальщик виправить.
Same in this file.
Suggest naming these 'origXXX' vs. 'oldXXX'.   Semantically more accurate.  Also - do we need all of these?  They suggest we're keeping them around because we're injecting new values, but only some of them are being overridden.  What's the reason for copying the pointers if not because we're changing them?
I really don't like how this is being done. Firstly, tons of stuff is getting leaked. I'd prefer this to be done like the others (timeout is another good example) in setupEvents. I'm not sure how the `source` property is set for button clicks but it seems to be correct so maybe it's automatically set by V8. Then you might just need to set the `socket` property to that value and clear the `source` like timeout does.
Same for the others.
We have simualtor for bb. Can't we use it instead everywhere?
we used to have "! " rather than "!" to better highlight negations. Not anymore?
The Lucene Spatial contrib module is deprecated https://issues.apache.org/jira/browse/LUCENE-2599
Removed in 3.6 and 4.0
This was the starting point of my incoming at Hibernate Search level =)) see https://community.jboss.org/people/nicolas.helleringer/blog/2011/09/22/hibernate-search-spatial--first-draft
There could be > BUF_LEN events, although they're being thrown away anyway...
Heh, unfortunately, the code compiled, executed and passed all tests.

I'm trying to find an automatic conversion that would lead StringArray to char pointer, that would explain why.
yes, i put a `use Symfony\Component\HttpFoundation\Request;` in the top, like `use Symfony\Component\HttpFoundation\Response;`
Did you check for deserialization compatibility, i.e. run the old plugin, then update and browse this action using the new plugin? I think it will be OK (just set to null by XStream) but ought to check it.
well, if you add classes later extending the older one, it does not break BC for the user (a child class passes a typehint for the parent class). when using the listener, you can know which event was fired by knowing which method was called, as the method name matches the event name in Doctrine Common.
My personal preference right now is to not print this hint at all. Just add the commented-out entries in john.conf to show that they are supported. But it's arguable of course :-)
of course I meant now ;)
unsafe. Use `None` instead.
```
products = [x['product_name'] for x in response['hits']]
```
?
I've pushed these functions to master already.
shouldn't the default be `django.utils.translation.get_language()`?
I thought there were still open discussion about it.
For instance, do you really find more readable and less character-typed the two alternative you suggested?
PHP 5.1?
where does this ``twitter_app`` comes from ?
There is the `Publisher.fromCsv` method which (sh|c)ould probably be used here.
I think you can combine the two "if" statements into one with an "AND" clause ("&&")
We started out with that idea but now that we are parameterizing our data we can do it safely.
Want to just name this guy FirebasePusher?
not sure what do you want to test (probably read(..) method) but there is readFully method
This `if` is just

~~~clojure
(or (= x y)
    (= (sort (replace {14 1} x)) (range 1 6))
~~~
The hand `[2 3 3 3 6]` would fool this one. Unfortunately we have no tests for this. The solution would be to first check if the hand is strictly monotonic: `(or (apply < arvot) (apply > arvot))`
@bodepd this is because [according to the documentation](http://keystone.openstack.org/configuringservices.html#configuring-swift-to-use-keystone), both ```keystone``` and ```authtoken``` middlewares must be declared in the ```pipeline = ``` directive.
And as the module loads a ```swift::proxy::…``` class for each middleware I prefered to split rather to add a special case.

Hope it makes sense.
```ruby
  'DefaultTarget'  => 0,
```
Just checked that, there might be the case that response is nil which means there is no "body". Therefore brandon is right. Fixed, thanks...
Yeah, good suggestion ;-) My code is lazy ;-) but yeah to align with my previous comment I can even check if j2plauncher.exe exists or if wab.exe exists and then consider abort execution if such files are not there.
There seems to be a lot of duplicate code between `build_laplacian_pyramid` and `build_gaussian_pyramid`. Maybe this can be refactored a bit.
It was just my initial impression. Plus I just realized that `np.bmat` returns a numpy matrix, which I'd prefer to avoid (it could be cast to an array, of course, but that's just another strike against my approach). Ignore my suggestion. ;)
I think you need parentheses:


```ruby
item_outside_array || raise(Exception)
```

That said, what about moving the exception into `disallows_outside_values?`? That makes it as close as possible to where it's actually being "used".

Like:

```ruby
def disallows_outside_values?
  if value_outside_of_array
    disallows_value_of(value_outside_of_array)
  else
    raise CouldNotDetermineValueOutsideOfArray
  end
end
```
JulianDate is ALWAYS in TAI now, users should never be passing around instances of a JulianDate which represent UTC.
This can be moved inside the `if` block below.
The material class needs to be able to do a recursive combine for modifying existing materials, so it's good to have this utility for that reason alone. Other areas of code that use combine are more straightforward, but it's still nice to have this utility instead of using some for-loop to combine properties together.
Any reason not to make this a script CFC?
Yeah, it's hard coded cause it's hard coded in CSS as well...I don't like it either, but I'm content to wait until we properly separate styling oriented CSS from functional oriented CSS in preparation for better theme support.
Spelling nit `paramaterized` -> `parameterized`
`if` is a statement not a function, so you should put a space after the `if` (like you've been doing for `for`)
If you move the files and packages variables to the class level you won't need the checkenabled function at all, and you can avoid calling a method on the superclass directly (good in case we end up refactoring to more plugin classes).
The problem is as usual with arrays as ranges. If I'm not mistaken they serve as output range just fine.
But then when passed to copyInto the changes to ptr+len won't get reflected to the outside.
Destructor call again.
The complete file is not i18n friendly, there's a seperate issue for that
I did this to make sure that the fields exist for a `VideoEdition`. Is there a better way to make sure that these fields are present as a test?
Could use the three-argument form here rather than concatenating.
without those reformattings the read would have been much easier...
While it makes sense to return this, this isn't how it was done in the legacy player.  We need to return the local member `speed` instead.
Is memcache a valid package name? When I tried to install it through `pip` I did not find any packages with that name.
Service Bus supports sending messages with empty body. However, I have added tests to verify that behavior.
I'd go with ```#essentials-section``` for consistency with the file name, helper name, etc.
But at the same time, wouldn't calling get_attribute here have already triggered those side effects?
URI дожен в нее передаваться снаружи
if we drop the autoload file, we should also drop the .gitmodules and completely rely on composer. or we keep both for a little longer.
Can't this function just call a private helper function that returns the value from `NumberOfPositions`? Seems like lots of duplicate code.
I kept the case one, too.  Did that in most cases, still.  Does that still seem like too much?
Seems like the comments indentation is off by a few spaces
`elsif dump_header.nil? and verbose`
Does anyone know if maybe we should use `version(D_LP64)` here? Perhaps not yet since most Windows OS versions are x86-compatible..
Fixed bug, where a soft limit of 0 didn't disable the soft limit.
I actually don't even think its necessary to mention EventDrivenConsumer and PollingConsumer. All you trying to say is that this parser will produce a complete Endpoint or just a MessageHandler
@brad Are you positive this works, and that it doesn't throw a exception that the request to Mailchimp isn't right? I had a problem with my ```requests-facebook``` lib when passing ```params``` and ```data``` to post. Maybe just posting one, like you are, is alright? Just wanted to be sure.

P.s. are you familiar with writing tests?
Maybe you should indicate from where to call phpunit because only works when is called when the workdir has a phpunit.xml[.dst]
end_formatted was missed
Nit-picking: if we use one period, should we use two?  Or maybe just make the period a semicolon?  I dunno.
Would you just use :locals then?
The method is too specific. the $parameters key cannot be configured and the catalogue is hardcoded. There is not benefit for this method as it.
Looks like you added some trailing whitespace here
Y'avait pas un paramètre de projet git utilisable pour dire "nous casse pas les couilles avec le retour chariot ?"
In fact it is 2011 :)
This is not new, we've had it a long time.
return values are ignored!?
ino ba margin bede
va hatman first-child ro estesna kon
bara border radius lazemet mishe :D
is the blur() necessary to achieve this behavior?
Just realized that trim() is a part of ECMAScript 5, which WebKit browsers support.  Excellent!
We customized the event framework you guys used for Ripple to better fit our webview model, where events are fired globally on the native side, and we wanted to expose an encapsulated object. To acheieve this we changed event registration to take an ID and a type.

Now Brent was saying there are places in the browser where they want to listen to all global events, and we figured it would be useful in this module to track all of the existent webviews and encapsulate them.
Alright, thans for your explanation.
You mentioned an extreme use case here:) One of my colleagues tried to make the example work. He had problems with setting interfaces and so (there might be problem also with firewall sometimes). When I mentioned a possible change of port-offset, he got it working in a while. 

I agree that for production use and debugging it's better to have different adresses but for such a simple use case when the goal is to make the example working, it's easier just to change the port-offset in one file. 
OTOH, I can put this tip back so that users can choose which one is better for them ;-)
If path isn't being used, please remove.
This entire file is wayyy too long.  So many handlers.  Can we split them up into logically separate files?
maybe call this webworks_utils.cpp, maybe i'm just being picky, I guess we can rename the lib to utils or something like that, can't think of a better name, of course that means the other extensions that use this need their manifest updated as well and meta.mk needs to add the native directory as an include path so it can find the header, if you aren't up for doing all this I can help out tomorrow
I was thinking about this one when I mentioned that it makes sense to be an ordered dict.