labofsahil/github-event-dataset-2012 huggingface dataset:
I do not know if this applies here. In fact, it is not really a parameter passing but further processing to get: "Francisco Jordano (2)" or "Unknown (2)" in the phone in case your phone number where assigned to a contact with you name or with no name. Don't know if I make myself clear :) Or maybe I am missing somethine here.
By the way do you really need this file?
Guidelines for if/else expression are to add brackets even if they are one-liner.
The label text it does, but the buttons are not invisible, they are the small "clear" crosses.
I will remove the hidden text!
Use the html5 'hidden' attribute
hi @vingtetun 

This code is in contacts, I know that it's not useful to have it in the shared folder, but do you fancy a way of still using the same code?

Any plans of add this as a web activity from contacts, or you prefer to keep it separate as an app?

Thanks, and good work btw :)
Since we don't have event listeners on the mozApp objects, we're trying to be good citizens with the callbacks, you can
`app.ondowloaderror = null;`

in `ai_downloadError` since you don't need it anymore.
I think that unless you tell me other wise, I would go for the previous approach (the one you propose but with the minor doc checking issue) since definitely all these variables would be reusable in the hide function ;-)
Did the type 'sdcard' not work for pictures?  Sorry about that, I thought it was supposed to work, but I had not tested.

Note that you probably want this to happen when there is a file type that is not an image/* or audio/*.  Maybe just set storageType to 'sdcard' when you define the variable above.  And also note that .ogg files get reported as video/ogg by device storage. I'm not sure how bluetooth will report them.
Same from above, what do you think about building a list of fqdns names and returning them instead? Also, I'd mostly avoid print and send anything we want to LOG.*
Should this.id get initialized (even if it is null)?
What a beautiful language Matz gave us. Nicely done @alindeman. /cc @myronmarston
From what I can tell `#magenta` wasn't really being used for any type of message (it isn't referenced elsewhere in the code that I can find): Do you think it would it be better to add one more generic configurable method or just put a message referencing the configurable methods in the second argument of the deprecation warning for `#magenta`?
I'm not a huge fan of `and` and `not` in conditionals as the precedence is really easy to mess up.

I think we can also use DeMorgan's laws to swap the `unless` to an `if` which reads a lot better to me with multiple conditions. 

Thoughts on this?

```ruby
if $!.nil? || $!.kind_of?(SystemExit)
```
最終的に、受け入れ可能な値を管理するパターンになるでしょう。

→

    最終的に、利用可能な値を管理し、その中から選択させるパターンになるでしょう。

オプションの値は予め選択可能な値をいくつか設定しておいて、その中から選ばせる、みたいな管理ができるみたい。なので原文より少し丁寧な訳になるけど意味は伝わりやすいのかなぁと
This is definitely a security problem - filter out bad characters from the app name before calling this.
ok.. what do you think about changing canLooseHealth in canTakeDamage, that should be more generic

also for the VanillaProtectionService I guess I will have to copy this one plus add the canHunger or?
I am assuming this a precondition check (the check happens before the block has actually been changed). Then there is no way to tell what the player is trying to build.

I suggest switching it to:

public abstract boolean canBuild (Player player, Block block, BlockMaterial toBuild);

That way plugins know what material is being placed.
The way we calculate visibleLines is really different from what's in upstream because of the inline editor stuff. However, the change here was pretty straightforward--the visibleTop is just the scrollTop if it's specified, otherwise we calculate it the same way as we used to.
I added a comment to the original report.
you should update the comment as the steps are now 1, 2, 4, 6 :)
regular expressions? sorry I'm not quite familiar with code writting
PERIST ? typo?
okay we can remove custom
This folder doesn't even exist.
Scalariform code formatting plugin is doing that.
Maybe make this have no default, and only use it if specified?
Nit: Unused variable.
BTW I talked to JS folks a while ago about `.bind` and they create regular closures behind the scenes and are no way faster or more optimazed than regular ones. Neither they had any plans to optimize that in a future.
The only reason I ask is that the change would decrease the amount of noise in the diff.  Many of these changes are happening around other changes, like at the top of `packages/api-utils/tests/test-content-worker.js`.
Yep, on Fennec nothing will happen, but it can be used in a add-on that would be expected to work on Firefox as well is all that I wanted to point out, so I agree that more information about the expected behavior on Fennec is a good idea.  I will do that.
*normally* the page templates are rendered on the server. For client side QUnit tests, main site templates are also rendered in the tests, causing a failure anywhere that "partial" is used. Got any ideas on how we can make this less crappy, or do you think once #2455 merges, we'd be able to take this statement out?
Hmm, this broke my code. I think it was a merge resolution error.

I think `certKey.args` is better as it reduces duplication. If we keep the code this way, I'll need `, 'forceIssuer': {type: 'hostname', required: false}` added back.
It would be clearer to place all of the object's properties on the obj when declaring it in line 19-21.
well, that's certainly a sentence cut off in mid thought :-)
Yeah, I had mentioned in the original pull comments that these management commands had gone fallow.  But I fixed them to use the new controller code and they work now.
nit: please add a blank between function and the opening bracket.
I violently agree!
Manual change also :)
nohup worked sanely in the previous pull against 2.7 (and was there to ensure that if someone has changed some environment defaults it would still work) but no longer seems to now. Same is true with the --no-daemonize flag. Not sure if puppetd had some different defaults, but when using /usr/bin/puppet we also need to add the `agent` face to the call in 3.x.

So this line should probably be `daemon $daemonopts $puppetd ${PUPPET_OPTS} ${PUPPET_EXTRA_OPTS}`, as long as ${PUPPET_OPTS} includes agent and --daemonize.

Then RETVAL=$? to capture the success.

Then I would use `pgrep -f '$puppetd ${PUPPET_OPTS} ${PUPPET_EXTRA_OPTS}'` to get the pid # and write it to the pidfile (as daemon doesn't seem to actually write the pidfile out).
在上面出异常的情况下，这里是执行不到。
Hi Bernard, why you have adding this declaration to point to UI classes? Wouldn't be better to generate rebel.xml inside -ui.jar?
so this is number of accesses before first miss? if so, can you rename accordingly?
CS, missing @throws in docblock
Can't do this in the LDAP exceptions, as the custom LDAP component exception class does not have previous exception support.
IMO really we shouldn't maintain the BC for this. Make the return of this function too weird.

boolean for some values that are too integers, PHP allows this behavior but is really weird the method signature
use "static", not "self".
What is the point of changing the position of this method? This is an unnecessary change.
``return false !== strpos($bundleClassContent, 'Symfony\Component\HttpKernel\Bundle\Bundle')``
Don't need braces around single lambda variables
That's fine, I think it's okay to leave it the way it is.
また「サルでもわかるGit入門」さんから引用しました
Don't animate
You don't need these helpers, just feed the template with the data instead
-> `var ENTER_KEY = 13;`
Again, instances are typically lowercased.
you probably need java-pkg-opt-2 eclass, see other ebuilds for examples
Typo. Should be iconaction_flash_torch_30x30.png
If you assume this value may change due to resising of the terminal, it makes no sense to store it in the environment.
possibles -> patterns
Hmm, is there a reason not to delete this line, the one before it and the one after it, after your changes?
rpath should point to the final destination, not the staging directory I think
As soon as the super is added you get:  TypeError: save() takes exactly 1 argument (2 given)
Without the super it works correctly
Why is this marked safe? Either way it should be `{{ personal_message|safe }}`
1 new line at the end of a file.
Ну мы используем 3 уровня: `===`, `---` и `~~~`
Was this intended to be changed?
how would that work without passing a proxy_id ?
I couldn't find a test case for this, that's why I just left it in there. It probably would work for search results - we can look at the link in the results.

The only problem is that I don't think search results work at all any more. From my poking around on their website, all the search queries are handled by openedition.org
This is just a minor modification to RepositoryChangeBus. Note we don't need the complex multi-pass loop as we thought we'd might. The change in RunningState.shutdown() is what keeps Infinispan from logging the InterruptedException.
I wasn't even aware that it was there and shouldn't have been removed. I must have based the code I added on a version that pre-dated that line. I am not that good at GIT, so not sure what I am doing most of the time.
I've just added one.
Maybe you should check the type here too :

```php
<?php
if ( $time === false )
    $time = microtime( true );
i'm afraid we need to build getSupportedQueryLanguages into Transport\QueryInterface. at the moment QueryManager::getSupportedQueryLanguages is not telling the thruth anyways, it has to return all languages we support. i guess we should redirect getSupportedQueryLanguages to transport and call it in createQuery to check if we should create that type of query.
No, I think `JINGO_MINIFY_USE_STATIC` should default to `True`, that the default should be to use the Django-1.4-style `staticfiles`, and if you want to keep doing it the old way, you should have to opt in.
I think you should not attempt to carry this file forward - commiting it was clearly a mistake, so feel free to just delete it
Sure thing, I have done so.
Yeah, this is fine
response.code.to_i was raising a NameError. Assumed dev wanted to use resp.code.to_i.
I dunno, I guess I thought it might provide a clue to the user that the value was configurable?  I can get rid of it if you like.
You are correct, and I need to change this. I belive this came in through my sloppy resolution of a merge conflict.
If the modules are preloaded at server startup, then this method can be a simple access to a hash. So there could be a method `initializeAvailableModules` that would be called from the initialization stage.
The tenant API doesn't seem to be used here.
I'm not sure this assumption is true, and you seem to handle that case as well (right-hand side of fields are expressions that are defined directly inside a class, same for constructor expressions).
one vote for DRY :)
Use !GTK_CHECK_VERSION(2, 12, 0) ?
Oops.  That check is backwards.  It should be if it is a UrlMatcher, not if it is not. :-)  I'll fix.
yeah, which is exactly why I said to remove all this commented section :)
Can we change '`\\'` to `DIRECTORY_SEPARATOR`? Get's rid of the backslash on non-windows installations.
Checking for 'options' but then setting 'selected'? It might be correct, I just want to make sure it's not an error.
Yes, as it's passed from the main driver.
uint. Меньше 0 не передастся
Новый генерируемый нуклеотид может совпасть со старым.
Ещё, как вариант, можно самому переопределить сеттер, и в нём при изменении значения уже увеличивать или уменьшать сам массив, это тоже будет корректно.
Would it still go into the controller if the sorting is done only for this page?
We don't really want to change the order unless it is the dashboard and the admin.
:reject_if =>  :all_blank
checklev is false in perform_analyze(), so any level checks should be ignored and the character buying the analysis should get full information.  Right?
Would it be possible to make one function out of renderFringe and renderBackground to a common function which gets the layer passed as a pointer? so in the render function you'd have
```
renderIsoLayer(background);
renderIsoLayer(fringe);
```
and the other layer (objects) would then need a function 
```
renderLayerMixedWithObjects(objects, renderables);
```
These functions would allow us to introduce even more layers later on very easily (not sure if we actually want that),
also it reduces code hence improves readability.
You're right, `id` is not really used. 
Although there are conversions between `String` and  `int`, your solution seams to improve execution time of handling `_DART_TEMPORARY_ATTACHED`.
I thought of a third solution : what about using the length of the value of  `_DART_TEMPORARY_ATTACHED` attribute to keep the number of elements temporarily attached at this point ? Doing this will use only `.concat('a')` for serialization and only `.substring(1)` for deserialization.
I made some tests and they confirm that this third solution is the most efficient.
ACME_Corporation is hardcoded. not good. 
I would rather rename the test case to: test_listProductDefaultOrg

and the default_org is accessible via: default_org (see superclass: com.redhat.qe.katello.base.KatelloTestScript)
Do we want functions to look like this or not? I do. Let's make it uniform. Line 60 has a different style than line 61.
Should anything happen in an `else` block?
Similar comment here: no need for `S` on the LHS, just use `HNil` on the RHS.
Then we actually arrive at a point where find_sources and get_sources can
be more or less merged...
Technically you could already do it. Just a matter of wrapping.

2012/10/29 Alexandre Gramfort <notifications@github.com>

> In mne/artifacts/ica.py:
>
> > +        Returns
> > +        -------
> > +        fig : instance of pyplot.Figure
> > +        """
> > +
> > +        sources = self.get_sources_epochs(epochs)
> > +        fig = plot_ica_panel(sources[epoch_idx], start=start, stop=stop,
> > +                             n_components=n_components, source_idx=source_idx,
> > +                             ncol=ncol, nrow=nrow)
> > +        if show:
> > +            fig.show()
> > +
> > +        return fig
> > +
> > +    def find_sources_raw(self, raw, target, sources=None, score_func=None,
> > +                         start=None, stop=None, sort_func=stats.skew):
>
> hum. How about being able to pass to the plot an order array
> (order=np.argsort(scores)) to order the plot according to the similarity
> with eg. the ECG?
> it seems weird to sort just to score.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/165/files#r1966062>.
>
>
if reject is not None:
> Yes, I think we can / should have both versions. Also note that we've got several whitening options now... our prewhiteners, PCA(whiten=True) and ICA(whiten=True). We should allow the user to select tuple combinations of those (prewhitener and pca, pca and ica, prewhitener and ica). Wdyt?

you need a MEG specific whitener using noise cov or channel z-scoring then
there should be only 1 whitening step and the whitening must be done
(not an option).
It's however unclear to me if it's easier/faster to do it manually or
let FastICA do it.
That is to say in terms of API the user should not be exposed a whiten parameter
but just noise_cov
Please remove the '\n'. 
(H&E)
Do we really need all these permissions here?
Don't think this check needs to be here. If it's a SquareMatrix, we can assume that it's square; only the data-filling methods should have to check that.
Oh, sure, sorry :)
@gajdaw I think it is better to say 'Hey, they say you need to do it that way, but that can cause problems, this is much easier'.
You should remove the space before the colon.
Reject when string is empty right after the split?

    parsed = response.split(/^|$/).reject { |i| String(i).empty? }.map do |block|
      parse_block(block)
    end
I think I'd probably fix this by declaring `d3_rgb_names` as `Object.create(null)` rather than switching to `hasOwnProperty` here.
Why is the sleep necessary?  I feel like it's very useful to comment steps like this for future readers of the code.
Thanks!
Fixed [here](https://github.com/emanuele45/playpen/commit/1da27d618e71a3074826cf55bc3db4fce8cfda59)
The grammar does not have to be a string. A RubySpeech grammar object is fine.
We don't support `@interrupt_digits` in this case. Raise if they are set.
This workflow is super-clear, thanks!
I'd say screw qemu.conf. /var/lib/libvirt/qemu is practice, qemu.conf is theory.
You assume wrong, because I just ran through fake mode tests and realized this broke everything! Whoops.
The same check is in the parent class. Extract method?
why does this need to be synchronized ?
Ok, noted.
This is defined for all the caches in the cluster. Wouldn't it make sense to be able to define it on a per-cache basis? e.g. a numOwners=5 cache would only trigger rebalance if 3 members left (or if there's a segment that's only replicated on 2 members) and another cache with a different policy..
All I did is remove the unnecessary brackets ;) (get rid of a warning message)
a bit more complex than that: this  file was written from scratch and then renamed (old file dropped), so the import is all new. I'll try to keep old import order.
1) We only count paths once
2) We don't use another variable to store the count
3) We don't need to include paths in a specific order

I know the loop could be further optimized by changing to
```php
for ($i = count($this->modulesPath) - 1; $i++;) {
```
But that lessens readability
Sorry, my bad. Totally forgot about the asset pipeline in that moment. Of course it has to be there.
This is a good catch.  We should probably be escaping all of the rst out of the commit messages here -- git commit messages aren't strictly considered rst, and there's no telling what other non-rst syntax they might contain.  But this one-off fix is fine for now.
And the leading ```..``` ?
Typo: preferd/preferred
Remove this.  It is defined identically in webapp style.
missing closing </ p >
Thanks! I thunk I should take a look af the FL library.
Can we hook this up to a data attribute so we don't rely on the ID of the object?
Please remove that later
If the preview/live toggle is here to stay (Sig and me don't like it, hehe), can we please generalize its styles so it can be used elsewhere? Should probably go into `_forms.scss` then, with a bit broader description.
Ano, to je zadání. Co tím opravdu mysleli nevím. Mám to v TODO listu.
IMO, the test should be ``if (isset($handler['publisher']['id'])) {`` (and a space is missing btw)
oops, you're right, I'll fix that.
For consistency and performance reasons the way to check the method here should be: `params[:method].to_s.casecmp('GET') == 0`.  Also I think we should just check nil on body and avoid using unless.  So altogether we would have something like: `if !(params[:method].to_s.casecmp('GET') == 0) && !params[:body].nil?`.  I guess we could just use a single ! and wrap the whole thing, using || instead of &&, but I think that this version is probably more understandable.  Let me know if that doesn't make sense or won't work (kind of writing off the top of my head while referring to response stuff).
Query layout is not correct
Should the type for parameter ``$resource`` not be "null|Acl\Resource\ResourceInterface" instead of "Acl\Resource\ResourceInterface"?
Are you sure ``$this->connection->getResource()`` has the correct type as ``object<PDO>|null`` is not a sub-type of ``object<PDO>``?
Opps, I thought I caught all those - let me see what I can do about cleaning those up
All of the std::remove calls are compile errors on my linux distro:

error: cannot convert '__gnu_cxx::__normal_iterator<char*, std::basic_string<char, std::char_traits<char>, std::allocator<char> > >' to 'const char*' for argument '1' to 'int remove(const char*)
We should get someone from UX to comment on this (drop into #ux sometime, ping shorlander).
I'm not familiar with epel. Would it just be something like:

```ruby
if node['platform'] == 'centos'
  include_recipe 'yum::epel'
end

package 'tmux' do
  action :install
end

...
```
?
Would it be OK to consider feature maps as a possibility here as well? I guess in 3D we could use PointCloud2 maps, but it would be nice to be able to have 2D feature maps as well. Maybe feature maps could have independent profiles in both 2D and 3D.
Maybe a stupid question: are these tests run anywhere? I mean on Hudson?
You may want to create a followup JIRA to update this (and other undocumented methods) later.
Eventually, yes. For the moment I'm hesitant about doing too much to the test suite since it's going to get a big pass over.
Yes, I understand.
My current Gemfile specifies:

'thin',   '~> 1.3'
'yajl-ruby',  '~> 1.0'

Thanks for the support.

On Tue, Jun 5, 2012 at 11:01 PM, Jordan West <
reply@reply.github.com
> wrote:

> >  gem "thor", "~> 0.14.0"
> > -gem "thin", "~> 1.2.0"
> > +gem "thin", ">= 1.2.0"
>
> I prefer, when possible to avoid `>=` dependencies due to the risk of
> unintended upgrade. What versions were you hoping to use specifically and
> I'll try to make the dependency specification more specific?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/stackmob/stackmob-ruby/pull/13/files#r931992
>
This is a good approach. But why is there is still mongoengine.DictField() defined on user document?
Move this one level up. Before our global vars. So that we can override in global vars if we will want.
Why exactly 31%? Such test-and-find numbers are dangerous. Try to find underlying fixed value.
Group imports properly.
You don't have to use `-1` value there. It could be simply an empty string. :-) Because it is strange to have strings for other values, and then number for this value. Empty string is much easier to understand: nothing should be mocked.
So, try with dict. You will see what MongoEngine maintainer says. :-)
Maybe don't use TemplateView but some other class-based view?
`os` is not needed anymore?
`HOROSCOPE_DATE_TOO_OLD` -> `HOROSCOPE_OBSOLETE` if you are using `error_obsolete` bellow. Or change to `error_date_too_old` bellow.
The regular way to write this (in Python < 2.7, at least) is `self.assertTrue(errors is None, msg)`.
Given the [GOOS](http://www.amazon.com/gp/product/0321503627/ref=as_li_ss_tl?ie=UTF8&camp=1789&creative=390957&creativeASIN=0321503627&linkCode=as2&tag=ploeh-20) advice that Mocks are for Commands and Stubs are for Queries, are we testing the right thing here?

The Invoke method is a Query. Do we expect it to have side effects? If not, then why are we (only) testing which parameters were passed. Don't we care about the returned value?
Accounted for in the next pull
Output would be weird with this syntax. Use following syntax to avoid the `\n`.

```php
    $exception = 'A value for the identity was not provided prior to authentication with'
        . ' DoctrineObject authentication adapter.'; 
```
Yup just pushed the changes
Not introduced in this change, but this error message should read "specified with --kiji" not "--instance"
there is a type parameter ( would suggest adding foe type="number" and few others i dont remember on the top of my head.. test cases for completion )
You seem to be changing this loop faster than I can comment on it, so I will just rollback to my previous criticisms:

 - mixing `onResolve` and `onReject` in the same array is confusing, so please at the very least document this in the data structure – or even better, return to one of your previous versions that introduced an array or an object;
 - these two uses of `shift` in the same expression are very confusing – you could afford the two lines to introduce named local variables, which would largely improve readability.
s/elemnet/element
Was also wondering about a good value here, when you access over dataset the key is actually isResponse, so the value could be that. Would want it close to a boolean attribute as I can have it..
This string is not new, but I would use "Pretty-print" with a dash.
Can I get a `local` on this guy? :)
rqServerHostname?
My 2cents: i think it's nice to show-off the convenient new features, but it might be wise to insert a word of warning.
Even if with nobody:nogroup, `openvpn_group` will still not be an empty string. Do you need this conditional here?
I'm a bit confused about how to exclude direct.  Do I disable the profile, or set direct.excluded?
grep -P '\t'
On Jun 13, 2012, at 11:34 AM, Barnaby Rowe wrote:

>>             if (nargs<2 || nargs>3 || !allNumbers)
>>                 throw SBError("SBParse: Bad arguments for SBMoffat: " + args.print());
>>             double flux=1.;
>>             double beta = dargs[0];
>> -            double truncationFWHM = dargs[1];
>> -            double re = (nargs>2) ? dargs[2] : 1.;
>> -            dbg << "**Returning moffat with beta, truncation, flux, re " << beta
>> -                << " " << truncationFWHM << " " << flux << " " << re << std::endl;
>> -            return SBMoffat(beta, truncationFWHM, flux, re);
>> +            double trunc = dargs[1];
>> +            double fwhm = (nargs>2) ? dargs[2] : 1.;
>> +	    SBMoffat::RadiusType rType = SBMoffat::FWHM;
> 
> P.S. Sorry, dumb question but how would you use grep to check for the wrong amount of whitespace?  That would be really useful!  grep \t * seems to dump a LOT of output...
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/GalSim-developers/GalSim/pull/189/files#r976969


-------------------------------

Rachel Mandelbaum
http://www.astro.princeton.edu/~rmandelb
rmandelb@astro.princeton.edu
On 21/09/2012 16:19, Thomas Gazagnaire wrote:
> how does it know in which path it has to install the package ?

Through `ocamlfind printconf destdir` (see the Makefile). I find that 
quite clever :)
if starting with a mgmt context it isn't clear whether we want this to completely override properties from the mgmt context.
(do we even need this?)
copied from Monterey
effectivement c'est mieux
console.log??
Spinning? ..maybe i'm confused?
Or a GET form rather than an anchor.
Sorry, my mistake. I'll remove these lines.
Thanks for the suggestions!  I've re-worked it a bit in my new commits.
It's better that title underline match the size of the title, Sphinx yields warnings when it's not. Also, if you can fix all other titles in this document, I'll gladly accept it. :)

In any case, the best thing to do is to compile the doc yourself and look at the output. 

<pre>
cd docs/
make html
</pre>
`forEach` can take a second parameter, so no "that" magic necessary.
I guess `functionNames` explains the purpose of the object already, no need for `functionNamesArray`.
Lets use the same implementation for both reg season and round robin.
Because it is?

It isn't a Rest thing per se but a REST on AG thing.  

Also because when I asked in chat that was the answer I got.
т.е. просто else для всего, что осталось и делаем toString?
Непонятно, зачем threads передаётся как аргумент?
You do need the `.http_proxy?` check, because we also support SOCKS5 and this authentication scheme will not work. 

Having said that, it looks like client_options needs some cleanup: drop @proxy, and move `http_proxy?` into connection_options.rb.

(or similar gymnastics)
Does this only exist for testing? ugh. Is there no way to just make a mock FlushTimer but have it called the same thing?
i'm rather confused. while FirePath finds this locator, it isn't visible on the page (FirePath isn't highlighting it, and i don't see "More..." on the page)
This was an attempt to implement an element_not_present with an implicit wait, just as element_is_present has an implicit wait. I need to check that an element _becomes_ not present.  I can see, though, how naming it thusly will be confusing so I will rename it as suggested.
left up to

は

   あなたに任されています

とか
oscillation even...
Is this faster than itertools.product? or what was the reason to implement this?
I should comment this. Per our undocumented api, transformers ("the" other class of unsupervised estimators) have to accept an optional ``y`` in their ``fit`` method. Therefore the ``else`` path goes through. If you look at line 237, Est can be a regressor, classifier, clustering method or transformer.
Maybe do a for-loop and a with-statement?
Should be
  
    return _dist_to_weight(dist)

The looping is done within the function
Yes, that's a defensive copy.
I think we could say that ``fantasy particles (particles in MCMC chain).``
Apart from that, I think reading the docs would be the way to go ;)
Yeah, I think it was a rebasing problem. I was able to solve that easily with the other "stale" commits though. Just not this one.

Mickey




On Sunday, February 19, 2012 at 11:32 , jmburges wrote:

> > @@ -0,0 +1,59 @@
> > +class BugsController < ApplicationController
> > 
> 
> 
> So I didn't re add it on purpose. Maybe it's cause you removed it while I was working on this and it re put it back in?
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/mickeyreiss/abtt/pull/19/files#r465434
> 
>
As discussed, in the case it makes more sense to crash.
AFAIK, the time limit is the same as in signing (it has only been shortened for testing purposes), so Signature::TIME_LIMIT_IN_MINUTES should be used instead.
Então, fique a vonts
Should be `Zend\ServiceManager\ServiceLocatorInterface`
What is the idea here. Two threads try to call _refreshAndGet_. The first acquires the _lockOnReplaceCurrent_ lock and does it stuff. The other thread need to wait for the lock. Once it receives the lock it first determines whether the  _preAcquireVersionId_ which he read before going into _WAIT_ has changed. If  not the current reader will be returned.

What you want to "save" is the _IndexReader.reopen_ call, right?

Is there a lot of contention around this lock? Is the _IndexReader.reopen_ really so expensive? Either way I would love to see the locking strategy a bi better documented.
2- rectangular shapes on screen transfer to weird shapes in spherical model and in more over weird shapes in internal planar projection : the way go is :

- calculate was is the max distance displayable on our rectangle on screen 
- ask for all points in that distance with center,radius search
- render all matching points  (yes some will may be outside visible area)
We really need a way to test in a more unit based approach. Weak test to just say the factory build, hence the issue is fixed.
The above variables seem to be unused
Also, should this function in a common place, do other activities do the same?
Have you tested this case?
hmm i dont know this all looks pretty hacky.. I dont want a large maintenance burden in here
Yes, something like `/lib/main.js` and `/data/content-script.js`, ...
I probably wasn't completely clear in my earlier comment... what I was getting at was instead getting a read lock first (versus write) and then upgrading to a write lock if the second null check returns true.  The first thread to come through will very quickly acquire the read lock and then the write lock, thus blocking any other threads from even acquiring the read lock.  Once the first thread releases its write lock, these late arriving threads will all (somewhat) concurrently be given a read lock and the second null check will fail.  If the threads each had to acquire a write lock, the can only proceed one at a time through the gate.  But again, this would happen very (very) quickly and may be perfectly adequate for this situation -- and simpler.
It's definitely non-intuitive that newDatabaseFactory takes an optional DatabaseFactory... if we're sure this is necessary, could we at least comment on what is going on?
If this is a private test, then it shouldn't be checked in.  If it's not a private test, then it should be named something other than troy-test.js, and it should have a few comments at the top as to how to run it.
I could chirp in, but I am not sure how much value I could add to this discussion.  IsAssocArray() is such a simple and trivial concept, I'm unsure why the implementation needs its own class.  This particular implementation (in the suggested pull request) is as expensive in C as it is as if you were to write it in PHP.  When comparing arrays, PHP (unless its the same exact zval by reference), will iterate both arrays and test each value. (see: http://php.net/manual/en/language.operators.comparison.php#example-116)

In general, I'd try to avoid crafting any kind of API that distinguishes between int-indexed arrays and associative arrays and switching behavior based on that.  To me, it seems more clear to just have two separate API's for introducing int-indexed array's and assoc-arrays to the party, especially since you're placing such significant contextual meaning to the difference between the two.

More to the point though, the assumption you're making is that any array that matches a copy of that array with the keys stripped, which the consumer would also have to know and agree this assumption.  In this particular implementation (before the suggested change), it was also expensive, but that is the cost of ensuring every single key is not a string - I am assuming that fact was important and widely used pattern such that it warranted inclusion in StdLib.

@EvanDotPro The only place I do anything of the sort (that I can remember) is here
https://github.com/zendframework/zf2/blob/master/library/Zend/Di/Di.php#L227
and that is only b/c I care about about the type of a particular entry of an array, not the "type" of the array as a whole.

There's not going to be a fast implementation of this concept.  To use the current implementation, you'd have to either be working with small arrays, or be prepared to place a large amount of importance on the perceived "type of the array" over the performance of figuring that fact out.

Hope that helps ;)

-ralph
NM, I see your bullet point in the PR about this.
You should first check for `oauth` driver, then lookup for that class.
I'm rather bother by this use of CamelCase - we prefer the "_" seperator to split words in CFEngine, as you can tell by the other variables/classes around. Could you use "apache_modules_to_update" instead please?
Right, I used View File. But now pasting that output into my editor, it looks like four spaces.
There was something in Canvas doc about how posts need to be application/form-urlencoded. Do posts still work properly without this header?
Cool, no reason to persist the inconsistence though. It isn't my call in the end.
Can MSF_LICENSE be used?
``$this->assertCount(9, $paginator->getPaginator())`` could do the job without the need to do the foreach loop in each test (ArrayIterator implements Countable)
I think the word "authentication" is wrong - we don't authenticate the users at all, we trust them to be who they say they are!

How about we just call it "simple log in".
Throughout:  why not move the shader creation code to the `if (pass.color)` and `if (pass.pick)` to make this function flow better?
Since `_context`, `_strict`, and `_template` are part of the material, I would assign them to `undefined` as part of the constructor function like we do for `_uniforms` so looking at the constructor function shows all members.  This is arguable I suppose since this function is called from the constructor function, but I think my suggested organization is cleaner and more consistent with the rest of Cesium.
Yeah, the only reason I didn't do it was because it's a breaking change and I was being lazy.  I'll switch them.
Longer term, I expect WebGL to get 16-bit half-floats, which are great for texture coordinates in the range `[0, 1]`.  I suppose we'll get typed array support too, and be able to use half the amount of memory.
エントリの例はrailsだからかもしれません。
最初から vendor ディレクトリがあって、そこに外部ライブラリを入れるお作法なので。 < rails
Deprecated. Use `button.titleLabel.lineBreakMode = UILineBreakModeWordWrap;`
Not removed from the children array
here you have an extra space before "is disabled", but below in the other case statements, there are no extra spaces, apparently causing the string to run together? this probably need to be fixed in each case statement below..
There are actually a couple ways to make this better:

* `#ifdef USE_TI_UIWEBVIEW` - Prevent the code from being compiled in unless we have webviews
* Only send the message if `[sView isKindOfClass:[TiUIWebView class]]`.

This will keep things performant for non-webviews.
Can you get rid of this commented out code?
Needs a html5 doctype
...and if you start with 1, 1, 2, 3, 5 like a real fibonacci sequence, you should probably run 9 or 10 rounds here.
i mean this note for doctrine-dbal - this note on setting up jackrabbit applies to only jackalope-jackrabbit, not in general.

.. note::

     Make sure you also configure the main ``doctrine:`` section for your chosen RDBMS.
    See `Databases and Doctrine <http://symfony.com/doc/2.1/book/doctrine.html>`_.
Это монада! :D
nit: here and throughout, we've gotten a fair number of reports that `hint_orange` is hard to read on some lower-quality screens since it's so light, so I've been using it less and less (except for some graphical elements). I'd suggest changing it to `hint_pink` or something throughout.
nit: Should computing the actual division be a separate hint? I don't have a strong opinion one way or the other. Probably depends where this exercise is in relation to other percent exercises--which you probably know better than me.
Since the working set of LaggedFibonacciEngine is so large, it may be worthwhile just making it a class. That would simplify implementation considerably. Make methods final.
My intent was just to prevent possible bugs introduced by maintenance, but your test against element [$-1]  is good.
This comment's placement is awkward. Move?
`mostNegative` won't even compile with user-defined types in almost all cases, because it checks whether `T.min == 0` (hence the compilation error with user-defined types and `min`). Also, given its description, I think that it's pretty clear that `mostNegative` was never intended to work with user-defined types, which is why pull# 611 adds `if(isNumeric!T || isSomeChar!T)` as a template constraint on it.
That's a hell of a line. Does it make sense extracting a method?
bind could presumably be top level? it looks like a static method
Let's not use print in tests (after the debugging phase). If necessary, let's setup logging so that it can be disabled/enabled without changing code
no - this is simply handling a case that Kerry wanted - it is not an error or an exception for this to occur 

----- Original Message -----

From: "nrporter" <notifications@github.com> 
To: "plymouthstate/plymouth-webapp" <plymouth-webapp@noreply.github.com> 
Cc: "bscoleman-psu" <bscoleman@plymouth.edu> 
Sent: Wednesday, October 3, 2012 7:57:44 AM 
Subject: Re: [plymouth-webapp] changes for adding new BM Report App (#425) 


In legacy/hub/HUBSQL.class.php: > +      return $data;
> +    }
> +
> +	return $null;
> +	}
> +
> +/**
> + * add a Round Entry for Building Manager Report to the MySQL hub database
> + * 
> + */ 
> +	function addBMRRound($now, $data) 
> +	{
> +		// don't save the round if the time is blank - didn't fill anything in...
> +		// might be just doing a submit - didn't do a round
> +	  if ($data['r_htime'] == '' && $data['r_mtime'] == '' && $data['notes'] == '')
> +			return; 
Should we do an empty return, or at least return something? Maybe even Null? You might even consider throwing an exception. 
— 
Reply to this email directly or view it on GitHub .
Are you sure this is supposed to be for money?
```
$('.create-file').toggleClass( 'visible', selectedComponents.length );
```
Change commited
It's easy to mismatch meaning of Request::getRequestFormat() method - it's about 'in which format should be Response returned' not like 'in which format is request given'. Im not sure if @fabpot named that method properly...
Could you use the note markdown? Like

>Note
>
> You have to visit space needle this weekend
ditto for s/CONSUMERNAME/SYSTEMNAME
By reference isn't needed, I'll remove it.

I'll look around a bit more, I see they are optional, but I'm not sure they are ever not sent by PHP unless you are using PHP < 4.0.2.
Stimmt, du hast natürlich recht.
Ich habe nun Members auf Array umgestellt. Bei Discussions und Orders ist es ja das selbe, oder? Deswegen habe ich es dort auch gleich gemacht. Sorry für das Versäumnis.
Leider habe ich die Reihenfolge der Getter/Setter Funktionen umgestellt -> um es besser lesbar zu machen (habe überall den Datentyp ergänzt).
Dadurch ist es jetzt aber wahrscheinlich etwas schwer vergleichbar. Werde mich bemühen, solche Spässe in Zukunft zu unterlassen.

Lg

_________________
Lukas R. Burkhardt, B.A. HSG
Luzernstrasse 55
6102 Malters | Switzerland
+41 79 265 56 16


-----Ursprüngliche Nachricht-----
Von: Pirmin Mattmann [mailto:reply@reply.github.com] 
Gesendet: Mittwoch, 25. Juli 2012 13:55
An: Lukas Burkhardt
Betreff: Re: [skillbaz-proofreading] User / Legi / Login (#6)

> @@ -59,4 +139,123 @@ public function getLogin()
>  	{
>  		return $this->login;
>  	}
> +	
> +	
> +//	/**
> +//	 * @return Member

Das selbe hier

---
Reply to this email directly or view it on GitHub:
https://github.com/skillbaz/skillbaz-proofreading/pull/6/files#r1234121
`std`
И сделай тайпдеф, да. VLCWrapper_ptr, типа.
I believe this should be 'event_type'
What is this? Why float? No no no?
What if no route is available ? should the user redirected to the dashboard screen or the show view ?
is it possible to add travis tests for sonata PRs? this would make merging easier.
As you said, stringifies everything ... I don't believe pre-check if is an object is desirable ...
and btw, your logic is broken if you use a form with an empty root name (which is not the case in this snippet but is handy when the request is sent by Backbone as it does not wrap it in the name of the form type)
I'll take into account all comments and re-submit a pull request :)
Will restore the original API. I agree that it is better to remain as much backward compatible as possible, and consistent with the rest of the code as much as possible. That was just my rationale/explanation for why I did this. Thank you for reviewing this, it helps a lot :)
please extract constant
you should not use unbound constraints whenever possible as tags should not contain them (we cannot release something that requires knowing the future as Chuck Norris is not part of the team)
I havent seen any traces of alphabetical ordering. Adding a new node in this pr looks for the current max sort_order of the siblings and adds one to it, to ensure that new node appears at the end of the child list as you have said.

So the only order by I have added is by the new sort_order field.
There is only one class in this file, therefor generally we try to use the same classname and filename. In this case BuildingGraphics might be a good choice. 
In general i'm not quite sure if this name is the best choice,  I think something like PreviewHandler might be better suited, as Graphics is very general.
I don't know about that.  

The "regular subscription verification email" if you mean the EmailService.Send() method, that is what the api should be doing.

Let's not do anything with ESP emails and this flag only triggers the GCS email sending.
why creating a method just for this ?
this is the ``$configs`` array passed to the method. you ``$config`` array here will be the last element of this array due to the foreach loop
Removed the concept of plugin_dir in favor of the plugins concept we talked about. Added in a :desc
@Ayoola I see your point, please see my comment on line 84 of models.py. This suggests that the get_ordered_group() function on QuestionGroup be re-factored to return a list of question objects, not question_order objects as it does now. That way we don't expose our method for ordering the questions in a QuestionGroup, we just return the ordered list of questions. This will address your concern of having to do question.question in this block.
Should there be test cases with one of ipaddr or IPy missing, perhaps not worth it?
Hum, you could make the checksum as a policy, then.
That can/should be done inside `inject_defaults`.  It's probably worth adding a TODO for it.
Actually, these border styles ARE supported.
Where does the "NewTransLang" stuff come from? :)
Looks like you don't actually need grailsApplication here, but rather just a config flag.  Try this:

    boolean rejectIfNoRule
Now that we have the formatting to worry about, should I use this 
formatter - http://pastebin.com/xnBxYZZk or the ones used by ShrinkWrap 
(and JBoss AS7)? Also, should I run this on this file alone? Or format 
the entire GlassFish adapter project, or just the glassfish-common and 
glassfish-managed-3.1 modules for now?
On 12-03-2012 14:38, Aslak Knutsen wrote:
>> + * limitations under the License.
>> + */
>> +
>> +/**
>> + *
>> + * @author Z.Paulovics
>> + */
>> +package org.jboss.arquillian.container.glassfish.clientutils;
>> +
>> +import java.net.URI;
>> +
>> +public class NodeAddress {
>> +        /**
>> +         * HTTP protocol URI prefix
>> +         */
>> +        public static final String HTTP_PROTOCOL_PREFIX = "http://";
> formatting ?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/arquillian/arquillian-container-glassfish/pull/17/files#r544428
Should use localize.translate("EXCEPTION_FEATURE_NOT_FOUND", feature.id) here instead of hard-coding the message.
It should. Removing.
yes, because it makes sense for the sessions to be ordered in the same
way in which one finds them in excel
ajaxload.info