Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

extensions/ext: Add VK_EXT_calibrated_timestamps extension #556

Merged
merged 1 commit into from
Jan 17, 2022

Conversation

MarijnS95
Copy link
Collaborator

This extension can be used to correlate timestamps on the GPU timeline (vkCmdWriteTimestamp, vkCmdWriteTimestamp2KHR) to the CPU timeline (ie. CLOCK_MONOTONIC and friends).

@MarijnS95
Copy link
Collaborator Author

If anyone knows why they called the function here calibrateable instead of the arguably more correct calibratable, do let me know!

@MarijnS95 MarijnS95 requested a review from Ralith January 13, 2022 18:53
This extension can be used to correlate timestamps on the GPU timeline
(`vkCmdWriteTimestamp`, `vkCmdWriteTimestamp2KHR`) to the CPU timeline
(ie. `CLOCK_MONOTONIC` and friends).
@Ralith
Copy link
Collaborator

Ralith commented Jan 14, 2022

Wouldn't be the first editorial mistake in the API; raise an issue upstream if you're sure.

@MarijnS95
Copy link
Collaborator Author

https://en.wiktionary.org/wiki/calibrateable Says that it works as an alternative, so I guess no harm done 😬

@MarijnS95 MarijnS95 merged commit 5169862 into master Jan 17, 2022
@MarijnS95 MarijnS95 deleted the calibrated-timestamps branch January 17, 2022 15:32
MarijnS95 added a commit that referenced this pull request Jan 24, 2022
Manual code files usually see extra documentation lines added with
`///`, making for a rather unpleasant (source code) reading experience
when mixed with a link wrapped in `#[doc]`.  This is just a copy-paste
remnant from autogenerated code (`quote` macro converts `///` to
`#[doc]`) and can easily be omitted in manual code.

[#556]: #556 (comment)
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