Uploaded image for project: 'GMAT'
  1. GMAT
  2. GMT-4580

Different results when setting OutgoingAsymptote state in different orders in mission sequence

    Details

      Description

      The attached test Params_LhsDeps_OutgoingAsymptote.script sets the OutgoingAsymptote state in different orders in the mission sequence. The report file should contain lines with identical data, but OutgoingRHA, OutgoingDHA, and OutgoingBVAZI differ in each case.

      Running Params_LhsDeps_OutgoingAsymptote_Debug.script shows why. This reportfile shows the truth data in line 1, then it shows IncomingAsymptote and OutgoingAsymptote data before and after each line setting a component of the state.

      When HyperbolicRadPer is set, GMAT is going through the IncomingAsymptote system, and the Incoming parameters are being held constant. This causes the Outgoing parameters to change, which is not what the user intended. This occurs with C3Energy also, if you place that line last.

      What we actually need is four new parameters, none of which are shared:

      • IncomingRadPer (instead of HyperbolicRadPer)
      • IncomingC3Energy
      • OutgoingRadPer
      • OutgoingC3Energy

      TA does not show the same behavior, so we don't need to split that off.

      SharedParamTests.zip contains script that try all state reps with shared params in different orders. It would help if someone else could bang on it also, since I keep missing things.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            jjkparker Joel Parker added a comment -

            When the code fix goes in, I can write new param tests/docs in the evening and kick off the packaging/testing.

            Show
            jjkparker Joel Parker added a comment - When the code fix goes in, I can write new param tests/docs in the evening and kick off the packaging/testing.
            Hide
            jjkparker Joel Parker added a comment -

            All script tests are updated to use the new params. All docs are updated too.

            Show
            jjkparker Joel Parker added a comment - All script tests are updated to use the new params. All docs are updated too.
            Hide
            gmatloj Linda Jun added a comment -

            Removed HyperbolicRadPer Parameter.
            Added IncomingRadPer, IncomingC3Energy, OutgoingRadPer, OutgoingC3Energy Parameters.
            Changed C3Energy back to read-only Parameter.
            Fix will be in 2014-05-09 build.

            Show
            gmatloj Linda Jun added a comment - Removed HyperbolicRadPer Parameter. Added IncomingRadPer, IncomingC3Energy, OutgoingRadPer, OutgoingC3Energy Parameters. Changed C3Energy back to read-only Parameter. Fix will be in 2014-05-09 build.
            Hide
            jjkparker Joel Parker added a comment -

            Verified as fixed using R2014a.

            Show
            jjkparker Joel Parker added a comment - Verified as fixed using R2014a.
            Hide
            jjkparker Joel Parker added a comment -

            Already tested and documented. Closing.

            Show
            jjkparker Joel Parker added a comment - Already tested and documented. Closing.

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 7 hours
                  7h