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

rankID improvements #376

Closed
adam-collins opened this issue Dec 14, 2023 · 1 comment
Closed

rankID improvements #376

adam-collins opened this issue Dec 14, 2023 · 1 comment
Assignees
Milestone

Comments

@adam-collins
Copy link
Contributor

adam-collins commented Dec 14, 2023

rankID is populated using an external reference file at the time DwCA names indexes are imported.

The ALA names index has this information but does not include it in the combined version of the names index. AtlasOfLivingAustralia/ala-name-matching#223

Improvements

  1. Use the imported taxonRankID if it is available https://github.com/AtlasOfLivingAustralia/bie-index/blob/master/grails-app/services/au/org/ala/bie/ImportService.groovy#L1445
  2. Permit external override of the internal taxonRanks.json file.
  3. Update ws/ranks to only return results that are present in the index. This is to include information branch notes othernames found in taxonRanks.json to rank and rankID that exist in the index.
  4. The openapi documentation method was recently changed to annotation so this needs moving.
  5. Update buildNameFormatted that uses rankGroup because this information may now be absent and need to be inferred from rank and rankID. e.g. rankGroup is the upper rank name where the upper rank is identified by a rankID in the 000's, e.g. 1000, 2000, 7000.
  6. Update ranks() https://github.com/AtlasOfLivingAustralia/bie-index/blob/master/grails-app/services/au/org/ala/bie/ImportService.groovy#L2767 so that otherNames are correctly added to the map for searching purposes. Prevent changes to ws/ranks output because it also uses this method.
  7. Update taxonRanks.json to match the current names index builder https://github.com/AtlasOfLivingAustralia/ala-name-matching/blob/901eeb929b0e2b3cba47817da83c67ba1578f255/ala-name-matching-model/src/main/java/au/org/ala/names/model/RankType.java#L37
@adam-collins adam-collins self-assigned this Jan 2, 2024
adam-collins pushed a commit that referenced this issue Jan 3, 2024
@adam-collins
Copy link
Contributor Author

  1. Added a TODO for updating when ala-name-matching-model is updated
  2. Override taxonRanks.json with location specified by config taxonRanksFile
  3. Limits /ranks response to rk_ index fields
  4. Added openapi annotations for /ranks
  5. Unnecessary, see 7
  6. Now using taxonRanks.json's otherNames when searching for rankID
  7. Now using ala-name-matching-model's RankType class as the source for the default taxonRanks.json. Leaving this now unused file as a reference when constructing an override file, see 2.

@adam-collins adam-collins added this to the 3.1.0 milestone Jan 5, 2024
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

No branches or pull requests

1 participant