labofsahil/github-event-dataset-2012 huggingface dataset:
по соглашению просто complete, и проверять потом его методами с вопр знаками. типа @video.complete?
For now, I suppose so. If you have any other ideas, that would be great.
Oops, I also missed it. Should I send another PR ?
the old ConstraintBuilder code
OMG this is some old code needed to be deleted.
no robots :)

http://beta.guardian.co.uk/robots.txt
DBsettings.pl is in the directory overpopulation.org, as is settings.pl - I don't think I currently use either one. They were for security purposes.
I think we should first add "collections" and then concentrate on FRB.
TBH, `extendsPrototype` / `implementsList` / `implementsPrototypes` or anything alike would be better than synonyms.
I think this should be 

    und.each(kpi_json, function (kpi) {
      if (! kpi.timestamp) {
        kpi.timestamp = new Date().getTime();
      }
    });

kpi_json is getting sent as a list, no?
createPrimaryUser was never used, so I removed the declaration from user.js as well as its tests.
I weep to think of what's in that whiteboard.
In routes.php?
you already set keyCount to activityKeys.length, so maybe put it to good use here? :)
Not fixed there
$error is an instance of ErrorException -> put it as previous exception to the thrown exception
I think the correct way is exclude `vendor/` in the main .gitinore
Because it is not needed according to the [PHP documentation for the S modifier](http://php.net/manual/en/reference.pcre.pattern.modifiers.php): "At present, studying a pattern is useful only for non-anchored patterns that do not have a single fixed starting character."
No -- because if we later want to add dependencies not installable via composer, we'd likely include them in this directory as well. As such, it's better to create .gitignore rules within the vendor directory itself.
Try to call this to_s
-> `completed: todo.done`
I don't think this file is needed
Since jQuery's `css` has the same form as `attr` and `data`, namely that it accepts a key and returns it's value, the `css` assertion should have the same form and behavior as the `attr` and `data` assertions:

### `css(name[, value])`
Assert that the first element of the selection has the given CSS property, using [`.css()`](http://api.jquery.com/css/).
Optionally, assert a particular value as well. The return value is available for chaining.

    $('#header').should.have.css('background');
    expect($('body')).to.have.css('background-color', '#ffffff');
    expect($('body')).to.have.css('font-family').match(/sans-serif/);
Maybe we can condition the emission event when really an item is focused.
Can parent be null?
``symfony/console`` should be in the ``suggest`` section (as it is optional, even if using the CLI tool is recommended) and eventually in ``require-dev`` (if you want it for the tests but the testsuite does not rely on composer currently). And the constraint is too strict. You should not force using only the 2.0.12 tag of Symfony as it would force changing Doctrine each time a bugfix release of Symfony is done (for instance now as the latest is 2.0.13) and would break things if you have a library forcing to use 2.0.12 and the other one forcing to use 2.0.13 only (impossible to use them together whereas they would generally work with any 2.0.x release as they are BC)
:+1: for abstracting this out
a `else` maybe ?
Defined by TestCase. So I can't change it.
In the else clause here where we switch back to a default easyblock:
                modulepath = "easybuild.framework.application"
                class_name = "Application"
This should be configurable in easybuild_config.py
Is this needed? Won't the cp overwrite?
I think it comes indirectly from pig.
If we depend on it, we should add it in the redelm-pig pom as scope test
Indentation here looks like 4-spaces or tabs (maybe a mix since one of the lines is not indented in)? Please me it all 2-spaces.
"latte" not "late".
Given that we have reverted the choice parameter to a string parameter this concern is not valid anymore, right? Can I go ahead and land the changes?
I've just read your comments for the TF2 plugin, will fix :)
This is the same thing I did with the work blurb a few pull requests back: separated the main content from the surrounding li so we can call it in multiple contexts.
If the source if always ready (`is_ready` below), shouldn't it be `Infallible` ?
These are still not valid (although Scaffold does attempt to interpret them). Take a look at the washington post translator  (https://github.com/zotero/translators/blob/master/washingtonpost.com.js#L101). More importantly, you do not want to generate these test cases by hand.

You are using [Scaffold](http://www.zotero.org/support/dev/translators/scaffold) right? If so, open up firefox, open Scaffold (Firefox Menu -> Scaffold), in Scaffold open your translator. In Firefox open the page you want to test. Go to the "Testing" tab of Scaffold and click "New Web". Then Click "Save", You can also run the test by selecting it and then clicking run.
toArray copies the elements of the list into an array I think. Might depend on the implementation, but there are better ways to loop this.
http://drupal.org/coding-standards#linelength
nit: we don't put spaces inside brackets any more, so use `ANGLE_LABELS[ANG_LEFT]`, not `ANGLE_LABELS[ ANG_LEFT ]`
Why not {}?
Change "compresed" to "compressed"
if this was why to make two methods you could do:
def commit_to_repo_and_hash(commit):
    return (commit.repo.remote().url, commit.hexsha)

and 

get_commit_url(*commit_to_repo_and_hash(commit))
We definitively need to call `setOptions` instead of instanciate the table in this way for readability.
I would use the global comment we have in the mozdownload.py file.
These are referred to by other plugin docs, and lead to hyperlinks pointing to not existing docs.
Those @@ should lead to these pages to exists, even if basically empty.

E.g. chordrest.html links to durationelement.html ( "inherits DurationElement"), but that doesn't exist.

Bye, Jojo

> -----Original Message-----
> From: Nicolas Froment [mailto:reply+p-1164094-
> 65b81b42ae26c31b8819f1ae5f17aa18bad61062-1786669@reply.github.com]
> Sent: Saturday, July 14, 2012 5:34 PM
> To: Joachim Schmitz
> Subject: Re: [MuseScore] some more plugin doc updates (#11)
> 
> > @@ -22,7 +22,7 @@
> >  class Spanner;
> >
> >  //---------------------------------------------------------
> > -//   DurationElement
> > +//   @@ DurationElement
> 
> Why did you add an annotation here? DurationElement doesn't seem to be
> exposed to the plugins
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/musescore/MuseScore/pull/11/files#r1164094
Comment on added line (4)
Yeah, but I wasn't saying put the loop in the template. Put the loop in the view, and make the template just the table with an empty `<tr>`. Then say something like
```
for i in 7
    $('tr').append('<td><div>Date</div></td>')
```
I think this should just be an empty array as there are no variables to bind to in the CQL query
In another PR I recall you using JSON.parse(JSON.stringify(..)); to avoid cloning methods to avoid methods. should we do that for this as well?
This should probably be radio
I'm not sure. What troubles me is the type ``String``. What about using an ``Option[IPath]``? But if it can't be fix, it is fine like it is now.
It would be truly good if this could be simplified/broken into pieces. Personally, I find this code hard to parse.
Right, just making the array up front.
Okay I just tried it with a Non-ASCII name in quotes and it worked just fine in GMail. I'll remove the commented code.
should be made available through plug-in manager.
Comment this out, not necessary when we have x:y coords.
Shouldn't it be :ca
Why did you add a space?
Here is where you'd typically include the `"license": "MIT",` line, or whatever license you like.
Ага, make sense: Преимущество этого алгоритма что рендом будет работать не со статичной длинной массива индексов, а с такой, которая постоянно уменьшается.
 А после удаления объекта из середины массива все остальные сдвигаются на единицу? Т.е. Если я удаляю 5ый, то 6ой смещается на 5ое, а 7ой на 6ое и тп?  Хотя это не важно ведь каждый обект содержит свой уник порядковый инт на каком-бы он месте в массиве не стоял ..

Меня это навело на менее абчтрактный алгоритм и более жизненный: вместо перемешивания корзины с номерками и вытягивания  порядкового номера по которому брать шарик с номером с полки, мы можем перемешивать сами шарики  (вроде как у массива есть рендом сортировка) и брать первый на полке
Хотя в жизни, в отличии, конечно проще перемешать бумажки чем шарики на полке))

Попробую этот вариант. 
С оценкой не ждите, т.к. Сегодня я не поправлю , да и по сути это не влияет на условия в задаче. Текущий солюшен не подойдет только  в случае больших объемов данных

СПС
Алекс
Отправлено с iPhone

Nov 9, 2012, в 9:31 AM, Hadush <notifications@github.com> написал(а):

> In DNA/DNA/main.m:
> 
> > +    //использовали ли мы его или нет. Если нет - то используем и складываем в спец. массив и тд
> > +    for (int i=0; i < percentInNumbers; i++) {
> > +        int random_num = arc4random()%[self.DNA count];
> > +        NSNumber *random_numAsObject = [NSNumber numberWithInt:random_num];//превращаем число в обЪект
> > +        
> > +        if (![numbersUsedinRandomChange containsObject:random_numAsObject]) {
> > +            [numbersUsedinRandomChange addObject:random_numAsObject];
> > +            int random_num2 = arc4random()%4;
> > +            [self.DNA replaceObjectAtIndex:random_num withObject:[codes objectAtIndex:random_num2]];
> > +//            NSLog(@"array count:%lu", [numbersUsedinRandomChange count]);// должно равнять заданному параметру percentage
> > +            
> > +        }
> > +        else
> > +            i--;// чтобы действительно изменить x заданных процентов 
> > +    }
> > +
> Я изначально тоже делал как Вы :)
> На данный момент мне больше всех понравился вот такой:
> 1) Формируем массив индексов длиной равной длине ДНК, значения которого NSNumber
> 2) Рендом выбираем из него значение от 0 до длины массива индексов
> 3) Меняем элемент ДНК с индексом полученным на пред. шаге
> 4) Удаляем запись с массива индексов и повторяем все начиная с шага 2
> 
> Вы будите исправлять/пере заливать свой проект? Или можно выставлять оценку.
> 
> —
> Reply to this email directly or view it on GitHub.
>
можно более строгое условие поставить - percentsToChange <= 0 чуть быстрее будет выполняться :)
должно быть далее условие
if (self) //если в конструкторе базового класса все прошло удачно
             //и он вернул корректный объект, а не освободив память вернул nil
{  }
Not a big issue, but maybe it shouldn't print square brackets if no `tag` has been provided
Maybe this is ok, but m_closeLiveBrowserCallback is a CefRefPtr<> and the common way I see to check those for non-NULL is:

  if (liveBrowserMgr->GetCloseCallback().get()) ...
Apologies.  I started adding this before uris were available in cucumber.js.  I then did a fetch of the upstream changes which introduced uris and clearly didn't remove this spec.  Will do so and push up in the morning.
Huh. Not sure what happened here. Push got a bit confused, I think.
As I mentioned above, I would put this as examples/add/server/main.go
`*mMyRepo` is a nice construct here... Get's you think a bit, but should work nicely. I think, usually "mMyRepo.data()` is easier to understand; but we don't have that (yet) :smile:
You mean non-in-place (on copies)? I figured folks could do epochs.copy() to experiment. I think it's cleaner than having every function give the option to return a copy or not (and only have that where it usually makes sense to return a copy).
can you add also the verbose param to the ``__init__`` that is None by default and defaults to raw.verbose? if True or False then it overrides raw.verbose
the verbose argument should also be added to the docstring

verbose: bool
    Use verbose output.
I don't think it is necessary to have this restriction. crop() could just create a mask and the epoch are cropped before they are returned in next() and __getitem__()
when I see scaling=scalings (singular plural it raises a red flag...)
Forgot to deprecate this. I'll also update the entire concept index here.
after -> before
this code never worked as it attaches the transformer to the wrong place. It should be

```php
<?php
$builder->add(
    $builder->create('issue', 'text')
        ->addModelTransformer($transformer)
);
```
put `$ ` (with a space at the end)
controller ?
请添加标点
How about pulling this out into `#storage_class_name` and `#store_class` private methods.
if you change the 3 500s to 50 in this test and divide the 1000000 and 100000 each by 10 the tests are very fast and still trigger the Overflow portion of the code.
И в чем тогда разница?
> каждый чих
> отметка всего как прочитанного
Если ты это делаешь регулярно, то просто один раз поставишь галочку Do
not ask me again.

И вообще, пойду у Техника спрошу.
Here the user preferences should be consulted for **vintageex_osx_terminal**. I think this is missing from the linux counterpart too. Update this if you want or I'll do it myself after the merge when I get around to doing it.
use ``@param  string|null`` instead of description
Please restore copyright year
You are correct, good catch.
cache.add() will fail if the key is already present in the cache i.e it will not update the cache. What you should do is to check the return value of cache.add(), which will be False if the value was not updated. If it is False, do cache.set() otherwise cache.add().

Moreover, you should first check if the current_space is already in the cache first, then try to add or set or get it.
erm... "lokalisering" might be a better word, but personal, "plats" isn't that good
Disabled it on the api controller anyway
Surely there's a way we can fold this into each class' __repr__() instead of this? __dict__ is effective, but it's a carpet bomb and a lot of what it'll hand us will only clutter our logs.
What are the cache contents at this point? Do you wanna be emptying the cache here to make sure that the entries used are the ones retrieved from the cache store?
And rename JGroups file too
@galderz You probably missed my comment on this? We need uniqueness in both segments and owners collections ...
Isn't a UUID simpler & safer?
The internet lied to me! *shakes fist*.

So here was my quarm with the UTF8String… if we are given the length of the header surely we can't assume then that it's null-terminated?
"With the exception of *spectral*, the above colormaps are based on those of the same name provided by Matlab."

I don't know how they were derived or extracted; alternative wording suggestions are welcome.
@apex13 Can we remove `userpages` here and in all the other files?
something is weir with indention here.
The problem with IE8 is that if the icon update (_addIcon) is triggered too early, the rendering engine does not pick up the class change and the icon is stuck until you hover. The new onliszt:ready handler above will fix the initial-icon-missing loading problem in IE8, but will not fix the subsequent mode changes. I couldn't find an event that's triggered late enough.

Triggering _addIcon by hand afterwards fixes the issue, so I presume using a timer would do the trick - but it's ugly... In any case now the icon will always be there, just not always the right one - definitely an improvement ;)

@chillu would you like me to try installing a timer at all or it's a no-go?
Minor note, but could you please tidy up the indenting here?
Same protocol relative url approach should be used here.
Try this again now - download.xml should be included now
Thanks for an explanation. I see the point, I would just introduce a variable like "mag_eq_tolerance = 1e-5" and made it clearer that this is just for float-point operations imprecision and (as you said) the tolerance value is several orders of magnitude smaller than the typical reported magnitude.
Are you sure you want to override the parameter ``$container``? This often makes code less readable.
Such method not exist in `Response` class.
Hi Dmytro,

I've seen examples using both. I started out using strings but then
switched to symbols.
I simply haven't taken the time yet to go back and change them all over to
symbols.

We can also use the new 1.93 syntax to set values instead of the earlier
hackrocket ( => ) syntax.
My next time through the models I was planning to change them over as well.

i.e.

*validates :person_id, presence: true*
*belongs_to :store, class_name: Store::Master*

Does this sound good to you?

Cheers,
Charles

On Mon, Sep 10, 2012 at 10:51 PM, Dmytro Kovalov
<notifications@github.com>wrote:

> In app/models/user/actor.rb:
>
> >    belongs_to :person
> > -  belongs_to :store, :class_name => "Store::Master"
> > +  belongs_to :store, :class_name => Store::Master
>
> Should :class_name be a String or Symbol?
>
> If both work, I'd vote to change all to Symbol. But if I remember
> correctly I've changed to String because Symbol didn't work.
>
> I can be wrong though.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/SimplyCubed/webtickets/pull/1/files#r1572970>.
>
>



-- 
Charles Green
cg@simplycubed.com
@charles_green
Skype: charles_m_green
Mobile: +1-810-449-3119
Direct:  +1-810-515-8345
Japan: +81 50 5534 5335

*SimplyCubed, LLC*
www.simplycubed.com
@simplycubed
Naja, du machst überall ein `replaceAll` mit jeweils dem selben Regex. Das macht keinen Sinn. Man könnte zB eine Funktion machen, die mit einem Compiled Regex die Ersetzungen durchführt und auch mehrere Platzhalter unterstützt.
I'm curious, is the `/` in the "/>" ever needed here?
![clap](http://assets0.ordienetworks.com/images/GifGuide/clapping/taxidriver.gif)
A comment on what m.group(2) is would be helpful
Tags should be self contained so it isn't a good practice to make the user manually included other resources.

What you could do is either merge the CSS of prism into the controls CSS or (better option) inject the link tag into the HEAD when your control is first loaded.  I would do the same with the prism.js dependency.
drop the empty lines below please
Should we give it a more useful name? like `getReplicationFactor`?
You told me that it's better to have test methods *not* throw exceptions, and I've found the same advice several places online.

Remember that the results readers still are marked as throwing Exception.
Учту.
No no no. You should not try to copy logic of their implementation. We are not creating new users here. We already have a user created. A lazy user. We just link that lazy user to user's BrowserID. Setting e-mail address at the login. See how we are doing this in other backends.
Do you really need ID on this div? You could just store jQuery div object into a variable and use that as a target of `appendTo`. No need to do the query against ID there.
sorry i don't get it. What is here constant string?
It is common in jQuery to chain things together. So you do not really have to define a variable here, but just chain all following calls together.
No need to rename a module if there is no conflict.
Don't the wording. Also if you can, supply links. Don't have users search for how to get to the solution of their problem.

BTW, I would not check this in the template. I would check in dispatch or somewhere. If e-mail is not configured, they don't have a reason to be on this page anyway.
I know where to put it. I do not know what should be the content of this fragment.
`<a href="..."><img src="..." /></a>`?
Fake edit: I sense that by fragment you mean all the code that constructs item?

I think values used in sekizai blocks must be known before rendering. I am saying that there is little to no gains using preloaded HTML as content of the item is very dynamic so you would still have to go through all this procedure:
1. Load HTML
2. Populate variables
3. Show/hide parts of not yet rendered HTML that are missing for current item (if node does not have website for example)
4. Insert item into list
Plus you would still need to either track all items in an array or enumerate/uniquely label/ these loaded HTML blocks so that you know which to add/remove when switching nodes.
But MongoEngine doesn't have a choice field? So I create custom TypeField?
Здесь проверять не надо
Remember the old wdb parser derp? i bet we didnt fix them all yet.
I'm confused about this bit. I think this whole method needs some treatment with comments.
Twig does not have its own CS for the PHP code. It uses the Symfony2 ones (the Twig CS are for templates).
And indeed, this rules comes from PSR-2, which is the base for both the Doctrine and the Symfony CS
@Koc I will push a new branch with such realization will see where it is good one.
* 1 -> $TRUE
* add a comment explaining why
Hey, we have upgraded to using jQuery 1.7.1. Can you update your code to use the new version please.
That's true, and the detailed documentation is added in the getter method for this field. Which customers are going to use.
0.5 according to the style guide.
This change looks a bit dangerous to me: have you taken all views which are using `header` in regard?
Added a conditional compilation macro?

etag appears after Nginx-1.3.3. It's not in the latest stable version Nginx-1.2.4 yet.
Instead of polluting the global sys module, use PyInstaller.hooks.hookutils and place your global variable there. That module already contains hook-specific code, so it's ok adding a pubsub-specific variable/function there.
No hurry :)
Do you mean I should write a new _nbinom_logpmf function and call that from _ll_nbin?
If there is a convergence failure, does "the final solution make sense"
Why not set just `title`?
Mismatched log levels for entry and exit.
Changes to this class seem unrelated to log4j cutover.
This can be simplified:

```clojure
(defn- submap? [map1 map2]
  "Are all the key/value pairs in map1 also in map2?"
  (every?
   (fn [item]
     (= item (find map2 (key item))))
   map1))
```

I'll just pull in the branch now though and change it then. Thanks a lot for your patience :-)
Don't disagree.  Should I make the same change to the Airy class, where obs is also used? (This consistency was the reason I selected obs).

On this subject, I would also like to make the Airy use `lam_over_D` rather than its current `D` = 1 / `lam_over_D`, but was thinking of pointing that out and fixing that in the radii -specifications issue...
Okay.  Hmmm, in that case, would you mind trying that initial rebase?  Is it possible for you to do that and then me to do the rest?
Lets get this right now. It's easier later on and serves as an additional reminder that there is more work to be done.
This comment seems to have been copied in from other code; I don't think it applies here.
Yes, on reflection it just seemed to me to be more natural to put the wavefront as a unit amplitude pupil with varying phase... I now only do the dx**2 division for the real space output, which is the PSF, so that SBInterpolatedImage can use it properly.  These conventions are reflected in the docstrings, but I'm not totally happy with them.
(could open an issue as an fyi for those who are not reading this pull request)

On Jun 13, 2012, at 5:24 PM, Barnaby Rowe wrote:

>> @@ -364,7 +367,7 @@
>>             double flux=1.;
>>             double re = (nargs>0) ? dargs[0] : 1.;
>>             dbg << "**Returning exp with flux, re " << flux << " " << re << std::endl;
>> -            return SBExponential(flux, re/1.67839);
>> +            return SBExponential(re/1.67839, flux);
> 
> This is Gary's code, so I guess it's just an old typo, I'll update it.
> 
> On a related point, maybe it might be worth a discussion to retire SBParse?  It does require a bit of maintenance to keep track with other changes, and I think that the Python front end supersedes it.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/GalSim-developers/GalSim/pull/189/files#r980266


-------------------------------

Rachel Mandelbaum
http://www.astro.princeton.edu/~rmandelb
rmandelb@astro.princeton.edu
All over here, get rid of this
Should this file be checked in?
i think this XML is nice and tight, not having needless additional nodes just to make it feel more java-like.  the DTO impl might look naive having all entries in a single list but it's fine to work with, i've found.
Very interesting. I hadn't considered expressing the idea of side-effect only code with different syntax. Given that context, I like this change.
Yes, just to create visual grouping of lines between adding to the connection/server array, and adding both the server an connection keys to the returned value :]
Interesting, since `COUNT` queries aren't cached, afaik.
Maybe a good idea to make it possible to force a download?

An simple setter like setForceDownload() which always sets the 'application/octet-stream'.
In some cases the mime-type is known but you don't want to file to opent in the browser, namely pdf en Office documents.
Or downloading an HTML document.
Why are you comparing to '5.4.0RC1' version? Why not just '5.4.0'?
You mean uintptr_t?  That's in stdint.h just like uint8_t.  It is a type that's wide enough guaranteed to hold a pointer on any platform.  Now that we have people trying to compile code on 64-bit machines, we should be changing all of the opaque device handles to be uintptr_t or maybe into an opaque structure definition.

I'll probably sweep through PiOS and make that change so that we can compile unit tests and sims more easily on 64-bit machines.
It that wasn't defined let's just removed the method. Wdyt?
System.in закрывать не надо (он принадлежит классу System, он должен заботиться об освобождении ресурсов).
Здесь закрывается System.out. Некорректно. И надо закрывать стрим в try..finally.
Тут ошибка компиляции. Я поправил.
Зачем префиксы у аргументов такие?
А в finally есть закрытие файла?
Что за пакет? Он должен соответствовать пути.
Tables are awkward in Sphinx apparently, these sorts of formatting issues need to be decided upon
lambda functions will prevent your code to run in parallel with joblib. Can you avoid it?
Ok for y but X can be huge and we should avoid unnecessary copy as much as possible. 

Le 9 juin 2012 à 14:42, ibayer<reply@reply.github.com> a écrit :

>> @@ -193,6 +246,72 @@ def fit(self, X, y, Xy=None, coef_init=None):
>>         # return self for chaining fit and predict calls
>>         return self
>> 
>> +    def _sparse_fit(self, X, y, Xy=None, coef_init=None):
>> +        X = sp.csc_matrix(X, dtype=np.float64)
> 
> Is the check for really necessary / useful? Besides that y is small compared to X, (np.asarray:
> Convert the input to a `chararray`, copying the data only if necessary.)
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/scikit-learn/scikit-learn/pull/891/files#r955587
I added a bit of details, as @pprett did for Gradient Tree Boosting. Early stopping of the learning procedure is also now documented (which serves the same goal as warm restart ~). 

Proper early stopping on an independent validation set should be done by the users though, using the `staged_` functions.
> The ever_active_set is also needed in the basic use case (Simple, one alpha fit with ElasticNet)
> Why? I don't understand. All we need is to learn the coefficients.

same feeling here.

if you do:

lasso = Lasso(alpha=1, use_srong_rule=True)
lasso.fit(X, y)
print lasso.coef_

do you need the ever_active_set attribute?
This isn't D, you need to prototype these functions.
My guess is the `argExpTypesToCBuffer` call is the only thing that needs to be inside the if block, printing modifiers applies to all functions regardless of the number of arguments.
is there a rationale to the order of setget or getset ?  I see that you're mixing both, i don't have any issue with either form, but i think you should choose one and stick to it.  This way we can learn it and know that it's there, otherwise we'd likely have to look up each time which order it is for each id.

On line 64 is another example of mixed ordering.
`NATIVE_PROPSETGET_CALLBACK GetSetterCallback(size_t prop)`
Add `show()` to the list of stuff still needing to be done.
I'd say we should throw an exception
Minor pedantry: this is a glob, not a regex.
I understand the "double" as I've seen the code, but I don't think the sentence makes it clear you're referring to the type of the property.
could decorate the Coordinates with the <classname>

Another good one for some example is <literal> for constants, variable names, etc..
This is only needed if the functions that are added are non-virtual.
@GordonSmith  - In general I can see advantages in maintainability by using WSDL based generated code (although this particular case is pretty straightforward). Anyway, beyond that, I'm having a hard time finding substantial performance savings by using SOAP as opposed to simple REST requests. Bottom line,  I'll be happy to re-implement if we feel SOAP requests are definitely advantageous. Just let me know. Thanks.
$ means 'current module'
So given that this code is in a if(ptype != PT_ADMINISTRATORS_ONLY) block + it's for authenticated users only
I think it would clearer if it said 'ptyye == PT_AUTHENTICATED_USERS'
Oh, I see what you mean. Github doesn't show it properly, but it is outside the loop (and just before the end of the try block)
white spacing around this expression and the prev. one a little inconsistent
Yes, that should be fixed.
Grammar might need correction.
You should not assume that the widget will be in addon-bar. Actually, the user may have moved in url-bar.
https://github.com/dglol/addon-sdk/blob/da8fc9b60a6ecc668194d15d5d624587e3c18208/packages/addon-kit/lib/widget.js#L614
Yeah, I'm up for discussing this. Maybe `InstallProcess` or `Installer`, though the latter is already used and could potentially be ambiguous.
good catch! done.
@shacker just as a FYI, we don't need the px here. Do you have the CSSLint checks enabled in sublime?
Kaikki `def` ja `defn` määrittelyt päätyvät ylimmälle tasolle, eivätkä siis jää esim. funktion sisäisiksi. Jos haluat, että `numit` ei näy ulos `rank` funktiosta, se täytyy määritellä `let`:llä.
~~~clojure
(apply str [n " (" by " - " dy ")"])
;=> (str n " (" by " - " dy ")")
~~~

So `apply` is not needed here.
as an FYI, this line broke this module on 2.6.x. I has been fixed in pull request #105.
I this construct is a little less awkard like so:
```
case type
when 'png', 'gif', 'jpg', 'jpeg'
    ...
end
```
As @jlee-r7  has pointed, could happen that RHOST is nil. It doesn't seem to have sense when running in active mode, has it?

If RHOST is always needed to run in active mode, in case of RHOST being nil, just print_error to warn the user and return.

Thanks @jlee-r7  for keep pointing!
Yup, that's perfectly fine.
Argh, my beautiful columns!
Thing is, I think this might be able to be modified to work for both.  We'd have to enable multiple "@Implements" annotations or something, but since the contract for the support class is the same as the real one, it might work.

That said, I didn't want to bother with any of that because I don't think there are any/many folks writing ICS-only apps at the moment (aside from Chrome), so almost nobody will be using the real Fragment class.
This is constant per-vertex and could be evaluated on the CPU:
```
u_northLatitude - u_southLatitude > maxTileWidth
```
It looks like from jsperf testing that if/else is faster than switches.
my pull requests has the necessary domain objects that will ease the above refactoring.
usually we write node != null ?
Why the empty method ? Maybe you should add a comment saying "intentionally left blank"
This seems to contradict the doc in Window.modal that says that all windows are modal in MW.

Does opening a tab window with modal: true automatically obscure the tabs? Or does the user need to specify a separate option, such as 'tabBarHidden', 'fullscreen', or explicitly set the size?
In my testing, I didn't see the view appear. Maybe it's hidden by the slider control? If so--see my comments to Max, below, regarding z-order.
This should be provided from the surrounding environment, right? How are you testing the cookbook with RSpec?
Генерик интерфейс — это то, что сейчас, когда вариант функции аплоада кодируется именем. Когда вариант выбирается на основе типа данных, которые передали — это не генерик, а самопальная перегрузка. Я бы послушал, какие у этого плюсы.
Thank you for the contribution! 

My only question is that this line appears like it will blow up if the private path already exists. Would it be better off with a condition, like this?

```ruby
run "#{try_sudo} test -e #{private_path} || mkdir -p #{private_path}"
```

I'm not sure what the rest of the behavior in `deploy:setup` is like; we probably want to be consistent with that.

Thanks again,
Scott
Library specific imports must be used before package imports.
s/typeof(haystack)/R1/
This line should probably be deleted.  There is not getQ boolean.
should this be the cinder user/group?
ah, ok, becuase its sbt&&play. maybe just rename the method then to `_has_playConfig` ?
This line can be removed since it is added above.
You are presented with a simple form to insert, edit, or remove contacts.

* To insert a new contact, click the `New Contact` button. Complete the form fields and then click the `Save` button. 
* To modify an existing contact, find the name in the **All Contacts** table and click the `Select for edit` button. The **Contacts** form fields are populated with the contact data. Modify the data and then click the `Save` button.
* To remove a contact,  find the name in the **All Contacts** table and click the `Remove` button. A dialog appears asking 'Do you want to remove this record?'. Click the `OK` button to remove the contact.

When you insert a contact, notice that both the **All Contacts** and **Audit Records** tables are updated.
When you edit a contact, notice that the  **Audit Records** table is updated with the new contact data.
`super.close()`?
you mean by letting this function throw EDBExceptions? Or should all exceptions which are thrown by this function be handled in that way?
here should be a catch of NullPointerExceptions and IllegalArgumentExceptions (if the query hasn't the correct format) added. Maybe also adding a feedback if the result is empty would be nice
Do we want all these dictionary operators, or should they go via datasets?
so upon further reflection, i'm thinking the best way to keep this general is by just replacing `:data` in the map with the json-parsed version.  ie:

``` clojure
(let [{:keys [class args] :as data} (json/read-json (:data msg))]
  (assoc msg :func class :args args :data data)))))
```

should also save you the pain of your dynamic binding in tests
If we restructure the registration data, this won't be necessary
Thx for adding these back. I accidentally deleted it in my previous pull.
Why was this confirmation removed?
and I think that the parameter should be optional
oops, dang.  thanks for fixing it.
Makes no sense to kick the old horse. Let the bones rest and the history stay in commits of a master branch.
I just removed those methods. This is a dead code (not called by anything)
ok, but I'm not adding String as a type
Why are some of the links changed to use the namespace-qualified version of the type name?
Вот базовый набор возможностей, ...
metadata[:method] and metadata[:route] no longer exist, so I believe this code is currently broken.
:) right. I removed it.
@apex13 linked both
@apex13 not OCD at all, just aiming for perfection - I changed it.
You need to set the catalogue to ``SonataAdminBundle``, and create a dedicated translation key ``export_format ~ format`` and add the related keys into the different translation files.
AFAIK, mongo requires _id as a "PK" if not, it will generate a new ObjectId
`[\'zenddevelopertools\']` ?
libpng is necessary to generate png images. On OS X the lib comes with the OS and resides in /usr/X11/include/ and /usr/X11/lib
The screenshots section is missing. I would to see a screenshot of a theme before I choose whether to include it.
Use interval instead of calling GetInterval again.
You can combine this into one import.
Because sometimes it can be a tuple.
You've got to admit English is insanely inconsistent in that regard.
Btw, you can have a global `.gitignore` file, eg:
```bash
alen@tlaloc:[~/projects/django-oauth2-provider ~master *]
$ cat ~/.gitignore 
*pyc
*egg-link
*egg-info
.project
.classpath
.settings
dist/
build/
TAGS
```
This isn't a bug, but I'm just curious why you chose to use for(;;) rather than while(true)
I love progressive enhancement but we intentionally don't worry about it. Why make it part of the test?
needs newlines between methods
test is commented out?
@dmclain re-correctness -- I know it changes it.  I still think we're going to end up with a lot of `nondefault` values that end up getting declared twice rather than just once in the templates.  I think display related code belongs in the templates, but I'll live with this model.
Returns an array of properties that must be set in a specific order, ordered from first to last. <<<I assume?>>>
Any properties which are not in this list are set after the listed properties, and are set in undefined order.
Should be a space between "properties" and "in"
As there is a note in the summary that indicates that it is only used on iOS together with the optional:true key, I really feel that we describe it adequately. I always  prefer not to duplicate information. Let me know what you think with this in mind.

Regarding your other comments - they were really helpful, and I am much happier with this apidoc now.  Thank you! :)
Also didn't know this...good to know.
and @rwmtse is working on it already :) - so speedy
Fair enough.  I wrote this prior to reading the entire PR.  What about removing `section_id` entirely and just taking `kwargs` and passing that to `get`?