Robert Brennan
Robert Brennan
@Umpire2018 we do actually do a bit of differentiation here: https://github.com/OpenDevin/OpenDevin/blob/f8d4b1ab0d85edfc584d0b4cd1960c3968b5f336/.github/workflows/ghcr.yml#L68 We don't `--push` if it's a fork. That said, I was considering switching this behavior so that we can...
HYG: https://docs.google.com/document/d/1GjyNU_rKAeX5ugFKDu8PsVY8IyDx-CxiTOJRoNRaEFU/edit
All good questions! Thanks @theckman. Even just having a function like `refresh_token, err := refreshAccessToken(access_token, refresh_token)` would be nice. There are also probably interesting ways to do an automatic refresh...
> One comment: it might be a bit rough to summarize the entire user intent into "latest user message". The latest user message may be a clarification message or something...
Re: inheritance, I added some more notes to the docstring on Plan. Ultimately, we have a tree structure representing the agent's plan, e.g. * Do x * Do x' *...
We need to e.g. override the `__init__` so that it doesn't try and set an `id` etc
That works! The bigger issue I think are the helper methods that are defined on Plan: `get_task_by_id`, `add_subtask`, and `set_subtask_state`. Calling these on a non-root Task would cause problems. Two...
🤔 what if we just rename `Plan` to `RootTask`?
I'm not sure it'll fix it, but I'm going to try setting a higher UID_MAX here: https://github.com/OpenDevin/OpenDevin/pull/1788
#1788 is merged--can you try using `main` to see if it helps?