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

ENH: Add more solar irradiance proxies #124

Open
1 of 5 tasks
jklenzing opened this issue Feb 9, 2021 · 11 comments · Fixed by pysat/pysatSpaceWeather#108
Open
1 of 5 tasks

ENH: Add more solar irradiance proxies #124

jklenzing opened this issue Feb 9, 2021 · 11 comments · Fixed by pysat/pysatSpaceWeather#108
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@jklenzing
Copy link
Member

jklenzing commented Feb 9, 2021

Description

Numerous solar proxies are available through https://lasp.colorado.edu/lisird/. Current, pysatSpaceWeather accesses some of the F10.7 products here. These routines could be generalized to support other indices such as Mg II core-to-wing, FISM daily bins, etc.

Potential impact

  • Is the feature related to an existing problem?
    • F10.7 does not capture short-term variability of solar inputs into the ionosphere
  • How critical is this feature to your workflow?
    • This would enable one-stop access of multiple solar irradiance proxies
  • How wide of an impact to you anticipate this enhancement having?
    • Potential use in driving ITM models with multiple inputs
  • Would this break any existing functionality?
    • This would be adding an instrument. Generalizing the routines may have impact depending on implementation

Potential solution(s)

A clear and concise description of what you want to happen.

  • Additional instruments for lisird, with methods file for access methods

Alternatives

A clear description of any alternative solutions or features you've considered.

  • The exact instrument structure requires discussion, as the existing F10.7 will probably remain as is

Additional context

Add any other context or screenshots about the feature request here, potentially
including your operational configuration.

@jklenzing jklenzing added the enhancement New feature or request label Feb 9, 2021
@aburrell
Copy link
Member

aburrell commented Nov 29, 2022

There are currently about 112 data sets available. Which ones do you want most? If you make a check list, this could be more easily addressed. A general LISIRD instrument would also belong on the checklist, though given the Madrigal example that's not terribly simple.

@jklenzing
Copy link
Member Author

FISM is probably the top priority for current projects.

@aburrell
Copy link
Member

There are many FISM options, make a checklist with the ones you want and I'll try to get at least one in for the next release.

@jklenzing
Copy link
Member Author

jklenzing commented Nov 29, 2022

checklist added to top-level issue

@aburrell
Copy link
Member

aburrell commented Dec 1, 2022

Thought: FISM2, as a model, may be more appropriately added to pysatModels.

@jklenzing
Copy link
Member Author

Thought: FISM2, as a model, may be more appropriately added to pysatModels.

I like this idea, but note that it would likely make pysatSpaceWeather a dependency there since we use the lisird interface

@aburrell
Copy link
Member

aburrell commented Dec 1, 2022

Not necessarily, we could instead add the lisird interface to each module individually, format it for an individual instrument (as is currently done), add it to pysat, or create a pysatLISIRD interface. So many possibilities! 🦑

@aburrell aburrell linked a pull request Dec 2, 2022 that will close this issue
10 tasks
@aburrell aburrell added this to the 0.2.0 milestone Dec 6, 2022
@aburrell aburrell transferred this issue from pysat/pysatSpaceWeather Dec 6, 2022
@jklenzing
Copy link
Member Author

I think all four FISM products could be a single module. Proposed properties:

platform = 'lasp'
name = 'fism2'
tags = {'daily': 'FISM2 daily averages',
        '60sec': 'FISM2 flare output, 60 second cadence.'}
inst_ids = {'': list(tags.keys()),
            'bands': list(tags.keys())}

Potentially could use 'hr' instead of '', which matches the syntax on the site. Any preference @aburrell?

@jklenzing
Copy link
Member Author

Experimenting with using the pysatNASA download routines here, since https is available.

@jklenzing jklenzing self-assigned this Feb 27, 2023
@aburrell
Copy link
Member

Potentially could use 'hr' instead of '', which matches the syntax on the site.

I would prefer that. As we add more and more data, it becomes increasingly unclear to have empty tags when there are other options available.

@jklenzing
Copy link
Member Author

OK. My current working is that tags define time resolution (daily or flare), with inst_id for wavelength resolution (full spectra vs bands from Solomon and Qian, 2005).

Just noticed the 60sec cadence is not consistent, this is 300sec for bands. Could change this tag to 'flare' if that would make more sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants