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

Follow default IMP: Cell Importance logic #657

Open
tjlaboss opened this issue Feb 7, 2025 · 0 comments
Open

Follow default IMP: Cell Importance logic #657

tjlaboss opened this issue Feb 7, 2025 · 0 comments
Labels
feature request An issue that improves the user interface.

Comments

@tjlaboss
Copy link
Collaborator

tjlaboss commented Feb 7, 2025

Is your feature request related to a problem? Please describe.

The MCNP 6.3.1 Manual § 5.12.1 describes the input syntax for imp: (cell importance) cards. Default cell importances follow some logic described in the proceeding paragraphs.

Image

The default particle importance for a cell (if any) depends on a number of factors, including:

  • Other particle importances on the cell
  • Presence of a WWN card
  • Cell-based vs. mesh-based weight windows
  • Whether importance is not set, or explicitly skipped with the Jump feature.

Describe the solution you'd like
Implement the logic described in the MCNP6.3.1 manual and set cell importances accordingly.

Describe alternatives you've considered
Unset cell importances resulted in fatal errors in MontePy, and this is better than a partial implementation that could set incorrect values.

Additional context
See Issue #381 and PR #656

@tjlaboss tjlaboss added the feature request An issue that improves the user interface. label Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request An issue that improves the user interface.
Projects
None yet
Development

No branches or pull requests

1 participant