labofsahil/github-event-dataset-2012 huggingface dataset:
Indeed correct - we are still figuring it out. Having class expressions looks like it would be a good way to go, should help in automating certain checks.

Multiple values should be comma-separated, to make it easier to parse them.

I'll put some more details in the wiki documents - also feel free to add your own. Everyone should be able to modify the wiki.
Greats! Please, add methods to official documentation
You can't (in Chrome at least) call ``.length`` on an object. I've just finished adding / revising the tests... you'll see what I've done instead when I push the latest changes.
I'd rather have them gone

On Mon, Dec 24, 2012 at 3:14 PM, Zeke Fast <notifications@github.com> wrote:

> In spec/parallel_tests/tasks_spec.rb:
>
> >      end
> >
> > -    it "should be whatever was set" do
> > -      ENV["RAILS_ENV"] = "foo"
> > -      ParallelTests::Tasks.rails_env.should == "foo"
> > +    it 'should be whatever was set' do
>
> All the changes for double quotes replacement were moved to 09538ae<https://github.com/grosser/parallel_tests/commit/09538aea66365d20f16f8ec3e886c2d6e3a62996>
> .
> So, they can be eliminated with git revert
> 09538aea66365d20f16f8ec3e886c2d6e3a62996.
>
> Do you what a separate PR for quotes or just do not want merge those
> changes to quotes style?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/grosser/parallel_tests/pull/170/files#r2499312>.
>
>
Actually the installer will need some more thought, because you need to rollback changes. The installer is notorious for not giving good error messages so I think we'll need a different strategy.  I think the DB error message will be enough to pinpoint failure, but rather than logging the error, I think it would be better to JLog the successes so that a developer can trace through the point of failure.  So, every time we do something, we could do a:

``JLog::add(JText::_('JLIB_INSTALLER_LOG_SOMETHING'), JLog::INFO, 'jinstaller');``

com_installer can then add a message queue and possibly file log listener and then a big try-catch (where catch will rollback).

Does that make sense?
add a body!
Don't you want this one to be relative as otheriwse it won't work through the portal?
Done in b1ebdaf.
See the first code example: http://symfony.com/doc/current/contributing/code/standards.html the first return in the `transform` method doesn't have a blank line before.
Somehow this section as it is isn't totally clear to me. Why not provide an example like this instead? I think it makes it totally obvious what `raw` does if you just show `s` versus `raw`:

    scala> s"a\nb"
    res0: String = 
    a
    b
    
    scala> raw"a\nb"
    res1: String = a\nb

wdyt?
some things I do just for scan-ability. can drop.
No, you can have only one list given to template. In Python you do a query to the database and remove duplicates (retain only the newest one). Then you can just add additional attribute to Python object with a check for stale date you want to do:

```
station.old_data = station.time (or how you will name the field, see how we named similar fields elsewhere) < now - 2 * (interval from settings, 60 seconds currently)
```

Then in template, display this data, and based on the flag, add a warning sign about stale data.
```
from piplmesh.api import models as api
api.Post.objects()
```
The same goes probably for e-mail. First time we want to check it only on blur, no?
This is also a constant. So write it with all-upper-case letters.
One word.
Space before `**`.
Okay, then tell me a solution how to destroy/completly despawn the chest after looting like a go spawned via spell cast... I know this isn't the best way, but it's working for me in this place. Maybe there's a spell for summoning the chest, I'll take a look at the dbc this evening.
Can you please use the header comment style used in the other source and header files?
Love this comment block. Very informative.
Yeah I didn't realize that discriminators could be any valid json construct, that will need some thinking and changes.
Please rename this method to just `click`
Must use cookie in response here, not in request!
To be extra safe, could we make this (I'm not sure which option is best):

kiji_jar=$KIJI_HOME/lib/${project.artifactId}-${project.version}.jar
OR
kiji_jar=$KIJI_HOME/lib/${project.build.finalName}.jar

I'm betting the second option is best.
Is this test needed? It looks like you get the intended coverage in the next one.
Beware of which warning you issue though. Python 2.7 now ignores DeprecationWarnings see http://docs.python.org/2/library/warnings.html#default-warning-filters

Here I think we want this warning to always appear, whatever python version we use
adding some comments here would be welcome, to avoid having to read everything to understand what's going on. I'm saying this as a general rule, as this code will be read way more often that it will be written, adding comments when things are not clear helps.

In this specific example: why are we removing self.config from the path? for instance.
This test sequence is going to be fragile. What if there's enough network latency that you miss the StartingVM state and it goes straight to running (from your perspective)?
´var use_raw_boundary = Boolean(do_raw && req.boundary);´. (Guessing `req.boundary` is not a boolean.)
No space inside object literals.
Yes. Init() adds a timer, even on error, so it needs to be removed from the rbtree again (unless PSOL calls ShutDown() when Init() fails. I don't think so but @jeffkaufman probably knows.)
The new parameters were actually never used--so I took them out.  

I ended up sticking with the idea that l1 regularization should simply be a different type of fit method.  So you just set method='l1', or method='l1_cvxopt_cp'.  This is consistent with the way things are implemented at present.
I'm not a nitpicker on copyrights, but I guess if we do it cleanly from start it helps others - care fix?
Thanks.

a more general question: com.google.common.collect.Maps is faster? what are the advantage of those?


On Feb 24, 2012, at 1:04 PM, Nima Asadi wrote:

>> +        * (-1.0 / 12.0 + f
>> +            * (1.0 / 120.0 + f
>> +                * (-1.0 / 252.0 + f
>> +                    * (1.0 / 240.0 + f
>> +                        * (-1.0 / 132.0 + f
>> +                            * (691.0 / 32760.0 + f * (-1.0 / 12.0 + f * 3617.0 / 8160.0)))))));
>> +    return r + Math.log(x) - 0.5 / x + t;
>> +  }
>> +
>> +  /**
>> +   * Approximate the trigamma of x.
>> +   * 
>> +   * @param x
>> +   * @return
>> +   */
>> +  public static double trigamma(double x) {
> 
> Yes, I believe that's what Jimmy is saying:
> 
> Map<T1, T2> object = new HashMap<T1, T2>();
> 
> Even better, use com.google.common.collect.Maps for Guava:
> 
> Map<T1, T2> object = Maps.newHashMap();
> 
> Nima
> 
> On Fri, Feb 24, 2012 at 12:59 PM, Zhai Ke <
> reply@reply.github.com
>> wrote:
> 
>>> +        * (-1.0 / 12.0 + f
>>> +            * (1.0 / 120.0 + f
>>> +                * (-1.0 / 252.0 + f
>>> +                    * (1.0 / 240.0 + f
>>> +                        * (-1.0 / 132.0 + f
>>> +                            * (691.0 / 32760.0 + f * (-1.0 / 12.0 + f * 3617.0
>> / 8160.0)))))));
>>> +    return r + Math.log(x) - 0.5 / x + t;
>>> +  }
>>> +
>>> +  /**
>>> +   * Approximate the trigamma of x.
>>> +   *
>>> +   * @param x
>>> +   * @return
>>> +   */
>>> +  public static double trigamma(double x) {
>> 
>> I don't understand, Map is an interface, right? Are you staying something
>> like Map map = new HashMap()?
>> 
>> On Feb 24, 2012, at 12:53 PM, Nima Asadi wrote:
>> 
>>> (that's sed syntax: substitute Map for HashMap)
>>> 
>>> Nima
>> 
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/lintool/Mr.LDA/pull/1/files#r486093
>> 
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/lintool/Mr.LDA/pull/1/files#r486124
What happened to the handler chain that was previously set?
Ought to just call this `name_`.
remove_template or removeTemplate?

I don't care, just asking....
How is that? npix below is an odd number and is passed directly to all functions in optics.py, where it is not modified and used as the size of the FFT array.
Might it be worth tracking the number that are discarded and issuing a warning?  I imagine that most cases where we're throwing away lots of the photons are due to some error in inputs that the user might want to know about.
Sorry for not saying explicitly but I will 100% fix the code based on your comments tonight. Won't be at my workstation today during the day. I'll change the name, revert the equality, and do you want ParVector implemented as I mentioned? Think overhead might be too much for that.
Should add this option to the "usage" printout.
It probably has to do with where this fragment is included from.  Use an absolute link
The body.append(String...) looks nicer from the caller's point of view.
But good that it's fixed. Will still merge, and we can discuss after.
Why is this renamed?
It was another memory leak that I found in this same code when checking with valgrind.
On Thu, 4 Oct 2012, Hannes Magnusson wrote:

> >  	if (db_start) {
> > -		mongo_manager_log(manager, MLOG_PARSE, MLOG_INFO, "- Found database name '%s'", db_start);
> > +		tmp_database = strndup(db_start, db_end - db_start);
> > +		mongo_manager_log(manager, MLOG_PARSE, MLOG_INFO, "- Found database name '%s'", tmp_database);
> > +	} else if (tmp_user && tmp_pass) {
> 
> Is password required? There is no way to have a username without a password?

Yes, a password is required (for now).

> The log message also looks a bit informal "so defaulting"..

Changed it.

Derick

-- 
{
  website: [ "http://mongodb.org", "http://derickrethans.nl" ],
  twitter: [ "@derickr", "@mongodb" ]
}
yeah, thats true. we would have to use a mixin type. getting more heavy. but whatever, the controller can work with several classes, so we can just do like this and add a more sophisticated later.
I squashed commits and your next comment:

    Can you revert this change, this was done on purpose to demonstrate how to use the render…

disappeared. Changes to the controller were reverted.
I think it's simpler to use `io.open` directly - it makes it obvious that we're not doing any extra magic. Also, it's identical to the Python 3 built-in `open()`, so there's less to change when we eventually drop py3compat.
rainy day work would be to extend HttpRequestBuilder which would then remove the awkward .httpRequestBuilder() thing cc/@aplowe @grkvlt
So, I think this was a screwup on my part. I had cache machine caching counts locally and didn't discover it until after this went out. I'm now using cache settings more like what's on stage.

Given that I had "bad settings", this should get backed out since the comment is wrong and the code change shouldn't have any effect.
I took out this comment because it's more complicated to do now that there are three groupings and the groupings aren't easily handled with a group_by.
PHP5 syntax please, that needs `public` on it.
Aaaaand another one.
No, it is our general practice to not require additional services to be running for testing
А почему не пользуешься общим?
Конструктор должен быть public.
Пробела не хватает
How you decided that this is the correct expires date? By all means it's good to add comment explaining that either if it's a heuristic from you or something dictated by Media Services guidelines.
Oh, the two chunks of code are identical because they act upon two classes which are identical. In that case, should the two classes share a common base class with those properties?
My previous comment was lost. This comment is not needed, and I suspect removing this blank line does not make the PEP8 gods happy.
why DateTime ?
そのように改めました。
「クエリー」と「クエリ」は統一したほうが良さ気
Should just remove the commented lines if they aren't needed.
Typo. This method should accept `*args` and call `super` as well.
Would it be possible to add a new feature type: `csr_matrix` where the integer feature indices would be hashed and the collided values summed so has to implement a kind of sparse random projection based on a hash function on existing high dim sparse datastructures?

I am not really sure this is really need but I think either @mblondel or @amueller expressed some need for it at some point in the past if I remember correctly.
doesn't this defeat the purpose of the test?
Maybe I should have waited for #1198... the current ``GridSearchCV`` contains some magic...
This should be `is_dir()`.
I thought we were going to zero out the turns variable when we changed directions. The way it is you have to know how far you went forward to know how far you have gone backwards instead of treating each move separately.
Is this going to end up being run on an MSC thread? If so and a deployment ended up spawning a few services with these in it, it could deadlock MSC.
License folder is added. licenses for madlib and madmark are added. Licenses for third party code (GPL, PyYAML, Python and argparse) are also added.
Same comment as for mscoff.h.
Our framework doesn't know about Date type. This class is for that purpose. Maybe there is other solution for that, but without it it got crashed with var date = new Date(); statement.
You need to setup the event object to be passed into the event handler function. You should at least add a TODO comment before the fireEvent method.
It's not needed now, it can be added later if there is a need for it.
it's not about performance - the process is slow - it doesn't make sense to check more often
I can't see the benefit of special casing a known number of records.

- Normally the number of deduped rows would be significantly lower than the number of input rows, so it is likely to end up with a hash table that is much too large.
- Even if a spill fill is being read in there should be duplicates - (using a new hash function/modulus).  In the unlikely event that a spill file had nothing added (should probably be optimized to the close-to-memory case?) then the rows should be directly streamed instead.
It probably should, although default handling in Thread would just be to trace it.
However, that still doesn't fix the issue that the local instance resolved directly by the directory exists at the same time as the transaction's cache one, but it does limit the lifetime of the copy.

To solve that problem, I must make sure all files are resolved within a transaction/cache context, so we can always guarantee that the same instance will be returned in case of double-resolution on the same context. For that, I need ```DFSAccess``` working, which is on the roadmap.
Not that I know of. There are a few assumptions that win32 means msvc, but should be harmless.
Just FYI
You don't need the `[` and `]` here. I think `dict()` can turn a generator into a `dict` object.
Feew, I was afraid I understood this totally wrong. No problem Peter, I'm happy if we agree that this works! :)
how is this idea?
```ruby
def ThreadStorage < Hash
  def [](key)
    ...
  end

  def []=(key, value)
    ...
  end
end
```
when julia switches to nginx, we use the version of homebrew, of course :-)
``false|string`` ? Please don't create crappy APIs like the one done in PHP for such methods. Returning either a string or a boolean is really ugly.
This should be more like:
`Invalid API driver specified, available are: 'oauth', 'api'`
The naming is weird as it applies to 2.2 too :)
@christianv clever double bang trick: http://stackoverflow.com/questions/784929/what-is-the-not-not-operator-in-javascript
may be in create builder?
This test is actually destructive since it involves click_create_addon_btn().
add an else warning("gstreamer packages not available, video thumbnailing disabled") or something like that?
As said, In my opinion hasn't sense to keep a temp file to store usernames you find, please switch to a data structure such as an Array.
Can you break this out into helper methods, like

```ruby
def all_possible_combinations_of_greater_than_or_less_than_methods
   [:greater_than, :greater_than_or_equal_to].product([:less_than_or_equal_to, :less_than])
end
```

or something like that. It's very hard to parse as-is - I understand your explanation, but I want it to be immediately readable to people who don't have access to this explanation in the future. For example `parameters.first` isn't immediately obvious as "the validation with `greater` in it", etc.
Yes, but I haven't done it yet, mostly out of laziness.  I added it to the roadmap.
disable and enable methods should go away once we have implemented the EventBus change, e.g., channel.
I "think" so :) There is another comment about this and my understanding is that an exception route does not have a path.
Missing space before "(".
Can you update this comment (since "scrolling" isn't used in the long run)? thx!
After looking at the builder scripts, they appear to detect the existence of the build directories specific to the platforms, but not whether or not they are empty. As far as I can tell at least. For e.g., 

support/android/builder.py line 244
{code}
if not os.path.exists(self.project_dir) or not os.path.exists(os.path.join(self.project_dir,'AndroidManifest.xml')):
			android_creator = Android(name, app_id, self.sdk, None, self.java)
			parent_dir = os.path.dirname(self.top_dir)
			if os.path.exists(self.top_dir):
				android_creator.create(parent_dir, project_dir=self.top_dir, build_time=True)
			else:
				android_creator.create(parent_dir)
			
			self.force_rebuild = True
			sys.stdout.flush()
{code}
 and the iphone builder script seems to do the same.
CAp.
Would this be clearer if it were inside the if block?
No space between parens and braces.
Maybe I'm missing something, but why would it work?
Doing so would require storing a copy of arr[pos..$] somewhere; inserting directly after pos would destroy the tail of the array. However, if one only built the tail using appender, this would be possible. But without punching into appender's internal implementation, (which insertInPlace could do), we'd have to hit he GC at least twice and the implementation would become less DRY. Do you think that's worth it? Also, do you think that the above static if conditional should be moved inside dupTo's implementation? (i.e. to ease generic programming)
Oh, drat. A change was made to std.algorithm, and I'm going to have to rebase this again. I'm getting tired of that.

I'll just add the extra restriction to `isRandomAcessRange` when I do that (hopefully this evening).
That's actually an *excellent* point, and it would have helped while developing too...
that's really the change right?
Also, maybe join both functions, because they're almost doing the same thing...
remove those here
A few sentences javadoc would be nice here
Seems like everything below here should be private.
This is the entire problem that we were not able to resolve with this - remember. so no - not going to change... comment removed 

----- 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 8:01:13 AM 
Subject: Re: [plymouth-webapp] changes for adding new BM Report App (#425) 


In legacy/hub/HUBSQL.class.php: > +		$data['lastname'] = $filer->formatName('l');
> +		$data['firstname'] = $filer->formatName('f');
> +    $data['bmr_date'] = date('Y-m-d', strtotime($data['idate']));
> +
> +		// get previously saved version of the report...
> +		$bmr = $this->getBMReportByPidm($filer->pidm);
> +
> +		$data['modified'] = $now;
> +
> +		// if created is set - already saved one "round" get prev. saved information...
> +		if (isset($bmr['created']))
> +		{
> +			$data['created'] = $bmr['created'];
> +
> +			// Since using MySQL, can use a single REPLACE instead of check for existence
> +			// and then having to use INSERT or UPDATE 
Shouldn't we be doing this then? 
— 
Reply to this email directly or view it on GitHub .
ok will make that change
Submitting one of these forms may also lead to the amount of time left for a drug changing. How about rendering the html partial server side, and simply replacing the DOM `tbody` node for the drug with the specified id? We then don't need our JS logic to have `alertDrug`/`unAlertDrug`-level of specificity.
whoops  how did that get there
This variable is nowhere used.
I prefer this fields on a row, it's more readable for me
be careful, your uri is not escaped
Good change!
Cute. Are we going to integrate this before having content?
Sounds like a plan. I did update the docs to reflect the change, but an upgrade document would probably help.
My vote would be to be consistent with how legends are done (but without "best").  So, map names would be best.
I'm not sure about performance, have you tested it? Here is I've put a little test case: http://gis-lab.info/share/DR/array/array_performance.html that can be used for testing in different web browsers. But my point of view that the use of the Array constructor should be avoided as much as possible. Literals are definitely preferred. They are shorter and have a clearer syntax; therefore, they also increase the readability of the code (http://bonsaiden.github.com/JavaScript-Garden/#array.constructor).
Can we chain this?

```javascript
.off('hover').hover(...
```
Is this creating any additional overhead?
S is the same thing you have, the covariance matrix computed over both lhs and rhs if lhs!=rhs (like it should be now).
I've definitely gotten in the habit of including helpers on the controller since I've primarily been using helpers within the ActiveAdmin DSL, but I suppose it does make more sense to register view helpers here. I've committed the change. Let me know what you think.
Your learning to speak my language!
Right and that is what he is listing  after stating 'and the latter for a SpEL expression to be evaluated. . .' after which he lists possible scenarios. So it does capture that its any valid SpEL
Just name 'lockTable' or similar - there's no concept of "available" here.
dont u have to call done at the end ?
Oops; IDE automatically removed trailing whitespace.  Doesn't seem to have an effect upon the rendered markdown, though.
まず、皆さんおっしゃる通り、「structural type」で実用的には十分意味が伝わると思います。なので、英語のまま「structural type」かその直訳的な「構造的な型」でも良いかと思います。訳としては自分としてこれ以上コメントできる事はなさそうです。

以下、型システム的な薀蓄。

----

私は、Scalaの「structural type」について、よく「structural subtype」、あるいはその訳語として「構造的部分型」という言葉を使いますが、これは、型システム一般の話としては、単に「structural type」だと、「subtype」の部分のニュアンスが抜けてしまうからです。

元々、subtypingのシステムには、大別して「structural subtype」(構造的部分型)と「nominal subtype」(名前的部分型)の二つがあります。前者は、型Aが型Bのサブタイプであるかどうかが、それぞれのデータ型のメンバの包含関係によって暗黙に決定されるようなsubtypingのシステムを、後者は、型Aが型Bのサブタイプであるかが明示的な宣言(これはScalaだとextendsだったりwithだったりします)に基づいて宣言されるものです。

ふつーの静的型付けオブジェクト指向言語のsubtypingのシステムは、ほとんどの場合nominalです。(Java, C#, ...)これはScalaも例外ではありません。ただ、Scalaの場合、2.6でstructural typeが入った事で、subtypingのシステムとしては、nominalとstructuralのキメラになりました。

これはどーいうことかといいうと、

val hoge: A { def hoge(): Unit } = ...　// A はクラスまたはトレイトと仮定。A自体がstructural refinementのaliasだったりすると、またややこしいので以下略

という変数を宣言した場合、「Aを直接・間接に明示的に継承しており、*かつ*、hoge():Unitメソッドを持った型」という意味になります。

さて、以上はsubtypingがある言語の場合の話です。一方で、subtypingが無い言語でも、型付けに関して「structural」と「nominal」という用語は使われます。そういう文脈では、二つの型が互換性を持つかどうかを判定するためにどのような戦略をとるかを意味する用語になります。「structural」では、型(普通、構造体)のメンバ集合が完全に一致していなければならず、「nominal」では、「型の名前」が一致している必要があります。「structural subtyping」とか「nominal subyping」というのは、この辺の話に由来しています。

----

で、この辺りのうんちくはともかくとして、subtypingのある言語で「structural type」を導入したら、常識的に考えて、subtype関係についても定義するだろう、とい気はするので、あえて「structural subtype」とか言いなおさなくても、ただの「structural type」でいいかも、と思います。ただ、OCamlerとか、その他、静的型システムに詳しい人に、Scalaの「structural type」を説明するときは、私は、意味を正確に伝えるために、「strucrual subtyping」という用語を使います。
Könnte es sein, dass hier nicht ``n + 1 > 0`` sondern ``n + 1 > n`` steht? Stimmt zwar beides, aber wenn man sich die nächste Gleichung anschaut, macht das erste mehr Sinn.
In `buildView` method, in same file, line 108.
Please don't change the CS.
could you add `1.0.*` instead?
Agreed, but I was in a "get this done by yesterday" mode :)
This class needs doc.
That's tricky to do with JSON Schema.  Better to just document that the parameter is ignored if the external document is accessed with XHR instead of an EventSource.
Im not so scared anymore!
Underline should match title length.
That's only works for a polling model which would be extremely wasteful in our case. Because we don't use Thread.sleep, there is no place to catch the InterruptedException. Regardless, I've added a Socket timeout for the server so that we can perform a graceful shutdown. 

InterruptedException also doesn't work because acceptSocket or socket.readline block by themselves and interrupted exceptions aren't handled in either of those cases.
The right way to get form data is from ``filter_form.cleaned_data``, not directly from ``request.GET``.  Make sure you've read all the django docs on forms: https://docs.djangoproject.com/en/dev/topics/forms/
fix for Prototype -> jQuery pxToEm
Should this be ```owner__id``` to be consistent with the use so far, and more explicit about what's being compared?
add a force option.
When the deploy() method return it's expected the deployment can be reached via info given in ProtocolMetaData. (no async deployment)
You probably want to provide the examples in separate lines, ie. so one can paste them in a 'pp' file and try them out.
Again, this still needs a better summary on what the behavior is. See above.
...should be larger by one than the...
Consider using the System.Threading namespace and removing the explicit declaration.
Is `uri` always valid?
Hmm, so what do you think about this:

https://github.com/throughnothing/Test-WWW-Mechanize-Dancer/commit/edeb72da42dc7d0860725cc14c06436d1225125e

Maybe I should just put it back to how you had it, and update the documentation?
That was the idea (to me). If the webview already exists (ie you've already created it, or it is the chrome) then you don't need to create one.
Stop this discuss.
It check only existing file template. If the file does not exist then occur phpbb template error. This removed line avoid the problem for non existing template. So now is removed and when no assigned template exists then occur standard phpBB template error.
First I wanted to call the Help-Command automatically and take also the description and have it as ToolTipText - but since i couldn't call any commands i withdrew the idea and kept it simple for the beginning