Task #6542 (closed)
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)
Change History (16)
comment:1 Changed 13 years ago by wmoore
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
Insight: see https://github.com/openmicroscopy/openmicroscopy/pull/842
Passing ticket to Will.
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
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.