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

Enable line highlighting for script lines that do not parse correctly

    Details

    • Type: New Feature
    • Status: Closed (View Workflow)
    • Priority: P1
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Script Debugger
    • Labels:
      None

      Description

      This is the first step towards building a GMAT debugging environment (GMT-6292). Let's break it into the following steps:
      Learn how the wxWidgets (Scintilla based) editor works
      Retrieve information from the Interpreter about the script line(s) that did not parse correctly
      Provide highlighting (change background color?) for the problem script line(s).

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              CRConway Claire Conway added a comment -

              For future reference: editor in src/gui/view

              Show
              CRConway Claire Conway added a comment - For future reference: editor in src/gui/view
              Hide
              djcinsb Darrel Conway added a comment -

              Code clean up and commit complete. There may be cases that we want to make a bit smarter – for instance, those that trigger during the interpreter's final pass, but at that point we don't have line numbers.

              Show
              djcinsb Darrel Conway added a comment - Code clean up and commit complete. There may be cases that we want to make a bit smarter – for instance, those that trigger during the interpreter's final pass, but at that point we don't have line numbers.
              Hide
              shughes Steven Hughes added a comment -

              hi Tetyana,

              This is reading for testing.. you can take a sample mission, add errors to different parts of the script before and after the BeginMissionSequence line, and then make sure the correct lines are highlighted.

              Thanks!

              Show
              shughes Steven Hughes added a comment - hi Tetyana, This is reading for testing.. you can take a sample mission, add errors to different parts of the script before and after the BeginMissionSequence line, and then make sure the correct lines are highlighted. Thanks!
              Hide
              shughes Steven Hughes added a comment -

              We've seen issues in the past with line numbers using #include, so use Ex_R2016a_IncludeMacro.script as one of the test cases and put in errors before and after the #include line.

              Show
              shughes Steven Hughes added a comment - We've seen issues in the past with line numbers using #include, so use Ex_R2016a_IncludeMacro.script as one of the test cases and put in errors before and after the #include line.
              Hide
              shughes Steven Hughes added a comment -

              Errors are pink , Warnings as yellow.

              Show
              shughes Steven Hughes added a comment - Errors are pink , Warnings as yellow.
              Hide
              troyzman Tetyana Royzman added a comment -

              Steve, could you use the script attached. If the error at the script, the Plot Orbit and Target color highlighted in yellow.
              Ex_HohmannTransfer.script
              Attached also is screen shots examples:
              GMT-6296.docx

              Show
              troyzman Tetyana Royzman added a comment - Steve, could you use the script attached. If the error at the script, the Plot Orbit and Target color highlighted in yellow. Ex_HohmannTransfer.script Attached also is screen shots examples: GMT-6296.docx
              Hide
              troyzman Tetyana Royzman added a comment -

              Tested with the updated scripts. The error is correct.

              Show
              troyzman Tetyana Royzman added a comment - Tested with the updated scripts. The error is correct.
              Hide
              troyzman Tetyana Royzman added a comment -

              Ticket fixed.

              Show
              troyzman Tetyana Royzman added a comment - Ticket fixed.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Time Tracking

                    Estimated:
                    Original Estimate - 2 weeks
                    2w
                    Remaining:
                    Time Spent - 3 days Remaining Estimate - 1 week, 2 days
                    1w 2d
                    Logged:
                    Time Spent - 3 days Remaining Estimate - 1 week, 2 days
                    3d