Skip to content

Synonyms for accepted names - #270

Merged
ehwenk merged 11 commits into
masterfrom
Synonyms-for-accepted-names
Mar 24, 2026
Merged

Synonyms for accepted names#270
ehwenk merged 11 commits into
masterfrom
Synonyms-for-accepted-names

Conversation

@ehwenk

@ehwenk ehwenk commented Mar 23, 2026

Copy link
Copy Markdown
Collaborator

Addition of a new function that indicates synonyms for a currently accepted name, outputting results in either a long format (each synonym as its own row) or as a collapsed form, with all synonyms for a currently accepted name collapsed into a single cell.

Synonyms are sorted by "type" in the same priority order specified in align_taxa, the synonym -> accepted updating function.

This function should be very useful if you have the current name of a taxon and want to work backwards and search the literature for references to the same taxon concept.

ehwenk added 3 commits January 7, 2026 11:49
Per user request, a function that summarises all past names by which a currently accepted taxon has been referred to.

While other APCalign functions are excellent for updating names, they aren't designed to take a currently accepted name and work backwards - for instance, if you want to know what names to search up in the literature to find information about a species that has changed names.

This function is in the testing stage - the exact output format and parameters have not been finalised.
- added tests
- added documentation
- added parameter that lets you collapse by accepted species or return one row per synonym
@ehwenk
ehwenk requested review from dfalster and wcornwell and removed request for dfalster March 23, 2026 05:05
wcornwell and others added 4 commits March 23, 2026 16:30
- Add missing @PARAM accepted_names documentation
- Add description for @PARAM resources
- Remove broken no-arg example (accepted_names has no default)
- Add name_with_status, synonyms, taxon_name to globalVariables()

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
dfalster
dfalster previously approved these changes Mar 23, 2026

@dfalster dfalster left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice work. I made some edits to make it consistent with other functions and help pass tests.

I suggest expanding the test group of names! We could probably generate some automatically from the resources if that helps

@dfalster
dfalster self-requested a review March 24, 2026 06:01

@dfalster dfalster left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice work! Added some small changes to work with recent changes on master

Feel free to merge once tests pass

@ehwenk
ehwenk merged commit e16d891 into master Mar 24, 2026
4 checks passed
@ehwenk
ehwenk deleted the Synonyms-for-accepted-names branch March 24, 2026 06:18
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