ncss-tech/SoilTaxonomy

CRAN Release 0.1.0

brownag opened this issue · 5 comments

brownag commented

I want to submit this package for a first CRAN release ASAP.

  • Run test code coverage analysis and tackle tests for any low-hanging fruit that are untested

  • Review all existing tests for duplication and thoroughness

  • explainST need to look for formative element dictionary edge cases -- RE: #15, #5.

  • Submit to Winbuilder, fix anything

  • Get into CRAN first submission queue

dylanbeaudette commented

Thanks. Do we need to fully solve #15/#5 before a CRAN submission? I'd like to make sure that most cases are resolved, but a general solution is likely going to take some time / planning.

brownag commented

No, I don't think we need to "solve" them completely. But I personally have spent very little time with the explainST code aside from my verification that it runs on all subgroups.

I essentially want to look closer at the ones that I know are problematic, and figure out why. I am not sure about the extent of the problems at this point... my feeling is it is actually not that much work.

Sure refining definitions could go on forever, but getting known missing entries in the table and ensuring that all multi-word subgroups at least populate with "?" for missing entries shouldn't be too hard, right?

dylanbeaudette commented

No, I don't think we need to "solve" them completely. But I personally have spent very little time with the explainST code aside from my verification that it runs on all subgroups.

I essentially want to look closer at the ones that I know are problematic, and figure out why. I am not sure about the extent of the problems at this point... my feeling is it is actually not that much work.

Sure refining definitions could go on forever, but getting known missing entries in the table and ensuring that all multi-word subgroups at least populate with "?" for missing entries shouldn't be too hard, right?

Works for me. Maybe we can divide / conquer.

brownag commented

We are in pretty good shape now! With a couple recent additions to explainST tests, we are now at 95% coverage package-wide.

brownag commented

We are on CRAN as of 2021-02-25