-
Notifications
You must be signed in to change notification settings - Fork 20
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
An error in kp_selector.py #2252
Comments
hey @chunyuma - I think ever since the changes made last summer to how Expand refreshes its KP info cache, if you're running ARAX locally, you need to manually run the KP info cache process before running tests (I think the cache needs to be refreshed within the last 24 hours or you'll see that error). you can do that by navigating to the
then you should be able to run pytests as usual. it'd be great if we somehow came up with a way of automatically running this command prior to running pytests locally (the process is automatically run by the BackgroundTasker on instances that are running the whole ARAX flask application, but when we run tests locally we're not actually running flask, so there's no BackgroundTasker..) |
Before a solution that runs automatically is implemented, do you think it would be worth adding such directions to the wiki in the Dev notes about local running of ARAX? |
Thanks @amykglen! It fixed the issue. I will keep this issue open to discuss if we need to |
Yes, I do. @sundareswarpullela - can you update the wiki with this step since you made the changes to the KP info caching process? |
@amykglen I will add this ASAP |
Its already mentioned in step 5 of Setting up for local dev work on ARAX |
@sundareswarpullela - ah, nice, thanks! maybe it would make sense to add a note to the pytest section too, since it needs to be run basically any time we run pytests? (not just the initial dev environment set up) https://github.com/RTXteam/RTX/wiki/Dev-info#arax-pytest-suite |
Updated the dev info. Closing the issue. |
Hi team,
I ran
pytest -vk test_ARAX_infer.py
to test ARAX_infer with the updated xDTD database. Most of tests have passed except fortest_xdtd_with_only_qg
. The error message seems to indicate the error is from kp_selector.py. I pasted the error mssage below:Can anyone help with this issue? Thanks!
The text was updated successfully, but these errors were encountered: