labofsahil/github-event-dataset-2012 huggingface dataset:
why is it called tShowPwd instead of showPassword?
Hmm... I like it! 

But. It's a bit awkward that we'd have to include `ProgrammingError` in that list when it's such an ad-hoc exception (no pun intended).
Also, "act_on_exception()" isn't a great name. Can you think of something better? 

I'll work on a revised version.
Moving this to the controller breaks the program.  Shouldn't the message be being saved within the model?
# is comment
Yeah, go ahead.
________________________________________
From: David Welch [reply@reply.github.com]
Sent: Thursday, July 19, 2012 10:52 AM
To: Matsui, Joy T
Subject: Re: [BRAINSStandAlone] Gtract eigenvalue fix (#95)

> @@ -75,7 +76,8 @@ gtractDiffusionTensor3D<TComponent>
>    this->Superclass::ComputeEigenValues( eigenValues );
>
>    // (lambda.2 + lambda.3)/2.0
> -  return ( vnl_math_abs(eigenValues[1]) + vnl_math_abs(eigenValues[2]) ) / 2.0;
> +  //return ( vnl_math_abs(eigenValues[1]) + vnl_math_abs(eigenValues[2]) ) / 2.0;

Can this comment be deleted?

---
Reply to this email directly or view it on GitHub:
https://github.com/BRAINSia/BRAINSStandAlone/pull/95/files#r1198107
*"\The [src]", please.
Yup, that did it. I'll apply your fix and pull request it. Thanks!
20 should probably be a #define.
The parentheses aren't needed here and below.
That's fine.

Should I change the constructor to not allow null arguments as well or just leave the if null check?
I am 100% certain I meant for it to be final.  Good catch
but I think it will be better to just use an `if` inside `forEach` so we're not iterating over the array twice.
occurred
Непонятно. Почему аргумент не может быть null?
if the request fail, the following assertion about the get access token call will fail.
I think this change was caused by tabs to spaces when saving the file. The change is to GetActionMethods method.
New namespace for user API, as discussed
no copyright notice here
Please use a single line when all fits in less than 80 chars.
Also grep for `filter_params` in the KernelPCA code. It's pretty useful.
missing `:` in ```:math`N<=20` ```
BTW, with your current solution, one would also have to inspect whether `y` is 1d or 2d...
you can put the too if in one
- Variable `$t` is never defined. It was in an older version.
  Because I improved tag handling, it is no longer necessary to check for the cases '' or '-' or '--', because they are simply never created on tags.
- Is this condition necessary? Isn't it easier to first check for no tags and then return false (as they are equal and not greater)
D'ac, on traitera plus tard.
Changed to use QStrings
I can't confirm production but it was breaking dev.
Since we have only one kind of observer at this point, why not just accept IConfigFileObserver objects in addObserver and get rid of the dynamic_cast. If you envision the need for different types of observers, then it's fine.
Yes, think so.
Not if we always created a huge heap for maxHeapSize()/2 and above.
Strange, I'll re-check.
Always catch Exception at least, never raw
Totally agree...I felt dirty creating that variant of BaseLinter's `find_file`.  I thing there could be value in moving `find_file_or_dir` into BaseLinter and then having `find_file` leverage it.
Already defined (L3)[https://github.com/caedes/sinatra-sprockets/blob/84bcebf6285ef98731f903c2ac9f3834ce82cb47/lib/sinatra/sprockets/helpers.rb#L3]
AFAIR we renamed <code>postData</code> to <code>data</code> recently, so it'll be good to change here as well (<code>postData</code> will be used only for backward compat). Or, alternatively we could accept <code>data</code> and <code>postData</code> as synonymous (I prefer this option honestly).
Moar docs?
This is some messy work, I really don't know if I was half way through finishing this.
@shacker making a mental note to test this out.
I'm not really against only having the one option. The only reason I
allowed someone to pass an argument was that they might want to ensure
that the association is _not_ autosaved. If you'd prefer only to have
the one case, I'd be glad to go back and change that to match your
suggestion.

As for the symbols, I am not really sure why I didn't just pass the 
literals, either. I'll go back through and change those.
Yeah - I've been around long enough to know why XHTML failed, I just wish it had been XHTML from day one. I would have made life much easier!

Also, I am an XSLT fan - so have other motives ;-)
This could probably be written as:

    catch: ^expected_type, ^expected_value
      expected_value
    catch: ^expected_type, actual_value
      flunk "Expected #{expected_type} #{inspect expected_value}, got #{inspect actual_value}"
    end

If we get anything besides ^expected_type (let's suppose we are waiting for a throw and an exit occurs), we should just let the exit pass through.
Can we add a TODO here so that once we have some way to return async responses to users we can go back and do it.
The imagery_layers branch currently renders front-to-back.  I was going to switch it back before merging, but if this goes into master first I won't need to.
Is returning `undefined` better than throwing here?
Does exist a RouteDescriptor without a path specified?
OK. I don't think it's supported on any other platform, but it showed up in the coverage report.
type dictionary
так не получится `new []("Беднева Дарья", "Петрова Марина", "Беляева Анастасия")` можно просто ["Беднева Дарья", "Петрова Марина", "Беляева Анастасия"]
switched this up to open multiple files
Did you forget to delete this file when you renamed it to terminal.dart?
This treats all paths as relative to the current directory. Hence, I think "/foo" is treated as "foo", even if you're within a subdirectory.
This is indented oddly.
i will put all the classes in the src folder. it is becoming slightly cumbersome to manage
You read the full file on reach change? That creates a lot of overhead, doesn't it?
I preferred the old "In the pending state, a promise" to "When in pending: / a promise".
issues: http://projects.puppetlabs.com/issues/14393 http://projects.puppetlabs.com/issues/14394 http://projects.puppetlabs.com/issues/14395 http://projects.puppetlabs.com/issues/14396
А нет, судя по докам, получишь те, которые будут с этим же нейспейсом. Те, которые вообще без нейспейсов. **Но не те, которые с другим неймспейсом.**

Какая идиотская система.
Markdown file name detection logic was simplified from previous multiple if-else expressions in above changes
This should be in the page object.
I'm following the example of completeSort, partialSort, etc., which all use 'less' for specifying the sorting order.
`@safe pure nothrow`(?)
This indentation looks odd.
No, this is core part of the bug 8705.
If `T` is `const(int[int])`, `Unqual!T` returns `const(int)[int]`. This is correct `Unqual` behavior, but has no meaning here.
Instead, `Unqual!V2[K2]` constructs `int[int]`, it has a *mutable* values, and as far as written in comment, it allows assign in whole building the result.
By the way, `typeid(T)` does not allocate anything when `T` is a type known at compile time. It literally compiles down to the address.
What the fuck is this variable ?
I think this line is too long.
is there no checkstyle error for this line? (because of wrong indention)
If not, it returns an Array
Format this one too.
Yeah, FreeBSD does work actually:

<pre>
[root@hal:/tmp]# su ben -c "git clone git://git.wincent.com/command-t.git"
Cloning into 'command-t'...
remote: Counting objects: 2640, done.
remote: Compressing objects: 100% (964/964), done.
remote: Total 2640 (delta 1445), reused 2613 (delta 1433)
Receiving objects: 100% (2640/2640), 489.04 KiB | 436 KiB/s, done.
Resolving deltas: 100% (1445/1445), done.
</pre>

And Solaris too. I stand thoroughly corrected (and surprised...). Grand!
@carlosantoniodasilva We have a local implementation of `hide! ` https://github.com/plataformatec/devise/blob/master/lib/generators/devise/views_generator.rb#L17
This is wrong. If there is several ignored classes, you will do the interface check several times.. You should add a second foreach loop over the ignored interfaces and use ``implementInterface`` in it.
could you please indent this code?
(but see above note about QByteArray use making all of this a lot easier)
Same comment here, host['group'] || "puppet"
вот поэтому и не хочу. Потому что это ад и содомия. Хранить начальный индекс, где мышь пресс, потом кучу проверок делать. Ну нахуй. Кому надо - сделает хук на перемещение и все. Или вообще к сигналу на перемещение прицепиться.
<< это push_back а не push_front
I changed the button to a hyperlink which should validate
While we're fixing this up, change this to String also
No "_mpl_axes" code here?  Believe it or not, but many users use "fig.gca()" as shorthand for "fig.add_axes()" or "fig.add_subplot(111)".
I think that delete_document/1 doesn't appropriate name here. It's about only mark document as deleted.
How about better name as mark_as_deleted/1?
Need to change these to V too
No, they are not redundant.
They can actually be different if group already exists. Look at my last commit, i made a few minor changes, it should be more clear. 
````
final Timestamp updatedDate = new Timestamp(System.currentTimeMillis());
		
final Timestamp createdDate = groupNotExist ? 
	updatedDate : 
	jdbcTemplate.queryForObject(getQuery(GET_GROUP_CREATED_DATE), new Object[] { groupKey, region}, Timestamp.class);
````
The bottom line, when commenting on a piece of code be more explicit about what you see, like or don't since it saves time for all of us. Sure after I stare at it i can probably see something and in most cases I did, but as I said we can be more efficient about our time.
You can use:
```
uri = this.uriExpression.getValue(uriContext, String.class);
```
`'required' => true` is probably wrong here, as this method doesn't need to be called every time... It is already required in the constructor.
Nope, it can be like 3 different things :) http://octopress.org/docs/blogging/
i will fix this in another branch.

[Opened issue in milestone 1.1.1](https://github.com/Publero/PubleroAppleMobileBundle/issues/7)
I think we will just expose the entire button and let people localize how they please.
Thanks, @stof.
Agree. Fixed
FreeBSD seems to use "acpiconf -i 0". Do you have an example of the output?
Point uses LineStyle and IconStyle. When Cesium can support polygon outlines, polygons need to use LineStyle. when we support extruded LineStrings, it uses PolyStyle.
remove color definitions here, apply link class to anchors in code
Great code review (as you can see I'm still pretty new to scala.. didn't thought about simple empty object *sight*).

My only question is here... Are you sayin' that register the custom class loader coming with Play is not needed anymore?
Because my guess is that it's still required (https://play.lighthouseapp.com/projects/82401/tickets/216-play-scala-20-recompilation-problem-using-salat)

Thanks again for this update
I let you change that in this PR (if this is needed?)
I don't think this is the best place for this. If it needs to be in the form array, it could be in it's own form element with markup.
http://api.drupal.org/api/drupal/developer!topics!forms_api_reference.html/6#markup
please do not use exceptions
I think that is the guest already has an account is a user of the system, here I woul put something like:

As a user
...here? For the other classes also.
`role="link"` is very redundant for an `<a>` element.
shouldn't it just be

    if (!state && $(this).find('input').is(':checked'))
      $(this).toggleButtons('toggleState');

only toggle if it's not the right state O_o ?
копипаст детектед
I'm guessing this should be `mozAlarms`, not `mpozAlarms`.
Bien sûr!
тут step нужно сразу уменьшить вдвое, так как он гарантированно указывает наружу (assert)
We typically just ask: "What does the following code print?". Therefore, "upon execution" is a bit unnecessary, in my opinion. I would also change word program into code, for the same reason.
(minor) I am not completely clear on what this multiplying by 10000 and then dividing by 10000 is for.

→ Please extract the literal '10000' into a variable with a descriptive name.
compress may help here too
Maybe a quick comment here to specify which condition is splash and which is icon.
I understand what the problem is. I tried on ready state changed and it was not being fired by the dom. That is why I went with this route.
As above. Needs to tell me the name of the controller.
I'd guess it is, but I can't find the class anywhere in the mark-up, and when I change the class to either fl or fls, I don't see a difference. Should we just delete it? And I forgot to add :focus styles. Can I still commit to my branch with a pull request out?

Johnny







On 2012-02-24, at 9:33 AM, Justin Obara wrote:

>> }
>> -.single .entry-content strong {
>> -    font-weight: 700;
>> +.fl-toggleButton{
>> +	background-color:#09c;
>> +	background-image:url("images/icon-uio.png");
>> +	background-position:center center;
>> +	background-repeat:no-repeat;
>> +}
>> +.fls-theme  .fl-uiOptions-fatPanel .fl-toggleButton:hover,
>> +.fl-toggleButton:hover{
>> +	background-color: #cc0;
>> +}
>> +.fs-uiOptions-panel {
> 
> Is there a type-o in the namespace for this class. I'd assume it should have been either fl- or fls-
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/fluid-project/studios.fluidproject.org/pull/45/files#r485127
shouldnt the services include equiplet id? becouse it is equipletNode specific