Skip to content

Preserve sub-millisecond event timestamp precision - #7

Open
bmehta001 wants to merge 7 commits into
mainfrom
bhamehta/fix-event-timestamp-precision
Open

Preserve sub-millisecond event timestamp precision#7
bmehta001 wants to merge 7 commits into
mainfrom
bhamehta/fix-event-timestamp-precision

Conversation

@bmehta001

Copy link
Copy Markdown
Owner

Summary

  • preserve nanosecond-derived 100 ns precision for default POSIX event timestamps
  • use GetSystemTimePreciseAsFileTime on Windows when available, with a Windows 7-compatible fallback
  • add regression coverage for POSIX timestamps retaining sub-millisecond ticks

Fixes microsoft#1514.

Validation

  • git diff --check passed
  • Fresh Windows CMake configuration succeeded
  • The unit-test build was blocked by existing configuration issues: missing gtest/gtest.lib and /WX failures from pre-existing HAVE_MAT_AI macro redefinition / exception-mode warnings

bmehta001 and others added 7 commits August 3, 2026 12:34
Use precise wall-clock time where available and retain nanosecond-derived 100 ns ticks on POSIX so record.time no longer truncates every event to milliseconds. Add regression coverage for POSIX timestamp precision.\n\nCo-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>\nCopilot-Session: 05d1030e-75b0-447f-9856-65091d59a97f
Resolve GetSystemTimePreciseAsFileTime once instead of repeating module and symbol lookups for every event timestamp.\n\nCo-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>\nCopilot-Session: 05d1030e-75b0-447f-9856-65091d59a97f
Ensure vcpkg-built Apple libraries match the consumer deployment target and avoid linker warnings.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 5f341bc5-f8ae-4259-b03b-8eeb87c06837
Propagate the resolved iOS sysroot to embedding builds and keep Apple vendored targets compatible with strict warning settings.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 5f341bc5-f8ae-4257-b03b-8eeb87c06837
Remove legacy Apple architecture, platform, and deployment-target inputs so standalone scripts and embedding consumers share CMAKE_OSX_* configuration.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 5f341bc5-f8ae-4257-b03b-8eeb87c06837
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.

Event timestamps (record.time) only have millisecond resolution despite 100ns tick field

1 participant