-
Notifications
You must be signed in to change notification settings - Fork 10
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
options for loading multi-sector light curves #90
Comments
You can currently call We could extend the lcviz parser to accept the output from
or
we could support:
I'd also like to some day have a split/join functionality as a plugin in jdaviz - do you think that would be helpful? Especially if from the MAST-side, you'll need to hardcode one of these options and the user won't be able to choose in advance. |
Yeah, I like the idea of supporting this syntax I think we'd also like the ability to pass in a general list of lightcurve files, with the In our case, what I want to avoid is having lcviz download any files again on the server-side. Ideally lcviz could be smart enough to know when it's running on MAST to lookup the isilon paths instead of downloading the files, but that might be a big ask. I think we could add some logic in our notebook to do that conversion. |
oh and yeah, I do think a plugin for splitting/joining would be helpful. If we didn't expose that interaction in the UI, I think the only option would be to write an event handler that calls a notebook function to do it programmatically, and reload the viewer. |
First stab at stitch functionality as a plugin (after loading into lcviz - haven't forgotten about the other use-case) is implemented in #107 - let me know if you have any thoughts/concerns. |
Great. I'll be able to take a closer look at this next week. |
Is there an option for loading lightcurves from multiple sectors of an observation, or from multiple observations? Is this something that is feasible, or does it have any performance drawbacks?
I could imagine a few different workflows:
If I run the
download_all
method here and load each file in LCviz, how performant is that?The text was updated successfully, but these errors were encountered: