Skip to content

Comments

correct nperiod calculation from nx0#352

Open
arkabokshi wants to merge 4 commits intounstablefrom
bugfix/gene_nperiod
Open

correct nperiod calculation from nx0#352
arkabokshi wants to merge 4 commits intounstablefrom
bugfix/gene_nperiod

Conversation

@arkabokshi
Copy link
Collaborator

nx0 = 2*nperiod - 1, but we specify nx0 which can be odd / even. How do we correctly handle this?

@arkabokshi arkabokshi requested review from bpatel2107 and dake0795 June 2, 2024 14:16
@arkabokshi arkabokshi added the bug Something isn't working label Jun 2, 2024
@arkabokshi
Copy link
Collaborator Author

This seems to handle the odd and even nkx "well", in that the mode is centered at theta=0 as expected for this situation. However for even nkx, the first 2pi segment seems to be zero suggesting an indexing issue somewhere (for odd nkx this is fine).

@bpatel2107
Copy link
Collaborator

Sorry this seems to have slipped through the cracks. This seems to be failing tests because the ballooning angle grid is not matching the golden answers. Do you know why?

@bpatel2107
Copy link
Collaborator

@arkabokshi is this still an issue?

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

Successfully merging this pull request may close these issues.

2 participants