Skip to content

Multi coverage by time plot#210

Open
EdwinB12 wants to merge 1 commit intomainfrom
Multi-coverage-by-time-plot
Open

Multi coverage by time plot#210
EdwinB12 wants to merge 1 commit intomainfrom
Multi-coverage-by-time-plot

Conversation

@EdwinB12
Copy link
Contributor

@EdwinB12 EdwinB12 commented Feb 5, 2026

Adds functionality to plot coverage by time steps.

Add to notebook by:

time_zones = [(i, i + 10) for i in range(0, 90, 10)]
plots.plot_coverage(preds, trues, time_zones=time_zones)

time_zones is None by default and therefore normal coverage plot is used.

Screenshot 2026-02-05 at 14 57 43

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@EdwinB12 EdwinB12 changed the base branch from main to 201-coverage February 5, 2026 15:10
@EdwinB12
Copy link
Contributor Author

EdwinB12 commented Feb 5, 2026

To be merged on @sgreenbury coverage branch

@EdwinB12 EdwinB12 mentioned this pull request Feb 5, 2026
Copy link
Contributor

@sgreenbury sgreenbury left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks @EdwinB12! A few comments:

  • Would it be possible to replace time_zones with windows in the API and docs etc since that's the API elsewhere?
  • Would it help for MultiCoverageByTime to subclass MultiCoverage and reduce any repeated code?
  • Perhaps we could add further class or extend API to support returning the coverage fully spatiotemporally too though I think we can look at this in a separate PR. Being addressed as part of #212.

Base automatically changed from 201-coverage to main February 6, 2026 12:25
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.

2 participants