labofsahil/github-event-dataset-2012 huggingface dataset:
Ok, so, here's the problem. In order for the transition to take effect, a reflow needs to occur *before* you add the class that causes the transition. While a setTimeout technically will work, it is unreliable and inconsistent. The real way to do this is to cause a reflow. One of the things that cause a reflow is if you request an element's offset/dimensions, the browser will then reflow the element and return the value once it has confirmed that all pending reflows/repaints have occurred. All you need to do here is:

$cellElement[0].offsetWidth;

The funny thing about this is that typically you'd be calling that to set a variable equal to it, but since you don't really care what its offsetWidth is, you can just access its property and the reflow will occur. Then, immediately following that line, you can add the class 'show-delete-button'.
Does everyone else hoist `.length` though? I'm genuinely curious about this now.
prettify, see libunistring-0.9.3-ictce-4.0.6.eb
add `on-invalid="null"` here
Also it's imo wrong to put this option here.
You can use just 4 spaces here, and it will look less odd when rendered in html :-)
That's some good debugging! 

Is this a known issue with JBoss logging? If not you should raise a Jira.
I'm not a fan of the double-nested `describe` blocks - easier to do inline.

Unless you have a ton of validations/associations, this isn't really necessary.
Why the cast to unsigned? It does the wrong thing when ftell() returns -1.
For consistency again, don't put a space after the keyword `function`.
I <3 the rigor of this matrix. Thank you.
It does, the VPN does that. Not sure why, but I ain't messing with that URL.
Also, why partially rely on google's CDN?
just noticed this while looking for this class - should test-utils.js live under tests/ ?
np, I just couldn't see it used that's all.
As Giorgos said, this could be simplified to:

```python
key, created = ApiKey.objects.get_or_create(user=self.user)
return key
```
When running on locally, process.env.PORT is undefined so I wanted the option to define OPENBADGES_PORT. When running on Heroku, the PORT is process.env.PORT.
This is so gross-looking.
This *looks* like a typo, which is why I suggest PopcornYT vs. newYT
In my original proposal for the PageProvider, the URLs are stored in the page provider class rather than in the page objects. I'm not too concerned with this alternative approach, the idea was only to keep page URLs and navigation in a single place.
Why rescue here just to reraise?
So we won't be giving people a message when their comment doesn't save?
The problem with this is that it doesn't automatically strip whitespace. e.g. `request.POST['name'] = '   something\n  '`.

Here's a piece code that has worked very well for me for many years:

```python
class _BaseForm(object):
    def clean(self):
        cleaned_data = super(_BaseForm, self).clean()
        for field in cleaned_data:
            if isinstance(cleaned_data[field], basestring):
                cleaned_data[field] = \
                  cleaned_data[field].replace('\r\n','\n')\
                    .replace(u'\u2018',"'").replace(u'\u2019',"'").strip()

        return cleaned_data


class BaseModelForm(_BaseForm, forms.ModelForm):
    pass
```
change this to `from django.conf.urls.defaults import patterns, url`
done.

On Wed, Apr 4, 2012 at 5:09 PM, Simon Kornblith
<reply@reply.github.com>
wrote:
>> @@ -103,7 +102,7 @@ function doWeb(doc, url) {
>>               var parser = new DOMParser();
>>               var doc = parser.parseFromString(text, "text/xml");
>>               itemType = ZU.xpathText(doc, '//display/type');
>> -             if ((itemType == 'book')) {
>> +             if (itemType == 'book'| itemType == 'Books') {
>
> This should be a boolean OR (`||`) instead of a bitwise OR.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/zotero/translators/pull/280/files#r648151



-- 
------
Sebastian Karcher
Ph.D. Candidate
Department of Political Science
Northwestern University
what was the deal with this?
`>.<`
You're right, I'll change it.

FYI I just made an unstable, in progress commit to the BIC branch.  I
refactored kmeansp, rewrote kemansbi, and added a higher level function to
submit with an upper and lower bound of K.

It is rough, but I think this type of structure will work better and be
clearer.  I'm going to concentrate on tests for the BIC related functions
before anything else.  A good way to spend a Sunday.

On Sat, Aug 11, 2012 at 8:02 PM, Dan Frank <notifications@github.com> wrote:

> In km.go:
>
> >  		p := points.GetRowVector(i)
> > -		dist, err := measurer.CalcDist(p, mean)
> > +		// Find it's centroid
> > +		centindex := clusterAssessment.Get(i, 0)
> > +		centRowVector, err := centroids.FiltCol(centindex, centindex, 0)
>
> if you had the row be the index of the centroid (ie, the centroids matrix
> would be KxM), then this would be a call to GetRowVector instead of
> FiltCol
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bobhancock/goxmeans/pull/11/files#r1357916>.
>
>



-- 
Bob Hancock

Blog - www.bobhancock.org
Twitter - bob_hancock
https://profiles.google.com/hancock.robert/about
everytime.
I have just pushed this commit 45fefe996ae96239ecd78e47aaab579cb437c857 which should definitively avoid the issue. You can now revert your change :)
обычно пишут комментарий //namespace graph
чтобы понятно было что за скобки такие
кто здесь?
for_each не работает. 
Возможно, он принимает функцию по значению и при этом теряется информация, про переопределенные методы.
Shouldn't it be after submodule commands?
if (!preg_match('/ [a-zA-Z]:\//', $newpath)) {
Oh, right. Forgot that was where it was.
I think this should be `queries-are-different?`
Too much spacing in the docs.
I think the intention here was to warn that the temp file was not removed properly, not to log the provided `error` in the cleanup method. So more like:

```javascript
fs.unlink(file.path, function(err) {
    log().warn({'err': err}...
```
I''m not entirely sure if we should check that the `isGlobalAdmin` method exists. If it doesn't, something is wrong (as it should always exist) and an exception should probably be raised.
I've added an `@throws` JSDoc statement to all the methods
The keys make sure that the extend is happening properly.
The schema on top of the file determines what the default value is for a new field. I changed it to an empty string.
I'd rather have a different name, without `reload`. After all, it **won't** reload the unit, it will only move it out of the ignore buffer. `moveToFront` may be a possibility (the unit will be the first one to be typechecked, if there are more than one open buffers).
Don't use `inject` with a fixed object to pass around. Use `each_with_object`.
ну якщо точно, тоді <code>.sonata-bc textarea.xxlarge</code>. я лишаю це як є?
you're nice
I haven't the faintest idea. I know `Array()` does some more complex logic, but I presume that `Integer()` is fairly basic.
Use the short class name here, no need for the FQCN.
What is force_dir?  No documentation given for users.
Spaces are off, please use this:

    if ($this->save_queries)
Зачем приводить реализацию в .h файле?
Вы же сами писали:
В данный момент переменная DNA у вас доступна для изменения из вне.

Cell *c = [[Cell init] alloc];
c.DNA = [NSMutableArray array]; // разрешенная операция
[c.DNA removeAllObjects]; // разрешенная операция

Определив сеттер извне она больше не меняется.
Здесь еще надо добавить использованный индекс в массив repeatCheckArray, как писали выше.
У вас он нигде не используется. Ну и следовательно метод может заменять значение по одному и тому же индексу несколько раз
Это зачем? должно ж быть число в диапазоне 0-3?
что самое интересное вы правы и неправы одновременно) я сделал проверку этого, это первый цикл do while,этот цикл будет выполняться, пока в массиве mutA(который я специально создал для проверки) по индексу что-то есть, то есть я уже использовал этот индекс. но забыл что либо туда заносить каждый раз, как я по этому индексу изменял мой массив ДНК. спасибо, исправлю ошибку
На самом деле всё дело в памяти, и в том, когда в какой момент времени её инициализировать. Это можно объяснить только на примере билетов на киносеанс. Вы можете позвонить в кинотеатр и заранее забронировать 25 билетов на компанию, а можете прийти в кинотеатр перед сеансом и каждый купит себе билет. В первом варианте у вас в любом случае будет 25 билетов (или не будет и вы поищете другой кинотеатр), а во втором варианте может получиться так, что кому-то билета не достанется - закончатся!
Не знаю на сколько уместно сравнение ДНК разного размера, возможно стоило возвращать ошибку в этом случае?
Is this really private?  Maybe just `e.graphInfo`.  I can add lumberjack support to decompose this into something filterable.
How are cases like `_links` or `self` missing handled?
Maybe here the BC Break could be done since it's not yet 1.0...
I suppose this must be:
     force   => $nginx::bool_source_dir_purge,
There's no reason to use the numbered assertions – this can just be plain `NSAssert`.
Why is this different/better than `+currentQueueScheduler`?
this should be left here as a test dependency
0.1 -> "0.1"
Why the `§§§§§` separator?
Sure, the user needs to be in the database as it is used later in the tests. That’s why the code was there in the first place ;-)  But it is not necessary to create this unused variable `manageruser` – which even causes a PyLint warning.
Good point. But many returns. 
I'm getting more and more enthusiastic about a little class / objects.
What about a subclass of dict (like bunch) with attributes that has all the returns 
plus bad components list which then can be passed to recompose?
Let's say: Decomposed

On 27.09.2012, at 23:20, Alexandre Gramfort <notifications@github.com> wrote:

> In mne/artifacts/ica.py:
> 
> > +    grad_idx = np.intersect1d(picks, pick_grad)
> > +    eeg_idx = np.intersect1d(picks, pick_eeg)
> > +
> > +    ch_type_idx = [mag_idx, grad_idx, eeg_idx]
> > +    # print mag_names, mag_idx, grad_names, grad_idx
> > +
> > +    ch_chan_types = reduce(lambda x, y: list(x) + list(y), ch_type_idx)
> > +    assert picks.tolist() == sorted(ch_chan_types)
> > +
> > +    out = np.zeros(data.shape)
> > +
> > +    for idxs in ch_type_idx:
> > +        if idxs.shape[0]:
> > +            # important: calculate zscores from flattend array.
> > +            # otherwise recomposition will fail.
> > +            out[idxs, :] = zscore(data[idxs], None)
> I would just do:
> 
> out = data.copy()
> ch_std = np.std(data, axis=1)
> out /= ch_std
> 
> I would not use zscore as you don't want to remove the mean (should already be zero is you high passed)
> 
> also I would return ch_std to multply its inverse to mixiing matrix to so we can just apply it to the output of raw[:, ...]
> 
> —
> Reply to this email directly or view it on GitHub.
>
that's nice, I like the title arg, was on my list but then I forgot about it.
agree.

what about simply channels or channel_specs

sorted(scaling) just unpacks the keys (similar to iteration over a dict),
so no need fpr sorted(scalings.keys())


On Tue, Dec 4, 2012 at 1:44 PM, Alexandre Gramfort <notifications@github.com
> wrote:

> In mne/viz.py:
>
> >      """
> > +
> > +    if scalings.keys() != units.keys():
> > +        raise ValueError('Scalings and units must have the same keys.')
> > +    else:
> > +        plot_channels = sorted(scalings)
>
> plot_channels is a weird name ...
>
> maybe
>
> types = sorted(scalings.keys())
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/214/files#r2305477>.
>
>
2. not 2 :)
delete debug things
I looked after this comment and yeah, we're tied to Django. For some reason I thought it was agnostic but now I have no idea why I thought so.
Is it really readOnly?
I think in addition to. At some point, I'm sure we'll revisit the structure/flow of the courses, and figure out what "first" will typically be.
language: "directed it to a single" should be "direct it to a single"
Really the dist macro only matters on el5. On any other modern platform it is already defined. So the check could actually be easy and just check if on el5 throw the define in, otherwise leave it out.
Are you dead sure about this? I just checked out origin/2.1, and there is a comment suggesting otherwise right here : https://github.com/symfony/symfony/blob/2.1/src/Symfony/Component/Form/Extension/Core/Type/FormType.php#L187 ...
There's also an `index.rst` file in this same directory, which also needs to be updated (map is used for the visual page, but index is used to calculate "where" the pages are behind the scenes). Make sure that the location of the filesystem component is the same in both :)
the title should probably be changed as this one covers too much things (we could use the same for an event dispatcher cookbook about using events to provide hooks)
骨灰 => 老式的
Note the use of `Vim.client` here to choose the most appropriate client; this means that if a user has to start a server with `gvim` or `mvim` they can still communicate with `vim` as long as it has `+clientserver`.
Just in case Mongoid decides at some point in the future to pass the same object to another class defining `demongoize` if this one fails or returns `nil`.
Do we really need this class? Can't the relevant code go into each of the other command classes?
I suspect this note was supposed to be an admonition, but it's not rendering properly.
Deque is only available in java 6 whereas fest-reflect must be java 5 compatible.
Why did you change this assert?
Oh, you've added the new flags as part of the enumeration.

So, why do you need ServerConstants at all? You're duplicating code...
Sorry for not suggesting an alternative. What about a SkipDefault annotations on the builders that you don't want to "read", and then have a readDefault() method on ConfigurationBuilder which which takes the annotations into account and does not invoke the read
I think you could use assertTrue(continue) instead.
The way these `append`s are wrapped is not consistent.
I was getting a failing test here because I was comparing the "tomorrow" figure rather than the "today" one. I assume this is related to dictionary ordering of figures, but I don't know for sure.
Yes, that's the part I'm not sure about the inner working. This logic seems to start around line 7999.

So, as far as I understand how limit work for multiple plots. The existing desired range is store in self.dataLim.intervalx/y, right? So, all i have to do is  make a super range that contains the new limit and old limit, right?
Yes -- that's my understanding of how it works.
typo in 'locator'
I have pulled the code directly and use gaiatest, not python gaiatest/runtests.py...
Use triple equals here.

Also - maybe I'm blind and I don't see it, but where exactly is ssid defined?
I would add the lines below

:focus:-moz-placeholder {  /* as it started to work as Chrome */
	color: transparent;
}

:focus:-ms-input-placeholder { /* just to be sure, I'm not able to check IE */
	color: transparent;
}
Einfach noch ein @author schreiben
This is not what want here.
Ah, you're right. I didn't know it worked like that.
If I'm not mistaken, a @MaxDepth(1) would currently always skip the property, no? If that's the case, I'd use ``<``.

I'm also wondering whether the name makes sense as the depth is not calculated relative to the property on which it is declared. What I would expect is that if I add the annotation on a property is the meaning "serialize the data of this property up to a maximum depth of X", but what it actually means is "serialize the data of this property if the current depth is smaller than X". I'm not sure if we can make this clearer somehow.
what do you think about exposing `transaction` on `DataImport::Database::Connection`? It somehow feels wrong to reach into this object everywhere just to open a transaction. I'ts more personal taste though. I don't have any good arguments except encapsulation.

@stmichael what do you think?
I believe this is the general style of this gem.
Why do we want to support XXX.XXX?
Won't this return holding the spin lock?
These notes have been cleaned up in 23944ea.
move the check into the function, also the inline if should be separated:
```$condition = (isset($qm_ary[1])) ? $qm_ary[1] : true;```
id also move the ```$user->lang[$qm_ary[0]];``` into the function, so you pass the language-key into the function, not the actual language.

also this should be ```$language = ((isset($user->lang[$lang_key])) ? $user->lang[$lang_key] : $lang_key;```
in the function, so it does not throw a "index does not exist" error.
Functional way of expressing this is a bit shorter than imperatively. Alternatively we could use element.attributes.map{...}
Is it possible to add a ``@param`` annotation for parameter ``$options`` which defines the array type; using ``array<SomeType>``, or ``SomeType[]``? Please also check line 23.
I messed this line up, fixing.
These line comments will be super handy. :D
ok, dann ist gut.
kannst du mir das mal noch erklären?
Seriously? `eval()`?
Ok, es la primera versión, ya estoy arreglándolo.
Ahora será del tipo `return _gridMap[row * _nMapCols + col];`
Fixed.  Thanks @caseywebdev!
Não existe música nas fixtures :) .
defensive coding, if the Object prototype ever gains a property named exactly like a CSS style that piece of code won't blow up into pieces. That includes both future standards and nasty users/3rd party scripts adding stuff to the Object prototype.
nit: `if (scrollTop >= navTop.top) {` <3 your spaces.
Heh.  No.
It seems so. I can't find where the dependency is (on packagist), but I did not add this bundle in the composer, only the following :

```json
{
/**
        "jms/security-extra-bundle": "1.1.*",
        "jms/di-extra-bundle": "1.0.*",
}
```

Which were in the deps file...
See comment on reduce_freq in JobArgs_POST__search_jobs.argspec
Confusing name, since you already have another variable called `task`.
Added / at the end of URL.
Like this:
    {% load pushserver %}
    {% load i18n %}<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">

or:

    {% load pushserver %}{% load i18n %}<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">

Does it matter here?
OK, I would argue here that for the consistency sake we should not have space around `=`, same as we do not have them for inline arguments.
Well I thought of something completely else :)

Anyway I have change it now.
Uh, you are complicating here. And ... moving even bit too much around.
Do you still have problems with this?
I disagree. Completely unnecessary and needless complexity, unless you can think of a use case where our binary format for XNA has different enums and features than XNA
In general I'd prefer `type(self)` to `self.__class__`, although I don't know if there's a reason to use the latter here.
sorry to fuss over the wording here, but how about something like

"Whole queues can be configured to expire as well. If `maxQueueAge` is set, `expirationTimerFrequency` is used to check the queue age. If the queue is empty, and it has been longer than `maxQueueAge` since it was created, then the queue will be deleted."
What does this code do if run?
For EFFECT_BIND, if no dbc target is specified, GetUnitTarget (the first parameter passed to CastSpell) becomes the target
Naoki used to have a variant of this power.  We errata'd it to 'from the player with the most salvage' to keep it from being a way to torture players that you don't like.
Okay, so as it turns out, I already had accounted for promo line item types. The `LineItem` class is supposed to be generic enough to represent all the various types of ChannelAdvisor line items: SKU, promo, invoice, etc. I think the issue is that the `LineItem` class also needs a `promo_list` property if its type is `SKU`. Then we would add an array of `LineItem` objects with type `Promotion` into that array.
(Realistically no page is going to be 100 times the length of the NYT front page, in fact most web apps serve pages smaller than 1 NYT front page which toko less than a millisecond).
Your OCD is beginning to wear on me.
If you're going to refactor this, any reason you're not just using ruby's built in temp file support

http://www.ruby-doc.org/stdlib-1.9.3/libdoc/tempfile/rdoc/Tempfile.html
Now we have MOD twice. Maybe remove the MOD16 instruction?
(Ah, that's where `elevHeatmap` is.)

Moving certain helper functions into `Heater` may eliminate the need for these.
maybe we could return "Watcher %r not found" for the message ?
what if we don't have a 'json' key? KeyError!
I don't really like `asResolvedPromise` and `resolution` seems natural to me. If you don't mind I'll leave it for now and let native English @Mossop  decide weather naming makes sense and if it isn't hopefully get some suggestions.
It depends. StreamWriter for example disposes the Stream received in the
ctor, so do others. Dunno.

/kzu

--
Daniel Cazzulino | Developer Lead | XML MVP | Clarius Consulting | +1
425.329.3471


On Mon, Mar 12, 2012 at 21:22, Craig Wilson <
reply@reply.github.com
> wrote:

> > +            this.handlers.Add(commandHandler);
> > +        }
> > +
> > +        public void Start()
> > +        {
> > +            this.receiver.Start();
> > +        }
> > +
> > +        public void Stop()
> > +        {
> > +            this.receiver.Stop();
> > +        }
> > +
> > +        public void Dispose()
> > +        {
> > +            var disposable = this.receiver as IDisposable;
>
> What do you think about not disposing this?  It is injected into this
> instance (this instance did not create it) and there it doesn't own it.
>  Seems like it is whoever created it's responsibility to dispose of it (if
> necessary)...
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/mspnp/cqrs-journey-code/pull/12/files#r549242
>
The microsoft package is named "x86", it refer to i386, i486, i586 and i686. It was the main reason why I used a regex expression (even if microsoft only exposed an old "i386").

For the solution:
I can change this by "i386" (even if it's not the "good" archi :) ), change regex to "/i[3-6]86/"... 
but in all cases we don't need another else statement (x86 will not work on windows RT or IA64, and I don't know the architecture type used by windows for ARM and IA-64).
`s/ngx_log_debug1/ngx_log_debug/` ?

`(%V, str)` exists?  I'd been using `(%*s, str.len, str.data)`

Would unparsed_uri be better?
Intended to be something like results.model.formula.desc.describe() since that's a bit verbose and I don't know that users will get so deep into the ModelDesign object.
I changed my mind about that `chunk` can cause an NPE.

That code is inside a `if (chunkLive != chunk) {`block. so if chunkLive is null, chunk can't be.
should delete the comment, too :)
Oh, good catch! You should submit a pull request to fix this
Yeah, I need a smarter way to handle that.
Yes, that's better.  Will do.
@fabrobinet That might explain why some stuff are missing now. Can you confirm?
@octatone no, it still doesn't work on `{}`. Looking for solutions.
But they've explicitly called configure?



On 10 Jun 2012, at 21:14, Garry Shutler<reply@reply.github.com> wrote:

>> +  require 'evenitron/collector_client'
>> +  require 'evenitron/collector'
>> +
>> +  # Public: convenience method for configuring the gem
>> +  #
>> +  # Examples
>> +  #
>> +  #   Evenitron.configure do |config|
>> +  #     config.system_key = 'your system key'
>> +  #   end
>> +  #
>> +  # Raises StandardError if no system_key passed or found in env variable
>> +  def self.configure
>> +    yield self if block_given?
>> +
>> +    raise StandardError.new("Evenitron system_key required") unless self.system_key
> 
> This seems a little OTT. I don't think we should stop someone's application from starting because they haven't configured Evenitron correctly.
> 
> I'd flip my shit if a third party library stopped my site from working because I hadn't configured it properly.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/evenitron/evenitron-ruby/pull/1/files#r957325
I think this should be `Stripe::Event.retrieve(params[:id], secret_key)`.
I would have loved to do this (named captures) but it only works on Ruby 1.9.
Per the Django source code, they're _legal_, but potentially confusing:
https://github.com/django/django/blob/master/django/contrib/auth/models.py#L172

"Note that the default value of allowed_chars does not have 'I' or 'O' or letters and digits that look similar -- just to avoid confusion."
Will change as advised here:
https://github.com/plymouthstate/PSU-Mobile/pull/1#r627825
Now only alphanumeric characters.
I was about to say the same thing ^.
What is `currentlySelectedMorph` and where is it defined?
There are "all-in-one" tools such as the Web Platform Installer, XAMPP, WAMP and others that will help get a windows development environment up and running fast, but do not use these for production.
This will break backwards compatibility with any code accepting only one argument.
I think it should be JBoss Enterprise Web Server 2 or Tomcat 7, just like we refer to JBoss EAP/JBoss AS.
It probably will. I'll merge and fix that.
I changed the names of the parameters according the original names in the authors' papers. That's probably the best approach. Additionally img_as_float is now used with a note about the range of values. Please check my phrasing... not sure if this is clear.
now I have to lookup reciprocal ^^
Personally  I would set it by default to `np.sqrt(n_components)` or something :) Of course this is an approximation and that new hyperparam would ideally be cross validated.
> where is this heuristic from?

It's from lobpcg :)
I sent a pull request for that as well now. Sorry for the delay.
as I squint at this wondering, shouldn't this be automatically set?
I think we shouldn't support rightImage property, since there is no such for BB. Otherwise there some inconsistency between titanium and BB native APIs. We have ability to allow user to add subview to row, so I think user can add rightImage in such manner if needed.
They're not, but this test kept failing
There could be, though fixedSizeHeaplet::maxHeapSize() - that it has just checked it is less than - is and will remain well under 4Gb...
I did that because I noticed the rowsToSort grows in small increments and it seems to me this would reduce the possibility of fragmentation.  Will change it to "capacity" as you suggest
This seems a bit too hard-coded... Isn't there a macro somewhere that tells you what's the import's trailing char?

Also, reporting "$" in this case is still misleading (ok, better than segfault, but). Isn't there any way you can get the full name ```$.x``` or at least print the line number/filename...
Well.. yes, it's at 'setup' time, not execution time, I think strictly speaking it's being done too early, but would only be noticeable if some prepare() action changed authentification permissions shomehow (which isn't possible) - so issue is probably moot.
unused afaics
If critical, might want to do it even if keycount < the min threshold.  Maybe on a second pass?
Dont want to take credit for your contribution? Please do ;)
We have the same code indentation rules in all our JS codebase, can't we document that instead of adding such header in all files?
Two failures in this line we should fix. :)
Why not default `$priority` to 1 as that's what happens at the EventCollection level?
Oui je sais, c'est pour ça que je l'ai changé dans la 2nde pull request.
Mais si tu veux, je peux le modifier ici. Le truc c'est qu'il n'est pas naturel d'avoir des Teacher sans User donc je faut que je fasse un select dans le controller. Tu verras! ;)
Cette méthode est appelée dans les 3 contrôleurs utilisateurs, et bientôt dans le 4è.
C'est pour ça que j'ai factorisé le tout et que je l'ai mis dans application_controller.rb.

D'ailleurs tu remarqueras que maintenant l'invitation est insérée dans un begin/rescue. Comme ça, ça évitera ce qui t'es arrivé l'autre jour lorsque tu n'avais pas paramétré les variables d'environnement. Donc en cas d'Exception à cause du serveur SMTP, hop on est redirigé !!
other include
Is an award with no lame legal?
This wasn't true in 1.8.2; why are we making this change?
Realised I took the version from a different project  :stuck_out_tongue: but you get the idea.
this should just be ```sysinfo``` which is a method defined on Post that caches the result, so multiple uses in the same module don't have to make unnecessary requests.
I don't see the point of these options at all? Just let the victim machine tell you everything you need to know about the environment. This is unnecisary
Please also update the Bento build script.
Not sure. I tried to keep with the original publication. If I get it working again we can just try it out.
FFmpegAdapter
This method seems to assume that `this` is expressed in UTC, which is different from every other method.  Would it be clearer to move this method out of JulianDate entirely?
Shouldn't we implement public `clone` and `equals` methods directly on Material and its prototype?
Why expose this?  Just double the width and height of the atlas.  Check the WebGL maximum supported texture size (on the Context) and document and throw an exception if it would be exceeded.

If we need to expose a scaling factor later, we will.  Less is more.  By not exposing it, we have less interfaces for users to learn, less interfaces to documentation, less interfaces to test, etc.

However, we should expose the initial minimum width and height of the atlas because if it is initially too small, it will get resized a lot early on.  For example, I think the initial minimum capacity for `std::vector` is 16 on many implementations.  Look at atlas resizing exactly like std::vector resizing.
Since FILL just sizes the content to scrollview bounds this is not required.
Still needs to be changed to use fast enumeration; more performant and standard.
Since this is a WM-specific NavigationGroup, should probably have a link here:

[NavigationGroup](Titanium.UI.MobileWeb.NavigationGroup)
Eu lembro da gente ter conversado sobre isso em sala, mas só pra garantir: esse setter não tá legal, pq vc estaria alterando uma propriedade fundamental do objeto por fora. No caso, a melhor solução seria passar o raio como parâmetro pelo construtor.
I removed this section because the author will be inputting dependency information in metadata.json, and there hasn't been compelling evidence that putting dependency in README captures any additional insight. A user would have the opportunity to see dependency information both on Forge and in the metadata file.
There is no need to require "base64" since you are using the following:
<pre>
[salt_hex].pack("H*")
</pre>
pure aestethic remark to try to avoid multiple newlines
`tpl of JST` ?
Callback is needed to include methods and define helper method
We could use `.expand` for that, though granted that reduces the attractiveness of the method.
Should these be updated to be correct? And in any case, if they aren't correct, why isn't the unit testing picking up on that. Perhaps it is not covered?
To @monarchdodra:
Things must not have been done this way. The comment will become invalid once Jonathan's slicing fix will be merged. And it will be his responsibility for incorporated breaking changes.
In current, compiler does not treat `@property` function so much specially than other normal functions in its semantic analysis.
A `@property` attribute for a function declaration just means that provides a syntactic sugar for the usage of the function.
In other words, compiler does not recognize that a `@property` function is getter or setter.
It depends on how the function is used.

```d
// Compiler recognizes just foo is annotated with `@property`, and has one default argument.
// Compiler *does not* recognize that foo is a getter or setter.
@property int foo(int n = 0) { return n; }

foo = 1;		// foo is used as a setter property. @property attribute for foo allows this *syntax*.
int n = foo;	// foo is used as a getter property. @property attribute for foo allows this *syntax*.
foo(1);			// foo is used as a normal function.
// compiler should be reject this syntax as a use case of foo. (But this is not implemented yet)
```
Should we even bother with the console.log here? It's no big deal if the code fails, and the message doesn't have any actionable content anyway.
Changed it. There thou go, beloved builder :)
> Is there a way that we can test this without actually doing a release?

Yes, it is a little tricky, but I added the information here - https://community.jboss.org/wiki/ReleasingHibernateValidator
With git you can keep all source changes locally w/o effecting _origin_. Also _release:stage_ allows you to deploy into a plain directory.
this looks ugly, maybe start the string on the next line and indent? or is that not allowed?
@lsmith77 Could you help me a little on this? I'm not sure on how to use loginAs() and makeClient()...
add them rather to the manual
I'm inclined to agree with you re: facter should be the authoritative source of record, I wasn't aware of the facter issue ~15792 so I'll wait for that to be released then I move back into the main branch.
In FX35, it seems covariance/contravariance isn't supported like this; in order to compile I had to explicitly specify what type to expect.
Dunno either, I've no such use cases. Being pragmatic I'm tempted to say that we should blacklist because I can't see many cases when it should be disabled compared to those when it shouldn't. But we may try to gather some feedbacks for this.
the objectSerialize method is static, thus it can be called by XmlSerialize::objectSerialize. I am wondering whether we have an issue to track this change.
Diese Liste ist somit natürlich fälschlich als Entity\Member deklariert!
Except web hooks there is also order_status_url for every order , where user can check current status, so web hooks were moved to order_status subpage
`return !(CommitteeWTS.FindCommitteeWTS(session, election.ID).Count <= election.VacanciesToFill * 2)`
Actually, as I know, on PHP 5.4 <?= ?> won't be considered as a short tag, but <? ?> remains
carl and I talked about this.  Some of my docs were out of data, but the class is format agnostic.  Just takes a dictionary.  the mgmt command figures out if its JSON or not.
Doesn't seem to be working for some reason.
FIXED ALL THE TABS!
No, go through all the email i sent yesterday since there are even more caveats ;)

But. . . for this particular scenario - its clearly mappable by convention where there are two maps and only one fits the headers criteria this means the other one is payload.
Its all goes to the issue of empty payload, but let's not discuss it here ;)
Yes true, but I deliberately did not wrap it in synchronized block since its  harmless . Think about it.
The this.replyDestination is never referenced anywhere else. Its basically only used during this determination/resolution process. For the rest of the request its what's being passed.
Its hard to write a yes to prove it but its easy to reproduce with debugger and I did so it works just fine. Extra synchronization would affect performance without any gain.
Why not use JUnit Assert's fail(String) method?
This doesn't look correct, wouldn't a + in a String context perform concatenation?
chybi phpDoc
Please pull these out and into a `#define` at the top of the file. Like:

    #define kMapLabelAnimationDuration 0.25
Ripped from DJango and does... what
Also people developing offline -- I do this on flights to conferences, etc.
I've added (hopeful) support in the next commit for checking the battery power on OSX - I've no system that I can test this on, though it should work fine.
This is a very bad thing to do
you need to use ``update``. ``install`` will use the lock file, not the new composer.json (and it will tell you your lock file is outdated)
While this will ultimately be useful from CI, we can't execute on it yet. There are too many sample snippets right now whose origin is unclear. We'll need to figure out what to do with these before proceeding.
This part is one of the main differences to the old model. The Controller will only create the Spotters/Disambiguators that are defined in the configuration file, so if you want to use only one, you just put that one spotter in the configuration file.
- preprocess file cleanup
Would it make sense to write this as: This is only meaningful when frequency is monthly on a day of the week or yearly during on a day of month?
If you're defining the route, shouldn't you define the endpoint? If nothing else, to serve as an example...
I mean, only if it can be done now
All this moves to the FinishAuthTask
I will soon (when I get time ...) move all http call to ``guzzle`` instead of ``buzz``. So adding a dependency can be an overhead for the bundle only, but not on the scale of the sonata project (ie, other bundles)
I like this signature, its a good standard
This will stop the linking of the require.js and bootstrap. Please change it back.
It would be nice to have some whitespace (two lines) between the includes and the code.
user cannot assign their own id.
This pattern can be written more compactly. You can either do:
```JavaScript
if (d.touchEnabled) {
    d.color = '#000';
}
```
or, if you want to specify the greyed out color (#aaa in this example, may need to be tuned):
```JavaScript
d.color = d.touchEnabled ? '#000' : '#aaa';
```
:triangular_flag_on_post: Use jQuery to `onload` things
Why we need to add "Doctrine\\Tests" here ?
Make that 2.060. ;)
Good call. This should be `basestring` in both places.