search

VEuPathDB / microbiomeComputations

1 stars 0 forks source link

Refactor correlations methods to support multiple correlation apps #62

Closed d-callan closed 9 months ago

d-callan commented 9 months ago

resolves https://github.com/VEuPathDB/microbiomeComputations/issues/61

this will represent an api change, in addition to some general refactoring. there are some new helpers for correlation methods, some new correlation methods, some changes to the get* methods of AbundanceData and a new method for SampleMetadata, among other things.

should be fully documented and tested now as well.

Should double check all needed validation for correlation w two data.tables is there

d-callan commented 9 months ago

this is ready for another look. the github action workflow seems to be failing for something unrelated. i promise the tests are passing locally :)

asizemore commented 9 months ago

No further comments or suggestions. LGTM!