Track compute time in cache metadata - #17
Open
dbeach24 wants to merge 2 commits into
Open
Conversation
Copy link
Copy Markdown
Sorry, something went wrong.
Include per-entry compute time in cache metadata and in summary table.
Copy link
Copy Markdown
Member
|
Somehow we missed this PR 4 years ago :) (I guess github notifications weren't really good). I can somehow see that this is useful. Can someone describe how to use this productively? Put caching everywhere and remove it on fast executing queries. I guess the problem is also using time and cache magics at the same time. |
Sorry, something went wrong.
Copy link
Copy Markdown
Author
|
Hi Till. When deciding whether to cache something and the cost of
recalculating, the time is often highly relevant and trivial to store. At
the time I wrote this, I was doing research from several notebooks, and
wanted to keep track of the time cost if I made changes to my data or
parameters.
…On Sat, Oct 8, 2022 at 9:13 AM Till Riedel ***@***.***> wrote:
Somehow we missed this PR 4 years ago :) (I guess github notifications
weren't really good). I can somehow see that this is useful. Can someone
describe how to use this productively? Put caching everywhere and remove it
on fast executing queries. I guess the problem is also using time and cache
magics at the same time.
—
Reply to this email directly, view it on GitHub
<#17 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACYRCIZ6IJSJEPLD6DQTSKLWCEUIZANCNFSM4GGEW2GQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Sorry, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Keep track of compute time in the cache metadata