labofsahil/github-event-dataset-2012 huggingface dataset:
This should be possible just by performing an xcodebuild, and then booting `ios-sim` directly. If you want to make it part of this PR drop by and talk to me about it. The new CLI will probably take this approach.
Yes, mobile web supports up and down, as well as left and right
Yep, I think this was a hold over from a previous design with more logic.  Oversight on my part and removed.
Type is only Number since it is defined as such on the proxy
Missing since tag
Missing update to UI.yml Titanium.UI.... does not link to anything. `docgen` will fail.
Sure a later PR is good.

On Mon, Aug 27, 2012 at 10:50 PM, Opie Cyrus <notifications@github.com>wrote:

> In anvil/driver/localMode.js:
>
> > @@ -20,6 +20,8 @@ module.exports = new function() {
> >  	var readlineInterface = readline.createInterface(process.stdin, process.stdout);
> >
> >  	this.start = function() {
> > +		util.setCurrentTiSdk();
>
> The idea is to test against the latest (or current) SDK version. The
> normal use case in mind was checking out a branch, working on changes,
> running anvil and them pushing changes back into git (the drillbit model).
> I can see value perhaps in specifying a different version but should that
> be included as part of the hub push or a new PR for enhancements (which
> there will be)?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/appcelerator/titanium_mobile/pull/2811/files#r1468449>.
>
>
JavaScript object.
Why is this development as opposed to any other module type? What happens if the module is production-only? I find nowhere else where this argument is hardwired like this.
I wonder if this is going to stop working at some point because they don't like treating strings as a list of characters (because of all the unicode edge cases). Maybe you should use .endsWith('/').
export_tags and exports are also unused.
Don't remove `Scoped_` prefixes. They need to avoid conflicts with `T`'s members.
I don't see why it matters much whether it's a mixin or a template like you suggest, but I can change it. However, I would think that it would better for the types to match for comparison, since that's generally the most efficient.
I'd suggest using `filter!"true"([1, 2, 3, 4])` rather than creating an `SList`. I'm surprised that std.algorithm even imports std.container (ideally, it wouldn't). Regardless, it's less overhead to use `filter`.

Also, please test the case where `upTo` is _larger_ then the `length`.
`if (isNarrowString!S)`
well, I was thinking we'd have the test use templates once they're in :)
hmmm. I think this should be set up in the run.sh script. Even if html5_utils_test is not run, we still need to ensure data/output exists
you should use ${project.artifactId} here.
SWFDomainWrapperになりました
Why aren't you just re-executing? Does that not work (because of entity reuse?)?
Just wondering: why did this final get dropped?
Thats a discussion for a different place ;). Though you know my opinion on this matter and that hasn't changed just because I removed this line ;). This change is purely driven by the fact that I expect 5.2 to be dropped from travis and that will break the test suite so better be ahead of that.
I don't think you want this $app->populate thing in there.  Makes a lot of assumptions about the app structure and will be heavier than most apps, I think.
if record = @sic_lookup.find { |lookup| lookup['NAICS 2002'].to_i.to_s == naics_code }
      record['SIC'].to_i.to_s
    end
typeo class name is `BrokerProperties`
Can't this end up with NPE? 

`clazz = null`

 when 

`clazz.getSuperClass() == null` in previous  recusion iteration?
Good solution. I like the name change to the simpler 'responsive' but definitely want to maintain backward compatibility.

Any ideas on how to alert/warn that the name has changed? Can we do that during compilation or something?
Instead of waiting for the scroll to stop, what if we used the method in the posts list where it detects if there are less than 3 rows remaining and that triggers the load? It loads even faster that way:

https://github.com/wordpress-mobile/WordPress-iOS/blob/notifications/Classes/PostsViewController.m#L185
not sure about this. The fact that the locking is eager  (BTW that has beed deprecated and can be identified through LockingMode.PESSIMISTIC) changes the way the system functionally. OTOH the fact that we use 2PC or TOB is an implementation detail of the optimistic transactions. IMO what we need to do here is log a message as trace stating that the TOB is ignored for pessimistic transactions and do just that: ignore the config.
you should test the output of the filter. Simply testing that it is not empty is not enough IMO
range filter JS. Adds the toggle.
This should not loop. it should just do the current encounter
The problem is the parser rejected ""

I don't think this particular issue is related to their non-standard server - just that no permissions for rename.
Done ! :)
I'm beginning to wonder if it's worth the effort for a simple bootstrap module. What's the use case where the device name (eth0 etc) isn't sufficient? The only one I can think of is where DHCP and DNS aren't on the same box - which must be an edge case, surely?
string[] = A list/array of only string elements, see http://www.phpdoc.org/docs/latest/for-users/types.html
I'd like these files to be included as they are used by the ModelMetadataProvider. Any objections?
``|prev_major_release|``? whatever the name, I think the goal should be to have as **few** hard-coded elements in these pages that need updating for every release. Anything in conf.py can be made overwriteable
I don't know enough about digital goods to know: does this deserve its own class? Or should it simply be added directly to PaypalExpressGateway?
Ah, of course, that's the way to go - thank you :)

I don't think you need a path like that if you want to get a sibling. In that case you could just do "$myBlock->getParent()->getPath() . '/block-1'". Or did I miss your point?
This is the same name as the form type for a one to one message. So users won't be able to pick one or the other. Maybe change to: `fos_multipersonmessage_new_thread`
Doesn't this remove the actions from ever being on top? Why does Grappelli need this removed?
originality ftw
So maybe its just me, but I've been having nothing but issues with this lock directory. Setting it to the home directory seems to have eliminated most of my woes.
Only reason I point it out is that these appear in the docs. Then again its kind of funny and harmless.
Fixed in 09b3b9b
I would indent this as:

```
url(
    r"^(?P<resource_name>%s)/(?P<pk>\w[\w-]*)/(?P<subresource_name>%s)%s$" % (self._meta.resource_name, name, utils.trailing_slash()),
    self.wrap_view('dispatch_subresource'),
    {'request_type': 'list'},
    name='api_dispatch_subresource_list',
),
```

Or:

```
url(
    r"^(?P<resource_name>%s)/(?P<pk>\w[\w-]*)/(?P<subresource_name>%s)%s$" % (
        self._meta.resource_name,
        name,
        utils.trailing_slash(),
    ),
    self.wrap_view('dispatch_subresource'),
    {'request_type': 'list'},
    name='api_dispatch_subresource_list',
),
```
Comment fixed and unicode cast removed.
Is it safe to always expect a RefinedManifest? What about cases like `class Foo extends Record`? I think we're on the safer side if `registerType` just works on any Manifest and uses pattern matching to identity RefinedManifests. That way we also get rid of the casting here.
On Thu, Aug 23, 2012 at 08:20:20AM -0700, Ralf Gommers wrote:
>    There are currently no examples/ dirs AFAICT, I guess you meant pyplots/?

It does need to be in an 'examples' directory. This part of the
scipy-lectures hasn't been ported to the new 'examples' style.

>    I copied what was already used in the scipy-intro section in several
>    places. Propose to fix this all at once in a separate PR, is that OK?

Separate PR is definitely the way to go.

G
I tried on my local machine and it worked.
Thanks, already got it pushed!

On Thu, Feb 16, 2012 at 11:51 AM, Ryan Bigg
<reply@reply.github.com>
wrote:
>> @@ -252,7 +252,7 @@ def order_opts(order, payment_method, stage)
>>          credits_total = credits.map {|i| i[:amount] * i[:quantity] }.sum
>>        end
>>
>> -      opts = { #:return_url        => request.protocol + request.host_with_port + "/spree/orders/#{order.number}/checkout/paypal_confirm?payment_method_id=#{payment_method}",
>> +      opts = { #:return_url        => request.protocol + request.host_with_port + "/orders/#{order.number}/checkout/paypal_confirm?payment_method_id=#{payment_method}",
>
> You can update it by pushing to your fix_namespace branch.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/spree/spree_paypal_express/pull/37/files#r458585
Is this solved since you have merged the pull request? I work on Windows 7 x64 and I think this is some kind of a Python or WinXP problem though I can test it if it persists.
I love changing names for aesthetic reasons, but I was actually going to avoid it this time. My original method was "test_pendings" to match the same noun_verb as the others. Then I realized that we were counting the number of tests in each state 3 times. Twice we were counting by looping through all the tests and manually incrementing counters, once we were using these methods. The counters were local variables, some of which had the same name as these methods. So I changed the name of the methods to not conflict. Then I realized that we should really only have one way of counting to begin with. Then I realized that one of the three times we were counting tests we weren't actually doing anything with the results....
Can also be a url to a video resource.
For consistency with Java API - they call it QueueInfo.
Sounds like a pandora's box to worry about typos...
@rwmtse Before we change this to `fs`; Rosa was there a reason we used `fsext` here instead of `fs`?
I think one way of avoiding global variables would be to pass it in within userdata.  We could create a struct readInfo { FILE *file, int chunkSize} and set it as CURLOPT_READDATA.
I just define a variable for it because it's referenced more than once. The other constants are not referenced within the code.
OK so it can't be tested with args. C'est la vie
It looked like that was the only change
My suggestions:

1) remove the 'Waits for the startup script' part
2) keep the list of startup script steps
3) add some info on how a URL is printed that allows you to see the result on cloud storage and shuts down the instance when receiving input from user.

Thanks for helping with these changes! :)
The consistency of the Batcher is key.

If we don't do this, then BatchTask.complete could overwrite the value of all_tasks_loaded and then CheckComplete would always fail on the batcher.