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.

Sign up for GitHub

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

Jump to bottom

Don't try to include binary cells in the /rows responses #3052

Open
severo opened this issue Aug 28, 2024 · 0 comments
Open

Don't try to include binary cells in the /rows responses #3052

severo opened this issue Aug 28, 2024 · 0 comments
Labels
improvement / optimization P1 Not as needed as P0, but still important/wanted

Comments

@severo
Copy link
Collaborator

severo commented Aug 28, 2024

https://huggingface.co/datasets/frutiemax/themoviedb_posters/discussions/2

I have some columns that the viewer do not need to load i.e. T5 prompt embeds and VAE features. Currently, the viewer freezes because the dataset is too big to load. Thanks.

Capture d’écran 2024-08-28 à 12 31 40

https://huggingface.co/datasets/frutiemax/themoviedb_posters/viewer/default/train lags forever to render the following:

Capture d’écran 2024-08-28 à 12 32 50

We should ignore/hide the binary cells (bytes (5KB) for example)
@severo severo added improvement / optimization P1 Not as needed as P0, but still important/wanted labels Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
improvement / optimization P1 Not as needed as P0, but still important/wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@severo