[ENH] Functions added to fetch the termURLs of diagnosis, assessment, sex and image modality - Githubissues

neurobagel / query-tool-ai

MIT License

2 stars 0 forks source link

[ENH] Functions added to fetch the termURLs of diagnosis, assessment, sex and image modality #20

Closed Raya679 closed 4 months ago

Raya679 commented 4 months ago

Closes #19

Changes proposed in this pull request:

Fetches the appropriate termURL of diagnosis and assessment tool from the respective api url.
Gets the appropriate termURL of sex and image modality from the hardcoded mappings.
Added tests for all the new functions.

Checklist

This section is for the PR reviewer

[x] PR has an interpretable title with a prefix ([ENH], [FIX], [REF], [TST], [CI], [MNT], [INF], [MODEL], [DOC]) (see our Contributing Guidelines for more info)
[x] PR has a label for the release changelog or skip-release (to be applied by maintainers only)
[x] PR links to GitHub issue with mention Closes #XXXX
[x] Tests pass
[x] Checks pass

For new features:

[x] Tests have been added

For bug fixes:

[ ] There is at least one test that would fail under the original bug conditions.

Raya679 commented 4 months ago

Thanks for making the changes @Raya679! I left one small comment other than that this is good to go 🧑‍🍳 Side note, how did you create the abbreviations?

Hi @rmanaem thanks for the review!! Regarding the abbreviations I had created them using llama3 itself similar to what was done in the annotation tool ai