Skip to content

Issue 1751: Update predictors on upload#2166

Merged
rmroot merged 3 commits intodevelopfrom
issue-1751
Nov 24, 2025
Merged

Issue 1751: Update predictors on upload#2166
rmroot merged 3 commits intodevelopfrom
issue-1751

Conversation

@rmroot
Copy link
Copy Markdown
Member

@rmroot rmroot commented Nov 24, 2025

connects #1751

This pull request adds new functionality to support updating weather predictor data after importing account data, along with UI improvements to guide users through this process. The most important changes are the addition of new methods and UI elements to facilitate updating weather predictors, as well as some supporting service and logic updates.

Weather Predictor Update Workflow

  • Added an @Input() fileReference to UpdateWeatherPredictorsModalComponent and updated its logic to use facilities from the imported file, enabling facility-specific weather predictor updates after data import. [1] [2] [3] [4]
  • Added a new method updateWeatherPredictorData(fileReference) to UploadDataService that calculates date ranges for each facility and calls the weather predictor management service to update predictors. [1] [2] [3]

UI Enhancements

  • Updated the import submit UI (submit-import-data.component.html) to show a success message and buttons for updating weather predictors or uploading another file after successful data import. The weather predictor update modal is conditionally displayed. [1] [2]
  • Added state variables and methods in SubmitImportDataComponent to manage the display of the weather predictor modal and track submission state, as well as logic to check for weather predictors and handle the new workflow. [1] [2] [3] [4]

Weather Predictor Calculation Improvements

  • Improved the warning logic in WeatherPredictorManagementService to also flag predictors when no degree days are available, increasing data integrity.

@rmroot rmroot merged commit 6d53cdb into develop Nov 24, 2025
3 checks passed
@rmroot rmroot deleted the issue-1751 branch December 3, 2025 16:32
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