datalad / datalad-neuroimaging

DataLad extension for neuroimaging research
http://datalad.org
Other
17 stars 14 forks source link

Update docs and CI build configuration #118

Closed mslw closed 1 year ago

mslw commented 1 year ago

Switched macOS appveyor build from "macOS" to "macos-monterey" to avoid stalling on older versions (following https://github.com/datalad/datalad-extension-template/pull/50). Fixes #116

Added instruction to install this package before building docs to readthedocs config. The config is needed to specify Python version, and apparently now also needs to include dependency specification. Fixes #117

mslw commented 1 year ago

Ok, so for Appveyor it worked on MacOS, but failed on Ubuntu... with an error while installing git-annex (downloaded from debian snapshots via datalad installer)... that apparently dpkg failed to unpack... although nothing has changed since last month when it ran successfully.

Help?

codecov[bot] commented 1 year ago

Codecov Report

Base: 0.00% // Head: 89.61% // Increases project coverage by +89.61% :tada:

Coverage data is based on head (09022de) compared to base (d04807c). Patch has no changes to coverable lines.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #118 +/- ## =========================================== + Coverage 0 89.61% +89.61% =========================================== Files 0 24 +24 Lines 0 1175 +1175 =========================================== + Hits 0 1053 +1053 - Misses 0 122 +122 ``` | [Impacted Files](https://codecov.io/gh/datalad/datalad-neuroimaging/pull/118?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad) | Coverage Δ | | |---|---|---| | [datalad\_neuroimaging/bids2scidata.py](https://codecov.io/gh/datalad/datalad-neuroimaging/pull/118?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad#diff-ZGF0YWxhZF9uZXVyb2ltYWdpbmcvYmlkczJzY2lkYXRhLnB5) | `82.50% <0.00%> (ø)` | | | [datalad\_neuroimaging/tests/utils.py](https://codecov.io/gh/datalad/datalad-neuroimaging/pull/118?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad#diff-ZGF0YWxhZF9uZXVyb2ltYWdpbmcvdGVzdHMvdXRpbHMucHk=) | `91.66% <0.00%> (ø)` | | | [datalad\_neuroimaging/extractors/dicom.py](https://codecov.io/gh/datalad/datalad-neuroimaging/pull/118?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad#diff-ZGF0YWxhZF9uZXVyb2ltYWdpbmcvZXh0cmFjdG9ycy9kaWNvbS5weQ==) | `91.57% <0.00%> (ø)` | | | [datalad\_neuroimaging/extractors/\_\_init\_\_.py](https://codecov.io/gh/datalad/datalad-neuroimaging/pull/118?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad#diff-ZGF0YWxhZF9uZXVyb2ltYWdpbmcvZXh0cmFjdG9ycy9fX2luaXRfXy5weQ==) | `100.00% <0.00%> (ø)` | | | [datalad\_neuroimaging/extractors/nifti1.py](https://codecov.io/gh/datalad/datalad-neuroimaging/pull/118?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad#diff-ZGF0YWxhZF9uZXVyb2ltYWdpbmcvZXh0cmFjdG9ycy9uaWZ0aTEucHk=) | `89.65% <0.00%> (ø)` | | | [datalad\_neuroimaging/conftest.py](https://codecov.io/gh/datalad/datalad-neuroimaging/pull/118?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad#diff-ZGF0YWxhZF9uZXVyb2ltYWdpbmcvY29uZnRlc3QucHk=) | `100.00% <0.00%> (ø)` | | | [datalad\_neuroimaging/tests/test\_aggregation.py](https://codecov.io/gh/datalad/datalad-neuroimaging/pull/118?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad#diff-ZGF0YWxhZF9uZXVyb2ltYWdpbmcvdGVzdHMvdGVzdF9hZ2dyZWdhdGlvbi5weQ==) | `100.00% <0.00%> (ø)` | | | [datalad\_neuroimaging/extractors/tests/test\_nidm.py](https://codecov.io/gh/datalad/datalad-neuroimaging/pull/118?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad#diff-ZGF0YWxhZF9uZXVyb2ltYWdpbmcvZXh0cmFjdG9ycy90ZXN0cy90ZXN0X25pZG0ucHk=) | `100.00% <0.00%> (ø)` | | | [datalad\_neuroimaging/extractors/nidm.py](https://codecov.io/gh/datalad/datalad-neuroimaging/pull/118?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad#diff-ZGF0YWxhZF9uZXVyb2ltYWdpbmcvZXh0cmFjdG9ycy9uaWRtLnB5) | `100.00% <0.00%> (ø)` | | | [datalad\_neuroimaging/\_\_init\_\_.py](https://codecov.io/gh/datalad/datalad-neuroimaging/pull/118?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad#diff-ZGF0YWxhZF9uZXVyb2ltYWdpbmcvX19pbml0X18ucHk=) | `100.00% <0.00%> (ø)` | | | ... and [14 more](https://codecov.io/gh/datalad/datalad-neuroimaging/pull/118?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad) | | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=datalad)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

mslw commented 1 year ago

I restarted the Ubuntu Appveyor run, and it succeeded! I'll blame it on a random occurrence.

Regarding Read the Docs, it seems like we don't have PR builds enabled, so we'll find out on merge...