labofsahil/github-event-dataset-2012 huggingface dataset:
Without the facade this method would like:
```ruby
def invite(investigation)
  if in_network(investigation.sender)
    UserPrivateMessenger.new(invitation, invitation.sender).invitation
 else
    UserMailer.invitation(invitation, sender)
  end
end
ah, it will be in same shape as in master .. ok.
There is no evidence because it isn't true. There is no noticeable performance impact. If Windows is discarding data, that might be a problem I didn't consider, but how is it "a known issue"? Citation?

The hashrate reported by my current code is correct AFAICT.
For a test like this you could also just register the blueprint on / (i.e., without a prefix).
you can return before iterating if ``null === $this->featureFilter``
I think you should just save the manifest from the original resource, rather than re-parsing it.
Great, do you have time to make the change now? If so, can you make the change and I'll merge it straight away.
yes, we need to pass it :)
Same as @gtorodelvalle . It's only a template for getting carrier info available in call log, but will be replace with the algorithm described by Ayman in his wireframes.
I am sorry that I didn't pick up this bug from the first time I review this part ....
let's rename those images so they are all lowercase.
Shouldn't the method be named `isMarkupFree()`?
I'll add in the code so we make full use of this error class (getting a Python trace, etc)
Return Err.notFound()
Most of the Array and Object methods cover both
This special requirement sounds odd, but there is a reason to the madness.

If the user calls "./githup-upload.rb file.zip some/repo --username IQAndreas", the script has no way of knowing if the current API token matches that username (you could store it, but it seems unnecessary). So we have two options left over:
 - Create a new API token using the specified username
 - Call the script with the specified username regardless of API token

The former can be achieved with `--reset-token` key, the latter is without the key.
Same as before. Consider opening an issue on GitHub to investigate this,
Previous way is less code. Sticking to that.
We don't have "colors" we have "materials", this whole part needs to be revised.
Would be nice to factor this out into its own method allowing you to write a unit test for it.
I think this should be a `.get(..)`.  In the tests, at least, it fails with a KeyError.
Will it remain `dev` by default? I think once everything has stabilized composer should only download stable packages unless asked otherwise.
[Rubular](http://rubular.com) is a good place to start for a crash course in Ruby regular expressions. :)

Gives you a quick reference, an easy way to test your regex and a link to the [Pickaxe](http://www.ruby-doc.org/docs/ProgrammingRuby/html/language.html#UJ) if you need more information.
(実際のコードでは `update_attributes` を盲目的に利用することはなく、常に変更できる属性をホワイトリスト化してください。)
IIRC this is  bash specific... let's see... yeap -- minimalistic POSIX shells (e.g. dash, default on e.g. many Debian derived distros) has no clue about RANDOM, so it would aways return

```
$> /bin/dash -c 'echo $((RANDOM%2000+10000))'
10000
```
@Lasall 
Got it
Fixed. Thanks for all the comments :)
If I understood correctly in some cases `selection` could be `null`, although that is not common. I also used `instanceof` to be consistent with the rest of the code, where it used before add and remove listeners from selection.
@ochameau could you please take a look only at this file once you'll have a chance. I tried to rewrite content-worker such that it no longer requires multiple sandbox or injection of objects from chrome. Although for whatever reason I still several warnings when running context-menu tests:

```
onsole: [JavaScript Error: "Exposing chrome JS objects to content without __exposedProps__ is insecure and deprecated. See https://developer.mozilla.org/en/XPConnect_wrappers for more information." {file: "resource://35343033-4e4d-460b-9404-bf9a2ba82202-at-jetpack/addon-sdk/lib/sdk/content/script.js" line: 114}]
```

Do you see what is the problem here or how to solve it  ? I believe problem is on line 145 (I think comments are ignored that's why error has a wrong line number). In the current implementation such errors are not observed and I don't understand why.

Thanks!
yeah.  I would like to improve this.  The only role the locale `db-LB` now serves is to convey to the front end that we're running a debug locale.  If it weren't for this, we could just use the debug locale straightaway.

One way we could make this better would require `jsgettext()`.  if we had that, then rather than including and running `gobbledygook` on the front end, we could dynamically generate a client.json for the debug locale upon request...  That way we'd still get the property that the debug locale is always up to date, and we'd excercise a bit more of the frontend code.

Dunno.  What do you think?  /cc @zaach
in PR-1489(https://github.com/mozilla/browserid/pull/1489), I am adding a helpers.log so we can have this sort of logic in a single location - https://github.com/mozilla/browserid/pull/1489/files#L1R70
If I put it into the describe block it doesn't include methods from that module.
You could really do with some comments about why it isn't a hideous mistake to allow the set of installed versions to be extended just by asking questions about what is installed.
`clear_everything_for_tests`?
Could it be `clean_backtrace` or `cleanup_backtrace`?
You've got the call to `rescued_pretty_json_render` repeated 6 times here.  You could probably refactor this so you only use it on one line.  This is not a big deal though; the refactored code probably wouldn't be much prettier.
Typo in `swissgri1d` (extra `1`).
`kwargs` is unused.
spaces surrounding %
If "call()" proxy the "__call()" method, it will be :

public function call($method, $params = array())
{
    return call_user_func_array(array($this, '__call'), array($method, array($params)));
}

I think there is a bug, but for me, there is more sense to use call() like a proxy.
Sure thing :) Will try to test this one too
Where are you thinking of putting it instead? I'm not 100% against this, tbh, particularly since you can specify an alternate ModuleManager factory in the application config.
Instead of looping through all validators, would it be safe to assume NotEmpty would always be first? Only check the first item in the array?
please remove the leading underscore. ZF2 follows PSR-2 coding style guidelines
This is duplicated with the line 80 but with a different value
I think that this code is not compatible with PHP 5.4, there is an `Array to String conversion` in the second argument of preg_match
Add a space before {. Update whole file
Missing @author and since. Update whole class. Not sure if it is needed when we work on ssGwt?
+1 for view
Huh, is it to fix a separate bug? Can you do this in a separate commit?
I'm a little hesitant to say what this does, but I'm guessing that setting var $xml = $(xml) will save some processing power, instead of running $(xml) every time.
`FALSE` needs to be lowercase here and `"."` should become `'.'`.
Also we'll need a test case for this. Usually if you provide one along a suggested patch yourself, patches can get merged much quicker.
sorry for my late response... unfortunately, updating the current version of todomvc using meteor and flatrion director doesn't has my highest priority at the moment, primarily due to other work... but it isn't actually a big deal to update, i will try to get it done till the end of this week... 

i am also working on my own framework, it's called "pervasive", having some of the sugar (reactivity, database-everywhere, etc.) in common with meteor, but will be more "open", e.g. using npm as package manager, database independence (use sql or nosql), persistent client storage, syncing client and server data, server-side rendering, etc... the core is already more or less working... but it's currently to hacky and usability has be improved...
It worked when I removed that path variable and it compiled OK. I will close this pull request and post the fix in the conversion from 2012 to 2010.
@harjis Tämä turha.
DONE !!
Btw, it is really weird that the Travis build is passing, because there is no setter for the platform.

Is the test skipped ?
oh awesome, thanks a bunch
I understand what you say but on method "collect", i need to know if legacy_mode is enable or not, on current siteaccess.. if you have a tip to do that i take it ;)
better use this syntax:

cat >>${D}/usr/share/applications/${PN}.desktop <<- EOF
[Desktop Entry]
...
EOF
I would let the previous form as well, and put what we expect with it.
Implemented running the demos as sanity check command, this works, but it seems like these don't detect the issue that occurs is packed groups is not enabled with an ictce build. I.e., the demos just work. :(
... of easyconfig files (.eb)
spaces around operators and after , and :
Yes, I'll refactor that later. It would be nice if you could tag services and then you'd be able to configure them.

-> tag service -> getKey() -> 'type: my_key`

Not sure if this is possible, but it would be nice to allow people to add extra specific implementations without having to change the configuration classes etc every time.
This indent is the same as the one above for 'mvn clean package...". 
Changed both.
thanks for fixing this
Oh sorry, I misread that... you are right.
Can you use

````
if (..) {
  ..
}
````

syntax to be more in lines with emscripten's coding conventions?
default value

anyway stolen from some stackoverflow question
nitpick: kwargs.get('latlon', False) might be more concise.
Cool ever seen this before.
Client must be signed in as a vouched user.
The issue is we shouldn't be calling `profile.save()` from a model form.
That's the way the style is for the rest of the file.
Just to be clear, do you mean the actual file name??
V8 doesn't have any trouble getting a name for that function.
Looks good, but can you fix the indenting on this line?
Use api_url instead of https://....
That sounds as a too large change for my documenting-only taste. (And I actually prefer RolesAllwoed to Rbac.) Feel free to do it yourself if you feel so :-)
Data members should be named with nouns, not (imperative) verbs. This code is hard to read.
This is the broadcaster ID, it's all in their docs.
This is meant to be the path where the queue file resides. It's a file that can be changed via an FTP upload, and Hanyuu will monitor this directory for any changes to the file, if there's a change, the file is read and queue is updated.

I'm not sure if this still works though.
Wouldn't it be fun if this were his/her own password on several online services?

Oh... And I think the hash is for "saudi", all in lowercase. Someone should update the comments on the fork.
what's up with this duplicate info? I see tilemill occasionally adds this but can't figure out why
should object_id be last one to be consistent with main keys value?
shoot, missed one here...
My head just exploded trying to read this ... do we not just mean:

SizeMatch := ~Strict OR StatsL.XMax=StatsR.XMax AND StatsL.Yman=StatsR.Ymax;

???
Seems like it is used in CSS here: https://github.com/jipiboily/monologue/blob/master/app/assets/stylesheets/monologue/blog/monologue.css#L86

Maybe you just change that to reflect the new structure?
`<p>` tag, that is.
Firebug is ok.  Adblock Plus is not ok.
Nice! Have found the same on stackoverflow. Ii just wonder if 16KB is the right amount of data we want to read-in for each iteration or if we should better go with 64KB, which I think is a better size.
Okay, not a bad idea.

+1 to unittesting.
There's a warning when the database cannot be opened, so I don't think it's necessary to flood the output on each SQL query as well.
whitespace around function arguments needed. Sorry to be a pain ;-)
Is there any code convention about the `@param` docblock? 
Should it the full classname or the short class name?
Можно было вовсе не создавать эту ссылку, а сразу работать с self.DNA.
мутируйте обе днк. Можно воспользоваться чем-нибудь вроде 

`[cell_1 mutate:arc4random()%[[cell_1 DNA] count]];
[cell_2 mutate:arc4random()%[[cell_2 DNA] count]];`
Возможно не понимаю, по условиям проекта нам нужна переменная DNA типа NSMutableArray, соответственно для этой переменой и были написаны сеттер и геттер, и через них я  и "общаюсь" с этой переменной.

Если не так поправьте.
Не только. Из процента ещё количество элементов для мутации нужно определить.
This sounds a bit like "so it's not our problem". Can you add a sentence at the end like "The following paragraph show how to do this."
looks great! would just suggest you document this function similar to how the others are.
partial here are well
Bah... sorry - editor doing it's thing...
typo... *chwon*
This looks right viewed in Xcode. It's in the Project group.
Yes, but your explanation here – in the documentation and this thread – was confusing.

It sounds like the real goal here is not "to ensure that subscription happens on a known scheduler" (because that Doesn't Matter™, as I pointed out above), but rather to defer scheduling when not already on a scheduler.

In which case, I would ask: why not be consistent and always defer scheduling?
Tazim, this branch (cfse-1.1) that you sending pull request intended for CFSE too :) so I don't think I'm gonna replace pulp-server here. sorry.
I would recommend implementing another method: call it stopHeadpin()

(same for: startHeadpin please)
Bruk gjerne også dash framfor underscore i attributter - "promises-tab" framfor "promises_tab".
e.g. compare epochs['1'] and epochs_read['1']
you can do a global import (not nested)
You mean always wrap it in np.abs or you mean intrinsically return absolute values? Pearson-r, for instance,  would also return negative values. But then you will say that the user is free to do what she wants with scores and source_idx and we are better of with less args and more simplicity.
"DEFAULT_INDEXES" is plural. But the docstring says it returns "the index"?
Ah ok I see how it works now - I was wondering where the extra data came from.  My bad :D

Now that I know that the dataset is consistent I'll add the extra test cases for the use case that this change affects as well as the regular use cases if you like.
Yes, but there's no way to know if a file is binary or Not,
Yes, it can be deleted and cleaned out. 
I'll repair it.
Added a few "tests" - basically a bunch of debug statements to see if everything is fine. There are a couple of bugs I'm trying to solve. Please find them in the next pull.
Done. Not sure I should keep this part though, as it has disappeared from the "How to Create a Custom Form Field Type" cookbook article. @stof any idea ?
如果不使用<var>$remote_addr</var>变量，而用<var>$binary_remote_addr</var>变量，在这里可以将每条状态记录的大小减少到64个字节。

使用<var>$binary_remote_addr</var>变量取代<var>$remote_addr</var>变量可以将每条状态记录的大小减少到64个字节，

添加“这样”联系上下文，这样1M的内存可以保存大约1万6千个64字节的记录。
``` js
.put('/' + db_name("a"))
```
Я бы ожидал от функции с именем `Find` какого-то возвращаемого значения или типа того. А она ничего не возвращает. Может, поменять название? Типа, `UpdateSyncableItemsList` или типа того.
put a else here no point evaluating this if the first if passed
Makes sense.
And since the original intention was to have more than one node be capable of fencing, we already covered that by having every node be capable.
``setCurrentUri`` does not exist anymore
Actually already does that, check a few lines above. This delete was from the original file, where we restart the nova-gridcentric service.
I tried Monkey Patching for most of yesterday only to find that doing it for the lifetime of the code was nearly impossible. It's just too hard to change all the functions you need to before they're used by anything else, which ruins the effect. If you think about it Mox and other tools are written to be used in unit test scenarios not in eventlet driven apps.

On the plus side the non-patched code is far easier to understand and reads cleaner. The only real blemish is the patching must happen here, which is really no different than a conditional import.
can you add a small comment of what this block is doing for easy reference?
Can you get on Skype for some other things?
Note to self: this comment is not correct.  Initially I was just going to do this for webm, but there are plugins for chrome at least that support mpeg4, so I am returning the EmbedHTML for all filetypes and leave it to the nb user to try and add support to their browser.  Just need to clan up the comment
As discussed on IRC I will revert the ordering of the parameters.
I think you can do `str(current_versions.append(FirefoxVersion(element.text))` here and that will fill the list with strings rather than version objects.
C'est inutile de l'utiliser en tant que variable d'instance si tu ne l'utilise pas ensuite.
Now a follow-up commit removes that.
this is already done about 25 lines above so this line can be removed.
It doesn't look like it would work to just map thrift/unpack right now.  But you're right that this function could be made a lot cleaner (and renamed, probably).
@christianv Missing a semicolon.
Не доверяй туториалу, мы так приватные методы не выделяем, дополнительные пробелы не обязательны)
It comes with OS X, so it's something that will only have an effect on Macs.
Hi Keith,

> That means if we have a lot of sessions at some time, and the number of
> sessions decreases, we're still looking at those empty session slots --
> but, I agree that's a NULL vs. non-NULL test, maybe we don't care?

Sure, but I don't think that's a common case and as you say, the test is cheap.

> That said, I don't see how we can go wrong if we decrement session_cnt in session close?

Well, it means we're explicitly ignoring some pointer that was previously in use.  Decrementing this count (by shuffling pointers around) is effectively the reason why the check can race with a close at the moment.

Michael.
On Wed, Aug 1, 2012 at 7:44 AM, Michael Cahill <
reply@reply.github.com
> wrote:

> > @@ -111,7 +111,9 @@
> >       if (F_ISSET(btree, WT_BTREE_DISCARD | WT_BTREE_SPECIAL_FLAGS)) {
> >               WT_ASSERT(session, F_ISSET(btree, WT_BTREE_EXCLUSIVE));
> >
> > -             ret = __wt_conn_btree_sync_and_close(session);
> > +             WT_WITH_SCHEMA_LOCK(session,
>
> What is the case where we have a btree handle with special flags or
> discard set, but we don't have the schema lock?
>

I can't reproduce it, so it may have been an artifact of a previous version.

I've backed out this change, I'll re-run the test suite to be sure
(__wt_conn_btree_sync_and_close asserts that we hold the schema lock, so
we'll find it if there's a problem).

--keith

=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
Keith Bostic
keith.bostic@wiredtiger.com
You'll notice all controllers inherit from ApplicationController, so you can put shared methods here.
I think this is what you want, if size==0? I'm not completely sure about it.
Are you sure ``$this->once()`` has the correct type as ``object<PHPUnit_Framework...t_Matcher_InvokedCount>`` is not a sub-type of ``object<PHPUnit_Framework...ect_Matcher_Invocation>``?
tiles are 16x16. So this value needs to be divisible by 16.
Please you make this just a simple # (and same thing on line 388)
so. i have it in two places. in my spec and here, so it can analyze code coverage during development runtime. 

I read about it in the simplecov README, but not totally sure if this will be used.. just kinda added it anyway.
previous test!
forgot C-c k before saving.

code review for the win!

2012/3/24 denlab <
reply@reply.github.com
>

> >
> >  (fact
> > -  (load-ns!) => ["fnx.meta.expose" "fnx.meta.example"])
> > +  (load-file-ns "namespaces-test-to-load") => ["fnx.meta.example"
> "fnx.meta.example2"])
> > +
> > +(defn load-fq-fn! "Get the namespaces from the file namespaces-to-load"
> > +  ([] (load-fq-fn! "namespaces-to-load"))
> > +  ([f]
> > +     (map (comp fun-str str) (mapcat #(ns-public-fn (symbol %))
> (load-file-ns f)))))
> > +
> > +;.;.  [31mFAIL [0m at (NO_SOURCE_FILE:1)
> > +;.;. Actual result did not agree with the checking function.
> > +;.;.         Actual result: ("fnx.meta.example/hello"
> "fnx.meta.example/one-arg-fn" "fnx.meta.example/hello-noir"
> "fnx.meta.example/three-arg-fn" "fnx.meta.example/two-arg-fn"
> "fnx.meta.example2/hello" "fnx.meta.example2/three-arg-fn")
> > +;.;.     Checking function: (contains ["fnx.meta.example/one-arg-fn"
> "fnx.meta.example/hello-noir" "fnx.meta.example/two-arg-fn"
> "fnx.meta.example2/hello" "fnx.meta.example2/three-arg-fn"] :in-any-order)
> > +;.;.     The checker said this about the reason:
> > +;.;.         Best match found: ["fnx.meta.example/two-arg-fn"
> "fnx.meta.example2/hello" "fnx.meta.example2/three-arg-fn"]
> > +(fact
>
> you forgot to future-fact ? or is just the output of a previous failling
> test?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/clojure-consulting/fnx/pull/46/files#r600162
>
This is wrong here. DataFixtures is in the **Common** namespace. It should not depend on the ORM (this code is broken for the ODMs currently).

Thus, all ODMs are now using Doctrine Common 2.2 as their requirement AFAIK so you should do the ClassUtils call first, without any assumption on the proxy (it checks it internally anyway), and eventually provide a fallback for the older ORM as done here but I'm not even sure it is necessary (we could mark this feature as requirement the ORM 2.2 as 2.1 will soon be marked as receiving only security fixes and no more bug fixes)
You can be more descriptive with the cache key than 5 characters
Thanks! Could you submit a pull request to fix these?
You're absolutely right! Good catch.
please remove the space after ``isset``
Just more a comment on the documentation: 

    $ ruby cd test && to_test.rb apps/some_app
    ruby: No such file or directory -- cd (LoadError)
Good. But also first add parent content `{{ block.super }}`, because maybe some other template has already used this block, to not lose its JavaScript code.
Ping. :-)
If you will not be updating fetch time here in this pull request, then it does not work properly! Because then all stations will display stale data, even if it is not stale, but just there were not changes in the station. You have to fix this here, cannot be a TODO.
Pretty sure this is equivalent to just gettext(text)
+XMLDocs?
I think it might be better to explicitly disallow errors in metadata processing (otherwise you'll end up with, e.g., a mix of ints and strings, where maybe the user really wanted to be notified if there was a casting error). I could see use-cases where this may be useful in catching errors. Maybe add an option to ignore failures by default?
In the original code $name was used in the logging messages without quoteSingleIdentifier being applied. I preserved the original output by using a new variable name. But I guess it's insignificant enough to warrant just modifying $name.
the RPM works on "amazon" platforms as well
Damm, could you fix it?
Why not add supports for default `Symfony\Component\Security\Core\User\UserProviderInterface` instead of own `IdentityManagerInterface`? Or create `UserManagerInterface` wich extends `UserProviderInterface`.

Look at this please
https://github.com/FriendsOfSymfony/FOSFacebookBundle/blob/master/Security/User/UserManagerInterface.php
https://github.com/FriendsOfSymfony/FOSFacebookBundle/blob/master/Security/Authentication/Provider/FacebookProvider.php#L96
Change to ``delete_box``
True, that's even better :-)
seems to ignore them if i echo $ofunction as they are loaded

 % ssh parsley
Last login: Tue Jun 19 09:08:53 2012 from 2001:dc0:a000:4:6d60:faea:9c0c:bb91
diff
dut
make
wdiff
git-branch-current
git-commit-lost
git-dir
git-hub
git-hub-short-url
git-info
git-root
git-stash-clear-interactive
git-stash-dropped
git-stash-recover
bohara@parsley> 

anyways, close the pull request if you dont want to support 4.2.6, i'll just keep my own fork
excellent point. revised accordingly.
This actually doesn't make sense to add this here.
Only enable/disable it in [MODULES] section.
^^ oh... that should not be there...
This seems high (although I see it was there in the original .cpp file too).  Perhaps question aimed at @gbernstein ... What's the rationale?

Also, are we jettisoning the convention of capitalizing magic number constants, e.g. `OVERSAMPLE_X`?  For the record, I like that convention, it highlights the special status of these variables on the code... (otherwise as a developer I spend minutes scratching my head wondering where they have been set...)
To add to that, in this code, just don't do the try/except block.  The only try/except you need would be in test_lensing.py.
Thinking POST here, since a new record is created each time.
直接require(source+"/package.json");
I've used the IDE to generate code for toString(), hashCode() and equals(). That's a lot easier than anything else.
How about this instead?  It's a bit more succinct.

```
_.all([null, {}, [], ""], _.isEmpty);
=&gt; true
```
mmm.. so user will get notification "Env has been renamed" even if someone else renamed env(because of polling), right? 
Do you think it's neccessary?
Yes, not sure if this method must be named "resolveObject" as in EC2 plugin, thought the name is good enough, but I'll add a comment to explain. 

If you have time, can you merge the instance-template-config branch and check why public-key  auth is failing ? 



On Wednesday, March 28, 2012 at 10:03 AM, Adrian Cole wrote:

> > 
> > + protected Object setCloudForTemplates() {
> > 
> 
> 
> comment could be helpful. I'm guessing that jelly doesn't wire up transitive deps like .cloud?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/jenkinsci/jclouds-plugin/pull/6/files#r613887
> 
>
Oh, right. Works for me then, sounds a better way to deal with validation errors. Sounds weird to put something missing in "validated" anyway.

Thanks again for the contribution.
If there's a value in the explicitness, I agree. To me this is just extra chars though, everyone knows in Python that return is None.
or `user.persisted?`
That was a bug. It should have been corrected right before you mentioned it.
I think these changes are spurious too (because of a bug in roxygen2 where it relies on the default collation), but I can fix that.
So instead of the if statement, you could document all these methods with `@throws` PHPDoc.
This file too, revert.
We can lose this -- that's what pry is for! Just use binding.pry where you would use debugger. plop right into a running ruby interpreter! :)
This won't work because the State you receive is the old #state record, not the new one.
Хватит copy-paste. Сделай в своём пакете какой-нибудь IoUtils,  у которого будет метод closeOrExit, принимающий один параметр типа Closeable. И используй во всех своих программах его.
А чем плох текущий вариант?
Хм, странно, в общем, я догадываюсь в чем лажа, щас слегка переделаю...
Планируется, что этот метод будет занимать конкретно десериализацией примитивных типов. Я неправильно назвал метод - название ничего не говорит о содержании.
Ну вот, везде поправила пробелы перед (, а тут нет.
If it weren't for floating point rounding errors, this point would be guaranteed to be within the `distance_range` from (0,0).
As I guess you'll see in a bit, that happened in a later commit.
i assume that since each field has its own error locator that not having re-populated the previously blanked field before testing the following field is not a problem.

DeveloperHub tests ran and passed for me
This test fails for me at this line. Here is the stacktrace: http://pastebin.mozilla.org/2003834
I think `threshold` can be removed altogether. As discussed on the mailing list, it's been deprecate for a couple of release cycles already, so we probably shouldn't keep it around.
No the shape is `(n_outputs, n_classes)`.
I forgot to remove the '\r' before my last commit, I'll remove them, thanks for noticing that. I'm not very familiar with the finer points of string formatting in python but having a fixed width sounds good.
This is not clear enough to me. Please add a line on the motivation of this field (building pairwise combination inside  of  group of samples so as to be able to train learning-to-rank models using the pairwise sample input representation).
why has this test been removed? It is redundant?
There is no test do to for the official API. The standard API selects automatically between sparse and non-sparse and the selection is tested below.
You mean about the difference between Gaussian and Bernoulli RBMs? Bernoulli RBMs have visible units that are either 0 or 1, Gaussian have any continuous value. Actually MNIST and digits are not really binary, but using it any way is more of a hack. The distribution of each pixel is heavily concentrated at 0 and 255 (or 1 if min-max normalized) still, so this can be modelled quite well with a binary distribution.

If you do PCA, you will lose the concentration at 0 and 1 afaik, so you need to model real values, i.e. use a Gaussian RBM.
why do you need both active_set and strong_set?
if active_set is given it should be the initial value for strong_set and then if kkt violations occurs
you should update it. It would avoid a kkt check for every point on the path although there is not
new variable to add or remove.
I've added it there
It would be also good to make the repo url as a parameter: in case someone wants to have more control over the packages he's using, will add it as another param.
Do we want to log this? If it's unlikely to happen maybe take out the try catch, if it's likely to happen shouldn't we be able to see the error?
You're right that seems to be the convention, and I agree it's good to be consistent here.  The reason I really wanted to change it in this case though was the keyword conflict between ui (the ui object passed in to our new command) and mercurial.ui (the ui package).  (This is one reason I don't like 'from foo import bar' in general).  I could do 'import mercurial.ui as ...' but that seems even worse.  I'm leaving it like it is for now, but if you have a suggestion that keeps to existing style while avoiding the naming conflict, I'm glad to do it.
There is a `typeDotIdExp` function in expression.h that you should probably use.
Oh, it's a just a bug. I'll fix it, and will add test case.
I was wondering about this. Can you please check this against the other platforms? In firefox, both clear functions work to clear both setTimeout and setInterval
Enum created.
This will have no utility as the value the getter will never get run.  I'm working on support to be able to set it directly in the JS object that will be usable from here.
Shouldn't this be static, or are events guaranteed to be handled by the correct instance?
there seem to be indent pb here
I'd probably use an `if else` here to show the fact that `inflaterClass` holds either `classes.LAYOUT_INFLATER` or `classes.HOLO_EVERYWHERE_LAYOUT_INFLATER`
+1, but having it in a separate commit would have been nicer
I'd probably made this a throwUnexpected or similar.
Some comment to describe how this works would be helpful.  I'm having trouble working out the flow.
no.

I don't think the two ports you are looking at are the same (though they are related)...

In standard mode (i.e. not runOnce) roxie can listen on several ports (with different security settings, query limits etc per port) as well as on the workunit queue. Thus it's looping round the topology information in the RoxieTopology file creating a number of listeners, ONE of which may be a WorkUnit listener.

runOnce is a global variable, and indicates that we are not running Roxie as a server at all... The global "port" that it checks is the command-line parameter you can pass to a compiled standalone roxie query to tell it to listen for query parameters on a port rather than just running the query once.
modifying a parameter like this is a bit confusing, especially I think because of the name.  (my first reading of the function thought that the maximum logic had been lost).
i think this is a bit misleading. the group was "bootstrapped" by a semi random group of framework representatives at a conference (php|tek what year again?) and since then new members have been voted in .. and these members are actually representatives from other frameworks .. where each framework can figure out on their own how they decide who to send as their voting member.
I think this value is overridden at run-time with `-isolateDevice` as passed by Bumblebee so it may not work when set in this config. Can you test it by filling a bogus value?
Wait, the tests show a different format https://github.com/SmileyChris/easy-thumbnails/blob/master/easy_thumbnails/tests/aliases.py#L23-40

The keys of the THUMBNAIL_ALIASES setting need to be the target.
I would also add a 

````$default : 0````
This turns the whole book into a set. No points from this one.
the api v2 will be discontinued in about one month. You should use the v3
Could we make it so tests automatically use the Riak::TestServer?
I would like to change the :info a little. With the actual reported info it's what I see:

<pre>
Services
========

host           port  proto  name  state  info
----           ----  -----  ----  -----  ----
192.168.1.133  8009  tcp    http  open   192.168.1.133 7.0.30, Application:Unknown
192.168.1.133  8080  tcp    http  open   192.168.1.133 7.0.30, Application:Unknown
</pre>

I think would be nice to change the report_service call to:

<pre>
report_service(:host => session.sock.peerhost, :port => r[2], :name => "HTTP", :info => "#{r[0]} Tomcat #{r[1]}, Application:#{r[3]}")
</pre>

(Note which I've added the "Tomcat" string to the :info value)

What do you think sectorix? Do you agree? I really think info would be much clear :)
I think this has different semantics on different Rubies
So, the `-fopenmp` flag actually does not harm the compilation if the OpenMP library is not available?
Hm, I don't quite understand what you mean. The current implementation in this PR allows you to do the following:

* warp(image, tform) -> for order 0, 1 and mode constant, reflect and warp this uses the cython version and uses tform._matrix
* warp(image, tform.inverse) -> for order 0, 1 and mode constant, reflect and warp this uses the cython version and uses inv(tform._matrix)
* for all other modes and orders this uses `map_coordinates`
At a pure mathematical level, I think undefined is correct, or perhaps NaN.  That being said, you are probably right and the "typical" expectation is that devs should never pass in invalid values.
"imageryProvider is required."
The only interval we have is a `TimeInterval` which uses `JulianDate`, I agree that introducing an `Interval` which has numeric start/stop properties is the way to go.  I have several other thoughts on this function.

1. Is `distance` the best name for this method?  It sounds like a property.  And distance from what?  Should it be `distanceFromPosition` or maybe `getDistancesFromPosition`.  
2. Maybe it should take a `Ray` instead of `position` and `direction` and be called `getRayIntersection`?
3. Also, I understand what this function does, but the description doesn't really do a good job of explaining it.  Maybe we can flesh out the documentation a little bit more to explain things a little better.
Haha, sometimes a Ruby guy inside me jumps up and says, "this is how we do in Ruby!"
Anything other than 27 is great. IMO we should avoid having resources named after a specific version which may change in the future.
`horizontalWrap` property.

Also, I think we should add a paragraph: 

On Android and iOS prior to Release 2.1.0, the horizontal layout **always** wraps and the`horizontalWrap` property is not supported.
instanceof should already checks for null, we don't need to explicitly check it.
Ping, please test this code https://gist.github.com/671380

Also see related tickets https://jira.appcelerator.org/browse/TIMOB-2015
Well spotted! :)
Fixed in next update.
Should fill at least one of either purpose/descr
"despite the returned value" is not very clear. Suggest making this a separate sentence, like, "some platforms return `false` for this value by default, but behave as if the value was `true`."
only one blank
Could simplify this method my inverting this check, returning NaN, making this a guard clause. Can then remove the indentation on everything below.
Why is this separate test needed (along with the markup)? Aren't those above good enough?
Either that, or I can do figure out (or write) a heterogeneous map function.  Ahhh, underscore.js, where art thou when when we need you (`_.map`)
I am merely trying help :) I did not wanted to make it sound bad or horrifying, sorry. Fix is easy: check if String and if not empty.
I have no issue with /etc/foreman-installer/answers.yaml. I don't mind the parameterizaton either, but lets do that in a separate pull.
I moved it to the generated sub on line 127; now it only gets executed conditionally.
а что по твоему такое параметризованный класс? это функция a -> Класс
Why not just plain memset from core.std.string? Just curious.
What about spurious wake ups? 
In general every wait outside of loop checking the result is a bug.
what I don't like about utils is that those methods typically do not match object thinking. For example; why is the getEOReferenceStringFromAnnotation not part of the OpenEngSBForeignKey object? or filterEngineeringObjectInformation of the EDBObject?
The default value looks weird here. Shouldn't it be an empty string (or null) ?
This is wrong as nothing enforces that the UserInterface implements a getId method currently.
Sorry, imprecise comment..
Thanks, I like this format much better =) for example.
It helps when used internally too - part of moving towards a management library.
I see, I wasn't aware of that. But why don't we just start doing it the right way now? It's not really any more work to create a new file. If it's about being able to see the actual table structure, the final database schema is generated by Rails in `db/schema.rb`.
what happened to the test?  It should be working.
file should be named `ESPREesSO-3.1.1-goalf-1.1.0-no-OFED-parallel.eb`, please rename it
Some programs (like python?) are (need to be) compiled with pic enabled, and so the dependencies need pic.

however, this should be in toolchainopts!
Off the top of my head, I can't think of any other alerts where HTML would be useful, so that works. We'll move it back if it applies to another alert.
хм, конструктор копирования по умолчанию разве не такой же самый?
Should always use single quotes unless you need string evaluation. e.g. `echo "hello $name";`
Fixed in inserter.js
Can you restore skin.css to it's original state and move your skin to another file? skin.css is primarily used to allow institutions to customize their installation.
Hmm, there's a lack of parallel between `import_suites` and `import_cases` that bothers me from an API perspective. `import_cases` and `import_suites` feel like they should be parallel, whereas here `import_cases` handles the full data, and delegates just the suite data to `import_suites`. I'd rather see the primary entry point method just be `import()` and then have it delegate suite data to `import_suites` and case data to `import_cases`.
The usual way to display an error from a Django management command is to raise CommandError, rather than writing directly to `sys.stderr`. Django takes care of outputting it then, and with a nice red style too :-)
this test will not work on any machine that runs an active mq. There are examples in ness-jms on how to use a random ports IIRC,
Typo for <i>entity-manager<i>. Was already there, but caught this one now.
done using __a__ JPA entit
May be a style question, but it feels like mGet(..) method is complex enough to have its value extracted into the variable and than construct a Message with the value. Would help with debugging
Yes, I am clearly stating that this list represents what's been supported by org.springframework.data.redis.support.collections.CollectioinType. 
Also, new types would depend on what's currently supported by Redis
I just think @ghillert has made copy/paste from ```StoredProcOutboundGatewayParser``` or ```JdbcOutboundGatewayParser``` ;-)
But never mind: it's not so important as we're discussing now
i think this change should be reverted
This looks the same as lines 75-77.
context_stripped is not used, right?
So, let's send right now just the WAtoken, we can think if storing and sending the URL has some benefits.
also need to add when using sqlite, directory needs to be writable so journal files can be created by sqlite else a cryptic message is created.
Same comment as singleFieldToSchema()
The same comment as singleFieldToFieldSchema too. :D
Declare these readonly because they are.
This class has a limitation that it cannot represent extents that cross the date line.  Make sure this is ok for your application.
There are a couple of ways to do this differently.  One thing that might look better is to use the dictionary literal syntax, like so:
```python
email = {'From': msg['From'],
         'To': msg['To'],
         'Subject': msg['Subject'],
         'Received': msg['Received'],
         }
```
However, it seems like you already have a dict that has all of this information in it.  So I'm wondering: why not just use msg to store your email?  Are you purposefully removing data?  If so, you could do something like this:
```python
email = dict((key, value) for key, value in msg.iteritems() if key in ('From', 'To', 'Subject', 'Received'))
```
This creates a new dictionary object with the keys and values from ```msg`` if the key is one of those strings in that tuple.  There are a couple of things going on here, most notably using the iterator version of .items() for a dictionary (I'm assuming here that ```msg``` supports the full dictionary interface), and the use of a generator expression, which uses the exact same syntax as a list comprehension but returns a generator instead of constructing a list.  Are you familiar with all of these Python concepts?  If not, let me know which ones I should explain :)  They're definitely more advanced Python usage.

This example shows that From, To, etc could be extracted into a tuple that identifies the important parts of an email that you could potentially re-use  in your template html.  I'm not super familiar with flask, so I'm not sure how feasible this is.

If you don't need to remove data, then you can just add ```msg``` to ```emails```, at least as far as I can tell.
For now I'll get the request from the container (not breaking BC), but for 2.0 it's probably nicer to update the function signatures anyway?

Passing the request because there might be additional parameters in the future (such as sort order, etc).
maybe have a specific exception for JSONObjectAdapterException
Everyone can rejoice. Now when the host VM is rebooted you wont lose your built image.
cos I didn't bother to change what existed.. but yeah a better descriptive name works.. will change that
we should best rewrite this to use a sub select to prevent concurrency issues.
чому далі все на рівень нижче пішло?
заголовок на тип application-xml не знайшов як перевірити?
Перекодируй в UTF-8
I may be showing my JS ignorance here, but shouldn't this check precede the if statement on line 160?  If we have no args.length, then args[-1], or args[null-1], et. al.,  seems like it doesn't make sense.
By the moment I made this I did not want to totally exclude that we use the advance switch as it was not clear if we will go that route. So I did not change the rest API. But now it seems clear that we are going to drop that forever so I will change API. This will imply some more changes in the code that uses capabilities client.
Hmm. Just to be clear, before addressing the semantics, the intent was for UnfilteredView to completely bypass the authorizer and for the other to do almost exactly what get_filtered does. `Getter` offers a bit more than just the authorization, in terms of dealing with some of the relationships so I wanted to be able to use that without necessarily having to go through the authorizer.

My understanding of 'filtering' in this context is the use of the authorizer to determine which attributes/rows are screened from the resulting django QuerySet. `Getter` does more than just restrict the returned rows, it limits the attributes in every row which works exactly like a filter. I also thought that was the implication of 'filtered' in `get_filtered`, not in the application of the filters argument which is used to create a specific query. I can see why the two meanings would get confused since the terms overlap. I suppose the question is which meaning of filter is more intuitive?

I'm not opposed to changing the names to something else. However, the term 'Censored' doesn't convey an obvious implication to me about how it will work (since it isn't used anywhere else).

I had initially considered AuthorizedView and UnauthorizedView but I was concerned that both imply some use of the authorizer which is only true with the former.

The rational for using FilteredView was that it should be a drop in replacement for the use of get_filtered in view methods. UnfilteredView was initially just View, but then I wanted to be more explicit about the bypass of the authorizer.

On the bright side if we do change it, it should only be 3 line change.
Does translate.foe-services.de require the translation dir to be `/languages/` or is it just your preference?
This one is tricky. I have a test but it is not related to a use case. Deep down I do not think there is a valid use case for this (ie overriding based upon a class hierarchy). But since many people do horrible things with their class hierarchy, I have added the feature to cope with that.
Can you add documentation for the `with_value` and `with_unconfirmed_value` methods as well?
This interface may have changed as part of PR #1523 and may need to be updated as a result.
KrollProxy can also be returned from Java to JavaScript without need for explicit creation. Since KrollProxy is the main base class of our binding (and therefore Module) system, we should probably add some more details here.

Take a look at the brief descriptions outlined at the bottom of the Android Module API guide here:
https://wiki.appcelerator.org/display/guides/Android+Module+Development+Guide#AndroidModuleDevelopmentGuide-AndroidTitaniumAPI
Extended note pushed into this pull request. The point is that the Serlvet API shouldn't be a dependency for the core management library because not all web frameworks we use/will use are Serlvet API based.
`Returns _YES_ if the font size is not set, _NO_ otherwise.`
ja, maar dit maakt toch nog steeds niet uit, eerst wordt gechecked of het token bestaat, indien dit bestaat wordt het user-object gevuld, anders is user gewoon nuIl.
Indien token niet null is zijn er dus nog twee opties:
  - $user is een object van User
  - $user is een string

Indien dan $user een User is; wordt $user in een logitem gestoken, in alle andere gevallen is het user-object in het logitem gewoon null.
Why did you remove this? Sounds like a handy tip IMO.
is there a reason why this this all caps? usually we use this for macros
add as an optional param.
Is this comma proper? I'm no grammar expert but it reads odd to me.
I don't understand what context you're looking for. If you pass something that isn't of type `settings.ARMSTRONG_SECTION_ITEM_MODEL`, weird things could happen. The code uses the model class to determine the name of the related manager to use. I don't know of a clearer way to say that.
`(- x (+ x x))` = `(- x)`