Apparently overestimating j2c size is always a bad idea
This commit is contained in:
@@ -363,8 +363,12 @@ S32 LLImageJ2C::calcHeaderSizeJ2C()
|
||||
return FIRST_PACKET_SIZE; // Hack. just needs to be >= actual header size...
|
||||
}
|
||||
|
||||
static S32 cald_data_size_help(S32 w, S32 h, S32 comp, S32 discard_level, F32 rate)
|
||||
//static
|
||||
S32 LLImageJ2C::calcDataSizeJ2C(S32 w, S32 h, S32 comp, S32 discard_level, F32 rate)
|
||||
{
|
||||
// Note: this only provides an *estimate* of the size in bytes of an image level
|
||||
// *TODO: find a way to read the true size (when available) and convey the fact
|
||||
// that the result is an estimate in the other cases
|
||||
if (rate <= 0.f) rate = .125f;
|
||||
while (discard_level > 0)
|
||||
{
|
||||
@@ -375,22 +379,7 @@ static S32 cald_data_size_help(S32 w, S32 h, S32 comp, S32 discard_level, F32 ra
|
||||
discard_level--;
|
||||
}
|
||||
S32 bytes = (S32)((F32)(w*h*comp)*rate);
|
||||
return bytes;
|
||||
}
|
||||
|
||||
|
||||
//static
|
||||
S32 LLImageJ2C::calcDataSizeJ2C(S32 w, S32 h, S32 comp, S32 discard_level, F32 rate)
|
||||
{
|
||||
static const LLCachedControl<F32> exponent("SianaJ2CSizeExponent", 1.0f);
|
||||
static const LLCachedControl<S32> offset("SianaJ2CSizeOffset", 0);
|
||||
S32 size = cald_data_size_help(w, h, comp, discard_level, rate);
|
||||
S32 size_d0 = cald_data_size_help(w, h, comp, 0, rate);
|
||||
|
||||
llassert_always(size_d0);
|
||||
S32 bytes = size_d0*pow((double)size/size_d0, (double)exponent) + offset;
|
||||
bytes = llmax (0, bytes - 600);
|
||||
bytes = ((bytes + (1000 - 1))/1000)*1000 + 600;
|
||||
bytes = llmax(bytes, calcHeaderSizeJ2C());
|
||||
return bytes;
|
||||
}
|
||||
|
||||
|
||||
@@ -278,29 +278,6 @@
|
||||
</array>
|
||||
</map>
|
||||
|
||||
<key>SianaJ2CSizeExponent</key>
|
||||
<map>
|
||||
<key>Comment</key>
|
||||
<string>Exponent of size estimator. Should be between 0.0 and 1.0</string>
|
||||
<key>Persist</key>
|
||||
<integer>1</integer>
|
||||
<key>Type</key>
|
||||
<string>F32</string>
|
||||
<key>Value</key>
|
||||
<real>1.0</real>
|
||||
</map>
|
||||
<key>SianaJ2CSizeOffset</key>
|
||||
<map>
|
||||
<key>Comment</key>
|
||||
<string>Number of bytes to add to size estimator.</string>
|
||||
<key>Persist</key>
|
||||
<integer>1</integer>
|
||||
<key>Type</key>
|
||||
<string>S32</string>
|
||||
<key>Value</key>
|
||||
<integer>0</integer>
|
||||
</map>
|
||||
|
||||
<key>SianaLegacyJ2CSize</key>
|
||||
<map>
|
||||
<key>Comment</key>
|
||||
|
||||
Reference in New Issue
Block a user