Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #6542 (closed)

Opened 13 years ago

Closed 11 years ago

Last modified 11 years ago

Bug: Different scripts shown in insight and web

Reported by: jrswedlow Owned by: wmoore
Priority: blocker Milestone: OMERO-4.4.7
Component: Scripting Version: n.a.
Keywords: n.a. Cc: jburel, wmoore, jamoore, pwalczysko, ux@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2013-04-09 (7))

Description

Different scripts are listed as available on web and insight-- web's list includes more scripts.

Attachments (1)

Script Menus.png (700.2 KB) - added by rkferguson 11 years ago.

Download all attachments as: .zip

Change History (16)

comment:1 Changed 13 years ago by wmoore

The reason that these extra scripts are removed from the menu in Insight is because we have custom-built UIs for those figure scripts. Since we don't have these UIs in the web, I have left the scripts in the menu, so that this functionality is still available to web users (even though the UI for running the script isn't as nice).

We could remove these from the scripts menu in web: It would just mean that you can't export these figures from web.

comment:2 Changed 13 years ago by jburel

  • Owner set to jburel
  • Remaining Time set to 0.1
  • Status changed from new to accepted

Following discussion with Jason:

  • I will display the full list in insight (as it was before)
  • For the scripts with a customized UI, the customized UI will be displayed.

comment:3 Changed 13 years ago by jburel <j.burel@…>

  • Remaining Time changed from 0.1 to 0
  • Resolution set to fixed
  • Status changed from accepted to closed

(In [7f89c3d4a5f95351877a15fe919b59aa23f5db97/ome.git] on branch develop) Display all available scripts in menu (close #6542)

comment:4 Changed 11 years ago by wmoore

  • Resolution fixed deleted
  • Status changed from closed to reopened

This is broken again in Insight: We now have all the figure scripts showing in the scripts menu AND we're not getting the customised UI, so users get to see the script-generated UI for figure scripts.
This leads to problems: E.g. Petr just reported that Channel map parameter wasn't working in Split View Figure.

I think we should just not show Figure scripts in the scripts menu. We should consider these as "built-in" OMERO functionality and not show them as scripts at-all. There's no advantage to showing them. They don't make good scripting examples (too complex) and they just lead to confusion. We have turned them off in web: #6544.

comment:5 Changed 11 years ago by wmoore

  • Cc pwalczysko added; atarkowska removed

comment:6 Changed 11 years ago by jmoore

Is this a case where we should actually think about not distributing these scripts under OmeroPy/scripts anyway?

comment:7 Changed 11 years ago by rkferguson

  • Cc ux@… added

Differences:

Screenshot attached.


All

  • Web uses .py suffix
  • Analysis Scripts: Order different - suggest:
    • Kymograph
    • Kymograph Analysis
    • Plot Profile

Export Scripts

Insight has:

  • Make Movie - not in Web

Figure Scripts

Insight has:

  • Movie ROI Figure
  • ROI Split Figure
  • Split View Thumbnail
  • Thumbnail Figure

Not in Web


Setup Scripts Submenu

  • in Web not in Insight.


Util Scripts

Different order - suggest:

  • Channel Offsets
  • Combine Images
  • Dataset to Plate
  • Images From ROIs

Note from Will:

Ticket covers turning off all the scripts that have a customised UI in Insight (Make Movie, Movie Figure, ROI Split Figure, Split View Figure, Thumbnail Figure).

So need to:

  • sort in the same way (alphabetically I guess!)
  • remove ".py" in web
  • not show 'Setup Scripts' in web.

Gus:
Need to remove analysis scripts from menu - Web and Insight.

Changed 11 years ago by rkferguson

comment:8 Changed 11 years ago by jburel

  • Milestone changed from OMERO-Beta4.3.2 to OMERO-4.5
  • Sprint changed from 2011-09-01 (4) to Bugs Fixing

comment:9 Changed 11 years ago by jburel

  • Owner changed from jburel to wmoore
  • Status changed from reopened to accepted

comment:10 Changed 11 years ago by jburel

  • Sprint changed from Bugs Fixing to 2013-04-09 (7))

Moved from sprint Bugs Fixing

comment:11 Changed 11 years ago by jburel <j.burel@…>

(In [c2239bfbc3df3e56e69e60360661e0fa509b9cde/ome.git] on branch develop) Remove scripts with UI from list (see #6542)

comment:12 Changed 11 years ago by Josh Moore <josh@…>

(In [9ef8240aa8342dce9307988c9397ad935ed7d900/ome.git] on branch develop) Merge pull request #861 from jburel/rebased/develop/scripts-menu

Remove scripts with UI from list (see #6542) (rebased onto develop)

comment:13 Changed 11 years ago by wmoore

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

comment:14 Changed 11 years ago by jburel <j.burel@…>

(In [9818c3ff7cf410adc659efc05c3f161b19cce305/ome.git] on branch develop) Removing setup_scripts and import_scripts from Web scripts menu. See #6542

comment:15 Changed 11 years ago by jburel <j.burel@…>

(In [f77f2ede3b7a5f34066e554024da876516302abe/ome.git] on branch develop) Replace '.py' with '...' and sort script submenus. See #6542

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.69333 sec.)

We're Hiring!