converted internal grid number to sequence number#200
Merged
victorkemp merged 3 commits intoMystranSolver:devfrom Apr 5, 2026
Merged
converted internal grid number to sequence number#200victorkemp merged 3 commits intoMystranSolver:devfrom
victorkemp merged 3 commits intoMystranSolver:devfrom
Conversation
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
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.
Fixes #150
Mass matrix MGG is ordered by grid point sequence numbers, not internal grid point numbers so we have to look up the sequence number before indexing MGG.
I also reverted an old edit for CONM2 which was putting point mass into the mass matrix in internal grid point order instead of grid sequence order. The effect of that cancelled out the use of internal grid point order in GET_GRID_6X6_MASS.
Now both element density and concentrated mass work correctly when the grid point order in the input file (grid sequence order) is different from the ID numbering order.