labofsahil/github-event-dataset-2012 huggingface dataset:
wz passed here to this function is written properly. It appears almost identical to the raw data, but has been aligned, and is in units mm. wx and wy do not appear correct. This is not just an issue with the write_xyz_bin function, because the same issue is with the ASCII x,y,z file created (below). wx and wy appear correct (the range is approx -2500 to 2500), but when plotting x and y they don't appear in the correct place... They are not paired correctly, or with where the Z data should be.

My test:
Display wz as a 2d image
Plot(wx,wy) with color as wz
The two should appear almost the same, although the first is a pixel image and the second is a point scatter plot.
Love the yardoc.
Sozusagen das Gegenteil eines RFCs? ;)
From examining the [jQuery source](https://github.com/jquery/jquery/blob/861a45b8fffc2073baa9f62b0648f395f0695f3e/src/core.js#L491-L497), it would appear that it is not necessary.
This was always returning `true` with an ajax request. I think it was reading the limit number as a string, where $var[1] would be the second character in that string. Validating the param as an array fixed the issue, though there may be a better solution I didn't think of.
It seems that this code could be simplified by doing the init view first and then over riding the defaultViewText if it has been provided using the old API.
Why not an && here?
gah, that must have snuck in when i was rebasing

sorry about that, will fix pull request shortly

On Wed, Aug 22, 2012 at 10:23 PM, Joshua Timberman <notifications@github.com
> wrote:

> In README.md:
>
> >
> >  License and Author
> >  ==================
> >
> > -Author:: Seth Chisamore (<schisamo@opscode.com>)
> > +Author:: Scott Chisamore (<schisamo@opscode.com>)
>
> His name really is Seth.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/opscode-cookbooks/java/pull/11/files#r1439240>.
>
>
I took all refs out.
You cannot have `<ul>` inside of `<span>`. Please check XHTML compliance.
Don't like the way you wrap here.
We go into newline after `}`.
No need to load `i18n`.
`-j#{ENV.make_jobs}`
Can you add the prefix strings to `strings.py` so that they can be used by localizations?
For future reference, nobody likes patches with whitespace changes. Don't worry about it though, I'll remove them in the cherry-pick.
Fix your indenting.
Could you un-indent this by 4 characters please. It's already indented for the exception.
if ((!factionEntry->BaseRepClassMask[i] || factionEntry->BaseRepClassMask[i] & classMask) && (!factionEntry->BaseRepRaceMask[i] || factionEntry->BaseRepRaceMask[i] & raceMask))
Will fix if AuthenticationService is staying.
I am propose take assumption that only data from openid `($identifier, $mail = null, $name = null)` is enough for user creation. For more complex cases users of bundle should use other approach.

How we handling 2 cases from above:

> if user with openid-identifier presend - log in
> if not - throw UsernameNotFoundException
>
> there is failture handler connected to auth provider. In this handler we store all data about user from openid to session and redirect to registration page, where some fields are prepopulated. After successfull registration we connect identifier with user and remove openid-related data from session.

Your bundle shouldn't provide ready for all cases sollutions. Users can add own success/failture/etc handlers and change beharvior.
I just came to think of something, the way `Gemino` is not is very similar to `Accio`
Mayby `gemino` should use `ARGV[1]` instead of `.`

Thoughts?
Reading this sentence, I have no clue which promise you mean.
All the deployment options has to allocate the deployment, because of that GetDeployment is used for all
It now defaults to "to bottom", so I left it out.
add_listener
NgxUrlAsyncFetcher::Init() should be called after new a object
What’s this for? It won’t go away before the end of this method will it?
Same comment about " formatting.
Forgot the 

```
        return -L
```
I guess better would be: stacked so that each column (row?) is one endogenous variable
No problem, take your time! :)
всмысле удолил да
I remove the parameter entirely in e501554. Basically if you enter "closetab" into the console, the event associated with the CLI entry is passed as the parameter. Thus any command-based method needs to either

1. Not rely on the argument, or
2. Determine where the method call originated and handle the argument appropriately

In this case the `closetab` method does not need the overhead of checking the argument -- it did not need the argument at all.
Style question/comment: I see you add explanatory text to every assert.  I generally don't.  Following the clean code 'eliminate comments' mantra, is that text noise?
Why don't you use UTF-8?
BTW, meta-charset tag should be placed before <title> tag, so that browser can parse title correctly.
I strongly agree that it's better to have custom copy methods in derived classes.
Looks like cfg.account.homePage shouldn't be checked here at all. It will never evaluate to true and then go on to have a type of undefined.

The overall point of this change seems to be to prevent error if cfg.account is null, which is accomplished by line 173.
amend: i see the above comment here about exam.xml_metadata may not be quite relevant, since you're using `question_times` in the video view.
This is basically a No-Op b/c the ProblemActivity model does not have a "problem" field.
The intent with `stable` is that it prevents scaling if we're fluctuating between growing and shrinking (or staying the same). If we're at 100 at the number required shifts from 10 to 11 repeatedly, then it should say that we have a stable request for 11.

If we're at 100 at the number keeps shifting from 100 to 10 and back again, then it will report stable=false.

That's the intention at least!
Could extract pattern as a constant, rather than re-compiling on each call, but not important...
i was using status==null to indicate success but i've tidied that up
This is in IdGenerator, so can be deleted from here.
Is there a library we can use for globs instead? Seems a shame to have to write/support/maintain this ourselves, particularly as it will be less documented than a standard glob library (if there is one).
`typeof` definitely beats out all other methods cross-browser for "is a function" checking:

http://jsperf.com/toarray-function-check

This might be an optimization to take further throughout underscore.js where _.isFunction may be getting over used as a wrapper to toString.call.

I'd be glad to do the legwork on that and report back with performance testing results.
Не заменить ли все урлы в опене на jdbc:h2.mem:.?
On Wed, 31 Oct 2012, Hannes Magnusson wrote:

> > +		safe_op = !default_fire_and_forget;
> > +	}
> > +	/* Then we check the options array that could overwrite the default */
> > +	if (options && Z_TYPE_P(options) == IS_ARRAY) {
> > +		/* Check for "safe" in options array */
> > +		if (zend_hash_find(HASH_P(options), "safe", strlen("safe")+1, (void**)&safe_pp) == SUCCESS) {
> > +			/* Check for bool/int value >= 1 */
> > +			if ((Z_TYPE_PP(safe_pp) == IS_LONG || Z_TYPE_PP(safe_pp) == IS_BOOL)) {
> > +				safe_op = (Z_LVAL_PP(safe_pp) >= 1);
> > +				goto ok;
> > +			}
> > +			/* Check for string value ("majority", or a tag) */
> > +			if (Z_TYPE_PP(safe_pp) == IS_STRING) {
> > +				safe_op = 1;
> > +				goto ok;
> > +			}
> 
> Any reason for not supporting specifying safe and fsync? (the fsync 
> check is done 2lines below but is skipped when safe_pp is string)

You misread, this is just a check whether it *is* a safe op. As soon as 
"safe" is set, it's a safe op so we don't need to check fsync anymore. 
We add the fsync option later on in around line 319 in 
append_getlasterror.

-- 
{
  website: [ "http://mongodb.org", "http://derickrethans.nl" ],
  twitter: [ "@derickr", "@mongodb" ]
}
sort of.. here's another way:

```ruby
@authors = Refinery::User.joins(:posts).select('id, username, count(*) AS post_count').group(Refinery::User.arel_table[:id])
```

But this also couples to `Refinery::User`
Does it make sense to handle the sending of emails using the ``Dispatcher`` and the ``CommunicationEventType``s? It might be more consistent with the way most (or all?) of the other emails are handled.
indentation is different from rest of the file.
btw, I used to have those functions exported... Now that they have moved here i supposed you wanted them global.
Should we be removing this style of getters across the whole codebase now then?
is underscore intentional?
Readd the isbreathing check.
It was sharing 1% of the air difference each time.  Now it is sharing 10%
`snow_leopard_64?` is just a compat wrapper around `MacOS.prefer_64_bit?` these days...
Please name your params according to the naming convention (which is the same than for services): ``markdown.sundown.extensions`` would be it (except the first segment should better be ``knp_markdown`` but this impacts the whole bundle)
Rebase your branch off of master and you'll get devstack_branch and devstack_repo in this conf dictionary
Pour ces variables, il serait probablement intéressant de faire des paramètres de classe.
extraFix made

2012/10/10 Dmitriy Komanov <notifications@github.com>

> In src/ru/fizteh/fivt/students/fedyuninV/wordCounter/ResultContainer.java:
>
> > @@ -30,7 +28,7 @@ public void add(ResultContainer given) {
> >          }
> >          for (Map.Entry<String, Integer> entry: given.container.entrySet()) {
> >              if (container.get(entry.getKey()) != null) {
> > -                container.put(entry.getKey(), container.get(entry.getKey()) + entry.getValue());
> > +                container.put(entry.getKey(), entry.getValue() + entry.getValue());
>
> á ÔÕÔ ÎÅÔ ÂÁÇÁ?
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/dkomanov/fizteh-java-task/pull/25/files#r1800535>.
>
>



-- 
ó õ×ÁÖÅÎÉÅÍ
æÅÄÀÎÉÎ ÷ÁÌÅÒÉÊ
íæôé æé÷ô ÇÒ. 195
И здесь System.exit(1)
and don't forget the indentation
Agreed. It was there because of the `STATIC_URL` code but that shouldn't have to be in the CSS.
I copied that code practice from the HOG-implementation. I also prefer imports at the top.
I think `sklearn/utils/testing.py` is the right location for such helpers. If there is redundancy it should be factorized here rather than `test_common` I think.
Yep ok, let's go for new mixins. I'll put changes at ndawe#4.
l1_reg has to be positive you don't need np.abs
Maybe this could stay at `info` level.
I understand. Then you should close the PR.
I think ``null`` would be better. Api looking like ``string|boolean`` don't look really nice IMO
To be clean this should save and restore the initial value of sys.stdout and not blindly set it to stdout
We've got "object false" twice - I'm guessing that's accidental?
Shouldn't be in the PR. Also, I'd first see if @prolic can suggest something given his https://github.com/prolic/HumusDoctrineHydrator
your new files are missing the copyright header
> What are you saying about the version numbers, are these not released using the same numbers as the rest of Lucene?

Unfortunately, the Regex libraries do have a different lifecycle, and I'm not sure why that is. They're not deprecated, but I can't really find any information about them on the [project website](http://lucene.apache.org/). (They used to be in the contrib area.)

> So whe're good using the default for HS ? One of my goals is of course to maximize reuse and make it more practical for your project.

Yes, you can (and probably should) use the default slot for HS. And if you're kind enough to include the Regex libraries and include support for Infinispan directories, we can happily migrate ModeShape to use the official HS module(s). Once again, [here](https://github.com/ModeShape/modeshape/tree/master/deploy/jbossas/kit/jboss-as71/modules) are our module definitions.
It's handy for when I want to compare what is happening in the dataset version
Gavin that was a relic from when I declared row as OwnedHThorRow, but later changed to const void **.   I removed the static_cast
I think github scrambled your comment..
Added them back in for now.  I'm going to do some more clean-up on this.
why bother? on the very next line it's used but not actually used.
shouldn't this be "1 download" (numerals, not string) to be consistent?
You should add here message from: `json_last_error()`.
You should leave this as ```internal```.  Please change it back.
User uppercase
Not a new class. Separate bug to fix javadoc here.
I was thinking more for the PayPal modal which lists the artists whom own
the artwork.
I agree, for the popover, tags would look nice inline.
Also, in terms of it not being displayed for very long, we can increase the
time for it to disappear. As well as, we could make it stay when the curser
is also on the popup not just the artwork.
@yuhunglin The indentation looks a bit off here.
This change dosen't increase code quality. It easier to go to class definition of localConfigFile when its type is declared explicitly.
Using ```tempfile.mkstemp``` might prevent multiple servers running in parallel having issues.
A better solution might be to use the DOM to store information about the goal--
<img class='experiments-goal' data-experiments-goal-name="{{ goal_name }}" src="{% url experiment_goal goal_name %}" height="1" width="1" />

Then in the javascript do something like:
$(function() {
     $("img.django-experiments-goal").each(function() {
        $(experiments).trigger('goal', [$(this).data('experiments-goal-name')]);
     });
});





This would also make using the javascript optional.


On Thursday, September 6, 2012 at 11:35 PM, sdcooke wrote:

> In experiments/templates/experiments/goal.html:
> > \ No newline at end of file > +<img src="{% url experiment_goal goal_name %}" height="1" width="1" /> > +<script> > + $(function() {  
> This will fail on our set up - jQuery is not loaded in at this stage in the page so it will error. This could be fixed using a similar technique to Google Analytics (in async mode) where you collect up goals through the page and trigger them when the JavaScript is loaded - a rough outline of what I mean:
> // In goal template tag goal_triggers = goal_triggers || []; goal_triggers.push('{{ goal_name }}'); // In experiments JS $(function() { // ... var triggerGoal = function(goal_name) { $(experiments).trigger('goal', [goal_name]); }; $.each(goal_triggers, function() { triggerGoal(this); }); goal_triggers = { push: triggerGoal }; // ... });  
>  
> but that's quite a lot of code for something relatively simple (and it creates another variable in the global context).
> Either way it would be nice to be able to turn this on or off globally with a setting.
>  
> —
> Reply to this email directly or view it on GitHub (https://github.com/mixcloud/django-experiments/pull/73/files#r1545330).  
>  
>  
>
ok, makes sense. Is it necessary for one to use mousedown and the other to use click? I assume in the article you linked they are using 'mousedown' because they want to catch drag events, so click makes more sense for us. Or maybe mousedown makes more sense in both cases so we catch it earlier for IE users?
Using `delegate` will bind events in the bubbling phase of event handling so there is no guarantee that if a click on an element fires an ajax request, the cookie will be set before that. `addEventListener` allows to bind in the capturing phase of event handling (if last parameter is set to `true`). Capturing happens before bubbling so we're sure that even in the case of another handler firing a request, the cookie would have already been set. The only downside is that this isn't supported by IE8 so we'll fallback to delegate for that. More details in this [article](http://37signals.com/svn/posts/3137-using-event-capturing-to-improve-basecamp-page-load-times).
please do this inside email/src/src.pro, not everything requires webkit/network
what about the server products? server 2000, 2003 and 2008?
Kwarg values shouldn't have spaces around them.
I notice there are two buttons with the same styling, we should target one or the other to avoid confusion.
Do you use komodo? 
http://www.activestate.com/blog/2010/09/writing-pep8-aware-python-code-komodo
Fixes camel case for table name
This is the doc for billboards.  Perhaps it needs to be updated throughout.
Right, I'll make it throw during a morph instead.
There's doc here, but no function.
I'd probably rename `polygon` to `positions` too so it is consistent.
Calling interval?  How about calling it "this interval"?
after taking a look at their initialization, I know what you are trying to convey with these names, but I still believe the word is not properly used. maybe something like translationOnX, translationOnY. If you don't like those, then use dragStartX/Y.
Magic number `2` as above. The code needs to be checked for instances of things like this and have a set of constants declared somewhere (whether `util` or elsewhere).
Is this code intended to fix another bug? :)
For comparison, you don't do `var setTimeout = window.setTimeout` but you also don't use `window.setTimeout()` when delaying code.
You should use qDebug here.
Is it worth guarding against empty strings here? Will postegres have a problem with `directive = ''`?
add a comment showing the format of the lines intended to be parsed
`__`!!1
http://d.puremagic.com/issues/show_bug.cgi?id=9198
Might have used readText, but this is fine too. Thanks!
Should I change the style in igammaTemmeLarge in some way?
You should be able to define these inside the unittest.
Should be able to remove it, I'll give it a go.
Yeah, I'll do it in the next revision.
spelling: framework
Yep, all good points. It does fail without this fix, but I like your idea of relying entirely on the DumpRenderTree output and not using throw/unittest. I'll resubmit today or tomorrow with these updates.
I did not feel so strongly about this that you needed to change it, especially since it is test code. But since you've done it, thanks :-)
We should discuss fluent APIs and builders over a beer some day.
I don't think there are any tests?
s/ we use compile scope as we need its API/We use compile scope as we need the API to compile against/
Previously, there was no comment.  I thought "optional encoded" was pretty clear: since `Utils.usernameFromAccount` is idempotent, we can always pass the input through the encoding.  The source of this bug is errors encoding at the callsite; why not fix any that come later?  (There are not enough calls to worry about performance.)
Just leaving a comment on this file to say that I checked as well, and the removed functions herein are not called in call log.
I think this should be `PSU_EXTERNAL_DIR` rather than `PSU_LEGACY_DIR`
Corrigir o Vermelhor
Or it looks from `levents` like the third parameter is `batching`. Definitely could use clarification :)
Is this function necessary? It seems to be translating a boolean into 1/-1.
I will test and fix this after the pull
Someone please confirm that this is the case.
Confirmed, what is the expected osfamily of arch? Probably Archlinux right?
sounds good.  Hope this new comment is better.  :)
Should we be requiring tinyMCE here?
Should be 4 spaces instead of 2
make an explicit getEntity test where you get the entity by ID and verify that the generatedBy is not null
Just top add. What I said may require modification to the original constructor in HREMH which now takes URL as String, but since its already a breaking change, might as well make it more concise now. wdyt?
If we really should be doing runtime channel resolution, this test should really send a message to verify the resolution is done properly.
Gunnar, this changes and many other about delayer in the 2.2 should be reworked as the 'id' attribute was introduced only in the 2.2 :-).
Take a look: https://github.com/SpringSource/spring-integration/wiki/Spring-Integration-2.1-to-2.2-Migration-Guide.
WDYT?
...2 has to be instantiated outside of the test class's scope, for other tests, so I would have to change the 'resolvedUri' field to be static too, which doesn't feel right. Plus, the test using ...2 has noting to do with 'resolvedUri'.
Artem

Also, its not about the invention. Its really about if its necessary or not for the given scope. Spring is a general framework while SI is a very specific one, so we always try to apply a KISS principle until there is a need to make things more sophisticated.
Thats why I said that all you seem to need at the moment is that filtering logic inside of that inner class, but in the end there was just to much stuff
Guys, you don't have to do this... The identity_column is ALREADY generic.
I see that the problem happens on oracle since it cant query a "select field1, field1, field2, ..."

The query on PR is this:

````php
$this->db->select($this->identity_column . ' identity_column, username, email, id, password, active, last_login');
````

The result fields are: ``'identity_column'``, ``'username'``, ``'email'``, '``id'``, ``'password'``, ``'active'``, '``last_login'``

There is no problem on the value of "identity_column" being the same of "username" or "email". It is just an alias to avoid the mentioned problem with oracle
The only change needed is on your line 876:

from 
```'identity'             => $user->{$this->identity_column},```
to
```'identity'             => $user->identity_column,```
Lucky for you I did look into all of them before deciding on `bootstrap-sass` for [brewdega](http://brewdega.com). :)
same as the other related notes
Also `AbstractController`?
You need an inheritence line as well:

```JavaScript
util.inherits(exports.FunctionType, Context.FunctionType);
``
startPage は 1 以上が有効な数字なので、本質的には (startPage < 1) とすべきかと思います。
startPage パラメータが指定されていない時に 0 が入り、デフォルト値の処理をするのでこうなって
いるのだと思いますが、それはそれでデフォルト値と明示的に 0 を指定された時の区別が
つかず、思わぬ誤解を招くので良くないと思います。
Define this and the other numerical constants above using `let` so that you don't have to copy-paste everywhere if you decide to change the constant.
Once you get to this point (which should be alittle way down the road, get the creation of the recommendations down first), this should be another namespaced area b/c you dont want those peopel to have to log in.
@christianv Can you remove the word 'as' and change the &amp; into the word 'and' ?
I can't, I'm not repo collab.

Try editing [the file](https://github.com/evgenyneu/coffee-script/blob/patch-1/examples/computer_science/luhn_algorithm.coffee).
Sorry -- I just deleted that comment.  I missed the actual end of the do block for "matching_nodes.map do"
Hi Philip, 

Sorry about including changes to the readme by accident. I reverted them and they are no longer part of this pull request.

Best regards,
Sytse
No problem, it makes a better looking code at the end.


On 9. apr. 2012, at 13:59, Mitar wrote:

>> +        excludes = getattr(new_class._meta, 'excludes', [])
>> +        field_names = new_class.base_fields.keys()
>> +
>> +        for field_name in field_names:
>> +            if field_name == 'resource_uri':
>> +                # Delete resource_uri from fields if this is EmbeddedDocument
>> +                if meta and issubclass(meta.object_class, EmbeddedDocument):
>> +                    del(new_class.base_fields[field_name])
>> +            if field_name in new_class.declared_fields:
>> +                continue
>> +            if len(include_fields) and not field_name in include_fields:
>> +                del(new_class.base_fields[field_name])
>> +            if len(excludes) and field_name in excludes:
>> +                del(new_class.base_fields[field_name])
>> +        
>> +        # Add in the new fields.
> 
> No dot at the end. (Sorry, I just cannot help myself noticing such things.)
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/mitar/django-tastypie-mongoengine/pull/1/files#r659757
typo in specifies
`return strip_accents` should do the trick :)
I agree, but for the benefit of more novice users, how about two settings, defaulting to "click" and "easytabs" and still concatenating within the plugin? So the options could be "event" and "namespace"?
Vishal says there was no changes made to TiRect.
Actually, if a title is not explicitly set, a dialog with no title bar is created.
No entiendo... este test viene asi desde hace rato 
https://github.com/laradiodelsur/laradiodelsur.web.policy/blob/f1f5e463ecc2d62a18722c049ee5dde2b9f6d044/src/laradiodelsur/web/policy/tests/test_initial_settings.py#L82
This should return boolean and not Boolean. It should return false if incrementalVersion is null.
Perhaps link to scala-user at the end? E.g.

"More discussion on [scala-user](https://groups.google.com/forum/?fromgroups=#!topic/scala-user/Lh0YS_RnBQE)."
I'm thinking something like this might work a bit better: 
  $lame_path = escapeshellcmd($lame_path);
  exec("$lame_path --help", $output, $return);

From that we should be able to print a nice version string and also know more information about lame. I believe this will also be platform independent.
Again, the earlier question, should there be combined fields, or calculated fields to support front page summaries?
Это не логика становится сложнее, раньше просто посту соответствовал `table` и всё было хорошо, кроме того, что ВСЕ функции поиска в посте выполнялись медленней (надо обходить больше нод). Я же сделал соотвествие посту ноды `td`. Это одновременно и ускорило поиск (в лисе, кекеке) и внутренние функции скрипта, но потребовало создание "обёртки" для нормальной подзагрузки постов.
Are you sure this is ok? If this is portuguese and you return pt for pt-BR a lot of people are going to be confused as the language is pretty different. Same goes for mandarin and cantonese with zh-CN and zh-HK.
Why isn't this test running?
You're right. Good catch. I shall make this change.
Klopt niet! zegt arjen
I think it should be parsed to the correct format before it reaches this point to avoid having this type check logic here.
external css?