Open
Conversation
…ality we only need one helicity value in the end, and it should be pulled from Sgn[Ip, B_t].
… gracefully and consistently with helicity wrapping direction. Issue likely arrises from COCOS reasons
… to make logic clearer
ZanderKeith
force-pushed
the
zk/test-polish
branch
from
b3f8c9a to
77189f5
Compare
…o verify which is timing out
…mments in test_toroidal_angles
ZanderKeith
force-pushed
the
zk/test-polish
branch
from
cb77893 to
d2d6158
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fleshed out tests for ThinCurr functions, Filaments, and EquilibriumField
Also trying to set up some CI