I think we are close to wrapping up the review of v3.7 example scene files 
so I wanted to summarize and get our plan "blessed" before proceeding with 
the actual work.



Division of work: This is how the scenes have been divided up.

advanced/animations (71)
camera/incdemo (74)                (Stephen has already spoken up for this 
group)
interior/language/lights/objectmods/qtvr/radiosity (69)
objects/portfolio (78)
textures (73)



Image size and format: 768 x 576 (4:3 aspect ratio) PNG. Using the file 
advanced/grenadine as a test case I noticed a difference in the preview 
window image (to dark) and the final png image file (to light). This led me 
to believe that I hadn't set display_gamma correctly. After some empirical 
testing this is what I came up with for my quickres.ini file:



[768x576, AA 0.3]

Width=768

Height=576

Antialias=On

Antialias_Threshold=0.3

Display_Gamma=2.3



I'm going to post the image over in the images group. One last thing about 
image quality. The rim of the glass had jagged edges until I used +a0.1 
+r4 ... we DO want to resolve quality issues like that don't we?



Scene file edits: This is how assumed_gamma and source file versioning is 
handled.

This example (below) has an assumed_gamma setting, so in order to leverage 
off the 3.7 default behavior the #version 3.5 directive has been 
conditionally excluded, as well as the assumed_gamma setting in the global 
settings block. Additionally any mention of version in header comments have 
been deleted , and a date stamp with initials was added. Example files that 
don't have assumed_gamma in global settings block should be (if not already) 
branded #version 3.5 as the default behavior is gamma correction off.



---begin scene file frag

// Persistence of Vision Ray Tracer Scene Description File

// File: grenadine.pov

// Desc: Glass with liquid

// Date: 1999/06/04

// Auth: Ingo Janssen

//

// -w320 -h240

// -w800 -h600 +a0.3



// Updated: 04Aug2008(jh) for v3.7 distribution



#if (version < 3.6)

  #version 3.5

#end



#include "glass.inc"

#include "lemon.inc"



global_settings {

   // conditionally comply with gamma correction handling

   #if (version < 3.6)

        assumed_gamma 1.0

   #end

   max_trace_level 5

   photons {

      spacing 0.01  // higher value 'lower' quality, faster parsing.

      autostop 0

      jitter 0.5

      max_trace_level 15

  }

}

--- end scene file frag



Notice from the scene frag above that it calls and include "lemon.inc" which 
is an include in scenes/advanced. It had the #version directive in the body 
of that include. I used the following common version preservation method 
used in the application includes.



--- begin code frag

// Added to comply with common version preservation scheme

// 04Aug2008 (jh)

#ifndef(Lemon_Inc_Temp)

#declare Lemon_Inc_Temp=version;

#version 3.5;



#ifdef(View_POV_Include_Stack)

    #debug "including lemon.inc\n"

#end



// code body goes here



// Added to comply with common version preservation scheme

// 04Aug2008 (jh)

#version Lemon_Inc_Temp;

#end

--- end code frag



Include files review:

The include files have been reviewed and  since they have a scheme that 
keeps the calling source file from inheriting it's #version branding it's 
been suggested that this is a moot point to change the #version branding for 
now. In other words it's purely a housekeeping issue for now. Since we're on 
the topic of includes, I've heard no reply on my proposal to change the 
textures in glass_old.inc to materials, and remove the I_Glass warning in 
glass.inc. The above scene file frag calls glass.inc so a minor mod to the 
scene file was in order. The scene file rendered fine. I don't want to do 
this because silence implies agreement. Someone yea or nay this change 
please!



Have I forgotten anything?



Jim

Post a reply to this message