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

patch.dft() does not change data_type #390

Closed
ahmadtourei opened this issue Jun 6, 2024 · 2 comments
Closed

patch.dft() does not change data_type #390

ahmadtourei opened this issue Jun 6, 2024 · 2 comments
Labels
bug Something isn't working transform Related to transform operations

Comments

@ahmadtourei
Copy link
Collaborator

ahmadtourei commented Jun 6, 2024

Description

patch.dft() does not change data_type attribute so when make waterfall plot from a patch with strain_rate data type (e.g., patch.dft("time").abs().viz.waterfall()), the colorbar has strain_rate label.

Example

Expected behavior

Versions

  • OS [e.g. Ubuntu 20.04]:
  • DasCore Version [e.g. 0.0.5]:
  • Python Version [e.g. 3.10]:
@ahmadtourei ahmadtourei added the bug Something isn't working label Jun 6, 2024
@ahmadtourei ahmadtourei changed the title patch.dft() keeps data patch.dft() does not change data_type Jun 6, 2024
@ahmadtourei ahmadtourei added the transform Related to transform operations label Jun 6, 2024
@d-chambers
Copy link
Contributor

hmmm.... should Patch.dft just remove the data_type tag? Or maybe it is still technically correct since it is derived from strain rate, its just not in strain rate units?

@ahmadtourei
Copy link
Collaborator Author

Yeah, I'm not sure either. Keeping data_type as strain rate can be a bit misleading. Also, do we need it for patch.idft()? If not, removing data_type sounds okay to me. Or, maybe updating data_type to amplitude if real=True and complex_number if real=False? Then, we may make the patch.abs() update patch's data_type to amplitude if the original data_type is complex. What do you think?

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

No branches or pull requests

2 participants