Skip to content

Update gray mode for compatibility with latest singularity-opac. - #33

Merged
RyanWollaeger merged 5 commits into
developfrom
up_sing_opac
Aug 4, 2026
Merged

Update gray mode for compatibility with latest singularity-opac.#33
RyanWollaeger merged 5 commits into
developfrom
up_sing_opac

Conversation

@RyanWollaeger

Copy link
Copy Markdown
Collaborator

Background

  • This is (currently) the minimal change set to get jaybenne working with the head of singularity-opac.
  • In the near future (maybe here as well) we will want to either update the multigroup mode to use the multigroup mean opacities or add a 3rd (true) multigroup mode.
  • Note this depends on changes in singularity-opac PR 77.

Description of Changes

  • Bump singularity-opac submodule.
  • Update oapcity type names constructors and API calls.
  • Use 1-group construction for gray mode in mcblock.
  • Access hard-coded group 0 for gray mode runs.
  • Fix mcblock field opacity update: use optional (missing) gmode.
  • Ignore dirty submodules (following A. Dempsey's artemis change).

Checklist

  • New features are documented
  • Tests added for bug fixes and new features
  • (@lanl.gov employees) Update copyright on changed files

+ Bump singularity-opac submodule.
+ Update oapcity type names constructors and API calls.
+ Use 1-group construction for gray mode in mcblock.
+ Access hard-coded group 0 for gray mode runs.
+ Fix mcblock field opacity update: use optional (missing) gmode.
+ Ignore dirty submodules (following A. Dempsey's artemis change).
@RyanWollaeger

Copy link
Copy Markdown
Collaborator Author

Will go ahead and merge this once the CI is finished, then follow it up with the (hopefully small) change to have multigroup opacity used in multigroup mode (for now replacing the "continuous" frequency currently used in "multigroup").

@RyanWollaeger
RyanWollaeger merged commit 8a9d628 into develop Aug 4, 2026
3 checks passed
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