Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Retrieve dataset by name instead of importing object (build a dataset index) #182

Closed
mlincett opened this issue Aug 4, 2022 · 1 comment · Fixed by #211
Closed

Retrieve dataset by name instead of importing object (build a dataset index) #182

mlincett opened this issue Aug 4, 2022 · 1 comment · Fixed by #211
Assignees

Comments

@mlincett
Copy link
Collaborator

mlincett commented Aug 4, 2022

Is your feature request related to a problem? Please describe.
Aside from navigating the source code, there is no easy way (as far as I know) to see which data sets are available.
In addition, datasets must be always retrieved by explicit imports and cannot be loaded by name (see #178).

Describe the solution you'd like
Every created dataset could take care of indexing itself in a central location. Such index will be exposed to the user, that will be allowed to load a dataset by key or name.

This could be achieved by instantiating a dictionary inside a module under flarestack.data, to which every dataset will "register" (more or less like it is done do for subclasses of the core modules).

Describe alternatives you've considered
Well, the current design works but it's less user friendly and provides little flexibility.

@robertdstein
Copy link
Member

I love this idea!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants