Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Import austraits #177

Open
wants to merge 12 commits into
base: develop
Choose a base branch
from
Open

Import austraits #177

wants to merge 12 commits into from

Conversation

ehwenk
Copy link
Collaborator

@ehwenk ehwenk commented Nov 19, 2024

Removal of functions from traits.build to accompany release of austraits 3.0

  • util_df_to_list and util_list_to_df are now sourced from austraits (as convert_df_to_list and convert_list_to_df)
  • build_combine is now replaced by austraits::bind_databases
  • plot_trait_distribution_beeswarm, trait_pivot_longer and trait_pivot_wider had been in both packages and have now been removed from traits.build
  • the function flatten_database is a replacement for database_create_combined_table, which was still on a branch
  • at the bottom of utils.R, the functions that have been moved and renamed are also assigned their old names, to ensure that the names included in our Eco Informatics publication are still "valid" and that people using old remake/build.R files are not inconvenienced by these changes

update DESCRIPTION file to import austraits functions from development branch - to begin testing removing the functions from traits.build and importing them
Remove traits.build functions `util_df_to_list`, `util_list_to_df1` and `util_list_to_df2` replacing them with the austraits equivalents - beginning with `convert` instead of `util`
Remove `build_combine` as a function in traits.build, instead using its replacement in austraits, `bind_databases`
install austraits from master
@ehwenk ehwenk requested a review from dfalster November 19, 2024 05:52
R/utils.R Outdated Show resolved Hide resolved
Copy link
Member

@dfalster dfalster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! See query regarding deprecated functions.

R/utils.R Outdated Show resolved Hide resolved
tests/testthat/test-setup.R Outdated Show resolved Hide resolved
@dfalster
Copy link
Member

Good start @ehwenk , good additions @fontikar . Just need to fix tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants