Skip to content

Conversation

@Roy-Haolin-Du
Copy link
Member

Description

Fixes the statistical inefficiency timestep units from femtoseconds to nanoseconds.

  • Multiplies stat_ineffs_all_winds by engine_config.timestep / 1_000_000 to ensures the statistical inefficiencies time are correctly reported in nanoseconds.
  • Fix import order and formatting by ruff --select I

Status

  • Ready to go

Minor modification, I’ll merge it once the testing is complete.
Thanks!

@fjclark
Copy link
Collaborator

fjclark commented Jan 20, 2026

Thanks for catching! Looks good to me. Worth bumping the minor version and adding to the changelog. Thanks!

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