Open neonWhiteout opened 8 months ago
Attention: Patch coverage is 65.85366%
with 98 lines
in your changes are missing coverage. Please review.
Project coverage is 61.20%. Comparing base (
7182495
) to head (b196953
).:exclamation: Current head b196953 differs from pull request most recent head 113bc7e. Consider uploading reports for the commit 113bc7e to get more accurate results
Files | Patch % | Lines |
---|---|---|
src/stockflow/StockFlowUnits.jl | 56.21% | 74 Missing :warning: |
src/visualization.jl | 0.00% | 14 Missing :warning: |
src/Syntax.jl | 89.79% | 10 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I'd like to change the way StockAndFlowF is mapped to StockAndFlowU; need to figure out how to define mappings between schema categories in Catlab.
Long, long term, there's gonna be a more complex version of this. One that uses dimensions and requires the units you use be of that dimension. May involve some dimension category.
May want to make a few more tests.