I wrote
> But trying parametric{function{}} object has so far given a
> consistent Windows "Illegal operation" and aborting of
> MegaPov  very quickly after hitting "Run".
[ ... ]

The next snippet gives trouble
( "MegaPov attempted illegal operation"):
//////////////////////////////////////
#declare parametrictest=
    parametric
     {
       function
        f( u,v),
        g(u,v),
        h(u,v)
       <-6,-6>,<6,6>
     }
object{
     parametrictest
}
///////////////////////////////////

But not:

object{
     parametric
     {
       function
           f( u,v),
           g(u,v),
           h(u,v)
          <-6,-6>,<6,6>
}

And something about this sure seems familiar, from earlier
Superpatch use.

Post a reply to this message

Nathan Kopp wrote:

> With all of these changes, there are likely to be bugs

 Ken's Bug Log for MegaPov. I'll submit more as I find them but it is
likely to be much later today before I can work with the program more.


The following is a program crash related problem ---

//	Persistence of Vision Ray Tracer Scene Description File
//	File: trace_hf_pattern.pov
//	Date: 15/11/99
//	Demo showing the use of pattern and trace function.

This crash was repeatable and occured immediately after parsing and when
the display window appeared.


MEGAPOV caused an invalid page fault in
module MEGAPOV.EXE at 015f:00422585.
Registers:
EAX=00000000 CS=015f EIP=00422585 EFLGS=00010246
EBX=00917f38 SS=0167 ESP=0306f568 EBP=0306f618
ECX=00000000 DS=0167 ESI=00000001 FS=4027
EDX=00000000 ES=0167 EDI=00003856 GS=0000
Bytes at CS:EIP:
66 8b 14 48 33 c9 66 8b 0c 70 89 54 24 0c dd 5c 
Stack dump:
00000000 0306f728 0306f740 00003856 004d0672 00000001
00519b98 0306f594 00000002 0000002b 00000001 00352e30
0306f5b8 0000001f 0306f820 0306f647



  All of the following are syntax bugs found in the example scenes.
I suspect there is a naming collision with a reserved function name
i.e. image_height and image_width. Changing to image_h and image_w
allows the scenes to render as expected.

//	Persistence of Vision Ray Tracer Scene Description File
//	File: double_illuminate.pov
//	Date: 15/11/99
//	Demo double_illuminate
	
This does not work

camera {
	location < 0.0, 5, -2.5 >
#declare image_height = 240;
#declare image_width =  320;
	up image_height*y
	right image_width*x
	angle 35
	look_at < 0.0, 0.0, 0.0 >
}

This does work

camera {
	location < 0.0, 5, -2.5 >
#declare image_h = 240;
#declare image_w =  320;
	up image_h*y
	right image_w*x
	angle 35
	look_at < 0.0, 0.0, 0.0 >
}

//	Persistence of Vision Ray Tracer Scene Description File
//	File: obj_controls.pov
//	Date: 14/11/99
//	Showing no_image, no_reflection, no_shadow and projected through

Same as above problem - Change image_height and image_width renders ok.	

camera {
	location <0.0, 3, -20>
#declare image_height=240;
#declare image_width=320;
      up image_height*y
   right image_width*x
	look_at <0.0, 0.0, 0.0>
	angle 25
}

//	Persistence of Vision Ray Tracer Scene Description File
//	File: warp_maps.pov
//	Date: 15/11/99
//	Demo showing the use warps in pigment
	
Same as above two plus the declaration has 2 duplicates that are not necessary.

camera {
	location <0.0, 1.5, -11>
#declare image_width=320;
#declare image_height=240;
#declare image_width=320;
#declare image_height=240;
	up y*image_height	right x*image_width
	angle 45
	look_at <0.0, 0.0, 0.0>
}

// persistence of Vision Ray Tracer Scene Description File
// Needs Isoblob patch.
//	File: isoblob_mace.pov
//	Date: 14/11/99
//	Isoblob example scene

Same as above

camera {
	perspective 
	location <0.0, 0.0, -8>
#declare image_height=240;
#declare image_width=320;
	up y*image_height right x*image_width
	angle 55
	look_at <0.0, 0.0, 0.0>
}

//	Persistence of Vision Ray Tracer Scene Description File
//	Isosurface function demo
	
Same as above

camera {
	perspective 
	location < 0.0, 0.25, -2 >
#declare image_width=320;
#declare image_height=240;
	up y*image_height right x*image_width
	angle 45
	look_at < 0.0, 0.0, 0.0 >
	rotate y*3.5
}

-- 
Ken Tyler -  1200+ Povray, Graphics, 3D Rendering, and Raytracing Links:
http://home.pacbell.net/tylereng/index.html http://www.povray.org/links/

Post a reply to this message

>Well, yeah, that's one of my goals.  Also, Nathan seems to be better able
>to actually create a shipping product from a bunch of cool patches, whereas
>I just sit around and tell y'all how cool the next one's gonna be without
>ever taking the time to write the documentation and put it online.


I think you've done a brilliant job anyway :)
I'm quite certain that I'm not the only one with this opinion.

Post a reply to this message

Ken <tyl### [at] pacbellnet> wrote:

> 
>   All of the following are syntax bugs found in the example scenes.
> I suspect there is a naming collision with a reserved function name
> i.e. image_height and image_width. Changing to image_h and image_w
> allows the scenes to render as expected.
> 
> //    Persistence of Vision Ray Tracer Scene Description File
> //    File: double_illuminate.pov
> //    Date: 15/11/99
> //    Demo double_illuminate
>   
> This does not work
> 
> camera {
>   location < 0.0, 5, -2.5 >
> #declare image_height = 240;
> #declare image_width =  320;
>   up image_height*y
>   right image_width*x
>   angle 35
>   look_at < 0.0, 0.0, 0.0 >
> }
> 
> This does work
> 
> camera {
>   location < 0.0, 5, -2.5 >
> #declare image_h = 240;
> #declare image_w =  320;
>   up image_h*y
>   right image_w*x
>   angle 35
>   look_at < 0.0, 0.0, 0.0 >
> }
> 

Yes, image_height and image_width are reserved keywords now.
They 'read' the image size entered in the options.

If you want to set (write) your own image_size, use the  keyword
ini_option "..."
or #declare Image_height and Image_width, using a capital letter in the
identifier.
I would like to emphasize the importance of using at least one Capital
letter in identifiers. Reserved keywords will always use only small
letters, therefore there will be no conflict with identifiers having a
capital letter. Not now and not in the future when new keywords will be
added!
The same goes for declared variables in macros. Please make it a habit
for future identifiers.

Smellenbergh


-- 
e-mail:sme### [at] skynetbe

http://users.skynet.be/smellenbergh

Post a reply to this message

Ken wrote:

>
>
>Nathan Kopp wrote:
>
>> With all of these changes, there are likely to be bugs
>
> Ken's Bug Log for MegaPov. I'll submit more as I find them but it is
>likely to be much later today before I can work with the program more.
>
>
>The following is a program crash related problem ---
>
>//     Persistence of Vision Ray Tracer Scene Description File
>//     File: trace_hf_pattern.pov
>//     Date: 15/11/99
>//     Demo showing the use of pattern and trace function.
>
>This crash was repeatable and occured immediately after parsing and when
>the display window appeared.

Same here, it also crashes in the superpatch.
I added a #debug after i=i+i and j=j+1, and it renderd ok, but only once and 
never since.

Ingo

-- 
Photography: http://members.home.nl/ingoogni/
Pov-Ray    : http://members.home.nl/seed7/

Post a reply to this message

Ken <tyl### [at] pacbellnet> wrote...
>
> This does not work
>
> camera {
> location < 0.0, 5, -2.5 >
> #declare image_height = 240;
> #declare image_width =  320;
> up image_height*y
> right image_width*x
> angle 35
> look_at < 0.0, 0.0, 0.0 >
> }

Oops!!!

That's from when I first tried to test some SuperPatch demos... I had not
yet incorporated the image_height and image_width patches, so I had to
#declare the variables to get the scenes to parse correctly.  If you just
delete the two #declare statements (or four for that one scene), things will
work fine.

-Nathan

Post a reply to this message

Ken <tyl### [at] pacbellnet> wrote...
>
> The following is a program crash related problem ---
>
> // Persistence of Vision Ray Tracer Scene Description File
> // File: trace_hf_pattern.pov
> // Date: 15/11/99
> // Demo showing the use of pattern and trace function.
>
> This crash was repeatable and occured immediately after parsing and when
> the display window appeared.

This is apparantly a problem with the 'pattern' image type (using a pattern
to generate the height-field).  Unfortunately, I don't know exactly what
causes the problems (a bounds issue, I think).  Anyway, I've made the
height-field code a bit more fault-tolerant so that this will no longer
cause a crash.

-Nathan

Post a reply to this message

Nieminen Juha <war### [at] punarastascstutfi> wrote...
>   However I noticed that something weird happens with normals (as someone
> else has already noticed). For example the following scene renders quite
> weird and completely different from my original patch or the (fixed) uvpov
6.2.
> The correct image can be seen in p.b.images (in a thread called
"interesting
> texture using the fractal patch" or something similar):

The fix in UVPov 6.2 was like just a band-aid.  Now I think I've fixed it at
the source of the problem.  See my recent post in povray.bugreports for more
info (I think it was a reply to somebody else's bug report).

-Nathan

Post a reply to this message

Nathan Kopp wrote:

> Oops!!!
> 
> That's from when I first tried to test some SuperPatch demos... I had not
> yet incorporated the image_height and image_width patches, so I had to
> #declare the variables to get the scenes to parse correctly.  If you just
> delete the two #declare statements (or four for that one scene), things will
> work fine.
> 
> -Nathan

I really brought it up in case you wished to fix your demo scenes.
I figured out how to work around the problem on the first scene I
encountered with that camera statement :)

-- 
Ken Tyler -  1200+ Povray, Graphics, 3D Rendering, and Raytracing Links:
http://home.pacbell.net/tylereng/index.html http://www.povray.org/links/

Post a reply to this message

> Nathan Kopp wrote:
> 
> > With all of these changes, there are likely to be bugs
 
 I'll submit more as I find them...


// Persistence Of Vision raytracer version 3.1e (isosurface patch)
// sample file by R.Suzuki   <rsu### [at] etlgojp>  Apr.'99
// Torus2.pov from the iso_demo directory

#include "isocom.inc"

#declare R0=5;
#declare R1=0.7;
#declare FUNC1=function {"torus",<R0,R1>}

#declare TH= function {"TH"} //function{"FUNC1"}

isosurface {
     function {FUNC1(x,y+cos(TH*7)*0.3,z)}
     bounded_by{ box {<-R0-R1,-R0,-R0-R1>,<R0+R1,R0,R0+R1>}} 
     eval
     accuracy 0.001 
     scale <1,1,1>
     texture {ComYw}
}


The above returns the following warning:

#declare TH= function{"TH"}

isosurface {
     function {FUNC1(x,y+cos(TH* <----ERROR

F:\Program Files\POV-Ray for Windows v3.1\megapov\demos\Multipatch NPov\
Isosurface_isoblob\Iso demo\torus2.pov:15: error: 

The function 'TH' needs arguments. Should be TH(...)

Returned from renderer (non-zero return value)


If I replace #declare TH = function{"FUNC1"} the program crashes with the
following page fault

MEGAPOV caused an invalid page fault in
module MEGAPOV.EXE at 015f:0048a304.
Registers:
EAX=00000006 CS=015f EIP=0048a304 EFLGS=00010202
EBX=00000003 SS=0167 ESP=0306ee28 EBP=00916b48
ECX=00000000 DS=0167 ESI=00000000 FS=1bcf
EDX=0a524f52 ES=0167 EDI=00916b18 GS=0000
Bytes at CS:EIP:
8b 14 81 52 e8 a3 98 00 00 8b 0d 10 b0 51 00 83 
Stack dump:
00000000 00493cb5 0043e92d 004dda24 004dc598 000006e1
00916b48 0041ab75 00000000 004dc598 000006e1 00000000
0041abe3 00916b48 00916b48 0046db54
 

-- 
Ken Tyler -  1200+ Povray, Graphics, 3D Rendering, and Raytracing Links:
http://home.pacbell.net/tylereng/index.html http://www.povray.org/links/

Post a reply to this message