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

Add in Recipe Update functionality to EVER #17

Merged
merged 9 commits into from
Jun 27, 2024
Merged

Add in Recipe Update functionality to EVER #17

merged 9 commits into from
Jun 27, 2024

Conversation

nsryan2
Copy link
Member

@nsryan2 nsryan2 commented Jun 19, 2024

This PR adds the lowest level fidelity case we envisioned for EVER (something the Cycamore Reactor already does), changing the recipe at specific times. As outlined in #10

This PR creates the header, source, and test files for EVER in this repo. It documents the functionality in the EVER/README.md, and adds a simple test scenario comparing to the Cycamore Reactor with an analysis notebook based on baseline.

Warning

This PR does not add the functionality to build the archetype (currently the workflow is you add the files to cycamore and build them there. An issue will be created before this PR is merged to discuss/address that.

Issue #18 has been created, and PRs #23 and #24 have half addressed it. There is still no functionality to build the archetype here in NEAR, but there is now an install script that copies it over and automatically builds cycamore with the new archetype that does work.

@nsryan2 nsryan2 self-assigned this Jun 19, 2024
@nsryan2 nsryan2 mentioned this pull request Jun 19, 2024
3 tasks
@nsryan2 nsryan2 changed the base branch from main to ever June 19, 2024 17:20
@nsryan2 nsryan2 added the New Feature Adds a new feature label Jun 24, 2024
@nsryan2 nsryan2 merged commit 5d96a93 into ever Jun 27, 2024
@nsryan2 nsryan2 deleted the ever_update branch June 27, 2024 12:46
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