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

added subfolders to txt search command in pyproject #3174

Conversation

shimwell
Copy link
Member

Description

When doing pip install . within openmc the .txt files from the subfolders of the /home/jon/openmc/openmc/data/effective_dose/ are not being copied over to the system install directory.

I noticed this as a CI failure while updating my neutronics-workshop examples

I believe the reason we didn't catch it in openmc CI when PR #3020 was merged in is because we don't use the src folder layout so when we run pytest it imports the local openmc folder as the package and this folder includes all the txt files as mentioned in this issue

Anyway it is a small fix to add ** so that the pyproject includes the subdirectories icrp116 and icrp74 and other future subdirectories

Fixes # (issue)

Checklist

  • I have performed a self-review of my own code
  • I have followed the style guidelines for Python source files (if applicable)

@paulromano paulromano merged commit c19b9b1 into openmc-dev:develop Oct 18, 2024
16 checks passed
@paulromano
Copy link
Contributor

Thanks @shimwell!

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.

2 participants