Re: Graphics properties implementation

[Kai Habel]

Michael Goffioul schrieb:
> On 1/13/08, Kai Habel <address@hidden> wrote:
>   
>> Hello all,
>>
>> here is a patch to add missing patch properties.
>> But have a question:
>>
>> I am not sure when to use the 'a' qualifier for radio properties.
>> Michael has used both versions for the patch object e.g.:
>>
>> ...
>> radio_property linestyle , "{-}|--|:|-.|none"
>> ...
>> radio_property interpreter a , "{tex}|none|latex"
>>
>>
>> For the new radio properties ones I have always used the 'a' qualifier,
>> but is this ok?
>>     
>
> This modifier is actually ignored by the genprops.awk script. Currently,
> a, o and O modifiers are ignored, because of the uniformization of
> the property code; all set_xxx methods performs assignment from
> octave_value. These flags are probably not needed anymore.
>
> Concerning the patch, it looks good. What's then needed now is some
> "behavioral" code to react to some property changes:
> - face-vertex to X-Y-Z representation conversion and the otherway around
> - automatic computation of the normals
> - setting vertexnormals manually should set normalmode to true
> If you're willing to do it, please go ahead. I implemented these kind of
> things in JHandles, so you might want to take a look.
>
> Michael.
>
>   
Hello Michael,

yes I will look into the additional method needed, but it will take some
time. After a first look into the respective jhandles file [1] it seems
straightforward to translate the methods for face/vertex <-->  X/Y/Z
conversion and normals computation into C++.

I assume these functions should be called (if needed) from a not yet
existing validate method as well.

Kai

[1]
http://octave.svn.sourceforge.net/viewvc/octave/trunk/octave-forge/extra/jhandles/src/org/octave/graphics/PatchObject.java?revision=4440&view=markup