Closed drwelby closed 4 years ago
Merging #767 into master will decrease coverage by
0.04%
. The diff coverage is100%
.
@@ Coverage Diff @@
## master #767 +/- ##
=========================================
- Coverage 81.44% 81.4% -0.05%
=========================================
Files 91 91
Lines 6404 6404
=========================================
- Hits 5216 5213 -3
- Misses 1188 1191 +3
Impacted Files | Coverage Δ | |
---|---|---|
tests/unit/test_workflow.py | 100% <100%> (ø) |
:arrow_up: |
tests/unit/test_meta.py | 100% <100%> (ø) |
:arrow_up: |
tests/unit/test_simpleworkflow.py | 98.9% <100%> (ø) |
:arrow_up: |
tests/unit/test_image_fetch.py | 100% <100%> (ø) |
:arrow_up: |
tests/unit/test_interface.py | 95.23% <100%> (-4.77%) |
:arrow_down: |
gbdxtools/auth.py | 87.75% <0%> (-4.09%) |
:arrow_down: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update d846fec...68d7ae1. Read the comment docs.
0.16.7 was supposed to be the last '16 release but in fixing up the Conda packaging I made enough changes to warrant a new release.
conda skeleton
and then edited downconda-build-config.yaml
since we build anoarch: python
packageenvironment.yaml
since it was comically out of dateIn the dependencies:
pip install dask
andconda install dask
are not the same thing!Conda's
dask
is all of Dask and all of its dependencies, even Pandas! What we want is actuallydask-core
, which is just all of Dask and equivalent topip install dask
.Meanwhile,
pip install dask
installs all of Dask, and we only need the array part, so we can just requiredask[array]
to lighten it up.Stole @cderwin's
makefile
from PyVeda to make OSX installs a little easier.make osx
sets the OpenSSL library options beforepip install
ingFixed up the tests so they all pass and got rid of the AssertEqual warnings.