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

Wrong defaults for StatisticsRejectFilter

    Details

      Description

      The current default values are:

      Create StatisticsRejectFilter BadCanberraPass;
      GMAT BadCanberraPass.FileNames =

      {'All'};
      GMAT BadCanberraPass.ObservedObjects = {All};
      GMAT BadCanberraPass.Trackers = {All};
      GMAT BadCanberraPass.DataTypes = {'All'}

      ;
      GMAT BadCanberraPass.EpochFormat = 'TAIModJulian';
      GMAT BadCanberraPass.InitialEpoch = '6116.00';
      GMAT BadCanberraPass.FinalEpoch = '58127.5';

      There are 4 fields with default All or 'All'

      This is wrong. The default value should be {} for all four of these fields.

      Tuan, is this easy to fix?

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            tdnguyen Tuan Nguyen added a comment -

            It is easy to fix I will fix it soon.

            Show
            tdnguyen Tuan Nguyen added a comment - It is easy to fix I will fix it soon.
            Hide
            tdnguyen Tuan Nguyen added a comment -

            It was fixed and wait for Darrel review code changes.

            Show
            tdnguyen Tuan Nguyen added a comment - It was fixed and wait for Darrel review code changes.
            Hide
            dcooley Steve Cooley added a comment -

            Lets talk about this at today's Nav meeting. (SES likes the current defaults. Note: current defaults and the Help doc are currently inconsistent)

            Show
            dcooley Steve Cooley added a comment - Lets talk about this at today's Nav meeting. (SES likes the current defaults. Note: current defaults and the Help doc are currently inconsistent)
            Hide
            djcinsb Darrel Conway added a comment - - edited

            Since this is not marked as a release blocker, we shouldn't commit it to the master branch. If it's not a blocker and we want this functionality, we should commit it to the dev branch. So is it desired and a blocker?

            Show
            djcinsb Darrel Conway added a comment - - edited Since this is not marked as a release blocker, we shouldn't commit it to the master branch. If it's not a blocker and we want this functionality, we should commit it to the dev branch. So is it desired and a blocker?
            Hide
            tdnguyen Tuan Nguyen added a comment -

            we agreed keeping default value of FileNames, Trackers, ObservedObjects, and DataTypes to be

            {'All'}

            . No code change is needed. This ticket needs to be closed.

            Show
            tdnguyen Tuan Nguyen added a comment - we agreed keeping default value of FileNames, Trackers, ObservedObjects, and DataTypes to be {'All'} . No code change is needed. This ticket needs to be closed.
            Hide
            dcooley Steve Cooley added a comment -

            The User docs will be changed accordingly.

            Show
            dcooley Steve Cooley added a comment - The User docs will be changed accordingly.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: