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

Openmc impot #22

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Openmc impot #22

wants to merge 2 commits into from

Conversation

makeclean
Copy link
Owner

This PR adds the ability to import OpenMC geometry and translate to other functions.

It still needs work on geometries that have lattices & universes, but other than that should be functional

@@ -9,6 +9,8 @@
from OpenMCSurface import SurfaceCard,surface_from_attribute, write_openmc_surface
from OpenMCCell import cell_from_attribute, write_openmc_cell
from OpenMCMaterial import material_from_attribute, write_openmc_material
from OpenMCLattice import lattice_from_attribute, write_openmc_lattice

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@makeclean Andrew, I'm interested in helping with building out the functionality in this branch. I have an interest in supporting translation back and forth between MCNP and OpenMC. I am curious though, if this addition is just a placeholder, or if you happen to have an uncommitted OpenMCLattice.py file (there doesn't seem to be one in this branch). Thanks.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @rtswanson I think I have a partial file somewhere, I'll push it and then feel free to work from that. I basically hit a wall that I didn't have the bandwidth for, at my inspection from the MCNP manual and the https://openmc.readthedocs.io/en/stable/io_formats/geometry.html#lattice-element OpenMC - I couldnt see an easy 1-1 mapping. But I would be thrilled if you want to take this on :)

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