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

fixing op2 subcase id #32

Merged
merged 2 commits into from
Feb 13, 2024
Merged

fixing op2 subcase id #32

merged 2 commits into from
Feb 13, 2024

Conversation

SteveDoyle2
Copy link
Collaborator

@SteveDoyle2 SteveDoyle2 commented Feb 6, 2024

Takes advantage of the fact that:

  • statics can have nsubcases (NSUB)
  • modes can have 1 subcase, but NSUB modes
  • craig-bampton is like modes and has 1 subcase and NSUB modes
  • buckling has 2 subcases with NSUB+1 (static+modes) therefore we explicitly call out the subcase_index for modes/craig-bampton and buckling as 1 or 2 depending

corrected eigenvector table flag (should be 7 and not 1, which would lead to displacements being output)

fixes: #31

Much better output

# test_op2 EB-ALL-ELEM-TEST-LANCZOS-MODE-2-DPB.OP2
eigenvectors[1]
  isubcase = 1
  type=RealEigenvectorArray ntimes=2 nnodes=13, table_name=OUGV1
  data: [t1, t2, t3, r1, r2, r3] shape=[2, 13, 6] dtype=float32
  node_gridtype.shape = (13, 2)
  sort1
  modes = [1 2]; dtype=int32
  eigns = [46891.77  61105.016]; dtype=float64
  mode_cycles = [216.545 247.194]; dtype=float64

# test_op2 EB-ALL-ELEM-TEST-GIV.OP2
eigenvectors[1]
  isubcase = 1
  type=RealEigenvectorArray ntimes=4 nnodes=13, table_name=OUGV1
  data: [t1, t2, t3, r1, r2, r3] shape=[4, 13, 6] dtype=float32
  node_gridtype.shape = (13, 2)
  sort1
  modes = [1 2 3 4]; dtype=int32
  eigns = [ 46891.766  61105.016 156801.047 216221.312]; dtype=float64
  mode_cycles = [216.545 247.194 395.981 464.996]; dtype=float64

 - statics can have nsubcases (NSUB)
 - modes can have 1 subcase, but NSUB modes
 - craig-bampton is like modes and has 1 subcase and NSUB modes
 - buckling has 2 subcases with NSUB+1 (static+modes)
therefore we explicitly call out the subcase_index for modes/craig-bampton and buckling as 1 or 2 depending

corrected eigenvector table flag (should be 7 and not 1, which would lead to displacements being output)

# fixes 31
@Bruno02468 Bruno02468 merged commit 4f9f7ac into main Feb 13, 2024
1 check passed
@SteveDoyle2 SteveDoyle2 deleted the op2_subcase_id branch March 27, 2024 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect OP2 subcase id
2 participants