-
Notifications
You must be signed in to change notification settings - Fork 33
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
Mapping multiple LipdSeries objects #267
Comments
That sounds good @alexkjames , but I would re-use as much of @fzhu2e 's |
This may fall under PyLipd outputs. |
revisit after #270 is closed |
@alexkjames , the |
No. |
True ; and issue #419 will address that. |
Is your feature request related to a problem? Please describe.
A lot of my paleoclimate analysis within Pyleoclim begins with loading a big folder full of Lipd files into a pyleo.Lipd object, turning them into LipdSeries objects with .toLipdSeriesList, parsing those results for specific features, and looking at what comes out. Right now all of this is easy except for the looking at what comes out part. There's no way to take a big list or group of LipdSeries objects and map them the way we can with pyleo.Lipd objects. There is also no way to easily get those LipdSeries objects back into a pyleo.Lipd object, so I have to come up with somewhat convoluted workarounds just to map my parsed dataset.
Describe the solution you'd like
I think the best option would be to create a new class called MultipleLipdSeries. It would be a child of MultipleSeries, and offer the ability to easily map collections of LipdSeries objects, perform spatial analysis, filter by archive, etc. The main thing that would be nice would be the mapping capability, but there's a whole host of potential use cases.
Describe alternatives you've considered
Making it possible to feed the lipd_ts attribute from LipdSeries objects back into a pyleo.Lipd object. Right now the structure of the lipd_ts attribute isn't what pyleo.Lipd expects of lipd dictionaries, so none of the functionalities work.
The text was updated successfully, but these errors were encountered: