0000627: IES Profile Crashes LuxRender

Notes
(0001716) Dade (developer) 2009-11-09 06:54	This is probably related to BUG 0000612.

(0001719) Heavily Tessellated (reporter) 2009-11-09 11:08 edited on: 2009-11-09 11:47	Oops! There are two IES profiles in his attachment, which leads to confusion. I re-re-re-read his bug until I figured it out, but @ ATOM: Please help us retarded types by only including the relevant IES! :P In this case it does crash Lux, but I've edited the IES so it works fine, it's a Windows/Unix cross-edit problem, the first group of 10 parameters and the 2nd group of 3 should not have hard breaks, I'd guess it's from editing on Windows or something that forces CR+LF. Editing the file to correct this enables it to work. Perhaps the IES parser could be a bit more flexible to cross-platform slight formatting errors? While I can delete any post I put up here, apparently I cannot delete any attachments? Nice. :) Could someone delete the first two irrelevant images using the -other- IES profile Atom included? Thanks.

(0001720) Atom (reporter) 2009-11-09 11:37 edited on: 2009-11-09 11:38	I still get IES failures. Should I report that under another Mantis task? LuxRender does fail to parse some/MOST IES files.

(0001721) Heavily Tessellated (reporter) 2009-11-09 11:49 edited on: 2009-11-09 12:01	Dade just fixed a related issue, which is why his 1st reply links you to the one I filed... I'll attach the fixed version, and a sample render of your scene... I have many (around 80,000?) IES lights and while it would be nice if they all worked, I simply don't have the time to even write a script to batch-test which segfault Lux and which do not. But even the ones that don't crash, might not work as expected, especially the >180Â° laterally-occluded asymmetrics, like elaborate wall sconces... Hopefully someone will "find" the full 2002 spec or whatever is current, and we can get full support into the parsing routine.

(0001722) Dade (developer) 2009-11-10 01:12	Atom, ERCO_22412+QT-ax12_50W-12V_0-tilt.IES doesn't really look like a correct IES file: they are supposed to start (after few optional comment lines) with line with the string "IESNA" and it is missing that. The second file instead leads to a spectacular crash. As highlighted by Heavily Tessellated, it was a problem in the parsing code not handling some list of parameters split among several lines (it was assuming they were only on 2 lines). I fixed the problem and now your scene doesn't crash any more. Patch commited.

(0001869) Dade (developer) 2009-12-13 07:22	Same problem of BUG 612, patch lost, re-fixing.

Issue History
Date Modified	Username	Field	Change
2009-11-09 05:02	Atom	New Issue
2009-11-09 05:02	Atom	File Added: lux_volume_spotlight.zip
2009-11-09 06:38	Dade	Status	new => assigned
2009-11-09 06:38	Dade	Assigned To	=> Dade
2009-11-09 06:54	Dade	Note Added: 0001716
2009-11-09 06:55	Dade	Relationship added	related to 0000612
2009-11-09 11:08	Heavily Tessellated	Note Added: 0001719
2009-11-09 11:09	Heavily Tessellated	File Added: lux_volume_spotlight.png
2009-11-09 11:19	Heavily Tessellated	Note Edited: 0001719
2009-11-09 11:22	Heavily Tessellated	File Added: lux_volume_spotlight_atmospherics.png
2009-11-09 11:35	Heavily Tessellated	Note Edited: 0001719
2009-11-09 11:37	Atom	Note Added: 0001720
2009-11-09 11:38	Atom	Note Edited: 0001720
2009-11-09 11:47	Heavily Tessellated	Note Edited: 0001719
2009-11-09 11:49	Heavily Tessellated	Note Added: 0001721
2009-11-09 11:51	Heavily Tessellated	File Added: T21997t-repaired.ies
2009-11-09 11:59	Heavily Tessellated	Note Edited: 0001721
2009-11-09 12:00	Heavily Tessellated	Note Edited: 0001721
2009-11-09 12:01	Heavily Tessellated	Note Edited: 0001721
2009-11-09 12:39	Heavily Tessellated	File Added: lux_volume_spotlight-T21997t(stockholm).png
2009-11-10 01:12	Dade	Note Added: 0001722
2009-11-10 01:12	Dade	Status	assigned => feedback
2009-11-12 02:45	Dade	Requires Documentation Update	=> No
2009-11-12 02:45	Dade	Requires LuxBlend Implementation	=> No
2009-11-12 02:45	Dade	Requires LuxMaya Implementation	=> No
2009-11-12 02:45	Dade	Requires LuXSI Implementation	=> No
2009-11-12 02:45	Dade	Status	feedback => resolved
2009-11-12 02:45	Dade	Fixed in Version	=> 0.6.1
2009-11-12 02:45	Dade	Resolution	open => fixed
2009-12-13 07:22	Dade	Status	resolved => feedback
2009-12-13 07:22	Dade	Resolution	fixed => reopened
2009-12-13 07:22	Dade	Note Added: 0001869
2009-12-13 07:36	Dade	Status	feedback => resolved
2009-12-13 07:36	Dade	Resolution	reopened => fixed
2009-12-22 01:49	jeanphi	Status	resolved => closed