Skip to content

Add metrics calculation app#29

Merged
dfulu merged 13 commits intomainfrom
add_metrics_app
Sep 22, 2025
Merged

Add metrics calculation app#29
dfulu merged 13 commits intomainfrom
add_metrics_app

Conversation

@dfulu
Copy link
Copy Markdown
Member

@dfulu dfulu commented Sep 17, 2025

Pull Request

Description

Add a new app for scoring the cloudcasting model in production. The new cloudcasting metrics app takes all of the cloudcasting it times from the day before and scores them against the satellite ground truths

Completes #16

  • New prediction scoring is stored in cloudcasting_metrics

Also:

  • Refactor tests
  • Update docs
  • Modify entrypoints
  • Change env name for clarity
  • The model inference has been moved to cloudcasting_inference

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked my code and corrected any misspellings

@dfulu dfulu marked this pull request as ready for review September 19, 2025 13:58
@dfulu dfulu requested a review from peterdudfield September 19, 2025 14:02
@dfulu dfulu requested a review from devsjc September 19, 2025 14:47
Copy link
Copy Markdown
Contributor

@devsjc devsjc left a comment

Choose a reason for hiding this comment

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

Nice, thanks for packaging it all sensibly!

@dfulu dfulu merged commit edf6df1 into main Sep 22, 2025
2 checks passed
@dfulu dfulu deleted the add_metrics_app branch September 22, 2025 10:56
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.

3 participants