-
Notifications
You must be signed in to change notification settings - Fork 394
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
Logging API for "temporal batches" #4184
Labels
Comments
jleibs
added
enhancement
New feature or request
user-request
This is a pressing issue for one of our users
labels
Nov 8, 2023
This was referenced Apr 12, 2024
|
Missing things to complete this workstream:
|
We should do a session on naming and usabillity of this prior to release 0.18 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
When translating existing time-series plots to Rerun our logging APIs have very poor ergonomics.
A user must iterate through every sample, calling set_time with the time value, and then logging the individual scalars.
The proposal from:
Would let us optimize this a bit by passing the time values into the log-calls, but will still be order-N which can be quite painful for large time-series.
A direct temporal batch logging API would instead let us collapse this into a single call. For example:
This would let us dramatically optimize the ingestion by letting us create a pre-batched uniform buffer of scalars that looks like what the eventual output of the batcher is going to be anyways.
The text was updated successfully, but these errors were encountered: