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

Vary command accepts repeated parameters

    Details

    • Type: Bug
    • Status: Reopened (View Workflow)
    • Priority: P3
    • Resolution: Unresolved
    • Affects Version/s: R2014a, R2015a, R2016a, R2017a, R2018A
    • Fix Version/s: Someday
    • Component/s: Vary Command (FRC-2)
    • Labels:
      None
    • Environment:

      Operating System: Other
      Platform: PC

      Description

      Created an attachment (id=1089)
      Vary_Validation_RepeatedSetting.script

      The attached script tries to set a parameter to the Vary command twice:

      Vary DefaultDC(DeltaV.Element1 = 2,

      {Perturbation = .001, Perturbation = .005}

      );

      This should be caught during validation.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            jjkparker Joel Parker added a comment -

            Attachment Vary_Validation_RepeatedSetting.script has been added with description: Vary_Validation_RepeatedSetting.script

            Show
            jjkparker Joel Parker added a comment - Attachment Vary_Validation_RepeatedSetting.script has been added with description: Vary_Validation_RepeatedSetting.script
            Hide
            gmat_wcs Wendy Shoan added a comment -

            CCB: Assign to Steve for documentation

            Show
            gmat_wcs Wendy Shoan added a comment - CCB: Assign to Steve for documentation
            Hide
            dcooley Steve Cooley added a comment -

            Added a Remark to the Vary Command feature spec describing this behaviour.

            Does anything else need to be done?

            Show
            dcooley Steve Cooley added a comment - Added a Remark to the Vary Command feature spec describing this behaviour. Does anything else need to be done?
            Hide
            dcooley Steve Cooley added a comment - - edited

            Per discussion with Steve H, can close.

            -Added comments to Vary spec.

            -The following regression tests have been removed:

            Vary_Validation_VF13ad1_RepeatedSetting
            Vary_Validation_fmincon_RepeatedSetting
            Vary_Validation_RepeatedSetting (script)

            Show
            dcooley Steve Cooley added a comment - - edited Per discussion with Steve H, can close. -Added comments to Vary spec. -The following regression tests have been removed: Vary_Validation_VF13ad1_RepeatedSetting Vary_Validation_fmincon_RepeatedSetting Vary_Validation_RepeatedSetting (script)
            Hide
            dcooley Steve Cooley added a comment -

            Per SPH (based upon DJC review of Vary spec): Re-open as a P3 just so we keep it in the system. Eventually, we will generate a warning or error on receiving repeated parameters.

            Show
            dcooley Steve Cooley added a comment - Per SPH (based upon DJC review of Vary spec): Re-open as a P3 just so we keep it in the system. Eventually, we will generate a warning or error on receiving repeated parameters.

              People

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

                Dates

                • Created:
                  Updated: