Skip to content

Conversation

@beojan
Copy link
Contributor

@beojan beojan commented Dec 11, 2025

This is a preliminary solution for #108. Output products still need to be specified as empty strings at this point.

The API for input product specification without a product name is:

.input_products(from(producer_name).in(layer_name), ...)

@codecov
Copy link

codecov bot commented Dec 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@            Coverage Diff             @@
##             main     #184      +/-   ##
==========================================
+ Coverage   81.13%   81.46%   +0.33%     
==========================================
  Files         116      116              
  Lines        2083     2083              
  Branches      332      331       -1     
==========================================
+ Hits         1690     1697       +7     
+ Misses        255      250       -5     
+ Partials      138      136       -2     
Flag Coverage Δ
unittests 81.46% <100.00%> (+0.33%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
phlex/core/edge_creation_policy.hpp 100.00% <ø> (+18.18%) ⬆️
phlex/core/product_query.cpp 66.66% <100.00%> (+2.38%) ⬆️
phlex/core/product_query.hpp 66.66% <100.00%> (+4.16%) ⬆️

... and 2 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7c11a93...1016ae6. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@beojan beojan requested a review from knoepfel December 11, 2025 19:38
@beojan beojan marked this pull request as ready for review December 11, 2025 21:17
@beojan beojan force-pushed the find-product-by-creator-name branch from 9816d89 to bee7fde Compare December 11, 2025 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant