-
Notifications
You must be signed in to change notification settings - Fork 14
Refactor: instruments have sensors #328
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
Open
j-atkins
wants to merge
58
commits into
main
Choose a base branch
from
refactor-sensors
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
ba585c2
remove CTD_BGC instrument type from InstrumentType enum, add SensorTy…
j-atkins 1f7e9b8
update utils: add sensor def mapping and remove old references to ctd…
j-atkins dbaa319
refactor: update SensorType enum and add source-truth for supported s…
j-atkins a2a7c81
add sensors configuration for various instruments
j-atkins 67a04d8
new registries and helper functions
j-atkins 057d9f9
update expedition models, now including SensorConfig model and associ…
j-atkins b82118d
modify adcp instrument class, also abstract expansion to u and v to h…
j-atkins 8d96d8f
dynamic particle class building takes JIT or Scipy particle
j-atkins 818e8f8
raise error when instrument has zero sensors enabled
j-atkins 07c8461
use centralised particle class builder for ADCP now as well
j-atkins 1bf517e
batch update instrument subclasses adapted to refactored sensor logic
j-atkins 961f1fe
rename list
j-atkins ee002d7
adapt argo subclass to sensor refactoring, also separate the sampling…
j-atkins 4777217
consistent particle variable naming
j-atkins c419399
add back in ctd_bgc for now
j-atkins daabfc4
fix import
j-atkins cece7bb
move sensor information to new sensors.py file
j-atkins b429331
update imports across codebase
j-atkins 882a419
add validator/serialiser for reading from YAML, remove unnecessary pr…
j-atkins 126ecc2
re-add JITParticle to particle class when creating instruments
j-atkins 9011b2d
remove CTD_BGC instrument type from InstrumentType enum, add SensorTy…
j-atkins 464b3e9
update utils: add sensor def mapping and remove old references to ctd…
j-atkins 2f7d82d
refactor: update SensorType enum and add source-truth for supported s…
j-atkins 1d7c158
add sensors configuration for various instruments
j-atkins f01bf0e
new registries and helper functions
j-atkins d9f9d10
update expedition models, now including SensorConfig model and associ…
j-atkins c50c43f
modify adcp instrument class, also abstract expansion to u and v to h…
j-atkins bb91f0c
dynamic particle class building takes JIT or Scipy particle
j-atkins b584d70
raise error when instrument has zero sensors enabled
j-atkins 6fb6284
use centralised particle class builder for ADCP now as well
j-atkins 243fb0d
batch update instrument subclasses adapted to refactored sensor logic
j-atkins 21f3b8b
rename list
j-atkins f6e17ac
adapt argo subclass to sensor refactoring, also separate the sampling…
j-atkins 0962261
consistent particle variable naming
j-atkins c9d0623
add back in ctd_bgc for now
j-atkins 8151a60
fix import
j-atkins 892c75d
move sensor information to new sensors.py file
j-atkins 13ded3f
update imports across codebase
j-atkins a30f72b
add validator/serialiser for reading from YAML, remove unnecessary pr…
j-atkins f0f8a19
re-add JITParticle to particle class when creating instruments
j-atkins f245288
Merge branch 'refactor-sensors' of github.com:OceanParcels/virtualshi…
j-atkins 92fe4a1
update with new analysis environment
j-atkins f4c6205
Merge branch 'new-pixi-env' into refactor-sensors
j-atkins 4351657
Merge branch 'main' into refactor-sensors
j-atkins 089d2b6
fix erroneous sampling during descent and drift
j-atkins 96ff22d
update docstring
j-atkins 73474e9
Add sensor configuration tests for various instruments and update con…
j-atkins c5ec69b
new tests for instruments, focused on new sensor logic
j-atkins 7e9c5e9
new test_sensors suite
j-atkins 18c1a2d
new tests for new sensor logic utils
j-atkins 89e184a
remove some overlap/duplication of tests
j-atkins bf61220
deal with circular import issues
j-atkins 51f4457
add ctd_bgc stationkeeping logic back in (accidentally removed earlier)
j-atkins bcbc28f
Merge branch 'main' into refactor-sensors
j-atkins f9f17f2
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 548297b
correct comment
j-atkins 7a163f7
update docstrings, rethink some testing
j-atkins 6978c95
Merge branch 'refactor-sensors' of github.com:OceanParcels/virtualshi…
j-atkins 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 hidden or 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 hidden or 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 hidden or 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 hidden or 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.
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.
This type hint isn't correct
Great use of typing across the PR - I think having this typing is really useful so that it gets us to think more about the structure of the codebase. Currently I don't think typechecking is run in CI (or if it is, we currently have a lot of failures). Should we map out a path forward for enabling typechecking across the codebase as well? The types are only really useful if they're enforced (something I'm working on in Parcels as well).