-
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
guide: consider/explain exp push --run-cache
(Sharing Exps)
#2687
Comments
I'd ask the core team first to understand the intention of |
It does the same thing as In a collaborative environment, if users are sharing experiments, they may want to run the experiments themselves (whether it's with or without modifications). In these cases, it is still beneficial to have pushed/pulled run cache so that stages which someone else has already reproduced do not need to be re-run. |
It is similar to what I understood. Run cache keeps the intermediate stage outputs so that other |
Run cache associates input state (a defined stage itself + its dependencies) with an output state (outputs) for a given stage. If DVC has the run-cache information available, and see's that the user is trying to run a stage where the current input state is in the cache, it will just skip the execution and Run cache is not really related to intermediate stage outputs https://dvc.org/doc/user-guide/experiment-management#automatic-log-of-stage-runs-run-cache |
:) My emphasis is not on "intermediate", but the run cache keeps the stage state and updates the workspace without running if the same inputs + dependencies are given. |
My point was in response to:
Run cache does not include any outputs/data from your workspace. You can look at the structure of the run cache yourself ( |
exp push --run-cache
(Sharing Exps)
So would you put this info in the Exp Sharing guide @iesahin ? Or somewhere else. What's the summary or key thing to document here? Thanks |
I think we may to tell this both in Running and Sharing, describing different aspects, but I still could not test this. |
K well maybe add a checkbox in #2768 instead and close this 🙂 Up to you |
As some of the items in #2618 seems to need more tests and the completion of rearrangement in #2654, I'll be updating the requirements for the document here.
This issue is a cascading issue on top of #2367Todo
--run-cache
fordvc exp push
to see its effectsindex.md
with links to "sharing experiments"The text was updated successfully, but these errors were encountered: