Skip to content

#2602#2625

Merged
saramsey merged 1 commit intomasterfrom
issue-2602
Feb 2, 2026
Merged

#2602#2625
saramsey merged 1 commit intomasterfrom
issue-2602

Conversation

@saramsey
Copy link
Member

@saramsey saramsey commented Feb 2, 2026

The changes to the requirements.txt fix the issue. I also fixed a couple of other things (leftover debugging code, and some ancient incorrect string literals in error messages) that I noticed along the way debugging this issue. I've already deployed the updates shown in the requirements.txt diffs, to arax.ncats.io in the system python3.12 inside the rtx2 container.

Copy link
Collaborator

@edeutsch edeutsch left a comment

Choose a reason for hiding this comment

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

looks fine, thanks.
Have you performed a full pip with -r requirements.txt on arax.ncats.io?

@saramsey
Copy link
Member Author

saramsey commented Feb 2, 2026

On arax.ncats.io, I think I only installed my specific new entries in requirements.txt.

But on my dev laptop, I have tested the complete new requirements.txt, as a fresh install.

@saramsey saramsey merged commit bf4b0c3 into master Feb 2, 2026
1 of 3 checks passed
@saramsey saramsey deleted the issue-2602 branch February 2, 2026 23:30
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.

5 participants