labofsahil/github-event-dataset-2012 huggingface dataset:
wrong formatting. It should look like this:

```php
switch ($cacheSize) {
    case 0:
        $this->_cacheSize = 'NOCACHE';
        break;
    case 1:
        $this->_cacheSize = 'CACHE 20';
        break;
    default:
        switch (true) {
            case (!(is_numeric($cacheSize)):
                $size = 20;
                break;
            case ($cacheSize > (ceil(pow(10,27) - $this->_initialValue))/abs($this->_allocationSize)):
                $size = 20;
                break;
            default:
                $size = $cacheSize;
        }
        $this->_cacheSize = $this->_cacheSize = 'CACHE '.$size;
}
```
Could we add  `, '~> x.x.x'` here? To make sure the new gem is installed?
Nope. Reduces clarity.
Nit: s/closed/close/
Awesome. I've seen people sometimes set SSL_VERIFYHOST to true, which causes it to actually be set to a pretty insecure 1 setting. Adding validation will definitely prevent that.
Don't delete this file, please.
I don't understand why this was moved from testinit.js to this file? What was the benefit?
When in C++ code the recommended way of raising errors is to throw a C++ exception rather than wrapping things in PG_TRY blocks and calling ereport.
So, this is not the exact number of records, but the fraction.  If so, the first line of the comment is confusing.
Please use only int16_t, int32_t, int64_t, uint16_t, ...
Everything related to futures is now handled in `_unwrap_result()`.
My thinking here is that I want the test to work exactly the same whether or not live indexing is implemented/enabled. So I want to do something that resets the index to a known state regardless of live indexing.

Do you think this is something that shouldn't be worried about?
I think usually the user will want to see how the report looks like and check the suggested config if supplied (so he can edit it if something is wrong or post a comment). He can also see the shot name of the configuration on top of the show_issue page (which is a link). 

What do you think?
`CC=#{ENV.cc}`
adding std_cmake_args results in this error during compilation:

http://bugs.mysql.com/bug.php?id=60449
Does this have a hard dependency on gnupg 1.x, or can 2.x be used as well? If so, this should use a `Requirement` instead, like the `signing-party` formula does.
```js
setTimeout( clear_old_warning_messages, ActiveOptions.interval);
```
Ce fichier non plus n'a pas sa place sur le git. Tout comme les autres logcom.server...
Au passage, est ce que tu pourrais me donner les changements que tu as fait sur la bdd? Parce qu'avec les changements, il y aura surement des trucs à revoir.

Aussi, pourquoi recommiter les librairy? Et même les librairy linux alors que tu es sous windows?
To clarify, leave it with the path unspecified.  Let include path do it's thing.
Agreed. Created issue https://github.com/praekelt/vumi/issues/329 for this.
We should probably avoid random whitespace changes.
Ligne de **commande** mais je tendrais plutôt vers "interpréteur de commande" voire "shell" qui semble ne pas être traduit en français.
Why are you using .each_index here instead of just .each?   I think you can clean this method up.
I think you should add some documentation to this register function and also to the other function in this module.
The `_get` function in tdb_cassandra seems to be unused elsewhere, and doesn't seem to add anything that `getattr` doesn't supply; could you change all the `_get` calls to use `getattr` instead?
If I'm a mod of *any* subreddit, I can always contribute to *all* wiki pages?
I think this should be fixed when we move to python3, since `subprocess.check_output` isn't available in 2.6.
How about using bug as a wrapper which queries bmo or bap depending on how big the ID is?
We normally have a newline here:

```cpp
class endian_buffer 
{
```
Nice approach! Didn't know about -ng before, great way!
> Is this not what `--continuation-indentation=4` would do?

Well spotted. Do I already need glasses for documentation now?

-- 
Mark Lawrence
In my opinion all links should open in new window. I think 'window' field is unnecessary.

If we delete 'window' field from model, we can change this ugliness ('_blank' if self.window) :)
should be only one name: modDir or modRoot
It doesn't, but seeing as that's apparently unused it puts a warning in the build process - or is that supposed to happen?  I can pull that back out if it is.
It looks like when this is called, it's only passed the siteId. Do we need all these arguments, then? Or do these actually need to be passed in? If so, I think it would be nicer to pass in an options object instead of so many arguments.
Thanks for aligning these :-)
externalize string ``{0} JSLint errors``
cmenu was already verified above, and it hasn't changed, so no need to check again.
I did a regexp scrub and didn't find any other problems.
1) In the details of the Products Backlog it was specified to use Ctrl+Up/Down as key bindings. So, I should do the following?
 - Windows: Ctrl+Shift+Up/Down
 - Mac: Alt+Up/Down

2) I'll reorganize the menu as you've specified.
Cool, will localize, again was trying to follow project testing style
Can you just align/format this plz :)?
Ah! That was a test. I was getting that because oAuthService wasn't injecting. I'll pull that out too.
Rename test (see above).
I vote to let it as is for now. That's sad, but it "worked" until now and we have more urgent things to do...
So if I understand correct, language should be stored in user model which would be default for that user and he could change it in his personal settings? So the best way for user to select his preferred language would be when registering. So the "footer language selection" would be actually for guests who don't have account?
Constant strings should be in `'`.
Is this comment needed? From the name it's not clear to me what this is locating.
Are the icons data-uri? I know that we reference `resources/bg.jpg` a few times in this file, so I followed suit.
@secretrobotron : okay IE9 is trying to make me look crazy. last week it was totally throwing. not anymore. will fix.

@jbuck : We think the warnings will be  helpful when malformed json or json that's not as we expect it is loaded.

also :v:
This *might* be dangerous.  In the DOM when I was doing Pointer Lock stuff and having to work with Mouse Events, you had to be really careful about someone holding a long-standing reference to an event object.  I know people won't generally do it, but there is nothing stopping them from holding a ref to your event object and doing something with it you don't expect.  Given that you've made all the properties read-only values, this might be safe.  But ponder it or if you should just clone.
I pointed this out a bit before (for a different place, I think), but is it not possible to use `trackEvent.view.resizing`? I'm not sure if this event happens before `resizing` is set.
I personally would prefer to keep only `target`s in `make.js` to keep it more readable -- auxiliary functions can go in external modules.

For this PR, let's just create an external folder `/external/carriagechecker` or something like that containing the module.

To prevent folder bloat, we can start a separate PR where we put all build helpers into a single folder, e.g. `/external/build-helpers`
This is not defined on OS X so it throws a timespec redefined compile error. You'll need to a more compatible way to handle this.
Yeah, it's probably unnecessary, but you may notice that the instance variable was not set at all in the edit view before this PR.  The fallback is retained through an abundance of caution.  Seems pretty harmless to me.
this name is kinda unintuitive. something like traceExists or tracesExist would be better
Why are you adding crl_regen here? I dont see deletes, maybe it is there twice?
Yeah I have a separate pull request for that, #67, I thought for this one I'd stick with the same style used in the rest of the file, and if there were any sign of #67 getting accepted I'd update on top of that.
Fixed in #7
yeah i agree, was thinking that at the time but just copy pasting for speed.
See 4830b92
just [this](https://newrelic.com/docs/php/the-php-api) but I can tell you, you don't have a way to handle level: that's why it's not there
Why do you think they should be removed? I think they are useful when dealing with code that uses both bigints/bigreals and normal ints/reals.
Its not overreacting, clean source is important, down to the last detail. When you let things slip you eventually end up with a huge mess.
Does this apply to all formats, or just votable?  It might make sense to move `overwrite` into the core so that all formats have that.  As it stands, this is handled in `io/votable/connect.py` (so outside of votable itself) anyway.
OK sure, that sounds good.
Good point, I moved this description to its own section.
I actually think it's better to have it in separate and at the root TOC so people notice it more easily.  Perhaps add a ``:ref`` (http://sphinx.pocoo.org/markup/inline.html#cross-referencing-arbitrary-locations) at the bottom of ``logging.rst``  that references the API section of  ``configs.rst``?
Actually ignore me, I've just realized that these setters will of course change the internal representation. I have no strong opinions though, so could be convinced either way.
HTTP uses ``URL:HyperText Transfer Protocol`` as description, so we could use ``URL:Bitcoin Protocol``.
This should not happen inside the forearm, IMO
There was a StaleElementReferenceException occurring on Chrome against the sign_in_locator at signout because the page refreshes. This locator will reliably allow us to determine the logged out state because it's only found after the page refresh. Note that I've also changed the explicit wait in click_sign_in() . I've detailed this strategy for avoiding STaleElements in the WebDriver blog post too.

sign_in_locator then removed because it's not being used.
The implementor of the view should be agnosotic to the pipe.  My view code shouldn't care whether the data is being passed from the server to the client directly, via pickled objects over 0MQ, or via web sockets to HTML.

If we force the user of an array image to guess how to encode their array so that it can pass over the pipe (particularly when they shouldn't know the client's capabilities) then we're losing a lot of re-usability of code.
identifiers: ah, i was not aware of this. in xml we have no such createphp-space names, there is only classes and the type in the rdf mapping.

creation: i think the crucial bit is getTypeByRdf. i don't see how we can properly handle creation without that method - we would have endless chicken-and-egg problems. if the rdf type is just something somewhere inside the array of your type, then there is no unambigous mapping. now the array driver could either be changed to use rdf as key and avoid that, or you could extract the map from the array and warn that creation will not properly work. or have a manual map to be used for creation, as the other types are probably never used for creation but just for reduced info rendering.
I am still not convinced that we want to add this capability.  It adds a function call to every single keystroke in the CodeMirror widget.  I am probably +0 on this but would like to get @minrk @fperez and @takluyver s thoughts.
Oh, I see, it's added in `magic run`.
Done. This also allowed me to get rid of the os check - whatever I was basing this proof of concept code on had something similar, but I see the `IPython.lib.inputhook` takes care of this for me.
return the only hardware we're using no?
I don't think there is an actual danger here, but if you are going to use the `weakptr` version of this function, you should probably move the call to fetch it closer to the place where you actually use the object. You make a lot of intervening function calls before using it, and technically, if one of them were to free and reallocate the config, your weakptr version is not refcounted and it could be freed.
You should come up with a better name. But yeah, this is a class act.
lets clean this up please?

we do not need such huge list of params ..
sure that the whole thing needs its own bundle, or should we just put the common bundle incl. jquery onto the page, and add the landing_devices one separately for stuff that's specific to that page?
can we get rid of that?
I'm not so sure, actually.  I think in the automation submission, always creating dupes is correct.  But if a user is coming in via the UI and wanting to "pick up where they left off" then they don't want to be creating dupes.  

Am I understanding you right?

But, yeah, I think the display of counts for results should be fixed to show distinct based on env/user in the results area.
I don't get this. Is there a particular reason why it's left commented out?
That's what we discussed on the phone. I chickened out and decided to just make the CronTabber class have config options.
I didn't like it in the rabbit hole. It felt like we were shoehorning configman almost against its will. Also, I feel like I'd rather release something, let it settle and **then** upgrade its features. 

No, the apps are NOT stand alone. It's deliberate. You're supposed to use `./crontabber.py --job=some-app` instead. That's so that the dependency stuff isn't ignore and also so that we record (every time) the last time it was run and what (if any) errors happened.
Doesn't ej:valid give you the expected type? Wonder if that would help simplify this at all? I'm not clear why we have to match specifically on the keys here. Feels more coupled than I would like.
no need to send null back as 2nd param.
Ah wth, missed this one..
Instead of checking on additional servers as they are added, use the regex to check the site

Change line 622 to
```
m = RE_NEWZBIN.search(uri)
if m and m.group(1) == 'newz' and m.group(2):
```
These two `Map.class` types are for client and server error deserialization. The result of that operation gets stored in a `ClientErrorException` or a `ServerErrorException` as an `Object` for you to cast? I'm not really sure what to do for this case...
файл можна покласти поряд і вказати відносний шлях. приклад https://github.com/stfalcon/PortfolioBundle/blob/master/Tests/Entity/Resources/files/projects/preorder-it/data/index.png (тільки непотрібно такої здорової ієрархії каталогів)
I was on the fence on that one and looked for prior art in Symfony codebase. I'm super happy to make this change if that is what is desired.
Maybe `toggleLeftPanels()`.
You should not remove this test. The constructor should fail in this case.
Sorry. But we need this feature in the form type extensions.
should we really add visibility in interfaces? everything is public here
As per the above comment, I'm not asserting the same twice, that already exists.
Thanks! For the ClientSpec, you need to have the redis binary installed for it to run
Thoughts on a view helper?

```ruby
def help_out(event_owner, recipient)
  if recipient == event_owner
    'me out'
  else
    "out #{event_owner.name}"
  end
end
```
这里有一处乱码，是不是编码的问题？
Decompiling the full python.pak is hardly noticeable on my not-so-fast laptop. So I'd say performance is not a problem.

Should the yesToAll/noToAll of the python files be shared with the one from the pyc files? I think not doing so will confuse the user.
Testing ubind_all tests the single unbind call also.
Note that we dropped the stat boolean parameter in favor of just checking for a non-NULL MaketxStat argument.
'Cartopy' or 'cartopy' - I don't have a preference, but we should be consistent.
@Baachi: the MongoClient class doesn't exist in versions prior to 1.3.0, so checking for Mongo is strictly for BC.
That's a good point. A comment explaining that will prevent someone from doing a drive-by cleanup and removing the protection later.
I had in mind something that this more in line of two different ways to do "same" thing...
$('img:first').attr('src') || '' 
would be enough. 

+1 for the absolute path check. I am not sure about the code you pasted, I could test it tomorrow.
These aren't drawing to the page, (no <%= tag)  Do we still need them?
replace this rule with `XXX` and a background image, just like the same workaround employed (to workaround a Gecko drawing bug)
So this does pass `gjslint` O_o
I was hoping you could change this to 'sdcard' and just use this one device storage object for everything. But it sounds like that didn't work. Sorry.
/me stop here the anonymous spam
move these `addEventListener` to `init`
* Definitely possibly for state to have keys not in participant.labels (1/2 the root of the problem)
* N/A
* Yes, hence the switch from dict[key] to dict.get(key)
We have a spec test verifying the return for commands invoking -w is JSON. This should make it easier for us to validate this requirement is met later on.
JUnit container should not be needed. JUnit should be added through dependency. #6
I adjusted the description and copyright text in: https://github.com/skyluc/scalatest/commit/fbbd17cc0ab98e472f9f7c22db9af8978046d6cd
Thanks man. The reason we use edge veritas is mostly because it accepts a hash in #restrict :)
please use dashes or underscores instead of spaces
Should this not be a method of ```Grouped```?
I haven't tested what happens if extra options (undefined) are sent to stylus, but if sending extra options doesn't raise any error then this can be changed (not needing `inline` as a temporary variable)
How about this: (though I don't know why the os which os_version is magic number 3 doesn't need this check)
```
if not os_version == '3':
    logging.info("Check fdisk info after conversion")
    fdisk_info = check_obj.get_vm_fdisk()
    if re.search('/dev/vda', fdisk_info):
        logging.info("SUCCESS")	
    else:
         raise error.TestFail("FAIL")
```
BTW, I guess you can't get which test step exactly failed from the test result summary (such as client/results/default/status file) with this kind of error message. Maybe error context mechanism can a little help.
I've since patched the `set-schema` API to clear the cache on all nodes on modification.  Now if that multi-call fails to hit a node then that node will still use the old version.  Currently this is only used for repair so that it can get `n_val` which really shouldn't change but I suppose there is some very rare edge case here where a user changes `n_val` and some of the cache clear calls fail.  Search also updates the bucket props when the `n_val` is changed so I could get it from there instead but I still need to do the list keys.
There are still some invocations of `bld.getSourceStamp()` in `buildbot/status/web/{builder,console,status_json}.py`.
did you try out the devise update?
user = key???
Heh...  add `.DS_Store` to gitignore, and yet this branch adds several `.DS_Store` files :-)
Am I missing something or does this unconditionally set `prev = null`? (According to the last clause of the `while` loop above.)
Yes, it helps simplify our config file management.  Doing it this way lets us keep our logstash configuration (not sensitive, and identical on our development and production networks) separate from our AWS credentials (sensitive, and different on dev and prod).
This bug was caught by  nodes global leak detection when I changed the wei weiqi-model's were being exporting.
Have you seen it being GC-ed before it is being called?
If yes, can you provide some details here?
Otherwise it sounds as a safe choice.
Can we have a followup bug for this. It's not really part of this change it was there already.
compellin -> compelling
needs to be comma separated
What I meant was that you should update the comment.
Fixed in b2b4822
Ok. I agree with your current implementation but have some comments:

* we shouldn't mention parent builder and rails 3.0.x if it doesn't help to make clearer implementation.
* we need to handle the case if association is not found, ArgumentError with a descriptive text will be good as it means user either misspelled association or forget to specify `accepts_nested_attributes_for`.

Thanks for your contributions!
To be specific, we should replace "old" with "SLE10" in these names.
Provide a comment showing what the output is?
Since 5.3.9 is_a() supports checking strings. Perhaps you could use that instead, relying on reflection for earlier versions?

http://nl3.php.net/manual/en/function.is-a.php
:+1: with the RenderListener to keep consistency & readable.
The above were correct -- it's a common idiom in ZF and ZF2 to align operators for readability.
You're right, I forgot to mention the pattern was:

static $matcher = null;
if (!$func) {
     $matcher = function () {};
}

Either way, micro-optimization.

-ralph

On 6/24/12 3:40 PM, Pádraic Brady wrote:
>> +
>> +    public function escapeHtml($string)
>> +    {
>> +        $result = htmlspecialchars($string, $this->htmlSpecialCharsFlags, $this->encoding);
>> +        return $result;
>> +    }
>> +
>> +    public function escapeHtmlAttr($string)
>> +    {
>> +        $string = $this->toUtf8($string);
>> +        if (strlen($string) == 0 || ctype_digit($string)) {
>> +            return $string;
>> +        }
>> +        $result = preg_replace_callback(
>> +            '/[^a-zA-Z0-9,\.\-_]/Su',
>> +            function ($matches) {
>
> Had an error using the static keyword - the function keyword wouldn't parse with it. Ended up pushing the closures to protected variables instead.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/zendframework/zf2/pull/1583/files#r1039331
>
Would it be possible for FormDateSelect to additively extend from FormMonth, rather than subtracting out the "day" functionality?
Interesting. I’ll keep the lastIndex approach then.

By the way, you can compare char codes like this:

```javascript
"a" < "b" // true
```
yes.
Ruby is interpreted language, so it's generally good idea not to write extra code (there are exclusions, like sort_order in users_controller; it definitely may be rewritten, but now it is more clear what is it doing).
As well nice to know that (a + " " + b) is much slower than "#{a} #{b}". In this case it adds readability and not used too much, so we don't care about execution time, but in general case you have to know it.
Could we leave it as a Tuple? Needless cost for doing reflection over dynamic
This line will always return false as FindJPEG is set to REQUIRED for the renderer.
You can remove this. It's already in the base.css
please, consider adding logging for this, also, using a False value rather than None seems better to me.
Hello!

After some consideration and careful watching of other eZ Publish Community Builds and Developers and after reviewing git branch support ... 

We think it would be best to create a git branch of AdminAid's current eZ Publish 4.x Support now rather than later.

With the remarkable ease of use of git branches you can provide better eZ Publish API Support with your extension repository.

Also we think think after you accept this pull request it would also be a good idea to create a git branch of AdminAid's more recent eZ Publish 4.3+ Support.

When the next version of eZ Publish is released to the public you will still have the option to create another branch to address template support as necessary.

Thank you for your continued support! Please let us know how this finds you.

Cheers,
Brookins Consulting
That's fine, maybe I didn't explicitly state it anywhere, but these days I target 2.6 and above. Especially when Python 3 support is desired, 2.5 is simply too old now.
Rick, could you update options for tabs here (take a look at the latest master). I had to fix the label issue, so your demands block needs to get up to date.
The test was not skipped, but the query was cached. however this still doesn't explain why the test passed
+1, they aren't used.
On Mon, Sep 10, 2012 at 3:16 PM, Kenneth Hoste <notifications@github.com>wrote:

> Are you OK with that, and fixing the pull requests you made that way? I
> don't expect that too be too hard...
>
Actually, that is very time consuming due to the requirement to provide
different pull requests,
hence distinct commits etc. I'd say, let's not invent a problem where there
wasn't one :)
To keep it short, let's drop one of the two: a) multiple pull requests b)
provide ebrepo files requiring manual edits

Sure, I'm all in favor updating them later on, together with the rest of
the current .eb files,
and then making it a standard to only provide the ones coming out from the
repo.
Part of the same business could be perhaps to have them pass through
pythontidy or such)
Use double quotes here.
Move to previous line if it fits.
"好きなエフェクト"、"☆を付けたエフェクト"...
do we not support fractions of a second?
For consistency sake, can you give this two args? (I realize it works with one, but `( true, message )` will be clearer for the next dev :)
Preps the `validate_mpl_graphs_transform_results` method to be used in a polling pattern.
Checking in this direction produces duplication of the "Drop Claim" line. It would be better to switch the test to test for the other direction, ie

if collection is open or user is mod
  show fulfill link
end
show drop claim link
Also, if user puts mixed string in the source field:

"http://www.example.com from Clark Kent"

auto_link can still nicely handle it.
I'll admit, I have simply copied the comment from the text() function above.
so you could actually login with that password instead of the key? in this case we wont need a key then.
Will fix, thanks.
Please don't name the dashboard instance here. Just make it a general documentation.
Well, if you uninstall brew using what is apparently the standard script (https://gist.github.com/1173223), it LEAVES /usr/local/.git in place. And because of that, brew will not get reinstalled.

Frankly it's sort of a mess. Adding the comment was the best I could think of.

If you want to try it, you would need to uninstall homebrew, remove your .dotfiles, get a new copy from the repo, switch to this branch, and do "thor new".
Add transforme this part to a module in vs-modules
If I need extra space I usually just do `\ ` (a backslash and a space) instead of `\quad` because it seems to make more sense. This works fine though.
> can of course be implemented in the session handler but the result will be the same

yes, but it will be cleaner :)
this is the modified version of _getBaseURL
the following is my test case.

input
> output
What do you think is better I think next? 
> class JSONConverter:
I'm agreeing with Sia , I think the initial review count should be picked up and then in this assert check that it has increased by 1.

Currently on -dev I can see the count for this value is one but the test is choosing this one for the test.

There is some disparity and potential bug going on here.
Will fix and resubmit.
To Clayton's point, this logic could be modified to load the static ruby config stub from a directory that each plugin owns.  That would definitely be cleaner from an RPM perspective.
Anything interesting in the XML should be inserted into the properties hash.  I'd recommend passing `osr->properties` to `parse_xml_description` and populating it there.  Interesting properties include:
* Creation date
* Device model
* Device version
* Objective
* Aperture

etc.  The application should never have to resort to parsing the XML itself to get some information it wants.
This is a weird change that we need to discuss.
I've moved most of the logic into the API.
I had a look at some other REST endpoints and it looks like we're doing the Array checking in the REST code.
is setting the `uri` to a default value above needed if it cannot be empty any way?
It's copy-pasted from Java.. It's the contract of sets, and Scala sets also return `true` if the element was not already in the set
One complication (and the reason why `Settings` objects were used before) is multiple output directories (for multiple source directories, like `src/main/scala`, `src/test/scala`, etc.).

I think the proper solution would then be for the `Sbt` builder to call the compiler several times, one for each source folder (as I believe is done in the command line Sbt. One benefit is that errors in tests won't prevent bytecode for being generated in the main sources.
I'm +1 for this, but as @stof mentioned that DriverChain shouldn't force advanced drivers, I did the check instead of an exception.

I see plusses in both situations, but feel that it may cause more problems if we don't add an exception here since the most common use case is to warm-up a cache, or work with all configured metadata (as I am doing).

Let me know what you feel is best and I can include it.
`3-1` 看上去虽然比更适合，但是操作难度更大，以社交类表单的兴趣调查为例，假设已有 12 个选择项，`12-N`，如果这时候增删选项，意味着所有 HTML 都可能需要改，如果需要改动顺序，那更容易改错。
Fair enough. I just rarely see someone course an Int like that. Integer() just call Integer.new underneath?
What about a `@link` to the GeoIPs website?
this `disabled` loop variable overrides the `disabled` dict on L:89
Да, моя ошибка, не дочитал что это NSMutableIndexSet.
работе отлично
Ниже разве не должна идти проверка на успешную инициализацию объекта. Как бы, мало ли что :-)
"Но пренебрегать входными данными, прописанными в задании - не стоит"
Именно это я имею в виду. Известно, что Cell всегда содержит массив из 100 элементов. Поэтому нет никакого смысла пересчитывать кол-во элементов для мутации из процента, ибо результат будет гарантировано совпадать с вводом - в полном соответствии с ТЗ.
Поэтому отсутствие пересчёта - не ошибка, а оптимизация. Что непонятно?
Just very little nitpick: could you cut lines at 80 chars ? Like it was before.
Don't know why this happend, this is an entire different commit :S
This is not needed?
no sprintf function, string interpolation won't happen
Curly braces on a new line
indeed this looks a little bit uncached :-|
Thanks @johnbender
no in1d so if np.in1d is present use it.
This part I still need to fix. Right now it just picks the freq in the mt-spectrum that is closest to the desired one. This is not ideal, as the real freq (e.g., 60 Hz) could lie between bins, depending on sfreq and n_times.

Chronux had some code for doing an f-test to figure out which components to remove, but I'd rather stay away from that functionality if we can. I worry users would utilize it without thinking about the consequences.
N00b!
It seems as if you're not using this file anymore? If so get ride of it.
Surprisingly instead of using PowerMockito to mock static classes you could do something like:
ServiceContext sc = mock(ServiceContext.class);
Context context = new Context();
context.setServiceContext(sc); // it sets the static serviceContext field! I actually think the setServiceContext was supposed to be a static method, but well.

AdministrationService as = mock(AdministrationService.class);
when(sc.getAdministrationService()).thenReturn(as);
It's cleaner to put the link on a new line.
正则表达式里可以包含有名的和位置的捕获，这些变量可以跟结果变量一起被其他指令使用。
It's using positional arguments.  The second one is for the URI query params, and the third one is for headers.  Use the block form if your needs are different.

```ruby
conn.delete url do |req|
  req.params[:page] = 1
  req.headers['Content-Type'] = 'application/json'
  req.body = "some stuff"
end
```
are you sure this has been resolved? see the comment on former line 19 above
Doesn't respect the XWiki codestyle.
We can add a @throws here as well, as far as I can see.
here and elsewhere, perhaps,

```python
whole, rem = divmod(pw, 2)
if not whole and (i, rem) in d:
```
Я сделал как в blogilo. Так лучше. Это редкая опция и мой вариант занимает куда меньше места.
А вот здесь `LeechCraft::` и не нужно.
Получается, что меню всегда заполнится, даже если плагин должен быть неактивен (в `services` нет  юнити).
I still would like there to be easy support for apertures that depend on the source (such as MAG_AUTO in SExtractor). The alternative is often thousands of calls to `aperture_photometry`. And then you have to compile all the resulting tables.

This may be needed for `psf_photometry(..)` as well. If the PSF varies across the image, it will be different for each source. Or will this flexibility somehow be included in PSF classes?
I was thinking the values in the array are like the set's values.  Of
course we should probably take in a Set<Double> but the double[] was just
the representation.  Either way, we need to clarify in the javadoc what the
input actually is.

On Wed, Sep 19, 2012 at 11:17 PM, Keith Stevens <notifications@github.com>wrote:

> In src/main/java/edu/ucla/sspace/common/Similarity.java:
>
> > @@ -899,20 +906,25 @@ public static double jaccardIndex(Set<?> a, Set<?> b) {
> >       * samples.
> >       */
> >      public static double jaccardIndex(double[] a, double[] b) {
>
> So the Jaccard index is based primarily on set overlaps. If we're treating
> the input arrays as some kind of count over how often or important a list
> of items are, where each item corresponds to an index in the array, then
> from a set point of view, the counts are irrelevant. All that matters is
> that the items occurred at all. In the previous code, we were treating each
> *count* as an interesting value to compare, but that really makes no
> sense. Under that interpretation if some word occurred 5 times and some *totally
> different* world also occurred 5 times, then that'd improve the Jaccard
> Index, which seems to not make sense.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/fozziethebeat/S-Space/pull/19/files#r1647758>.
>
>
Not a reason to reject, but I believe python allows line wrapping within tuples / function argument lists.
Should be after class, and just in case, call `super.teardown();` too so that the cache manager is closed too.
Were you aware of [numpy.testing](http://docs.scipy.org/doc/numpy/reference/routines.testing.html)? In particular the ```numpy.testing.assert_array_almost_equal``` function.
Sorry about the spelling.
Nit - one blank line, not two blank lines here
same as previous import comment
I don't agree. That means that as a user you can pass it anything and it should cope, but even if an implementation casts to string it doesn't protect it from failing on non-strings. Casting a string that does not implement __toString yields: `Catchable fatal error: Object of class $class could not be converted to string`.
This is wrong. `isset` returns false if you have the key but value is `null`.
Null values are accepted as part of composite PKs, so this needs to be addressed here.
To correctly fix the issue, change the method to `array_key_exists`.
resetting experiment = purge existing + create new one, and default enabled (as of now) is true.
These seems unnecessary seems like you can just do this below:

```coffee
module.exports = (robot) ->
  robot.hear /chicken\?/i, (msg) ->
    msg.send(msg.random chickens)
```
This is probably more accurately "cursor_insert_bulk" -- number of times insert was called on a bulk cursor in the current session.  We can't know across a reopen which records were bulk loaded.
The else if style is alternating around in this file.

    } else if (foobar) {
 
   }
    else if (foobar)
that's sounds usable (sorry it's becoming late over here)
It isn't very semantic, but it's perfectly valid and will make a difference to the page. I'm assuming that UB thinks it looks better like this.
Is it possible to define the element type for the array of parameter ``$options`` (using ``array<SomeType>``, or ``SomeType[]``)?
`[].map(...)` is `[]`, and `[].max` is `nil`, so this can be simplified to just line 59 alone.
This still needs a code example. All API methods should have an example in the docs. Even if it's something as simple as:

```javascript
var selection = editor.getSelection();
console.log(selection.baseNode); // the text that is highlighted
```
I love this line.
Using `$::osfamily` here would make it support Ubuntu as well, and simplify the RedHat part.
In this particular case, it turns out that association should always be embeddable? and we can reduce that to just proxy.load_from_database.

In any case, the main difference is that load_from_database sets the @_from_db attribute to true and replace sets it to false.

In general, when calling the proxy's find_target, @_from_db controls whether it will do a load or a new. load will  generally go to the one defined in keys, which may be replaced with the identity map. It is the method that would have been called by load_from_database if we didn't skip loading the associations.

If we used new, the _new property would get set to true, which we don't want.

The whole point is to defer the evaluating the proxy until it is needed. This saved us quite a bit of time when loading lists of documents. Especially, when it contained embedded documents we didn't need for the particular operation we were doing.
Should this be omnibus_options since the other install strategies wouldn't use it?
PessoaImpl => PessoaJdbc
Should the `Odometry` message be from `base_frame` to `map_frame`? This is typically not what the SLAM nodes will compute, but is what makes sense as input to an EKF filter. Furthermore, using an `Odometry` message is nice because it allows seamless plug in `robot_pose_ekf`, but what typically a SLAM node will compute is a `PoseWithCovarianceStamped`. Of course, Twist can be estimated by differentiation, but I am not convinced this is the responsibility of the SLAM node. Also, if we plan using an EKF to compute the robot pose, to avoid conflict SLAM nodes should provide a setting to disable the `tf` output.
why ending twice ?
Since you may want to choose between pdb and ipdb, and also for consistency, we should have two filters: pdb and ipdb.

This means that the autodetection code below should only apply for the ipdb case.
And return the results (that is, a set of pending tasks objects).
We put `{` on line up.
OK, improve the subject how we discussed it elsewhere.
Ehm, not timestamp, but our fetch time.
I have run the test on the current version (where we don't even pick the constructor). An exception of type *Ploeh.SemanticComparison.ProxyCreationException* is thrown with message *The proxy of 'FooProxy' could not be created using the same semantic heuristics ...*

Perhaps, what we could in that series of commits is to throw the same exception (instead of *System.MissingMethodException*).
vielleicht wäre um das get und release das noch sinnvoll:
try{
cr.get();
}
finally{
cr.release();
}

Dann werden die Resourcen auch im Fehlerfall freigegeben. Schätze, dass es bei der Kommunikation auch noch ein paar unchecked exceptions geben kann.
@emmanuelbernard yea that isn't going unnoticed that you changed style ;-) but I'm not going to dig up all my emails from you somewhere in 2008 or 2009 were you asked me to not to, I'm sure you did. (especially as it likely was MSN at the time)
this doesn't affect people using the gem does it?
Can you assist me with this? I'm not sure what either does or the benefit of adding them.
Yes, model is a class, so it's the class real name
With the latest changes to BIDPOM this should no longer be necessary.
this isn't even supposed to be in there. According to Mr. D, it doesn't
need to be. I guess when you move it, the stuff that's blocked comes back,
I'll take care of it.

On Sun, Sep 16, 2012 at 11:00 PM, Todd Pickell <notifications@github.com>wrote:

> In GroceryInventorySystem/GroceryInventorySystem/inventoryItems.cpp:
>
> > @@ -0,0 +1,7 @@
> > +#include "StdAfx.h"
> > +#include "inventoryItems.h"
> > +
> > +
>
> Why is the implementation file empty and all the code is in the header?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/myappleguy/ciss243Final/pull/3/files#r1615841>.
>
>
Would be nice to have more detail here. What's the form? A hash? Array? String?
Would like to postpone changing to addListener if that's okay. Lots and lots of changes to network_view.js in branches I'll merge this to.
I think that would work.  Let me rewrite it that way and see how it looks.
With it in the header but non-static I get a linker error:

    objs/addon/src/ngx_rewrite_options.o: In function `ngx_http_pagespeed_str_to_string_piece(ngx_str_t)':
    /home/jefftk/ngx_ps/ngx_pagespeed/src/ngx_pagespeed.h:29: multiple definition of `ngx_http_pagespeed_str_to_string_piece(ngx_str_t)'
    objs/addon/src/ngx_pagespeed.o:/home/jefftk/ngx_ps/ngx_pagespeed/src/ngx_pagespeed.h:29: first defined here
    objs/addon/src/ngx_base_fetch.o: In function `ngx_http_pagespeed_str_to_string_piece(ngx_str_t)':
    /home/jefftk/ngx_ps/ngx_pagespeed/src/ngx_pagespeed.h:29: multiple definition of `ngx_http_pagespeed_str_to_string_piece(ngx_str_t)'
    objs/addon/src/ngx_pagespeed.o:/home/jefftk/ngx_ps/ngx_pagespeed/src/ngx_pagespeed.h:29: first defined here
Sorry for the mistake from me. I should notice the buffer was clear....
done, and moved this whole section to its own function
Replace the else with an assert?
Makes sense. Stata uses hascons. Could we also just check if it's full rank since we're computing the rank anyway? I think this is how R does this for the purposes of ANOVA, etc. Then we handle the panel case on its own. Not sure.
In general, we need more continuous kernels than just gaussian, especially ones with bounded/compact support
Probably I'll do it after merging.
设置工作进程使用的非特权账户名称及权限。

感觉也不是很通顺，呵呵，user就是worker进程起来以后使用的账户名称，一般master进程是root权限的，而worker就是一般的用户。
true, but nexus will fall down at that point.  
We want to make sure we are not deleting good data.  

What happens if the existing attribute file is valid, the nexus users runs out of file handles, and nexus cannot overwrite the existing file.  We would not want to delete the file in this case, as the old data would be beter then no data.
Fluke, or an elaborate troll-within-a-troll? #trollception
Yes.  We have to do it that way if we want our multiple process stuff to work deterministically.  But even if people don't set a random seed explicitly, I don't think there's any danger of repeating initial seeds by accident because of the time initialization.  Check this out:
```

>>> vals = []
>>> for i in range(20):
...     rng = galsim.UniformDeviate()
...     vals.append(rng())
... 
>>> for i in range(20):
...     print vals[i]
... 
0.945653629256
0.848873705603
0.980820850702
0.431047609542
0.213930027792
0.552609677659
0.949710144429
0.835133100627
0.296914643608
0.496374197537
0.308267343091
0.452824495733
0.244326961692
0.963690048549
0.677749620983
0.803915831493
0.263021907536
0.249254386406
0.526536547113
0.781795559451
```
No repeats.  And that's about as close together as you can put the rng initializations.
Pushed this change in most recent commit but one...
Is there a reason why these three are assigned before being used only once or twice below? Could you not simply use the `self.kx`, `self.ky` directly? It's not big deal, as it costs little to label a reference in Python, but would like to understand if I'm missing something?
This last comment is also obsolete.  The 1d Interpolant is now passed by shared_ptr, so the shared_ptr "owns" it.  No need to worry about keeping something in existence elsewhere.
I think the error should be clearer on what fields the comparison was performed on and if we were ignoring null fields of other object.
What do you think ?
We should externalise this using a config key; but that should be a separate pull request.
Worse, if we create a new entity passing in an owner in the constructor - it is immediately "deployed", based on getApplication()?.isDeployed(), so we were getting these warnings - even though the entity was not yet officially managed!

Agree it's messy.
What is the convention for port sensors like this? I would call it postgreSqlPort or similar, and always have the entity or protocol name as part of the flag?
Why are these two lines necessary, given that TestUtils.connectToURL also calls TrustingSslSocketFactory.configure?
I tried commenting them out, but then the test failed.
Should we assert something about the content of the partial?  Same with the one below.
I’d put ticks around module and variable names.
Non le pb cest la coherence syntaxique: _because of how difficult it is and the idea_ ca sonne bizarre. vaut mieux mettre _how difficult ... and how researchers approached the problem_ ou un truc du style
This filename is too generic. Rename to match the name of the view requiring this functionality.
comments :-)
that returns  pages.profile.Profile it's a totally different page
Indentation is off. Not a huge deal, but if you end up changing anything, fix this too.
Can you ensure the message handler check evt.data == 'visibilitychanged' to be sure this code does not fired on any messages passing.
to force the download, the proper way is not to force the mime type but to change the content disposition to ``attachement``
Folks might be confused in the future, and think you're referring to name = get_visible_name() :P
That you'd like an ice cream cone? ;)

Before deleting it, I'd like for someone familiar with the code to tell me which version is the correct one.
No no no no NO.

There are other 64-bit architectures, and this game may well run perfectly well on some of them.
the rich text editor we use in admin use <div> to split lines.
Помимо пробела могут быть другие разделители.
useFileUploadSupport() is deprecated, please revert this to `getFileUploadSupport()`
access token should be scope specific.
in the get ETag the comment says it's snapshot ETag and in the set it says it's blob. I think it would be good to use same terms in set and get
Common mistake, I forgot to push that part of code.
It's on my home machine, I'll push it when I get back from the office...
文の順番入れ替えについても、意味が通れば行っていく方針か確認したいですね。
Tested and working for me too. Won't matter which one for me as long as it's fixed :+1:
What about setting `sample_weight=np.ones(n_samples)` if it is `None`? That way we could simplify some parts of `_tree.pyx`. 

(I can make just the changes if you will.)
Seems the comments are misleading. I will correct it. Thanks :)
This can be accomplished more efficiently with
```
self.class_prior_ = np.bincount(self._y).astype(float) / len(self._classes)
```
I believe this should work
Actually, I meant "small" literally; variants of the hashing trick have been used in embedded devices and once I get my Raspberry Pi running I'd love to try and port scikit-learn to it. Maybe I should change "computer" to "device".
I don't like treating the binary case differently.
Fixed in followup
if,if,1,2... Seems to me, like responsibility to decide what menu to display is completly on the measruementActivity, so, why it is not in that class?
Bad coding standard, the opening bracket should be on a newline
This is fine, I mostly had this in as a placeholder
From the apidoc: `The listen call will attempt to listen on the specified host and/or port property for the socket if they are set.`
Anvil seems to only set the port (util.js line 63) so this probably needs to be reworked.
constats defined
That's something that should be consistent, probably not used much, but we think it should be the actual native control handler rather than the container_ returned.
Same for the others.
In the create view, there must be no code!!! This code must be in the new view!
Use `getOneOrNullResult` instead of catching this exception
that was a test. Sorry. Deleted on e92d8d8
This looks wrong - cur is going to be updated at the end of the loop.  

I think a double byte quote character preceeding a separator at the end of the file could never terminate.
Same way as the slave code does...
Line 3850 is:
>   *(index[finger++]) = NULL;

>In the above code I set the rowsToSort entry for the returned row to NULL (see line 3850). 

You're not setting rowsToSort entry to null, you're setting a copy of the ptr in 'index' to NULL.
The original rowsToSort members are untouched.
Don't raise a raw Exception. In this case you could use ValueError.
Use a bracket instead:

```python
return (
  '%s  %s - %s - Build ID# %s' 
        % (item['product'], 
             item['version'], 
             item['platform'], 
             item['buildid'])
)
```
"missing signature" is one of two possible reasons it's invalid. So, comment is "wrong".
Tu proposes quoi du coup ?
Probably want to take out the "you" here.
OMG WTF NO
This case solve this security issue. In this case I sure this class is a valid controller. if controllerName is sms_service this case  create error.
I known the problem, This'll break the linux on IA ,but HP-UX　has no /proc/cpuinfo.
I'll modify the code asap
profiler 译成『测定器』，有什么准则依据么？
@shacker Totally agree, they should have been removed now.