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

Adding regular mesh based shut down dose rate example #258

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

shimwell
Copy link
Member

@shimwell shimwell commented Dec 10, 2023

Not ready yet but this PR aims to add a mesh based R2S shut down dose rate simulation to the examples. There are a few more parts that need to be added to openmc before this PR can be finished. Currently the meshes that contain two or more materials are not modelled correctly

@shimwell shimwell changed the title Adding mesh based shut down dose rate example Adding regular mesh based shut down dose rate example Apr 2, 2024
@shimwell
Copy link
Member Author

shimwell commented Apr 22, 2024

Paul has a nice PR that solves the multiple materials on a single mesh voxel problem.

openmc-dev/openmc#2971

usage is
list_of_MIXED_materials_one_for_each_voxel = mesh_void.get_homogenized_materials(model, n_samples=1_000_000)

so it should be possible for me to finish of this example now

@rlbarker fyi

@shimwell
Copy link
Member Author

shimwell commented May 9, 2024

The recent improvements in openmc reduce the complexity of this time of simulation considerably many thanks to @paulromano

I've tidied up the example and it is more concise but it is still not quite ready for use yet, sorry @rlbarker

Either I have made a mistake with my meshes or there is an openmc bug that when using a mesh source and a mesh tally then they both end up using the same mesh instead of different meshes. In this case I've got two meshes and want to make the source on a course regularmesh and tally the dose on a finer regular mesh but it appears to be using just the one mesh. I can look into this more.

@shimwell
Copy link
Member Author

we will need the features in this PR as well openmc-dev/openmc#3000

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.

1 participant