Skip to content

Conversation

@jim22k
Copy link
Contributor

@jim22k jim22k commented Dec 6, 2022

Hopefully this doesn't duplicate work already done by Henry. This was a quick and dirty attempt to make dask-grblas work with the latest version of python-graphblas.

Not all tests pass, but most do.

More work is still needed to rename dask-grblas to dask-graphblas and bring the API up to the same state as python-graphblas, as that has been a fast-moving target over the past year.

  • import grblas is now import graphblas
  • python-graphblas has been refactored to clean up the main namespace

- import grblas is now import graphblas
- python-graphblas has been refactored to clean up the main namespace
@eriknw
Copy link
Collaborator

eriknw commented Dec 7, 2022

CC @ParticularMiner

@SultanOrazbayev
Copy link
Contributor

SultanOrazbayev commented Dec 8, 2022

Looking through the failed test, it seems that the main problem tests are aborted due to incompatibility between black/click versions. Hopefully, this PR should resolve it #22
(another option is to pin the click to an older version)

Testing locally, I do see a lot of tests failing.

@SultanOrazbayev
Copy link
Contributor

I started working on the tests, fixing some easy issues (at least seem to be), this PR should fix the tests in test/test_scalar.py jim22k#1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants