Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 379365 - media-gfx/luminance-hdr 2.0.2 batch processing at minimal size
Summary: media-gfx/luminance-hdr 2.0.2 batch processing at minimal size
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Graphics Project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-16 09:59 UTC by Jonathan Müller
Modified: 2011-08-26 10:43 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---

Attachments
Fixes the batch file size problem (size.patch,838 bytes, patch)
2011-08-16 10:00 UTC, Jonathan Müller 		Details | Diff
Fix for the batch file-size problem (size.patch,836 bytes, patch)
2011-08-16 11:42 UTC, Jonathan Müller 		Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Müller 2011-08-16 09:59:32 UTC 
In luminance-hdr 2.0.2: When using the batch window to tone-map hdr files, the output is always at a minimal size of 400px width, instead of the width of the original file.

Reproducible: Always

Steps to Reproduce:
1. Open luminance-hdr, choose Tools (menu) -> Batch Tone Mapping
2. load some file, some options for tonemapping and an output directory
3. Click on "Start"
Actual Results:  
The resulting JPG file is only 400px wide with a 15MP input file

Expected Results:  
The resulting JPG file should have been >4500px wide

The problem is in the file src/Batch/BatchTMJob.cpp. The author apparently forgot to remove the line "opts->xsize = 400; // DEBUG", which causes this behaviour...
Comment 1 Jonathan Müller 2011-08-16 10:00:18 UTC 
Created attachment 283533 [details, diff]
Fixes the batch file size problem
Comment 2 Jonathan Müller 2011-08-16 10:20:58 UTC 
Unfortunately, the patch above causes luminance-hdr to sometimes crash... Which it does not when processing 400px-files.
Comment 3 Jonathan Müller 2011-08-16 11:42:39 UTC 
Created attachment 283545 [details, diff]
Fix for the batch file-size problem

This time, the patch really works. Sorry for previously uploading a not-working one.
Comment 4 Tim Harder gentoo-dev 2011-08-26 10:43:03 UTC 
Fixed in 2.1.0 that I just added to CVS.

Also, please use unified diffs in the future for attached patches. Thanks.