'[Sandy] more vector->Point3D quirks'

[prev in list] [next in list] [prev in thread] [next in thread] 

List:       osflash-sandy
Subject:    [Sandy] more vector->Point3D quirks
From:       petit () petitpub ! com (petit at petitpub ! com)
Date:       2009-01-11 23:18:39
Message-ID: 20090112001839.1iv1oqgxic8wswkg () webmail ! datakultur ! com
[Download RAW message or body]

Indeed :-) as those are in the data package and not the highest level API.
( Math gurus only here :)
/P

Makc <makc.the.great at gmail.com>: says:

> how about...  vectorMult3x3 ???
>
>
> On Sun, Jan 11, 2009 at 3:39 PM, kiroukou <kiroukou at gmail.com> wrote:
>> simply rename that method to something more readable.
>> transformPoint3x3() or something else.
>>
>> Thomas Pfeiffer
>> kiroukou at gmail.com
>>
>>
>>
>> Le 11 janv. 09 ? 14:32, petit at petitpub.com a ?crit :
>>
>>> That looks a ugly indeed.
>>> Also it breaks the naming convention that we tried to maintain.
>>>
>>> It was vectorMult3x3, so when doing that search and replace, I think it
>>> would have been better to tick the "March case" checkbox.
>>>
>>> What could we do?
>>> /P
>>>
>>> Makc <makc.the.great at gmail.com>: says:
>>>
>>>> I dont know about you guys but lines like
>>>> invModelMatrix.Point3DMult3x3 (m_oCurrentV); makes me feel
>>>> uncomfortable.
>>>>
>>>> On Sat, Jan 10, 2009 at 12:16 AM,  <petit at petitpub.com> wrote:
>>>>>
>>>>> I would think so yes.
>>>>> Just my 2 cents of course :)
>>>>> /P
>>>>>
>>>>> kiroukou <kiroukou at gmail.com>: says:
>>>>>
>>>>>> Hello guys,
>>>>>>
>>>>>> Well I thought I should do that change for 2 reasons:
>>>>>> - More consistent with the class name. Why calling that a Vector when
>>>>>> there's no Vector class (at least 3D one) in the API itself.
>>>>>> - People who upgrade their applications will only have to do what I did
>>>>>> myself, a simple find & replace.
>>>>>>
>>>>>> Does it make sense enough?
>>>>>>
>>>>>> Thomas Pfeiffer
>>>>>> kiroukou at gmail.com
>>>>>>
>>>>>>
>>>>>>
>>>>>> Le 9 janv. 09 ? 19:36, petit at petitpub.com a ?crit :
>>>>>>
>>>>>>> I know, replying to myself once again :)
>>>>>>>
>>>>>>> I see what you mean. There's always some important philosophical
>>>>>>> questions in math. Here, whether a triplet describing a point in
>>>>>>> space is
>>>>>>> a *point* or a *vector*.
>>>>>>>
>>>>>>> It certainly depends on how you see it or how you use it.
>>>>>>>
>>>>>>> If it's just to describe a point in space, it's a point.
>>>>>>> If you're into cross products, it's certainly a vector :-)
>>>>>>>
>>>>>>> /P
>>>>>>>
>>>>>>>> True. On the other hand it may be consistent with the change from
>>>>>>>> Vector to Point3D. What we get is not the new flash Vector, but our
>>>>>>>> new
>>>>>>>> Point3D.
>>>>>>>> I'd still want it to be SVector, SandyVector or something like that.
>>>>>>>>
>>>>>>>> As we now call this animal Point3D, I guess we could say getPoint3D
>>>>>>>> to,
>>>>>>>> well, get the bastard :-)
>>>>>>>>
>>>>>>>> /P
>>>>>>>>
>>>>>>>>> yeah but there was no real reason for getVector to getPoint3D
>>>>>>>>> change.
>>>>>>>>>
>>>>>>>>> On Fri, Jan 9, 2009 at 8:09 PM,  <petit at petitpub.com> wrote:
>>>>>>>>>>
>>>>>>>>>> Hmm.. I'm not sure about this.
>>>>>>>>>>
>>>>>>>>>> In the latest documentation for 3.1 it says for the Vertex class:
>>>>>>>>>>
>>>>>>>>>> getPoint3D():Point3D
>>>>>>>>>> Returns a Point3D representing the original x, y, z coordinates.
>>>>>>>>>>
>>>>>>>>>> This I believe is quite consistent with the code.
>>>>>>>>>> In the SVN trunk version it is the same.
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> Makc <makc.the.great at gmail.com>: says:
>>>>>>>>>>
>>>>>>>>>>> besides broken documentation, we now have method names changed,
>>>>>>>>>>> such
>>>>>>>>>>> as getPoint3D in Vertex class... imho we should roll back all such
>>>>>>>>>>> occurences
>>>>>>>>>>>
>>>>>>>>>>> _______________________________________________
>>>>>>>>>>> Sandy mailing list
>>>>>>>>>>> Sandy at osflash.org
>>>>>>>>>>> http://osflash.org/mailman/listinfo/sandy_osflash.org
>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> _______________________________________________
>>>>>>>>>> Sandy mailing list
>>>>>>>>>> Sandy at osflash.org
>>>>>>>>>> http://osflash.org/mailman/listinfo/sandy_osflash.org
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>> _______________________________________________
>>>>>>>>> Sandy mailing list
>>>>>>>>> Sandy at osflash.org
>>>>>>>>> http://osflash.org/mailman/listinfo/sandy_osflash.org
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> _______________________________________________
>>>>>>>> Sandy mailing list
>>>>>>>> Sandy at osflash.org
>>>>>>>> http://osflash.org/mailman/listinfo/sandy_osflash.org
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> _______________________________________________
>>>>>>> Sandy mailing list
>>>>>>> Sandy at osflash.org
>>>>>>> http://osflash.org/mailman/listinfo/sandy_osflash.org
>>>>>>
>>>>>>
>>>>>> _______________________________________________
>>>>>> Sandy mailing list
>>>>>> Sandy at osflash.org
>>>>>> http://osflash.org/mailman/listinfo/sandy_osflash.org
>>>>>
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> Sandy mailing list
>>>>> Sandy at osflash.org
>>>>> http://osflash.org/mailman/listinfo/sandy_osflash.org
>>>>>
>>>>
>>>> _______________________________________________
>>>> Sandy mailing list
>>>> Sandy at osflash.org
>>>> http://osflash.org/mailman/listinfo/sandy_osflash.org
>>>>
>>>
>>>
>>>
>>> _______________________________________________
>>> Sandy mailing list
>>> Sandy at osflash.org
>>> http://osflash.org/mailman/listinfo/sandy_osflash.org
>>
>>
>> _______________________________________________
>> Sandy mailing list
>> Sandy at osflash.org
>> http://osflash.org/mailman/listinfo/sandy_osflash.org
>>
>
> _______________________________________________
> Sandy mailing list
> Sandy at osflash.org
> http://osflash.org/mailman/listinfo/sandy_osflash.org
>

[prev in list] [next in list] [prev in thread] [next in thread] 