Raffriff42: reply

2016-01-18T01:06:42Z

← Older revision		Revision as of 03:06, 18 January 2016
Line 1:		Line 1:
			[[Category:Talk]]
	This filter is broken Not sure about its purpose either. As you say it progressively darken the top of the image. So the luma goes from black (at the top) to the luma of the image (at thebottom). It has two defects though:<br>		This filter is broken Not sure about its purpose either. As you say it progressively darken the top of the image. So the luma goes from black (at the top) to the luma of the image (at thebottom). It has two defects though:<br>
	1) It is broken when you apply it to an image with height > 255 pixels. You will see a repetitive gradient, because of the line <br>		1) It is broken when you apply it to an image with height > 255 pixels. You will see a repetitive gradient, because of the line <br>
Line 5:		Line 6:
	2) It results in a crash if intercept > height of the image. It either should throw an error or use min(height, intercept) instead. <br>		2) It results in a crash if intercept > height of the image. It either should throw an error or use min(height, intercept) instead. <br>
	No wonder nobody uses this filter. Please file a bugreport at sf. [[User:Admin\|Admin]] 20:30, 17 January 2016 (CET)		No wonder nobody uses this filter. Please file a bugreport at sf. [[User:Admin\|Admin]] 20:30, 17 January 2016 (CET)

			:Yes, this thing should be retired; MaskTools can do a better job. --[[User:Raffriff42\|Raffriff42]] 02:06, 18 January 2016 (CET)

Admin: Created page with "This filter is broken Not sure about its purpose either. As you say it progressively darken the top of the image. So the luma goes from black (at the top) to the luma of the i..."

2016-01-17T19:30:59Z

Created page with "This filter is broken Not sure about its purpose either. As you say it progressively darken the top of the image. So the luma goes from black (at the top) to the luma of the i..."

New page

This filter is broken Not sure about its purpose either. As you say it progressively darken the top of the image. So the luma goes from black (at the top) to the luma of the image (at thebottom). It has two defects though: 
1) It is broken when you apply it to an image with height > 255 pixels. You will see a repetitive gradient, because of the line 
p[x*2] = max(0, BYTE(p[x*2]-subtract)); 
p[x*2]-subtract can get negative, and p[x*2] won't be zero in that case. 
2) It results in a crash if intercept > height of the image. It either should throw an error or use min(height, intercept) instead. 
No wonder nobody uses this filter. Please file a bugreport at sf. [[User:Admin|Admin]] 20:30, 17 January 2016 (CET)

Talk:FixLuminance - Revision history

Raffriff42: reply

Admin: Created page with "This filter is broken Not sure about its purpose either. As you say it progressively darken the top of the image. So the luma goes from black (at the top) to the luma of the i..."