Skip to content
This repository has been archived by the owner on Mar 6, 2023. It is now read-only.

make dask and xarray "optional dependencies" #11

Merged
merged 2 commits into from
Sep 17, 2020
Merged

make dask and xarray "optional dependencies" #11

merged 2 commits into from
Sep 17, 2020

Conversation

soxofaan
Copy link
Member

Addresses issue #5 :

  • eliminate explicitly importing dask/xarray where possible
  • make dask and xarray optional dependencies
  • update README

@soxofaan
Issue Open-EO#5: eval_datatype: replace isinstance() checks with `t…
c0f0a5a 
…ype` inspection

This avoids having to import numpy, xarray, dask
and reduces overhead of if-elif chain and mulitple isinstance checks
@soxofaan
Issue Open-EO#5: move dask and xarray to optional dependencies ("extr…
b5a4134 
…as_require")
@lforesta
Copy link
Contributor

Thanks for the PR

@lforesta lforesta merged commit 58a547e into Open-EO:master Sep 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@soxofaan @lforesta