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

Rename Run to Tracker #396

Open
andyk opened this issue Jun 9, 2022 · 0 comments
Open

Rename Run to Tracker #396

andyk opened this issue Jun 9, 2022 · 0 comments

Comments

@andyk
Copy link
Contributor

andyk commented Jun 9, 2022

We could go further than #335 did and instead rename pcs.Run to pcs.Tracker, and then make Component.run() and Command.run() return a Tracker.

  • a Tracker is like a logger where everything that it logs is associated with a unique ID
  • a Tracker maps to an MLflow Run.
  • Trackers can be nested, in which case they show up as nested in the MLflow UI
  • Children trackers should have links to parents for sure (they do today)
  • Should parents have links to children?
  • A Tracker is a Component
  • Run is now only a verb in PCS/AgentOS (not a noun anymore) and corresponds to running an MLflow Project.
  • In this case we would rename agentos.AgentRun to agentos.AgentTracker
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

No branches or pull requests

1 participant