Skip to content

Conversation

@dfulu
Copy link
Member

@dfulu dfulu commented Jan 12, 2026

Changes in this Pull Request

This PR adds a line to the logs which is something like

2026-01-12 14:33:50 INFO     83.33% progress through time range. 0.89 seconds/image. ETA 0:00:10.724019

On the main branch we do have the % progress, but this PR adds the seconds/image log and the estimate time to remaining/

The seconds/image is done using an exponential moving average (EMA) which was largely taken from tqdm. The ETA uses the EMA and assumes that all images between the current one and the end of the time range will be available and that the process continues at the current speed.

Contribution Checklist

  • Have you followed the Open Climate Fix Contribution Guidelines?
  • Have you referenced the Issue this PR addresses, where applicable?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added a summary of the changes?
  • Have you written new tests for your changes, where applicable?
  • Have you successfully run make lint with your changes locally?
  • Have you successfully run make test with your changes locally?

@dfulu dfulu requested a review from devsjc January 12, 2026 14:42
@dfulu dfulu force-pushed the add_speed_log branch 3 times, most recently from ed21f56 to 0572d21 Compare January 12, 2026 15:48
@dfulu dfulu merged commit 8d44cc5 into main Jan 12, 2026
2 checks passed
@dfulu dfulu deleted the add_speed_log branch January 12, 2026 17:50
@dfulu dfulu changed the title Add loop speed logs chore(logging) Add loop speed logs Jan 12, 2026
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.

3 participants