labofsahil/github-event-dataset-2012 huggingface dataset:
True. I guess I assume the user is has run "ipython --pylab" and "from scipy.signal import *" here. However, just adding the imports will not make the example run (ipython --pylab does some threading magic). Can we assume an "ipython --pylab" environment? If not, this is what's needed:

from scipy.signal import *
import matplotlib.pylab as plt
w, mag, phase = bode(([1], [1, 1]))
plt.semilogx(w, mag)
plt.show()   # close plot to continue
plt.figure()
plt.semilogx(w, phase)
plt.show()   # close plot to continue

It's a bit more verbose than what I was aiming for. Maybe I could put both plots in one figure, but I'm not sure it will be easier to read. I was aiming for a short and simple example... hm... what do you think?
Line should be wrapped to <80 characters.
This is not a header though... it's just information.
nit: (uint32_t) FILE_HARD_SANITY_LIMIT
Does the function actually distribute any factor?  If so, I'd recommend writing it like `Turn a * [X, Y] into [a * X, a * Y]`.
s/Remark/We remark/ or s/Remark/Note/ I think.
Don't use relative imports, please.
There is the speed matter - http://hakre.wordpress.com/2010/05/13/php-casting-vs-intval/
same check is done in __call in CComponent: https://github.com/yiisoft/yii/blob/master/framework/base/CComponent.php#L264
@samdark  I dont  understand - there is an example already few lines above:

@var string name of foo {nillable = 1, minOccurs=0, maxOccurs = 2}

What else example should be there .. ?
but the current url is Theme,user,disableMobileTheme
I'll try it and see, but name is supposed to be more consistent across versions, supposedly.
This can be moved to the PrincipalBasic.email validator, i.e. ``validator=colander.All(deferred_email_validator, colander.Email)``
'Permission denied' is an `IOError`.
thread.links_as_read.append(link)
An alternative:

/**
 * @return RegisterService   // Or what ever Service it is.... (For syntax highlighing)
 */
public function simulate()
{
    return new ServiceSimulator($this);
}

===========================================
===========================================
class ServiceSimulator
{
    private $service;

    public function __constructor($service)
    {    $this->service = $service;    }

    public function __call($method, $args)
    {
         ServiceBase::$simulated = true;
         call_user_func_array(array($this->service, $method), $args);
         ServiceBase::$simulated = false;
    }
}
I think the delete is because you then pass the options through to active record, which validates that all the keys in the options hash are valid for the method you're calling.
Раскомментить и использовать TestApp
Force pushed.
:LINIX => :LINUX
Once again, list manager.
well
they appear

I'll check if they actually go into the database
I'm tempted just to limit the size of the upload, and force users into doing partitioning if they exceed it. This could eliminate the need to doing streaming, and encourage larger users to use a more bandwidth and processing considerate upload technique.
journaling stores raw operations that allow MongoDB to reconstruct the following operations: document insertion/updates and index modifications, and changes to the namespace files: we need to find away that describes the data as the user experiences them and also what it actually reflects (it took me a long time to figure out that metadata = ns files.) 

journaling stores journals?
It is. We require vendor/name just as we do for extensions so we can get titania & composer integration.
Why does this need to become public? Can't _js_include just prefix it's parameter with $this->phpbb_root_path?
How is forum_topics_real determined now?
Not sure, wait for a dev to post.
@return void and shouldn't that be added elsewhere in the diff also?
They should each go in their own file then. We store one class per file. So the name of this file should be phpbb_database_connection_odbc_pdo_wrapper.php
camelCase is the "official" term (recognized by wikipedia). camelCaps is one of a bunch of synonyms: http://en.wikipedia.org/wiki/CamelCase#Variations_and_synonyms
I prefer to use the word camelCase because I've heard it before, whereas camelCaps just sounds weird. But since i don't read the coding guidelines very often, I don't much care.
use the name, Luke, not the full module path.
do we need `method_missing` here? i like to limit `method_missing` to situations where you don't know which methods are going to be called on the object. here it seems like authentication types are registered beforehand. so either i missed something or authentication methods could be defined as soon as they get registered.
Not sure that this is needed to be in OpenGLWindow.h -> included by other files. It is needed only by OpenGLWindow.cpp implementation.
Could you tell me the motivation to use the Delegator instance?  Could this have been accomplished with a Shoes.backend for logger -- Shoes::<Framework>::Logger.  Maybe Shoes::Logger by default uses Ruby::Logger if none is defined by <Framework>?
Would you mind to replace all `[].forEach` with `tools.forEach` please?
Darn(er) you're right. It makes copies of ``buffer`` but not the string. Well, try with strlen then. Btw, you could templetize it and handle the special case of the static ``char []`` to get a compile time ``sizeof`` instead of the runtime ``strlen`` ;)
no, john sent me a file and according to that i created this json file.
will this test still work when we change our program to use REST instead of local JSON?
no need for these, just add into existing moduleTestData file
> Why would I want a space after a carriage return?

That was my question ... which I obviously failed to make clear. ;-)
In this case, the current syntax turns out to be fastest.
escpaing
? There is choices
What for? Am I missing something here?
long and descriptive is better than cryptic names, but I would prefer a name like `matchCount` - FWIW.
Is it really necessary to also rescue StandardError? If only recovering from java.lang.ClassNotFoundException is enough, we should only rescue that one probably.
I just added a commit replacing every occurencies of fetchMode instead fetchStyle
Works fine when I used this model for config in hasjob.
Seems like a good change, but it might be clearer to outsiders (and/or me) if the conditional was still in this location (but just used the PgUtils.addon_name call).  That way I don't have to be confused why this is always called only to find out it usually isn't.  Mind making that change?  Also open to discussing this.  Thanks!
List the expected test outcome (fail/pass)
@devcritter see Hui's comment in the code here. You can probably answer this better than me.
I believe whatever it was that I thought I recalled was caused by another issue. Thanks for pushing on this one, patch to follow
If we don't need this, simply remove it.
Nit: Since we're still inside the parentheses here, 'relaxng_...' should be lined up underneath os.path.dirname...
ENV vars should be set in a separate step, `ENV['LFLAGS'] = ...` and this should be done as a list:

```rb
system "./configure", "...", "..."
```
md5 '75a1720be83b1d787ba465e6bdabcf42'
So it does not overwrite it if it is already there.
The sources are not available to `brew test`; this should be a runtime test of the software, if possible. Grep through existing formulae for examples.
Is there a reason you're not using super() to call the super class constructor?
I think this line should be

```scala
(isEmpty(f1) && isEmpty(f2)) == isEmpty(plus(f1, f2))
```
I don't think you need @ Override to get this behavior . . .
sorry, I was trying Bojan dev and forgot to remove this line
I'm just a bit picky :) but ... milestone or rather warning?
破折号和书名号之间不必空格吧。
@marcphilipp 
What I mean is(...) is better and more simple, typically narrative code.
Bug fix: jQuery event parameters must be given as an array or bad things happen!
I think this function is a no-op -- I don't see any other references to "color_swatch", so I think no such element exists.
Good catch! I was not thinking carefully when I copied it from an existing one.
We already have the logic for resizeStart, so calling requestAnimationFrame there for the first time should be easey and seems like a nice final cleaning touch
haha, that makes two of us. I had to think about it too.

I started with the comment, but then thought renaming the variable to "fileInfo" was more concise and removed the need for a comment.
OK, then I will have to add break for each case.
Seems like this should be combined merged into the above block -- endQuote couldn't become set any other way, and I don't see any case where hasEndQuote would be true but endQuote would be falsy...
Any particular reason we're using `position: absolute` for this instead of doing a flow-based layout with floats (or inline-block)?
The circular dependency should be ok but I thought I'd mention it in case you had any concerns.
We currently only provide completions of the content of the file `css.json`
why i became global?
I do not see the documentation claiming that enid_mutex_create will fail if two threads create mutex with same name.  Is that what this code is depending upon?
I don't think there's any... I couldn't figure it out at least. Here's the source: http://www.nicprice.net/diptest/Hartigan_1985_ApplStat.pdf
Plus, if you use the HashSet, you won't need to use Contains. HashSet.Add will return whether it was successful, though you won't need the return value.
Why not also the original string *e-mail*?
Then don't put home there. ;-)

https://docs.djangoproject.com/en/dev/topics/http/urls/#django-core-urlresolvers-utility-functions
space between if and (. `if (`
done & good catch!:D
help i'm drowning in whitespace
And, by `test` dir, I mean `templates` dir.
store reference to projects.length
again, needs to be handlePositions
Neither does this.
Don't need this. Can be button styles only.
Add a `this.off( ... )` call here too so that the normal case doesn't leak.
You're using this function before you define it here
Ganz gekippt ist das Thema noch nicht (zumindest im Core), aber ich glaube, die Textile-Version in 4.4.1 läuft auch schon nur mit PHP > 5.0. Daher wäre das ok, denke ich.
I think this would be a little better as:

```ruby
Then /^I should see the current RDS security groups includes "(.*)"$/ do |sg_name|
  page.should have_css('.rds_sg_name', :text => sg_name)
end
```

(Includes the ID/class change)

There's no particular need to make this specific.
Please restore
I would reduce it to:
```ruby
if defined?(Bundler)
  Bundler.require(:default, Rails.env)
  Bundler.require(:foreman) if Katello::BootUtil.katello? # require backend engines only if in katello/cfse mode
end
```
Updating with commit 6e481e64be63a0482858f508b57fec4e820ee6ae .
at first I thought this was unnecessary but it looks like you are only saving if the pools need to be updated, which is fine!
it has to be directory always? could not end with foo.cgi?
is it good approach to consider success if the error_code was not found?
`inspect` appears to be unused.
Removed

On Mon, Sep 24, 2012 at 10:53 AM, Paul A Martel <notifications@github.com>wrote:

> In tests/frontend/org/voltdb/regressionsuites/TestUnionSuite.java:
>
> > +    private void load(Client client)
> > +    throws NoConnectionsException, IOException, InterruptedException {
> > +    }
> > +
> > +    static public junit.framework.Test suite() {
> > +        VoltServerConfig config = null;
> > +        MultiConfigSuiteBuilder builder = new MultiConfigSuiteBuilder(
> > +                TestUnionSuite.class);
> > +        VoltProjectBuilder project = new VoltProjectBuilder();
> > +
> > +        project.addSchema(TestUnionSuite.class.getResource("testunion-ddl.sql"));
> > +        project.addPartitionInfo("A", "PKEY");
> > +        project.addStmtProcedure("InsertA", "INSERT INTO A VALUES(?, ?);");
> > +        project.addStmtProcedure("InsertB", "INSERT INTO B VALUES(?, ?);");
> > +        project.addStmtProcedure("InsertC", "INSERT INTO C VALUES(?, ?);");
> > +        project.addStmtProcedure("UnionABC", "SELECT PKEY FROM A UNION SELECT I FROM B UNION SELECT I FROM C;");
>
> Yes. Might as well remove it.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/alexeevm/voltdb/pull/3/files#r1673760>.
>
>
this should be ``from ... import table``, right?
I think it might be worth showing an even simpler example here first, e.g. ``u.pc.to(u.m, 1)``
Why `spawn_link` a function that is going to immediately exit with an error?
Why change this to a pointer instead of a reference (&)? IMO `&(*it).second;` is slightly more ugly code, and in C++ it's generally advised to use references instead of pointers where possible (for type safety etc).
`$data` should equal `''` not `null`
Yay, no more linkify!
The thought here is that if we are assuming that the lowest common denominator of message serialization is JSON, then we need to special case raw binary for efficiency - yes, we could send it as a string in JSON, but it would be better in base64 or something similar so there's no issue with escaping JSON special characters or problems with unicode, which I suspect will be very inefficient.

However, in cases where we have a richer pipe we'd like to be able to just hand the object across with no further work.  It's the pipe which knows its capabilities, hence its the pipe we ask to do the appropriate conversion.

Serialization is a tricky topic when you are dealing with potentially non-Python clients, but even javascript now has homogeneous array objects on most bowsers, so we want some sort of binary data support.

If we standardised on a serialization format a bit more capable than JSON as the lowest common denominator (eg. some of the binary JSON-ish formats), then this becomes less of an issue.
no, put the real docstring back too :)
It's a pity we're losing history here, i guess because 'git mv' was not used?
Did you test this code on a windows client?  While SSH on windows is not common, SSH *clients* aren't totally unseen.  In particular, I'm wondering if this shouldn't be done with `os.sep` instead of hard-coding `/`...
What do the `str()` calls here solve? They look redundant.
That is a good question...I would imagine so since it's using perl, and will setup a whole new perl environment within that seperate binary?
Will this break code for anyone who has used `integer('foo', true)`.   Granted no one should because the `increments()` method serves that purpose, but since the `integer` method is public it is possible someone has used it that way.  It looks like this can be achieved without changing the order of the parameters.
Женек, блин, даже не смешно :)
:+1:

That's what Microsoft does on the Windows API. \*shudder\*
Well, I'm not sure. I think "Click to hide this topic" is misleading, it's like hide everything on this topic, and that's not what a click on the button do.

I have thought about it and I cant come up with a better phrase.
List addition appears to create a new list:

```
Python 2.7.1 (r271:86832, Jun  6 2011, 14:29:03) 
[GCC 4.2.1 (Apple Inc. build 5666) (dot 3)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> a = [1, 2, 3]
>>> b = a
>>> c = [4, 5, 6] + b
>>> a
[1, 2, 3] 
```
__xabolcs__ wrote:
> And what shoud `saveScreenshot` do if `asyncCopy` fails?

I mean `asyncFetch`...
Why are you keeping those commented pieces of code?
Same thing: `if not params.datatype:`
French!!!
Yep you reverted valid changes here too.
should be `\-` in the middle ?
should also be fixed in other test names
AFAIK it can be anything, so also null and string etc.
Should this be done *after* the vifs have been moved?
Seems like a lot of logic for the view.

Also, image already handles the 'no_photo.png' by displaying the placeholder (which we want to do) here:
https://github.com/yammer/sched.do/blob/master/app/models/user.rb#L52
Tap returns the variable the message was sent to. you could also just make the last line of each method `self`. I think tap is a little cleaner. I did this to keep with the fluent nature of the existing code.
基础方法，draggable中会引用到
Yeah, I was just trying to differentiate it from the old `Check` interface, but it was a poort name choice. I'll go with `AgentCheck` for now.
Remove reference to linear.
Extract = good, instance = bad?
What if they change the instance?
PatchCollections will be drawn before Features with the same zorder.
It does, but it's less visible when a dependency is changed (intentionally
or unintentionally), and it's harder to control the versions being used. It
should also be faster and in rare cases, you can get very long dependency
resolution times.

On 14 November 2012 16:51, Jake Benilov <notifications@github.com> wrote:

> In Gemfile:
>
> > @@ -16,7 +16,8 @@ gem 'gds-sso', '2.1.0'
> >  gem 'jquery-rails'
> >  gem 'plek', '0.5.0'
> >  gem 'zendesk_api'
> > -gem 'less-rails-bootstrap'
> > +gem 'active_attr'
>
> That's fine, although doesn't the Gemfile.lock file do this already?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/alphagov/support/pull/7/files#r2126537>.
>
>
would be nice if there were some doc strings describing what this does and how it's meant to be used
I not sure that I understand why we are using "i" tags here. A "span" tag would be more semantically correct here, I think.
This generates an Uninitialized variable notice for $data. We should avoid notices.
do we need to be nulling out the callbacks here to prevent leaks?
```javascript
  tryable == null;
  finalizer == null;
```
Huh? This is not an el.
this is why the data list checkboxes stopped working, too much of a good thing( preventdefault)
Sensei,

Ouep en fait à la maison, J'ai pour habitude de tout faire à l'éditeur de texte, etc... pour l'exercice...

Mais c'est étonnant, je n'ai pas ce rendu chez moi, je vais creuser voir ce qui cloche !

F.
Running comparison I see this. I don't see anything explaining what these module options are in the xsd?

--- Problems for relative address to root ["security-domain" => "*"]:
Missing attributes in current: []; missing in legacy [module-options]
Missing parameters for operation 'add' in current: []; missing in legacy [module-options]
Better use File.separator instead of front slash here. I know the previous version was (is) using the front slash, but now that you are changing something in this area, I guess it's better to use the File.separator
null 表示一个初始状态，表示其实你还没有设置这个值，而不是每个值都需要一个初始值。这个地方的到期日期本来就是需要人在页面进行设定咧，所以初始值自然就不能有值，就想你现在搞一个时间，如果这个用户一直都不得来使用，你就不晓得他是啊个时间到期咧，还是就根本不得设定，除非你有其他字段来表示。

--  
zhoudi
已使用 Sparrow (http://www.sparrowmailapp.com/?sig)


在 2012年11月24日星期六，下午3:41，longhengyu 写道：

> In grails-app/domain/com/pkgplan/auth/User.groovy:
> > @@ -9,8 +9,8 @@ class User { > String username > String password > String email > - Date dateCreated > - Date dateExpired = new Date() > + Date dateCreated = new Date() > + Date dateExpired  
> 我觉得有值比不得值好。判断是否有效的时候，代码更直接更可读，而且少判断一次。有效的条件就是 expiredData > new Data()
> 如果可以为空，你就要多判断一次。条件1 && 条件2，这个肯定不好。  
>  
> —
> Reply to this email directly or view it on GitHub (https://github.com/longhengyu/dream/pull/5/files#r2214944).  
>  
>  
>
list -> getInstalledApplications

Exceptions does not really make sense here, remove it from here and do this loop next to where is called getInstalledApplications.
Just following the wireframes :) HTML5_SMS_20120810_R2S1_V7.0 p6

Didn't think about localizing separators too
`#quick-settings > *[data-initializing] {`
Now that you're using :active in the css file, do you still need these event handlers here?
because 'selectedVideo' is set to null in deleteFile() function already. Or I just simply don't understand something?
Nope is for nothing, probably a debug option that wasn't completely removed
name of the function is typo sensitive and does not reflect its purpose. I suggest calling it buildMenuEntry
This will probably bitrot my pending change to convert pick to use blobs.  Or if mine lands first, then it will bit rot yours.
We could just remote the `style/gridview/` all together afterward :)

and you really cannot tell the relative path of `document.styleSheets[0]` ...
I think I would also rather just pass my own full string for the media type I want to accept. Maybe keep going on this path, but also allow me to set my own *Accept* header?
I'd prefer to avoid this as I've seen this pattern cause lots of confusion in the past. We need to expose `format_backtrace` on something, so I'd recommend that we extract out the backtrace-specific material from this module and expose it in a separate module as module functions. Then this module can `extend BacktraceFormatters` (for example) and other consumers can invoke the functions directly. WDYT?
Right again. `!!` is a habit of mine for coercing booleans - not always necessary, such as in this case. Bleh ;) You are thorough :v:
may the force be with you...
See https://github.com/sk89q/worldedit/pull/201/files#r1239562
-I や -X には `` ない
У меня в проекте используется gem 'russian' и one:, few: и many:.
А в Spree есть что-то, что позволит писать разные формы для разных случаев?
I have restructured the test so that all subcases follow the same structure
Not sure about this change, did simple aliasing cause any problems?
The outer parentheses are redundant
Uh this is a mistake and thx for point it out.
Lost an E
If a scheduler uses codebases and also has a Trigger step then the related Triggerable scheduler will also need the argument codebases. Else it cannot handle the sourcestamps from the trigger step. 

The Triggerable scheduler does not need to have the same codebases as the triggering scheduler. But there must be some overlap of  codebases to be able to exchange 'change' information.

If no codebases are used then the scheduler has a default codebases of {'':{}}. In this case the ``Trigger`` step will sent a ``Sourcestamp`` with also an empty codebase, so the codebase will be processed.
Please declare `i` before this line, i.e. inside the *if* block.
No because they're part of the package metadata. The root interface is for runtime stuff that's created.
Hi @MrZYX , Which translations should I add?
Sorry about the delay to reply, but the commit added to my master branch should fix it, ssl and non-ssl vhost with redirect rule for nginx.
as i said, this should be done in the highest priority, reading thread
You boys are up to something loosely typed here... That poor option! The intention here is to try a route then a fallback then error, so maybe something like:

    def routeCompetition(tag: String): Option[Action] = {
      Competitions.competitions.find(_.url.endsWith(tag)) map { CompetitionFixturesController.render(tag, _) }
    }
    
    def routeTeam(tag: String): Option[Action] = {
      TeamMap.getTeamWithName(tag).map { case (x, team) => team } map { TeamFixturesController.render(tag, _) }
    }
    
    def renderTag(tag: String) = routeCompetition(tag) orElse routeTeam(tag) getOrElse Action(NotFound)
I think this should be part of either "detect-engine:" or maybe nfq/ipfw.
If there is no such view but there is a folder (i.e. any kind of `ItemGroup`) with this name, ought to list that.
Add a space between the hash and the following word.
return quired -> returns queried
Shouldn't you be using `isWindowPBEnabled `here?
geeze.  nice catch.  a consistent misspelling!  go lloyd!
when running locally, don't stomp user specified config files!  This is especially useful when debugging l10n stuff, which is why it's bundled with this PR.  (yeah right, I just got lazy)
Yes, definitely.  Thanks for your comments, @ozten!
Don't we add an infinite amount of warning classes now?
why don't we use this in defaultTitle() and tabTitle()?
I'd prefer not to make another option for this but rather just as something that is configurable in knife.rb.  If we set a sane default it shouldn't have to be changed very often.
Do people run with both south and nashvegas installed in the same project?
Just a suggestion, I've learned to avoid "old" and "new" in code because this stuff tends to have a much longer shelf life than I usually expect it to when I write it.  I've switched to "current" and "previous" (conceptually, not literally) in most situations which is a bit more...  timeless.  Alternatively names that have versions or numbers or something may help since they're also timeless.
This regex is hurting my eyes. Break that up yo.
MRI check hash type
```c
if (OPTHASH_GIVEN_P(opts)) {
  randgen = rb_hash_lookup2(opts, sym_random, randgen);
}
```

**OPTHASH_GIVEN_P** is defined as:

```c
#define 	OPTHASH_GIVEN_P(opts)   (argc > 0 && !NIL_P((opts) = rb_check_hash_type(argv[argc-1])) && (--argc, 1))
```

and **rb_check_hash_type**

```c
VALUE
rb_check_hash_type(VALUE hash)
{
  return rb_check_convert_type(hash, T_HASH, "Hash", "to_hash");
}
```
I think creating extra `Zend\Uri` object is expensive and regex will do just fine
I mean that you want to say getHeaders()
This should be called with each key value as argument.
use constants here
This has to be recoded. It's a nightmare to maintain. Please make it more modular instead of a 200-line function
Nether portals are not supposed to work in the end.
you can write `e.toString()` here
This is not really a string, just a message key. I think you can just use "login-autoLogoutMessage" directly since the point of configuration would be changing the value in message bundle rather than the message key itself.
I'm not sure to get what we're testing here? The fact that we don't have any exception? Shouldn't we test also that changed does contain what we want?
I think it's okay to tweet the current URL instead of the original URL. It's meant to help share a video in the context of a playlist.
I think we have a problem with this step. `standalone.xml` changes quite a lot, so we're going to have a hell of a time keeping it in sync. Is there a way to do it without having the user copy these files in?
Is this supposed to be tags around a space?
Nitpick: e is not used, remove.
I think it's okay to err on the side of permissiveness until we have a problem. Templates are only editable by a subset of wiki users anyway(?) so I'd rather empower the template writers.
You can do that with `border: something; padding: 2px; display: [inline-]block;`. But if there's other reasons OK... if it's just for a border with padding you can omit the div.
Actually this is a pretty good idea. It simplifies it a lot and exception handling is fairly expensive.
What do `st` `sb` `et` and `eb` stand for? Maybe wanna spell out the names.
making things work that way is definitely on my list of Things To Do, but not in this pull =)
maybe we should not call it oauth, but rather auth in general (as we have a few auth related settings)
I was thinking on one visible select, that changes values based on the
radio?
Also use ZU.cleanISBN here to clean up the ISBN. This will help us stay consistent with the way we clean up ISBNs across multiple places in Zotero.
I have no idea how that proxy got in there - I'm not at Rutgers, never have been, don't have a proxy login and don't have Zotero set-up to plug in rutgers proxy addresses. Could they be re-directing this on their side?
Please add emoji syntax in pull requests.
I think this should be `@include border-radius(0 5px 5px 5px);` if it goes in clockwise from TL
Or maybe optionally specify a `filename` param, but pulling the filename from the URL gets old fast if you don't have a nice URL to work from.
There's no “d” in “Ajani”. ;)
`fetch_lang` is very similar to `fetch_card`. It'd be great to get rid of this repetition, though looking at the code it's unclear how best to do so.
Can we leave the previous defaults here until we have euconfig working the way we want. After that we can pull this info from there.
жалко, что тут size_t лучше value_type от итератора
yeah, I'm not sure what the purpose of this is, I just changed it from reading a value out of xmlDocs to using reflection.
This returned function is the actual event handler bound to the `"breakpoint"` event. It handles all of the complications of `options.delay` in setting and clearing timeouts.
Wouldn't (By.CSS_Selector, '.nomenu.logout') work as well?
Oops! It's "timesigN" pth lowercase T. Sorry
This is fine as a quick test to avoid XML parser overhead on other file types, but doesn't detect a newer or older version of the XML schema that we perhaps can't read.  So, I'd also like to see the document's namespace checked against `http://www.leica-microsystems.com/scn/2010/10/01`.
My preference would be to have this function just take a timestamp (the cutoff), and leave the computation of that timestamp to the caller.
Please avoid checking in files without newlines at the end.
In case it helps, the manager role should always win.
Moved + switch parameter order.
But this is the only way to know if ANYTHING works!
I'm wondering if this shouldn't be named ``Scala JUnit 4 Test``, or even ``Scala/Java JUnit 4 Test`` (a bit verbose, but maybe clearer for the user)
Ah, I see what you did - you changed the selector in the `attachVote` and `attachNotice` functions below to look for this.
hmm, I think my scenario was the opposite: joining a protocol to a filepath. Still, if we can assume forward slashes for urls, we can split on the slash to get the basename.
Тут надо добавить пустую строчку.
Люди разные бывают =)
Зачем мне возвращать NSArray, мне нужно значение именно NSMutableArray.
Лучше вынести создание категории в отдельный файл. У xcode есть удобная функция для этого - File -> New -> File или ⌘N
не самое удачное решение для подготовки случайных индексов с вложенным циклом, лучше подготавливать массив индексов по порядку, а потом случайным образом перемешивать его (https://u.hexlet.org/courses/4/discussion_topics/26#c98)
да-да, я уже понял
I think it will be better to have this sort of documentation in the wiki, what do you think?
I don't think you included your changes to overlay.html to add the cancel button
... or do what reflow in Adobe Digital Editions does (as included in Hanlin V3 firmware), i.e. jump to the top of the physical page, i.e. to the very first virtual page.
What's the point of this edit ? Isn't the code the same after edit ?
when with shake_input for added clarity and to disambiguate from temporary "for shake in shakes"
2 lines here too.
how about having tmin and tmax that can be None and default to self.tmin and self.tmax in that case?
this is confusing, can you just use `tris_connectivity`?
That would be nice since it would automatically be inherited by all functions beneath it.

I can give that a shot and see how it compares.
multi-dimensional (a missing)

Also, maybe we can mention that numpy.unravel_index can be used to recover the multi-dimensional indices.
Yes, that makes sense, something like viz._plot_topo_imshow. One could do the same with plot_tfr and then have plot_power and plot phase_lock. The other option is, as you said to have title parameter and keep the tfr.
Use status code 500 to illustrate internal server error
body{font-size:10pt;font-family:arial,sans-serif;background-color:#ffffff;color:black;}p{margin:0px;}That was the suggested change for this ticket on the wiki. Will change it if you like. -----Original Message-----
From: Ben Wolfe Sent: Sep 26, 2012 8:40 AM
To: openmrs/openmrs-core Cc: andreapat Subject: Re: [openmrs-core] ValidateUtil validate method: added null check  (#36)

In api/src/main/java/org/openmrs/validator/ValidateUtil.java:
> @@ -82,6 +82,10 @@ public void setValidators(List<Validator> newValidators) {
>  	 * @should throw APIException if errors occur during validation
>  	 */
>  	public static void validate(Object obj) throws APIException {
> +		if (obj == null) 
> +	        throw new APIException(Context.getMessageSourceService().
> +	            getMessage("error.null"));

Looks like this translates to "Cannot be empty or null".  I suppose that is good enough.

            
              —
              Reply to this email directly or view it on GitHub.
consistency: load-balancing vs load balancing. I suspect we should be using the term "load balancer" and "load balancing". There's about 13 instances in this commit.
We have magic syntax! RFC can be done like this:

:rfc:`1918`

And this syntax should render a link there.
No I think it is way better to let the user have the full control over this. Typically while I was using this feature, strange bugs happened because some files was processed by rjs on my back.
I've replaced this sting with [this](#L2R24), because objects, which returns by `zlib.createUnzip` don't have `setEncoding` method by some reasones.

Also note, that function `mime.charsets.lookup` passes two arguments, where second is a default charset.
This function returns default charset for all mimes, which don't passes `^text/` regexp, for all other mimes it returns `UTF-8`.
And if you use `setEncoding` to change encoding of incoming data from something like windows-1250, or koi8-u, then you should know, that this function and `toString` [can't](http://nodejs.org/docs/latest/api/all.html#all_stream_setencoding_encoding) handle that kind of encodings. Also you deletes information about encoding in [this](#L2L18) line.
I shouldn't be removing the comment though.
Not sure, is $this returned elsewhere in Aura that we might look at what @pmjones has done previously?
This should be:

    $locale = $dialog->select(
        $output,
        'Please select your locale',
        $availableLocales,
        0
    );
I think this sounds a little better:

>
You could of course do this by customizing how this field is rendered in a template.
I would maybe add:
* Aliases (a list of strings containing all the alias used  to access the wiki in domain or path based multiwiki)

There is other stuff that are a bit less important (not used much and even less from REST):
* Secure (indicate if URLs should be forced as https)
* Home page
* Template (indicate of the wiki should be listed as available wiki template)
A single line short docstring followed by a blank line and then explanation

"""Simultaneous slice-timing and motion correction algorithm

This interface wraps ...
"""
Ну название кривое какое-то.


2012/12/12 Maledictus <notifications@github.com>

> In src/plugins/blogique/blogiquewidget.cpp:
>
> >  			}
> >  		}
> >  	}
> >
> > -	void BlogiqueWidget::SetParentMultiTabs (QObject *tab)
> > +	QList<QStandardItem*> BlogiqueWidget::CreateItemsToView (const Entry& entry) const
>
> что?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/0xd34df00d/leechcraft/pull/153/files#r2390429>.
>
>



-- 
  Georg Rudoy
  LeechCraft — http://leechcraft.org
`QVariant`
I could go on... :-P
Hi, why did you remove this line ?
Shouldn't the logic be a bit different?

Try to test if `reload_templates` will be True by default.
Probably don't want to leave code in that is commented out... In future months you'd wonder if it wasn't valid, or wasn't completed.
Tested. It works as expected without the guard and without the __init__ too :)
Zoom does not stay the same when switching between OSM and Google with this change.
I'm not sure why this +1/-1 strategy worked with 0.2.4 version, but gives errors with 0.3.

I am going to close this pull request until I have time to look at this further.
The key is part of the entry but this method also gets called with null entry (and I need the key). Yes, the calls to StateConsumer can be extracted into another method and then the need for the key param is moved there :) Let's leave this as is.
Yes, will do so! Thanks Manik
Also I don't see `ActiveSupport` in the `gemspec` you should probably add this with the same kind of conditional for it to only be required if in 1.8.
i hesitated a long time to find where and how to named it ;)
you should not force having the class in the include path (especially as you don't make it work if it is not the case). You should configure the location in the [phpunit config](https://github.com/kriswallsmith/assetic/blob/master/phpunit.xml.dist) and then load the file on [bootstrap](https://github.com/kriswallsmith/assetic/blob/master/tests/bootstrap.php) instead, as done for CssMin.
Once it is done this way, you could even configure Travis to use it by getting the class [before the testsuite](https://github.com/kriswallsmith/assetic/blob/master/.travis.yml) and configure the location for travis in the [dedicated phpunit config](https://github.com/kriswallsmith/assetic/blob/master/phpunit.travis.xml)


Thus, you code would trigger a fatal error if a second test is added as it would load the class twice.
LessPHP is not required. Maybe "suggest" is a better place for this?
Personally, if it makes the code more readable then that's preferred. Ideally, @WeatherGod will weigh in with his PEP8 ninja chops :)
In contrast to python docstrings, godoc wants to have the explanatory comments immediately before the entity they're describing.
cf #21, this should be replace by a link to the original readme.
я за PSR2 =)
не увидел "@@ -171,6 +203,7 @@ public function clean()"  ))
THis method can be removed. I replaced it with something else
Could you please check the indent? It does not look correct.
coding style violation : should be } else {, see PEAR documentation : http://pear.php.net/manual/en/standards.control.php
Oh shiiii I hope Sam doesn't see this. Shhhh...
Should we use UUIDs?
@apex13 1 space after the `if`
@gsilver can you make sure there is always one space after the colons? `: `
Shouldn't this all be lower case?
no need to specify GET
are, not is.
Dobre, v zajmu skvelosti a funkcnosti predelam! :) Jeste tu neco doklepnu a pustim se do zapracovani zmen.

Udelam to tedy tak, jaks navrhl ty. Akorat misto  temp_schedule navrhuji nazev schedule_timer_callback.
Hrm. That sounds like you're grabbing elements by tag when you should be using DOM traversal instead then.. I'll take a better look through the importer in the morning
Instead of genfromtxt you can use loadtxt with parameter ndmin, like

```python
context.completeness_table = np.loadtxt(
    context.config['completeness_table_file'], delimiter=',', ndmin=2
)
```
This functionality needs to reside in a different file - perhaps diffa-common.js. Putting it in heatmap.js means that the links won't be enhanced when the settings or status pages are shown.
Are you sure the class "Zend\Di\Definition\RuntimeDefinition" actually exists, or did you maybe forget to add a USE statement?
comment could be more precise.  there is no header property in TableView.  The possibilities are headerPullView, headerTitle, headerView.
This formatting is weird. I'm going to be merging but you should fix tomorrow to something like:

    if (($validateXml
            || ($this->configuration
                && $this->configuration->get('validateXML')))
        && !XMLValidator::validateXML($xml))
    {
The indentation is slightly wrong for the rest of the method.
Is there a reason to prefer plural vs singular in the URL? pub.dartlang.org/package/foo/version/1.2.3?
Could you refactor this section? I realize that the repetition was already there before you made changes, but it would be really good to get rid of it now.
You'll have to replace `JFactory` with `self`, or you'll get a code style error. Also the convention right now is to use $input, that isn't really important thou.
others url are build in action
Here is the problem, you are pointing the stdevp method to the stdev function. It is quite simple to fix, I will merge and fix myself.
ST's official instructions say to create a symlink to the tool in `/usr/local/bin` but it has to be done manually. There's no install-command-line-tools menu option like you find in BBEdit or TextWrangler. OS X guidelines say `/Applications`  or `~/Applications` for `.app` packages so they're reasonably safe assumptions for places to search.
The question of features is indeed interesting; note that we could use PointCloud2 in 2D as well, should we specify this? This leads to the question of specifying the fields in PointCloud2, which are currently only informally defined by their usage in PCL. I have been talking to people here at Willow maintaining PointCloud2 and they agree that fields should be defined and that they plan to work on this in the next weeks. So once these fields are defined we can link to the specification from this REP. What I am not sure is how much information we want to give here. Maybe we could say that points can have additional features (like normals and density for instance), but that the REP does not enforce any of these?
haven't seen this one before, what is it for?
Removed, fixed indentation on marker variable.
Rename settings to account.
Define all arguments function takes, even if you are not using them. In this case, `event`. This is useful later on for somebody to know what is all possible here.
See, this is the problem. This is part of the template, not code. The name of CSS class is designer's job to decide how to do. You should just return some value to signal if you should put the class or not.

I would recommend using filter, not tag. You could then do in the template:

```
{% if request.path|is_active:"account" %} class="current_item"{% endif %}
```
Rename to `POLL_BICIKELJ_INTERVAL`.
Turn the logic around? if not check -> raise?
Аналогичиная ситуация как и в комменте выше, код планировался быть вложенным в этот элемент списка.
You removed the only place the constant is used in the class, so there's no reason for the constant to exist.
and here ^^
Structure:
data << uint32(BgWeeklyWins20vs20);
data << uint32(BgWeeklyPlayed20vs20);
data << uint32(BgWeeklyPlayed15vs15);
data << uint32(unk);
data << uint32(BgWeeklyWins10vs10);
data << uint32(unk);
data << uint32(unk);
data << uint32(unk);
data << uint32(BgWeeklyWins15vs15);
data << uint32(unk);
data << uint32(unk);
data << uint32(unk);
data << uint32(unk);
data << uint32(unk);
data << uint32(unk);
data << uint32(BgWeeklyPlayed10vs10);
data << uint32(unk);
data << uint32(unk);

the unknown fields seem to have something to do with arena.
4 of them are unused, btw.
I'm stumped on how to revise the regex to not match a new line. Using `[ ]?` (or `_?`) in the regex seems to match `\n` just as `\s?` does. Thoughts?

    /Controller([ ]?Test)?$/i =~ "WidgetControllerTest" # => 6
    /Controller([ ]?Test)?$/i =~ "WidgetController Test" # => 6
    /Controller([ ]?Test)?$/i =~ "WidgetController\nTest" # => 6 <- Ugh!
    /Controller([ ]?Test)?$/i =~ "WidgetController\tTest" # => nil
    /Controller([ ]?Test)?$/i =~ "WidgetController\rTest" # => nil
    /Controller([ ]?Test)?$/i =~ "WidgetControllerXTest" # => nil
This docstring could contain all required fields you are want to be filled up.
I've talked to @rangadi about pushing out the hadoop-lzo to our repo or a github repo. Hopefully we can do this before we merge in this branch.
ok. consistency all around now. methods at the table level are from and to, for more specialized methods I specialize the name to what is being read/written. layoutsToBackup(), for example.
Ah, OK. I didn't know that these were called core entities in MusicBrainz. In this case, for consistency with general JDK naming convention I suggest a package private 'AbstractCoreEntity'.
maybe should we refactor this in utils? I see this pattern used a lot.
You mean `global_options()` not `options()`, right?

Added comment.
This came from shortcuts originally being implemented here, before I decided that they needed to happen in KEY_UP instead. What was left matched the original logic, how about that :)

I think this is clearer, mostly because of the comment. Lets just leave it like this.
Please add test to the set of tests at line 50 where you test that business_day(date) returns the correct time when the business day start value is a different number (i.e. 2am).
I do not know if this would be faster, but it would save a lot of appends/deletions and make the code clearer (to me at least ;).
Also you would end up with iin_keep with only the solutions to be pooled at stage 2, which avoids testing ```ssr > ssr_keep[n_keep-1]```.
how about taking and OLS results instance as an argument in the ``__init__`` ? This way we wouldn't redo the OLS when we just use the methods on the existing OLSResults instance.

To me this class looks more like a results mixin, than an estimator, at least in this usage, even if it might not be the case for the actual calculations or class hierarchy.
I will follow this per your recommendation.  However, I am not familiar 
at all with Sphinx.  I will do my best to learn it but in the meantime 
please let me know if there are any other documentation issues.

On 06/21/2012 01:40 AM, Ralf Gommers wrote:
>> +            variable.
>> +
>> +        gamma_low: lower bound for gamma when finding lower limit.
>> +            If function returns f(a) and f(b) must have different signs,
>> +            consider lowering gamma_low. default | gamma_low =-(10**10)
>> +
>> +        gamma_high: upper bound for gamma when finding upper limit.
>> +            If function returns f(a) and f(b) must have different signs,
>> +            consider raising gamma_high. default |gamma_high=10**10
>> +
>> +        epsilon: When using 'nested-brent', amount to decrease (increase)
>> +            from the maximum (minimum) of the data when
>> +            starting the search.  This is to protect against the
>> +            likelihood ratio being zero at the maximum (minimum)
>> +            value of the data.  If data is very small in absolute value
>> +            (<10 ** -6) consider shrinking epsilon
> When you use symbols like ``*`` or ``|`` in a docstring, you should surround them by double backticks so they're rendered by Sphinx as is. Now it will consider those symbols like special characters (``**`` is bold I think) and therefore you'll get rendering errors.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/statsmodels/statsmodels/pull/326/files#r1023991
This part has to be removed from the toplevel README.txt.
README.txt is the main description for statsmodels for pypi.

I don't know where this should go. As a description of the files it's more oriented towards maintainer notes.
Maybe park it in a readme_L1.txt until we go over changes to the general documentation for the enhancements.
Won't this show validation and remove right after? Kinda making the UI to blink?

No idea if that is gonna happen just asking =D
If these dimensions aren't needed for remote images, are they needed for any images?
I have constant problems with indentation, keep fixing it but it's never ok, hmm
Just confirming we're using tables this time around instead of http://twitter.github.com/bootstrap/javascript.html#typeahead
I'd rather make it so all attributes _are_ copied.  Are there other cases of things that are not correctly copied currently?
Er, no.  Sorry.  I realize now that the shear at the grid points gets directly returned in that case.
don't think you need this function anymore
rename to should_return_no_diff_if_file_and_array_have_equal_content ti be consistent with my comment on SUCCESS
sorry, no inline styling allowed :)
Perhaps better to just get Throwables.getRootCause(e) here. Things like NPE usually have a null message, but the toString on the exception would still be useful. Similarly, some exceptions you need the context of the exception type to make most sense of the exception message. So below you could just do a toString on the exception.
Could use guava's Objects.hashCode
oh, I was confused by http://wiki.ecmascript.org/doku.php?id=harmony:number.isfinite. Seems to be an error in example.
Sure - not a problem. I've made a ticket for that already: https://jira.mongodb.org/browse/PHP-542
Tobbi change this to use `find_element` and `send_keys`
"not anymore send requests" --> "not send requests anymore"
Could you be more precise about what you don't understand?
Remove that?
I think you meant 'hastily' here.
So this where the problematic functions, I previously defined them as "protected" so they should be accessible to the derived class (FilteredUavTalk) as private members.
This worked as it should (IMO) on my machine and Ed's but it fails on jenkins and 1 other with the following error:
"builder/ground/openpilotgcs/src/plugins/uavtalkrelay/filtereduavtalk.cpp:144:31: error: ‘updateNack’ was not declared in this scope
/home/jenkins/workspace/phoenixpilot-pull-request-builder/ground/openpilotgcs/src/plugins/uavtalkrelay/filtereduavtalk.cpp:161:30: error: ‘updateAck’ was not declared in this scope"

I would prefer to comply with proper OOP thus avoiding making this functions not public.

Any ideas?