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

Provide millionths of solar hemisphere unit #6842

Closed
wants to merge 7 commits into from

Conversation

Telomelonia
Copy link
Contributor

@Telomelonia Telomelonia commented Mar 13, 2023

PR Description

Adds the millionths of solar hemisphere units in the map folder

Fixes #1851

TODOS

  • changelog
  • make an equivalency func

@Telomelonia Telomelonia requested a review from a team as a code owner March 13, 2023 03:02
@nabobalis nabobalis added No Backport A PR that isn't to be backported to any release branch. (To be used as a flag to other maintainers) map Affects the map submodule labels Mar 13, 2023
Copy link
Member

@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.

To start with, please could you update this so the pre-commit check passes? There's instructions on how to do this here: https://docs.sunpy.org/en/latest/dev_guide/contents/code_standards.html#formatting

@ayshih
Copy link
Member

ayshih commented Mar 14, 2023

See #1851 (comment) for how this PR needs to be modified

@nabobalis nabobalis marked this pull request as draft March 14, 2023 21:34
@Telomelonia
Copy link
Contributor Author

pre-commit.ci autofix

@nabobalis
Copy link
Contributor

Hi @Telomelonia, I am going to close this pull request. If you have time to come back to this, please reopen it.

@nabobalis nabobalis closed this Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
map Affects the map submodule No Backport A PR that isn't to be backported to any release branch. (To be used as a flag to other maintainers)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide millionths of solar hemisphere units
4 participants