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

Trading On-Demand reactor #66

Merged
merged 4 commits into from
Feb 24, 2025
Merged

Trading On-Demand reactor #66

merged 4 commits into from
Feb 24, 2025

Conversation

nsryan2
Copy link
Member

@nsryan2 nsryan2 commented Feb 24, 2025

Summary of changes

TOD is a Cyclus reactor archetype based on Cycamore's Reactor, except that it
has the ability to update the power output over time. TOD achieves this by
creating a private variable that is updated at the end of the Tock phase when
the reactor refuels. Each intervening step checks if the current time step
equals this private variable.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Associated Issues and PRs

  • Issue: #

Associated Developers

  • Dev: @

Checklist for Reviewers

Reviewers should use this link to get to the
Review Checklist before they begin their review.

@nsryan2 nsryan2 added the New Feature Adds a new feature label Feb 24, 2025
@nsryan2 nsryan2 self-assigned this Feb 24, 2025
@nsryan2 nsryan2 merged commit 3ab5e5b into arfc:main Feb 24, 2025
1 check passed
@nsryan2 nsryan2 deleted the thesis_reactors branch February 24, 2025 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Feature Adds a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant