Skip to content

Allowing a Snapshot to not write to DB #2032

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Allowing a Snapshot to not write to DB #2032

wants to merge 1 commit into from

Conversation

onufer
Copy link
Member

@onufer onufer commented Dec 11, 2024 •
edited by john-science
Loading

What is the change?

Allow for Snapshot not to write to DB if db: false

Why is the change being made?

Sometimes (very rarely) snapshots are desired without writing a new DB. this supports that setting

https://github.com/terrapower/armi/pull/1783/files for more context

SCR Information

Change Type: features

One-Sentence Description: Allowing a Snapshot to not write to DB.

One-line Impact on Requirements: NA

Checklist

@john-science john-science added the feature request Smaller user request label Dec 11, 2024
@john-science john-science self-requested a review March 14, 2025 21:26
@john-science john-science changed the title support not writting db in snapshots Allowing a Snapshot to not write to DB Mar 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@john-science john-science Awaiting requested review from john-science

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
feature request Smaller user request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@onufer @john-science