Added return methods for URL-references to the pushed dataset
MEMBER
Hi ! Maybe we'd need to align with `transformers` and other libraries that implement `push_to_hub` to agree on what it should return.

e.g. in `transformers` the typing says it returns a string, but in practice it returns a `CommitInfo`.

Therefore I'd not add an output to `push_to_hub` here unless we had a chance to discuss more broadly.

Anyway in my opinion it should no just return the URL of the repository, but ideally the URL at the revision where the data were pushed
2023-02-21T13:05:57Z
https://github.com/huggingface/datasets/pull/5545#issuecomment-1438454736
https://api.github.com/repos/huggingface/datasets/issues/5545
IC_kwDODunzps5VvRPQ
https://api.github.com/repos/huggingface/datasets/issues/comments/1438454736/reactions
2023-02-21T13:05:57Z
https://api.github.com/repos/huggingface/datasets/issues/comments/1438454736
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
COLLABORATOR
Perhaps a mixin or something similar could be defined on the `hfh` side to ensure the `push_to_hub` API is aligned across our projects. 

PS: this would also mean that the PRs such as https://github.com/huggingface/datasets/pull/5528 would no longer be our responsibility

cc @Wauplin 
2023-02-21T14:08:26Z
https://github.com/huggingface/datasets/pull/5545#issuecomment-1438557414
https://api.github.com/repos/huggingface/datasets/issues/5545
IC_kwDODunzps5VvqTm
https://api.github.com/repos/huggingface/datasets/issues/comments/1438557414/reactions
2023-02-21T14:08:26Z
https://api.github.com/repos/huggingface/datasets/issues/comments/1438557414
https://avatars.githubusercontent.com/u/47462742?v=4
https://api.github.com/users/mariosasko/events{/privacy}
https://api.github.com/users/mariosasko/followers
https://api.github.com/users/mariosasko/following{/other_user}
https://api.github.com/users/mariosasko/gists{/gist_id}
https://github.com/mariosasko
mariosasko
MDQ6VXNlcjQ3NDYyNzQy
https://api.github.com/users/mariosasko/orgs
https://api.github.com/users/mariosasko/received_events
https://api.github.com/users/mariosasko/repos
https://api.github.com/users/mariosasko/starred{/owner}{/repo}
https://api.github.com/users/mariosasko/subscriptions
User
https://api.github.com/users/mariosasko
public
NONE
I agree, with universability and the idea is more about returning at least something that references where to find the uploaded file/model or otherwise. 

Ideally, the referenced PR would work.
2023-02-21T14:16:08Z
https://github.com/huggingface/datasets/pull/5545#issuecomment-1438568520
https://api.github.com/repos/huggingface/datasets/issues/5545
IC_kwDODunzps5VvtBI
https://api.github.com/repos/huggingface/datasets/issues/comments/1438568520/reactions
2023-02-21T14:16:08Z
https://api.github.com/repos/huggingface/datasets/issues/comments/1438568520
https://avatars.githubusercontent.com/u/25269220?v=4
https://api.github.com/users/davidberenstein1957/events{/privacy}
https://api.github.com/users/davidberenstein1957/followers
https://api.github.com/users/davidberenstein1957/following{/other_user}
https://api.github.com/users/davidberenstein1957/gists{/gist_id}
https://github.com/davidberenstein1957
davidberenstein1957
MDQ6VXNlcjI1MjY5MjIw
https://api.github.com/users/davidberenstein1957/orgs
https://api.github.com/users/davidberenstein1957/received_events
https://api.github.com/users/davidberenstein1957/repos
https://api.github.com/users/davidberenstein1957/starred{/owner}{/repo}
https://api.github.com/users/davidberenstein1957/subscriptions
User
https://api.github.com/users/davidberenstein1957
public
MEMBER
imo this would be a good use case to just use `huggingface_hub` and align to what we do there :)
2023-02-21T14:17:28Z
https://github.com/huggingface/datasets/pull/5545#issuecomment-1438570467
https://api.github.com/repos/huggingface/datasets/issues/5545
IC_kwDODunzps5Vvtfj
https://api.github.com/repos/huggingface/datasets/issues/comments/1438570467/reactions
2023-02-21T14:17:28Z
https://api.github.com/repos/huggingface/datasets/issues/comments/1438570467
https://avatars.githubusercontent.com/u/326577?v=4
https://api.github.com/users/julien-c/events{/privacy}
https://api.github.com/users/julien-c/followers
https://api.github.com/users/julien-c/following{/other_user}
https://api.github.com/users/julien-c/gists{/gist_id}
https://github.com/julien-c
julien-c
MDQ6VXNlcjMyNjU3Nw==
https://api.github.com/users/julien-c/orgs
https://api.github.com/users/julien-c/received_events
https://api.github.com/users/julien-c/repos
https://api.github.com/users/julien-c/starred{/owner}{/repo}
https://api.github.com/users/julien-c/subscriptions
User
https://api.github.com/users/julien-c
public
NONE
@mariosasko, can you give me some pointers to where I might help implementing this for the `huggingface-hub`?
2023-08-11T09:46:28Z
https://github.com/huggingface/datasets/pull/5545#issuecomment-1674475728
https://api.github.com/repos/huggingface/datasets/issues/5545
IC_kwDODunzps5jznjQ
https://api.github.com/repos/huggingface/datasets/issues/comments/1674475728/reactions
2023-08-11T09:46:28Z
https://api.github.com/repos/huggingface/datasets/issues/comments/1674475728
https://avatars.githubusercontent.com/u/25269220?v=4
https://api.github.com/users/davidberenstein1957/events{/privacy}
https://api.github.com/users/davidberenstein1957/followers
https://api.github.com/users/davidberenstein1957/following{/other_user}
https://api.github.com/users/davidberenstein1957/gists{/gist_id}
https://github.com/davidberenstein1957
davidberenstein1957
MDQ6VXNlcjI1MjY5MjIw
https://api.github.com/users/davidberenstein1957/orgs
https://api.github.com/users/davidberenstein1957/received_events
https://api.github.com/users/davidberenstein1957/repos
https://api.github.com/users/davidberenstein1957/starred{/owner}{/repo}
https://api.github.com/users/davidberenstein1957/subscriptions
User
https://api.github.com/users/davidberenstein1957
public
CONTRIBUTOR
> @mariosasko: Perhaps a mixin or something similar could be defined on the hfh side to ensure the push_to_hub API is aligned across our projects.

> @julien-c: imo this would be a good use case to just use huggingface_hub and align to what we do there :)

I (finally) opened a PR to harmonize return types: https://github.com/huggingface/huggingface_hub/pull/1921. It should hopefully be shipped in next release later this week (:crossed_fingers:). 
2023-12-18T16:57:55Z
https://github.com/huggingface/datasets/pull/5545#issuecomment-1861056492
https://api.github.com/repos/huggingface/datasets/issues/5545
IC_kwDODunzps5u7Xfs
https://api.github.com/repos/huggingface/datasets/issues/comments/1861056492/reactions
2023-12-18T16:57:55Z
https://api.github.com/repos/huggingface/datasets/issues/comments/1861056492
https://avatars.githubusercontent.com/u/11801849?v=4
https://api.github.com/users/Wauplin/events{/privacy}
https://api.github.com/users/Wauplin/followers
https://api.github.com/users/Wauplin/following{/other_user}
https://api.github.com/users/Wauplin/gists{/gist_id}
https://github.com/Wauplin
Wauplin
MDQ6VXNlcjExODAxODQ5
https://api.github.com/users/Wauplin/orgs
https://api.github.com/users/Wauplin/received_events
https://api.github.com/users/Wauplin/repos
https://api.github.com/users/Wauplin/starred{/owner}{/repo}
https://api.github.com/users/Wauplin/subscriptions
User
https://api.github.com/users/Wauplin
public
Hi,

I was missing the ability to easily open the pushed dataset and it seemed like a quick fix.
Maybe we also want to log this info somewhere, but let me know if I need to add that too.

Cheers,
David