Skip to content

fix: correct spelling errors in documentation and code#518

Open
albedo-c wants to merge 1 commit into
openclimatefix:mainfrom
albedo-c:fix/typo-20260303
Open

fix: correct spelling errors in documentation and code#518
albedo-c wants to merge 1 commit into
openclimatefix:mainfrom
albedo-c:fix/typo-20260303

Conversation

@albedo-c
Copy link
Copy Markdown

@albedo-c albedo-c commented Mar 2, 2026

Summary

This PR fixes 14 spelling errors found throughout codebase, improving code readability and documentation quality.

Changes

Fixed 14 spelling errors across 7 files:

Documentation:

  1. README.md:114 - guarenteed -> guaranteed

Scripts:
2. scripts/backtest_sites.py:15 - seperate -> separate
3. scripts/backtest_uk_gsp.py:56 - avaiable -> available
4. scripts/backtest_uk_gsp.py:56 - accordng -> according
5. scripts/mae_analysis.py:95 - indicies -> indices

Training code:
6. pvnet/training/lightning_module.py:257 - persistance -> persistence
7. pvnet/training/lightning_module.py:337 - Uplodad -> Upload

Model code:
8. pvnet/models/base_model.py:419 - timestemps -> timestamps
9. pvnet/models/base_model.py:426 - ouput -> output
10. pvnet/models/ensemble.py:1 - mutliple -> multiple
11. pvnet/models/late_fusion/late_fusion.py:32 - paramters -> parameters
12. pvnet/models/late_fusion/site_encoders/encoders.py:16 - encodeded -> encoded, wih -> with

Model cards:
13. pvnet/model_cards/empty_model_card_template.md:5 - automaticall -> automatically
14. pvnet/model_cards/empty_model_card_template.md:33 - nessessary -> necessary

Impact

These are simple text-only changes that improve code readability and maintain professional documentation standards. No functional changes to the code.

@dfulu
Copy link
Copy Markdown
Member

dfulu commented May 12, 2026

@albedo-c thanks for these changes and sorry its taken so long to get to review this

I see one of these files has been removed since your PR. So if that merge conflict is fixed I'm happy to merge this in

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