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.

Sign up for GitHub

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

Jump to bottom

Pulse - Grad refocus profile calculates wrong if xaxis in cm vs kHz #8

Closed
bsoher opened this issue May 4, 2023 · 1 comment
Closed

Pulse - Grad refocus profile calculates wrong if xaxis in cm vs kHz #8

bsoher opened this issue May 4, 2023 · 1 comment

Comments

@bsoher
Copy link
Collaborator

bsoher commented May 4, 2023

Calculations in 'cm' axis result in values > 1, vs 0.5152 etc. type values when axis is in 'kHz'.
bsoher added a commit that referenced this issue May 4, 2023
@bsoher
PULSE - tweaking GradRefocus profile calculation fix for an interacti…
70fe0be 
…on it had with changing default plot xaxis units to 'kHz'. Also went through and changed all default values for plot units to 'kHz' from 'cm'. This commit fixes Issues GH-8 and GH-3
@bsoher
Copy link
Collaborator Author

bsoher commented May 4, 2023

This works the same regardless if x-axis units kHz or cm. Caused a small display/plot issue. Found a workaround by moving display code around in plot_1d.

@bsoher bsoher closed this as completed May 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bsoher