timholy / SnoopCompile.jl

Provide insights about latency (TTFX) for Julia packages
https://timholy.github.io/SnoopCompile.jl/dev/
Other
309 stars 48 forks source link

enhance invalidations docs - increase cov #337

Closed t-bltg closed 1 year ago

t-bltg commented 1 year ago

https://github.com/timholy/SnoopCompile.jl/pull/303 is a great addition for a quick glance at invalidations. As an overview, I believe it should go into the tutorial.

codecov[bot] commented 1 year ago

Codecov Report

Base: 83.70% // Head: 86.11% // Increases project coverage by +2.41% :tada:

Coverage data is based on head (494c04c) compared to base (a230cf8). Patch has no changes to coverable lines.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #337 +/- ## ========================================== + Coverage 83.70% 86.11% +2.41% ========================================== Files 18 17 -1 Lines 2221 2182 -39 ========================================== + Hits 1859 1879 +20 + Misses 362 303 -59 ``` | [Impacted Files](https://codecov.io/gh/timholy/SnoopCompile.jl/pull/337?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Tim+Holy) | Coverage Δ | | |---|---|---| | [src/visualizations.jl](https://codecov.io/gh/timholy/SnoopCompile.jl/pull/337?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Tim+Holy#diff-c3JjL3Zpc3VhbGl6YXRpb25zLmps) | | | | [SnoopCompileCore/src/snoopi.jl](https://codecov.io/gh/timholy/SnoopCompile.jl/pull/337?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Tim+Holy#diff-U25vb3BDb21waWxlQ29yZS9zcmMvc25vb3BpLmps) | `67.21% <0.00%> (+11.47%)` | :arrow_up: | | [src/deep\_demos.jl](https://codecov.io/gh/timholy/SnoopCompile.jl/pull/337?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Tim+Holy#diff-c3JjL2RlZXBfZGVtb3Muamw=) | `92.30% <0.00%> (+33.33%)` | :arrow_up: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Tim+Holy). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Tim+Holy)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

t-bltg commented 1 year ago

No, https://github.com/timholy/SnoopCompile.jl/pull/303 for report_invalidations.

Sorry, increasing coverage is unrelated, and got in the middle of this PR (too lazy to make a separate PR) :/