Skip to content

Irr299 final#8

Open
irr299 wants to merge 18 commits intoGFZ:mainfrom
irr299:irr299-final
Open

Irr299 final#8
irr299 wants to merge 18 commits intoGFZ:mainfrom
irr299:irr299-final

Conversation

@irr299
Copy link

@irr299 irr299 commented Feb 17, 2026

This would be the Final PR, which includes Aurora Video, and has also uploaded index.html

irr299 and others added 18 commits February 10, 2026 11:28
Updated the html based on the discussion
Fixing the error with "datetime.strptime"
update on aurora forecast
Updated the caption! & KP_comparsion
Co-authored-by: Sahil Jhawar <55475299+sahiljhawar@users.noreply.github.com>
kp_index_monitor, which creates html including video
The patch with the aurora video and also edited HTML format.
mock files for testing
changed the source path to assets folder
Added index.html (including Video)
- add --test CLI flag with mutually exclusive mode validation
- support synthetic Kp data generation for test runs
- thread test mode through run_single_check() and fetch_kp_data()
- update forecast caption text and aurora note wording
- improve logging format with level and source line details
- adjust forecast caption styling and minor message formatting cleanup
- Updated a few UI tweaks

- KeyError for np.float64(nan)
  - When there was no future forecast (all CSV times in the past), max_values.max() was NaN and DECIMAL_TO_KP[max] raised. The code now treats a NaN max as 0.0 and uses DECIMAL_TO_KP.get(max_kp, "N/A") for logging so a missing/max value never causes a crash.

- Round() UserWarnings
  - high_kp_records.round(2) and next_24h.round(2) were applied to whole DataFrames that include datetimes. Rounding is now applied only to numeric columns via select_dtypes(include=[np.number]), so the datetime warning is gone.
Changed the mock data to trigger the alert
This reverts commit 444f29d.
@sahiljhawar
Copy link
Collaborator

sahiljhawar commented Feb 18, 2026

add probability table for different Kps (7-9)

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

Comments