Project

General

Profile

Feature #5098

Add ability to select the filename of an open cube in qview

Added by Kristin Berry 6 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Applications
Target version:
Impact:

Minimal -- Added functionality should have minimal impact.

Software Version:
Test Reviewer:

Description

Add the ability to select (for the purposes of copying and pasting)the filename of an open cube in qview.


Related issues

Related to ISIS - Documentation #5140: Document new qview feature: creating cubelists from open cube. Acknowledged

History

#1 Updated by Tammy Becker 6 months ago

  • Category set to Applications
  • Status changed from New to Acknowledged

#2 Updated by Stuart Sides 5 months ago

  • Status changed from Acknowledged to Assigned
  • Assignee set to Adam Goins
  • Target version set to 3.5.2 (2017-01-31 Jan)

#3 Updated by Adam Goins 5 months ago

  • Status changed from Assigned to In Progress

#4 Updated by Adam Goins 5 months ago

  • Status changed from In Progress to Resolved
  • Impact updated (diff)

The filename of open cubes can be copied by using ctrl + c while in the qview window. If multiple cubes are open then the filename of whichever cube has focus will be copied to the clipboard.

#5 Updated by Robert Sucharski 5 months ago

Ran a test and the application worked as expected.

#6 Updated by Kristin Berry 5 months ago

I also tested this and it looks great! Thanks for adding this feature!

#7 Updated by Cole Neubauer 5 months ago

CubeViewport.cpp
With the line
QFileInfo file(p_cube->fileName());
We have a class to handle fileNames called FileName, use this instead of QFileInfo. Look into its documentation to know it better but you will probably want to use either baseName() or name() depending on what you want returned from the path you pass it.

CubeViewport.h
Looks good

#8 Updated by Cole Neubauer 5 months ago

  • Status changed from Resolved to Feedback

#10 Updated by Tammy Becker 5 months ago

I've tested this new feature and the ctrl-c captures the filename displayed as expected, I 'pasted' successfully into an open 'emacs' document.

1) I tried it with multiple files open - worked as expected
2) I tried it under the browse window functionality - it worked as expected, the filename changed accordingly as a new image was loaded into the window.
3) I tried it on an "isis2" cube file that automatically invokes pds2isis with a temporary filename- the ctrl-c captures the original filename rather than the "Temp..." filename - this is good.

Where will this new feature be documented? With this question in mind, notes follow that might be worthy of user documentation:

I work with ISIS on a PC through Xming/Putty, the only way the captured filename works smoothly for me is with an open 'emacs' document with 'paste' (it might work with vi, etc.). I cannot seem to recover the captured filename on my xterm window, and cannot paste it in an open Word document. This might be my environment and simple ignorance on my part.

One limitation is that the directory path to the file is not captured...I displayed all the images from different directory. Only the root file (file.cub) is captured. Not the end of the world, but worth noting (we might not always want a path if it creates an incredible length of characters). A future option? maybe.

#11 Updated by Adam Goins 5 months ago

  • Status changed from Feedback to Resolved

ctrl + c now copies the absolute path to the cube, not just the cube name.

Also, the 3 relative qview tickets (5097, 5098, 5099) have been consolidated into a single directory in the work area.

#12 Updated by Tammy Becker 5 months ago

I need /work/projects/isis/latest/m05097_05098_05099/ built for astrovm4.

Thank you!

#13 Updated by Adam Goins 5 months ago

m05097_05098_05099/ has been rebuilt for astrovm4.

#14 Updated by Tammy Becker 5 months ago

All features work great!

1) The path is now included with the image display ctrl-c feature. Now consistent with the export list.

2) Export list still works great with the paths included.

3) The behavior of supplying qview a list of cubes is now nicely flexible with input filename extensions; it no longer requires a .lis. The error encountered if a line read in from the input list cannot be displayed behaves very nicely prompting the user to continue or not. I especially like the name of the problem file read in is displayed and included in the error pop-up.

New request: the input list feature is so nice, is there a possibility to add "open cube list" from the qview-File drop down window?

Thank you.

#15 Updated by Tammy Becker 5 months ago

  • Related to Documentation #5140: Document new qview feature: creating cubelists from open cube. added

#16 Updated by Makayla Shepherd 4 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF