microsoft / ToolTalk

Evaluating tool-augmented LLMs in conversation settings
MIT License
72 stars 15 forks source link

Fix: prevent double-login, actually #12

Closed chokkyvista closed 6 months ago

chokkyvista commented 6 months ago

As neither UserLogin nor RegisterUser has their requires_auth attribute set to True, the current code path for the "double-login" check will actually never be reached and activated (due to the following precondition https://github.com/microsoft/ToolTalk/blob/a8d8befb1ea098cd9fc363a7515028b0b2aeb903/src/tooltalk/evaluation/tool_executor.py#L95).

This change will fix it and also make the error message a little bit clearer.