Skip to content

Make timestamps consistent#10

Merged
amd-pworfolk merged 3 commits intomainfrom
pworfolk/minor_fixes
Mar 23, 2026
Merged

Make timestamps consistent#10
amd-pworfolk merged 3 commits intomainfrom
pworfolk/minor_fixes

Conversation

@amd-pworfolk
Copy link
Copy Markdown
Contributor

Per Issue #9, I've reviewed the timestamping behavior. These changes:

  1. Use a single build time taken at the launch of the run and now stored as part of the state.
  2. Use local time for the build folder name (instead of UTC), making it much easier to find the appropriate build
  3. The timestamp field of the stats file still contains the offset to UTC so absolute time can always be determined.

I also added one fix to exit cleanly if the Lemonade cache folder doesn't exist.

Closes #9 .

@amd-pworfolk amd-pworfolk self-assigned this Mar 18, 2026
@amd-pworfolk amd-pworfolk marked this pull request as ready for review March 19, 2026 00:31
Copy link
Copy Markdown

@ramkrishna2910 ramkrishna2910 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@amd-pworfolk amd-pworfolk merged commit d39278e into main Mar 23, 2026
2 checks passed
@amd-pworfolk amd-pworfolk deleted the pworfolk/minor_fixes branch March 23, 2026 20:06
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.

Make timestamps consistent

2 participants