View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0014426ParaView(No Category)public2013-11-26 13:162014-01-11 08:39
ReporterGena Bug 
Assigned ToDan Lipsa 
PriorityhighSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Versiongit-master 
Target Version4.1Fixed in Version4.1 
Summary0014426: png screenshots have a huge size in 4.1.0rc1
DescriptionLast 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.
TagsNo tags attached.
ProjectSandia
Topic Name
Typeusability
Attached Filespng file icon screenshot.pv410rc1.png [^] (1,976,424 bytes) 2013-11-26 13:16
png file icon screenshot.pv401.png [^] (36,857 bytes) 2013-11-26 13:17

 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.

 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


Copyright © 2000 - 2018 MantisBT Team