labofsahil/github-event-dataset-2012 huggingface dataset:
let me know when you have this working up to the idxs/vals, and then I
can take over from there if you like.

On Thu, Feb 23, 2012 at 2:42 PM, Dan Yamins
<reply@reply.github.com>
wrote:
>> +                    orig_result = result[0]
>> +                    assert hasattr(orig_result, 'keys')
>> +                    assert all([isinstance(x, tuple) and len(x) == 2 for x in result[1:]])
>> +                    assert ([hasattr(x[0], 'keys') and hasattr(x[1], 'keys') for x in result[1:]])
>> +
>> +                    return_val = []
>> +                    orig_result_return = handle.update(self.current_job,
>> +                                           dict(result=orig_result))
>> +                    return_val.append(orig_result_return)
>> +
>> +                    num_new = len(result) - 1
>> +                    new_ids = self.trials.new_trial_ids(num_new)
>> +
>> +                    for new_tid, r in zip(new_ids, result[1:]):
>> +                        spec, res = r
>> +                        new_rec = copy.deepcopy(self.current_job)
>
> OK.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jaberg/hyperopt/pull/84/files#r481939
Not sure about the naming, though...
if the file is called petsc31, the formula should too. But if possible, I'd avoid the number alltogether.
It comes from factoring out d. Let's stat by using the arc parameter 't'. (In fact, if you wanted to update notation to match the link to Matlab, I think it's generally better.)

N0=norm(p2-p1)*rotation
n=N0/norm(N0) = norm(p2-p1)
Q=P0-T*n; = P0-T*N0/norm(p2-pa)

D=p2-p1
d=norm(D)
T^2=r^2-(d/2)^2=r^2-d^2/4 = d^2*(r^2/d^2-1/4) == > T= sqrt(d^2(r^2/d^2-1/4)) = d*sqrt(r^2/d^2-1/4)

Q=P0-T*N= d*sqrt(r^2/d^2-1/4) *n = d*(r^2/d^2-1/4) * N0/norm(p2-pa) = d*(r^2/d^2-1/4)*N0/d = (r^2/d^2-1/4)*N0
Yeah casting a QObject * is probably a more C++ way and more readable.  I'm just not terribly proficient in C++ trickery :)
I'm confused about this test.  If I haven't registered a StepRegistry, why is it required that Job be an instance of StepLocator?  Wouldn't you want it to just not attempt to register the steps since there is no StepRegistry provided?
I don't think the datepicker class is needed here anymore.
We should add a space around ">"
why do you have the import here?
bump me to 0.7.x
is -ldl realy needed ?

because it's doesn't exist and useless on FreeBSD
"make use of" rather than the awful "leverage"?
Given that you have two `self.` definitions I personally think it would look better this way:

```ruby
# Refinery::User isn't available when this line gets hit, so we use static methods instead
@@user_class_name = nil
class << self
    def user_class=(class_name)
      if class_name.class == Class
        @@user_class_name = class_name
      else 
        raise TypeError, "Expecting configuration input of type Class."
      end
    end
    
    def user_class
      @@user_class_name || Refinery::User
    end
end
```
Here, you should use 'store_precision=None, assume_centered=None' in the function definition and assign them to the object only if they are not None, raising a DeprecationWarning.
It shall be done :)
Yes. If only `len(X)` worked for sparse matrices...
What is the reason that negative numbers don't have the usual list semantics?
Why did you remove this? Both methods POST to this URL, unless I'm missing something.
It might be better to call this method ```self.save``` to resemble to ActiveResource method name.
Ok, io.File and readLine is used now.
API says: If offset and length are omitted, all of the data in the buffer is written to this stream.
The index is an integer to begin with is it not?  Is there a specific reason to create the QVariant with a string rather than the integer itself?  I see no reason why the index should be converted to string in the first place.  

Furthermore, more validation is needed to ensure that the argument to scrollToIndex is indeed an integer.
Shouldn't be necessary as it's already a parent class of NDTP
Do we still need icon.png and icon2.png?
We have blackberry NDK detection in blackberryndk.py, so probably no need in this.
!== ?
Les index de recherche utilisés par le moteur de recherche
Just renamed the file to match EditData.hpp - header files that define C++ classes have .hpp and header files that don't have .h
This only works if you execute dump.py from the examples directory. If you execute it from somewhere else ".", ".." and "../src" could be anything.
Is this plugin used? I don't think it is used as part of the basic build steps. What does it do?
perhaps 

```
typedef CInterfaceOf<StringArray> CStringArray;
```

Would have been clearer?
Are you sure you're allowed to modify this memory???  This looks highly questionanable - it could be a memory mapped file being passed in.
Saw that later ;)
Here again, array shortcut can be used
`{User, Track, Tracks, Playlist, Playlists}`
It appears to work for the purposes of initializing searchbird. I just picked the latest versions of the three finagle libraries (and did not specify the util-* libraries). But of course searchbird is only one use instance.
Given that none of our addons are getting unzipped anymore we should get this fixed in mozprofile so that addon_details supports XPI files. Thanks for that check Dave. I filed: https://bugzilla.mozilla.org/show_bug.cgi?id=797832
theoretically you could just have multiple iptables-multiport in a single jail, e.g.

[asterisk]
action   = iptables-multiport[name=asterisk-tcp, port="5060,5061", protocol=tcp]
               iptables-multiport[name=asterisk-udp, port="5060,5061", protocol=udp]
               ...

imho that would be cleaner... ?

also by default jails should be disabled (not everyone is running asterisk)
@logged
This assertion failed on my machine, which is pointing at real Oracle data: 

```
org.junit.ComparisonFailure: 
Expected :Stu Test-300846
Actual   :IWBEIWETWTB,EWOPEOQEQW
```

Can you weaken the assertion to just not null?
@shacker can we fix the indentation here and below?
why are you throwing an exception here and then immediately catching it further down?
is all this code commented out?
Yeah, I thought about doing that, but sometimes when the shell is broken it leads to timeouts for every call to ```shell_command_token``` so it would be another 5 seconds before the user gets any feedback.  That's why I went with del and rm in the same command to keep calls to ```shell_command_token``` to a minimum.  However, I agree that it would make it more reliable.  So I guess I'm saying I'm on the fence about it.

In unix, it could just be ```[ -f #{file} ] && echo yay``` since that should all be shell built-in stuff.  In windows, I'm not even sure how to do it.
is #{tpath} valid ? domain_path maybe ?
For ease of reading.  It is not a strict requirement, especially for code lines where it's sometimes difficult to keep a line under a certain length, but for comments and free-form text such as module descriptions, we like lines to stay within 80 columns.
Thanks, that assignment should work in all cases.  Updated.
It might also be worth it to add a note that the IPython display protocall isn't implemented in the `except` case. I'm iffy about this, since it could be too much noise and a bit confusing. On the other hand, it may be more confusing if I'm looking at an example that shows an image, but the example doesn't behave the same on my system.

.... I'm not really sure.
dirname this is not windows-save when manipulating paths. can you do a string operation to achieve this please?
So the main problem i see with having these in 2 places is making sure they stay synced. If someone in invision requests a very minor change the designer will have to make sure to also push that into git. It just seems like a big risk and that we might end up developing screens off wrong mock ups.
agreed. but tim never pull requested sneaky so i guess this wont be happening this weekend :( @TimSimpsonR pull request the sneaky code!!
addMimeTypeToExtension
Needs an `@exports`.
By doing this:
`var f = function f() { ... }`
rather than this:
`var f = function() { ... }`
Chrome (and probably other browsers) will have a value for `f.constructor.name`.  In some of my tests, that was useful in order to provide better failure messages, so I got in the habit of doing it everywhere.
A few points:

1. The circles here are not wheels; they are widgets to modify vertices of the polygon. You may go to http://linzhp.github.com/botprint2/, and see them in editing mode. So there is nothing to do with model here.
2. It's not an intuitive way to depict wheels using circles in Canvas2D. The top-down view of wheels is not a circle. I remember either Gillian or Jim brought this up in a meeting.
3. `circle` already extends `View` in above code, although I didn't explicitly define a `CircleView` class. I am not against having a `CircleView` class either. 
4. I think you could spend some time trying to implement your idea, so as to understand the problem better. Just pull this to a branch in your git repository, then you are free to make any changes. If you are able to implement a better solution, just send another pull request that supersedes this one. I feel I am not good at explaining code to you. So I guess you might spend less time understanding-by-refactoring than listening to my awkward English:) But I am happy to help when you really have problem in understanding.
Np, if it will cause collateral effects on preflight, nevermind
MW supports add on switch.
The problem I have with this is that "controls" is confusing here.
The name createViews() describes pretty well what is going on
underneath.

    Karl

From: ayeung <notifications@github.com<mailto:notifications@github.com>>
Reply-To: appcelerator/titanium_mobile <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Sat, 29 Sep 2012 12:32:26 -0700
To: appcelerator/titanium_mobile <titanium_mobile@noreply.github.com<mailto:titanium_mobile@noreply.github.com>>
Cc: Karl Rowley <krowley@appcelerator.com<mailto:krowley@appcelerator.com>>
Subject: Re: [titanium_mobile] [TIMOB-10712] Android: label 'opacity' property won't change for TableVi... (#3043)


In android/modules/ui/src/java/ti/modules/titanium/ui/widget/tableview/TiTableViewRowProxyItem.java:

> @@ -97,7 +98,11 @@ protected TiViewProxy addViewToOldRow(int index, TiUIView titleView, TiViewProxy
>               return label;
>       }
>
> -     protected void refreshControls()
> +     //
> +     // Create views for measurement or for layout.  For each view, apply the
> +     // properties from the appropriate proxy to the view.
> +     //
> +     protected void createViews()


createView seems a little generic for the context of the table view, maybe we can use something like createControls() so it aligns with terminology in this class?

—
Reply to this email directly or view it on GitHub<https://github.com/appcelerator/titanium_mobile/pull/3043/files#r1723800>.
On Android, one or more tab groups can exist and a tab group can be closed.
On Android, bottom, top, left, right, height and width only work for lightweight windows.
should be '"it's" not "its"
Remember that alert dialogs have a show() method that when called, then does it access those properties. As long as they are set before calling show (either through the constructor or directly), everything will work.
Oh ok, yeah our equivalent is Evented. Why is it called Proxy? That's not very intuitive.
Spurious old comment...the animation is cancelled on line 76
add comment regarding rationale for change and how it related to emulator only please
should we have a test for this case?
+1 for getOrCreateProfile
Are you sure? John Gruber is site?
No it doesn't I tried to do it without the .html but it wouldn't pick it up.
> However, Chunks can function perfectly well on a slice-able infinite range

I just added a requirement for *input range*. *Infinite range* as also *input range*, see [std.range.isInfinite](http://dlang.org/phobos/std_range.html#isInfinite).
Part of the question is what actually constitutes the style. Depending on how strictly defined it is, differing indenting like this would or wouldn't be considered part of the same style. And the more complicated the style is, the harder it is to determine whether something follows it unless it exactly matches how you think. And to be fair, glancing over the file, there are very few cases where a function call gets split over multiple lines like this, so determining what the style of the style of the file is in that regard is not necessarily easy, except for the fact that this particular edit is explicitly altering a line which followed one style to follow another.

Regardless, if you want to say that the style for std.algorithm is that all indentations are multiples of 4 even with expressions which are on multiple lines, then fine.

I tend to react very negatively to any insistence of people changing the style of their code precisely because I hate being told how to format my code (it's one of the worst parts of having to deal with other programmers IMHO).  So, in almost all cases, I'm generally going to fight against any attempt to require a particular coding style. And while I agree that having an inconsistent style in a file is undesirable, I'd honestly rather be able to just always format the code that I write however I want to even if that means that the resulting overall style is inconsistent. But I do try and keep what I write consistent with the rest of the file, because it matters to other people like you.
This would be replaced by `forward`.
There isn't any space before an ? or ! in english.
Needs a comma at the end of this line...
no comma needed
jump between singular and plural (models/model)
The > 0 is unnecessary.
Is there a reason to be using eq instead of ==?
`200` even if there's an error?
if ($this->smtp_crypto == "tls")   >>>  if ($this->smtp_crypto == 'tls')
`hasValue`?
I love Depency Injetction
also why would anyone have a reference to ``JMS\SerializerBundle\DependencyInjection`` ?
Unbedingt ein $aktive Flag hinzufügen...
Wir werden eher keine Pricings löschen können, solange noch Aufträge dazu existieren....
Und trotzdem möchtest du evtl ein PricingModel aus der Auswahl entfernen....

Ein weiteres Mal frage ich mich, ob man PricingModele anpassen darf (weil sie vom Auftrag referenziert werden)
Oder ob es nicht richtiger wäre, ein neues Model zu erstellen und das alte zu deaktivieren....

Was wieder dazu führen würde, dass die meisten Felder hier (vielleicht Name und Desc nicht)  readonly wären....
und somit keine setter brauchen.... dafür im Konstruktor gesetzt werden sollten.
If we're only using the timeout inside this function, no need to set it to a variable
Fair call, I'll fix.
Trial and error, really. There are asserts in the `.d` file that help get the actual value, which is then put here (at least that's how I did it). This approach was put in place by @dawgfoto so he can probably better comment on this than I can.
What's with this change of spacing? Looks like it makes things misaligned.
added internalSolrParams, which contains all parameters passed through the url. (minus 'page' and 'q')
Why was this ever a link.  You should remove that and the message.property for it too.  Eventually it will probably be for settings.
This description's taken as is from the JDBC chapter for the <i>reply-timeout</i> attribute. This i mean here is, when a response is received from the underlying data source after querying it, the adapter will not wait indefinitely to publish the message to the reply channel if the send blocks, but will wait for the given number of milli seconds before throwing an exception. This attribute is relevant to say <i>QueueChannel</i> and is ignored in case of <i>DirectChannels</i>
Is it kosher to use setStringProperty() instead of setJmsCorrelationId() when the header is JmsCorrelationId? Maybe it works with AMQ, but I don't know if some brokers treat well-known headers and arbitrary properties differently. (I seem to recall SonicMQ does, but I could be wrong).

Previous implementation did this...

			if (this.correlationKey.equals("JMSCorrelationID")) {
				jmsRequest.setJMSCorrelationID(correlationId);
				messageSelector = "JMSCorrelationID = '" + correlationId + "'";
			}
			else {
				jmsRequest.setStringProperty(this.correlationKey, correlationId);
				messageSelector = this.correlationKey + " = '" + correlationId + "'";
			}
maybe wrapping into `<code/>`?
Moving my comment from Commit to Diff

I think we can simplify it by modifying forceComplete() method which already holds the lock.
Basically modify its inner 'try' to look like this:
```
MessageGroup messageGroupNow = this.getMessageStore().getMessageGroup(group.getGroupId());
long lastModifiedNow = messageGroupNow.getLastModified();
if (group.getLastModified() == lastModifiedNow) {
	if (releaseStrategy.canRelease(group)) {
		this.completeGroup(correlationKey, group);
	}
	else {
		this.expireGroup(correlationKey, group);
	}
}
```
The explanation makes sense. As long as the end user only configures CCF (or any other CF impl that is *not* part of our lib), that's fine.
This doesn't seem to be used in the file.
I think this is platform specific problem (probably something around case in-sensitive on windows). Without this I can't run ceylondoc test in eclipse. But I agree, that this special `if`  are ugly. Do you have some better solution?
デフォルト値と例外処理。
I'll refactor this into a separate pull request, but it basically just makes the code cleaner.  Should have used separate branches for both of these requests anyway.
«то для разнообразия» можно убрать.
This should extends ``RuntimeException``
Agreed.  If it's possible, it would be much better to use a stream-type architecture where each row (or whatever unit is possible) is converted to CZML as it is read.  That way the entire Shapefile doesn't need to be loaded into memory.
Interesting idea... There would need to be a pair of functions - one to get the state from the request, the other to store it in the response, so the app would define something like:

    (defn get-state [request]
      (:state (:session request)))

    (defn put-state [response state]
      (assoc response :session (merge (response :session) {:state state})))

You could pass this pair of functions in the oauth-params map:

    ...
    :state-management [get-state put-state]
    ...

Does this look about right?
The Original Code is about:telemetry integration code for Thunderbird
This is funny - we're using the old syntax to specify the new. Reminds me of rspec's early days when rspec was tested w/ test/unit.

Do you think `expect(expect(7).target).to eq(7)` would be too much of a leap at this point?
Lo cambié al paquete agenda porque recurso no la implementa.
Style conventions for node & the node SDK:
  Indent 2 spaces, not four
  Use single quotes, not double quotes, unless necessary

For the other new files I don't have an issue, but this is an edit to an existing file and it should fit in with the style conventions already in that file.
Automated and manual tests added in a separate pull request.

@erikj54 - you should drop your changes to this file
Why this change? Also, some compilers do not handle nan, so using the builtin == does not always work.
please don't remove the ?src=,  this is needed when users first uses a URL and next uses the fileuploader, if a src= arguments has any value then always will use the manual src argument avoiding the file upload.  setting "src=" ensure that fileuploader will be used instead.  
(Do you remove the manual src option ?)
Should also accept undefined and empty string to exclude. Basically anything that evaluates to false in javascript (null, undefined, false, "", 0, etc) should work.
Will not having this currently cause problems on emulator with this fix?
Maybe in the next pull request ;-)
The parent type is able to use a string when you set the ``widget`` option to ``single_text``. Simply do this in the default options (I do this in my own project actually)
Должно исправить #291. Если и это не исправит, то хз что делать.
I would probably throw the configParser.parse call into a notToThrow() expect call to ensure it doesn't throw an error instead of a warning.

    expect(function () {
        configParser.parse(...);
    }).notToThrow();
permission(self) for permission in self.permissions
Why are you defining `fs` again when it's already defined at the top?
Shouldn't these be using the published remote functions?
In my experience it does not do a terminate. It does kill the app, but it appears to do so by the navigator while terminate seems to more quickly and reliably kill the app.
Can you please test with jQuery 1.4?
Why does this listener get removed?