Submit irispy-lmsal as an affiliated package

[nabobalis]

Package Details

• Package name: irispy-lmsal
• Maintainer(s): Nabil Freij (@nabobalis)
• Repository: https://github.com/LM-SAL/irispy-lmsal
• Documentation link: https://irispy-lmsal.readthedocs.io/en/stable/
• PyPI link (if applicable): https://pypi.org/project/irispy-lmsal/
• Continuous integration link(s): https://github.com/LM-SAL/irispy-lmsal/actions
• Link to Code of Conduct or page describing adopted Code of Conduct: errrrrrrrrrr

Description of Package

A Python package that provides the tools to read in and analyze data from the IRIS solar-observing satellite.

Package Review

Editor Submission Checklist

• Is the submission appropriate (compatible with the SunPy CoC)?
• Does the project have an appropriate license?
• Is the project in an online version control system?
• Does the project provide a Python interface?
• Is the project on PyPI?
• Is the project useful to the solar physics community?
• Version that was reviewed: v0.2.0

Instructions to Reviewer

Please copy the following and select the ranking for each criteria, the full review criteria can be found here:

* Functionality           : General Package / Specialized Package / Not Relevant
* Integration             : Well integrated / Partially Integrated / Minimal Integration
* Documentation           : Extensive / Some / Little
* Testing                 : Excellent / Good / Needs Work
* Duplication             : None / Some / Major
* Community               : Excellent / Good / Needs Work
* Development Status      : Stable / Subject to Change / Low Activity / Needs Work

[Cadair]

@nabobalis Thanks for the submission. The package looks good and it's great to see more officially supported instrument packages emerging. My review is below, the only reason I have marked Community down from excellent is that I can't see a code of conduct in the repo and/or the documentation which is a requirement, if you can add that I will happily bump the review and we can accept irispy-lmsal.

• Functionality :
• Integration :
• Documentation :
• Testing :
• Duplication :
• Community :
• Development Status :

[nabobalis]

I have opened/merged LM-SAL/irispy-lmsal#46 that adds a COC to the repo. It is based on the sunpy one with changes. Unsure if this is the best COC to use for non-sunpy project projects.

[Cadair]

With that change, here is the final review:

• Functionality :
• Integration :
• Documentation :
• Testing :
• Duplication :
• Community :
• Development Status :

@nabobalis can you push a release after that CoC change and then you want to make the PR to the page with this review?

[nabobalis]

I tagged a release: https://github.com/LM-SAL/irispy-lmsal/releases/tag/v0.2.1

The CI wheels are turning.

[nabobalis]

Anything else to be done here?

[Cadair]

I need to PR to the site with this and also #431