Opened 6 years ago

Closed 6 years ago

#17043 closed defect (fixed)

fix scilab interface

Reported by: was Owned by:
Priority: major Milestone: sage-6.4
Component: interfaces Keywords:
Cc: Merged in:
Authors: Frédéric Chapoton Reviewers: Dima Pasechnik
Report Upstream: N/A Work issues:
Branch: dfc49d3 (Commits) Commit: dfc49d364d612b494b0dc83b4e32fbb8129a81ca
Dependencies: Stopgaps:

Description (last modified by was)

The Sage Scilab interface is broken. The fix is pretty trivial, and I guess is a change in a command line flag to scilab: 

diff --git a/src/sage/interfaces/scilab.py b/src/sage/interfaces/scilab.py
index bc4a508..7e1d475 100644
--- a/src/sage/interfaces/scilab.py
+++ b/src/sage/interfaces/scilab.py
@@ -218,7 +218,7 @@ class Scilab(Expect):
         Expect.__init__(self,
                         name = 'scilab',
                         prompt = '-->',
-                        command = "scilab -nogui",
+                        command = "scilab -nw",
                         maxread = maxread,
                         server = server,
                         server_tmpdir = server_tmpdir,
@@ -495,7 +495,7 @@ def scilab_console():
     Scilab, like Sage, remembers its history from one session to
     another.
     """
-    os.system('scilab -nogui')
+    os.system('scilab -nw')

Change History (5)

comment:1 Changed 6 years ago by was

  • Description modified (diff)

comment:2 Changed 6 years ago by chapoton

  • Authors set to Frédéric Chapoton
  • Branch set to u/chapoton/17043
  • Commit set to dfc49d364d612b494b0dc83b4e32fbb8129a81ca
  • Status changed from new to needs_review

Done

New commits:

​dfc49d3trac #17043 correct call to scilab as required by wstein

comment:3 Changed 6 years ago by dimpase

  • Status changed from needs_review to positive_review

LGTM

comment:4 Changed 6 years ago by dimpase

  • Reviewers set to Dima Pasechnik

comment:5 Changed 6 years ago by vbraun

  • Branch changed from u/chapoton/17043 to dfc49d364d612b494b0dc83b4e32fbb8129a81ca
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.