tejrai07 commented on issue #97: URL: https://github.com/apache/otava/issues/97#issuecomment-3938878749
@henrikingo , I’ve finished a local technical analysis of the codebase to prepare for this refactor. Here is my proposed plan to ensure a clean removal: Impacted Files Identified: My grep results show that fill_missing is defined in otava/analysis.py and utilized in otava/series.py (lines 31, 248, 398). Refactor Strategy: - I will remove the fill_missing() definition from otava/analysis.py. In otava/series.py, I will refactor the logic to handle raw data points directly, following your guidance that running the algorithm on available points is preferred over padding. Verification: I will remove the corresponding unit tests in tests/analysis_test.py and run the full test suite to ensure no regression in the core series logic. I’m ready to start as soon as I’m assigned. Thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
