Open brycefrank opened 3 hours ago
Here is a preliminary list of things to consider
R
load.R
combine_taxa
, aggregate_taxa
, load_models
publication_processing.R
boilerplate.R
testthat
test-install.R
allometric
. After all, users install models via allometric
test-publication-processing.R
test-update-models.R
Large portions of
allometric
are dedicated to processing publications and, as we develop the database, handling conversion of model objects into JSON. This is making the "root" package quite expansive and muddying documentation for typical end users.Instead, we can move these pieces of
allometric
directly tomodels
, which is already an R package by definition. We need to ensure that moving publication processing, especiallymap_publications
is feasible first. I will start this work onmigrate_processing
branches for each of the repos.In general we should have the philosophy:
models
can depend onallometric
but not vice versa, because most users will not want/need to installmodels
directly.