Skip to content

Conversation

@youngbloodcyb
Copy link

@youngbloodcyb youngbloodcyb commented Dec 24, 2024

Fix year handling in wave forecast bulletin parser

First off, love this project and it's been a blast to use!

Fixed an issue in the wave forecast bulletin parser where forecasts spanning across a new year were incorrectly dated. Before, when a forecast crossed from December into January, the January dates were assigned to the same year instead of incrementing to the next year which caused some issues when charting and reading the data

ex: forecast dates were incorrectly showing:

  • Start: 2024-12-23
  • End: 2024-01-08

The fix properly handles the year transition, ensuring January dates are correctly assigned to the next year

Changes

  • Added year tracking in the date parsing logic
  • Updated the month rollover handling to increment the year when transitioning from December to January

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.

1 participant