biolab / orange3

🍊 :bar_chart: :bulb: Orange: Interactive data analysis
https://orangedatamining.com
Other
4.82k stars 1.01k forks source link

Exporting plot to currently open ppt #4883

Closed VesnaT closed 1 year ago

VesnaT commented 4 years ago
irgolic commented 4 years ago

Supposedly CTRL+C CTRL+V-ing in a widget copies the plot. This should be tested by someone else too.

ajdapretnar commented 4 years ago

@thocevar for Windows, @lanzagar for Linux?

thocevar commented 4 years ago

I tried all widgets in our Visualization section on Windows. Almost all of them work.

lanzagar commented 4 years ago

Similar on linux. Scatter plot also has (unnecessary) bottom and right borders when I paste the image

ajdapretnar commented 4 years ago

I think white/gray background is actually expected. I assume images have transparent background, with the image being on gray in Distributions and Venn by chance. @janezd Correct me if I am wrong.

lanzagar commented 4 years ago

That the copied image has a different background is not so problematic. But the case in Distributions, where pressing Ctrl-C changes the look of the widget is a bug (even if pretty harmless).

borondics commented 4 years ago

Is Mac OS going to be supported too?

On Fri, Jul 24, 2020 at 12:09 PM Tomaz Hocevar notifications@github.com wrote:

I tried all widgets in our Visualization section on Windows. Almost all of them work.

  • Distributions: background in the widget changes from gray to white on Ctrl-C.
  • Venn Diagram: the copied image has white background, while the widget has gray.
  • Pythagorean Forest: Ctrl-C doesn't work.
  • CN2 Rule Viewer: Ctrl-C doesn't work (it's a table not an image).

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/biolab/orange3/issues/4883#issuecomment-663467080, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABQDIGW2J5K3Q3XSIMSRHC3R5FMW7ANCNFSM4ORIS6GA .

ajdapretnar commented 4 years ago

@borondics I have asked for Win and Linux just because I have already tested on Mac and it works. :) However, I assume these bugs are present regardless of the platform.

borondics commented 4 years ago

Ajda, you are right indeed it works. For whatever reason I had to restart my Power Point and it worked fine. :D

On Fri, Jul 24, 2020 at 1:11 PM Ajda notifications@github.com wrote:

@borondics https://github.com/borondics I have asked for Win and Linux just because I have already tested on Mac and it works. :) However, I assume these bugs are present regardless of the platform.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/biolab/orange3/issues/4883#issuecomment-663488454, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABQDIGUEFT4JOVC7K5TSHBDR5FT5ZANCNFSM4ORIS6GA .

irgolic commented 4 years ago

Perhaps a 'Copy plot' button should be added to the status bar?

ajdapretnar commented 4 years ago

I would be cautious with extending the status bar. It is growing rapidly and some widgets can't handle many more options below.

Also, I don't think it is really needed. Anyone can simply save an image on a computer and use it in ppt. A shortcut is just a welcome feature and it is intuitive.

borondics commented 4 years ago

Would it be possible to somehow match the Copy/Paste image resolution to the screen resolution? Or at least increase it so that the Copy/Paste image becomes better quality? What I really like is of course that it is only the graph that Copy/Paste gives and that the last number on the axis is also fully captured, while it is not visible in the window.

Here is what I am getting on a Mac with Retina screen.

Copy/Paste Resolution: 913x691 image

Screen Capture tool of OS X (let's ignore that the whole widget window is on the picture) Resolution: 2024x1668 image

richardharrishk commented 3 years ago

I can't seem to export data easily to an excel - or power point. it's so easy to import - impossible to export - why do I get pickle files...?! I just want excel, or png/jpeg.

janezd commented 2 years ago

I confirm: on Retina screen, the image is captured with non-Retina resolution, i.e. it shows 72 ppi instead of 144 ppi. @markotoplak, @ales-erjavec, do you know how to fix this?

janezd commented 2 years ago

Attempted fixes for retina are in https://github.com/biolab/orange-widget-base/pull/208.

It appears, though, that macOs clipboard does not know the actual dpi of image data and always assumes 72 dpi. Copied images will have the full resolution but will appear too large.

janezd commented 1 year ago

Fixed via https://github.com/biolab/orange-widget-base/pull/208.