Ticket #3206 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

intellisense is broken after pythonsimple removal

Reported by: Nick Draper Owned by: Nick Draper
Priority: critical Milestone: Iteration 29
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Stuart Campbell

Description

The intellisense aspect where it listed the arguments of an algorithm no longer works. This is probably associated with the changes to the simpleAPI.

Change History

comment:1 Changed 9 years ago by Nick Draper

  • Status changed from new to assigned
  • Owner changed from Mathieu Doucet to Nick Draper

comment:2 Changed 9 years ago by Nick Draper

  • Status changed from assigned to accepted

comment:3 Changed 9 years ago by Nick Draper

  • Status changed from accepted to verify
  • Resolution set to fixed

(In [12753]) fixes #3206 This is a bit slow and inefficiant but it works. To speed things up you would need to do more of this in C++, and stop calling for every method. A single call to C++ to get all of the details for all algs, and then match them up.

comment:4 Changed 9 years ago by Nick Draper

  • Status changed from verify to reopened
  • Resolution fixed deleted

comment:5 Changed 9 years ago by Nick Draper

  • Status changed from reopened to verify
  • Resolution set to fixed

(In [12774]) fixes #3206 Slight improvement to restore [ ] around optional arguments

comment:6 Changed 9 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:7 Changed 9 years ago by Martyn Gigg

  • Status changed from verifying to closed

Verified in 1.28.12844 for both the script window and interpreter. Both algorithm and their counterpart dialog functions were verified.

comment:8 Changed 9 years ago by Martyn Gigg

  • Status changed from closed to reopened
  • Resolution fixed deleted

On a similar test of the intellisense, I've only just spotted that the calltip is not quite right.

It is displaying output parameters that are not workspaces, for example the IncidentEnergy output property on GetEi is displayed so that it looks like an input property.

comment:9 Changed 9 years ago by Martyn Gigg

  • Status changed from reopened to verify
  • Resolution set to fixed

(In [12884]) Fix the Python intellisense to exclude output properties that are not workspaces. Fixes #3206

comment:10 Changed 9 years ago by Stuart Campbell

  • Status changed from verify to verifying
  • Tester changed from Martyn Gigg to Stuart Campbell

comment:11 Changed 9 years ago by Stuart Campbell

  • Status changed from verifying to closed

Seems to work for me. Although I still get the initial grey list box on RHEL6, but once you type the '(' it looks ok.

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4053

Note: See TracTickets for help on using tickets.