-
Notifications
You must be signed in to change notification settings - Fork 3
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
fastsimrust is now a submodule of fastsim #21
Conversation
this solves various module not found issues and is consistent with altrios.
deprecated `RUST_AVAILABLE` and `rustext`
@nreinicke, could I ask you to take a look at the failing tests for this PR? I'm unable to reproduce the error on my end and very baffled by the discrepancy. |
…stall-fix # Conflicts: # README.md # python/fastsim/demos/output.html
…st as fsr` to check if this works on server
…astsimrust as fsr` to check if this works on server" This reverts commit 734b72c.
@calbaker - What's your definition of done here? I was hitting the same error locally using pytest and so I just switched over to using the unittest library and everything is working. Is there a specific reason you need to use pytest to run the test suite? Of note: I could get pytest to work when installing the rust extension in a develop mode (either |
@calbaker - FYI - looks like using a dev install with pytest works fine on the server. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gave this a spin in a fresh Conda environment and everything seems to work beautifully. Nice!!
this solves various module not found issues
and is consistent with altrios.
Made it so that tests don't need any dev dependencies.