enh: add Macaca radiata (bonnet macaque) to species_map#1865
Conversation
yarikoptic
added
patch
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1865 +/- ##
==========================================
- Coverage 76.76% 76.74% -0.03%
==========================================
Files 87 87
Lines 12769 12769
==========================================
- Hits 9802 9799 -3
- Misses 2967 2970 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| ["bonnet macaque", "bonnet monkey", "radiata"], | ||
| None, | ||
| NCBITAXON_URI_TEMPLATE.format("9548"), | ||
| "Macaca radiata - Bonnet macaque", |
There was a problem hiding this comment.
I think this version is better than just
| "Macaca radiata - Bonnet macaque", | |
| "Macaca radiata", |
as we did for "Macaca nemestrina" above . I think we should expand all others into similar
{current name} - {Genbank common name} form which seems they all follow.
@copilot -- prepare a PR where all other entries follow the same format for their "name" and also expand description above on the expected format of the "name" field
|
@yarikoptic I've opened a new pull request, #1866, to work on those changes. Once the pull request is ready, I'll request review from you. |
Adds NCBI taxon 9548 (Macaca radiata, common name "bonnet macaque") to the species lookup table so the species can be recognized by common name, "radiata", or its NCBITaxon URL. Requested by a user via email. Co-Authored-By: Claude Code 2.1.154 / Claude Opus 4.7 <noreply@anthropic.com>
yarikoptic
force-pushed
the
enh-macaca-radiata
branch
from
1a2d7a9 to
813fb8e
Compare
Adds NCBI taxon 9548 (Macaca radiata, common name "bonnet macaque") to the species lookup table so the species can be recognized by common name, "radiata", or its NCBITaxon URL.
Requested by a user via email.