Replies: 1 comment
-
I can see the argument either way. I lean slightly toward (at least trying) Coordinator because it feels like we have a fair bit going on in the concept space and not reusing a term we already use in a lot of places to denote different things might make it simpler to grok the overall picture. Just brainstorming, but what would happen if we renamed |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Renaming would reduce confusion about the difference between the role of the agent component spec file (i.e.,
agentos.ini
) and the component (that we currently call "agent") that coordinates all the other components.However, the role that component currently plays maps pretty well to the classic definition of an RL agent. I.e., it can be trained, and run. It uses trainer and policy. Etc.
I think once we support hierarchical components (e.g., via component dependencies), the name Agent might be a better fit than Coordinator.
Beta Was this translation helpful? Give feedback.
All reactions