labofsahil/github-event-dataset-2012 huggingface dataset:
I was trying to keep things consistent throughout the filters, but re: your note in chat, it does make plenty of sense to stay in sync with the libraries being depended on.  Updating this now.
In regards to changing johns code to include instead of require_once its not the case that require_once is realistically slower then include, you also have the problem that require !== include, they are very different by using include your saying at runtime you know for 100% that all the files are 'includeable' whereas require you have a last line of defence in terms of if the file cant be included the application will die.

It was the case with PHP 5.2- that require & require_once was very slow this however isn't much of a case within PHP 5.3/5.4+
cluuuuuuuuuube http://www.rathergood.com/zoology
Okies, then it must be the feather's fake title bar - seems the title is hard coded then.

As for your second question...I think we want to keep the songbird@songbirdnest.com namespace? I'm not sure, we need to discuss that one...I prefer using one specific to ngale, but if we can have multiples we could add both and remove the songbird nest later on.
The `total` ATTR needs to be available for calls done in _afterHostRender, but _afterHostSyncUI isn't fired until later.  I feel like improvements can be done to limit unnecessary multiple calls to set(TOTAL), but this was added to mimic the same logic as pre-3.7.0 ScrollView (which sets it in multiple places). So for a late-addition, this was the safest route.
Does this mean that if a developer does not have memcachd running on 'localhost:11211', then the build is going to fail?
I'd cap it to 128 since it will have changed semantics on a0 otherwise
Where do I find them? They are not listed at http://akka.io/community/
So it looks like we added back the member (two lines down) when we received a `MemberDowned` before this fix. Is that correct?
%% + cross Cossversion.binary ?
You mean that this event is sent out even if there is no convergence? Is that a good idea?
I think I got all the changes to cardname.rb that weren't already in the gem, and I noticed then that there is one dependency that I missed, the _user processing that had to change to Account.authorized.  I think we can do a better job with how this configs and connects, so this will be fine for now.

0.1.1 has different _user binding for earlier Wagn branches.
The former is the  most common case, so I don't like that is is long and too complicated. Maybe fetch_existing_trait for the one that is now ? and leave the other alone?  That's what I will do, then we change it again if you have another idea.
I vote we only use one-line defs to highlight parallelism (as in cardname).  otherwise it just makes things harder to read.
Marking the window as transient usually signals the window manager that this window is a popup on behalf of another window, implying that the window manager should treat it differently (no decorations, no user interaction required to show it, etc). Dialog boxes and other windows of that type fit this scenario. If your window manager is not respecting that flag and/or not doing what is necessary for it, I guess that can be corrected in code, but the transient flag still needs to be there for window managers that respect that flag.
try {
..
} catch { 
..
}
Into the docstring!
Adding a final comma is good practice
I think it works without the =1 not sure what it does, could be wrong
Do we have a good usecase for these two params?  Is there a reason we can't process the list in our controller etc and need to do it in the template?
Hmm, not sure if these really needs to be a logic function, but if it does it needs a proper auth check, either like what dashboard_activity_list() has or maybe better an auth function.
I think your bug is in here.  you now require both referenced_name and referenced_card_id; before you only required one.  It is possible for a card to have name but not id (as in a reference to a previously virtual card)

I don't exactly understand it, but this looks less right to  me.
I don't think ant jar would do anything for SCN
This doesn't seem right. There is no free/malloc in this file? only new and delete?
missing heading update:
<pre>SET_CONTROL_LABEL(40,g_localizeStrings.Get(_audio_info_string_id_));</pre>
Yes. HTML pages are the only thing a addon editor cannot link versioned (at least no the index.html) - so probably better don't let browsers cache them that long just to be really sure updated addons are loaded by the browsers.
Crap, thought i deleted that. Thanks for the heads up
в else точно `BEMHTML`, а не `BEMHTML()` ?
Ну в том-то и фигня, что он нигде не хранится.
И сейчас мы на распутье:
 - либо вписывать этого оператора в специальное поле на услуге с помощью григгера на закрытие действия "Заказ услуги"
 - либо `select a.assignedTo from casetbl c jon actiontbl a where c.caseId = c.id and a.name='orderService' and a.closed` с поправкой на то, что `a.caseId="case:123"`, но `casetbl.id = 123` (или такой же join по id услуги можно сделать).
good catch. I think I rather dumb it down a bit to only accept str and let the py2 version be a lambda x: x
to much magic will only hurt
Make them the same as the seeds please
Perhaps you could put the headers last and default so that they can be omitted unless needed?
why not using the positive condition instead of a negation and ``else`` ?
I know, but it uses more than one variable from inside that. It'd be hard to change that.
these two lines is unnecessary
I'd rather leave out the d3.xhr.get and d3.xhr.post aliases for now, and favor parsimony until there is sufficient evidence these convenience functions are needed.
can you explain why comparing two maps doesn't make sense? isn't that what's done in NPAIRS to evaluate the impact of a parameter choice?
The grammar of this error message is somewhat bust.
Just ndarray. The description can say 1-D array and explain the length.
'o'(utput?) is an unusual choice of variable name. Maybe something like tgt (target)?
A few lines above this, there is a sanity check preventing a year larger than 9999 to be supplied, so that limits the difference to ~253,202,544,000. Updated the length to 13, to account for the minus sign.
`!locked?` is better imo
Not sure about this - it'll trigger autoloading of constants which may be a security issue as the constant name comes from user input. I think I'd prefer if we got a list of subclasses and then checked against their names. I know that it won't trigger autoloading of subclasses but I tend to use `require_dependency` to load my subclasses at the end of the base class file so that they're all immediately available anyway.
This is not my code and wanted to change as little as possible, but I agree with you, the indentation is not good :)
I liked the idea, but I bumped with a problem with how that module works.

`I18n::Backend::Fallbacks` modifies the I18n module (https://github.com/svenfuchs/i18n/blob/master/lib/i18n/backend/fallbacks.rb#L10-23)
This means, that the fact that I load I18n::Backend::Fallbacks is enough to make the test pass (but, has the cons of changing the global state of the I18n module)

Any idea?
I'll remove this check file altogether
i missed the deprecation warning -- yeah, use zipped then
Formerly "capturedPre" was maintained as a map of Symbol -> TypeParam.  If clazz -> qvar is already in the map, then stabilize returns qvar.tpe.  Otherwise it updates the map with clazz = new existential.

So what that means is: the list of captured params uniquely identifies the last of captured prefixes, because they are the owners of those params.  If it would have been in the map, it will be the owner of a member of capturedParams, and if it would not have been in the map, then it won't (this being assured by the "getOrElseUpdate" logic being applied - every call to stabilize either finds a pre-existing qvar for a clazz or creates one, so there can't be two.)

That is perhaps not my clearest explanation ever, let me know.
so the common problem w/ the error case is if someone is using a macro or something in the statement block, which can lead to badness. but you're not doing that, so I'm fine with the less expanded version. i actually think taht the latter does read better, but maybe because it resembles java more. :)
It breaks example in spectra guide:  image.subract_bg() raise this exception! why?
the keyword 'mutable' can be set to False now.
So in those cases where auto-evaluation *should* happen (in particular, when the final object is returned at the end), do you use `cls(*args)` instead of `Basic.__new__(*args)`?
agreed: it makes more sense to handle the creation here than in diag. But how about:

```

def eye(n, cls=None):
    if cls is None:
        from sympy.matrices import Matrix as cls
    mat = [S.Zero]*n*n
    mat[::n+1] = [S.One]*n
    return cls(n, n, mat)
```

I will put this in #1567.
If the string had reserved SymPy names then sympify would have failed as discussed in the OP of this PR.
A question - is metadataKeyTag always -1?
do you really want to put this in the Silex doc ? :smile:
有个上限，可以读完所有存在内存中的数据。之后需要等待载入。
- it's a document not a section

- db.stats **is not a function** and should be cross referenced. 

- "free disk space" is idiomatic, "free disk storage" is not.
Trailing whitespace on this line.
I do not like this block. I do not think it is very readable. How about using `sprintf()` on this line, making `$filters` an array and using implode in `assign_vars()`?
The problem is, that this is also used to determinate whether the group exists at all, and not whether it is in the teampage
"Extension" would be the seconds word so shouldn't it be lower case then?
consider using underscore when iterating through object fields: 
   http://documentcloud.github.com/underscore/#each
   http://documentcloud.github.com/underscore/#keys
(many of the iterations and mappings in this codebase could be clarified with underscore; live and learn)
Alternatively you could just poke the port with a fixture hello or something. I am OK with this though.
Why check if state is falsish here?
linia asta nu funcționează așa cum te-ai aștepta.

first_index va fi pointer către last_index, care va primi valoarea 0.

cred că vrei să faci ceva gen: `first_index, last_index = 0, 0`
Parens are optional in ruby (except where needed to prevent ambiguities), and while this aspect of the language can and does get abused, I think they're totally unnecessary on a method call that passes no args.  The standard ruby idiom here would be `job = queue.pop`.

I know python is different here....
4 spaces per indent
add encountertypes: 
prescrip, PRESCRIPTION - Prescription encounter
prescripfille, PRESCRIPTIONFILL - Prescriptionfill encounter
It seems like my suggestion of the try, except, else construct hasn't helped at all. ```compatible``` is no longer used, but is needed to catch multiple coords associated with a single dimension. I  hesitate to offer another suggestion but here goes - it might be an idea to move lines 1500 to 1508 out of the try (as line 1498 is the source of exceptions we want to catch), and down into the else, putting the ```if compatible``` or equivalently ```if len(target_dims) == len(unique_dims)``` back in with it?
For a multi-line docstring there should be a blank line before the closing quotes.
I think a programmatic warning is too harsh. But now you've raised the issue, I'd like to repeat the docstring warning from the module header.
@rhattersley thanks.
I guessed you'd say this.
But I think,  when you have each arg assignment on a separate line, the extra spaces make it **read** much easier ...
Prefer...
```
result = function(
    long_named_arg = squiggle_diddle_widdle,
    another_long_name = complex + expression / numbers
    yet_another_getting_tired_of_this = how_about(x, adjustment_param=117)
    )
```
 -- to --
```
result = function(
    long_named_arg=squiggle_diddle_widdle,
    another_long_name=complex + expression / numbers
    yet_another_getting_tired_of_this=how_about(x, adjustment_param=117)
    )
```

It seems to me that in these cases, extra spaces help to visually separate the name from the value (which may itself have spaces in it).
Any further Comments?
``replaces``
You are right, I just mentally added an extra colon at the end of the 'opencl' string.  I'll change it back and add a comment.
Cut&paste error in sense of test: if (len(hosts) < max_hosts) abort?
Did you intend to test for the negated form of these three conditions?
that was my fault - bad copy/paste job
Look in the ScorecardSubcell, it is sorted.
Why are you cloning here? Can't you just append the fragment?
What about adding a validation check for `int(eps_rnd_seed) < sys.maxint`? And also leaving a comment describing why is it needed.
Our parser should be throwing an error as this invalidates rule 1 in the above comments.
Also missing whitespace here and on line 147
What is pre-steps?  When I looked at your comment on the prelim function, I thought you said the trans function (here) would make sure there is no duplicate.
I know it's from the existing source, but shouldn't we avoid mentioning it in the source?
Hmm, wondering what it is for.  If the special character is needed, why isn't it only NULL?
ArrayHandle::operator[] does bounds checking. Previously, we were also using mStorage[5], but only calling operator[](4).
no comma here too
same wtf :)
Noted, and I will fix TyrGodOfWar's work myself for the replacement version.
`skip_clean` goes above `def options` (in general we keep the DSL elements above methods).
Nit, but I would do this:

```rb
inreplace 'wemux.conf.example', 'change_this', ENV['USER']
etc.install 'wemux.conf.example' => 'wemux.conf'
```
I think my main complaint about Twisted web resources is I have no idea what any of the API URLs are and I've been reading the code for ten minutes.
I'm not totally clear on the terminology of comments either. So lets just be save and go with your suggestion.
Let's set up some short url redirects from yeoman.io, eh?
makes sense.
i can write such a maven launcher in ant.
#bashpro
I think ``$($(_getHTMLMenuItem(relativeID)).parents("li").get(0))`` can be changed to ``_getHTMLMenuItem(relativeID).closest("li")``
Good catch. I submitted a pull request here: https://github.com/adobe/brackets/pull/1392
You're absolutely right. So no objection to your proposal.
Use the double-back-tick to code-format ``Strings = require("strings")``
At this point, this doesn't make a lot of sense. 

If we want to keep `doResize` as a separate variable just in case, we should move this directly to `mousedown`. Otherwise, we could drop the variable or assign it to `newSize!==previousSize` and just check for it inside `doRedraw`
Should the code inform the EditorManager that we no longer have focus by doing the following?

EditorManager._doFocusedEditorChanged(null);
This is admittedly a very unlikely edge case, but there's a small possibility that you might replace the wrong thing here (if the last path component happens to be repeated somewhere else in the path). Would it be better to map the containing folder instead--something like:

```javascript
var parentUrl = _pathToUrl(matches[1]);
doc.url = parentUrl + encodeURI(matches[2]);
//...
rootUrl = (_isHtmlFileExt(matches[3]) ? parentUrl + "index.html" : doc.url);
```

(I guess that's more like the original logic, but still takes advantage of your new `_pathToUrl()` function.)
Converting to integer it shouldn't.

```
Time.now.to_i == Time.now.utc.to_i
=> true
```
This kind of thing is very tempting, and might even be faster, but should be avoided as a violation of DRY. I'd use `CouchUser.get` instead of `get_db().get` and then call `comment_user.full_name` directly. The overhead really shouldn't be that much.
This code repeats a query twice, causing potential future maintenance problems. Write it like this instead:

```python
query = basequery.filter(
    JobPost.status.in_([POSTSTATUS.CONFIRMED, POSTSTATUS.REVIEWED])).filter(
    JobPost.datetime > datetime.utcnow() - agelimit)
if sticky:
    query = query.order_by(db.desc(JobPost.sticky))
return query.order_by(db.desc(JobPost.datetime))
```
Might want to make this requestAnimFrame.
and other places it defines it as window.requestAnimationFrame/cancelAnimationFrame...
I do'nt understand why you need a second loop. Is it faster to set intervals directly in a JTable ?
This duplicates line 20. Instead, we should probably wrap the `allow_user_registrations` around `or Register`.
It's all ugly anyway, but I'd do this ugliness more explicit: `if instance_key is None or instance_key == ''` to withstand the temptation to change it to `if instance_key is not None`.
Why commented this out? Leave it in.
Problem is that form_valid doesn't get "request" passed to it. So we couldn't use auth.login(request, new_user). Have to do it in get() unless we pass request variable to the function as well, which would make it uglier.
Only a couple changes marked inline...
This background colour doesn't seem anything close to the gradients you're using, shouldn't it just be the top or bottom gradient?
Note: this was just clipped out of `src/editor/editor.js` to put it in its own file for cleanliness.
Probably, guess we'll see in the rebase
Instead of specifying "all" I think it would make sense that if you don't specify a property, then it just returns all track events.

``` javascript
if ( !property ) {
  return allTrackEvents;
}
```

And tests for this too
use `.backgroundColor`, this will apply extra settings like `background-position`, etc. that are unnecessary
These should probably be relative, or use {{baseDir}} if they can, no?
This isn't totally true. It will just default to the first target if one isn't specified.
https://github.com/mozilla/butter/pull/808/files#L1R11
can you update these last two occurences to look like the ones on 118 and 119
Fix this, file a bug to fix this, but don't leave it like this.
Do these urls not need to be in quotes?
I knew they looked wrong!
How are these changes related to the test?
On Wed, Jun 20, Josef Reidinger wrote:

> > +     * @id list.reverse
> > +     * @short Creates new list with reversed order of elements.
> > +     * @param list<flex1> v list
> > +     * @return New list. Changed if offset is correct, otherwise return unchanged list 
> > +     *
> > +     * @description
> > +     * Creates new list with reversed order of elements. Return nil if list is nil. 
> > +     *
> > +     * @usage list::reverse ([0,1,2,3]) -> [3,2,1,0]
> > +     * @usage list::reverse ([]) -> []
> > +     */
> > +    if (v.isNull ())
> > +    {
> > +	ycp2error ("Cannot reverse 'nil' list");
> > +	return YCPNull ();
> > +    }
> 
> After reading documentation for you method I strongly don't see reason for such error log - ```Creates new list with reversed order of elements. Return nil if list is nil.``` so it is documented as common and expected behavior
> 

This is first time I add an ycp builtin.

I just copied code of another builtin and changed functionality
so I have no idea if this error log has some reason or not.

Tschuess, 
	Thomas Fehr
-- 
  Thomas Fehr, SuSE Linux Products GmbH, Maxfeldstr. 5, 90409 Nuernberg, Germany
  GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer, HRB 16746 (AG Nürnberg)
  Tel: +49-911-74053-0,  Fax: +49-911-74053-482,  Email: fehr@suse.de
  GPG public key available.
is it still an option not to use it? just default to true?
Good point. Fixed here and in ARPACK and SuiteSparse as well.
Could you send me a link to the file where it is used? I would like to see whole usage context.

Nice trick! But it still feels little bit hackish. My solution is not perfect too, but in my eyes its a little bit better because its nor loading any files, which may be time-expensive if it is done for every test. 

In my opinion it would be best to replace all Glue modules with object instances (with behavior similar to a observer). Model object then can have the glue object assigned or not which would represent if the particular orchestration is on or off. It would simplify a test stubbing a lot. Not to mention it would be good OOP: more objects with simpler responsibilities, no god objects (current models where everything is included).

Anyway replacing current Glue modules with objects is probably out of this discussion scope. I think the questions are:
Do we want to unify approach to disabling orchestrations now? Which option should we pick?

I would leave both as it is and add story to the backlog to unify disablement of all three orchestrations.
ah, moved an existing method into this class, will update
Can you do a :through 2 levels deep?

It would need to go  ContentView -> ContentViewVersion -> ContentViewversionEnvironment -> KTEnvironment
Reasons are:
- Once you include the glue-code, it cannot be un-included without stubbing (which works only in tests).     
- To be able to create admin user in rake db:seed without foreman orchestration active. An admin user in foreman is already there. see https://github.com/Katello/katello/pull/818#discussion_r1804377
- To be able to disable foreman orchestration in specs selectively. I actually forgot to push the changes for tests to utilize this, remeding in aaeb782
will do ! :)
align these variables.
Use `switch (evt.target)` instead of `if` and `else if` here.
This link goes to `astropy.org`, but probably should go to `https://github.com/astropy/astropy`.
Hm. In the case of a CreateNewBlock JustCheck it would just be pointless computation on an operation which is supposed to be fast. Effectively in this case we don't know— and can't know— the hash yet.
No changes to txdb are written in the loop, only to mapQueuedChanges.
I'm a bit more leery of extract() these days. It makes it hard to figure out what exactly the local scope looks like.
Fixed in [bddea2b34bfa89b07c0bd0b2043445f884832460]
Yes, I got into an issue because Kernel is not initialized when I access the configuration here. There is no way to specify order in which Pre/Post methods run http://stackoverflow.com/questions/9041558/define-an-initialization-order-of-webactivator-preapplicationstartmethod-classes?rq=1
My implementation is here. It doesn't contain lifetime checks. It relies on var_export and __set_state only:
https://github.com/asm89/common/compare/master...DCOM101
Doctrine uses more or less the same CS than Symfony2. I'm not sure about how Doctrine wants to handle this (ping @beberlei) but Symfony did the choice to omit them in such case: http://symfony.com/doc/current/contributing/code/standards.html#documentation
Conditional logic based on class name isn't ideal. It makes it hard to test, and it makes it less modular. Try and take advantage of polymorphism where possible. How about something like

```ruby
render :partial => "message_types/#{message.presentation_type}"
```
Not that I'm calling a method on the method rather than checking it's class name. Now you might have that method returning something basically the same as class name, but at least it's possible to write other classes (with different names) but still return the presentation_type "vote".
oh yeah :-) thanks
Are you on IM? It's be helpful if we could have a quick chat about this.
Thanks, Robert. I'll try this.
Did you add setTimeout() because the parent div's width/height weren't available?  Won't this cause flashing of the UI when the png is resized after 250ms?

I had the same problem, and chose to grab the width of the input field (which is known at the point this code is running).  
https://github.com/mcelrath/ipython/blob/51514ef9c8dd7f8aeea35cb7da744ca16957e4e1/IPython/frontend/html/notebook/static/js/outputarea.js#L271

Because resize png/jpeg/svg will duplicate this code 3 times, we should think about unifying them.
Agreed :) I think the right margin should probably be a constant amount.
Bro, `giterr_set` already does printf construction. Why are you using a temp buffer?
Marcela, can you please remove these lines rather than just commenting them out? Thanks.
You can use double-quotes in sql. So this'd be a little easier as:

    'AND wiki_revision.content NOT LIKE "REDIRECT%%" '

Where "easier" is defined as "fewer escape character required".
oops will fix this
This was set but never used.
It would be more conventional for the progress listener to just take a float between 0 and 1. It's sort of weird that a UI decision (progress update granularity) is being decided here.

In slow transfers (say 1MiB over 3G) using percent is lame because multiple reads may yield the same percentage of progress. (10KiB and 19 KiB are both reported as 1% done).
This was done on purpose.

And please keep using ``integer``, not ``int``
Would you mind explaining? I don't understand.
CS are:
- Start with a one line summary (end the line with a '.'),
- one blank line before `@return`,
- one blank line before `@throws`
Well, a simple fix would be to inject Twig through a setter in the engine I think. But I'm not sure it will be enough. We may need to do it for each of possible entry point in the loop, i.e. Twig (already the case), the renderer and the engine. Btw, do these services really need to be public ? Making them private would limit the possible entry points in the loop (and making both the renderer and the engine private services would let only twig as public entry point, for which the issue is already fixed)
FYI, removing this ``continue`` does not break the testsuite
remove extra space before `)`
This should be reverted as this file is automatically generated when we release a new version.
I don't know why it shows diff here...
This change should be reverted as it breaks BC. This can be discussed for the master branch though.
I'm going to use len(self.workers) as the worker_id instead of i -- we'll need it when we signal TTIN
What is this file? Remove please. Make sure you pull before push.
LastId is used in ordering the feed (addPostsFromArray: postsArray: in CPVenueFeed).
`s/ID/identifier`
Unfortunately we don't really have a mechanism to tell the user that their ident isn't valid and have them try USER again with our current setup.  I was just thinking that we reject the connection, but... we could make one up?  I don't know what it'd be, but it's a possibility.
das sind im prinzip auch nur shortcuts. die machen deine formel fuer die auswertung aber ein ganzes stueck lesbarer
I'd rather not do this. I know why you have done it, but I would sooner find another way than have two identical (different instances) projections not equate to one another.

Is that possible?
C'est Test pas TEest
Вот так писать никогда не надо. Вообще писать такой код в одну строчку - плохо.
Правильно так:
if not save_to_path:
    save_to_path = self.result
@mockee 推荐一般情况下 property 使用 nonatomic 的。nonatomic 表示非原子性访问，不加同步；加 nonatomic 比不加会性能更好些，因为不加的话会多出原子事务访问的开销。
You don't need this with chef_gem. It will run at compile time by default since it's aware these gems will be needed at convergence. I have a similar pull request at #125
The ``-D`` switch from ``GNU`` ``install`` is a ``GNU`` extension that have no equivalent in ``BSD`` version.
What's the right approach here? Currently `_setOption` does various checks, then calls `_super` in the end.
idem (call not needed + anonymous function)
I wonder if instead of setting display none on all of these only to set it back again whether we should just overlay the results section over the top (making sure it covers the whole panel) with a z-index. That way we don't have to show it all again afterwards. What do you think?
`bodyHTML`
`notification-detail` not needed anymore I think.
`airplaneOff`, and use the same UI label. Let's confirm with UX on this later.
Why 'ui' in the path?
Remove this line of code. If it's commented, it should die.
it is needed because Workers does not have access to Console, thus this Object only forwards trace messages to the worker
I think we'll need to bump the revision here, to simContacts-imported3 (all seven strings).
These made up names should return to Open Sans if #2329 resolved.

CCing @ferjm @fcampo
Placeholder text until we get translations?
Good point, re: aq_base() -- see: https://github.com/seanupton/plone.uuid/commit/78d2387f7b0f6d8f66e882d85ff461408b52c312 as an attempt to do this without introducing a dependency on Zope2 in plone.uuid (conditional imports).  If that looks good, I'll close this pull request and submit a new one with the fix.

I need to use getattr() in place of hasattr() because in my own add-ons I have a r/w property proxying to another attribute name actually storing -- but I still want my object to provide IAttributeUUID.  hasattr() would return false positive, and I want to replace a None value.
But didn't we run into cases where npm was dying without outputting anything? Or is that a separate issue not yet addressed?
#210 has a better fix for this issue.
didnt think about snapping forward, was ok with it fetching replacement on forward requests...That could be a future add.
Wouldn´t `date = new Date value` have the same effect as these two lines?
Does this run? I would have expected a 203 on the first call, followed by a 404 on the second
I don't think these group binders should search for all input tags, but rather take an option `name` that identifies the group:

```html
<span data-bind="foreach: colors, radioGroup: { value: selectedColor, name: 'colors' }">
  <input type="radio" name="colors" data-bind="attr: { value: $this }"/>
</span>
```

Further, how does this respond to changes in the radio group? Above, if an element is added to the list `colors`, and a new `<input>` is created for it by the `foreach` binding, and the new `<input>` is selected, will the group binding do the right thing and change the variable `selectedColor`?
I am not allowed to write sexy code yet.
Rather than pulling the documentation out of the app file temp directory (since it no longer exists there), I pull it directly from its original location.

All the documentation files now appear in the DMG disk image mac users download.
tighter with

if (expPkgName.endsWith(".*")) {

or even

if (expPkgName.matches("[^\.]\.\*$")) {
This isn't defined.  Which worries me a little, since this has no unit tests -- did you run this? :)
There is also a discussion about `Interpolate` in the features section.
this search should be a scope in user, and most likely needs tests! :)
OK，这个我们到时一起来做吧~~
I don't think we should hardcode blob. I don't even think we should have a tag option. Ideally the url should be like this:

    https://github.com/elixir-lang/elixir/blog/master/%{path}#L%{line}

And then path and line are injected via a regex. :)
最後に`change`が呼ばれてから、silentに変更があった属性のハッシュ。次にコールされた場合、この属性は保留中となる。
"Изпуска се параметърЪТ след..."
И махни този коментар след `case`.
https://github.com/rails/sass-rails#features
hm it makes sense. At least validation should not be so strict like for path
Since this is never called, you can just remove the function.  We still have a copy of it in `udp_client.c`, so there's no reason to preserve it here.
If we use a template, let's maybe use a templating engine, 
We now require Jinja2 for notebook why not for nbconvert. 

Especially, me might be able to use jinja2 for loop to do things more cleany and just pass it an array of cells.
Ah, but it's not a legit word in Scrabble.
Cool, I didn't know about the "!" syntax :)
Trying to get me head around $title and $text to pre-empt a platform response.  In the context of this function I'm wondering if it might be cleaner to leave $title as the column heading (matching the original parameter description) and add a new $tip for an optional tip prefix.
fileno is posix, we compile with c99, so I enable posix extensions
probably should link to a specific version?
This is not necessary `event/core` will log exception if there is no handler for error events.
No, this is just an attempt to detect the data type if omitted – or if "image" is given. If we are not to able to auto detect the datatype, we proceed with the defaults and the information we have.
I forgot about strings :) I'll add it
Why use `renderWait` here?
Gobbledygood is very cool! Works on node or in the browser, so we can use it everywhere.
This kind of UI-focused modification (joining into a comma-separated string) should happen as close to output as possible (i.e. in the management command). A model-layer function should return pure data structures so it's as flexible and reusable as possible; in this case a dictionary where the values are all sets or lists of project names.
this makes perfect sense.  reminds me of a technique we used on MozTrap, too.  :)
Can you please add a short comment which explains why we can't easily read the package.json file?
__whimboo commented__
> ... Have you thought to make that method a property instead? Would be nice to get access to it via nightly.changeset`.

Issue #70 could contain that change.
Have I mentioned that Zend Engine could use some documentation? :)
@zaphod42 not opposed to delegation, but it seems like overkill
Yes, application can override command line. It should be clear that it is the default that is being set. It is probably just "master" that actually overrides the default.
I would just replace 'hidden' instead of relying upon 'suite hidden' be an exact match.

`.replace(/(\\s|^)hidden(\\s|$)/), '')`
Will investigate on spaces :)
Changed in $appName
You can declare it since newer PHP versions if the signature is compatible with the inherited one.
I would let it in because the signature specifies the return type much more exact (IteratorInterface extends Iterator) and it should be uncommented if the min. PHP version supports it.

You are right to more document on class level ;)
Looks good. I wasn't sure you'd dig it :)
I don't think model is proper place for this. Maybe some view, or addiotional method in existing extension.
Spaces still here.
if (i in results) { 
  newResults.push(results[i]);
}
this will fail if you get Doctrine and run composer to get its deps as it will be in ``../vendor/autoload.php``
can you do those tests with combined operations to see if things break? i am a bit afraid of merging without knowing what happens. if we hit problems that are hard to solve, i would prefer to throw some exception rather than create a hard to debug invalid state with hidden data loss.
I just realized that the purpose of #488 was to remove the `SITEURL` for relative urls with this change:
```python
if self.settings.get('RELATIVE_URLS'):
    assets_url = './theme/'
else:
    assets_url = self.settings['SITEURL'] + '/theme/'
```
and this change is useless if we need to use `{{ SITEURL }}/{{ ASSET_URL }}` in the template. 

With the lack a proper solution to have relative urls, I think it is better to remove these changes, and advice people to set `SITEURL` even for local development (with `http://localhost:8000` and a python SimpleHTTPServer for example).
Not sure about style here. You are not using isbooleanoption further down the code path, so maybe do not assign and simply conduct the test? I'm fine with it either way, we should just adopt some style and stick with it.
Erm yeah.
Maybe you are right, but IMO we should add some per resource scope validation, as IIRC this is fully custom to implement according to spec.
Konacha also supports `test/javascripts`.
I don't see a test for this yet?
```python
eq_(r.status_code, 200, 'this is why i failed')
```
indeed, left-over from trying to coerce facebook iframe into cooperating, which ultimately failed. I shall remove
matches doesn't exist anymore, just set this.media = entity
they both should work but self.collections is clearer -- I'll fix that
An even better suggestion: show that EDN.register takes a block as well.

EDN.register("clinton/date") { |date_array| Date.new(*date_array) }
I believe the behavior is if you set innerHTML on a parent the child nodes references you have will become null unless they are explicitly removed.
the example text should say: "bold because not filtered"... fix that and I'll merge :) thanks!
What does this do? I added it and it killed the footer entirely.
In this scenario, upon restart the projection would be considered a "new projection" and created. As far the the old projection goes, that would still exist (in other words the federated segment on the old parent).

I really don't know how we can determine that a projection has "changed"  and therefore remove it from its old location (which also brings up the aspect that we don't really have a way of removing projections atm)
Why "formatter" and not "ui"? Is there something specific meant by "formatter"?

Most libraries use "ui", so far as I know.
The tests for the Pro features will fail if the user does not have Nexus Pro installed. While I do have a scenario that first checks if Nexus Pro is installed, the rest of the scenarios will continue to run (and fail if Nexus Pro is not installed).
emmited -> emitted, and backticks around socket.send
"$template"??????

use only

$template
OK typo. Should mention *FS* instead of *DFS*.
adds articles relationship
oh right I forgot this isn't backend! gosh darn it. I'll rebase :)
In other words, it doesn't hurt to have them in the .gitignore! I'm indifferent with it.
Maybe something like "New version X.Y.Z available (you're currently on X.Y.Z). Visit <url> for details." ?
Should "global" be accessed as a constant somewhere rather than hard-coded?
This one can do reply as well, with an optional `replyTo` parameter.
Yes, this method may be needed by the JDT, but we provide completions using an extension point. Maybe we can remove it altogether.
Ok, it seems we can't do better than that... I'll shut up :)
I thought so as well, but `getCoreFormatterOption` goes through a map, so maybe we get `NoSuchElementException` as well, in exceptional cases..
It is not backward compatible with lower versions of python (remember, lint tool is different from django code as we want to keep it compatible to 2.4+), and to be honest, I find it very hard to read in this specific example.
All of the text above that explain what CE and EE are and what are the differences are covering a topic that is a bit delicate. I think we should avoid covering that in the documentation and instead point to the related sections within our website. This is specially important as we are right now changing a bit our messaging around it.

All I would say in this document is that EE Subscribers will have access to additional maintenance releases as well as specific Service Packs and Fix Packs that allow applying fixes to production environments safely and faster.
那这理应是一个更好的 vaildateReturn 实现，而不是新开一个

在 2012-10-21，下午11:10，Chris Yip <notifications@github.com> 写道：

> In validator.js:
> 
> >  
> > -    // 通过则返回 false
> > -    return result;
> > +  // 返回的错误对象 = {
> > +  //    $el: {jQuery Element Object} // 当前表单项
> > +  //  , type: {String} //表单的类型，如 [type=radio]
> > +  //  , message: {String} // error message，只有两种值
> > +  // }
> > +  packageErrorObject = function($item, klass, parent, type, message){
> validateReturn 达不到想要的效果，因为 validateReturn 是验证和样式的处理是一起处理的，:checkbox:checked 会被跳过，即使这一个组是 invalid 的。
> 
> 所以我把验证和处理分离了出来，在 182L 里单独调用。
> 
> 而且在 171L 已经有一个 validate.apply 了，再调用一个 validateReturn 不划算。
> 
> 我这里的思路是从 validateReturn 的单个元素验证改成一组元素验证，验证完统一修改样式，所以封装一个可以复用的 packageErrorObject。
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
Can't we sort this in python? One-time effort
was in de eerste instantie om hem te resetten als hij van spotted naar niet spotted gaat, en niet flip= false en flip = true door elkaar haald met links en rechts.

moet nog even goed kijken of dit nog nodig is
This will fail on Windows OS, try using

    $expected = getcwd().DIRECTORY_SEPARATOR;
Кстати да, NSUInteger предпочтительней, иначе очень хочется придраться а зачем тут именно этот тип :)
Вам дали ТЗ, в котором попросили: "Сделай нам, пжалста, класс, у которого в состоянии DNA типа NSMutableArray* будет храниться ДНК, и чтобы, эта, можно было посчитать хэммингово число с другой ДНК, да ещё и изменить случайным образом ДНК, вот!"

Есть пункты, которые нужно выполнить, в данном случае, исходя из ТЗ, должен быть класс Cell, у которого ДНК хранится в DNA, у которого есть метод -(int)hammingDistance:(Cell *) и который может изменить с помощью метода -(void)mutate:(int)x. Вы начали с того, что не выполнили пункт относительно DNA, я, как заказчик, тупо подключу ваш класс к своему проекту, и не получу желаемое.
Соглашусь, проверка x не проводится. Но с другой стороны, x при DNA = 100 никак не сможет  быть > 100.
same comment on path as above.  Also, I may not fight this for the moment, but I am sort of concerned about having the public rng file embedded in our app.  We did that with previous incarnations of this and had issues with the public one being updated and us using a different version because we were not aware that, say, factory had updated what they were using.  Really I think rng needs to get a version added to it, which would make this a non-issue.
Esta es la nueva interfaz del update.
WARNING which is XXX
Think we're missing a command here. Looks like the original doc header had two commands.

```
# hubot sqs status - Returns the status of SQS queues.
# hubot ec2 status - Returns the status of EC2 instances.
```
yeah, it's kind of dirty ;p You mean an absolute path?
does this work? np.inf is a float and has no len(), you could just use fmin < 0
and raw.copy(preload=fname)
would preload in a new memmap or if preload=True (in RAM) just do a
deepcopy of self
I would call it source_freq_connectivity

In terms of parametrization would it make sense to pass bands? I am trying to anticipate use cases
the same about returning the error
```return response.end();``` and remove the else statement
This is an example where I think having the argument names in the spec just add noise.
How about:

    Unfortunately, life isn't quite this simple, since most browsers do not supporting sending
    PUT and DELETE requests.
you should use ``:method:`` to reference the method
sdumitriu was fast on the commit button, I've removed them in commit https://github.com/jamiemaher/xwiki-platform/commit/417106d4dbbd7906d31361ddbbe156fa913faa87
Why this waste of developer time?
there is a little header that goes before each class, check the fsl or spm preprocess that will give you access to dummy data files for the docstrings
these dummy files are located in nipype/testing/data
You can use `add_option` method here.
should we log every time it recurs?
Why would the user provide an empty string? But even in this case you can use `if something:`. The if block wont be executed for `something == None / "" / [] / (,) / similar_empty_objects`. Anyway, it's rather minor remark, don't worry about it.
How about renaming it to $reuseDependentData or some other more explicit name?
To be sure that the lang exist, true ?
Need to assert that the package is installed on the client machine.  (on all tests that do installs)
Looks to me like it returns a function that 'Returns true if the given notification contains the specified request type.'
Tested this and it works as it should (at first glance it seemed like it would produce something like `[1,1,2,1,2,3,1,2,3,4,...]`). 

But it looks really weird. How about just doing this?
```
story = [Paragraph(x, headers[x]) for x in headers.keys()]
```
Or perhaps splitting that in to two lines:
```
story = []
story.expand([Paragraph(x, headers[x]) for x in headers.keys()])
```
I think you mean when the index does matter -> the index being the order of the elements in the array. I'm guessing you mean that when you don't have additional information about an element (like a key value) that you would want to reference it by.
Use the magnificent RecorddRequest Object ?
I'll move the first line into the else statement. I missed that the activity would be restarting.

Regarding having mapController in CGeoMap at all: I wasn't sure why the MapView didn't keep a copy of the controller. It only creates a new MapController every time getController() is called. But, I'll leave that one for rsudev as he still understands the mapping code better than I do.
`_(i8n)`

it begins
u have to explain this to me. It seems like this is duplicated code with whats below.
I increased lifespan 10x and maxIdle 5x.
This method should assert that the argument isn't `NULL`.
Trailing backslash.
>
> I have used None as an alternative to 'default', 'neither' is used in the
> 'extend' argument which already existed and is nothing to do with my
> code. Also, None is not surrounded by asterisks in any of the other
> docstrings in colorbar.py, I thought it best to be consistent? Do I need
to
> change anything here?
>

I don't think we should have two argument values that have identical
affects.  Standard practice in mpl is to have None as default.  Eventually,
this could be an rcparam and the practice there is to have None mean "get
the rcparam value".

As for the asterisks, doc styles have not been enforced, until now.  I am
working on updating the existing docs, and it makes it easier on me if I
make sure all new code follow this.
Also, needs a docstring, and the name of the method is misleading.
Unclear.  Maybe say, "has no effect in plot() function; see axes.color_cycle"
@bp323 Since these `ifs` are near copy/pastes, can't we use a separate parameterized function for this?
With a bit of trial and error, this seemed to work best for 1, 2 and 3 column layouts for displaying the content title.
sometimes, m_length_y is NULL. You have to check for that
same holds for  nested for loops  (and you do that a couple of times in the patch)
Because there is two of these, and we are only targeting one of them. I don't think it makes sense to generalise in this case.
Thanks for the comment. I added a third commit to this pull request that fixes that problem too.
I don't think they have a global timeout, maybe it would be a good idea to add one.

If we expect a timeout of even "just" 30s on the IRC notifier then that would require more work since currently the notifier would block and everything else would be delayed (I think per notification type).
We can say for sure that engine versions after and including 536.11 do not have the bug. Earlier versions may or may not have the bug, which is caused by Chrome for iOS reporting an incorrect Webkit engine version. So yea, we can limit the check to versions below 536.11. I don't see any harm in running it on all Webkit versions (or even all browsers), but I don't have any strong arguments for or against (other than that I didn't notice any problems with running the check.)
so I just looked up the doc on mongodb and ```:upsert=>true``` does exactly what create_if_not_exist does.
So I'm pretty sure you can get rid of ```create_if_not_exist``` and change ```set_experiment_created_at``` to what it originally was and everything would still work as expected.
Api documentation.
This section has the same concerns as my long comment for the previous section.
@carhartl Actually, if I change that and the user didn't pass an option it would fall to $.cookie(key, undefined) eventually deleting the cookie and not allowing the check.

I think the only cookie removal should be === null cause then we would be able to create a proper signature by checking
```javascript
if( value ) {
  //setting cookie
} else {
  //getting cookie
}
```
Why do we need to fire a change request here and how does it differ from the one fired on line 647?
To avoid the costly `count`:
~~~clojure
(not (my-some (complement pred) a-seq))
~~~
I tried to get rid of those, but admittedly I was in autopilot mode, just copying messages there.

@eddieajau - thanks for doing this. Getting closer to getting codesniffing and testing in place. With phpStorm v 4, we can use codesniffing in the IDE. That'll be very helpful. I have the beta installed but have not got the Joomla standards in place yet. https://plus.google.com/109030869001230031837/posts/A2BeQNsXheR
Yeah, agreed. :/
this could be simplified as all of them are defaults except ``colors`` and ``bootstrap``
How about,

```ruby
platform_family?("rhel", "fedora")
```
No, not `user.language` (although it is the same as user.language, but this is just by our design and middleware), but `translation.get_language()`. So simply you display in current language. We might someday allow to override language without changing the preference. Or something.
Mnja. Let it be. :-) But you should add TODO, that it should not be hardcoded like this. :-)