Skip to content
This repository has been archived by the owner on May 28, 2022. It is now read-only.

merge into MLDatasets? #1

Closed
CarloLucibello opened this issue Nov 16, 2020 · 2 comments
Closed

merge into MLDatasets? #1

CarloLucibello opened this issue Nov 16, 2020 · 2 comments

Comments

@CarloLucibello
Copy link

Hi,
there is a lot of nice work here, would you consider merging it into MLDatasets.jl?
We have recently removed datasets from Flux and point users to MLDatasets.jl, it would be nice to have a central repository there.

@CarloLucibello
Copy link
Author

bump on this. I can move over the code
(I wouldn't do anything more than a copy and paste, homogenization can come later)
although is better if @lorenzoh does it to preserve authorship

cc @johnnychen94 @darsnack

@lorenzoh
Copy link
Owner

Might be better to move the datasets code from https://github.com/lorenzoh/FastAI.jl/tree/master/src/datasets into MLDatasets.jl as it currently supports more datasets already, what do you think?

DLDatasets.jl has some pose estimation datasets which could be ported over, though one of them depends on PyCall which is a big dependency.

I can get a pull request started over the next few days 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants