labofsahil/github-event-dataset-2012 huggingface dataset:
I'm not sure that's used, but it's for the case where bsid is None - in that case hopefully we expected no buildset (expected_buidlset is None).  It should return right after this, and should use `assertEqual`.
You seem to be right. This is a somewhat old change. I don't know why I worked out that bf.perspective and bot were referring to the same object, I'll have to rework that out. SIGHUPs seemed to work so I assumed gracefulShutdown was working. I have made the same assumption in https://github.com/buildbot/buildbot/pull/469/files#L0R582 .
Latest phing release has pretty version == '2.4.13snapshot20120629164452'. When it written to lock file it can't be read back cause lock file reader  uses ArrayRepository::findPackage that uses VersionParser::normalize that throws on string '2.4.13snapshot20120629164452'
Can you add a comment before the colorscheme section describing marking that section
I did not study the data sheets, but would it be possible to include the standard definitions hackery_<base_mcu_type>.m4 to reduce the places where to make changes if we introduce new defines. See hackery_atmega328p.m4 for an example.
no, I am not

if `filter` method return `false` than execution will continue(no filter)
if `filter` method return `true` than execution will stop at the filter

So allow means `false` in filter context.
Not trivial, I even think that you only should use the `@` for property definitions, not function calling or as a replacement for a standalone `this`, like in this case.
`unless tree.try(:tree)`
This should be removed as well.  Injecting a file into the class path like this is dangerous and it would be best to remove the code entirely.
any chance config.get() returns non-array value?
nice catch!  as you mention in the other line comment, the reason it appears to work is because I defended against calling shouldAsk with undefined inside storage.js as well.
verified removed controller wasn't references anywhere else
You mean reverse what is currently happening? Make transitionComplete the default, and change submit to verify if it is not needed?
Again, use real comments, not multi-line string literals
#74 (re)introduces `titlebar.css`. If that pull lands, move this styling there!
Could you please explain reasons to use `formtastic-bootstrap` from "github.com/saegey/formtastic-bootstrap" instead of the rubygems?
Example of how to use new infrastructure
we try to keep imports in alphabetical order. com.google.* before com.twitter.* before java.* etc.
also FYI you can do `td.source!= line.source`
Kinda agree, but min/max is used in PHP rand()/mt_rand() docs. I think we should be as close to PHP docs as possible
Don't import from a subnamespace of the current namespace.
No, and that was the reason because the mentioned bug was not detected.



>________________________________
> De: Roman Vidyayev <reply@reply.github.com>
>Para: Maks <github.maks3w@virtualplanets.net> 
>Enviado: Viernes 4 de Mayo de 2012 20:01
>Asunto: Re: [zf2] Refactored Zend\Validator\AbstractDb to compy with the latest Zend\Db changes (#1054)
> 
>> -     * Test when adapter is provided
>> -     *
>> -     * @return void
>> -     */
>> -    public function testAdapterProvidedNoResult()
>> -    {
>> -        //clear the default adapter to ensure provided one is used
>> -        AbstractTable::setDefaultAdapter(null);
>> -        $validator = new RecordExistsValidator('users', 'field1', null, $this->_adapterNoResult);
>> -        $this->assertFalse($validator->isValid('value1'));
>> -    }
>> -
>> -    /**
>> -     * @return ZF-8863
>> -     */
>> -    public function testExcludeConstructor()
>
>Isn't it a duplicate of testExcludeWithString then?
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/zendframework/zf2/pull/1054/files#r775800
>
>
>
use $this->setExpectedException instead of try/catch block
Zend\Acl\Resource\ResourceInterface
what is wrong here? session expires when (last modified + lifetime) is older than current time
Shouldn't this be false by default?
Support empty uploads.
this property access should be protected.
By convenion the interfaces must have the suffix Interface (e.g. ExceptionInterface)
Fixed (again).
Nope, I don't have this in my repo local repo.  So it isn't my code at all.
will be exchanged with an IF-EXIST statement
Comments and more tests added.
would the `PHP_EOL` suffice?
Corrected in [0ec17aa3624a20eb81df6a5929707d0b17677e2b].
Align both equal signs
I've learned the error of my ways. Create new migrations now for adjusting database tables.
This is one issue I was asking about. What I'd like to do is implement `extend` once, although I don't know exactly how. You may say that `extend` is one level of abstraction higher than other functions in GenDict protocol, because it is implemented in terms of one of those function (`GenDict.put`) and thus it can have one single implementation for all dict types.
Since `|` is an operator, we could use it instead of `or`.
@emodric Then @patrickallaert will be :angry: ;-)
Good interpreters optimize this for you. I wouldn't never optimize something like this unless it came up in a profiler.

Here's a presentation from a Chrome developer in which he explains how this case is optimized in V8: http://s3.mrale.ph/jsconf2012.pdf
Spaces around = and after , 
Why not use 'licenses' ? i.e., ' instead of "
Put a newline before blocks.
It is a tough choice.
TEXT -> Text
This is actually testing that it works without passing the id around.

There was a bug where it worked one way, but not this way.
ENCRYPT() generates a randomly hashed password compatible with php's crypt()

I have removed the 'activkey', since it seems like a security risk to have a default value
right. fixed the syntax and changed `nil` to `true` to match the other examples.
no sense in keeping new action for api
I think Rackspace have stated they intend to deprecate the V1 API in the future - they want everyone to move to V2. I suspect it will be a long time before they turn it off though, so being clear makes sense.
On second thoughts either this name is badly choosen, or the one for the slot in the pager is. I need to sent the exact contrary of what this signal name implies for the semantic of the toggleListening() slot in the pager to be respected.

I'm all ears if you have a better name for either of them.
On 16 April 2012 16:01, Henrik Skupin <
reply@reply.github.com
> wrote:

> > @@ -53,3 +53,13 @@ Now click the `Launch` button and the node should
> automatically connect to the m
> >  Using the Jenkins master as executor
> >  ------------------------------------
> >  If you want that the master node also executes jobs you will have to
> update its labels and add/modify the appropriate platforms, e.g. 'master
> mac 10.7 64bit' for MacOS X 10.7.
> > +
> > +Running on-demand tests
> > +-------------------------------
> > +1. Navigate to your Jenkins instance: http://IP:8080
> > +2. Open the +admin tab and look for the 'trigger-ondeman' row
> > +3. Click the green arrow button, 'Schedule a Build'
> > +4. Upload your ondemand.cfg file (see examples in ./config/ondemand/)
>
> Feel free to add a link to the folder on github here.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/whimboo/mozmill-ci/pull/50/files#r690899
>


How do I do this? What is the URL I should use?
https://github.com/whimboo/mozmill-ci/tree/master/config/ondemand?

-- 
Anthony Hughes
Excellent work Westley.  I'm good for commit on this one.
I think we can make the problem more concise by asking, "What is the value of x in the following arithmetic sequence?" here, and eliminating the line below.
getApkIcon_fromAPK这函数删掉，或者把getApkIcon这个函数删掉吧
can you review your indentation? Because I see it's not correct.
djangopy -> djangopypi
а верно ли, что тут условие может быть 
do {
} while (top_vertex != v)
ah, or maybe not. foreach resets to the first item which is something we want to avoid to not re-iterate over everything again.
Yeah, we need to check overflowing in almost all date/time operations, but
ceylon.math is using java.math API and its not avaiable for javascript, we
need to find another way to implement these operations.


2012/12/25 Roland Tepp <notifications@github.com>

> In source/ceylon/time/Period.ceylon:
>
> > @@ -261,19 +259,19 @@ shared class Period(years=0, months=0, days=0, hours=0, minutes=0, seconds=0, mi
> >
> >          value years = this.years + this.months / 12;
> >          value months = this.months % 12;
> > -
> > -        variable Whole total := wholeNumber(this.hours * secondPerHour.getMaximumRepresentation())
> > -                              + wholeNumber(this.minutes * secondPerMinute.getMaximumRepresentation())
> > +        //TODO: We need to remove ceylon.math reference because it only compile to JVM
>
> On the other hand - do we *really* need to use math module here.
>
> Initially I used it because it uses BigInteger internally and that would
> allow to do calculations without overflowing.
>
> We need to test extreme values for periods to see when the use of Whole is
> needed and see if the use is really warranted.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/luolong/ceylon.time/pull/8/files#r2500636>.
>
>
is there going to be a signup/register link in the dropdown?
One statement per match with the participants in it would probably be enough.
This should be /sys/fs/cgroup on Fedora and newer distribution
Config file should not be required.  Breaks BC
Remove this logic.
Agreed.  There needs to be a CommandQueue class.  Not sure on the timing of that refactor.
My thought was that we would have a canned message label or something. I'll default this to displayName as the username isn't always available.
I incorporated both of these comments in my LoginTenant branch.
This is out of order.
I'm always a bit suspicious at ``foldRight``, because it can lead to ``StackOverflowException``. I'm not sure what's the data set size to expect for this scenario, but if possible it would be better to change this.
Instead of using pattern matching, why not having proper (case) classes for the different ``MethodClassifier``s. I find it would be easier to read.
Whoops, even though pig is working in local, I need to get rid of this comment.
On Feb 23, 2012, at 6:13 PM, Steve Johnson wrote:

>> +        # Get all script parameters as a list
>> +
>> +#        abs_input_paths = []
>> +#        for i in self._input_paths:
>> +#            abs_input_paths.append(os.path.abspath(i))
>> +#
>> +#        input_paths_str =  'INPUT=' + ','.join(abs_input_paths)
>> +#        log.debug('Pig_input_paths_str: %s'%input_paths_str)
>> +#        output_path_str = 'OUTPUT=' + self._output_dir
>> +#        log.debug('Pig_output_path_str: %s'%output_path_str)
>> +#
>> +#
>> +#        args.extend(['-p', input_paths_str])
>> +#        args.extend(['-p', output_path_str])
>> +        return args
>> +
> 
> If Pig isn't working in local mode, we shouldn't keep dead code in the master branch. Definitely hang on to it for the future but clear it out of this version.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/Yelp/mrjob/pull/378/files#r483582
Для сравнения строк в методе hammingDistance  и  mutate вы используете оператор !=, это не всегда корректно. Оператор != сравниваяет значения указателя (адреса в памяти), а не сами объекты. 
Например:
       NSString* foo = @"Foo";
        NSString* bar = [NSString stringWithFormat:@"%@",foo];
        if (foo == bar)
            NSLog (@ "%@ = %@", foo, bar);
        else;
        NSLog(@ "%@ <> %@", foo, bar);
согласна, что тут есть ошибка, но программа не будет вываливаться с ошибкой, если numCell  не пройдет по условию, то просто ничего не выполнится. Надо исправить
вчера чистила зубы и думала над вашим кодом:)
И пришла мне в голову такая мысль, что в случае если нажо мутировать 99 элементов из 100 мы долго будем ждать пока ваш код сгенерит эти 99 случайных неповторяющихся чисел.
примерно столько же длого как и в самом популярном решении этой задачи, когда хранится массив уже измененных индексов.
I hope these aren't real keys.
I think there's some value in making this more generic. I think we would want a "reveal in finder" option in Brackets at some point either in core or as an extension. Thoughts?
My secret hope is that they'll turn into links when _not_ in backticks. Backticks would just be preformatted text.
What is `mLocalKey` used for? Seems like it could be used for a mapping of some sort but wouldn't using `mImageid`, `mGuid`, or `mLocalPath` be a better choice than creating a new variable (as long as it is unique)?
Maybe better to use ```$( document ).on( "pageinit", "#pageID", function() {``` in here, since that is what we recommend in general?
Please do not remove the obsolete `LibGit2Exception` yet.This will be done after the next version (v0.10.0) is released. @dahlbyk has already pushed #181 to deal with this
This line is off-by-one, right?
I think we should return both indices and scores.

and I would rename criterion to select_mode = 'all' | 'max'

one might want to remove the sources for which corr > 0.9 ?
@rrjamie Any reason not to print these out in both branches?
Another thing is that, map.keySet returns a set view, meaning that when the user properties are updated, the keyset is updated, meaning that your assertion on line 1180 is confusing about what you are testing, i would expect that blank string to be the removed property's value, or even better you would just test newUserProperties.values().contains(original value) == false
Hmm... I get your point. I confess this caused me some problems during development, but at the end it passed all the tests (and it seems to be working correctly on my application). I could not figure out a way to test these other than just running the original devise tests.
I doubt you want to internationalize these two. Removing the gettext call will also avoid having the problem mentioned in a75a4af0
Does this do `notify_or_ignore`?

Whenever I use Airbrake raw like this, I do `Airbrake.notify_or_ignore(exception)` so then if you choose to ignore types in the initializer it still works. This is probably just a controller thing it dumps in though, so I imagine it's fine.
These two lines of code seem like they could be extracted into a method somewhere.
Yes, you're right. That's the part I personally don't like about the Symfony CS, but I will change it soon.
No problem, but why? I've looked at other parts of the document, and `:class:...` parts are inlined everywhere.
"the --the-- look and formatting of HTML"  
"Together they form"
Could you fix this indentation first?
Using the main connection pool will be annoying in this test because it's basically a global pool.  We can't easily isolate it from the rest of the tests.  It's better to use the `@pool` variable supplied in the setup method of this test suite.  The pool created in the setup should be isolated from the effects of the rest of the tests.
А ничего, что он еще нигде не создается даже, а просто класс?
En este método ¿una misma imagen puede devolver varios slots con referencias a attachments? ¿Es eso correcto? No entendí bien por qué y no me hace mucho sentido.
```
@using (var permissionHelper = Html.BeginPermissionTrimmingForm("GenerateOrder")) {
    //Template
}
```

`permissionHelper` podría tener atrbutos como `IsAllowed`, `ActionName` y `ControllerName`, también otros métodos simples para renderizar condicionalmente y hasta para renderizar un `<input>` o un `<span>` dependiendo de si está permitido o no.

@AlphaGit que opinás?
How about `pattern.all? { |e| e.is_a? @array_type }`?
Relies on `steps` and `results` to grow in parallel.
Removed some dead code.
Fixes gh-3
Actually I'd like to get this out of debug. It does pollute the logs.
Mircea, I meant removing the LockInterceptor completely from the
interceptor stack for local-mode caches. I don't think we support this
right now, and I didn't see any users clamoring for it :)


On Wed, Dec 12, 2012 at 12:53 PM, Mircea Markus <mircea.markus@gmail.com>wrote:

> yes, especially for JDG in server mode which uses non-transactional caches.
>
> On 12 Dec 2012, at 08:02, Dan Berindei wrote:
>
> @mmarkus, this seems like a new feature, do we really need it?
>
>
> On Tue, Dec 11, 2012 at 8:37 PM, Mircea Markus <notifications@github.com>wrote:
>
>> In
>> core/src/main/java/org/infinispan/configuration/cache/LockingConfiguration.java:
>>
>> > @@ -53,6 +55,15 @@ public int concurrencyLevel() {
>> >     }
>> >
>> >     /**
>> > +    * For non-transactional caches only: if set to true(default value) the cache keeps data consistent in the case of
>> > +    * concurrent updates. For clustered caches this comes at the cost of an additional RPC, so if you don't expect your
>>
>> @danberidei yes it does make a difference for local caches. when this is
>> explicitly disabled (it is enabled by default) we don't add a locking
>> interceptor to the interceptor chain. So concurrent updates are very
>> un-predictable. I think (this would require more though though) that this
>> would only affect consistency, but I really want to discourage people doing
>> concurrent updates on a cache that doesn't have the locking interceptor in
>> the stack as this would open a can of worms.
>>
>> —
>> Reply to this email directly or view it on GitHub<https://github.com/infinispan/infinispan/pull/1520/files#r2382476>.
>>
>>
>
>
>
Please use Objective-C `NO` here instead of `false`.
This file shouldn't be committed, it it's generated every time by the build scripts.
Dunno... Reading the code as a "not matplotlib expert", it didn't seem obvious what ``s`` was. I have for rule not to use one letter variables as they are often confusing for beginners, and I want my code to be easily read and modified by anyone. Explicit is better than implicit.
I think this is probably a private attribute (users shouldn't fiddle with it).
Dan's right, there's a bit more than just using the thrift/unpacker going on here, and the feature vector has to be constructed exactly this way. It's definitely unwieldy though, so would be great to clean it up.

Create an issue?
@bp323 seems like I didn't need that after all (some other code was using it) - using `sakai_i18n` did suffice.
@bp323 The underscore is meant to emphasize that we're not manipulating the groupData variable that's outside of the function's scope.
@arnodeclercq This will cause an extra request for every single page which won't be good for performance reasons.

My suggestion would be to add something like the following to `sakai.base.css` instead:

```
@media print {

}
```

We should verify this with @bp323
Hi, thanks for the comment. That's on the TODO :p. This is only a working prototype, I'll make it more native to shogun in the following days. :)
This is going to fetch all categories and create objects just to count them, could you use a count query instead?
```ORM::factory('category')->count_all();```
Can we remove the commented values?
I actually wonder if this file should be in the .gitignore and remove from the repo. I'm pretty sure it gets updated or created when running `android project update`. Could you test this while I continue listening to music and blissfully ignoring everything? :-)
Why are we reducing it from 24 hours?
Die in Hell Timeline-Domain-Object!
Same as above. -p must also be removed.
Totally agree with you.  On my first pass with this, my intent was to be as "like-like" as possible with the implementation details.   My thinking was that, for most examples I could go on and on with alternative approaches in the .net world, so I thought to just keep it simple for now?
I'm a Python noob. What does the `:` here mean?
Yes, native constructors always return a blank new value when called without `new`. Non-native constructors generally only return a value when called without `new` in special cases, and that value is likely to be specified for the purpose of changing the return value when called with `new`.

Given this, it is likely that the return value of a constructor is valuable and should be preserved. For instance, the previous conditional case `child = protoProps.constructor` will not discard the return value of the `constructor` property. However this breaks down when someone applies `extend` to a native type and attempts to create new subclasses of that native type using `extend`. While this is not a documented feature, it is functionality described in #555 that is being preserved by the maintainers.

Preserving this functionality requires identifying extension of a native type and ignoring the undesired blank object that the constructor returns. I have not found a cross browser means of separating native types, so I am enumerating them. There are twelve native types (`Arguments` `Array` `Boolean` `Date` `Error` `Function` `JSON` `Math` `Number` `Object` `RegExp` `String`), but only some of them are constructible (`Array` `Boolean` `Date` `Error` `Function` `Number` `Object` `RegExp` `String`). Furthermore, some of them (`Date` and `Number` on WebKit) have methods that are not compatible with extension:

```
Date.extend = Object.extend; (new (Date.extend())).toString()
> TypeError: this is not a Date object.
```

The confusion that this could lead to is why extension of native types is not a great idea, but I include a special case for `Object` as a the case of most common misuse documented in #555.
should this code block read as Python?
Definitely agree, UnexpectedValueException is a better fit.
Is this the right documentation (the parameter is named code)
Related to this, you should add a migration which removes the no-longer-needed resized thumbnails.
either would work? If anything I'd have thought 'body' would be better optimised but I'm a JS nub.
The idea is that of least-surprise - if you just run "smart-proxy" it requires rubygems. Debian deosn;t like that though, so now you can run "PRODUCTION_MODE=true smart-proxy" and it won't require rubygems. The init script uses this for Debian compliance.

I guess normally you'd have "DEVELOPER_MODE" but I didn't want to break anything people we're already doing.
Why `value="value"`?
I thought that was camel-case, that's why I changed it :D
Yes I was thinking of making an ul. I don't know, never liked dl. But can make a dl for a change :)
```


Aha, this is the test of what it happens if it is invalid form. :-) So it tries to find template to display error with. :-)

Hm, maybe we should not require the field. Then it will work? Because in some way, we could say that not providing the field is disabling it. And empty string is probably the same as not providing it.

```

Sorry didn't read you post careful enough.  You already provided a solution here. Yes, not requiring this field fixes this issue.  I ```id:''``` means an absence of this field.
It would work anyway. I have no idea what is the problem, maybe plural forms in the header of po file are not correctly set?  But they are the same as in python .po file. I must leave it like this and add TODO here, otherwise the whole javascript crashes.
`as about_url`. But yes, like this.
UPS. This is completely wrong space for this comment. I wonder why is here??? You made a change while I was commenting. ;-)

This is for the if-block inside the authorization class. Where you have a inline if-then-else block.
Why if? No, do an `assert`. We are assuming that it will be authenticated. We want an error if it is not.
You should group imports. So this is another group, different than Django imports. So newline before it.
I think the input needs a non-empty value field.
Try <input type='hidden' name='discontinue' value='discontinue' />
Also, consider that leaving this in Creature allows it to be used by people in a creature script. Putting it in PetAI means it's not available.
Codestyle: Naming of public fields
Sorry! This is why your nitpicks are good.
First, we surely need to link the ZF2 docs about authentication here.

Let's strip all the form stuff and reduce it to
```php
public function loginAction()
{
    $data = $this->getRequest()->getPost();
    
    $authService = $this->getServiceLocator()->get('my_auth_adapter'); // let's use a custom name to avoid confusing users
    
    $adapter = $authService->getAdapter();
    $adapter->setIdentityValue($data['login']);
    $adapter->setCredentialValue($data['password']);
    $authResult = $authService->authenticate();

    if ($authResult->isValid()) {
        return $this->redirect()->toRoute('home');
    }
    
    return new ViewModel(array(
        'error' => 'Your authentication credentials are not valid',
    ));
}
```
We don't need to explicitly read from stdin?
This used to be inside the "useShards" if — will the row coming in always have this property if the DB is unsharded?
As mentioned above, probably want `node['mysql']['server']['packages']` here.
use the  ngx_create_temp_buf()?
Hm, but ``django-tagging`` and ``django-tagging-ng`` are not compatible, because they use the same package directory in site-packages ``tagging``...

Another package that is also dependent on both [django-antichaos](https://github.com/svetlyak40wt/django-antichaos/) simply ignores the alternative dependency fact and hopes that the user reads the description.
standard terminology nobs, k_vars
I'm not sure what you mean by interpolation.  Shearing is normally done by a direct coordinate transformation.  With SBInterpolatedImages, I guess this requires an interpolation, but not for most of our profiles.
OK, I'll do that as it was my erroneous edit...
And I'm really sorry, I should have flagged that in your script, I just realized you wanted to specify things in terms of FWHM.  I've been talking with lots of galaxy people who all use r_e, so that's where my mind was.  (Not that I did the coding.  But I might have suggested we don't need FWHM for sersic/exponential in our discussion, because people mostly use FWHM for PSFs but not galaxies.)
Removed and pushed in all deviates.
You mean a paper / textbook / website that defines the PS and Delta^2?  Or a justification for using P(k) rather than Delta^2(k) as our chosen input?  If the former, I could try to dig one up (there are loads, but we want something good and clear).  If the latter, it really is kind of arbitrary and we might in fact want to eventually allow specification of a keyword indicating that the input function is Delta^2 rather than P.
Typically we indent comments at the same level as the code.
Strange white space change here?
could do with some comments -- i guess idea is we keep either 100 tasks per entity/tag, or for up to 1 day ?

i suggest using a ConfigKey (which BrooklynProperties supports) as that is better typed and self-documenting

in future might want to configure this per entity but this is a great start
Do we really need these logging managers in the core library?
I just added a comment in the docstring for the method. Since XORing two bitstrings of unequal length is always an error (right?) I think we should fail loudly when it happens. If not, we could add a return value to the function and check it every time.
I can be talked into letting this slide for the moment, but this strikes me as a very brittle test.  Anything looking for exact strings tends to be destined to fail
Разве они не для многопоточных программ?
{} обязательны для if
Пакет должен соответствовать пути (пропущена папка wordCounter).
Объясни-таки, зачем нужен synchronized здесь.
I think you are right, interesting enough, media service does not use the URI of the request as the scope of the token. updated the design to reflect that.
This should be $mobile-portrait-width, no? I don't necessarily expect anyone to change the values from 300px/420px, but it's still cleaner if it all lives in the variable declarations at the top of the page.
Wrap lines to length 80.
Why are these functions all commented out?
I agree, will do!
Yes, it's probably best to combine those two steps in one function: check if r, c is within image and then check if mask is true.
I can change this now since I am responding to other change requests.
what happens if pos_label == None and labels == 2? does it break?
I think it's useful, even if only for advanced users, for instance see the usage with `fbeta_score` to set a custom value of beta. For for not making it private even if not imported into the `sklearn.metrics` namespace.
This is important - GBRT now uses ``'gini'`` for feature importances - this was a bug in the current code that @glouppe pointed out a while ago but I thought it was ok - sorry about that - feature importances don't differ much though.
Without loss of generality. Maybe write it out as there are not so many mathematicians around ;)
ok I will, I'll try to make another commit later
It exists indeed sorry. http://dictionary.cambridge.org/dictionary/british/cannot?q=cannot
The docstring is too short, and makes it hard to figure out what this function is doing without digging in the file. In particular the input and output arguments should be described.

Also, if we are going to have a Cython-level argsort in the scikit, it might be useful to put it in utils, so that other contributors find it and reuse it. The codebase is getting big and it is easy not to realize that such functionality is in the scikit.
please don't comment the old code. Remove it. Keeping the history is done using git, not using some really old-school comment-based system
what is the best way to write this so that when I  run first_level for all subjects (after running preproc separately, say) preproc won't run again? The way its written now, the preproc workflow's base directory is root_dir/subject/preproc and first level's base is root_dir/subject. If I make preproc like the resting workflow, won't this have problems with first level?
if the destructor is protected, thi object is gonna be deleted (when parent object will destroy childObjectMap_)?
La réponse est au format JSON.
totalRows is an unsigned __int64, the -1 != totalRows isn't correct anyway actually, should be:
> if (RCMAX != totalRows && totalRows > 0  ..
I don't think so. For a start in the default configuration (and many others) we tend to put the eclccserver on the same machine as other processes, secondly eclcc itself may use multiple threads for the compile phase. Thirdly, it's really memory that is liable to constrain the number of eclcc processes you want to launch before CPU does.
Not a bug - a comment for future thought.
We need to be careful that there aren't too many different size buffers allocated, since each different block size will use a whole page (currently 1Mb).
Due to that change my builds results in "Skipping Cobertura coverage report as mojo did not run". I execute cobertura via ant and not via the cobertura-maven-plugin so i can get a complete report over all my submodules. Can you use another check, instead of asking the maven build? For example, check of existense of a file?
Awesome, thanks for the explanations and experience sharing Peter!
This won't work, ``operating_systems`` is a list of dict. 

<pre>
if os_name not in (os['name'] for os in operating_systems):
</pre>
I have no objections to your implementing this.
@cordoval it is highly recommended, but it is not required. You can use the bundle wihout Zend Cache (and you could want to write a Doctrine Cache implementation instead of the Zend one). Only required deps should go in the require section
The column type is Timestamp in the db. Spring JDBC automagically converts from java.sql.Timestamp for us. So this is the cleanest way to represent "now" in GMT.
That makes sense. I didn't even realize Django had a reversed template
command. I'll have to dig into the 1.2 docs.

On Tue, Jul 17, 2012 at 10:24 AM, Eugene Fang <
reply@reply.github.com
> wrote:

> > @@ -56,7 +56,7 @@ <h1 class="end_header">FRC Team
> {{team.team_number}}</h1>
> >                           <span class="caret"></span>
> >                         </a>
> >                         <ul class="dropdown-menu">
> > -                     {% for a_year in years %}
> > +                     {% for a_year in years reversed %}
>
> I was thinking about doing it in the template because, for example, if you
> have a list that appears twice on screen, one a->z the other z->a, you want
> to only reverse it once in the template as opposed to twice? I don't know.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gregmarra/the-blue-alliance/pull/136/files#r1180688
>



-- 
http://grgmrr.com
860.263.9018
What's with these changes (and those below)?
That's correct. But if they are part of the admins for some reason, they shouldn't be counted towards the limit.
since there's no 'else' here this could infinitely wait if packet_len doesn't get to 0.  Would it make sense to change it to:
    if packet_len ==0 or count > 5
    # or just add an else statement
    else
        break
    end
This is kinda a weird habit we do.  We tend to leave BadChars => "\x00" in there when there are no bad chars at all.  If this is the case for you, please feel free to remove the Payload option.
this blank line should be put back (two blank lines between functions)
Indeed, so we should remove this line, and raise instead an error if `multichannel` is True and data.ndim < 3.
Why not use ``tform`` on the corners?
This check seems fragile to me. Is there any harm in just letting the whole method body be the ``for`` loop & just letting the ``delete`` do the no-op?
SHould be form_rest(form) ?
The best way to get the record name is to do `elem(other, 1)`
I don't know node very well, sorry. But I think it is reading up to 1024 chars from looking at their docs. So I suspect this might not work if the input is bigger than that.
Why the double quotes?
We could import the view source page class inline and return the page object.
In what situation would you want to pass these in rather than computing them?  Shouldn't they at least be optional, and computed if not supplied?  What if we need a radii to the fifth in a future version for some reason?  Do users have to go back and change everywhere they constructed an ellipsoid previously?  Ok, so users probably won't actually use this constructor, and will use something like fromRadii instead, but then why do we have it?
Matt means `Array.isArray(uniformValue)` actually.
Instead of having checks like this throughout, would it be better for `fromPoints` to return `undefined` or a sphere with a radius of `-1.0`?
If we queued it, we'd have to have some way to dequeue or reorder pending tasks.  Once the message is posted to the worker, we no longer have any ability to do that.
I'm not sure about this description.  This function computes the geodetic surface normal at the cartographic position on the surface of the ellipsoid, that is, the normal of the plane tangent to the surface at the position.  It is the "correct" normal for non-spherical ellipsoids.
"If use case Y, then the other's methods will be overriden with the self's methods, right?"

If `other`'s method is overridden by `self`'s method, how could `self`'s method call `other`'s? It will only result in infinite recursion.

Take a look at the first lines of `self.dragMove` and `self.dragEnd` below, if you haven't.
should probably have copyright (applies to all, but I'll just say it here.)
Should we also check if (point != null) like in line 124?
move var i out of for loop
Same as backgroundDisabledColor
Why a body do dat?
reconnectScheduled is more clear
Perhaps it would be best to move the dispatching stuff to a another base class from which this class and the BaseReflector can extend? that way no duplication is necessary within this component and we can simplify the assignment in phpDocumentor\Application
a FQSEN is not the same, a Type is (as defined in the PSR draft) a collection of keywords that have no relation to a Structural element
Lo de la lista me parece mejor.

Pero sólo para anotar es mejor usar `<br />` que 2 espacios para hacer
el breakline, hay editores que eliminan los espacios en blanco al final.
`var entity`
this should be moved inside the loop
no spaces around parameters.
I thought you only had to put the div onto your page now. Also, obviously, you have to load the JS, right?
Agree, it's important to display "DokuWiki" somewhere for marketing purposes.
Also, this would provide a clean migration path to the fixed implementation: Just schedule the `isSafe` and `areAllSafe` for deprecation, and add only `isSafelyCallable`.
"Narrow strings don't have length." dstrings do, so this is incorrect. It's also bad English, because the subject (String) is singular, whereas the verb (don't) is plural.
move to after if/else
constructor="GenericIfComponent"
added TODO - for now I wanted to leave this around  for debugging purposes.
would have helped if i looked at the code :)
個人的な好みですが、`class FlattenOutput < Output`直下にある方が好きです。
Operator at end of line, not beginning; align arguments.
still unneeded new lines
Maybe add the name of the interface to the previous sentence and skip this one?
Are you requiring autoload in your index file? If you are, you shouldn't need this require. Remove it, and test it out.
Please use the PmsConfiguration getters (and register appropriate event handlers to call the setters when values are changed in the GUI), rather than probing the GUI. For starters, there is no guarantee that there is a GUI (headless servers).
Assertions are valuable tools for detecting errors.
It's not a presenter.  It only enforces an interface.

On Wed, Sep 26, 2012 at 11:56 AM, Joannou Ng <notifications@github.com>wrote:

> In app/presenters/woople_theme/content_item.rb:
>
> > @@ -0,0 +1,8 @@
> > +require 'explicit_delegator'
> > +
> > +class WoopleTheme::ContentItem < ExplicitDelegator
>
> Can we call it ContentItemPresenter?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bigbangtechnology/woople-theme/pull/99/files#r1697076>.
>
>
I think the more common `id<Protocol>` example would help. Perhaps also add that, where possible, extend a new protocol from the root `<NSObject>` protocol. That would be nice...
change this to

```
sources = ['abyss-%s.tar.gz' % version]
```

This makes updating this to a new version a one-time change, and it will also make the `--try-software-version` command line option to work as expected on ABySS. That option only tweaks the `version = 'X'` in the easyconfig, and trusts that it is being used throughout the easyconfig file.
Aye, good point. I don't write much Java code. This place seems abandoned though.
I believe that would change the default behaviour from listening on all interfaces to only listening on lo. According to this documentation - http://www.rabbitmq.com/configure.html, it looks like just setting it to the empty string would be the right default value to not change the rabbitmq module/package's default behaviour. Would you like me to make that change instead of the UNSET hack? It is a slightly more intrusive change to set that parameter to a reasonable default than just to not have it there if unspecified, though the change would allow the good people over at UNSET.com to use this feature :). Let me know what you want.
`self.companies.where(id: current_company_id).first`?
Was a description and name supplied for the widgets in the patch?
This is a models-level programmatic API; it shouldn't accept a string name and version, it should just accept a `ProductVersion` instance. The management command should be responsible for taking the string name and version and getting the `ProductVersion` instance, since that's a concern that's only relevant to command-line usage, not programmatic usage (where you're more than likely to already have a `ProductVersion` instance anyway).
I still don't agree with this. Dereferencing `0x1` is *not* dereferencing a null pointer.
setting for variable range gaps
What, not even a simple javadoc? c'mon... :-)
looks like a race condition here... another thread may have passed the *if (this.replyDestination != null)* check above and yet not made it here yet. Meanwhile the thread that got here first (but after that other thread passed the null check) would be setting the instance var ref itself - only to be replaced by that second thread's created queue.
probably make more sense to do the element.webkitMatchesSelector null check as the first thing.  Might even add an empty line after that.
Removed.  Thanks.
I think this needs to go away, now that it's all in the form code. Plus, the URL is inaccurate?
I get burned on readme and meta more than anything else.  Diego,  can you publish example templates or clearer requirements for these things.
@apex13 Good catch, updated the naming.
Thank you, I see it now. Will fix it in the next commit with Brain Link modification
The SecurityContext is already injected into the AclHelper service, so there's no need to pass it in...
if this is ever null, then move it after the null check!
这种用temp_file中转内容的方式太低效了。
我看pdf2text_content()这样的函数是你自己从textlib.php修改出来的，是吧？
可以把整个textlib.php修改一下，改成直接处理内容，而不是读文件的。但不是你现在这种copy&paste代码风格的修改。

首先把原版textlib.php导入，提交一下，提交信息中说明这是import original version from crot
然后修改为直接处理content的版本，再提交一下，说明这次修改了什么（传content注意效率，因为有些文件内容可能很大）
做好这两项工作，再修改moss.php，提交
这样能把代码的演化过程清晰地说清楚，并便于回溯
Awesome! You taught me something new!
Actually, on further thought, I'd rather that the main getMessage blocks and if needed, we define a separate non-blocking getMessage call. The reason being that the expected behaviour of "getMessage" is that it will eventually return a Message regardless of the current queue state.
I was a little unclear...  I agreed with you, the main content on left is good

Sent from my iPhone

On Feb 16, 2012, at 7:18 PM, Tristan O'Neil<reply@reply.github.com> wrote:

>> -                 </li>
>> -              {% endif %}
>> -              
>> -            {% endfor %}
>> -       </ul>
>> -      </div>
>> -       
>> -      <div class="well sidebar-nav">
>> -       <h3>Past Events</h3>
>> -       <ul class="nav nav-list"> 
>> -          {% for event in site.categories.event %}
>> -            {% if site.time > event.date %}
>> +  <div class="span8 well">
>> +    {% include event.html %}
>> +  </div>
>> +
> 
> For consistency reasons I think it should be on the right. In all other cases the sidebar is on the right, its a bad user experience to move common elements around.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/btvwag/btvwag.github.com/pull/4/files#r460138
Ayuda en hacer debug , fijate cuando haces break point y mira el estado de variables
i think reassigning it is the way to go ...
role="option" is missing
Maybe we can change this function to take an optional third parameter which is a transform function which would be parseInt where I use it, and nothing here?

And/or as a more dramatic change, I've been getting a code-smell feeling about all the duplicate string parsing code being strewn about the code-base to extract account and folder id's from the message suid's.  Maybe it's time to just create simple helper functions and do performance tricks later as required.  (The easy non-trick is to pull it out into a const at the top of the module, the more debatable trick is to use something like the closure compiler that we could tell to inline those specific functions.)  In that case, you could just pass extractMessageIdFromSuid as the second argument to this function.

No need to change this immediately, but I think it's fertile ground for refactoring and am interested in your thoughts.
Not sure why this is in awkward parens. If it's just for PEP-8, please don't & just let the line be a little over 80.
For reference: https://github.com/django/django/blob/1.3.X/django/db/models/loading.py#L182-191

It would require us to use the Django `app_label.Model` syntax instead, so I guess it's not a viable option, though.
is_end()
You can also do this:

    if (~historyStackIndex) {
please, no version change in this feature branch
> wondering whether foreach sb.append(...); sb is some kind of for-like syntax

I see that. Was wondering, though, whether people might be tempted to read it as `foreach { sb.append('*'); sb }` in which case it's no longer surprising that it compiles. The multi-line variant makes it more obvious that the function body really is just `sb.append('*')`.
> But using update_attributes allows anyone to inject their own parameters and potentially change other user's ratings and even rate puzzles which they haven't solved.

I don't think he can do it because I have a validation in Rating model here https://github.com/samnang/puzzlenode/blob/6f6c44e4d1e3caa656958aa3f5c2fda9d6e4fd24/app/models/rating.rb#L9
Indention is messed up.
undefined checks should be done using typeOf
I'm not sur what logical reason there is for separation. We looked through the file and it seems like all of it is tied into the UI of the contextmenu. I don't see the advantage to having two single purpose 300 line files instead of one single purpose 600 line file
Lint had been complaining about too many var definitions.  That's why I had changed it to this.
This is the exact line to put in - (this will pass lint)

 window.BlobBuilder = window.WebKitBlobBuilder || window.BlobBuilder;
Can't do this.  This is a BC break.
1.0.21 worked for us