labofsahil/github-event-dataset-2012 huggingface dataset:
Why 0.5?  What does that mean?  If the credit is not a number do we instead want to indicate an error?  (this is what the existing answer choice validations would do right?)
I guess this means invalidateCacheForDomain, but is out of scope to be made consistent with the other methods?
This is scary.  For anyone who actually has a zkcluster that they are using, they will never be able to use this utility because it's gonna dump everything and a real ZK cluster will probably have a *lot*.  It's probably best to not do this and either explicitly go for the information that you were trying to dump with this, or just not do anything and have people rely on things like exhibitor to provide visibility into ZK.
And the canonical link is surprisingly https://groups.google.com/group/mongodb-user.

@tychoish can we make the link a macro somewhere?
to return statistics on the plan used to select indexes and fulfill the query. 

we need a provisional link here to the explain output that kay is documenting.
can we add a test for <h2><h4></h4></h2> too?
Design/architecture-wise it is much better to have them in the drivers.
Is array_merge() required? Why does `$config['phpbb_functional_path'] = $_SERVER['PHPBB_FUNCTIONAL_PATH']` not work?
This conversation was never about whether it would be possible to update or not, just about how this particular bit is implemented.
Because than the else is not tested in the same test
1. Take this logic out of the query string.
2. The code already knows the set of forum ids that the query is restricted to. Filter f_read_other forum id list against that set before giving it to the database.
3. Once you do 2, check if user has f_read_other in all forums being queried. If so omit this condition entirely.
What's that?

On 14 Nov 2012, at 11:05, Erik Frèrejean <notifications@github.com> wrote:

> In phpBB/includes/api/api.php:
> 
> > +		}
> > +		else
> > +		{
> > +			header('Content-type: application/json');
> > +			echo json_encode($data);
> > +		}
> > +	}
> > +
> > +	/**
> > +	 * Throws a 404 error in the format used by the API.
> > +	 *
> > +	 * @param string $message The message to error with.
> > +	 */
> > +	public function throw404($message, $format = false)
> > +	{
> > +		header('HTTP/1.0 404 Not Found');
> Should probably be calling the send_status_line function here.
> 
> —
> Reply to this email directly or view it on GitHub.
>
$php_ext.
fixed in fcac8ef
A matter of style, but if this requires `mojito` module then we should just be able to access this as `YUI.Env.mojito.DataProcess` no?
That was my thoughts, just giving a helping hand with what I could do ;)
почему нужно указывать точное значение?

UPDATE `unit_stat` SET `shows` = CASE `unit_name`
WHEN 'test' THEN `shows`+10
WHEN 'test2' THEN `shows`+20
ELSE `shows`
END

такой код работает. и он помоему намного понятнее инсерта, но вопрос в производительности
The indention in this file is a bit off
https://github.com/maxwe789/Nancy/blob/95bc8962ecff1c39787f95959b55ade806a9dabe/src/Nancy/ModelBinding/ModuleExtensions.cs
This (bogus?) is code formatting.
yes -- I have mentioned the universal extractor.  But I thought that the beauty here is partially in abstracting user away from statsmodels API while providing them with a simple function for GLM analysis they might have done before.  With the extractor, to e.g. for multiple contrasts, I would need to become quite aware of the statsmodels API to accomplish the goal.  So, I thought, that at least for the common cases extractor should not be necessary and could be hidden away from the user with suggested API enhancement without sacrificing the simplicity.  Then this could be used beyond a rudimentary feature weighting for the sake of feature selection.

Just my 0.1 cents -- otherwise I do not have comments and it could be merged
model_level -> model_level_number
Not sure what this check is trying to prevent? The interface only offers a list of selectable databases, security is done by giving the database user proper rights to query the database?
This class has a lot of duplicate code from theano.compile.debugmode.BadThunkOutput. Moreover, the functionality in theano.gradient.numeric_grad and theano.gradient.GradientError is really close, not to mention the logic in TensorType.approx_eq may benefit from it.
I think this is the opportunity to finally refactor those exceptions into one (or a base one, and other that inherit from it).
Its a typo/spelling fix. Not even worth arguing about.
On Thu, Aug 2, 2012 at 5:33 PM, Łukasz Langa
<reply@reply.github.com>
wrote:
>>  def _prepare_model_groups(request, query, tree=False):
>>      query, sort, page = _prepare_query(request, query, tree,
>>              MODEL_GROUP_SORT_COLUMNS, default_sort='-count')
>>      for g in query:
>>          g.count = g.get_count()
>> +    query = [g for g in query if g.count]
>>      if sort in ('count', '-count'):
>>          query = list(query)
>
> Queries are always lists already, this is not needed anymore.

This is not to make it a list, but to filter that list on g.count,
whoch is not a database field.
-- 
Radomir Dopieralski, http://sheep.art.pl
Нужно переделать как-то более правильно.
Ditto. Looks like this piece of code is copy-pasted.
Nitpick: please keep spacing the same, there're spaces around the `=`.
similar question here as to `collection.each` see above
I would simply use ``if ( ! empty($params['host'])) {
you should explain that the closure passed to extend will receive the object created by the previous definition as first argument
Ah, I get the version number change now. I think we should get rid of `python-noq` and just stick with `python-oq`.

I don't think we should maintain 2 different streams of version numbers. If we haven't created a package yet for it, can we just call this oq-engine v0.8.4?
The object would be local to the thread because it's stack allocated. I was only noting that it would allow you to make a single write call without concurrency issues.
This is a strange thing... It's really bothersome that IE10 actually released a browser that allows a clone's style property to retain reference to the source element.

Anyway, yeah this belongs in `jQuery.support`, maybe called `jQuery.support.whenWillTheyLearn` ;)
The hard numbers are masked by rounding, but http://jsperf.com/jquery-fragcache shows a ~7% speed improvement from the cache on IE6-7, no significant difference on IE8-9 and Chromium 20, and ~13% on Firefox 17 (not that we really care on newer browsers).

There *is* a benefit, but it's so modest that I'm still in favor of dropping the cache.
is there anyway we could split this over two lines? :)
At this point we don't know whether the array has been allocated. Checked bellow in this function.
For exception safety, use a boost::scoped_ptr to hold the result of readDocument().

boost::scoped_ptr<BSONObj> o(readDocument(file, buf, BUF_SIZE));
I think this line should be indented to match the one you removed.
We don't want to do this.
A good question, sir!

Probably I should change it to `args << '--without-gcc' unless ....` ... how do I best check if we build with clang/llvm-gcc ?
why disable readline? If libedit of OS X does not work well, we have `depends_on 'readline'`.
Maybe worth logging the exception / traceback here?
Can you please restructure this and the 2 other `unsafe()` calls so the markdown is applied in the templates?
I'm ambiguous on this... NullPointerException, or IllegalArgumentException.  The former is more of a "null dereference" whereas the latter is what's happening.  But if I left it be, there would actually be a real null dereference further down the chain, which I'm just bringing forward here.
Replace <COPYRIGHT HOLDER> with Steinwurf ApS
I think it does add readability and one notch less of cognitive overhead
Hm... i guess you are right. I will remove this functionality.
Are you talking about lines 284, 287, and 288, or 287, 288, and 291? 

In the case of lines 287/288, that's what @jason-sanjose suggested for sizing those correctly. I generally get what you mean but that might be beyond the scope of this specific pull request. I can create an issue for it though.
Oh, I didn't mean to single out parsing vs. searching so specifically.  It just seemed like it'd be useful to isolate the pure-JS stuff that _findAllFunctionsInText() does from the JS-plus-noisy-disk-IO stuff that _getFunctionListForFile() does.  But having timing for Document creation seems to provide most of the same distinction, so I'm fine with what's in the pull request now.
Yes, the original intent was to use shouldShowHintsOnKey to determine whether to open or close the hint list. But indeed this breaks the HTML hinting and isn't necessary for the JS hinting that I'm working on separately. I will amend the pull request.
@redmunds:
I think we can be sure: in search(), selectorInfo is set on every searchResult that's returned.  The value it's set to (itemInfo) is guaranteed to be non-null by line 75 where it's dereferenced (although also it should be safe to assume that the input list before filtering doesn't contain null entries).  Same is true in the HTML & JS versions of this code.

In the old code that was using the slightly fragile getLocationFrom*() utils to map back from the DOM node to a search result item, it sort of made sense to be worried that something could go wrong and give you null.  Now I think we can safely assume it's always non-null.
@domenic -- agreed. filed issue #1034 -- for promises that get handed to third-party code (e.g. extensions) we should use a more robust implementation

@jason-sanjose -- well, extensions are likely to listen for the 'ready' event. We don't want extensions to be able to _break_ the 'ready' event. I added a comment as to why we aren't using JQuery deferreds (and filed bug #1034)
On second thought, since this is "quick" open  it should probably ignore case when searching, but respect case when sorting.

For example, in the case where there's multiple matches that are all lower-case and also multiple matches that are mixed-case, I think that it would make it easier to differentiate between the choices if they were separated and not mixed together.
And if x is NULL?  Placement_dtor is substituting for "delete" which is NULL safe.
You should be specifying the exception types you want to intercept. Using a catch all except at this level is prone to hiding silly mistakes that would otherwise be quite obvious.
I thought the data location type needed to be passed to get questions as there are different questions activated for different types?
Ah okay I see. Even when you're logged out this still gets called. This is
a hack, but I guess it doesn't matter.

On Wed, May 9, 2012 at 6:33 PM, Daniel Trauner <
reply@reply.github.com
> wrote:

> > @@ -6,5 +6,6 @@ class ApplicationController < ActionController::Base
> >
> >    def current_user
> >      @current_user ||= User.find(session[:user_id]) if session[:user_id]
> > +     rescue ActiveRecord::RecordNotFound
>
> So what would that line do that prevents one from having to do that?
> And I did sign out, that didn't fix it.
>
>
>
> On May 9, 2012, at 6:32 PM, "Gareth G. Aye"
> <reply@reply.github.com>
> wrote:
>
> >> @@ -6,5 +6,6 @@ class ApplicationController < ActionController::Base
> >>
> >>   def current_user
> >>     @current_user ||= User.find(session[:user_id]) if session[:user_id]
> >> +     rescue ActiveRecord::RecordNotFound
> >
> > You have to log out (ie visit /signout or clear your cookies).
> >
> > On Wed, May 9, 2012 at 6:29 PM, Daniel Trauner <
> > reply@reply.github.com
> >> wrote:
> >
> >>> @@ -6,5 +6,6 @@ class ApplicationController < ActionController::Base
> >>>
> >>>   def current_user
> >>>     @current_user ||= User.find(session[:user_id]) if session[:user_id]
> >>> +     rescue ActiveRecord::RecordNotFound
> >>
> >> Dude I probably dropped and re migrated the database 15 times and
> >> couldn't get rid of the user with ID 1 not found bug.  That fixed it;
> >> I can give you the Stack Overflow link.
> >>
> >> Dan
> >>
> >>
> >>
> >> On May 9, 2012, at 6:25 PM, "Gareth G. Aye"
> >> <reply@reply.github.com>
> >> wrote:
> >>
> >>>> @@ -6,5 +6,6 @@ class ApplicationController < ActionController::Base
> >>>>
> >>>>  def current_user
> >>>>    @current_user ||= User.find(session[:user_id]) if session[:user_id]
> >>>> +     rescue ActiveRecord::RecordNotFound
> >>>
> >>> Kill this. If there is a user_id key in the session hash, then it
> should
> >> be in the database. The only reason you'd ever lose it is if you change
> the
> >> database in which case you'd want to have the user log out.
> >>>
> >>> ---
> >>> Reply to this email directly or view it on GitHub:
> >>> https://github.com/gaye/rideshare/pull/22/files#r798494
> >>
> >> ---
> >> Reply to this email directly or view it on GitHub:
> >> https://github.com/gaye/rideshare/pull/22/files#r798524
> >>
> >
> >
> >
> > --
> > Best,
> > Gareth
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> > https://github.com/gaye/rideshare/pull/22/files#r798533
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gaye/rideshare/pull/22/files#r798538
>



-- 
Best,
Gareth
Yeah, the tests are probably a bit of a shitshow at the moment
*django* -> *Django*
Remember the variable you tried to remove from the settings file? There is something commented out there. :-)
also should store the value of options.linkUrl in a variable since it's accessed so much
Don't need khtml
I didn't even see the containerWidth calc. Nvm. This is fine :).
Might be better to call this something that shows it won't fail with options that aren't ment for sqlite.
Totally forgot!

I knew I left a reminder there for a reason. This is for me to sort out, here an now, I think.
let's simplify the `end` calculation by using ?: operator, something like e.g. `end = Math.min(query > 0 ? query : url.length, anchor > ...`
The docs you are pointing to is only for auth.chi which should use the endpoint '/cgi-ssl/auth.cgi', but for capture the endpoint should be '/cgi-bin/..'
Compare the docs for auth.cgi and capture.cgi and you'll see the difference:
http://tech.dibs.dk/dibs_api/payment_functions/authcgi/
http://tech.dibs.dk/dibs_api/payment_functions/capturecgi/
echo “copying $NGX_OBJS${ngx_dirsep}${dso_binout}${ngx_soext} to $NGX_PREFIX/$NGX_DSO_PATH"
```ruby
def execute_command(cmdline)
  puts cmdline
  system cmdline
end
```

というメソッドを定義して、タスクの定義では以下のように並べる方が可読性高そう。

```ruby
task :watchr do
  execute_command 'haml src/index.haml index.html'
  execute_command 'sass src/style.css style.css'
end
```

ところで、このタスクって watchr じゃなくて compile っていう名前なんでは。
Oh, wild! I've never seen that before.
According to the specs of how to create a IQN, it's supposed to reflect the domain you're running on AND the date the target was created. Not a generic one, nor is it a reflection on 'today'.

But you're (partly) right about the date. In the sence that it needs to be static. I'm working on a patch to instead set this (the whole IQN) in 'shareiscsi', much like the 'sharenfs' property does it. This because once every month, my VBox can no longer find it's targets (because the've been "renamed" - i.e. reshared). So that is defenatly a bug that will have to be adressed.

Either that, or I need to figure out a way to 'store' the first IQN create, and reuse it every time I do a share on the ZVOL...
`saved_layer` appears unused and the docstring is misleading
Didn't think of that, but you can only do that from the API/CLI and would allow you to add packages that might not be real, or exist.  Is that something we want?  Both the ability to add any ole package name you want and to have that sort of inconsistency between UI and API?
You are right, there should be
```ruby
          return nil unless self.class.rpm_command_available?
          return @rpm_full_name || nil unless @rpm_full_name.nil?
```
Did you add the wrong version of the file? This seems to be the tool header.
shouldn't you use the channel available in the record instead of hardcoding a channel in the handler ?
`'use strict';` to enable ECMAScript 5 strict mode here.

Since this Javascript lives in the system app and might grow to and become complicated someday, please rewrite it into `KeyboardManager` object.

(The above can be done in a follow up)
Done in bbd0f8a.
OK, the `80 - sum()` is needed, so scratch this.
@wkerzendorf - the way I see it, there should be one way to do it, which is via the classes. We don't want a functional *and* a class-based way to do things. One could just envisage having a caching=True/False option when initializing the coordinate object for large datasets.
Won't this have the possibility of breaking css on some apps?  We could do `input text textarea` to be more fully backwards compatible.
Don't need that since we use Migrations.
What you wrote here is the duration of the request sending.
Please add unit tests for RemoveNotAllowedCharsInRefName. Maybe move it out of GitRepository, too... I think an extension method on string would be nice...  `label.ToGitTagName()`.
Hi, thanks for the pull. However I'm not sure is it mean that everything will work fine once "reindex" is completed? Normal when is such process take place?

Currently Haxe is disappeared from the languages listed [here](https://github.com/languages). Also new Haxe projects (eg. [this one](https://github.com/andyli/puzzle)) are not indicated as Haxe projects.
I have to update the HideMyAss::HTTP.start method for Ruby 1.9.2 and 1.9.3, but the intention was to take the same arguments as a regular [Net::HTTP.start](http://apidock.com/ruby/v1_9_2_180/Net/HTTP/start/class) call and work [exactly alike](http://ruby-doc.org/stdlib-1.9.3/libdoc/net/http/rdoc/Net/HTTP.html), just behind hidemyass proxies.

And

    proxy = HideMyAss.proxies.sample
    conn = Faraday.new proxy: "http://#{proxy[:host]}:#{proxy[:port]}"

seems straightforward enough to me, I guess it's just part of __*your*__ implementation, and avoiding repetition is left up to you.

Here's an example of how to control the connection properties on a typical Net::HTTP.start (via HideMyAss::HTTP.start)

    HideMyAss.options[:local] = false
    res = HideMyAss::HTTP.start(@uri.host, @uri.port) do |http|
      http.open_timeout = 2
      http.read_timeout = 2
      http.request(@request)
    end

I really like Faraday when it comes to complex middleware, in fact I use it in [some](https://github.com/jassa/the_tv_db/blob/master/lib/the_tv_db/connection.rb) [gems](https://github.com/jassa/the_pirate_bay/blob/master/lib/the_pirate_bay/connection.rb), but I don't want to use both faraday and net/http. In that case, it would be better to switch completely (and support every Faraday method).
can you add \@Beta to these two new methods?  we've expressed some doubt about final form, so adding this is probably correct.   once in, I'm ok merging
Should be Executors.newFixedThreadPool(size).
Sensei's default analyzer is the StandardAnalyzer (I think), so perhaps it's a better default default choice
Yes Mongoid is 1.9.2+ for 3.0.
You can leave out `.keys()` here.
s/Affiliates/Bedrock
if we import frame I would say we should do it globally.
__whimboo__ wrote:
> ... I think having the duratino printed out for the module level should be enough. ...

My vote is to drop these split times:
Because this was only an a misreading, and these times are already stored. 
See comments on [Bugzilla][1]!



[1]: https://bugzilla.mozilla.org/show_bug.cgi?id=767315
With that WSGI warning, I found that webtest lint doesn't like a content-type being returned with no content, as tastypie does with delete.  So this snipped removes it before returning and removed the error we are seeing:

    def delete_detail(self, request, **kwargs):
        res = super(SuiteResource, self).delete_detail(request, **kwargs)
        del(res._headers["content-type"])
        return res

With this addition to SuiteResource, the warning is removed and all the tests pass.  Would you mind poking around with this change to make sure things are still working right?  Since all the tests pass I feel pretty dang confident that this is good.  But worth a check on your end, too.
`asnamePath`?
You are calling that function without a parameter a lot of time, if you don't add some kind of verification here you will create a lot of JS errors. Just make sure that ``inputContainer`` is not ``null`` nor ``undefined``.
what does this specific issuer do?
Use EMBEDDED_MODE from sage.misc.sageinspect, rather than having yet another copy of this variable in sagenb.misc.sageinspect.
btw, here, it should simply be ``{@inheritdoc}`` as the doc is already available in the interface
hmm, we could indeed inject the context directly. But this would force the user to create it explicitly instead of relying on the default instantiation in the router class (not an issue for Symfony or Silex as they are already managing the context explicitly)
Yes rebasingsing !
urlencode already escaped these chars. so the str_replace is redundant.
The indentation here should be fixed.
I don't think, why ?
Should you check the type ?
Fixed

On Mon, Apr 16, 2012 at 6:14 PM, Joe Tessler <
reply@reply.github.com
> wrote:

> > +import java.util.*;
> > +import javax.persistence.*;
> > +
> > +import org.elasticsearch.index.query.QueryBuilders;
> > +import controllers.Application;
> > +import controllers.Messages;
> > +import controllers.Skins;
> > +import controllers.Users;
> > +import play.db.jpa.*;
> > +import play.modules.elasticsearch.*;
> > +import play.modules.elasticsearch.annotations.ElasticSearchIgnore;
> > +import play.modules.elasticsearch.annotations.ElasticSearchable;
> > +import play.modules.elasticsearch.search.SearchResults;
> > +import play.libs.Crypto;
> > +
> > +@ElasticSearchable
>
> Probably want to remove this line, because we don't want to be able to
> search unverified users.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/w7cook/OpenBook/pull/172/files#r692063
>
Just for the record this isn't a decorator :) Just saying.
Remove reference to linear per above comment.
Is there a reason why you prefer to keep this one?  Normalizing here will end up doing either nchannels divides, or nchannels*3 divides if derivatives are needed.  Normalizing in the other spot requires nsamples divides (related to the amount of anisotropy, and skips the divide for the common cases of just 1 or 2 probes.
Genau wie der BaseController, nur hier fuer die Commands. Die anderen Commands muessen aber noch umgestellt werden, dass sie auch den BaseCommand extenden.
better avoid allocations/deallocation via a local variable "ZnmInfo Znmparam;"
i think we can go ahead and move this one to the standard response format
With the change above, you'll need to add "utils." to the class name ...
The property definitions and these attribute names are out of sync, too (_ again).
Could method toArray be on ChangeLog object directly? it could cascade with Version and Feature
We can either bump the MANAGEMENT_API_MINOR_VERSION to 2 and leave MANAGEMENT_API_MICRO_VERSION at 0, or we have to port this fix to the 7.1 branch. We can't leave the 7.1 branch in a situation where we need to fix something and can't use the next number without pulling in undesired other changes.

I vote for just bumping the minor version to 2. It's simpler, and if we end up changing anything else in this subsystem for the 7.2.0 release we'll bump the minor version anyway.
This is because I need to shut it down directly in the subclass
Tests that need to restart the server must be in the manualmode module, not integration/basic. Restarting the server will break later tests as arqillian looses its connection
Yeah, that's mapped properly. targetElem is the event object; `target` is a jQuery object, `targetElem` is the first item of `target`. `$( obj )[0] === obj`; there's even proof in the conditional. I probably wrote the original code, I'm not sure why I would've switched between the conditional and the assignments.
In dialog we use `&#160;`. The result should be same as `&nbsp;`, not sure about the reason for one or the other.
A `WebRequest::getJSON` method sounds great. Make sure you allow for a different default (some usage might want to default to something like `{}` in case of invalid, others might want to get PHP boolean `false` and abort instead of silently failing).

Content validation, I'd put it a public static method of `ResultAction`.
Hi @jmcanterafonseca 

is there any way of decoupling the settings from fb? if not, should we add this listeners after checking if fb is enabled or not?

Thanks!
I only send the first blob for the share request. It means that Bluetooth AP only provide sending one file for each sharing action now. I think that I should set a filter to check the "number" with one item. Then, I will file another issue to have a discussion with UX for the multiple files sharing by Bluetooth.
please drop the dead code
`reqBt` -> `bluetoothReq`
I remember that I have done this before...so strange. :(
No problem on my side. The point it that both actions can be whatever the code invoking ConfirmDialog.show() wants them to be. For example, the developer could do something like this:
ConfirmDialog.show(
   "Title",
    "Message",
    {
         title: "Confirm",
         callback: whatever function
    },
    {
         title: "Cancel",
         callback: whatever function
    }
);
and consequently in this case action1 would be the Confirm button and action2 the Cancel one, and maybe this could be even more disturbing :-)

As I said, if you prefer us to use actionCancel instead of action1 and actionConfirm instead of action2, just say: "Yes, I do" to this comment and I'll include it :-p

Anyhow, I will ping you tomorrow via Skype to make a decision on this :-)

Thanks!
What about using the same object {type} formatting for the window_close message? You will be able to just switch(data.type) { }
Let’s keep this paragraph-in-list please.

In this app, most list items have one text + one input, so all texts are included in a `<a>`, `<span>` or `<small>` tag; and when we need some multi-line text, a paragraph-in-list is used (and defined in the Settings stylesheet).
May I know why this is "all" instead of just "transform"?
Thanks.
What about using encodeURIComponent on this side and decodeURIComponent on the other side? I think it will help with urls too.
Indeed, this is fixed now
It's actually called ahs-plugin-defalt-face
http://www.eecs.harvard.edu/~michaelm/CS222/countmin.pdf

I see now. The paper has this and other query methods.
True, but notice that my fix does not directly address the reg exp issue for one specific use case (punctuation at end of input string), but rather the whole issue that words() should never return an empty string as part of the array. The fix handles the use case that we don't know about yet too.

That said, matching on \w+ in addition to the filter() rather than as an alternative to the filter() might be even better.
Space these globals properly and document them.
It should be `$.off`ed only when we're sure it can be safely be `$.off`ed. Currently it won't work after a connection error anymore.
This appears to only be used in one place?
This seems to be missing `basedir`
This should use `self.build.path_module.join`
This loop may probably lead to the issue we had here:

https://github.com/chaplinjs/chaplin/pull/315
I always saw the variable at the right for comparaisons on symfony code but didn't know why, that makes sense now! :)
whitespace formatting in this block is inconsistent
already exists in master
@NARKOZ Thanks for noticing. Fixed.
@vsizov All tests were passed
> each your request contains a bug
not true
refactor all this code to
self.questions.sum{|q| q.abc(user_id)}

* .questions OR .questions.not_mark_as_deleted
As we discussed on vydio, this is out of the scope of this pull.
After some thinking about it I realized that things handled by that code will be required in scenario with a new manifest format as manifest consists of relative paths. I really don't want to run into case where we land simplified version to FF and then we'll have to workaround limitations until we'll land a followup patch.

I'll try to comment more to make it easier to follow that code.
Handle the xhr failure.
why did this make it into the database layer?
via shane - never supposed to have a return value - code cleanup.
@lloyd - is "debug_lang" supposed to be defined in config/production.json?  What happens when somebody translates Swiss-Italian?
Yes, I saw that issue yesterday.  Thanks, I will update this branch and put the whitelist on the client side.
Seems like we miss one thing. How does the build script behave if we supply a build version? Would it also execute the build task because its a dependency? Or would it be skipped?
I might just be missing it, but do you ever call this method?
Nope: open in append mode and you can't write before the end of file.  Feel free to give it a try if you want to confirm that for yourself.  These open modes are actually pretty terrible for doing exactly what I want, but nothing else is portable. :(
extracted a method.
I think 3 reporters use this now, so we should put it in the utils file
Good point. I started that way, but `getElementsByClassName()` does not return an array, but an array-like structure. When you start iterating this, it brokes at the half. After several tries, recursion behaved better as a solution.
I was pick up the wrong syntax :p 
This PR can be close, I was just to highlight a 404..
Is this comment correct? Shouldn't it be the selected menu item count?
phpstorm? or just avoiding use cases at the top? or none of them?
Yes. 
Since the reset column function can be applied to any search page, this method can be override by any search manager. But it makes more sense to implement this method, and instead, add a "setDefaultSortColumn" method to set the default sort by search managers.
These execs are very dangerous. The proper way is to stop writes on the master either by a global write lock or some other way. I prefer to not have these things done by puppet as they are typically done once only anyway.
I found that sibling.data was a more reliable method for getting the content of the comment - particular with IE8

```
jqLite('<!-- /ng-repeat -->').data === '/ng-repeat'
```

Since you are checking the node type anyway you could simplify the regex and match logic.
Wrote a better sql query to delete the first duplicate, I've checked and sessions always have one duplicate, seems like a double callback or something.
I don't know if this is the most reliable method for retrieving the zoom level, but it works on iPhone 4S and iPad 2.
@vesak Pilkunviilausta, mutta tässä ois kiva jos minimierästä puhuttais vain jos $trow[minimi_era] > 0
@harjis "$automaaginen" takoittaa, että "Toimita selkeät rivit automaattisesti". Sitä muttujaa ei siis voi käyttää kun haluat päätellä poimitaanko rivejä käsin vai kutsutaanko jt-poimintaa skriptistä.
This phpdoc is still wrong
Philosophical question. I just find typing `Enum.t` easier. 

If we think about it... Enum.Iterator is a *protocol*, what it works on is "an Enum". Thoughts?
Its not really the query's string representation, its Solr's query format
Maybe hoist this into a small function, something like:

```python
def encode_class_name(name):
  return EBCLASSPREFIX + encode_string(name)
```

That way, we have a single point of change if we ever want to tweak it for whatever reason, instead of two (one for applications, one for packages).

Also then, I guess we do not longer need the EBCLASSPREFIX global constant. Just hardcoding that in the `encode_class_name` function is OK imho.
Why not equals?
In most cases pig will give them an alias, but in some cases it won't. The most common example is if a UDF doesn't give it a name (which I could change, but is how it is currently)
re can't match `/mongoskin/-/mongoskin-0.4.0.tgz`
This test doesn't seem to test anything.
No trailing white space please :)
Please use double quotes here instead of `\'`: http://mozweb.readthedocs.org/en/latest/coding.html#whitespace-matters
This assumes `navigator.id.getVerifiedEmail` will always be successful. If a user closes the popup without completing the login, for example, `assertion` is null.
sounds like a bug ought to be filed against persona docs.
all three of these need to be: `function(` (no space before the first paren)
```js
var length = arguments.length, 
    eventType = "trackadded",
    options, trackEvent, sec;
```
We should add an assert here to check that the section has expended. We might want to do this click in a separate method since this section is used in other tests.
This change was necessary and I need to illustrate to everyone, we CAN NOT embed python objects into granules, don't try it we're not there yet. The decoder can't handle it because of conflicts with `msgpack` and `numpy`.  I tried writing a decoder but the problem runs deeper than I thought. *Granules should be monolithic and not nested.*
semi-colon please :)
We should probably test this change, too...
It's a she!
When would you use the "no solution, pass anything without errors" option? Seems like any question of that type could be answered by just deleting all the code.
`.*images`?
Having an interface and 2 implementations would be the cleaner design, once we agree on the input/output types of the interface methods. This is just a draft atm
I cleaned this up with my latest commit.
Also we will not under any circumstance "pull" PRs that remove features unless its for a very good reason. That is completely backwards logic.
blah blah blah6
@iwada :  What did you want to point out?
Are you really sure this is right? STATIC_URL is for things like your .css files. MEDIA_URL is for things like files uploaded by users. 

Django 1.4 makes these things clearer. Also the documentation in 1.4 has improved. 

I would be very very careful with "messing" with these two. Especially since it might negatively affect people on django 1.4.
@bryanwb can you undo this change (or I can). This will break existing installs and force a restart.
Whitespace/Indentation
I cannot find any reference for `urllib.splitquery()` in the documentation: http://docs.python.org/2/library/urllib.html. Where have you found this method? Is that frozen and allowed to use by external code?
Should log the value of `rel` as well
I believe this is unused.
This can probably be simplified quite a bit as well. IO.loadJSONConfig should check whether the file exists, probably not this file. This can probably also be a synchronous read (as it would only happen on start-up), making the recursion not necessary.
As discussed during the daily stand-up we'll hold off on adding a validation framework until the UI has been brought in. There are some validation methods in our 3akai-ux API's that we can leverage.
Should split this out into a `it'(verify validation create reply')` test
Unless I'm misreading this function, it looks like this function is not checking the role/permissions that is directly associated to the principalUuid, only for its member groups. This seems to be reflected in the unit test as well.
Does ``index`` really need to be a ``var``? I'm asking because turning it into a ``val`` seem to result in no compilation errors...
``Nil``?
Sorry my bad, the references module has two sub-modules one for node_reference and the other user_reference. These are the functional equivalent of the CCK modules in D6. So the dependency is named node_reference on d.o it should be downloaded from the drupal.org/project/reference
а навіщо тут 'php' йде другим аргументом?
I mean ca:
I'll make a PR for this :)
I don't know if this is safe to treat a url like a file. I recall being burned once with that, but I don't remember the exact scenario. Best would be to at least get rid of the protocol part of the url.
Ik zou deze regel dus neerzetten bij wherever je de code verandert naar 200.
Ik zou dit handlen met collision types. Ik zou het ook neit doen in deze branch. gewoon nieuwe branch voor maken en meteen de fix shippen
`$php_error` is (supposed to be) always `TRUE` or `FALSE`. So this `if` statement is pointless.
Again, just 1 tab.
On 731 and 737, the program flow as I read it will be inside of a case where the originating MIME type is 'text/html'.  The protected method picks up the $type parameter to determine whether or not to execute the code that strips the <script> and <style> tags.

My understanding of how the main method works is that CSS and JS in a 'text/html' doc will be encapsulated in script and style tags, but CSS and JS in a text/css or text/javascript MIME type document would not have those tags in them.

I just carried the MIME type from the minify() method into the protected function to determine if tags needed to be dealt with because I did not want to strip spaces inside the tags, for concern that properties in the tags would get messed up.
Не очень оптимальный, хоть и рабочий, метод мутирования.
So this needs to be an interface with an implementation?
Ну, я и сам пока что не сильно тороплюсь. Проект поди от нас никуда не убежит, а сейчас опять началась эта солянка из учёбы и работы, хочется уже поскорее от кого-нибудь из них отделаться)
There was a discussion about this, and we decided to put the sprint number as the second qualifier for now. We thought about having the sprint number be the first qualifier (and eliminate the concept of "major" versions), but that would limit our flexibility for defining API-breaking changes. We can always adjust our numbering, if needed, when we get to 1.0.0.0.
this spec is empty :smile:
it should only set title for Gadgets as it's using Portlet wrapper for displaying
http://stackoverflow.com/q/5813311 :)
... and the string is otdated I guess. As even with random state set to zero it seems not be 100% predictable and we now have the score func we don't have to tell where we expect it..
Yes, that's also a good option. However, then we need to modify the info
passed to compute the whitener, at least with the code we have there.
Otherwise the compute whitening procedure will try to remove the bads from
the data that have already been removed.....


On Wed, Dec 5, 2012 at 1:42 PM, Alexandre Gramfort <notifications@github.com
> wrote:

> In mne/preprocessing/ica.py:
>
> > @@ -743,9 +743,12 @@ def _pre_whiten(self, data, info, picks):
> >              data *= pre_whitener
> >          else:  # pick cov
> >              ncov = deepcopy(self.noise_cov)
> > -            if ncov.ch_names != self.ch_names:
> > -                ncov['data'] = ncov.data[picks][:, picks]
> > -            assert data.shape[0] == ncov.data.shape[0]
> > +            if ncov['names'] != self.ch_names:
> > +                ncov['data'] = ncov['data'][picks][:, picks]
> > +            assert data.shape[0] == ncov['data'].shape[0]
> > +            if info['bads'] != ncov['bads']:
>
> it's common not to mark bad channels in the noise cov and to rely on the
> data bads to remove the proper lines and columns in the noise cov but based
> on channel names. I think it would be much more robust. Basically I would
> take from the noise cov the channels present in the data and raise an error
> if one data channel in not present in the noise cov.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/222/files#r2319840>.
>
>
I'm trying to work with this one. What about "The system administrator can't be blind to writing some code." ?
Is this still working correctly?
Can the tests use `matrix.toArray()` rather than accessing the data property which, even though it's public for now, should be seen as an internal detail.
typo: ``to send``
模块请放后面
duh!! Sigh
braces superfluous
pas de deux points après les tags @. et mettre à jour le cartouche du début du fichier. respecter le coding style http://developer.jelix.org/wiki/fr/conventions
Que bolu!.  Pense que no entraba ese commit. Perdón.  Como me conviene sacarlo?
Doesn't this just return the whole list of subscriptions as one big text blob?

What you want to do is return it as a list of strings, not one big string.  You cheat here because your test only has one subscription, but we want this function to do what it says and be useful elsewhere.
Ah, then I'd suggest we drop `+` for composing strings. It's slow and doesn't read well. Interpolation with `#{}` is the fastest.
Is the interpretation required here?
This probably won't work as expected. you do not know the rest of the where condition in here.
good because i like the more detailed message :)
My fault but I think with the find_all and find_by methods no longer being overridden this import isn't needed
Ympäröivä `if` on tarpeeton. Pelkkä `(and (empty? (rest coll)) (not (empty? coll)))` tuottaa saman totuusarvon.
In the first loop we can 'see' the same segment multiple times and we want them to be unique to minimize the number of iterations in second loop. I can't think of how uniqueness would work efficiently with an array. Not sure I understand your point though. Could you clarify?
the year should be ``2012`` :)
This is where the magic happens. Credit goes to @tvdeyen.
Not necessarily out of scope for PEP8.  PEP8 is all about readability and making sure you have maintainable code.  However, I don't know this code enough to understand the distinctions between these variables.  However, I would be willing to file a bug report instead.
@WeatherGod Do you mean that you would put the "for" on the first line?  Regardless of what PEP 8 says or doesn't say, that would look wrong to me; "for...artists" forms a phrase that should not be split.
For those of us new to this function, can you explain what this does?
Nothing needed here in order to merge in my opinion
Is it possible to cache the instance of the JsonMarhsaller (in a static variable)?
You missed a safeUserInput here
extracted into super-interface from SolrSearchResultProcessor to separate render from search
@iglesias yeah postpone for later but better file a bug report such that this is not forgotten.
setModifiers() does the array_map as well, so it probably isn't needed here.
Did you want to have a function arg here?
Nenastane tahle možnost náhodou spíš obecně když bude timer být umístěn na poslední místo v seznamu (tedy když je seznam prázdný, zároveň na první). Fungovat by to teda mělo tak jako tak, jen ten komentář mi přijde zavádějící. Anebo to špatně chápu...
Foreign key does not have end with _id, I'm wondering if we really need this second condition.
I'm not sure why that was commented out. It came from an external patch. It might be worth asking upstream
The copyright here conflicts with the one in L10 above - https://github.com/ushahidi/Ushahidi_Web/pull/958/files#L48R10

Regarding the `@package` tag we should update all references to `source.ushahididev.com` so that they read `https://github.com/ushahidi/Ushahidi_Web`

The following perl oneliner can achieve this:

`find application -name '*.php' -print0 | xargs -0 perl -pi -e 's/http:\/\/source.ushahididev.com/https:\/\/github.com\/ushahidi\/Ushahidi_Web/g'`
One thing though. `if ((isset($qm_ary[1])) ? $qm_ary[1] : true)` should default to `false` rather than `true`, if there is no permission check _(for what ever reason)_ the assignment should be skipped rather than setting it.
This is handled as `$apple[0].attribs.data` MINUS `test`. The result should be `NaN`.
fyi: we just had a major fuckup as the hdds of our ec2 instances were out of space. I will add an options, which enables one to keep the downloaded files. also i will add an option for a limit of max used tmp space.
@philfreo, definitely! Also, I would recommend adding those to the code examples throughout the document. Wherever you have
`ExampleCollection.on('change' ...)`, change it to `ExampleCollection.on('change' function(model, value, options) {...})`.

There are a few code examples like that scattered throughout.
It would be more coherent if, when possible, we keep all short form options in lower case (-a instead of -A).
Based on that definition it's not entirely clear if the close should close the connection nor the result, but every example I've seen seems to leave that up to the caller. 
I will continue to null out the PrepStmt internal objects.
Also, calling any of the execute methods on a close prepstmt will cause an exception w/ an explicit message.
CMS error is this:

    JModel::getName(): Can't get or parse class name.
Again, as @dianaprajescu said, this prevents it from decoding URL's. Both tests that I added on lines 620-630 would fail in the current version of this function. This causes *significant* problems because this function is used in JURI to help explode URL's for things like HttpTransports.

In my use case, JHttp::get('https://www.googleapis.com/calendar/v3/calendars/me%40aaronschmitz.com/events?singleEvents=true&orderBy=startTime&timeMax=2012-09-01T00%3A00%3A00%2B02%3A00&timeMin=2012-08-01T00%3A00%3A00%2B02%3A00') turned into a request for https://www.googleapis.com/calendar/v3/calendars/me%40aaronschmitz.com/events?singleEvents=true&orderBy=startTime&timeMax=2012-09-01T00:00:00 02:00&timeMin=2012-08-01T00:00:00 02:00 which perhaps unsurprisingly failed.
Nitpicky I know, but would you just remove this line instead of commenting it out.  Everything looks like it passes so we will get it merged when you clear that out.  Thanks a bunch!
i have no problems merging the above two scripts (just in case they were in a different pull request ;))
Really interesting way to put context back into the view. Really crazy ideas. :-) (In a positive way. Sometimes it is interesting to see such solutions because I would never imagine them because they are so out of the "right way". But I can see that this could maybe be of use someday. But for current problem this is really a hammer to simple problem of including template.
We are not doing beautiful template output (because it is impossible to keep track of all indentation there), but beautiful (and readable) templates.
OK, other things now all look good. But this we will still have to improve. ;-) So this is still not done correctly. Again: we want that no error is displayed until user enters something in both password input fields and leave (blur) them both. Then, after that, we want that fields are checked on any change/keystroke.

So I would propose you leave here only `blur` handler. This is when we want first checks to be done. Only on blur. And then, in `checkPassword`, I would attach to change/keyup event first time check fails, after `$("#id_password2").after(message_info);` for example.
The function was initially outside the Post. Maybe it would be better this way so we can reuse it for post comments?
you call replace on whole string, it is not needed. You can do
```
aliases[u]  ="update-" +  String::Replace (p["display_name"]:p["name"]:"repo", " ", "-");
could you change this to `dataSet = +dataSet` looks good otherwise.
why are we removing the title?

Also, you'll need to add a migration using South, now that we're using South.

merge from master to get south.

Issue ```./manage.py schemamigration dashboard --auto```

This will make a database migration. It will prompt you, asking what do to if we do a backwards migration. I think you can safely pick the third option (don't allow backwards migrations).

Then commit the new migration file.
I think you're confusing /* */ and /** */
I don't think the OgoneResponse has enough code to justify its own file; instead just embed the class in this one.
Could you prettify this please?
no dreamweaver javascript
make this class home
Sean needed to play someone besides X-2021...

Feel free to put your stamp on the flavor text and such. Not sure I got the tone right.
Are all the `snippetResult` params in this class being set to nil because you don't plan to implement snippet results with this ticket? I'm assuming that is the case and I'm just validating. :