labofsahil/github-event-dataset-2012 huggingface dataset:
you can use 

from bigml.util import (
	invert_dictionary, 
        localize, 
        is_url,
        ….)

     
On Dec 21, 2012, at 3:53 PM, mmerce <notifications@github.com> wrote:

> In bigml/api.py:
> 
> > @@ -56,7 +56,8 @@
> >  except ImportError:
> >      import json
> >  
> > -from bigml.util import invert_dictionary, localize, is_url
> > +from bigml.util import invert_dictionary, localize, is_url, \
> > +    clear_progress_bar, reset_progress_bar
> I believe it is to use multiline imports
> …
> —
> Reply to this email directly or view it on GitHub.
> 
>
I put everything in the User model...you're right, it makes a lot more
sense there.

I looked at eager loading...I don't think it really makes sense here, as we
would probably load a lot more data than we need since we're really only
making a couple of queries, at least for bookmarks. It will definitely be
useful for reviews and comments though.

That aside, fixed everything else.

*Nikhil Goel* | Vanderbilt University, 2013 | Computer Engineering,
Economics**

LinkedIn <http://www.linkedin.com/in/ngoel1> |* ngoel36@gmail.com | ( (256)
503-9183 | NikhilGoel.me <http://nikhilgoel.me/>



On Thu, Nov 29, 2012 at 6:40 PM, Luke Steensen <notifications@github.com>wrote:

> In app/helpers/videos_helper.rb:
>
> > @@ -1,2 +1,9 @@
> >  module VideosHelper
> > +  def bookmark_listing
> > +    return current_user.bookmark_listings.find_by_video_id(@video.id)
> > +  end
> > +
> > +  def bookmarked?
> > +    return bookmark_listing
> > +  end
>
> In my opinion, these should be on the User or Video models. For example:
>
> if current_user.bookmarked? @video
>
> or
>
> if @video.bookmarked_by? current_user
>
> What they're really doing is checking for the existence of a Listing
> record in the database. Helper modules (at least in my understanding) are
> more for helping out with complex view logic than getting information about
> models.
>
> Now we obviously don't want to be hitting the database for every one of
> these calls, so we're probably going to want to do a slightly bigger query
> at first and bring all that data in at once. Then we can add some method in
> one of those models that quickly checks whether such a Listing exists.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bitcast/bitcast/pull/22/files#r2273403>.
>
>
If we leave it, does Bitcoin-Qt open properly?
Would it not try to send to the mq, send properly, and then exit?
Making anyone open Bitcoin-Qt several times until the mq fills up?
Why add spaces here? Most if the ifs() are like this, and IMO that's fine.
I think this constant should go
The results can change per rule of a field. Similar reasoning as I explained above for `checkRequired()` as `_allowEmpty` can also take values true, false, 'create', 'update'.
Yes


On Mon, Dec 17, 2012 at 3:57 PM, Mark <notifications@github.com> wrote:

> In lib/Cake/Network/CakeResponse.php:
>
> > @@ -444,10 +444,18 @@ protected function _setContentType() {
> >  		if (in_array($this->_status, array(304, 204))) {
> >  			return;
> >  		}
> > -		if (strpos($this->_contentType, 'text/') === 0) {
> > +		$whitelist = array(
> > +			'application/javascript', 'application/json', 'application/xml', 'application/rss+xml');
>
> it should be in a new line then right?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/cakephp/cakephp/pull/1007/files#r2435468>.
>
>
to please @guilhermeblanco who does not like ``else``, I would write it like this:

```php
<?php
if (!isset($this->classNameHashes[$class->name])) {
    $this->classNameHashes[$class->name] = sha1($class->name);
}

$key = $this->classNameHashes[$class->name];
```
no it cannot be removed. See the line just after this creating a ReflectionClass and the issue linked in the description.
[MEDIUM]

Month_ is rather a quarterStart_ here.
Please rename.
Please move the closing brace to a newline, like the old code above.
Please incorporate 84c668f943d.
yes, of course.  Sorry I got it backwards.  Thanks!
And I should note that the reason I asked for no "_" in this case is that there are other filenames in this directory that are ~equally complex, none of which have underscores.
I am fine adding it but the parser does not check for it (and preferably won't). We can't fix incoming correctness in any case only incoming.
You should peek at my tests, you might find some with even longer names :D
You're not using these conditionals to avoid appending the boost info or the minimum should match.
shhh, i only asked for @rehandalal's opinion here :-P
This `num_voted` is bothering me, oh my OCD :-). I see that is how it was defined in the form.
Having it be a list of multiple errors might be too forward-looking - right now there is no more than one allowable error per execute() statement
oh, right, thanks :-)
fixed in b9fe980
"출판일시: "
should be
"출판일"
额 好吧。我去掉它。
`@see` or `@link` before link could help a bit IMO.
My point is, having a string key is not false usage. And the implementation shouldn't (and doesn't) care about the array key (no need vor array_values). For example currently you allow to use `array(5 => 'option1', 89398 => 'option2')` (non consecutive intergers), but `array('5' => 'option1', '89398' => 'option2')` (same with string key) raises exception.
arf, indeed
$extraData can still contain values if the forms children remain unchanged within the loop. See testBindChildKeepsLoopingUntilNoFieldsAreAdded
I don't think any distro ships with this on, it's a legacy switch that
some people compiling php manually still have for historical reasons
more than concrete reasons IMO (except for the few oracle+php users).
Just waiting for feedback on whether we cleanup this command or close the PR.
As said previously (but my comment disappeared because of the renaming of the file), this driver should follow the DI pattern instead of instantiating the dependencies itself (write the unit tests and you will see it will help you)
this should be simply a ``if`` as the previous block returns. And you should use ``&&`` instead of nesting the conditions
We only add to the benchmark group test that are really slow. That's not the case here.
More on the dagLevel can be found here:
http://en.wikipedia.org/wiki/Layered_graph_drawing

It's found by finding the longest path and creating levels from each node. Then every other node is assigned a level such that each branch is a short as it can be.
This dummy function makes me believe that it would be fine if A+ modified the original post parameters. But is it clear that we need to modify them in some case? And what does this have to do with the actual feature of this pull request.
There's got to be a way to handle multiple photo upload methods without copying so much code. If anyone wants to change the upload pipeline, he/she will have to change code in at least 3 places.
Good call.  Verified and fixed.
hameye comment haro yek joor bezar
index ro bebin
since the method does not return a bool, this is not a predicate. Only closest, instead of closestPredicate ?
Class doc missing.
I like this addition.  Might be more appropriate to check for `ENV['logging']` instead of adding to the `Runcible::Base.config` so that someone reading the tests for implementation details doesn't think there is logging option.
Yep. 
But you are welcome to make specific formatting commit (in the master branch).
When don't we have a `loaded?` method? Non-AR stuff? I hate checks like these. :/
We don't have a strict convention for this, but the event name should probably be the same name as the function without the "get_" prefix.
Possible rephrase: `I am signed in as a tenant`.
There can be multiple persistent units in a persistence.xml file, I think this should be a list
this one I'm not sure again. should it be "problem" or "problems"? /cc @steveklabnik
Good riddance. I don't know why I ever thought this was a good idea.
if (!alarm)
     alarm = this.getDefaultAlarm();
We should revoke the url we created to release some memory

URL.revokeObjectURL(url);
https://developer.mozilla.org/en-US/docs/DOM/window.URL.revokeObjectURL
Maybe we need to change the right padding to 1rem as well (RTL compatibility).
Better story ever.
plantype -> `planType`?
better to do this by toggling a className
Sure, thanks :)
Why do you dispatch a click event? can't you simply do myElement.click() ?
that is install gjslint https://developers.google.com/closure/utilities/docs/linter_howto
and run |make lint| from the root of gaia
Resolved - Event has been removed.
From L123 to L126:

   if (req.result.length > 0) {
      self.contactData[options.filterValue] = req.result[0];
   } else {
      delete self.contactData[options.filterValue];
   }


Overwrite the cache instead of `delete` it then write it.
What happens if this activity is invoked while the music app is already running and playing a song? I don't know anything about the new audio API and how it handles contention.  Do you have to do anything here to ensure that this song overrides any music that is already playing?
I like early returns as well, but here it's not an early return (which in my view is "a condition => a return") because you're actually doing stuff :)
`setEnabled(true)`
If sats is not used, you can remove it :)
you miss the uploaded++ for the OsmbugsPOint
I think the period might need to be inside the docstring.
You could link to ProjectRazor::Error classing here and tie into proper HTTP response binding we have there.
这个不应该泄露，请作废这个 access/secret key。
If you use the `request` function directly, you can pass the http method right in; thereby removing the need to condition on the http method.  Something like this --

```javascript
request({
   url: xhr.url,
   method: xhr.method
}, function () { /* handle */ });
```
Check which table version we're using (old vs new(blob))
if old, put the Object into varchar;
if new, put a byte[] array in to the MEDIUMBLOB that can be read again as an Object by an ObjectInputStream.
fixes flags not being able to be parsed while using mysql db, and scalable: It knows objects, so MANY more flags can be enabled and stored this way.
Personally, I would find `[a-zA-Z0-9]` much clearer, I don't have to look up whether `:alnum:` includes caps for example.

Instead of the kleene star, it would be more correct to use `+`, as this will currently match the empty string.
see above -> lets remove these `disabled` checks by switching to real buttons instead of divs?
I prefer `getRelativeSpeed` ... or make it a computed property.
By the way, this code handles both cases efficiently (both of your if branches)
Do we want to make a last assertion here?
If (!$ourFileHandle) fclose...
I knew that it wasn't necessary to handle it here, but I though it made the code easier to understand if I explicitly showed that this block of code will raise exceptions in some cases and that they will be re-raised from this function.
Added a currency mixin. All currency related computations can happen in the mixin. For now, it just has a property called centesimalValue, which returns 3 for a few currencies and defaults to 2.
I've seen that you use it nowhere, so I removed it. Get rid of the useless `require`.
Since these get_*_uri functions only can called inside this class, I think it's better to add '_' in front of these func names, to mark them as private methods.
The "evalRegexp"? I double checked and I don't need to override it :) made it private again
More generally, I will probably need to rewrite various constraints to make them imperfect, so I'm making the class extensible. 
If you prefer, I can make "protected" only the ones I strictly need now and then submit more PRs if I need anything else
Not a big deal, the reformats in this case don't really make the patch harder to review but given the discussions on geotools-devel recently about keeping reformats out of patches i thought it worth mentioning. Be nice to keep things consistent for everybody and lead by example for new devs.
The ID or code name of a project
"as a regular script tag"
Wouldn't it be better to force system module to only use `sdk/...` form?
So that we wouldn't call `normalize` here.
This is a smart change.
Has the concept of verified vs unverified emails gone away completely, even from the backend? What happens if the user requests an email re-verification but they have not verified the email in the first place?
Very cool. Didn't know how to do this before :)
There isn't ;) It's indirectly tested, since I lifted it from the private function in user.js
If you want to send JSON over the wire I think you want: `res.json({message: 'success', clicks: clicks});`
I found the <echo> tasks to be handy for MemChaser. Any reason for removing them?
__whimboo__
> ... I'm happy with a follow-up for this RFE.

Mee too! :) Filed #80.
By default it could be interpreted as OvR (only if it is square), but any output code could work -- we would just need a convention for it (e.g. `-1 = False`, `+1 = True`, `0  = ignore`), at this stage we should just assume it's `OvR` and let the user know in the docstring, but possibly open up the possibility of other output codes (e.g. `OvO`).

Thoughts?
Why the change to lazy creation of `*root*`?
Understood... but I have left a few comments scattered about re: futher improvements to how we deal with "run_mode".  Do you want me to remove this comment?  Revise it to be more explicit and reference other places where I've left notes about run_mode? Or..?
Missing a `¿`
Use `proplists:get_value/3`
Any reason you don't use latex() here?
Haha whoops. Copy/paste error -- fixed.
comma after 'above'
@weierophinney It's not necessary regenerate the packages. This files are not used by Composer, at this moment only they are used as replacement of ZF2Packages/dependencies/*.ini

Notice Debug, Version, and Registry don't have any composer.json definition yet
That's fine, as the listener that triggers init() always passes this argument.
it is already defined in line 4
in that case we should reference the .git repository and it should be in some place or repo special for forks? bower can fetch these deps
Do these work? I've never actually used these helpers. Twitter and Facebook seem good, but does anyone use Delocious anymore?
awesome. I will change my code. Other than that, are you fine with the functionality in question?
What're your thoughts on renaming this to "guice-modules" or "bind-modules"? I actually thought at first this defined a module in some larger, plugin-type sense rather than just a Guice module.
seems that the bug was fixed already :)
missing die here
(I mean that would be useful to tell it here, I can see that this info is available in this file, if we read a bit more)
There is a HTML tag you can add to tell Facebook to use a particular image. Ashwan found it once.
use `self.moduleGenerator` below
Why not create this once and not worry about it in all these methods?
the client looks only at the next repetition level at the end of the column. See javadoc for ColumnReader.getCurrentRepetitionLevel()
https://github.com/julienledem/redelm/blob/master/redelm-column/src/main/java/redelm/column/ColumnReader.java
Sorry @AlinT, it should be fixed now.
I understand why you're doing it but I think that instead you should make sure that two results come back so the the test is actually testing the search page since we should have other tests for returning one result.
Any reason not to use the fetch function here instead of manual peek & calcout?
vmware (and others) cant co exists without the fog group, should they be separated?
I pushed a small fix about that. Right now it would add periods after [A-Z], but we can expand that to include Cyrillic and other letters. I'm not a huge expert on asian languages, so maybe someone else can figure out the unicode ranges for what can be punctuated.
actually I don't think we're pushing translators pre 3.0 anymore.

On Wed, Sep 12, 2012 at 11:13 AM, Aurimas Vinckevicius <
notifications@github.com> wrote:

> In APS-Physics.js:
>
> > @@ -0,0 +1,142 @@
> > +{
> > +	"translatorID": "f318ab1e-71c6-4f67-8ac3-4b1144e5bf4e",
> > +	"label": "APS-Physics",
> > +	"creator": "Will Shanks",
> > +	"target": "^https?://(?:www\\.)?(physics)\\.aps\\.org[^/]*/(articles)/?",
> > +	"minVersion": "1.0.0b3.r1",
>
> Since you're using the ZU shortcut below, this should technically be 2.1.9
> (
> http://www.zotero.org/support/2.1_changelog#changes_in_219_september_3_2011).
> It's not really that important in this case, because translator updates are
> not pushed to clients that are older than 2.1.9 (IIRC) anyway. Other than
> that, this looks ready to be merged.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/zotero/translators/pull/456/files#r1589386>.
>
>



-- 
------
Sebastian Karcher
Ph.D. Candidate
Department of Political Science
Northwestern University
It knows to wait for ```Zotero.done()``` the same it knows when translation is done: by tracking the number of outstanding asynchronous requests. If you return a value, it should be ignored if ```Zotero.done()``` gets called.
"When you generate config iby default it will go into APPPATH unless you specify a module. There are two ways to specify the module:"
Fixed, I removed all comments like this.
Continuing a line on another line should be indented once.

var privatePluginConstructor = pluginType.GetConstructors(System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance)
    .Where(_con => _con.GetParameters().Count() == 1).SingleOrDefault(_con => _con.GetParameters()[0].ParameterType == typeof(PluginCore));

Again, add using statements instead of fully declaring type names.
As the comment above that block says, I figured we would just adopt the units from the object on the left. I'm not sold on that idea, but I don't like silently switching to SI units. What does @eteq think?
This guy really should be bounded. It's *probably* safe to just assume it won't grow too large right now, but when we get to the time when we're dealing with legit thousands of subscriptions, I really don't think it's safe to assume they'll mostly be for the same products. Also if we're loading 1k content sets on some products we could be in real trouble. Would be much safer if we just hang onto the last maybe 100 products requested.
I'd suggest having the `Server` instance set on the `Environment` instead of the `Service`. The semantics of `Environment` fit better and it's much more flexible.
Change this condition, the multiple files only works on linux servers
Does this have to be a one-liner out to the horizon?
Changes the default footer row height to support non-multi-select scenario, but only if user has not specified something explicitly.
will this line also be a bottleneck, like other fixed in https://github.com/evert/sabre-vobject/commit/30283a5f4f494432f8b0c81528a98a31da58417f ?
Will not be easier just to define that method in the resource class:

def id
@table[:id]
end

So you avoid monkey patch.
This still seems odd to me. What about having distinct 'date' and 'time' arguments?

    def get_build_info_for_date(self, date, time=None, build_index=None):

I think this would read better.
I think at this point it is too late to say "bye" to the channel. Why call the command?
Would it make the code cleaner to also use XPath for these early queries?
please only 2 spaces for indentation..
Right, that makes sense.
You could also wrap all the async stuff in a `process.nextTick(function() {` which would make it truly async from the request I guess?
Yeah, I'd say there's no need to check the type of callback
the FinalMatcherInterface does not have a setContext method
Shouldn't this still be `…gh.git`? Also, this doesn't really have anything to do with this PR, does it?
I would expect to be able to use accept to do that. So we have this in our code:

```javascript
if (req.accetps("json")) {
     res.contentType('json');
     // ... some code...
     res.send(200, jsonObj);
} else {
      res.contentType('html');
}
```

But since it always returns true it always goes to the if branch, which is not what I would expect.
"По заданию кол-во символов == 100, значит совпадает с процентом."
Oh, you are right. It cure with small utpgading of algorythm:
in do - while generate new number (from 0 to 3) and check it equality to that we already have.
If it is equals - generate it again and return if OK.
Or another method - create an copy of array with nucleotids, remove one element that equal with current and generate number from 0 to count of new array. This if funny method, but take some memory.
Хм, а разве так получится? Ведь переменная по умолчанию приватная... Но попробовать стоит, спасибо. 

Уже думаю написать модификатор по индексу и сделать проперти только для чтения. Надо было сразу так и делать, но подумалось, что написанное мной будет проще. 

--
Your sincerely,
Sergey Lebedev.

Twitter: @drasgo
Facebook: s.a.lebedev
Usually, var statements with multiple lines are indented once more to the right (IMO better readability), but your call, that is an opinion, and a nitpick. :-)
Maybe it should be interesting to port those deleted lines to its specific spec just to guarantee we didn't fuck up with something.
This method is called for both test cases in this class. It may make more sense to use `setUp()`, which is already intended for that purpose.
@Haacked You're right. They have to be created yet as discussed in #137.
could be combined with the above "if connectivity is None" into a single if/elif/else statement (but it doesn't reallt matter ;) )
Please remove this parser and use the one that you defined earlier.
Looks like this translates to "Cannot be empty or null".  I suppose that is good enough.
No need to use reflection here. We can have a static reference to the `Global` object.
I'm -1 on using VERIFY_NONE when we actually have a proper SSL certificate in place. In this case, people are dealing with a trusted resource, and that should be properly enforced by checking the certificate as well.
not every browser supports `map` (in particular IE8 and below). Using `map` here would either require a shim or dropping support for older browsers
What kind of timeout is this and why isn't it confgurable?
There's nothing Akka can do to make arbitrary Extensions TS. The only thing Akka can guarantee is that there'll be at most one instance created of the Extension. You'll need to make sure that the Extension is TS.
That's a bit extreme ;) I'd leave a few arguments on the first line and not put the closing brace on its own line.
Using () is preferred in these scenarios to clearly indicate what is involved in the expression.
No creo

On Tue, May 29, 2012 at 10:58 AM, Alpha <
reply@reply.github.com
> wrote:

> > @@ -115,6 +115,22 @@
> >              initialize: function () { }
> >          });
> >
> > +        // Nervoustissue.DataLinking.Compound
> > +        // -------------------------------
> > +
> > +        // Se encarga de registrar el evento change en un modelo
> complejo de backbone.
> > +
> > +        m.Compound = m.Base.extend({
>
> ¿Esto está declarado dos veces?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/CommonJobs/CommonJobs/pull/89/files#r892262
>
Ese por que lo sacaste?  (yo igual no los uso más, me instalé RavenDB como servicio y listo)
There shouldn't be a www in this url
it's not 1, but l (lowercase 'L'), for 'level'
here the starting_point isn't a string, but it's a Place object... so you're comparing point.name to starting_point. Either of these would work:

```
start_index = points.index { |point| point.name == starting_point.name } 
```
or
```
start_index = points.index { |point| point == starting_point } 
```

This would break your program though... You'd need to change the salesperson.rb to something like:

```
places = all_boroughs.take(5).map do |borough|
  place = Place.build(borough) 
  phil.schedule_city(place)
  place
end

starting_point = places.find{|b| b.name == "Westminster"}
```
What should I do then, add the feature to the other sources or implement it in RowAction::setRouteParameters?
In my opinion setRouteParameters is the way to go
sleep with parent
I think it is okay since we would need arcseconds mostly when offsets are displayed. We may interpret "s.sss" as arcseconds only when offset mode is used.

```python
ax.set_coordinate_system('fk5', offset=(272.3, 44.3))
```

It would be also good if methods like *set_xticklabels_format* also take a formatter-like object so that users can implement their own fomatters.
In this case though, I am not sure if we can reuse the Formatter class from the matplotlib, as locator and formatter is more tightly coupled in our case.
Nope. If you don't have any arguments for an exception, you don't need to instantiate it.
Fixed, just waiting to commit. I hope that code has same functionality, and I agree it is at least more readable.
This should be `"mongodb/bind_ip"` (minor typo)
Id like to see this a a config value as well
Exactly this. Of course it would be a nice exercise to do this without `count` also.
No need for this. That's what scala.collection.JavaConversions is there for (used throughout the code base).
Again, no need for toString()
Hmmm, why organize or re-organize the imports? AFAIK, we're normally Ok with using .* if there're loads of imports of same package.

Such reorganization makes pull requests noisy ;)
Please place the string in the Log
This LON names need to match the site names in the Infinispan configuration, right?
Please revert all these white space changes. If we want to have any hope of merging these upstream, we need to tread as lightly as possible.
ArrayListを返すため、メソッド名は"getPoketCards"のような複数形にしてください。
Why < 2?
It **wasn't** guess work. It's a fallback if the user has _safe_mode_ turned on - which renders init_set calls useless.
This should be updated[:-1] + (spec,), no?
From the context here it's unclear if this is in some EMI-only case (I guess it is from the OverWorld-mode-mention above, but asking to make sure)?
I think the number 0 will throw an exception here, which doesn't seem right to me.
Used `the content` instead of using `it`
please use CMath::{max,min,pow,swap}, SG_ALLOC / SG_FREE and friends
Will change that
Are you sure ``'Unable to locate scrolling style plugin manager with class "invalid adapter"; class not found'`` has the correct type as "string" is not a sub-type of "string"?
Is it possible to add a ``@param`` annotation for parameter ``$resource`` which defines a more specific range of types; something like ``string|array``, or ``null|MyObject``?
???? ?? ??????:
??? ??? ?????? view, ?? ??????? ??????????? ??????? ?? iPhone ?????? 60.
?????? ?????? ????????? - ????? ????????, ? ???? ?? ?????? ?????? view
??????????, ??? ????? ????????.

On 04.12.2012, at 16:47, "Evgeny V. Kokovikhin" <notifications@github.com>
wrote:

In SDK/src/WPBannerView.m:

> @@ -89,7 +89,10 @@ - (id) initWithBannerRequestInfo:(WPBannerRequestInfo *) requestInfo
>  		
>  		_shildImageView = [[UIImageView alloc] initWithImage:[UIImage imageNamed:@"wp_banner_shild.png"]];
>  		[_shildImageView setHidden:false];
> -		_shildImageView.frame = CGRectMake(0, 0, 9, [self bannerHeight]);
> +		
> +		float bannerHeight = [self bannerHeight];
> +        	float bannerWidth = bannerHeight>60? _shildImageView.image.size.width*2 : _shildImageView.image.size.width;

@DenisBaryshnikov <https://github.com/DenisBaryshnikov> ??? ????????
????????? 60 ? ?????? ??? ?????? 60, ? ?? 42, ? ????????

--
Reply to this email directly or view it on
GitHub<https://github.com/WapStart/plus1-ios-sdk/pull/4/files#r2306868>.
Sollte man da noch null prüfen, falls es gar keine solche Relationship gibt? Sonst wirds hier wohl ne NPE geben...
Guter Test!
OK. I change this.
Why are we changing this?  The class has still existed since 11.1
And from this choice depend documentation too. It is still missing from the PR.
too many empty lines :) let's keep just one
Yes. What should it be called? sendEvent?
the menu name is the name of the node under `menu_basepath`. for example if your repository stores the menu nodes under `/cms/menu` , rendering "main" would mean to render the menu that is at `/cms/menu/main`
Sounds a lot more consistent. Should I do this in a separate CL?
Why two divs? Should outer have `panel` class?
*svojega e-mail naslova*

Space after dot.
Add also bundle. You never know what is useful to others.
Add TODO here: we want user to redirect back to where she came from.
it should be ``elseif`` without space
~/.appscale and move this to a constant? Or is there already a constant for this?
'/acccounts/' links are for authentication views handled by auth. We probably just want these links to be `user_settings` and `house_settings` respectively, so they link to views (which I will create) in the dashboard app.
This is not our dependency, it is somewhere in the rails stack, so we should not have to add this ourselves.  Did you hit this on fedora 17?  If so, it is because of the broken ruby 1.9 they ship, which does not include stdlib, where minitest is
I think the `percentage` should be a double to make for a smoother movement (not because you can draw half a character, but because the double rounding makes it jumpy).

Example (width of 40 characters):

- 1 % = 0.4: 0 characters
- 2 % = 0.8: 1 character
- 3 % = 1.2: 1 character
- 4 % = 1.6: 2 characters
- 5 % = 2.0: 2 characters
- 6 % = 2.4: 2 characters

(These values will not change, but allowing the input to contain decimals improve the overall calculation.)
Jep mit dem Backbutton.
Funktioniert zwar nicht, wenns in der resource schon Platzhalter hat, in diesem Fall ist das aber nicht so und deshalb ega.
query string params for the sake of the test, yah?
IMHO we should add 1st argument (providerKey) always even if user provider not used. I plane open PR fixed this to FOSFacebookBundle. Explanation: we can not put providerKey into auth provider here but authentication listener always contains providerKey. And method `supports` will always return false for authentication providers without user provider.
Using different guess-the-MIME-type approaches depending on whether the resource has a name or not seems like it would lead to surprises. Are you sure that mimetypes can't interrogate a stream, so as to at least use a subset of the same logic in the stream case?

Also, is there a way to explicitly specify the MIME type, in cases where the caller knows already?
I can fill a bug on launchpad, the different importers are entangled with the persistence layer i.e. CatalogueDatabase.
I would suggest to rethink about the interface of the different importers, my idea is that each importer should return a representation and then a different component storing it in the db.
Yeah that makes sense, leave that up to the subclass. I'll take it out.
Принято обрабатывать случаи когда instance is None
Kernel.puts ?
Actually maybe I'm missing something. How do these functions differ from the fulfillment and rejection functions above? Just more lightweight versions?
The hosted services.
Is that just content for testing?
Unnecessary memory allocation? Doesn't `options->x_header_value().c_str()` suffice? You'll probably need to const_cast it but c'est la vie.
This should reserve `n` items rather than 4.
is LOC in every subject? surely not. But oh well
Этот кусок спионерен из жуйквери. Здоровая простыня, но что ж поделать...
Make the second Nodes "NODES" as it is essentially read only and final
Unnecessary empty lines?
Wait... why are we deleting the bean for the Nhin Proxies?  We should delete the passthrough ones.
I'm guessing this is `so if we let the...`?
Why not fwhm here?
ditto this, how about uniform_deviate?   (admittedly this is kind of long so I understand if you prefer not)
LEN should be lowercase to use the len() function.
Please do not bump the version in your PR.
yeah, as part of cleaning up JcloudsLocation we should make it more piecewise.  could even link to jclouds unit testing which has template support etc.
Perhaps return boolean to say if the id was found (similar semantics to Set.remove)?
catch (Exception e) { Exceptions.propagate(e); }   :)
Can you add a case for an invalid date time format? with associated exception being thrown?
Everything looks good up to this point. I'm reluctant to have a mousemove handler tracking on the document because it's kind of like a nuclear option. It can put quite a strain of a client's resources (kind of like resize and scroll handlers).

Also, this is going to produce errors if the document contains x-domain iframes.
whitespace before ')'
I don't think this does what you think it does! `itsoffset` is a timestamp offset, not a seek time. You're looking for `-ss`. (Also, it's dangerous to put the seek before the `-i` flag - this will work for some formats, and not others.)
Using a hash here would be a more idiomatic ruby way.
The iteration you do later on wouldn't even change.
By Maven specification, AFAIK, inside `src/main/java` or `src/test/java` should only be placed *.java files. If you need other files (E.g.: *.ini, *.xml, *.properties, *.txt, etc.) they should be put either in their own directory (E.g.: `src/main/xml` - not so sure about this) or put inside `src/main/resources` (most likely).

In our case, in my opinion, the following files are misplaced:
- src/main/php/xp/compiler/default.xcp.ini, inlining.xcp.ini, etc.
- src/main/php/xp/grammar/php.jay, xp.jay

By default, when you run `mvn compile`, the xp-maven-plugin does the following:
Step 1: Copy all XP-Classes (*.class.php files) from `src/main/php` to `target/classes`
Step 2: Compile all *.xp files from `src/main/xp` to `target/classes`

At Step 1, xp-maven-plugin expects that into `src/main/php` there are only XP-Classes that match the default pattern `**/*.class.php`. You can overwrite this behaviour using the `${xp.compile.phpIncludePattern}` property.
I'm not 100% convinced I like this animation
Odd indentation again?
I would propose a new Config option "undoLogging" or something that is tested for in lively.Main and if set it requires that module instead of loading it unconditionally.
Board configuration panel for instance? Like the pipx configuration? That's
not possible to put into a config file, and adding it to the xonfig gadget
is just dirty...
On Dec 11, 2012 12:06 PM, "PTDreamer" <notifications@github.com> wrote:

> In ground/openpilotgcs/src/plugins/boards_openpilot/coptercontrol.cpp:
>
> > @@ -0,0 +1,71 @@
>
> I still don't see the point. All those examples, image, widget for config,
> etc, don't require a new class for each board and can be dynamically found
> and used. I don't see any benefit on this, I only see one more think to
> remember and handle for each new board.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/PhoenixPilot/PhoenixPilot/pull/79/files#r2383870>.
>
>
typo s/Instante/Instantiate
Не корректное оформление JSDoc блока http://code.google.com/p/jsdoc-toolkit/wiki/TagReference

```
/**
 * @constructor
 * 
 * @param {String} data description
 *
 * ...
 */
```
Надо использовать Generics.
В таких случаях надо исключение кидать.
Perhaps instead of converting these to arrays and using %0 and %1 in the string template. We could switch to using %language and %medium in the template and change the values to being an object. This would probably make it easier to read the template and ensure that the correct data is put in each token.
Randall, Randall, Randall.  You are quoting the man page from Mac OS X.  I am referring to the man page from Ubuntu Linux.  However, if you insist, we can change it to -L.
Juan, we added this return statement, so if the execution won't step into the above if statement the methodInvocation.proceed() at the end of the method won't be called again.
I still prefer ``s'`` but don't have a particular US/UK fight to pick (:smile:). I'd leave it to Helen to ensure consistency.
And http://eclipsesource.com/blogs/2008/08/22/tip-split-packages-and-visibility/ gives a workaround for when you do want to export split packages twice (or more).
In the example, you pass a list rather than an array, so this should read array-like.

It might also be worthwhile to mention that the target values may be arbitrary hashable and comparable Python objects (since that's what `np.unique` accepts).
I'll add a comment that this tests the deprecated stuff, though.
matrices (and then a comma or colon?)
that might make sense!

but maybe you could start this change by storing a `@songs` array on the jukebox?
Y'a une raison particulière ?
Sorry to confuse you this should look like the same from the instance create:

`"flavorRef": "https://ord.databases.api.rackspacecloud.com/v1.0/1234/flavors/2"`
As far as I can tell the only difference between the "if postgres" block and this one is startup and teardown.  The rest is duplicate code.  This could be simplified.
This line is broken.
All platforms return lowercase information
shouldn't this be conditional on the control being currently disabled?
Jira updated: https://jira.appcelerator.org/browse/TIMOB-8821
I try to avoid negations as much as possible, maybe change it to: `if options[:note].present?`
See discussion at doctrine/DoctrineModule#14
It's not user-visible until 4.0, and this commit is targetted to 3.10.2
I expected the interface would be

CBucket * queryBucket(n);

with CBucker::queryWriter() etc.

rather than various arrays in this class.
Wouldn't it be more efficient to start from the last going backwards?
Doesn't `urlencode_params()` already take care of turning things into safe strings?
Can you de-dent this massive string a bit so it fits in (or more nearly) 80 chars?
Not directly related to the docs, but the 404 page really should differentiate between a - d in dev mode!
Isn't it equivalent to
`this._loadClassAndGenerate(classDef, logicalPath);`
The scope will still be `this`
Do we need a `if (ServiceTimeoutException.class.isAssiginableFrom(scanClass))` clause here and in the other branch of the FSM?
These arguments are not in the function anymore - you could just say that the t dictionary is going to be submitted as kwargs to matplitlib's axhline
Read more about why I used `target="self"` here on 
http://docs.angularjs.org/guide/dev_guide.services.$location
and
https://groups.google.com/forum/#!msg/angular/basidvjscRk/Whth9JMFAKwJ
I'll test this locally, but I think this change also requires a change to timeshift.rb (to make the fake injected date match).
style suggestion (feel free to ignore): when you have so many members to initialize, split them one per line, prefixed by comma, e.g.

: QObject(parent)
, q(parent)
, manager(new ...)

... that way, removing them is a lot easier to parse in a diff view (you see exactly what got removed), and adding them is also often only a single line change:
+, foo(bar)
instead of

-moo(cows)
+moo(cows),
+foo(bar)

this also makes it trivial to make sure ordering is correct compared with the .h
This should also only add a single DC like the ALL flag:

				found_dcs = enum_dcs(domain_name) 
				# We only wish to enumerate one DC for each Domain.
				dcs << found_dcs[0] unless found_dcs.to_a.empty?
Fix tabs, please.
You do have to re-watch (because that sends the watch to the zookeeper server).

The register call is a library-specific construct — you only need to
do that part once.
This can be speeded up by using: `return result * mask`
Maybe `image > dtype_in(dtype_range[dtype_in][1] / 2)` would be better?
yeah, one thing that is very sparse and constantly out of date with openvz is its documentation.  apparently simfs is the way to go if you want to get around the quota thing.
Please remove the console.info
Is this a better default that `Assets/Imagery/earthspec1k.jpg`?  I'm undecided.
Can we omit the output=false and displayname attribute since output defaults to false in script and display name isn't doing anything?
Do you need to explicitly give these namespaces on every property? Doesn't it pick up the namespace from the XmlRoot attribute on the class?
This should be a *very* rare scenario, but it's best to be certain we have the right config element
CloudConfig.Role != null) { do happy path }
perhaps rename 'toRemove' to 'childViews' for clarification.
Here we try to update the property of mViewProxy, so we should use mViewProxy.updateLocalePropertyNameAndValue().
There are leading spaces on this line just before `bos`.
I don't think it's needed right now.  I think a feature pass will happen in more depth once Anvil gets some time in the field and people have time to digest the pros/cons.  I mean, we could jot down these ideas but I don't see it as any kind of reliable roadmap at this stage of the game.  My 2 cents.
Why would the view be null?
Why do you need "var myWin ="?
Dude! Unknown Error trumps all errors :)
There's no need for 2 blank lines here.
Could this be a stub?
The previous version was really long. How about:

1. `then` may be called multiple times on the same `promise`.
    1. If `promise` is fulfilled, `onFulfilled` callbacks must execute in the order of their originating calls to `then`.
    1. If `promise` is rejected, `onRejected` callbacks must execute in the order of their originating calls to `then`.
Should this be `--[no-]fail-fast`?
On Thu, Mar 8, 2012 at 10:12 AM, Daniel Pittman
<reply@reply.github.com>
wrote:
>> @@ -0,0 +1,17 @@
>> +module Facter
>> +  module Util
>> +    module PuppetSettings
>> +      class << self
>> +        def with_puppet
>> +          begin
>> +            Module.const_get("Puppet")
>> +          rescue NameError
>
> This is a whole lot of magic, right here.  It should have some comments to explain what it is all about.

I found myself back in the code today and this comment should be
addressed in https://github.com/puppetlabs/puppetlabs-stdlib/pull/50

-Jeff
I think you mean `\d+` in this regexp, no?
We check access to route
It's exactly the same code as is used in the C implementation for Python. It's under a "BeOpen" license, which is only a few pages of text and seems very lenient for the most part.

Code - http://svn.python.org/projects/python/trunk/Objects/listobject.c (merge_compute_minrun)
License - http://svn.python.org/projects/python/trunk/LICENSE
All right, I have no more complaints. :-)

The `hasSlicing` debate is worth having, I think. It is not obvious (at least not to me) that a slice needs to have the same type as the slicee. At one point it was even suggested ([by Walter](http://forum.dlang.org/thread/h5nbj3$17is$1@digitalmars.com), no less!) to make built-in arrays and slices different types... But that is for a different forum, and not this pull request.
We should migrate this condition into the template constraint.
ブラウザ組み込みの非同期APIは当然として、優れたアーキテクチャのアプリケーションは、低レベルのAPIも非同期に呼び出せるようにするべきです。
Sim entendi.. Tranquilidade :P
Was simply copying what we had in others. If we don't see a point in having
remote we could drop it. Tasks uses remote as well.

On Thu, Aug 2, 2012 at 4:10 PM, Pete Muir <
reply@reply.github.com
> wrote:

> > +            </dependencies>
> > +        </profile>
> > +
> > +        <profile>
> > +            <!-- An optional Arquillian testing profile that executes
> tests in a remote JBoss AS instance -->
> > +            <!-- Run with: mvn clean test -Parq-jbossas-remote -->
> > +            <id>arq-jbossas-remote</id>
> > +            <dependencies>
> > +                <dependency>
> > +                    <groupId>org.jboss.as</groupId>
> > +
>  <artifactId>jboss-as-arquillian-container-remote</artifactId>
> > +                    <scope>test</scope>
> > +                </dependency>
> > +            </dependencies>
> > +        </profile>
> > +        <profile>
>
> Do you really want this profile?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jboss-jdf/jboss-as-quickstart/pull/324/files#r1297961
>



-- 
Jason Porter
http://lightguard-jp.blogspot.com
http://twitter.com/lightguardjp

Software Engineer
Open Source Advocate
Author of Seam Catch - Next Generation Java Exception Handling

PGP key id: 926CCFF5
PGP key available at: keyserver.net, pgp.mit.edu
hmm. The code using this line is broken since 2.0 :O
Is PSU_DataObject like this because it has yet to be namespaced?
Then more pythonic to go with "is True".
Not sure. Exception's description in MSDN implies that it is used when accessing elements of an array by an invalid index, which is not exactly our case.
I am not sure I understand you correctly. Which line of code does your command apply to?
header and unit tests added.
Why was this validation removed?
Thinking about it, I'd like to use filtermetalink for meta4. Instead a new prop filtermetalink3 should be added  that reads "Metalink 3 (Legacy, XML)".
Don't forget to update tree.js reflecting these changes
У всех этих классифкаторов должно быть текстовое описание title. Можно пустое
Well we actually use it already.. because in one case we have some weird strategy that depends on a who sees the data, and we exclude fields based on that, so on some actions we just add that strategy to it. I don't see what a builder would bring apart from more code to write.
It will be shown when you run app/console router:debug ... so maybe a nice name makes sense here :-)
:thumbsup:

Makes a lot of sense to me (for the same reasons I propose wrapping of method call parameters and array initialization with a trailing comma on the last line before a line followed with `);`
Yep, $focusable is an array of jquery objects and we want to focus on the first or last.
Reverted to `isX86_64`, but my Programmer OCD still doesn't like that name. ;)
I intended __gshared.
Maybe we should tag these cross-posts? How about a link to the original article?
This is a bit worrisome since someone could have followed ``Windows -> Developers -> oops, Unix``
Good question - the best thing I know to do is to just pull the branch into your fork, make your changes, then do a new PR.
what did you mean by the comment: * This map contains a groovy.lang.GString descendant* ? Is this important for the test?
So, I know it's a little hokey, but I've kindof used the dates as a means of ordering the pages, so: this could be `2011-5-1-widgets.markdown` to go with the flow of the doc pages...
move JSONOBjectAdapterException into onSuccess and use proper client model error message
Should we make this a strong type?

List<ListKeyAndUserSubscriptionPreference>()
cleaning up thefailcallback to pass errors, this is crucial for the errors to show up in the welcome modal.
self.assertEqual(a, b)
It will never happen, so you can remove that.
You mean flagging it as "BB10 API Changes"? Good idea.
вроде можно делать Comparator cmp = std::less<T>
вместо объявления еще одной функции
Should be `self.timer`
We'll probably want to log all of these at the same level, so let's use a single LOG_TAG for each: <tt>SyncJPakeStage</tt>.
I think this test is a little too ambiguous. The region manager itself also has a `$el` attribute on it. If we changed the show method to return `this` instead of the view, this test would still pass. 

A better test would be:

```js
var view, returnedView;
beforeEach(function(){
  view = new WhateverView();
  // ...
  returnedView = myRegion.show(view);
});

it("should return the view", function(){
  expect(returnedView).toBe(view);
});
```
Is there value in moving this into the Webplatform?
two usb interfaces sounds a little strange?
missing copyright/license boilerplate
Just `(== (mod n divisor) 0)` is sufficient, as it returns a boolean.
idem que ci-dessous.