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

eval loading state should be true as soon as it's data method is called #73

Open
ShiboSoftwareDev opened this issue Dec 18, 2024 · 1 comment

Comments

@ShiboSoftwareDev
Copy link

current behavior is that the loader starts spinning only when the task method for that eval is called, so when the data step takes too long you will not see immediate feedback and it's confusing
image

@mattpocock
Copy link
Owner

mattpocock commented Dec 27, 2024

This is non-trivial, and cosmetic, so will likely not push for this soon. The best way to mitigate this is to make the data functions resolve quickly. You shouldn't be doing LLM calls inside data, since traces won't be captured.

But - I agree that it should work in the way described.

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

2 participants