-
Notifications
You must be signed in to change notification settings - Fork 17
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
BREAKING CHANGE: InjectionDynamics #1296
Merged
dbirman
merged 23 commits into
release-v2.0.0
from
1235-20-injections-replace-volumetimeconcentration-with-injectiondynamics-2
Mar 7, 2025
Merged
Changes from all commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
7e9e86e
feat: testing out ideas for injection dynamics
dbirman a6cbfa4
fix optional rate/duration
dbirman e04a367
Merge branch 'release-v2.0.0' into 1235-20-injections-replace-volumet…
dbirman 3c5513b
Merge branch 'release-v2.0.0' into 1235-20-injections-replace-volumet…
dbirman 8ac6070
Merge branch 'release-v2.0.0' into 1235-20-injections-replace-volumet…
dbirman e5a57c2
refactor: starting to rip out volume/volume_unit
dbirman 1d10410
refactor: updating examples/tests to the new dynamics pattern
dbirman d26bf9a
fix: some typos
dbirman 082137d
fix: improving Nanoject validator
dbirman 137a31f
fix: missing InjectionProfile in a few places
dbirman 9e2b28f
feat: add validator for bolus profile + test
dbirman aa5cac4
chore: lint
dbirman 91e7a1e
tests: fixing test that never tested what it was supposed to
dbirman bc0adbb
chore: lint
dbirman 6f4bb97
Merge branch 'release-v2.0.0' into 1235-20-injections-replace-volumet…
dbirman e8fdcaf
Merge branch 'release-v2.0.0' into 1235-20-injections-replace-volumet…
dbirman c2da78e
refactor: move profile into dynamics, move current into dynamics
dbirman 9850eeb
chore: dependency gone
dbirman 67e9ac5
chore: moving profile into dynamics
dbirman a70fc53
fix: accidental deletion in procedures, and missing [] on plasmids
dbirman 11b3bcc
chore: lint
dbirman e5ca5c6
fix: remove old validator, fix plasmid list
dbirman 5b9e8a3
chore: lint
dbirman File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't remove the time here - it's not a duration field, it's the time when the injection is done. This is for the NBA project where they do timed injections prior to blood collection.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added back