Plures/uarray

Quansight

@quansightai_twitter

[Unknown, Quansight]
Pull request opened by joaosferreira
Quansight-Labs/unumpy#66
Picking up where Quansight-Labs/unumpy#61 left off by extending class overriding to other backends, more specifically, Dask and Sparse.
Reviewers: @hameerabbasi

Quansight

@quansightai_twitter

[Unknown, Quansight]
*1 new commit pushed to master*
Quansight-Labs/unumpy@e9d5d8d - Add overriden_class to other backends (#66)

[Unknown, Quansight]
Pull request merged by hameerabbasi
Quansight-Labs/unumpy#66

[Unknown, Quansight]
Pull request closed by hameerabbasi
Quansight-Labs/unumpy#61

Quansight

@quansightai_twitter

[Unknown, Quansight]
Pull request opened by joaosferreira
Quansight-Labs/unumpy#67
This PR intends to expand unumpy's coverage of indexing routines. The multimethods added so far are the following:

Generating index arrays

• indices
• ix_
• ravel_multi_index
• unravel_index
• diag_indices
• diag_indices_from
• mask_indices
• tril_indices
• tril_indices_from
• triu_indices
• triu_indices_from

Some observations:

The failing tests are in the Sparse backend. Because ones creates a COO with a fill_value of 1.0, the call to the next method fails since a zero fill_value is necessary.
I'm working on default implementations for ravel_multi_index and unravel_index. I'll push them as soon as I have a working prototype but tips on how to implement these are welcomed.
There are three classes that should be added as well since they are part of these routines, they are c_, r_ and s_.

*Note:* I'll be adding more multimethods that target these routines later.

joaosferreira

@joaosferreira

Hey everyone, hope you are doing well. Here is my last blog post. Have a nice week!

Also, not sure if the mentors can see it, but I completed my 2nd evaluation of the organization/mentors for GSoC. :)

joaosferreira

@joaosferreira

@hameerabbasi Can we move this week's meeting to tomorrow? Same time is fine by me.

Hameer Abbasi

@hameerabbasi

@joaosferreira I'm on leave tomorrow. If you're blocked, you can ask on Gitter or on the PR, or we'll have a meeting on Monday. Whichever you prefer.

joaosferreira

@joaosferreira

I'll use those channels.

Hameer Abbasi

@hameerabbasi

Great. You're good at writing up the issues either way!

Sudeep Sidhu

@sidhu1012

Hi guys

Hameer Abbasi

@hameerabbasi

Hello!

Sudeep Sidhu

@sidhu1012

Hi Hameer

Sudeep Sidhu

@sidhu1012

I want to get started with this organization , from where shall I begin

Hameer Abbasi

@hameerabbasi

@sidhu1012 Here's a blogpost that explains the basics. https://labs.quansight.org/blog/2019/07/uarray-update-api-changes-overhead-and-comparison-to-__array_function__/

Sudeep Sidhu

@sidhu1012

Okay I'll give it a read ,thanks @hameerabbasi

Quansight

@quansightai_twitter

[Unknown, Quansight]
Pull request ready for review by joaosferreira
Quansight-Labs/unumpy#67

joaosferreira

@joaosferreira

What's the best way to check if an array is of a given dtype in a default implementation, in this case a boolean array?

Quansight

@quansightai_twitter

[Peter Bell, Quansight] a.dtype.kind == 'b'?

joaosferreira

@joaosferreira

Hey everyone! Here is my latest blog post. Have a nice week! :smile:

Quansight

@quansightai_twitter

[Unknown, Quansight]
Pull request opened by hameerabbasi
Quansight-Labs/unumpy#68

Quansight

@quansightai_twitter

[Unknown, Quansight]
*1 new commit pushed to master*
Quansight-Labs/unumpy@8d1ca5c - Fix up backends and metaclass so subclasses work correctly. (#68)

[Unknown, Quansight]
Pull request merged by hameerabbasi
Quansight-Labs/unumpy#68

Quansight

@quansightai_twitter

[Unknown, Quansight]
*1 new commit pushed to master*
Quansight-Labs/udiff@f6f0407 - control the base of the differentiation (#30)

[Unknown, Quansight]
Pull request merged by hameerabbasi
Quansight-Labs/udiff#30

Quansight

@quansightai_twitter

[Unknown, Quansight]
Pull request opened by joaosferreira
Quansight-Labs/unumpy#69
As first discussed in #67 (review), this PR intends to extend class overriding to CuPy.
Reviewers: @hameerabbasi, @peterbell10

Quansight

@quansightai_twitter

[Unknown, Quansight]
*1 new commit pushed to master*
Quansight-Labs/unumpy@cc124c2 - Add overridden_class to CuPy backend (#69)

[Unknown, Quansight]
Pull request merged by hameerabbasi
Quansight-Labs/unumpy#69

Quansight

@quansightai_twitter

[Unknown, Quansight]
Pull request opened by sangyx
Quansight-Labs/udiff#31
I have added the diff function of np.stack.

When we extend the code to higher-order derivatives, the situation becomes a bit more complicated. There are some places to improve:

• In some derivative functions, the program may enter an infinite loop. For example, np.sum is used when registering the gradient of np.sum. I plan to add depth attributes to DiffArray or to() to control the order of gradients.
• The function registered in nograd_functions will register the gradient when it is called in the form of ufunc.__call__.
• Consider a cleaner way to control whether the calculation should derive.

Do you have any idea about the above questions?

Quansight

@quansightai_twitter

[Unknown, Quansight]
Pull request opened by joaosferreira
Quansight-Labs/unumpy#70
This pull request adds multimethods for statistical functions. The multimethods added are the following:

Order statistics

• percentile
• nanpercentile
• quantile
• nanquantile

Averages and variances

• median
• average
• mean
• nanmedian
• nanmean
• nanstd
• nanvar

Correlating

• corrcoef
• correlate
• cov

Histograms

• histogram
• histogram2d
• histogramdd
• bincount
• histogram_bin_edges
• digitize

A few things to discuss:

Why is dtype not being dispatched in std and var? Other multimethods that use _reduce_argreplacer also don't dispatch dtype.
The previously added min and max seem to be equal to amin and amax that this PR intends to add:

In [2]: onp.amin                                                                
Out[2]: <function numpy.amin(a, axis=None, out=None, keepdims=<no value>, initial=<no value>, where=<no value>)>

In [3]: onp.min                                                                 
Out[3]: <function numpy.amin(a, axis=None, out=None, keepdims=<no value>, initial=<no value>, where=<no value>)>

I'm looking into implementing some defaults as well. Multimethods like median that reduce array slices along an axis might be the easier ones for now. As I understand these require a for loop over the given axis, so in terms of complexity it would be O(n) where n is the length of that dimension.

joaosferreira

@joaosferreira

Hey everyone! Here is my latest blog post. Have a nice week! :)

Quansight

@quansightai_twitter

[Unknown, Quansight]
Pull request opened by peterbell10
Quansight-Labs/unumpy#71

Quansight

@quansightai_twitter

[Unknown, Quansight]
*2 new commits pushed to master*
Quansight-Labs/unumpy@76a279f - Misc fixes found by the cupy backend
Quansight-Labs/unumpy@9a42b49 - Merge pull request #71 from Quansight-Labs/cupy-fixes

[Unknown, Quansight]
Pull request merged by hameerabbasi
Quansight-Labs/unumpy#71

Quansight

@quansightai_twitter

[Unknown, Quansight]
Pull request opened by joaosferreira
Quansight-Labs/unumpy#72
This fixes some issues regarding the correct dispatching of methods and classes like the one mentioned in #67 (review). It also allows proper dispatching of methods in modules such as unumpy.linalg as demonstrated by the fixed tests.

Quansight

@quansightai_twitter

[Unknown, Quansight]
*3 new commits pushed to master*
Quansight-Labs/unumpy@a52e773 - Add _get_from_name_domain to __ua_function__
Quansight-Labs/unumpy@986b492 - Fix tests for Linear Algebra routines
Quansight-Labs/unumpy@8f1d962 - Merge pull request #72 from joaosferreira/fix-backends

[Unknown, Quansight]
Pull request merged by hameerabbasi
Quansight-Labs/unumpy#72

Quansight

@quansightai_twitter

[Unknown, Quansight]
*15 new commits pushed to master*
Quansight-Labs/unumpy@0a168c9 - Add multimethods for indexing routines
Quansight-Labs/unumpy@0981d47 - Add defaut implementation for ravel_multi_index
Quansight-Labs/unumpy@bda0ab9 - Add default implementation for unravel_index
Quansight-Labs/unumpy@a30f2d4 - Add order kwarg to ravel_multi_index's and unravel_index's defaults
Quansight-Labs/unumpy@810e580 - Add more multimethods for indexing routines
Quansight-Labs/unumpy@ea0e0fe - Add default implementation for indices
Quansight-Labs/unumpy@57cf4de - Add c_, r_ and s_
Quansight-Labs/unumpy@b1e875e - Add lib.Arrayterator
Quansight-Labs/unumpy@eaefc2d - Add default implementation for ix_
Quansight-Labs/unumpy@5c46c50 - Add overriding metaclass for Arrayterator
Quansight-Labs/unumpy@3413b30 - Add dispatching of choices to choose
Quansight-Labs/unumpy@e7aa4f9 - Add default implementation for putmask
Quansight-Labs/unumpy@3833cc6 - Add backend fixes in #71 and #72 by merging branch 'master'
Quansight-Labs/unumpy@95321f9 - Add CClass, RClass and IndexExpression
Quansight-Labs/unumpy@f5a1d62 - Merge pull request #67 from joaosferreira/indexing-routines

[Unknown, Quansight]
Pull request merged by hameerabbasi
Quansight-Labs/unumpy#67

Quansight

@quansightai_twitter

[Unknown, Quansight]
Pull request opened by peterbell10
Quansight-Labs/uarray#247
Should fix the segfaults in Quansight-Labs/udiff#31

Returning selected_backend.get() instead of .release() from determine_backend meant the refcount was decreased without dropping the reference. The net result is decreasing the refcount of the backend once for each call to determine_backend. Eventually it gets GC'd and the python interpreter accesses invalid memory causing the segfault.

The GC issues are unrelated but were spotted while debugging this. The context managers had tp_traverse defined but the GC will only use these if the type has the Py_TPFLAGS_HAVE_GC flag set.
Reviewers: @hameerabbasi

Quansight

@quansightai_twitter

[Unknown, Quansight]
Pull request opened by hameerabbasi
Quansight-Labs/uarray#248

Kartik Shukla

@kartiks-21

Hello I am Kartik,I would like to know more about the projects and would want to contribute to the same can any mentor help me out with the information?

Quansight

@quansightai_twitter

[Hameer Abbasi, Quansight] Hello, Kartik. Are you looking for GSoC? If so, the selections are already over and have been for some time.

If not, head on over to https://labs.quansight.org/blog/2019/07/uarray-update-api-changes-overhead-and-comparison-to-array_function/ and read the blog. Play around with it, explore, and come back to us with any specific questions you may have.

[Unknown, Quansight]
*2 new commits pushed to master*
Quansight-Labs/uarray@33c6558 - Remove MyPy linting.
Quansight-Labs/uarray@dc9b68f - Merge pull request #248 from hameerabbasi/remove-mypy

[Unknown, Quansight]
Pull request merged by hameerabbasi
Quansight-Labs/uarray#248

Kartik Shukla

@kartiks-21

Thank you sir

Quansight

@quansightai_twitter

[Unknown, Quansight]
Pull request merged by hameerabbasi
Quansight-Labs/uarray#247

[Unknown, Quansight]
*2 new commits pushed to master*
Quansight-Labs/uarray@60deba9 - Fix refcount and GC issues
Quansight-Labs/uarray@2e03d53 - Merge pull request #247 from Quansight-Labs/refcount-gc-errors

Quansight

@quansightai_twitter

[Unknown, Quansight]
Pull request ready for review by joaosferreira
Quansight-Labs/unumpy#70

Where communities thrive

People

Repo info

Activity