Ticket #611 (closed task: wontfix)

Opened 7 years ago

Last modified 6 years ago

Combine Add Axis with Add Data (and Delete cases also)?

Reported by: bdezonia Owned by: bdezonia
Priority: major Milestone: imagej2-b4-plugins
Component: SCIFIO Version:
Severity: serious Keywords:
Cc: Blocked By:
Blocking:

Description

Users might get confused by there being two ways to add data (along an axis and by adding an axis). Same with deleting data.

Maybe make Add Data automatically hatch an axis if you specify one that is not there. If we do this then the insertion position of the dialog gets ignored (which is better than requiring it equal one).

Delete Axis is still necessary to reshape Datasets so they don't have multiple dimensions of size 1.

Ignoring insertion position may be confusing. So might the fact that we have a Delete Axis but not an Add Axis. Discuss this with others and come up with a solution.

Change History

comment:1 Changed 7 years ago by bdezonia

One thing to note: if add data and delete data add and remove axes as needed we don't need to have a add/delete axis at all. If we do that then delete data should always collapse Datasets so that dimensions of size one disappear.

comment:2 Changed 7 years ago by bdezonia

  • Milestone changed from imagej-2.0-beta2 to imagej-2.0-beta1

comment:3 Changed 7 years ago by bdezonia

Another thought: combining them makes sense if the plugin shows all possible axes. If it only shows axes currently in the Dataset then a separate Add/Delete Axis are needed.

comment:4 Changed 7 years ago by gharris

Confusing Label -
I used the "Delete Data..." (DeleteHyperplanes) plugin today and found that it
It appears that the Position is 0-based - as a result when I tried to delete the 4th channel from the OrganOfCorti, I set the Position to 4 and Quantity to 1... nothing happened (incl. no warning). When I used Position=3, it worked. So the label of that field is confusing... or perhaps it should be 1-based.

comment:5 Changed 7 years ago by gharris

Alternately, or in addition to, it could prohibit setting the value out of range.

comment:6 Changed 7 years ago by bdezonia

The plugin is 1-based but had a bug that kept last N hyperplanes from being deleted. Fixed in 77a70f6c9c2748f7b14ff5114e62432c241691b5.

Alternative name suggestions welcome.

comment:7 Changed 7 years ago by bdezonia

  • Milestone changed from imagej-2.0-beta1 to imagej-2.0-beta2

comment:8 Changed 7 years ago by bdezonia

  • Milestone changed from imagej-2.0.0-beta3 to imagej-2.0.0-beta4

comment:9 Changed 6 years ago by bdezonia

  • Status changed from new to closed
  • Resolution set to wontfix

I think the current implementation (the four combos of Add/Delete Data/Axes) is simple. The suggested fixes are more complicated and I'm don't think better. I'm retiring this ticket.

Note: See TracTickets for help on using tickets.