-
Notifications
You must be signed in to change notification settings - Fork 14
Improve test coverage for provider implementation #182
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
Closed
Closed
Changes from all commits
Commits
Show all changes
50 commits
Select commit
Hold shift + click to select a range
32d4124
introduce test
sabasehrish 635874b
make provider_test work
marcpaterno e696b2d
Add development plan notes to provider_test.cpp
marcpaterno 29546d1
Introduce non-trivial data product
marcpaterno 81dc861
Add stub provider with struct data product
marcpaterno dc16810
Add provide Method And Documentation
marcpaterno c5ad9de
Add provider node support to framework
marcpaterno a71595b
Add provider node support to framework
marcpaterno 6d96e2a
Adjust the matching between providers and downstream ports
knoepfel 5f74b75
Make sure there is only one entry per unique provider name
knoepfel 697fcb3
Change provider API from input_family to output_product
knoepfel 144a67d
Remove output_products from provider_api
marcpaterno 19c78f0
Add providers for tests
knoepfel bed7e95
Add providers for tests.
sabasehrish b1b217f
Modify tests to use providers
marcpaterno 223dc18
Modify more tests to use providers
marcpaterno bcd4ecf
Add provider support to graph_proxy
marcpaterno 9f63d3f
added providers to benchmark tests
sabasehrish fc8d46a
added providers to check_parallelism tests
sabasehrish 2811ad1
added providers to the job:add test
sabasehrish 9d8ce91
added provider to the many_events test
sabasehrish 53ff93a
added provider to the mock-workflow test
sabasehrish a5a962c
adding the provider
sabasehrish 146eafc
changing data_store to data_cell_index in graph
sabasehrish 15c324e
More work toward function providers
marcpaterno f3b0b3e
Adjust fold bookkeeping; remove unnecessary product_store functionality
knoepfel 141496c
Apply clang-format
knoepfel 0bfb7bc
added algorithm header
sabasehrish 6904f36
Adjust driver to accommodate API change
knoepfel 96cb5d4
updated multiplexer to work with providers
sabasehrish 3be11b3
removed fancy comments
sabasehrish 2c9ebed
added execution checks for providers
sabasehrish 32ab6a4
Apply clang-format and update code comments
knoepfel 8efa8f0
Move making of provider edges to dedicated function
knoepfel 3e164c7
Reduce severity of (or remove) logged messages
knoepfel 611de81
Relax check on execution counts for now
knoepfel bd98914
Simplify multiplexing
knoepfel 94dfb00
Further simplifications to multiplexing
knoepfel 0ece841
Another tweak
knoepfel e820868
Reduce duplication
knoepfel 7448456
Add caching to providers to prevent unnecessary re-execution
sabasehrish f8d0196
Allow head ports to be empty for driver-only jobs
knoepfel 6242bd8
Simplify declared_provider class
knoepfel 589d19a
Some cleanups; and a typo fix
knoepfel 40cccf4
Remove trailing whitespace
knoepfel 46e0bdd
Some renaming
knoepfel 92c09c9
Set type of provided product
knoepfel fb945c0
Initial plan
Copilot cf2a0da
Add test coverage for provider flush message handling
Copilot cb658df
Merge branch 'providers' into copilot/sub-pr-180
greenc-FNAL 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
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.
Uh oh!
There was an error while loading. Please reload this page.