Skip to content

Translator feedback issue1350#2851

Merged
chunyuma merged 4 commits into
masterfrom
translator-feedback-issue1350
Jul 20, 2026
Merged

Translator feedback issue1350#2851
chunyuma merged 4 commits into
masterfrom
translator-feedback-issue1350

Conversation

@chunyuma

Copy link
Copy Markdown
Collaborator

This PR is created to fix the issue of missing the expected source_record_urls data in the MVP1 query (reported by https://github.com/NCATSTranslator/Feedback/issues/1353). Basically, it did the following main updates:

  1. For infer_utilities.py, add code to recover the expected source_record_urls data from the xdtd database.
  2. Add a test into test_ARAX_infer.py for confirming source_record_urls data is indeed recovered.

@chunyuma

Copy link
Copy Markdown
Collaborator Author

Hi @edeutsch, could you please let me know whether we are ready to merge it into master and then deploy the fix? This fix doesn't change anything in the existing xDTD database but only updates the code to extract the related info. Thank you!

@edeutsch

Copy link
Copy Markdown
Collaborator

Hi @chunyuma this looks fine to me to merge and deploy.
To clarify, you will perform the merge?
And you'd like me to deploy to arax.ncats.io?

@chunyuma

Copy link
Copy Markdown
Collaborator Author

Thanks @edeutsch. I will merge it to the master branch. Could you please help me deploy it? Thank you!

@chunyuma
chunyuma merged commit fe73c46 into master Jul 20, 2026
4 checks passed
@edeutsch

Copy link
Copy Markdown
Collaborator

deployed to arax.ncats.io

@chunyuma

Copy link
Copy Markdown
Collaborator Author

Thank you so much @edeutsch.

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