Remove `.valid` from `/valid` endpoint?
We recently added to fields to `/valid`:
- `viewer`: all the datasets that have a valid dataset viewer
- `preview`: all the datasets that don't have a valid dataset viewer, but have a dataset preview

And the Hub does not use the original field `valid` anymore. We still fill it with the union of both sets.

Should we remove it, as it doubles the size of the response and increases the response time, with no benefit? cc @huggingface/datasets-server 

Note that it's used in the notebooks (https://github.com/search?q=org%3Ahuggingface+datasets-server.huggingface.co+repo%3Ahuggingface%2Fnotebooks&type=code), for example, so it is a breaking change.

I would vote in favor of removing it, and updating the notebooks (and the docs obviously).
Remove `.valid` from `/valid` endpoint?: We recently added to fields to `/valid`:
- `viewer`: all the datasets that have a valid dataset viewer
- `preview`: all the datasets that don't have a valid dataset viewer, but have a dataset preview

And the Hub does not use the original field `valid` anymore. We still fill it with the union of both sets.

Should we remove it, as it doubles the size of the response and increases the response time, with no benefit? cc @huggingface/datasets-server 

Note that it's used in the notebooks (https://github.com/search?q=org%3Ahuggingface+datasets-server.huggingface.co+repo%3Ahuggingface%2Fnotebooks&type=code), for example, so it is a breaking change.

I would vote in favor of removing it, and updating the notebooks (and the docs obviously).