Skip to content
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

testing for multiple python versions #36

Open
shimwell opened this issue Nov 19, 2021 · 2 comments
Open

testing for multiple python versions #36

shimwell opened this issue Nov 19, 2021 · 2 comments

Comments

@shimwell
Copy link
Member

Not sure if this will work as we run ci in a container.

However it would be great if multiple python versions can be tested

Here is an example

https://github.com/C-bowman/inference-tools/blob/760061e77bdc8ddeacb951196a3deb783a6ccffd/.github/workflows/tests.yml#L16-L17

@LiamPattinson
Copy link
Contributor

I found the openmc container to be quite difficult to work with. Any change to the Python environment using the setup-python action caused all tests to fail, so simply adding a python version matrix won't help. I think it'd be best to work with a fixed Python version as long as we're using the openmc container.

Another option would be to move away from the openmc container and simply use ubuntu-latest. However, whenever I try running examples locally, I'm met with complaints about missing .xml files, and I'm not familiar enough with the workings of openmc to solve these issues. If we can figure out a way to fetch the necessary .xml files from within ubtuntu-latest, I think testing multiple Python versions would then be fairly straightforward.

@shimwell
Copy link
Member Author

Ah yes the openmc docker image has just the one version of python inside. So it is not quite as straight forward as it could be.

I guess the missing xml message just needs some minimal xml files. Perhaps the ones attached help.
openmc_xml_files.zip

Another option is to make use of a miniconda docker image and install openmc using conda install -c conda-forge openmc then create different python envirments with conda create --name py37 python=3.7

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

No branches or pull requests

2 participants