labofsahil/github-event-dataset-2012 huggingface dataset:
I displayed 2 buttons in the same page and the .caniuse-nfo was displaying underneath the second element instead of on top as expected

[z-index issue](http://oi41.tinypic.com/35c4c1x.jpg)
I don't really care about this kind of boilerplate code, it doesn't do any harm really.

If there is a pattern we could change our build tool to add this.

Also note that some implementations give "new" copies of require/define etc. to their modules, so overwriting doesn't help too much.
1. surround intval($_SESSION['authId']) with the attr() function
2. surround Me with echo xlt() function
:) it is in the newer versions of php.
Proffesor putricide get's aggro from 3000yards away before even going on the stairs, when u get in fron of stairs where abominations stayes...from there proffesor can be aggroed
Juraría que relativa funcionaba, es una sorpresa para mi esto. Tendrá que ver con el casing?
Can you avoid inline control structures? (add braces)
Ok, I will prepare something on saturday.
Não, pões na raiz do repositório. Lá dentro, em cada linha, é que defines as pastas/ficheiros que queres que o git ignore. Por exemplo, o conteúdo do meu é este:

    /bin/
    /gen/
    /.classpath
    /.gitignore
    /.project

Vai ignorar as pastas bin/gen que não precisam de estar num VCS, é mesmo inútil. Ignora os ficheiros .classpath e .project que se referem ao projecto do Eclipse, isto fica ao teu critério incluir ou não; eu não acho necessário, qualquer um pode criar um projecto novo no Eclipse (ou outro IDE) com o resto dos ficheiros. E o ignore do próprio .gitignore é somente porque este projecto especifico não estou a publicar no GitHub, é local, logo não necessário estar num VCS. Se tornares um projecto público ou trabalhares em equipa (ambos os casos podes ter forks e depois merges de código e bla bla), convém todos terem o mesmo .gitignore para ignorarem os mesmos ficheiros no VCS.

Ficou mais claro?
@Dulumun actually Im asking the same :P
no worries. for some reason now I'm getting a weird delay this keybind though between when i fire it and when the 1 line buffer appears for me to type in. Minor Issue
Please fix this also on line 44:

``` ruby
    # actual invokment of a given task (through @deployment)
    def invoke_task!
      options[:actions] = deployment.task
```

Deployments don't work unless you do. I just changed it to deployment[:task] and it works.
#fcd700 is too bright imo. Would you consider changing it to a more eye pleasing softer shade please.
There is a function there is a method StringBuffer::reserve(size32_t size) that combines these steps and you don't need to cast (char *).
It doesn't break because of line 163, which skips every line that isn't in a 'key = value' format...

So basically line 162 only strips the inline comments (options # blah blah), and line 163 ignores every line that is not in the right format...

I guess that's OK?
Doesn't appear to be a problem; cf. http://packagist.org/packages/GrindKit/GrindKit
ToDo: Refactor existing code to use the LineSegment class. Try to move existing logic into LineSegment class where applicable.
The removed coords have been my "personal coords" of the cache (changed on GC.com). I restored the original values and as a result adjusted the test.
Using the "long" format isn't very human-friendly but is an additional test for Geopoint btw.
Why do we need this?

All the browsers support the innerHTML property, and being a property that doesn't suffer of the overhead of the html() function call in the stack.

jQuery syntax compliance shouldn't be part of our roadmap.
Better having a new branch for v3. When it's complete it can move to master
i get command not found git_compare_version . moving line 73 to end of file fixes the prob
Hm, I will look into it again, however, please change your patch according to what I said before, so I can see which parts of the code you have actually modified.
Oh, I would never have guessed that `mark-defun` actually worked for anything else than lisp defuns.
Not sure this should be included here - especially since the transform can be far from simple.
Uuuh.. Okay, this was indeed a bug then. Thanks for fixing it!
And yes: I did not understand, what yo wanted to tell me in the forum. I was not even sure, that you 
were talking to me..
ooops! good catch :))
Here's a screenshot of her saying it :)

http://images3.wikia.nocookie.net/__cb20070313201454/wowwiki/images/d/db/Aurosalia_willyoustop.jpeg
What about cmake ?
If I can fill up the Crisis and Rare Salvage decks a little more, I could build the initial prototype tonight.
Don't tell me this wasn't copied a bit. 1 of the files is 1000+ lines :)
Thanks for the reply - I wish that I could but I just don't have the
skills.  Maybe I can get one of our teams developers to do it.

Janet

On Tue, Feb 21, 2012 at 10:57 AM, vitali-ausianik <
reply@reply.github.com
> wrote:

> Sorry, I no longer work with this project. May be in future... But if you
> need it right now - you should do it by yourself.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/vitali-ausianik/lettuce/commit/ad21d4b850bef290b27c0318514dc182fdae2a9a#commitcomment-990417
>
Desde esta línea (70) hasta la línea 75...Deberían ser eliminadas...Esas no son del sistema si no estoy mal...Son parte del código de tracking de Google Analyticis...Si no estoy mal
Basically you just subclass admin.User and add an objectmanager to it, then you wrap an auth_backend that always returns the subclassed User object instead of admin.User.

No big deal though, I can look into it! Just some food for thought.
Yes it was. My bad.

-----Original Message-----
From: Nicholas Martin
[mailto:reply+c-932304-c5b6e7b7fb01dcd1add4f4f17a0e83dc86c2165a-205465@reply.
github.com] 
Sent: Friday, February 03, 2012 6:59 PM
To: Cameron Fischer
Subject: Re: [Timberwolf] OOPS (3ff8cf5)

Was this just moved from the wrong plugin section?

--- 
Reply to this email directly or view it on GitHub:
https://github.com/RiparianData/Timberwolf/commit/3ff8cf5ea6a0f4bb8744a09a4c2
58ee96c3039d3#commitcomment-932304
да серега алкаш ебаный
Entries should now be separated by space?
Dooh I was just pushing this exact same thing :P
This description does not contain the full recursive tree structure, which might make error detection difficult. Is there a way to print the full expected tree structure while also showing the new information?
My reasoning was to prevent us from having to put ```python if somepath:``` everywhere, but if you don't anticipate using os.path.basename outside of util.py then it should be fine to do it your way.
Indeed, you are right, you can forget about this pull request, I closed it.

Thanks,
-joe
¿? can add 2012_03_09_00_world_conditions.sql with drop tables? i think is better for all :) no all knows to do that...
Hmm, cool update :)
Thanks :)
I love the sentence:
``The parameter in \epsilon-SVR becomes a parameter here.``
But for SVR it makes (kinda) sense. As long as it's not in the SVC ;)
`LabelBinarizer` encodes the positive and negative labels with 1 and 0 respectively. This was not a problem with classifiers as they can handle any encoding as long as `neg_label < pos_label`. Hence, for classifiers, the decision threshold is always 0 when using `decision_function`. For regressors however, the decision threshold must be in the middle (0.5 here). Another fix would have been to do initialize the label binarizer with `LabelBinarizer(pos_label=1, neg_label=-1)` but this currently not possible. I opened issue #548 for that.
Email fehlt,
Wrong wurde zu worng
Szerintem dropold a táblákat az adatbázisodból és importáld újra ezt az sql-t (a sequencekre majd hibát ír, de don't care).

Bocs, hogy ennyi módosítást csinálok, de így van esély, hogy hamarabb befejezzük (némi áldozattal) :)
Hi

When using custom hints, you need to call toggleUseObserverHints () on your observer widget. It uses the hints of the displayed observer by default.
Off by one error!
This is throwing compiling errors btw.
OK, I fixed it (and silently updated the GitHub tag "v1.3") with commit #8e6576f171

The reason for this "crazy workaround" is that when there's an @2x texture, but we're NOT supporting retina displays ([SPStage supportHighResolutions:NO]), this method still loaded the HD texture, and you'd get the same kinds of problems you experienced now, only under different circumstances.

To fix it, I added an extension to the NSString class: You can now ask a string which content factor it contains:

´´´ objc
NSString *filename = @"/some/folders/filename@2x.png";
float scaleFactor = [filename contentScaleFactor];
´´´

Now, this should finally be settled. It would really be awesome if Apple had added a method to UIImage allowing to load a **specific** file, without automatically adding the scale factor part.
I don't think I'd call this an "Extender" — what's it extending? Neither is it anything to do with Android.

Let's go with "ClientsDatabase".
I think the fix is https://github.com/startsiden/startsiden-playpen/commit/2d02f5311ed835da0acc09a02ec78ae5ffc3dd35
Flying Water speaks of the truth
why not no newline?
удалил к хуям, но еще не коммитил
Does this really need three non-breaking spaces, or should some CSS be used to add a bit of margin / padding?
What if, instead of manually specifying a comparison length, we just selected the first unambiguous match? Something like this patch I committed to thor: wycats/thor@a0e3e20c7ea902f0678d741e8451093c36f5d912.
This is generally a bad idea, hardcoding sizes.  If my screen or window is small enough, it's impossible to access the bottom of the dialog as it does not scroll it's contents.

I'd prefer we set max, min widths and heights in hard pixels only if necessary, then use percentages for the default width and heights.  Also make sure the contents of the dialog can be scrolled if the dialog it's self is too small.
Should there not be as many pairs of dims as there are pairs of maps?
Why do you only want to test Set._lib_handle? (I presume that it would be valid to test _lib_handle for all classes)
`cdim` and `dim` property docstrings are a bit confusing, `"The size of a single data item"` would be `np.prod(self.dim) * self.dtype.itemsize`. Then if i am not mistaken, `cdim` should be `"""The number of values at each member of the dataset."""` and `dim` should be `"""The shape of each member"""` or something like that.
changing ScatterGatherFirstCompletedRouter?
RE mySelf - thanks, do that
RE controller - got it, thanks
Is there any test which tests massive concurrent usage?
This is expensive: SortedSet.empty[Member] + member

Cache the SortedSet.empty[Member] in an object
what do you mean? pulse-interval? heartbeat-pulse?
we use interval for everything else
heartbeat is the normal term for this kind of messages, I think
I will sleep better if "quick and dirties" are clearly flagged ;-)
Yes, I forgot that you had added that you had added that after I wrote this test.
This section will be extended later?
This comment seems to be repeated througout the docs, is this intentional
If I do {% position XXX for category %} and the {{ category }} var isn't defined it will result in Category.DoesNotExist "Category with tree_path="category" doesn't exist." That's suer confusing.

Don't catch the variableDoesNotExist (or log and return '') and if the result isinstance of basestring, then try and fetch it by tree_path:

cat = template_var.resolve(context)
if isinstance(cat, basestring):
    cat = Category.objects.get_by_tree_path(cat)
return cat
right.  it should be just moving a bunch of methods over by reopening `class Session`.
I have it so it goes through send_account_info for the account_info mail and I change the method parameters to make for less argument handling.
Wasn't sure if that preference was an absolute requirement, but thank you for pointing it out, has been amended to match to the stipulated standards
You need additional white space after comas to be PEP8 :)
I think it would be great to have more PEP-8 compliance, and we can do it here by adding a blank line between standard library imports and current project imports. So a blank line below "import traceback" would take care of that.
BLOCKER: This should raise the exception, not just create it.
BLOCKER: I'm guessing there's no unit test that verifies a 404 in this situation.
typo in "rpeomd.xml"
we don't have to wait long, but this request is very manageable right now, so I'd rather not add to it.
Shouldn't we strip _data suffix as well to be consistent?
I think that this functionality should not be placed into OpenDefaultStreams .. it open streams, not clear or modify the streams. If we can not ensure that no streams disappear, then we leave it the way it is.

Clear the stream list and update them with the new ones.
is_readable мне кажется тут не нужно, смотри, мы создаем папка, если она есть но не доступна для чтения.
может быть проще так:
    if (!is_dir($path))
        return mkdir($path);
    if (!is_writable($path))
        return false;
return true;
But `_b_mgr` is pretty opaque. `_bootstrap_dir_mgr`?
Just curious. Any reason why a method over a simple variable assignment?
This changes shouldn't be attached.
actually, the fix is easy: instead of moving back to the project and then inside the vendor dir (thu assuming its name), this line shoul use ``__DIR__.'/../../..'`` as we know that this file will be placed in ``$vendor . '/behat/behat/bin/``
This should probably should move with the rest of the code.
this file should be updated with the output from running this test.
shorter:
if cvmfs_server list | grep -q $repo; then
We haven’t released 0.2.2, so you don’t have to change this.  ;-)
@Fustrate, @henrikbjorn already mentioned that is_empty is not a php function.
Hmm, no it was used in the code before I made some changes.
GitHub isn't too helpful here: you can only see branch changes at my branch or by reopening this PR. galaxas0/twui/upstream-button-additions
I realize that, but when one changes the `cursorColor`, it should automatically call `[cursor setNeedsDisplay]` immediately afterwards.
@jspahrsummers Ohhhh! Got it! Thanks. /feels oddly left out :panda_face:
That's probably even better.
Should this not be <h3> like the previous subsection?
Le fichier modifié au complet de nouveau? Tu dois faire qqch pour que ça arrive tout le temps. Vérifies tes commits.
Votre vote a été enregistré.
Already realised and closed.
```php
unset($selection, $book);
```
Thank your for your review, I've updated the code, follow coding standard and move parameter parse to main function, and remove a duplicate function call. it should be better now. thanks ;)
ok, fixed and all the xpath usage in jackalope is fixed too.
shouldn't this list all the alpha div metrics?
с табуляциями что-то не то
то же
My suggestion was text_punct_phrase for dynamicField but no change to email or newemail.  WDYT?
No, that should be enough. I just thought it'd be worth, even a projects.sum(:id) should do :P. But that's fine already, thanks!
I used the word "middlewares" because it's used in the code, but I can change it in the docs, if you guys want, something like:

    Here's how it loads the middleware

or

    Here's how it loads middleware

Actually "middlewares" is used all over the place, we can agree in something and I'd change the guide accordingly, check this out: [https://github.com/rack/rack/issues/332](https://github.com/rack/rack/issues/332)

    Here's how it loads middleware components
    Here's how it loads middleware filters

Sounds like good candidates.
OMG! how did a tab sneak into my commit... thanks for the hawk-eye!
Cool. Lets wait to merge that commits in another pull request after merge this one.
Space :scissors:

Not sure if we could/should move the `register_task` calls to *inside* the `DatabastTasks` module, just to avoid typing it again and again here. Thoughts?
What this change means? There no mention of in commit message.
Fear of using a poor, undefined interface (shallow or deep copy?  NO CLUE)  that I'm not sure what it's doing.  I can adapt to that, if you think it's more regular.
You are correct.  I'd put that in the category of intentions for which there should be both a) a comment and b) a test.  That's strike fifteen against this file.  I'm thinking we need a major reassessment of how we're doing code review.
What does this link to? ``bracket`` doesn't seem to be a parameter or function.
line width must be below 80 characters.
specifically, this is in network order (big endian)?
Perhaps a better test would be `assert not s1.has(cos, sin)`. If you do this you will notice that it fails for `x = pi/255`.
I vote to normalize to one space only on each side of `=`.
You should ask if ``expr.base`` is positive, rather than doing not negative. I don't think it really matters, since ``Rational``'s cannot be created symbolically, where you could run into problems with ``ask``'ing if it's negative coming out as None.
No need to define a ```__str__``` function if it's just going to return str(self). But keep in mind the comment I left above.
you should be defining `__eq__`, not equals...see the difference between the two in dense.py
Thanks for encapsulating the `sign_in*` methods :heart:

Now we just need to move them to `test_helper` to make them reusable for other tests.
allow duplication of meta tags → now allows to register multiple meta tags with the same name
Single quotes for consistency? Actually, looks like there's a mix of quotes in this file :s.
Thank you Chris! PR updated with your optimizations. I don't think the `existsSync` from `bin/grunt` is available here so we have to check again. Maybe I'm not understanding. Removed the unneeded `obj` definition though.
Can't the client derive that from the URL of the queue sending the message? The queue address is included in the message format.
This should be part of the `ifeq (Linux,$(uname_S))` block.
Tab copy for July is missing. Is that intentional?
Sorry about that. What is your fun award name to replace it with?
Single quotes (there is no difference in JS)
But no, that would work because, looking at it, cache_name is used to retrieve the extensions as well https://github.com/imkingdavid/phpbb3/blob/8406db9402b99346a8fc56c25eb8a2d2e63b3792/phpBB/includes/extension/manager.php#L55
How about:

	OR '. (int) $auth->acl_get('f_read_other', $forum_id)

?
$input is already a string?
in place (the second one).
@imkingdavid this code was written on the fly, so I am not sure it is according to coding guidelines or if it works - needs to be checked.

About question whether it goes to overall_header: I am not sure, but if no one have spoken about it at phpbb yet - JS files should be included just above closing body tag (</body>) to speed up loading of a page. Another possiblity is to use asynchronous JS loading... This probably needs to be discussed more at area51 or somewhere in case it is not discussed already...
Space between `)` and `{`.
Should probably return different integer values (use class constants) based on what fails validation. Also, eventually we should consider other requirements than phpBB and PHP versions, such as specific PHP modules enabled, specific server requirements, etc.
The third parameter can be trivially dropped, what does it gain exactly?
Users can turn off email notification in their UCP, so this should check to make sure they have email turned on.
Hardcoded language here. please use language variables
The query can just be limited to 1 row (using sql_query_limit()).
This will reduce its expense but will do the job.
This should be done after the sql query.
Indeed the null assignments should not be needed.
Passing `err` to async.map short-circuits it, right? Should not being able to resolve one of the endpoints be fatal?
Any reason you don't reuse args here?
QOTD_BONUS_PROB isn't determined with a formula. but I guess QOTD_BONUS_POINTS has to go
You probably know this but `and` is a lower precedence version of `&&`. Not sure if that's what you're looking for.
When you `require('yui/[something]')` in moduleA.js and in moduleB.js, it uses the same YUI instance under the hood, and therefore you get back the same `Y.Handlebars` object.

These two ways use a shared YUI instance:

```javascript
var Y1 = require('yui/handlebars'),
    Y2 = require('yui').use('handlebars');

Y1 === Y2 // => true
```

If you want to create a separate YUI instance, you can do the following:

```javascript
var YUI = require('yui').YUI,
    Y   = YUI({useSync: true}).use('handlebars');

// Notice the `useSync`, that's important. The way of using a shared instance 
// already does that.
```

There's some more details on our [YUI on Node.js](http://yuilibrary.com/yui/docs/yui/nodejs.html) user guide.
I'm being picky but our code style means  there should be a space after (int)
overlook sorry
good point... removed it
They're not deprecated - they're removed!
this should be protected and not public
It's not important, but I prefer String.format for this kind of string building. This comment applies to a few other places in the change set, if you choose to apply it.
I must have added that the last minute. Thanks.
good question. I know in windows I cannot say -Dsome.prop='value with spaces', probably I can on Linux. ... I'll check
I don't know the meaning of `Last`, and I see that is the only subtle difference between this fun and `try_preflist/3`. My first response when reading the code is that `process/3` could just call `try_preflist/3` and you'd never have to remove the current node from the preflist, making this whole mod shorter and easier to read still.

So, I guess I'm asking what `Last` means, and why it means don't try other vnodes?
an endpoint
Raising an error sounds good to me (I think it is better for frameworks to detect this on their own and prompt the user when the migration is generated, not when it is run, but that is a little off topic here).
Prefer the form you just replaced here.
`s/provides/provide/`
We shouldn't be landing code that has been commented out like this. We should fix it up or remove it from the pull request.
@gibson042 Done - the gzipped version is the same size with either version, `".5"` is one byte less otherwise. Thanks!
I don't think it's in a path that hot; this particular code only runs once to pull in the data attrs.
move to previous line? it looks a little odd is all :)
@dmethvin - It's a function that adds a "timer" function to be called by the jQuery animation timer loop.  `tick` is a function that is expected to return a number of ms til the end of the animation (for support with rAF maybe?) or 0/false to "remove" itself from the timers loop.  The `element`, `queue`, and `animation` properties attached to `tick` are all used to identify/manipulate it in the timers array when you call `.stop()`
length as a name issues aren't a problem here. Let's leave this as a length expression for speed.
Because this should map to the backend's RNG (which could change in future versions).
Would be good to have a comment to that effect.
The following section that initializes the notes should be refactored in the base class in a separate method (e.g. ``_setup``):

```javascript
Notes.prototype._setup = function () {
    var self = this,
        notes;

    // find notes loaded from session and re-create them
    notes = this._root.find('.note');
    notes.each(function () {
        var note = Note.create($(this));
        self._bind(note);
        self._notes.push(note);
    });

    this._root.find('.add-note').on('click', this._addNote.bind(this));

    // socket used for saving notes
    this._socket = this.context.messaging.getSocket('/example/3.0/notes');
}
```

The ``FloatNotes`` class wouldn't need to do anything more, but ``FlowLayoutNotes`` would just some lines:

```javascript
FlowLayoutNotes.prototype._setup = function () {
    var self = this;

    this.context.find('flowLayout', function () {
        self._layout = this;

        // call base method
        FlowLayoutNotes.super_.prototype._setup.call(this);

        self._root.find('.remove-note').on('click', self._removeNote.bind(self));
    });
}
```
there is no need for both `@class` and `@constructor` in the same docblock. `@constructor` will suffice
i think it's wiser to actually use ``projectPath = projectPath || '.'`` because for ``null`` or empty string values you want the same behavior.
ahso.. k, gefeudelt wird am schluß.. ;)
This method definitely wants to be defined in the same class as its twin, ModuleAdapter.get()
Entirely re-worded.
I strongly prefer to use RuntimeException over checked-Exception
That seems like the best approach. Ideally, the scaffold templates would be composable.
I opened a pull request with a different solution (similar to Kevin's proposal): https://github.com/KentBeck/junit/pull/442

ht kcooney
It looks like this function would probably be better on the AbstractBinding prototype, since the given node is always the binding's node. Also:

* should this function name have "Event" in it, if it's not actually firing events?
* should this function in fact be firing an event instead of calling a function?

The callback function approach is probably just fine, but the function name is too general for what it's doing. From the function name, I thought maybe it was setting up a propagation chain for _any_ events on a binding to bubble up to parents.

```coffeescript
Batman.DOM.AbstractBinding::notifyParentsOfBondage
```
??? ;)
It's the interface that already exists to what is passed in to ijod.getOne and getRange/Bounds, either a string of the idr, or of it's "hash_part" format, or of a raw hash (the latter shouldn't be used anymore by our code but could come in from older apps yet, hence the warning)
The branching here kind of got away from me :)
There an ```{Object<K, V>}``` syntax that you can use to document map objects like this and _commandMap
It looks like all the colors to `setColorAsHsv()` now pass `false` for the second parameter except for the opacity keydown handler--though the opacity *drag* handler does pass false, so I'm not sure that was intentional. I wonder if we should just remove this parameter make it so `setColorAsHsv()` *never* normalizes the value (i.e., always commit the color without resetting the internal hsv state). That would capture the intent that setting an individual HSV parameter should directly edit that parameter's value while in HSV mode without normalizing.
When will attrName be the empty string?  That seems potentially problematic -- it will fail the truthy check in search() and wind up showing tag name hints instead.  Is that the desired outcome in cases where it's set to ""?
Now that this is in the in-product menu, we should remove it from the native menu on Windows.
The second ``if`` case above also restore ``newColorObj`` to the previous valid color object by calling ``tinycolor(newColor)``
To be honest I'm still struggling a bit with ender's code formatting conventions. Actually trying to reconstruct this for this post, I think that even one more level of indenting is need here.

Before:

```
var run = function (...) {
  childProcess.exec(...)
}
```

Now:

```
var run = function (...) {
    childProcess.exec(...)
  }
, createFixture (...) {
    ...
  }
```

Should be:

```
var run = function (...) {
      childProcess.exec(...)
    }
  , createFixture (...) {
      ...
    }
```

Right? I'll rebase my commit if you agree.
Could you put everything inside a div automatically (or specify through some property that the custom tag should render as a div tag?), and then you could just do something like... attrs.happy == 'true' or in this case attrs.class == 'report-value-true/false' for the value type bool case?
The new algorithm is much simpler than the previous one, so we need less unit tests than before.

Do you think there is new code that is not properly covered?
No the tests in GeoExt 1.1 only covered the Reader and not the records or store. This extra fields logic is untested.
This is machine-handled and -generated, so no fear of typography here. ASCII only :)
А где же документация к методу? :)
Are you sure Facebook is doing GET callback? Or it can also do the POST callback?
Should the order of `CsrfViewMiddleware` be changed?
Please elaborate why would `reverse_lazy` be better in this case? So this comment is not the best. It should be more verbose, like "we have to do ... because ... which would ..."

Aaa, you mean because it cannot be as an attribute of the class. Yes.

You know, there is nothing wrong to push the requirements of Django to 1.4 release candidate version. Maybe do that? The rationale is that while you are developing a future product, it is OK to use a development version of the library as it will very probably become a stable version once you also finish your product. But you can already use the latest features (and thus be better).

So, bump the version in `requirements.txt`, go for lazy version.
Alignment. Be consistent.
Put mongoengine imports into a separate import group (after Django).
Improve. This is not English.
This error is thrown also when password is incorrect, so this message can be misleading
Nuke the name and projectID from this json template and the rest as well.
All callers of this version of defaultValue() pass only a single argument, which means val === undefined is always true.  Simplify your defaultValue to get rid of the other code path.
You're not using this anywhere
I guess it really doesn't matter but why not use the class name?
Please remove :P. I think this is actually what you're *supposed* to do, regardless of the browser. See "Completing a Drag" on http://www.html5rocks.com/en/tutorials/dnd/basics/.
all these elses need to be moved like I mentioned before
So, you initially had me convinced on this.

Once I got in there, I changed my mind.

I don't see these things as something that should change often, or ever.

There are only 4 and 2 cases of them being used, and each are each very close to each other.

You name the object stateClasses.ready = "ready" Now, what if we do change ready to be something else. Wouldn't we also have to change stateClasses.ready to be stateClasses.newThing.

This works in cases like var offset = 23432 // magic number

Not seeing it work so much here.
Why do *all* of the properties get updated when one checkbox is clicked? Need this happen?

If not, can this function just be a wrapper around `attachCheckboxChangeHandler`? Seems like that could eliminate some duplicate code.
If your going to have spacing for some of the changes you are making, probably should for all :)
It should be '-h'.
Should we not be calling this "SNAPSHOT" anymore, since we'll be actually calling it the "0.7-T1" release?
Perhaps not. But it works.
remove dev_classpath
Sorry, thought I had that in there.  Must have gotten lost in my edits.  I'm not sure how to efficiently create a bitmask.  What I did instead was

```julia
C.chunks[n] = (C.chunks[n] << (chunk_bits-last_chunk_bits)) >>> (chunk_bits-last_chunk_bits)
```

(where last_chunk_bits would be calculated, of course).

Although it looks like none of this will be necessary if these are reimplemented using _jl_copy_chunks, as you suggest below...
Can I ask for more whitespace here, too, to set off the command from the rest of the text when it's printed out? I'd also suggest printing out `newpath`.
Permissions of the foreman directory? SELinux?
the point is that Monolog never enforces having ``$record['context']['type']``. So if you simply try to use it, it is likely to be empty in many cases.
And no, you should not throw a notice. the logger is a debugging tool, it should not be the place triggering the errors in your app.
Don't need both, this ctor calls the other one.
The implementor expects a collection of raw JSON objects passed to the `Invoker`. The correct deserialization of those is on the implementor.
No wrapping please :)
Let's use `deleteConversation`, `deleteConversationForReal`, `undoDeleteConversation` in conversation list view? What do you think?
Please:

1. Remove these comment,
2. Add other settings and set them to false

Thanks.
You reckon? I pondered this for a while..
Why O2?
I'll write something for `is_votable`.  Expect a PR shortly.
I think this line should say

    if mask is not None:
That is indeed a very good idea :), thanks.
Couldn't this condition be merged with the on `777`?
These errors would be easily avoided if you use the [code sniffer](https://github.com/cakephp/cakephp-codesniffer) :)
This line is not long enough to be split into multiple lines
no need to call `bind` on the image. It is called at the end of the `initialize()` method, which was invoked below.
I've made this default to None if not present.
This check used to be in the requireGlobals() method. It's all we need to retain from that method.
Eigenlijk heeft title-attribute hier geen meerwaarde. Onlangs te weten gekomen van Anysurfer.
commit in ba83f00
Do you happen to have a favorite javascript "beautifier" tool, like indent is for C?  For example, http://jsbeautifier.org/.
Nope, these don't replace each other. The `GIT_OLD_ERRORS` is a temporary workaround for compat, and it will go away shortly.
That was added before I realised libgit2 had a built-in regex which wasn't selected for build, therefore this is no longer requried.
However -L../../build I believe is perfectly valid (and required), as libgit2.a and/or libgit2.so end up in the build dir, not the root of the build tree.
one more test case

{#.}

{$idx}

{/.}
Check out `mozorg.utils.page`, it'll let you get rid of `views.py` completely.
Nice! As long as the text isn't drifting off the sign or something. Looks good to me.
Jeez :( We really need to have a talk about this as a team.
If controls are enabled when the page loads they're visible by default, which is kind of ugly and doesn't match the design. Adding the controls attribute once we initiate play means they're not visible until we're ready for them.

And true, changing the poster doesn't seem to do anything, I just wanted the poster to match the video because I'm anal like that.
but up to you :)
Just override the social block and customize the three links there.

{% block social %}
<nav><ul> ... </ul></nav>
{% endblock social %}
Actually, something is off. If I post in the forum, the time shows up 5 hours in the future for me (I am in ET). If I set myself to UTC, it give me the Pacific Time, which is what we store in the db. So it seems to be assuming that the time from the database is UTC when it really is whatever is set in settings.TIME_ZOME. I am pretty sure you just have to take `value` and add this `tzinfo` to it. Then the conversion should work.

I *think* if you undelete the line below then we will be good.
Oh, whoops. I was looking at the wrong file. Bah.
We (as a team of python developers) should talk about this. 
The reason it's there is because then you never have to worry about accidentally running tests against databases you might need.
It's about the same as raw_input, I'll switch it to make it more readable (I generally avoid input() since it tries to automatically coerce input into Python objects - it evals)

In python 3 input() behaves like raw_input (you can eval(input()) to get the old behavior)
Huh? If ``$versions`` is different from ``null``, then pass ``$versions`` else pass ``null``. Why not simply pass ``$versions``?
we shouldn't need to do this lookup, if we move to the numeric constants, but I'd use an intermediate variable so you don't have to do odd wrapping like this
This is a good idea, but I think it would be better implemented as a general su vs sudo command runner. Something like [here][]. Then it could be used for both this `mv` and the other commands that run as root.

[here]: https://github.com/pbrisbin/downgrade/blob/master/downgrade#L42
до цього екшена доступ повинен мати тільки адмін. де це?
why using the setter injection for a mandatory dependency ? The code will break if the setter is not used.
"explicitly" (multiple occurences)
Uhm, sorry, I can't really follow you. Which way would be better?
there is an issue if you keep using ``substr`` and ``strlen`` as they don't support unicode
the default case does not respect the contract of the method as it does not return a CacheItemInterface (which will break the code btw as the other methods of the class expect the contract to be respected)
I'm fine with change this interface.
^ `var`, no need for globals here
These VDIs may not all be in the database - destroying a VDI doesn't cause all of the VBDs referencing it to be immediately GC'd - we should wrap the Db.VDI.get_SR with an exception handler.
Usually, you see the wrapping behavior written in a constructor because all objects have one but Draper added a `decorate` class method.
I'm happy with this abstraction of the text to the `erb` template, but the rest of this module feels like its too physically removed from it's actual usage.

For me, reading a class like `GroupPrivateMessenger` is now more difficult with use of this model..
@remh -- `logger.exception()` is equivalent to `logger.error(..., exc_info=True)`, so the exception should be logged in any event. Agreed that catching `Exception` makes more sense.
That method call can throws exception, you need to add a try {} catch { case e:DataAccessException (or even Exception) => Failure(e.getMessage, Full(e), Empty)
Or perhaps

```C
    if (A.spec().format != TypeDesc::TypeFloat  ||
        A.nchannles() == 0                      ||
        channel < 0 || channel >= A.nchannels() ||
        bins < 1                                ||
        max <= min)
        return false;
```

would be more in keeping with the prevailing style.
Hmm. Are they your only two options of interrogating a WMS server? I'm surprised nobody has made a web service where you just point it at the WMS server URL and get capabilities...

Perhaps we need to think about introducing this interrogate functionality then. 

Personally, I'm torn between having this useful functionality in cartopy and having to maintain the supplementary code. That was the reason I was particularly keen to hear if OWSlib would take it on.
See comment in https://github.com/ding2tal/ddbasic/pull/2
I think this warning can go away, since it was marked as deprecated in previous releases.
I usually call the blueprint "blueprint" so the reference in the app module is something like "system.blueprint". It's not worth holding this up, but I thought I would mention it.
What's it set to? I don't think I've ever had one I've used set to anything other than the view.
This could be locale sensitive. Painful thing is the default "unknown-host.unknown-domain" is locale-independent.
`Gruntfile.js`
A few points for next time:
* There was trailing space on almost every single line.
* Don't use underscores, they are meaningless in javascript and only confuse the scope. Either way, the jQuery code conventions have decided on not allowing dangling _ in variable names (use node-jshint to automatically run the code against the rules in `.jshintrc`)
* Accessing .length on a HTMLCollection is pretty expensive and can actually cause an infinite loop. The result from getElementsByTagName is not an array but a live HTMLCollection.
* Inconsistent whitespace presence/absence of the space after the `function`operator.
It is used in UserAction and ProjectsAction, centralized here so that the key format is consistent throughout the API and also to avoid duplication of logic. I've added a comment in 452cfb6.
this doesn't actually reset any parameters so there wasn't really any reason to reorganize the code. all that this code does is copy the pointers to the RBMs' parameters into the DBM. If you actually want to reset the values of the parameters you will need to recursively call reset_params on the RBMs inside the DBM prior to copy the pointers to their parameters over.

If you don't want to mess around with the DBM code I'm fine with just reverting these changes and having the DBM not implement reset_params for now. I think that's fine since the DBM code isn't complete anyway.
I'll fix this up now.
the line goes through the circle with this margin.
-> `7.8rem` looks like it does the trick.
What if I turned on vibration before entering silent mode? You are turning it off for me?
Yeah, such text seems useful.  @caseyyee?

The form labels is a problem because the building blocks don't have something that quite lines up.  It uses fieldsets to label things, but those work very poorly on our device's screen width.  If you've typed an e-mail address in the contacts app and had it wrap itself into hiding, that's what I'm talking about.
nit: let's name it `incoming-locked`
Is it possible to use the same style for quote here, i.e.
`x='-1.3'` -> `x="-1.3"`
Really good thing to do, a little birdy told me that we might need to remove all the inlined javascript.
80 characters, in general review lint problems in this library
It looks reasonable at other resolutions, the prompts most definitely need a visual design pass later
This is wired if the user press home button on SIM PIN Dialog shows. The homescreen would 'shift' to left page because
https://github.com/mozilla-b2g/gaia/blob/master/apps/system/js/window_manager.js#L740
and
https://github.com/mozilla-b2g/gaia/blob/master/apps/homescreen/js/homescreen.js#L33
These states are used in the debian model
Is the local assignment for clarity?
Stubbin a value object (like `RSpec.configuration`) is a bit of a code smell, I think.  You can just set `color_enabled` and `success_color` through the configuration APIs provided by `RSpec::Configuration` -- no need to stub.

The `sandboxed` thing in `spec_helper.rb` takes care of preventing changes to the configuration object in one example from leaking into other examples.
Ok, just making sure.  You can change this if you want and squash it in with the rest.
Add your email in.
Please move constant strings to global constants.
I do think it was correct the old way
I mean all plugins in kunena group. But that's not important. What is important is to show if the plugin is enabled or disabled.
Without limit this query will fail on almost every forum. For me it returns 200k messages and runs out of memory.

We need to break this into smaller parts. Easiest way is query something like this:

SELECT id, subject FROM #__kunena_messages WHERE parent != 0 AND subject LIKE 'Re:%' LIMIT 1000

and do it a few times. I think we can even make a query that updates subject in MySQL without needing any PHP...
Already merged, but found typo in here $this->profile->userid = $this->me->userid, can you fix it?

Another question: should user be able to delete all his attachments regardless if topic is locked etc?
OK. Wonder if this is usable also for other sharers now?
Sorry, I meant ChatChannelManager -> ChatManager... put it in the wrong file.

And the name should just be a plain string. like "global" or "local"
This should probably only count retries where no additional inserts occurred.  Otherwise, in a pathological case (such as a race between two changes with the same set of tags) you might successfully insert one more record in each try.
`@settints.titleTemplate {title, subtitle}`
Right, I'll add it.
I wish there was a way to report a value in frequently, say, only on change.

Any thoughts there?
Please add an empty line between the namespace and the use statement
think you want `/uk` not `/href`...
the full comment makes more sense:

  97 >--->--->--- * registration timeout -- didnt send USER/NICK/HOST
  98 >--->--->--- * in the time specified in their connection class.
Erm.... Yes. :) Maybe I should continue working tomorrow... ;-)
Right... the hit counter wasn't working, so I fixed it. The database connection change was done to eliminate repeated code.
Can you raise ``SkipTest`` here? See: https://github.com/mapproxy/mapproxy/blob/master/mapproxy/test/unit/test_client.py#L137
I don't see how, but I'm open to suggestions.
I've added some comments about these two globals: `globalHash` and `bestMatchingLocale`.
These two variables are the two main data exposed by this module,
but as I can't retrieve these data synchronously I ended up using such module globals that are set on `init()` call.
In this case yes, I will add a comment that says so.
user.canSetPassword is that abstraction, it already exists.
Mind moving this callback method into the global scope? You will have access to the tree via `event.originalTarget`.
Just making it possible.
If we're adding them to the queue of search paths, do we want to prepend them or simply append them like we have here? The advantage / disadvantage of prepending is that it would be impossible to run a non-official version from elsewhere in `$PATH`.
<pre>
                       user     system      total        real
returns only       0.020000   0.000000   0.020000 (  0.027393)
begin and return   0.020000   0.000000   0.020000 (  0.028760)
raise and rescue   0.900000   0.050000   0.950000 (  0.950110)
</pre>

Yes, raise and rescue is substantially slower.  Not quite two orders of magnitude less performance...
Since some of the modules are pending deletion or large changes, we could probably just exclude them for now too:

    pep8 --exclude=scheduler,locations,ajax ./rapidsms/
any rationale for choosing 128 as the default? If not, just add that "128 is an arbitrary default"
the same typo
Really in SendResponseListener and not in ResponseSender?
Why is this file included in the patch?
Remove space after !
In fact each time I add a <code>$</code> it's because there only <code>IPBoard</code> or if there is <code>IPBoard_foobar</code>  I'm certain to find <code>IPBoard</code>. If I didn't do a mistake it will match exactly the same pages
If I don't I'll have to add a trait during instanciation. That trait would be misleading as the function will never be used.
@MStumpp I think things like user/admin pages get into the realm of providing complete application boilerplates, which might be tricky with so many frameworks but I definitely think there's some value in exploring things like auth. We'll keep it in mind for the future.
Should we suggest Tomcat even though we are running Jetty in production?
Missing line break here.
There're 2 logical blocks here. One is assignment of local variables and the other is assignment of class variables.
Just the API key, not API key or username
Oh...sorry.
I will fix it and try implement it without changing the `unicode.ex`.
Fixed in a1a4579
import X before from X import Y
drop the -t, just offer --toolkit

If you change the toolkit, you'll also need to change the version, and for that you need to use --toolkit-version already. So just offering long options for changes to toolkit makes sense imho.
why are " (buy, checkout and getCartContents)" in brackets?
usually you would do 
StringBuilder sb = new StringBuilder()
writeToStringBuilder(sb)
return sb.toString();

either you write yourself to an existing StringBuilder or you return a string, but it is unusual to create the StringBuilder and return it for further modification.
An empty list in Python is falsey, so we don't need the explicit length check `{% if news_stories %}` will suffice
Rename this to getMethod()
update url not title
Yes that's a copy&paste I should be more explicit
Although not forbidden, it's highly encouraged to **avoid** the star import
http://readthedocs.org/docs/mozweb/en/latest/coding.html#index-2
This will always be English, because it runs at import-time. When we have strings like this, we need to use the `_lazy` versions of the gettext functions. The `_lazy` versions return an object that isn't actually localized until it's used as a string, so we need to be careful with them.

This is true of any module-level code. If it's wrapped in a function or a view, you're fine to use the non-`_lazy` versions.

```python
from tower import ugettext_lazy as _lazy

PERMISSION_LEVELS = (
    (VOUCHED, _lazy('Vouched')),
)
```

And that may well need context (`msgctxt`), it's worth asking Milos, or just doing it.
nice catch, can't believe I missed this
Make this plural, like the others
We can't use resource registry for handling process definitions, you must use CEI services directly.
FWIW, if we used `[set(sid.keys) for ...` to create set_keys we could articulate this as

```
UNWANTED_FIELDS = set(['portfolio', 'sid', 'dt', 'type', datetime'])

self.field_names = set(sid_keys[0]) - UNWANTED_FIELDS
```

Should be functionally the same, but again leveraging some of succinctness of sets.
I'd add another section for `Bourbon (before version 3.0)` with a link to the old README on Github.