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.

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

Feature/plugin #6

Merged
merged 29 commits into from
Aug 12, 2024
Merged

Feature/plugin #6

merged 29 commits into from
Aug 12, 2024

Conversation

ChrisspyB
Copy link
Member

A collection of changes made to faciliate gribjump's use as a plugin to FDB. We register callbacks in FDB to trigger creation of .gribjump files on-the-fly when fdb.archive() is called

@ChrisspyB
Add fdbplugin logic, tidy aggregator
cf686de
@ChrisspyB
Use new plugin.setup
48acd5c
@ChrisspyB
Use single archive callback
dcac876
@ChrisspyB
Merge branch 'develop' into feature/plugin
4cf749a
@ChrisspyB
Add dump info tool
0658c3d
@ChrisspyB
Remove unused multipliers from JumpInfo
2659337
@ChrisspyB
Improve logging
8efc4ed
@ChrisspyB
Add callback/scanning consistency test
b4c3af0
@ChrisspyB
Use futures in InfoAggregator and register a constructor callback wit…
8dfe485 
…h fdb
@ChrisspyB
Move constructor callback registration to FDBPlugin
477a6a8
@ChrisspyB
Small tidy up
238a416
@ChrisspyB
Have cmake write the plugin manifest
25b78c8
@ChrisspyB
Dont use eckit::resource before eckit::main
6a1a8ca
@ChrisspyB
Missing header
93de651
@ChrisspyB
Tool test depends on FDB tools
be941fe
@ChrisspyB
Conditionally include test
97c60a2
@ChrisspyB
Correctly check for fdb tools
dcccdcd
@ChrisspyB
Use eckit exception instead of std runtime_err
6ebd2cc
@ChrisspyB
Merge branch 'develop' into feature/plugin
bd7719f
@ChrisspyB
Fix bug finding ccsds offsets for a constant field
90293fe
@ChrisspyB
Add and use SerialAggregator
f3e2000
@ChrisspyB
Appendable cache, and fix flushes on parallel aggregator
1fee4e4
@ChrisspyB
Update infocache print
6a18316
@ChrisspyB
Use parallel aggregator
b0a7285
@ChrisspyB
Improve configuration
2e6d64e
@ChrisspyB
Remove plugin.enable from config
057c56f
@ChrisspyB ChrisspyB self-assigned this Aug 7, 2024
@FussyDuck
Copy link

FussyDuck commented Aug 7, 2024 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@ChrisspyB ChrisspyB marked this pull request as ready for review August 12, 2024 15:27
@ChrisspyB ChrisspyB merged commit 94706fd into develop Aug 12, 2024
2 checks passed
@ChrisspyB ChrisspyB deleted the feature/plugin branch August 12, 2024 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@simondsmart simondsmart Awaiting requested review from simondsmart

Assignees

@ChrisspyB ChrisspyB

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ChrisspyB @FussyDuck