More consistent naming #2611
More consistent naming #2611
Conversation
There was a problem hiding this comment.
Thank you for doing this polish, @mariosasko! Great work.
I left a few small remarks in the changes.
README.md
Outdated
| @@ -25,7 +25,7 @@ | |||
| <a href="https://zenodo.org/badge/latestdoi/250213286"><img src="https://zenodo.org/badge/250213286.svg" alt="DOI"></a> | |||
| </p> | |||
|
|
|||
| `🤗Datasets` is a lightweight library providing **two** main features: | |||
| `🤗 Datasets` is a lightweight library providing **two** main features: | |||
There was a problem hiding this comment.
IMHO, here and everywhere else in this doc needs to be either datasets or 🤗 Datasets - no backticks!
backticks are for the module name, which is not 🤗 Datasets but datasets
There was a problem hiding this comment.
since there is only one python package called datasets if you go with the former, you probably don't need the emoji.
I'd say to be consistent with transformers docs, most of the time ideally it should be: 🤗 Datasets (as in 🤗 Transformers).
There was a problem hiding this comment.
Unless you are writing:
Main differences between
🤗 Datasetsandtfds
a few paras below, which looks very unbalanced.
Here you probably want to go for one of:
datasetsandtfds- 🤗 Datasets and TensorFlow
tfds
Just a suggestion of course...
docs/source/index.rst
Outdated
|
|
||
| 🤗Datasets originated from a fork of the awesome TensorFlow Datasets and the HuggingFace team want to deeply thank the TensorFlow Datasets team for building this amazing library. More details on the differences between 🤗Datasets and tfds can be found in the section Main differences between 🤗Datasets and tfds. | ||
| 🤗 Datasets originated from a fork of the awesome TensorFlow Datasets and the HuggingFace team want to deeply thank the TensorFlow Datasets team for building this amazing library. More details on the differences between 🤗 Datasets and tfds can be found in the section Main differences between 🤗 Datasets and tfds. |
There was a problem hiding this comment.
| 🤗 Datasets originated from a fork of the awesome TensorFlow Datasets and the HuggingFace team want to deeply thank the TensorFlow Datasets team for building this amazing library. More details on the differences between 🤗 Datasets and tfds can be found in the section Main differences between 🤗 Datasets and tfds. | |
| 🤗 Datasets originated from a fork of the awesome TensorFlow Datasets and the HuggingFace team want to deeply thank the TensorFlow Datasets team for building this amazing library. More details on the differences between 🤗 Datasets and tfds can be found in the section Main differences between 🤗 Datasets and `tfds`. |
but see my earlier comment.
There was a problem hiding this comment.
I think it's fine this way. We could also change it to "Tensorflow Datasets"
Co-authored-by: Stas Bekman <stas00@users.noreply.github.com>
Co-authored-by: Stas Bekman <stas00@users.noreply.github.com>
docs/source/share_dataset.rst
Outdated
| @@ -103,7 +103,7 @@ Push the changes to your account using: | |||
| Sharing a "community provided" dataset | |||
| ----------------------------------------- | |||
|
|
|||
| In this page, we will show you how to share a dataset with the community on the `datasets hub <https://huggingface.co/datasets>`__. | |||
| In this page, we will show you how to share a dataset with the community on the `Datasets Hub <https://huggingface.co/datasets>`__. | |||
There was a problem hiding this comment.
For consistency may want to add 🤗 here and in all the instances of Datasets Hub below?
As per @stas00's suggestion in #2500, this PR inserts a space between the logo and the lib name (
🤗Datasets->🤗 Datasets) for consistency with the Transformers lib. Additionally, more consistent names are used for Datasets Hub, etc.