tobitege
tobitege
> hey there, thank you! have incorporated your suggestions, kindly do have a look! Thanks so much! Looks good to me, but want to have @neubig look over it for...
Fyi, edited title and top post for clarity 😃
@enyst - I now reverted the changes related to `latest_session_id` in `manager.py` as per our convo.
> @tobitege this looks cool! > > My hope is that (per #3794) we can avoid having an extra CLIENT_READY, and just make sure that AGENT_READY means the runtime is...
> > Maybe it's semantics here, but the agent shouldn't "appear" ready before the client. We can disguise the state then accordingly, I guess. > > 100% agree here. This...
Finally fixed the "blank page" error so that at least an actual status message can be displayed. Right now, the first message that _would_ appear is in `EventStreamRuntime`, at the...
> > Maybe it's semantics here, but the agent shouldn't "appear" ready before the client. We can disguise the state then accordingly, I guess. > > 100% agree here. This...
Seems to work now, but need advice on the UI changes if they need further refinements.
> Maybe you can remove the "`(may take up to 10 seconds)`" text entirely and leave in the "`Initializing agent`" since the status message already mentions the estimated time (`Preparing...
How the heck is there a lint error after going through pre-commit checks? 😔