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

Synthetic test has x and y swapped #348

Open
kujaku11 opened this issue Aug 14, 2024 · 2 comments
Open

Synthetic test has x and y swapped #348

kujaku11 opened this issue Aug 14, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@kujaku11
Copy link
Collaborator

kujaku11 commented Aug 14, 2024

I'm not sure where this comes from, it looks like it might be in building the H5's but it looks like either Ex and Ey are swapped or Hx and Hy are swapped. When plotting the synthetic result with MTpy I get this:

Aurora Results

image

And the phases look like:

image

EMTF Results

image

Looks like EMTF results also has a phase flip. Could be mtpy-v2 or just a channel nomenclature issue.

@kujaku11 kujaku11 added the bug Something isn't working label Aug 14, 2024
@kkappler
Copy link
Collaborator

Yep, something does not look correct here.

Quick sanity check:

We expect (theoretically) for a homogeneous earth:
phase_xy to be +45 degrees
phase_yx to be -135 degrees

If that is correct, then we can fix the phase issue by

  • flipping the polarities of the electric channels only, or
  • flipping the polarities of the magnetic channels only

Not sure whether to fix this in the data, or in the reader.

@kkappler
Copy link
Collaborator

This has been fixed in the reader, with a non-robust fix.

A better fix would probably be to migrate the synthetic data to csv with headers, and make the reader assign the nomenclature after reading with the standard channel names, per mth5 issue 236

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants