Closed Cadair closed 11 months ago
dkist
packageI reviewed the v1.0.0b15 of the dkist
package.
Functionality:
This package is very useful to anyone wanting to search for, download, and load data from the new Daniel K. Inouye Solar Telescope. DKIST is the largest solar telescope in the world, thus this package is very relevant to solar physics.
Integration:
The package makes use of the
sunpy
package primarily for data search via Fido
. The package also makes use of NDCube as its base data structure.
Documentation :
The documentation is written in restructured text, rendered with sphinx, and hosted on Read the Docs. The documentation includes a high-level guide to the concepts underpinning the software, and API documentation. I am only marking the documentation as "good" (as opposed to "extensive") because I did not see a tutorial (similar to the SunPy Tutorial) that showed a user how to get going quickly. The User Guide contains elements of this, but mixes reference material with conceptual material. It would be beneficial to provide better separation between these types of documentation (e.g. the diátaxis approach). I would also recommend providing more info in the README on GitHub.
Testing:
I am only marking this as "good" as opposed to "excellent" because I could not easily find what the test coverage actually is (though looking at the codebase I assume it is high). Adding a codecov badge to the README would bump this to "excellent". Code coverage information is now available on the README and is sufficiently high (96%).
Duplication:
This package does not duplicate any functionality already provided by the SunPy Ecosystem.
Community:
There is a CoC and there are multiple contributors to this repository. The documentation provides explicit instructions for getting help and contributing.
Development Status:
The package is available on PyPI. The repository has seen development activity in the last six months.
Given that the package has a "green" score in "Functionality" and no "red" scores, the dkist
package should be accepted as a SunPy Affiliated Package though you are of course still free to respond to the review!
Thanks for the review @wtbarnes I have added a codecov badge (and a bunch of other badges) to the readme. I completely agree on the documentation, there's still lots to do there.
Package Details
dkist
Description of Package
The DKIST Python tools provide a package (dkist) which aims to help you search, obtain and use DKIST data as part of your Python software. The
dkist
package provides DKIST specific functionality on top ofsunpy
,ndcube
,astropy
,gwcs
anddask
to search for, download, work with and visualise DKIST data.Package Review
Editor Submission Checklist
Instructions to Reviewer
Please copy the following and select the ranking for each criteria, the full review criteria can be found here: