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

Extract interface from UngriddedData, implement alternative #1515

Open
heikoklein opened this issue Feb 11, 2025 · 0 comments · May be fixed by #1517
Open

Extract interface from UngriddedData, implement alternative #1515

heikoklein opened this issue Feb 11, 2025 · 0 comments · May be fixed by #1517
Assignees
Milestone

Comments

@heikoklein
Copy link
Member

Is your feature request related to a problem? Please describe.
UngriddedData is very difficult to read and it is unclear what is the programming interface and what is internal necessity. A interface with only the needed functionality should be extracted, with the current UngriddedData as an implementation. Another implementation should be added, with the hope to improve readability and memory consumption. In the long range, several implementations could co-exist, e.g. depending on different reader e.g. pyaro-base (timeseries/stations) or satellite data.

Describe the solution you would like to see
A clear and concise description of what you want to happen.

  • Extract an interface
  • Implement current UngriddedData as implementation of that interface
  • Add another implementation
  • Compare implementations
  • Decide, which implementation to use for the next release
@heikoklein heikoklein added this to the m2025-03 milestone Feb 11, 2025
@heikoklein heikoklein self-assigned this Feb 11, 2025
@heikoklein heikoklein changed the title Extract interface from Ungridded, implement alternative Extract interface from UngriddedData, implement alternative Feb 11, 2025
@heikoklein heikoklein linked a pull request Feb 11, 2025 that will close this issue
9 tasks
@heikoklein heikoklein linked a pull request Feb 11, 2025 that will close this issue
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant