Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename MAE plot line from "Daily Total MAE" to "Latest Forecast" #241

Merged
merged 1 commit into from
Jan 6, 2025

Conversation

MAYANK12SHARMA
Copy link
Contributor

Pull Request

Description

This PR resolves the issue of renaming "Daily Total MAE" to "Latest Forecast" in the MAE by Horizon plot as described in issue #193 . The changes ensure consistent terminology and improve clarity in the plot labels.

Fixes #193

How Has This Been Tested?

Run it locally

  • Yes

The changes were tested to ensure functionality and consistency.

Steps to Reproduce:

You can reproduce the changes by opening the file located at src\plots\forecast_horizon.py and inspecting line number 27. When you create the forecast_horizon chart, you will see the updated label, "Latest Forecast," displayed correctly.

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

@peterdudfield
Copy link
Contributor

thank you

@peterdudfield peterdudfield merged commit be3fe22 into openclimatefix:main Jan 6, 2025
@peterdudfield
Copy link
Contributor

@all-contributors please add @MAYANK12SHARMA for doc

Copy link
Contributor

@peterdudfield

I've put up a pull request to add @MAYANK12SHARMA! 🎉

@MAYANK12SHARMA
Copy link
Contributor Author

Thank you so much for merging, @peterdudfield! 🙌 I’m excited to contribute more in the future.

@MAYANK12SHARMA
Copy link
Contributor Author

Hi @peterdudfield, I’ve created another PR (#244) for issues #243 and #226. Could you please take a look at it as well?

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.

Rename "Daily Total MAE" to "Latest Forecast" in MAE by Horizon plot
2 participants