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

Clear confusion about geomagnetic field vector #85

Merged
merged 1 commit into from
Nov 19, 2022

Conversation

yakir12
Copy link
Contributor

@yakir12 yakir12 commented Oct 17, 2022

The input variables to these functions are date (i.e. year), radius (i.e. Z), latitude, and longitude (i.e. X and Y). There's no mention that the components of the returned geomagnetic field vector are X, Y, Z. While this might be an obvious convention, I had no idea and assumed that the order of the components would be the same as the order of the input variables to these functions (i.e. Z, X, Y). I tried to "fix" that with that tiny notation in the parenthesis, but maybe you have a better idea.

As a side note, it might be good to add functions for the magnetic components listed for example here.

The input variables to these functions are date (i.e. year), radius (i.e. Z), latitude, and longitude (i.e. X and Y). There's no mention that the components of the returned geomagnetic field vector are X, Y, Z. While this might be an obvious convention, I had no idea and assumed that the order of the components would be the same as the order of the input variables to these functions (i.e. Z, X, Y). I tried to "fix" that with that tiny notation in the parenthesis, but maybe you have a better idea.

As a side note, it might be good to add functions for the magnetic components listed for example [here](https://www.geomag.nrcan.gc.ca/mag_fld/comp-en.php).
@ronisbr ronisbr merged commit f6e5618 into JuliaSpace:master Nov 19, 2022
@ronisbr
Copy link
Member

ronisbr commented Nov 19, 2022

Hi @yakir12 !

Sorry for the delay! Perfect, thank you very much!

@yakir12 yakir12 deleted the patch-1 branch November 19, 2022 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants