planetarypy / TC

PlanetaryPy Project Technical Committee
https://planetarypy.org/
1 stars 2 forks source link

kalasiris application to PlanetaryPy #49

Closed rbeyer closed 3 years ago

AndrewAnnex commented 3 years ago

the 1st external reviewer declined to review kalasiris. Waiting to hear from the alternate.

AndrewAnnex commented 3 years ago

*updated with anonymous reviewer comments 05/21/2021* *updated testing score 05/22/2021* This package has been reviewed for inclusion in the PlanetaryPy affiliated package ecosystem by a member of the PlanetaryPy community as well as myself, and I have synthesized the results of the review here.

You can find out more about our review criteria in Reviewing affiliated packages. For each of the review categories below we have listed the score and have included some comments when the score is not green.

Functionality/Scope General package

Anonymous reviewer stated that kalasiris did not provided enough added value over pysis or bash scripting for them personally.

Review coordinator notes that review guidelines are broadly defined, and notes the potential of added-value capabilities.
Integration with PlanetaryPy ecosystem Good

Anonymous reviewer pointed out the existence of pysis causing kalasiris to potentially be in conflict with the objective of ecointegration. Anonymous reviewer goes on to note the lack of an established user base for kalasiris relative to pysis. Anonymous reviewer however also noted the lack of activity on pysis, and that kalasiris stands a better chance of being developed further, justifying approval to the reviewer.

Review coordinator notes that pysis is not currently affiliated with planetarypy, so kalasiris meets the definition of establishing base functionality in the planetarypy ecosystem specifically.
Documentation Good
Review coordinator: I think more examples in the documentation outside of the quickstart could demonstrate the additional benefits/features of kalasiris such as `k_funcs` more visibly to new users.
Testing Good
No further comments
Development status Good
No further comments
Python version compatibility 3.6
No further comments

Summary/Decision: This package meets the review criteria for affiliated packages, so we are happy to confirm that we'll be listing your package as an affiliated package! Keep up the good work, and we encourage you to improve on the areas above that weren't "green" yet.

If you have any follow-up questions or disagree with any of the comments above, leave a comment and we can discuss it here. At any point in future you can request a re-review of the package if you believe any of the scores should be updated - contact the coordination committee, and we’ll do a new review.

rbeyer commented 3 years ago

I have added Codecov to the project's GitHub workflow, and test coverage is now being updated a reported with each push.

AndrewAnnex commented 3 years ago

testing review has been updated to reflect changes

AndrewAnnex commented 3 years ago

Also, anonymous reviewer supplied general comments that have been amended to the review