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

Timestamp capture #851

Merged
merged 2 commits into from
Nov 10, 2023
Merged

Timestamp capture #851

merged 2 commits into from
Nov 10, 2023

Conversation

davidplowman
Copy link
Collaborator

This commit adds the ability to capture a request but where nothing was exposed before a given timestamp (which may be "now").

The second commit is an example/test for this functionality.

This means that no pixel in the returned request started being exposed
before the given timestamp. Any such requests are flushed out and not
returned to the caller.

The "flush" parameter may be an explicit timestamp (in nanoseconds,
for example obtained from time.monotonic_ns()), or just True, meaning
"now".

Signed-off-by: David Plowman <[email protected]>
@davidplowman
Copy link
Collaborator Author

@naushir @will-v-pi Do we like this feature? I do get asked this kind of question from time to time, so I think it may be useful. I wonder a bit whether all the capture_xxx methods should allow it, but then again, most folks who care about this level of detail can probably use capture_request.

@naushir
Copy link
Collaborator

naushir commented Nov 9, 2023

I can see this being useful. LGTM.

@davidplowman davidplowman merged commit fe755dc into next Nov 10, 2023
4 checks passed
@davidplowman davidplowman deleted the timestamp-capture branch November 10, 2023 13:22
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