Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Install Instructions Not Complete #25

Closed
dhensle opened this issue Aug 22, 2022 · 2 comments
Closed

Install Instructions Not Complete #25

dhensle opened this issue Aug 22, 2022 · 2 comments

Comments

@dhensle
Copy link

dhensle commented Aug 22, 2022

Following install instructions from the documentation will not pass the tests. First issue is that openmatrix was not included in the package list. After installing openmatrix, got another error that says "ValueError: unrecognized engine zarr must be one of: ['scipy', 'store']" in many places.

Instead of trying to figure out the package issues one-by-one, I just created a new environment using testing.yml followed by "pip install -e ." in the cloned directory. The tests then passed successfully.

The installation documentation should be updated to include the extra packages or should just point to an environment file (or both?).

@dhensle
Copy link
Author

dhensle commented Aug 22, 2022

Suggest adding the jupyter related package requirements to the environment yaml as well.

jpn-- added a commit to jpn--/sharrow that referenced this issue Aug 22, 2022
jpn-- added a commit that referenced this issue Aug 25, 2022
* do not drop dims when analog is missing

* fix shared memory for sparse arrays

* flows work with numba jit disabled

* fix error in looking up from empty sparse rows

* fix reverse in sparse

* allow None in attrs

* import .dataset by default

* limit to setuptools<64

editable mode is broken in 64, python/importlib_metadata#402

* fix install info in docs

addresses #25

* re-loop aster for mixed spacenames, fix np.clip

* faster dataset from dataframe

* update install reqs

* add 3.10 to test matrix
@jpn--
Copy link
Member

jpn-- commented Sep 17, 2022

This has been fixed.

@jpn-- jpn-- closed this as completed Sep 17, 2022
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

No branches or pull requests

2 participants