View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0014426 | ParaView | (No Category) | public | 2013-11-26 13:16 | 2014-01-11 08:39 | ||||
Reporter | Gena Bug | ||||||||
Assigned To | Dan Lipsa | ||||||||
Priority | high | Severity | minor | Reproducibility | have not tried | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | git-master | ||||||||
Target Version | 4.1 | Fixed in Version | 4.1 | ||||||
Summary | 0014426: png screenshots have a huge size in 4.1.0rc1 | ||||||||
Description | Last version of ParaView (4.1.0rc1) makes the huge size screenshots when choosing png format. Changing image quality isn't help -- the image size is always the same. I attached two screenshot of the same scene (see below), one from 4.1.0rc1 (screenshot.pv410rc1.png) and another from 4.0.1 (screenshot.pv401.png). Steps to reproduce: 1. open pv 2. make a sphere (Sources -> Sphere) with the default settings 3. make a png screenshot with the default settings Env: both ParaView (4.0.1 and 4.1.0rc1) downloaded from the size and installed on Debian testing amd64. | ||||||||
Tags | No tags attached. | ||||||||
Project | Sandia | ||||||||
Topic Name | |||||||||
Type | usability | ||||||||
Attached Files | screenshot.pv410rc1.png [^] (1,976,424 bytes) 2013-11-26 13:16 screenshot.pv401.png [^] (36,857 bytes) 2013-11-26 13:17 | ||||||||
Relationships | |
Relationships |
Notes | |
(0031888) Paul Edwards (reporter) 2013-11-30 04:18 edited on: 2013-12-04 17:02 |
Looks like this commit was adding support for compression with PNG images but set the default value to be 0 (no compression). http://vtk.org/gitweb?p=VTK.git;a=commit;h=5d97d7df5c754d8609a746fdcb8d65736ab61f8b [^] The compression level in the dialog isn't used unless jpeg is selected - it could be added in: ./ParaViewCore/ServerManager/Rendering/vtkSMUtilities.cxx (but would need to translate to the 0-9 that png writer expects). |
(0031921) Utkarsh Ayachit (administrator) 2013-12-11 11:43 |
Thanks for the solution, Paul. That makes it easier :)! |
(0031941) Utkarsh Ayachit (administrator) 2013-12-16 11:40 |
VTK gerrit topic: http://review.source.kitware.com/#/t/3720/ [^] |
(0032024) Dan Lipsa (developer) 2013-12-18 17:04 |
Hi Paul, Note that in the 'Save Snapshot' dialog you can select the Image Quality. You cannot link that with the compression level for PNG as the image quality stays always the same - PNG is a loss-less format. Someone selecting highest image quality would get no compression for PNG, not a desired behavior. So, in the fix I just set the compression level to 5, as per advice from http://refspecs.linuxbase.org/LSB_3.1.0/LSB-Desktop-generic/LSB-Desktop-generic/libpng12.png.set.compression.level.1.html [^] |
(0032034) Utkarsh Ayachit (administrator) 2013-12-20 08:26 |
Topics merged into master: 14286_fix_orientation_widget (VTK) 14426-huge-png 14464_fix_specular_highlights 14466_fix_calculator_panel disable-unified-bindings-by-default fix-collaboration fix_parallel_amrconnectivity gio-cleanup-and-warning-fixes pvweb-enhancements-events-protocols-docs |
(0032041) Paul Edwards (reporter) 2013-12-20 09:25 |
Hi Dan, You are right that it would be confusing and the dialog would need to be updated to accommodate it. One minor point: the level would have originally been 6 as that is the default in zlib - see Z_DEFAULT_COMPRESSION here: http://www.zlib.net/manual.html. [^] Regards, Paul |
(0032049) Alan Scott (manager) 2013-12-24 16:06 |
Tested local server, both ParaView and pvbatch, master, Linux. |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2013-11-26 13:16 | Gena Bug | New Issue | |
2013-11-26 13:16 | Gena Bug | File Added: screenshot.pv410rc1.png | |
2013-11-26 13:17 | Gena Bug | File Added: screenshot.pv401.png | |
2013-11-26 14:28 | Alan Scott | Priority | normal => high |
2013-11-26 14:56 | Utkarsh Ayachit | Project | ParaViewPro => Sandia |
2013-11-26 14:57 | Utkarsh Ayachit | Product Version | => git-master |
2013-11-26 14:57 | Utkarsh Ayachit | Target Version | => 4.1 |
2013-11-30 04:18 | Paul Edwards | Note Added: 0031888 | |
2013-12-04 17:02 | Paul Edwards | Note Edited: 0031888 | |
2013-12-11 11:43 | Utkarsh Ayachit | Note Added: 0031921 | |
2013-12-11 11:43 | Utkarsh Ayachit | Assigned To | => Dan Lipsa |
2013-12-13 15:40 | Dan Lipsa | Status | backlog => active development |
2013-12-16 11:40 | Utkarsh Ayachit | Note Added: 0031941 | |
2013-12-18 17:04 | Dan Lipsa | Note Added: 0032024 | |
2013-12-18 17:04 | Dan Lipsa | Status | active development => gatekeeper review |
2013-12-18 17:04 | Dan Lipsa | Resolution | open => fixed |
2013-12-20 08:26 | Utkarsh Ayachit | Fixed in Version | => git-master |
2013-12-20 08:26 | Utkarsh Ayachit | Status | gatekeeper review => customer review |
2013-12-20 08:26 | Utkarsh Ayachit | Note Added: 0032034 | |
2013-12-20 09:25 | Paul Edwards | Note Added: 0032041 | |
2013-12-24 16:06 | Alan Scott | Note Added: 0032049 | |
2013-12-24 16:06 | Alan Scott | Status | customer review => closed |
2014-01-11 08:39 | Utkarsh Ayachit | Fixed in Version | git-master => 4.1 |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |