-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
REL: pytao v0.4.1 #25
Conversation
@conda-forge-admin, please rerender |
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( For recipe/meta.yaml:
|
…nda-forge-pinning 2024.10.01.17.37.14
- test -f $PREFIX/bin/pytao-gui # [unix] | ||
- git clone --depth 1 https://github.com/bmad-sim/bmad-ecosystem bmad | ||
- ACC_ROOT_DIR=$PWD/bmad pytest -v --pyargs pytao.tests | ||
- echo | pytao -lat $PWD/bmad/bmad-doc/tao_examples/erl/lat.bmad | ||
- TAO_REUSE_SUBPROCESS=1 ACC_ROOT_DIR=$PWD/bmad pytest -v --pyargs pytao.tests -k "not help and not test_plot" |
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.
Explanation of -k "not help and not test_plot"
:
not help
-tao.help()
interface command test appears to be causing a segfault in the non-subprocess version. This is only on Azure Pipelines runners (i.e., this feedstock). I can't reproduce it locally on MacOS or Linux. Skipping it for now until we can get some more debug details back to Sagan.not test_plot
- this is a broad removal of the majority of plotting tests, which are exceedingly slow on Azure Pipelines. The build has been timing out, so this is an arbitrary way to reduce the amount of time spent on tests in the feedstock.
@@ -41,10 +41,12 @@ test: | |||
- pytao.plotting | |||
- pytao.plotting.bokeh | |||
commands: | |||
- echo | pytao --help | |||
# Add back once https://github.com/bmad-sim/pytao/issues/101 is resolved: | |||
# - pytao --help |
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.
Checklist
0
(if the version changed)conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)