fix: reset when ClimateSimulation=None

[mjaquiery]

No description provided.

[Copilot]

Pull request overview

Updates the climate cache reset utility to safely no-op when no ClimateSimulation instance is available (i.e., None), avoiding crashes when callers attempt to reset a missing simulation.

Changes:

• Broaden reset_climate_cache to accept Optional[ClimateSimulation] and return early on None.
• Add Optional typing import in climate.py.
• Minor formatting-only change at end of tests/unit/test_misc.py.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 2 comments.

File	Description
src/openbes/simulations/climate.py	Allows reset_climate_cache to be called with None by adding an early-return guard and updating type hints/imports.
tests/unit/test_misc.py	No functional change; adjusts the file’s final lines formatting.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The function now accepts Optional[ClimateSimulation] and returns early on None, but the docstring/Args section still states climate_sim is a ClimateSimulation instance. Please update the parameter documentation to reflect that None is allowed and that the function is a no-op in that case.

Suggested change

	climate_sim: The ClimateSimulation instance to reset
	climate_sim: The ClimateSimulation instance to reset, or ``None``.
	If ``None`` is passed, the function is a no-op and returns immediately.

[Copilot]

New behavior: reset_climate_cache(None) now returns without error. Please add a unit test covering the None input to prevent regressions (and to document the intended no-op semantics).