Fix allociné dataset configuration
MEMBER
Actually when there is only one configuration, then you don't need to specify the configuration in `load_dataset`. You can run:
```python
dataset = load_dataset('allocine')
```
and it works.

Maybe we should take that into account in the nlp viewer @srush ?
2020-06-16T08:11:26Z
https://github.com/huggingface/datasets/pull/271#issuecomment-644607895
https://api.github.com/repos/huggingface/datasets/issues/271
MDEyOklzc3VlQ29tbWVudDY0NDYwNzg5NQ==
https://api.github.com/repos/huggingface/datasets/issues/comments/644607895/reactions
2020-06-16T08:11:26Z
https://api.github.com/repos/huggingface/datasets/issues/comments/644607895
https://avatars.githubusercontent.com/u/42851186?v=4
https://api.github.com/users/lhoestq/events{/privacy}
https://api.github.com/users/lhoestq/followers
https://api.github.com/users/lhoestq/following{/other_user}
https://api.github.com/users/lhoestq/gists{/gist_id}
https://github.com/lhoestq
lhoestq
MDQ6VXNlcjQyODUxMTg2
https://api.github.com/users/lhoestq/orgs
https://api.github.com/users/lhoestq/received_events
https://api.github.com/users/lhoestq/repos
https://api.github.com/users/lhoestq/starred{/owner}{/repo}
https://api.github.com/users/lhoestq/subscriptions
User
https://api.github.com/users/lhoestq
public
CONTRIBUTOR
@lhoestq Just to understand the exact semantics. Are you suggesting that if there is exactly 1 configuration I should not show the configuration menu and just treat it as if there were 0 configurations? 
2020-06-16T14:11:28Z
https://github.com/huggingface/datasets/pull/271#issuecomment-644791494
https://api.github.com/repos/huggingface/datasets/issues/271
MDEyOklzc3VlQ29tbWVudDY0NDc5MTQ5NA==
https://api.github.com/repos/huggingface/datasets/issues/comments/644791494/reactions
2020-06-16T14:11:28Z
https://api.github.com/repos/huggingface/datasets/issues/comments/644791494
https://avatars.githubusercontent.com/u/35882?v=4
https://api.github.com/users/srush/events{/privacy}
https://api.github.com/users/srush/followers
https://api.github.com/users/srush/following{/other_user}
https://api.github.com/users/srush/gists{/gist_id}
https://github.com/srush
srush
MDQ6VXNlcjM1ODgy
https://api.github.com/users/srush/orgs
https://api.github.com/users/srush/received_events
https://api.github.com/users/srush/repos
https://api.github.com/users/srush/starred{/owner}{/repo}
https://api.github.com/users/srush/subscriptions
User
https://api.github.com/users/srush
public
MEMBER
The configuration menu is fine imo.
It was more about the code snippet presented in the viewer.
For example for Allociné it currently shows this snippet to load the dataset:
```python
!pip install nlp
from nlp import load_dataset
dataset = load_dataset('allocine', 'allocine')
```
However for datasets with one or zero configurations, the second argument in `load_dataset` is optional. For Allociné, that has one configuration, we can expect to show instead:
```python
!pip install nlp
from nlp import load_dataset
dataset = load_dataset('allocine')
```
2020-06-16T14:22:24Z
https://github.com/huggingface/datasets/pull/271#issuecomment-644797824
https://api.github.com/repos/huggingface/datasets/issues/271
MDEyOklzc3VlQ29tbWVudDY0NDc5NzgyNA==
https://api.github.com/repos/huggingface/datasets/issues/comments/644797824/reactions
2020-06-16T14:22:24Z
https://api.github.com/repos/huggingface/datasets/issues/comments/644797824
https://avatars.githubusercontent.com/u/42851186?v=4
https://api.github.com/users/lhoestq/events{/privacy}
https://api.github.com/users/lhoestq/followers
https://api.github.com/users/lhoestq/following{/other_user}
https://api.github.com/users/lhoestq/gists{/gist_id}
https://github.com/lhoestq
lhoestq
MDQ6VXNlcjQyODUxMTg2
https://api.github.com/users/lhoestq/orgs
https://api.github.com/users/lhoestq/received_events
https://api.github.com/users/lhoestq/repos
https://api.github.com/users/lhoestq/starred{/owner}{/repo}
https://api.github.com/users/lhoestq/subscriptions
User
https://api.github.com/users/lhoestq
public
CONTRIBUTOR
> Actually when there is only one configuration, then you don't need to specify the configuration in `load_dataset`. You can run:
> 
> ```python
> dataset = load_dataset('allocine')
> ```
> 
> and it works.
> 
> Maybe we should take that into account in the nlp viewer @srush ?

Oh ok, I didn't expect it would work! 

Anyway, I think it's intrinsically better to simply remove the optional parameter. 
The dummy data folder architecture seems also more logical this way.

2020-06-16T17:07:05Z
https://github.com/huggingface/datasets/pull/271#issuecomment-644893894
https://api.github.com/repos/huggingface/datasets/issues/271
MDEyOklzc3VlQ29tbWVudDY0NDg5Mzg5NA==
https://api.github.com/repos/huggingface/datasets/issues/comments/644893894/reactions
2020-06-16T17:07:05Z
https://api.github.com/repos/huggingface/datasets/issues/comments/644893894
https://avatars.githubusercontent.com/u/37028092?v=4
https://api.github.com/users/TheophileBlard/events{/privacy}
https://api.github.com/users/TheophileBlard/followers
https://api.github.com/users/TheophileBlard/following{/other_user}
https://api.github.com/users/TheophileBlard/gists{/gist_id}
https://github.com/TheophileBlard
TheophileBlard
MDQ6VXNlcjM3MDI4MDky
https://api.github.com/users/TheophileBlard/orgs
https://api.github.com/users/TheophileBlard/received_events
https://api.github.com/users/TheophileBlard/repos
https://api.github.com/users/TheophileBlard/starred{/owner}{/repo}
https://api.github.com/users/TheophileBlard/subscriptions
User
https://api.github.com/users/TheophileBlard
public
CONTRIBUTOR
Fixed in the viewer. Checked that allocine works.
2020-06-16T18:00:14Z
https://github.com/huggingface/datasets/pull/271#issuecomment-644920034
https://api.github.com/repos/huggingface/datasets/issues/271
MDEyOklzc3VlQ29tbWVudDY0NDkyMDAzNA==
https://api.github.com/repos/huggingface/datasets/issues/comments/644920034/reactions
2020-06-16T18:00:14Z
https://api.github.com/repos/huggingface/datasets/issues/comments/644920034
https://avatars.githubusercontent.com/u/35882?v=4
https://api.github.com/users/srush/events{/privacy}
https://api.github.com/users/srush/followers
https://api.github.com/users/srush/following{/other_user}
https://api.github.com/users/srush/gists{/gist_id}
https://github.com/srush
srush
MDQ6VXNlcjM1ODgy
https://api.github.com/users/srush/orgs
https://api.github.com/users/srush/received_events
https://api.github.com/users/srush/repos
https://api.github.com/users/srush/starred{/owner}{/repo}
https://api.github.com/users/srush/subscriptions
User
https://api.github.com/users/srush
public
MEMBER
Awesome thanks :)

Closing this.
2020-06-18T07:41:20Z
https://github.com/huggingface/datasets/pull/271#issuecomment-645841086
https://api.github.com/repos/huggingface/datasets/issues/271
MDEyOklzc3VlQ29tbWVudDY0NTg0MTA4Ng==
https://api.github.com/repos/huggingface/datasets/issues/comments/645841086/reactions
2020-06-18T07:41:20Z
https://api.github.com/repos/huggingface/datasets/issues/comments/645841086
https://avatars.githubusercontent.com/u/42851186?v=4
https://api.github.com/users/lhoestq/events{/privacy}
https://api.github.com/users/lhoestq/followers
https://api.github.com/users/lhoestq/following{/other_user}
https://api.github.com/users/lhoestq/gists{/gist_id}
https://github.com/lhoestq
lhoestq
MDQ6VXNlcjQyODUxMTg2
https://api.github.com/users/lhoestq/orgs
https://api.github.com/users/lhoestq/received_events
https://api.github.com/users/lhoestq/repos
https://api.github.com/users/lhoestq/starred{/owner}{/repo}
https://api.github.com/users/lhoestq/subscriptions
User
https://api.github.com/users/lhoestq
public
This is a patch for #244. According to the [live nlp viewer](url), the Allociné dataset must be loaded with :
```python
dataset = load_dataset('allocine', 'allocine')
```
This is redundant, as there is only one "dataset configuration", and should only be:
```python
dataset = load_dataset('allocine')
```

This is my mistake, because the code for [`allocine.py`](https://github.com/huggingface/nlp/blob/master/datasets/allocine/allocine.py) was inspired by [`imdb.py`](https://github.com/huggingface/nlp/blob/master/datasets/imdb/imdb.py), which also force the user to specify the "dataset configuration" (even if there is only one).

I believe this PR should solve this issue, making the Allociné dataset more convenient to use.