Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Grapher does not print values #212

Open
5 tasks
stefanhahmann opened this issue Nov 22, 2022 · 3 comments
Open
5 tasks

Grapher does not print values #212

stefanhahmann opened this issue Nov 22, 2022 · 3 comments
Labels

Comments

@stefanhahmann
Copy link
Collaborator

stefanhahmann commented Nov 22, 2022

Bug description

The grapher view does not plot values that are actually existing in the table view.

Steps to reproduce the bug

  1. Start Fiji
  2. Start Plugin Mastodon
  3. open Mastodon project grapher-bug.mastodon: https://cloudstore.zih.tu-dresden.de/index.php/s/XgbnqRDgb9jLXLb
  4. Confirm dialog image data not accessible
  5. Open grapher
  6. Select "Spot position - X" for X axis
  7. Select "Spot position - Y" for Y axis
  8. Select "From context" - "full graph"
  9. Press "Plot"
  10. Check the result in the grapher view. No points are plotted in the diagram, even though the plot window is updated. It zooms to a location without data points. Giving the user the impression that there is no data available.

Additional information:

The way to zoom out is to first click on the plot and afterwards use Shift+Crtl+Scroll or Z + Z to zoom out. After successfully zooming out once, everything works nicely.

GIF 22 11 2022 12-33-23

Version:

ImageJ 1.53t
Mastodon: 1.0.0-beta-26
Windows: 10 + Ubuntu 20.04 LTS

Tasks:

  • Reproduce bug
  • Find reason
  • Fix bug
  • Add "Fit axes" button for convenience
  • Optional: Make Alt + Scroll the same behaviour as Shift+Ctrl+Scroll ( Alt+Scroll currently is the same as Ctrl+Scroll, i.e. zooming on x-axis)
@maarzt maarzt added the bug label Nov 22, 2022
@tinevez
Copy link
Contributor

tinevez commented Nov 22, 2022

I can get it to work:

Screenshot 2022-11-22 at 15 40 13

But I had to unzoom first, as the grapher window was showing the bottom left part of the plot, where there is no data point.

@stefanhahmann
Copy link
Collaborator Author

stefanhahmann commented Nov 22, 2022

I agree. It also works for me now. I actually even tried zooming, but wasn't aware of the shortcuts (particularly ctrl + shift + scroll wheel to zoom on both axes is unusual to me). Now I am also aware of the 'Z'-button functionality (had problems with this, since the machine that I am currently using is bit to slow for the dataset I wanted to analyse).

Nevertheless, I would like to suggest a usability enhancement:

  • Next to the plot button, it would be nice to have a second button "Fit axes". This button would zoom to the full extent of the current data. Users that are not aware of the 'Z'-button doing this as well would then at least see something more intuitively.
  • When I set "From context" and "full graph" and press "Plot", the Grapher window makes an animation of the axes. This gives the user the impression that the Grapher window tries to zoom to the full extent of the data. I suggest to omit this behaviour, since it is a bit irritating. At least for me, it was not clear what happened during the animated adjustment of the axes.
  • Alt + scroll wheel currently does the same as Ctrl + scroll wheel. Perhaps, it would be nice that Alt + Scroll wheel does the same as Ctrl + Shift + Scroll wheel, i.e. zooming in both axes. (Btw: this would be the same behaviour as in Photoshop).

Additionally, https://mastodon.readthedocs.io/en/latest/docs/partB/table_bdv_navigation_keys.html would need a similar section for the Grapher as well. Without @maarzt telling me, I would not have found found how to scroll on both axes in the Grapher viewer (and similarly a newbe user perhaps would also not find out on its own).

@maarzt
Copy link
Contributor

maarzt commented Nov 23, 2022

@stefanhahmann wrote:

  • When I set "From context" and "full graph" and press "Plot", the Grapher window makes an animation of the axes. This gives the user the impression that the Grapher window tries to zoom to the full extent of the data. I suggest to omit this behaviour, since it is a bit irritating. At least for me, it was not clear what happened during the animated adjustment of the axes.

I think there is indeed some bug. After opening a new grapher window, selecting "Spot - X", "Spot - Y", "From context: full graph" and clicking "Plot", the plot window is updated. But it zooms to a location without data points. Giving the impression that there is no data available. The way to zoom out is to first click on the plot and afterwards use Shift+Crtl+Scroll or Z + Z to zoom out. I myself couldn't find how to zoom out, despite knowing the short cuts. So I too got the impression, that the grapher window is buggy. After I successfully zoomed out once, everything worked really nicely. So there seems to be a problem only with the initial state after opening a new grapher window.

Let's try to fix the initial zoom state of the grapher window. And also add a "Fit axes" button for convenience.

tinevez added a commit that referenced this issue Oct 18, 2024
With a short delay so that the bounds of the transform are
properly computed.
Fix #212
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants