labofsahil/github-event-dataset-2012 huggingface dataset:
When returning in the if-statement the variables on line 104 and 105 are never used.  It would be best to move the variable declarations to just after the if statement.
space in between slash and comment
I'll review this entire spec. I assumed since it had been reviewed and committed already (I was only moving it) it wouldn't require extensive review.
Here's another strike against use of the module: we're testing it inside `group_private_messenger_spec.rb` which seems to cross county lines.
Github is at fault for believing that the tab character is 4 columns instead of 8. I'll leave it as it is here and come up with editor settings and commit hooks to improve it for future commits.
what is domain.size()/2 ??
the .size() method is not in the domain concept and a Value is expected...
hah good catch, it did work until i changed it ;)
It seems that we do not have aws-java-sdk-1.3.14 in artifactory yet. This will cause internal build problem.
Isn't this already in fmath.h?
I think I will get rid of the ```cartopy``` on the attribute name.
remove this. we have no dependency on pyplot here...
A transform rather than a projection....
Two comments:

1. The idea is that within a `recover()` block we'd store the stacktrace info from panics. I'm calling them "exceptions" to maintain compatibility with [semantic_logger](https://github.com/ClarityServices/semantic_logger#semantic_logger).

2. The `Exception` type is removed in later commits; this info should be part of the `Payload` struct.
Pffffff ;)
Very good point piquadrat. It would be embarrassing if someone decided they had to bundle their own version of django CMS with their project because it had an unstable API and too few warnings about significant forthcoming changes...
@Sunight reuse identifier的值`@Cell`最好自己指定一个，另外这自动生成的变量名我一直觉得怪怪的，强迫症发作时一般都会改掉
This could be less tied to the implementation, e.g. "When I click the link to create a new listing"
Also: this feature doesn't make it clear that the landlord is creating the apartment. If I read down this scenario, I see that a landlord exists, that someone clicks a link, fills out a form (what form? "fill out the form" is pretty generic, and will probably bite you later on), and sees some apartment.

Here's an improved version:

```
Scenario: A landlord lists an apartment
  Given I am signed in as a landlord # you can create the landlord (via the site!) in this step too
  When I click the link to create a new listing
  And I fill out the new apartment form for "97 Walden Street" # I'd submit the form in this step - there's no scenario (yet) where you fill it out but don't submit
  Then I should see the "97 Walden Street" apartment in my listings
```

The advantages of this version are:
* Less generic, higher-level steps: "the link to create a new listing" (which deals with function) vs "the Create a new Listing" (which deals with implementation); "fill out the new apartment form" vs "fill out the form"
* Less need for shotgun surgery down the road: explicitly deal with a unique aspect of the apartment, like the address, rather than assuming that you'll always use the same test data. The other advantage of introducing variables is that you can use descriptive names like "123 Not Listed St"
Tu passes un username (par exemple "jdubois") dans une méthode qui attend un login (donc un email), je ne pense pas que ça doit être bon
I plan to add that later.
I don't recall seeing "MESSAGES.failedToGetModuleAttachment" happening in the past, so I'm not worried about the change to ignore it currently.  In the future, that could change though.  

Could you be more specific in the comment about which type of (unresolved) OSGi bundles would not have a module attached.  

How about the case that the OSGi bundle is resolved for an OSGi Persistence bundle.  How does that integrate with EE processing?  I mean, can the developer use both OSGi persistence (e.g. no EE @PersistenceUnit/PersistenceContext injection or equivalent) and EE container managed support?
Richard D. Worth please. Thanks.
In both case you're checking the same value, let's make an intermdiary variable above and return early if it does not match what you expect.
That sounds a nice change. Let's ask Jose to validate it.
the same, it have to be localized in other pr, thanks
Couldn't this just be

this.onReceivingFileConfirmation.bind(this)
This callback has the same name than the function we use to 'show' the logo, makes sense for you set a different name?
Updated! Now we have 'error' and 'failed'!
mess -> mass. What a mess! :)
I guess I’ll rather make it async right now.
Why not removing this style if you don't use it at all?
we need to talk F2F today about this issue
`<a data-l10n-id="lowBit">Low Bit</a>`
@ctshryock You're correct, format is required here.
Somewhat strange logic. Would be easier to read if there was a 'SolutionContainsProject' boolean helper.

Also, what's the scenario where the matching count is more than 1? A solution containing the same proj multiple times??
Sorry misread. Never mind.
https://github.com/greggroth/rspec-core/commit/f620228916b2b549266838fdd98a58de0aafe54c
I'll patch that in now.
Nope. Without the delay the browser won't animate :-)
yey, our first macro! :+1:
Presumably you would never use ```HBaseSchemeSource``` outside of ```HBaseSource``` so why not merge them into a single class? That means that you would not need the class ```HBaseTable``` anymore
just a style correction, please change to:

    if
      __rvm_has_opt "errexit"
    then
      rvm_stored_errexit=1
    fi
    set +e

or:

    if __rvm_has_opt "errexit"
    then rvm_stored_errexit=1
    fi
    set +e
These constants are cassandra specific and not global to all datastores.
strpos()
maybe just `this.isURL = false` with no condition
You don't need to do that on init.
There's g.BOARD for the current board.
You shouldn't use this value anyway because of inlined cross-board posts.
same as above comment about _ vs - compared to the rest of the ids/classes
@zml2008 Will do
better to change the above 3 lines to:
```
pool_options = params.get('pool_options', '')
```
Well, the import part is just print out a waring msg, because we may import this file in some framework modules, and these module would be use in a no-json env, eg RHEL5 host. But these env doesn't really use this `GuestAgent` class, so it's safe to import this module in framework.
These lines above aims to make sure we really get the correct json module which is a must for `GuestAgent` class, if there is no json module, then raise an exception.
With these 2 parts, I think we can make everything goes well, both test cases and framework.
If `platform` needs to be specified and `None` isn't a valid value, you shouldn't be setting it as a default value here. Also, if `None` isn't a valid value, you should check that it isn't set to none, and do `config.error` in that case.
This decorator should return something meaningful.
Probably just template.
These three methods (`getRepository`, `getBranch`, and `getRevision`) are only used in this file, and each only once.  I think that the invocation in `create_sourcestamp_without_changes` could be refactored to use `.get` and eliminate the need for these functions entirely.  Barring that, I'd like to see them named e.g., `getRepositoryForCodebase` so it's clear what the argument means.
Sniffs added in local dev setup. Have taken them from here: https://github.com/opensky/Symfony2-coding-standard
Well, this doesn't work. I'm happy to help with this but I'm out of ideas. I can't seem to issue standard python commands from within the ipfw.conf file without an error
https://github.com/tsigo/gitlabhq/commit/6033b200fc185ea379b593d8cf40b507fe56d14f
try wait() in conjunction with run() ?

see https://github.com/pivotal/jasmine/wiki/Asynchronous-specs
afio_exti_num is deprecated and STM32F1 only. This breaks the build on STM32F2.
shouldn't this return an element? not a node?
Ah, I didn't know Map existed. MDN doesn't show a way to iterate one though.
Ok. I won't use a png then, but just a string.
Yeah, duh. Earlier this morning I was intending to do that, and then completely blanked on doing it. I will change this.
Well, sorry. It has been addressed sort of but requires another update as given by my latest review.
All the new code should use MPL 2. We can update the rest of the files with a separate commit.

http://www.mozilla.org/MPL/headers/
__whimboo__ commented on commit
> ...
>  Also please separate out getAddonsByTypes so it can be read and understood easier.

I had no idea how to do this nicely without #90. :(
After checking 392a9b8bd4a4 please advice me how to separate out `getAddonByTypes` more!
I gave it a try in the commit above, but any comments are welcome!
if (null === $validator) {
Nevermind, talking in IRC this doesn't seem to matter.
Why is the symbol :utf8 used here? Isn't the flag on `toprint` boolean?
I think this change is no more compatible with Mac OS.
We should also introduce architecture in the name of the agent, something like "libgcprof.Linux.x86_64.so" for instance.

What do you think?
what's the intention here? we can write to the document or something instead
if __debug__:
I expect an update(s) similar to postfix one. So, I left it here as a clue.
there are spaces, there used to be tabs.
I fixed the above when merging.
Those comments are quite useless... Could you fix them with real text (or strip them?)
For each character strtoupper was called - that slows down and isn't needed by spec. Therefore I removed it and lower case this one, too.
This component must use this way because there is a code for try to guess the setters.

```
foreach ($options as $name => $option) {
$fname = 'set' . ucfirst($name);
$fname2 = 'is' . ucfirst($name);
...
$this->{$name}($option);
```
@Maks3w What @bakura10 says is correct -- data is not bound to the object unless validation is successful.

@bakura10 instead of doing a conditional, you could simply call ```$form->isValid()```; you'll get an exception raised if you then call getData() anyways, which will cause a test failure -- which you'd want to have happen anyways.
@weierophinney @ezimuel I think that we should rename `timestamp` to `date` if this PR go on.

Breaks the backward compatibility but I think that is better to have the correct name for the value that it represent.
написать try ?
You are correct. I did some additional testing, and any helpers would be included in the final array twice. I think removing the files array before applying the files_filter will fix that.

``` ruby
specs = Dir.glob(File.join(specs_dir, '**', '*.rb')) - files
```

Will that work?
When I ran it against lithium core there were already a few syntax checks that would need to be fixed. It's definitely on my todo list! :D

Being a new dad, I'm sure you can relate to the lion king "There's more to do than can ever be done".
Line break :/
@samiluukkonen Tässä sit ku tän siirtää muutaman rivin alaspäin, niin on laskurow haettu ja tulosta_tyomaarays.inc requiretettu
我赞成
Missing throws documentation
I was always a fan of moving PersistentCollection into `Collections` namespace.
These classes seems to me more like a `Collections\Filter` namespace or a more specialized namespace.

Also +1 on moving PersistentCollection
can you make the nagios name a variable? incase it will become eg the icinga user?
deal with this in ldap.search ?
파일 이름 뒤에 버젼을 넣어주어 설정으로 빼두어도 괜찮겠네요.

업데이트시 버젼만 바꾸면 될것 같아요.

static.js?20120820 처럼요.
1. I'm not sure I understand what you mean here. I add `name` only in line 50 so there is no double `name` entry in the filters. About the OR operator: `name` is indexed in elasticsearch both as `name` and `fullname` with different elastic search analyzers (`not_analyzed` and `whitespace`) so to query both I use the `|` operator.
It's a sentence, it should end with a period.
Should underscore properties that are plugin only.
i almost went that route! ^_^ will try the accessor way
ohhh ok I'll just change the string to mention `req.signedCookies`
というか .all 明示的に書かないとエラーになるのかな?
Munoh だと"Undefined Method 'map'"というエラーになりました・・・
renderするときに必ずSELECT * FROM munohsされており、
Munoh.allでもMunoh.scopedでも挙動が同じだったため
Munoh.allとしました。
If something requests an update after run has been called (e.g. a custom plugin that does something weird) , then it won't be sorted.  It'll just be at the end of the list.  This should only be called a few times per run, so there's no worries about time lost.  Also timsort is about O(n) for mostly sorted lists, which it will be since it's called every time.
If you are running foreman on a machine in a colo, this way you get the correct external IP of the compute instance.

If you are running foreman on a machine in EC2, you get the correct internal IP of the compute instance.

This was the simplest way to determine what effective IP foreman should use to communicate with a compute instance, without tying it to any particular cloud provider's networking details.
Wait?! Where did this come from? Can you get rid of all `@import` statements?
Gotcha. Please make sure that this is updated on the phoenix thorax 2.0 branch.
Add comments here describing what setup is doing here.
I wanted to change as little code as possible, since I can't test it. This is just copy-pasted from below, but you're right. Someone who has access to Scopus can perhaps bring the whole translator up to date.
you're right. Fixed

On Thu, Mar 15, 2012 at 4:28 PM, Simon Kornblith
<reply@reply.github.com>
wrote:
>> @@ -306,6 +306,29 @@ function addHighwireMetadata(doc, newItem) {
>>               }*/
>>       }
>>
>> +     //Deal with tags in a string
>> +    //we might want to look at the citation_keyword metatag later
>> +    if(!newItem.tags && !newItem.tags.length)
>
> I think this should be an ``||``, no?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/zotero/translators/pull/210/files#r565901



-- 
------
Sebastian Karcher
Ph.D. Candidate
Department of Political Science
Northwestern University
bumped to f.



not really, renamed to event ;)
Weird linebreak.
Do i understand this correctly?
You don't merge all elements from $sequences[$k] but only the first element of every sequence?
This is not very clear in the comments
`function(elem) {`
why reg. exp? this is really just a find/replace thing :)
connects

Also emphasise that the below code is server-side.
I'd prefer this to be at or towards the end.
This is not necessary as ActiveAdmin contains an AdminUser.
It's bieber, not beiber...
No worries :) Good job cleaning up some of our mess, haha.
No debería hacer diferencia pero la hace (en las pantallas de detalles), ¿no se estará cargando correctamente el reset?
this can all be done seamlessly with one check:

```ruby
def content_for(part_title)
  content_for(part_title).present?
end
```

`.present?` returns true for non empty strings / non nil objects.
I can have the config honour "strategyClass" and "strategyOptions", but I don't think we want to default to it in our dev. environments. I think there is extra configuration of a snitch required. "OOTB" configuration doesn't let me manipulate data if I create the keyspace with NetworkTopologyStrategy.
What is the problem with this test? It no longer calls a callback.
I was worried about that, but couldn't find anything that suggests that `nextTick` actually gets prioritized as the very next tick, v.s. just getting tossed into the event queue. If that's the case, I think I'll just ditch the nextTick, as `_reindexContent` is asynchronous. The difference is that it's not as clear that there is something asynchronous happening.
Is there a reason why you take this out of the anonymous function?
Probably not all that much overhead. It checks if the Content-Type header is set to application/json.
I'll add it back in.
The UI needs to know where to post to. Before, when the endpoints were at `/api/config/:tenantId` for all servers, we didn't need this. Now, when on a tenant server or the global admin the UI posts/gets to/from `/api/config`, if on global but looking at a tenant it posts/gets to/from `/api/config/:tenantId`.

So making this part of /api/tenant would mean that I do not know if this data would be at /api/tenant/:tenantId or not.
s/shutted/shut/
Our style guide says that this should be kept under version control: https://github.com/thoughtbot/guides/tree/master/style#rails
Lets switch this method name to be call instead of perform with the idea that procs could be involved in the future
why do we need to prevent propagation? wheee is not clear ;-)
Since it will not allow you to choose any production centre now, can you please take the checks for json out of it?  It'll only be needed over REST.
`init` - это как бы "конструктор по умолчанию", и мы его для нашего класса **переписываем**. Обратите внимание, что конструктор не принимает никаких параметров. 
--
Your sincerely,
Sergey Lebedev.

Twitter: @drasgo
Facebook: s.a.lebedev
he doesn't use it i guess
Tук не е ли "higher precedence", а не "lower precedence"?
Nvm, ik zie de @deprecated nu pas :-)
@codyhoag 
- With regards to a site running more efficiently and being easier to maintain by using OpenSocial gadgets, I think this needs to be fleshed out a bit.
- Did you mean to emphasize OAuth's support for numerous web technologies or Liferay? I wanted to double check.
I ran into a problem with using an optional argument.   It could have been a bug in my original implementation, I'll revisit that in a subsequent commit
How about updating the module's lackluster docstring to be: `Preprocessing with SSP, artifact detection, and ICA`. Then you can delete the admittedly somewhat confusing intermediate heading. I don't think they need to be separate anyway...
It wasn't broken. Have you actually tested what your change does?
I see no test for this
Can we get a sample of "not found"?
Good catch! Removed in 21df288.
I'm unaware of any other way to do a cpk in oracle.
You should double each `\`:

    :class:`Symfony\\Bundle\\FrameworkBundle\\Console\\Application`
We shouldn't use `var_dump` functions in our doc. `var_dump` is to debug code and documentation examples shouldn't be debug coding. Place something like:

    $input->writeln('The locale you choose: '.$availableLocales[$locale]);
then no need to revert
should be `/{_locale}/contact` (same for xml and php format)
<registered>Microsoft Visual Studio</registered>8和10是已知可用的
=>
已知<registered>Microsoft Visual Studio</registered>8和10是可用的
可以加上
应该加上
cool, added
NSKeyValueO-<b>b</b>-vserving_BlocksKit?
could we turn this ino a hyperlink?
Just tried that, but it does not work.
не наш. Я сам офигел от этого ужаса
Зачем тут `QObject::`? И так покатит.
Me parece que son distintos `m`, puede ser (no tengo el código enfrente)?

On Tue, May 29, 2012 at 11:02 AM, Alpha <
reply@reply.github.com
> wrote:

> > @@ -115,6 +115,22 @@
> >              initialize: function () { }
> >          });
> >
> > +        // Nervoustissue.DataLinking.Compound
> > +        // -------------------------------
> > +
> > +        // Se encarga de registrar el evento change en un modelo
> complejo de backbone.
> > +
> > +        m.Compound = m.Base.extend({
>
> Está declarado dos veces, uno en la línea 123, el otro en la línea 624.
>
> Seguramente el segundo esté pisando la definición de este.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/CommonJobs/CommonJobs/pull/89/files#r892293
>
No it cant because if $selectedBasketId is empty && $baskets->count() === 0 $selectedBasket will be null.
It's not bad idea, but i don't know how it would be integrated... Maybe just managing the addition and deletion automatically on access to its elements, overridding __getattr__() and __setattr__(), so user is totally unconscious if it's using a cache or not? Are you thinking something like this?
Are you sure that was a Sat/Sunday? Would be an unrealistic camp otherwise.

:trollface:
UA requirements.
Vielä pitäisi tarkistaa, että sekvenssien alkiot ovat samat. Ikävä kyllä testit eivät tätä huomaa.
that's tempting indeed. I didn't want to change the semantics: since we
might have multiple instances instantiated at different times, it currently
reacts to logger configuration changes.
+1 to make it static - as we usually do - as long as it's master only (not
5.1)

On 12 August 2012 14:22, danberindei <notifications@github.com> wrote:

> In core/src/main/java/org/infinispan/marshall/VersionAwareMarshaller.java:
>
> > @@ -52,7 +52,7 @@
> >   */
> >  public class VersionAwareMarshaller extends AbstractMarshaller implements StreamingMarshaller {
> >     private static final Log log = LogFactory.getLog(VersionAwareMarshaller.class);
> > -   private boolean trace = log.isTraceEnabled();
> > +   private final boolean trace = log.isTraceEnabled();
>
> Why not static as well?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/infinispan/infinispan/pull/1238/files#r1358388>.
>
>
This will no longer be necessary if `[Verified]HTTPSConnection.__init__` accepts a `socket=…`.
Of course, a comment would also work fine to explain that...
this returns some false positives:
self.TableRegion(self.testsetup, 2) == self.TableRegion(self.testsetup, 1)
s/_header_loactor/_header_locator
Changed it to "Indentation" now
Yes, that should work :)
`editingGroup` needs to be false here since we're creating a new group.
No I don't think so, images seems to scale down correctly (both long and tall images).
I did notice however that the height of images doesn't get scaled down in IE8, but they do in IE9.
If we want images to scale properly in IE8 I think we'd need a javascript solution such as jquery.imagefit.js that I had in a previous commit.
What do you think?
Good point. Only the last one should have the undefined check. Fixed.
Could you please describe what is the motivation to have 2 parameters here?
Is there any reason to not make this private?
Sure, it could be useful -- e.g., if cursors were open during a previous checkpoint, keeping an obsolete checkpoint alive.
nit: I'd say this and the removal of `url =` in the previous block makes the test look a little confusing. (ie. What does this statement do?) Assuming this is a cleanup change and not functional
From what I have seen so far, it looks like brunoais is experiementing on his branch without creating a new branch - there was something else which shouldn't have been on this pull request, too.

@brunoais: In the future, either experiment on develop (it's very difficult to accidentally commit stuff), or create a new branch.
Are the indexes for this array documented anywhere? ... Should they be?
Rspec is absolutely my preference.  I believe @Stubby feels the same way on this for testing.
It might be worth adding can_do and set_client_id to this list.  can_do is the primary way to for workers to announce their presence to a server, which might be nice to see.  I mention set_client_id only because the gearman-ruby worker will send a set_client_id packet first, before any other packet, when it connects to a server.

Thoughts?
I would also recommend file comments at the top of each file. It seems lame in this fast past world of node, but these are the kinds of things that will encourage hacking but others.
Вы же делаете это в index.php
I think that es_server and es_config should be part of the "settings" passed in.   This function is supposed to create datastore from a parsed settings object, and if you need to split out es settings, then that reduces the "convenience" of it.  Also, I'd prefer the default be that there is no ES plugin enabled so that agamemnon remains backward compatible with existing applications that use it.
It should be, since a nil check should just be a check if the `object_id` == 4, whereas a comparison has to perform a value check. The big optimization there is avoiding creating a new string to compare against for every invocation though.
Código Java
Probably smart. Though i was hoping you could fix some of them. There's a lot of == misuse in the code.
That's the default output of `bundler gem`. Is it wrong, then? Should this be fixed upstream?
Aggh. Why doing this in dispatch? I bet you do not know that in `self.request` you get `request` when used in any class method?
This is not the best way. You should keep global variables to minimum. This would be much better to define image in template, as HTML, just by default you do not display it (with CSS). And then you use it wherever you need.
You should use `django.utils.timezone.now()`. You probably want that?
Do at the end.
You could make a form which simply have the same hidden name as comes from Facebook (`code`). And this form would then automatically be populated from GET data. Which would be missing choice. So it would fail. And after resubmit it would succeed.
Style: space after method name.
Does `a ++= "c" ++ "d" ++ "e";` work?
URL needs to point at a stable release, master is a moving target.
now everything works fine
You forgot {% endfor %} before this line.
Перенести в use
This was the only instance that is not pre-formatting and validating the order-id. I added a test case.
Why did you remove the lazy loading? This will fatal if the event manager has not been set and githubToLocalUser tries to call trigger on null.
I think this `action` needs to be needs to be ``http://astropy.readthedocs.org/en/latest/search.html``
Can we create a URI from $authority to make it resolvable?
It seems we could simplify this to:
hours_data = {'dates': []}
I think this class is not used anymore
globalObj isn't defined. It will lead to compilation errors.
I agree that this note is a distraction from the point of the chapter.

On the other hand, I do think our examples should use raw strings, because that's what `startproject` puts into `urls.py` by default.  Therefore, I feel that the question would naturally occur to new users if they look at our examples vs. their urlconf and think "Why do these strings start with 'r' and the ones in the book don't?"

What do you think about using raw strings, but leaving out the explanation?  Or, we could change this inline explanation to be something short like "The `r` indicates a raw string instead of a normal string.  To find out why you would use one instead of the other, read the note at the end of this chapter." and then put the longer explanation at the end of the chapter.
Yep will fix in the am
On Mar 13, 2012 6:38 PM, "Jim Klo" <
reply@reply.github.com>
wrote:

> > \ No newline at end of file
> > +        yield ']}'
> > +    def _orderParmaByView(self,params,view):
> > +        startKey=[]
> > +        endKey=[]
> > +        def populateTs():
> > +            if 'from' in params:
> > +                startKey.append(self._convertDateTime(params['from']))
> > +            else:
> > +                startKey.append(self._convertDateTime(datetime.min))
> > +            if 'until' in params:
> > +                endKey.append(self._convertDateTime(params['until']))
> > +            else:
> > +                endKey.append(self._convertDateTime(datetime.max))
> > +        def populateDiscriminator():
> > +            if 'disciminator' in params:
>
> RE: lines 80 - 81
>
> Shouldn't it be dis**crim**inator instead of dis**cim**inator?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jimklo/LearningRegistry/pull/4/files#r554742
>
Haven't tested. I'll test and update. Please defer merge till you see another update. Thanks for feedback!
yes, will be removed here once he merges master.
please add documentation here, otherwise it is not simple to see the intent. As I understand it :

"for the the first record returned, pos_ should be 0 if the recordreader is reading the first split, otherwise it should be end of current lzo block.
 This makes pos_ consistent with LzoBinaryB64LineRecordReader"
As you can see in the diff above, i didn't touch this line in this PR, just replaced "available_commands" by "it._commands['all']"
Be careful here - this works in English, but doesn't work in Turkish to my understanding (something to do with having two different letter 'i's is my understanding). Depending on which human languages you expect this to be used on you may be in for a world of hurt here.
No, it's not. Nice. It would work the same way without Boolean, but `use_raw_boundary` would be a string then, I guess it's cleaner.
Please update to point to the orig project not a fork
GetFrameTransform will return the identity matrix if the players frame is the root frame. Tomm appears to have put it there just in case hyperspace somehow ends up with additional frames in the future..
If you check the result of `${^path}/virtualenvwrapper.sh(.,@N)` you should be able to know if there is `virtualenvwrapper.sh` in your path (even if it's a symlink). With that you don't have to worry about wether it's executable or not.
Thanks for the suggestions.
I will incorporate them in the next version.

Regarding the functions vs. classes for kernels. I thought it would be
easier to implement the generalized product kernel estimator with
functions rather than with classes but I may be wrong... I will spend
some time on this issue.

Thanks again.
George

On Mon, Apr 16, 2012 at 3:15 PM, Ralf Gommers
<reply@reply.github.com>
wrote:
>> +
>> +    #TODO: Extend this to handle the categorical kernels
>> +    var_type=var_type.lower()
>> +    mykern=kernel_func[var_type]
>> +
>> +    n=len(X)
>> +    LOO=LeaveOneOut(X)
>> +    L=0
>> +    i=0
>> +    for X_j in LOO:
>> +        f_i = sum(mykern(h,-X_j,-X[i]))*1/((n-1)*h)
>> +        i += 1
>> +        L += np.log(f_i)
>> +    return -L
>> +
>> +from scipy.optimize import fmin
>
> Imports should go at the top of the file.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/statsmodels/statsmodels/pull/228/files#r690550
certainly is. 

fixed.
We talked about this over Skype, but I'll record the answer here too:  There are conceivably times where we would want to merge arrays and not just hashes.  I'll change this code to merge arrays more intelligently - that is, iterate through all the elements and recursively merge arrays and hashes that occur at the same indices.
Are you sure that this function call is working in Zikula 1.3?
Is it still possible to use these kind of legacy hooks?
The usual convention I follow is to put `function` on the same line and then indent the contents by a single tab
This function rounds any number, so perhaps a more general name could be used, such as ```round```, or ```roundNumber```
broke out into the following methods:
    private void whenProcessingMessageReturnMockResult() throws MessagingException {
        when(mockSmtpAgent.processMessage(any(MimeMessage.class), any(NHINDAddressCollection.class),
                        any(NHINDAddress.class))).thenReturn(getMockMessageProcessResult());
    }
    
    private void processAndSendMultipleMsgs(int numberOfMsgs) {
        for (int i = 0; i < numberOfMsgs; i++) {
            intDirectClient.processAndSend(getSender(), getRecipients(), getMockDirectDocuments(), ATTACHMENT_NAME);
        }
    }
My only objection to that way of doing things (which, I agree, is quick and works) is that I imagine then the "blame" feature doesn't work, since I will seem to be responsible for that file?   I mean, I don't think GitHub will recognize that this is something that was there before and will consider it a new file created by me.  But, it's documentation rather than code so maybe that's okay?  Do you have any thoughts on this?
higher -> smaller
Also might be worth clarifying that e==distortion definition of ellipticity.
No edit, really like this as a multi-document config file.
I admit raised my eyes a little at this but chose to retain it from the earlier version out of deference to whoever wrote it first time around.  But you're right, this is a little disingenuous.  will push an edit.
merge bug!
defined at https://github.com/RiotGames/rbenv-cookbook/blob/master/libraries/chef_mixin_rbenv.rb#L84
It turns out this recipe already had a stanza to make the old working dir location world-writeable, so just repurposed that.
Isn't this a private function and should be prefixed with _?
What is this for? There doesn't seem to be any extra functionality - is it intended to be a mock point or something along those lines?
Ah, I see: the "name" is a logical name. But I think the `keystorePath` is a bug.
There is no lapse of time. He's paid till 4/1/12. 
To change from monthly to yearly we're going to need to cancel the previous subscription & create a new one starting on 4/1/12.
Why 10%, this seems like an arbitrary limit? You can push as much data as you want, you'll simply tradeoff against collisions.
Why is this `redirect_to` inside the `scope :locale`?
Yeah I agree with @crynobone here. I think we should remove that.
Then, why not using that on all the methods?
don't align by the equals sign. it makes your eyes read down the page, instead of reading what the code is actually doing.
2 typos here: should be  "or @Template annotation as demonstrated in DemoController"
No, I left it out completely. Either way, the project would fail to build on it's own, you would need to follow the steps outlined in the tutorial (Add the Airship folder and link the library). I took this route to avoid having  a dated version of the library appear in red.
I'm actually working on this exact transition right now :)
It is getting create_message.html.  See line 39 of inbox.html.
Does the concat method is not enough here?
Please revert this file and redo the pull request.
Why create a new plugin for this task? It can go directly into the CopyOnlySubmorphsPlugin.
Just a note, since this is an interface.... the Pipeline and the DataManager are concrete classes;
Not everybody has a bashrc in their dotfiles project, maybe make this configurable so zsh'ers can use it?
Там ведь получится два обратных слеша
Не понимаю, если директория существует, в чём проблема туда скопировать что-то. Почему обязательно надо создавать?
В выражении вполне могут присутствовать пробельные символы.
Thank you for your explanation. This confirms my understanding of bug 751949. I was also worried about these changes, hence the time I took with testing. I will test more.

Would you like us to avoid these changes?
Hey Ionut, here we need to be careful here with the distinction between present and visible, for a few reasons: the main reason is that `is_element_present` merely checks that it is present on the page whereas `is_element_visible` acutally verifies that the element is visible to the user (and hence interactive!). To know which one you need to use you need to look at the step of the test and which it requires.
Then you need to label the property with the right one so that it is consistent for future users to use and understand :)
Could you write:

```
try {
    if (...) {
        return true;    
    } 
} catch (\Exception $e) {
}

return false;
```
@stefanv concerning the auto_examples, I moved some files into the directory, but the doc compilation fails (make html), I guess that the skimage used is the systems one, not the dev_ one.

Any idea on how can to force the use of the local skimage module ?
Could you add a note that this test is known to fail on OSX 10.6?

See the following thread for more information:
https://groups.google.com/d/topic/scikit-image/hOJhyDRzOG4/discussion

I never tracked down the source of the error, but @stefanv was able to reproduce the failure on a similar system so I don't think it's just me.
Thanks.

On Wed, Oct 3, 2012 at 5:43 PM, Lars Buitinck <notifications@github.com>wrote:

> In sklearn/rbm.py:
>
> > +        """
> > +        return self.mean_h(v)
> > +
> > +    def mean_h(self, v):
> > +        """
> > +        Computes the probabilities P({\bf h}_j=1|{\bf v}).
> > +
> > +        Parameters
> > +        ----------
> > +        v: array-like, shape (n_samples, n_visibles)
> > +
> > +        Returns
> > +        -------
> > +        h: array-like, shape (n_samples, n_components)
> > +        """
> > +        return self._sigmoid(np.dot(v, self.W) + self.b)
>
> There's no input validation here. Also, with safe_sparse_dot, sparse
> matrix input can be supported trivially.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/scikit-learn/scikit-learn/pull/1200/files#r1755913>.
>
>
Still missing an ``l`` here ;)
supporting `n_jobs` would be nice :)
Can you revert this chunk ? The convention is this way:

    local context curcontext="$curcontext" state line ret=1
    typeset -A opt_args

(remove what you don't need)
We are trying to reduce usage of jboss-common-core. 

I would suggest moving org.jboss.as.messaging.deployment.PropertiesValueResolver into somewhere more central (probably the server module) and using that instead.
True, but to me it was either make all command methods static so the SimpleInjector instance could be removed (since that object isn't really necessary either), or pass in the plugin which was used by both command classes anyway.
Added Jira task for it:
https://jira.appcelerator.org/browse/TIMOB-9600
change to
if (right_ != NULL)
That scenario crashes on android, so probably we don't to do that.
need to check the other bounds
This handles Visual Studio's floating WPF windows. I could not yet see any bad effects on any other windows.
Another package, eh? Making more work for me ;0
bah.  This would have had a +/- 0 stat score if not for this newline
You may do just as well if you spilt the whole hash table for the last row, but kept it allocated so subsequent rows were still deduped.

You would lose possible dedups against existing keys - but if they're common they'll occur again soon - but may gain if the dedups tend to be clustered.  I suspect it is highly data dependent.
If you remove the microseconds from one of these, will the date parsing bit still work?
I loved that you used a namespace. Thanks! :-)
Is it using same algorithm than you? (sub-objects with obj[attrName])
Sounds like we should either implement same algorithm and switch to it when it works.
Or just do not plan such thing and completly ignore this component.
Hum I think we should make a decision. It feels weird to write in documentation that a method has multiple names. (serialize sounds like a good choice)

As said  yesterday, I disagree with the extreme Applescript way of doing, where we try to write english sentence instead of write imperative/functional/object/prototype code. I'm totally fine to try to use short name, if we do not loose important meaning/context.
I'm not suggesting to go into the other extreme and use 10 words for each name, either.
Yes, maybe this example can be confusing. I'm changing it.
You haven't implemented `isActive` setter, that was implemented before:
https://github.com/mozilla/addon-sdk/pull/312/files#L3L49
Fix the spacing on the <h3 > please
This is a detail, but my eyes would be a lot happier if these "(for logging only)" strings were all lined up :)
For naming, I'd be more comfortable with:

* `_cached_user_permissions`
* `_get_cached_user_permissions`
* `_user_permissions_cache_filled`
* `_cached_group_permissions`
* `_get_cached_group_permissions`
* `_group_permissions_cache_filled`

In light of the `check_groups=True` being user-inclusive issue, you also might consider adding another cache that's `_combined_permissions_cache` containing the highest permission level combination of user + group (so if either has approved=True, then it's contained).
Don't actually think False would be valid here.

In the template:
* `crop`  means `{'crop': True}`
* `crop=smart` means `{'crop': 'smart'}`
On vérifie que le paramètre is_admin n'est envoyé que par un administrateur, sinon autant le mettre en tant que attr_accessible.
I don't think there was any way this could ever evaluate to true before change made?
PEP8 - function name should not use camel case. How about ```filehash()``` or ```file_hash()```
I have used a temporary file to let puppet-tftp handled the file permissions.
On the other hand, if this was on purpose and you expected execution of the script to continue, you should make this a print_warning. The other print_errors should still be fail_with
Unnecessary, already registered by HttpClient
Oh wait, I see that you're iterating over `x`, which spans the height, and `y` which spans the width... that's a bit confusing.
well this is simplecms so fetching content and route is just two noop method calls that return $this.
Never this style of indention please.
Avoiding contractions is a good idea - we've avoided it so far :-)
That makes sense. Do we want to just add it to the description or should I add a new tag either inside or after <desc> so we can style them in a certain way?
Seems to be something odd going on with Scaffold. Test passes in Scaffold as it is here, but if you go to Edit Import and then run doImport, all fields show up as before.
Let's just keep the `instanceof` for now.  When we add the post-processing stuff ([wiki](https://github.com/AnalyticalGraphicsInc/cesium/wiki/Screen-Space-Rendering-Details)), we'll change this into something more generic.
Why do we need static versions of functions like this?  I get it for arithmetic operations - the things we might use operator overloading for in another language - but why does every method need a static equivalent?
why preserve newest?
hi - could you file an issue in http://issues.apache.org/jira/browse/CB so this issue can be tracked and you can be notified of updates? Thanks!
Is the intention to have one codebase for the UI that will run on all our platforms, albeit with some tweaks to handle platform-specific behavior and support, or to have a separate codebase for mobileweb? If we change the purpose of something as fundamental as a window in mobile web, then I imagine we have the latter.

Our approach so far has been for the APIDOCs to reflect the actual, true, behavior. I will speak to Arthur about whether we should do that in this type of situation also.
Will do, for some reason the other day whatever build of dart editor I had kept on blowing up with the error callback handlers. I'm guessing it had something todo with recent  positional optional argument changes.
не сработает в этом случае по-моему так что хер с ним
просто читабельность

    panel.append(JST[tpl]()) if JST.hasOwnProperty(tpl)

стремится к нулю, на мой взгляд, в сравнении с скажем

    panel.append JST[tpl]() if tpl of JST
Good point. Switched to:

(cast(size_t*)&this)[0..PageSize/size_t.sizeof] = 0;

Which should hopefully be optimal on any given platform.
Use fixed gem numbers, so this line should be:

```ruby
gem 'tariff_importer', '0.0.5'
```
via the mailinglist IIRC :-)
wonder if it should be OF_SCALEMODE_FIT, OF_SCALEMODE_FILL etc ? 
as with alignment it could be used for other parts of OF ?
Just out of interest, why is table = "ScreenAnnotationLink" here? Also, why does this block duplicate the above block exactly?
Isn't this doing the opposite of what the comment is suggesting?
Currently there is a space between the slash and state, but not between priority and the slash. Add one.
Style: We never really use for loops, but prefer each style loops. e.g. (2..nb_files).each do |i|; <do something with i>;done
apparently this isn't as fast as the CSS3 selector, but the selector style doesn't support what I'm trying to do. The performance difference will likely be negligible since it's not working on more than 10,000 elements.
Just corrected spelling on "separated"
This should be a case statement and should use ::osfamily instead
I am confused about what to use for flow_flag. which functions I need, which one I should rely to from flow_table that's why I wait to see what to do in flow flag modules before including anything in the main
I think this is an API change.  Not that I disagree with it, just that we need to have thought about the tradeoffs.
Oh, forgot to add -- I'll add it.
Why not move the Product/Productversion loading outside the "with"? Less deeply nested code, and if they don't exist you can bail faster, there's no need to ever open the file in the first place.
Isn't `else if` more readable?
This opens/closes list items for the 'more widgets' section in the inserterbar.
These product assertions are also made in the above test, don't need to be replicated here.
one other minor thing I try to stay consistent on: always use double-quotes for quoting strings, not single quotes. It's not a big deal, but I think it's odd when code goes back and forth for no particular reason. Consistently using double quotes avoids problems with apostrophes in a text string. The only exception is if the string itself should contain a double quote, then I single-quote the string.
The type is defined in the field above. I don't think it is any clearer, simply an old habit.

In kernel's source code, this is done using define. The type is specified nowhere, the important point is that it is a shortcut to a deeper field in the struct.
I total agree, but I can't find a CMake command that will ensure that the ign_logging.sh script will be copied into the build directory without using the add_dependencies command 'the wrong way round'.
Conserning the objections: 

  1. The exception you get with an oracle database is 

Caused by: java.sql.SQLSyntaxErrorException: ORA-00936: Ausdruck fehlt
I already fixed that an hour ago, but I am glad you noticed ;)
Right, but the problem is that the current approach ties us to two types only where the enum approach will expose possibility of adding other structures. 
That is entirely up to you, all I am saying is that its a very similar concern I faced with Redis outbound adapters where I have 5 collection types but aside from different types to return the process of building them is identical to yours (IF/ELSE)
Either way; but I disagree about new JIRA - this is a new bug; the previous implementation doesn't have a group table so this problem doesn't exist there.
Are you joking? ;-)
What is the reason to have here ternary operator?
How about createForPool (pool is asking callback to create) Vs. removedFromPool (pool is telling callback it's no longer in the pool and should have its resources released).
LIFO can happen only when we extend existed MS or overwrite `JdbcMessageStore#getQuery()`.
However I understand your opinion: we can add here some indicator (CountDownLatch) which won't allow to handle messages until `reschedulePersistedMessagesOnStartup()` schedules all existed messages in the MS on startup moment.
WDYT?

And why I use here `pollMessageFromGroup()` and not just iterate over messages in the `MessageGroup`? I want to avoid messages' duplications in the cluster.

I haven't once said that we need to add check on result of `removeMessageFromGroup()` if it realy removed message during current transaction.
At least make it inside ``pollMessageFromGroup()`
Similar comment as for TaskExecutor. You probably want all instances of TaskScheduler, right?
Seems like it would be smarter to keep track of whether or not the item is checkable and always defer to it. This was if it changes by some other means you are not misrepresenting it.
@lsmith77  @dbu Since it's a http_basic authentication, the logout doesn't work (the browser takes care of the login). Do you want me to try a JS hack to be able to logout anyway?
what about fragments?
where is the locale_Add_Extra_Columns button? did you mean to remove it?
typo in mobile
We should extract this to some other place to make it configurable in the future to preview end points etc.

Example from other places:
    var managementEndpoint = url.parse(utils.getManagementEndpointUrl(account.managementEndpointUrl()));
+1. I meant to apply the terms as outlined in your mail but obviously mixed it up here and there :)
це можна зробити програмно в скрпиті. див. Галерею
Holy shit !!!

no maj at the word locale for initializing the variable...
Do we really want to talk about unsupported plugins in the OPC docs?  Or should we just admit that, at this point, knife-acl is supported?
Wow, pretty complex style rule, is there an easier way to do this?  Also, it looks like the first word has a typo (should be ".container").
Can you clarify, why is the addition of conn->close(true) necessary in addition to conn.clear(); ?
where the semantics of a IRemoteConnection are to commit on release.
If the conn.clear() for some reason didn't commit, it would imply the connection was still linked somewhere, i.e. it had been leaked..
Actually no, FileSystem.Combine gives me "lib\imported", while I need "lib/imported"
This line should not have been removed