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

Propagator Applies Integrator Type When Invalid Value

    Details

      Description

      1. Open New Mission
      2. Open DefaultProp and change Integrator Type to 'PrinceDormand45'
      3. Change Step Size to invalid value 'eee'

      4. Click Apply. The error pops up
      "The value of "eee" for field "InitialStepSize" on object "DefaultProp" is not an allowed value.
      The allowed values are: [Real Number]."

      5. Open Show Script - the Integrator Type changed to 'PrinceDormand45'

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            jraymond Joshua Raymond added a comment -

            This issue also happens when an invalid entry is placed in the Degree and Order fields when the force model is changed.

            Show
            jraymond Joshua Raymond added a comment - This issue also happens when an invalid entry is placed in the Degree and Order fields when the force model is changed.
            Hide
            jraymond Joshua Raymond added a comment -

            Code is ready for review. The commit can be found in the IntegratorChangeFromInvalidEntryFix branch in the repository.

            Show
            jraymond Joshua Raymond added a comment - Code is ready for review. The commit can be found in the IntegratorChangeFromInvalidEntryFix branch in the repository.
            Hide
            djcinsb Darrel Conway added a comment -

            Setting the field to eee, hitting apply, and then setting it to a valid setting is broken now. So there is a bit more work to do here.

            Show
            djcinsb Darrel Conway added a comment - Setting the field to eee, hitting apply, and then setting it to a valid setting is broken now. So there is a bit more work to do here.
            Hide
            jraymond Joshua Raymond added a comment -

            Added fix to remove the new bug that was created. Code is ready for review.

            Show
            jraymond Joshua Raymond added a comment - Added fix to remove the new bug that was created. Code is ready for review.
            Hide
            djcinsb Darrel Conway added a comment -

            Code changes have been reviewed and pushed. Please check the change in tomorrow's build (or later).

            Show
            djcinsb Darrel Conway added a comment - Code changes have been reviewed and pushed. Please check the change in tomorrow's build (or later).
            Hide
            troyzman Tetyana Royzman added a comment -

            Tested with the latest 9/28/17 Build. The error fixed.

            Show
            troyzman Tetyana Royzman added a comment - Tested with the latest 9/28/17 Build. The error fixed.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - 4 hours
                  4h
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 3 hours, 30 minutes Time Not Required
                  3h 30m