Skip to content
This repository has been archived by the owner on Aug 24, 2023. It is now read-only.

Examples using multiple SFT simulation output #335

Closed
wants to merge 1 commit into from

Conversation

mfcorchado
Copy link

Two python scripts that show the evolution of the PFSS solution as a Surface Flux Transport model simulates the 11 year solar-like cycle, see the following issue.

Two python scripts that show the evolution of the PFSS solution as a Surface Flux Transport model simulates the 11 year solar-like cycle, see the following [issue](dstansby#334).
@lgtm-com
Copy link

lgtm-com bot commented Dec 5, 2021

This pull request introduces 1 alert when merging 0cd13c9 into f1fecd0 - view on LGTM.com

new alerts:

  • 1 for Module is imported with 'import' and 'import from'

Copy link
Owner

@dstansby dstansby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! I'm not sure what new functionality this is adding, or what functionality is being shown that isn't already in other examples? Could you briefly explain?

@dstansby
Copy link
Owner

Thanks again for opening this. As it stands I don't think this is mergeable because it the examples aren't self contained, ie. you need external data to run them, which means we can't generate the example output when automatically building the documentation. If you can modify the example to fix this feel free to open an new PR, or comment here and I can re-open this one.

@dstansby dstansby closed this Jan 14, 2022
@mfcorchado
Copy link
Author

mfcorchado commented Jan 14, 2022 via email

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants