Accuracy difference ended up being second case where invert() did not
produce quite as close a result as the previous code expected.
Andrea provided a lot of feedback, two of which were signification to this
discussion:
1) GeneralMatrix methods without javaodcs
Previously these were covered by the vecmath superclass. I have added
additional methods to XMatrix for method compatibility. This covers basics
like sub, mul, etc ... some of which we had already implemented.
2) Matrix3 duplicating functionality of GeneralMatrix, not optimized etc...
There are two ways to take this:
- Remove Matrix3,Matrix4 from the MatrixFactory - tried it and geotools
builds fine.
- Previously Matrix3 was worthwhile as it extended an optimized 3x3
implementation. I have found a FixedMatrix3x3_64F which we could use as a
delegate.
There is a pull request here: https://github.com/geotools/geotools/pull/836
for discussion/review.
--
Jody Garnett
On 7 May 2015 at 08:40, Jody Garnett <jody.garn...@gmail.com> wrote:
> Not yet still working on a succinct testable difference.
> On Thu, May 7, 2015 at 8:31 AM Eric Engle <een...@google.com> wrote:
>
>> "A wise man said: do not bring two compasses; bring one, or three."
>>
>> Anyone fired up a math tool to check the true exact result?
>>
>> Depending on the API you're using, EJML may do smarter things in several
>> cases to produce more accurate results.
>>
>> On Thu, May 7, 2015 at 8:23 AM, Jody Garnett <jody.garn...@gmail.com>
>> wrote:
>>
>>> I also captured the different inverse result encountered earlier (as a
>>> test case). I think I could take this one to the EJML mailing list as a
>>> bug/accuracy report.
>>>
>>> --
>>> Jody Garnett
>>>
>>> On 7 May 2015 at 00:41, Andrea Aime <andrea.a...@geo-solutions.it>
>>> wrote:
>>>
>>>> On Wed, May 6, 2015 at 11:43 PM, Jody Garnett <jody.garn...@gmail.com>
>>>> wrote:
>>>>>
>>>>> I am quite discouraged by the regression shown with GEOS-7009
>>>>> <https://osgeo-org.atlassian.net/browse/GEOS-7009> - in particular
>>>>> the same WMS request produces two different GetMap results (see the issue
>>>>> screen snaps) I am not sure what to make of that as all our rendering
>>>>> tests
>>>>> pass at the GeoTools level.
>>>>>
>>>>
>>>> The difference seems rather severe, even if it happened before the
>>>> rescaling, we are talking about a shift of 3 pixels up, 1 left, that seems
>>>> unlikely to be just a numerical glitch.
>>>> Trying to make a guess, and assuming the obvious math is taken care of,
>>>> how about matrix inversion? We do a lot of transformation inversion in our
>>>> code.
>>>>
>>>> Cheers
>>>> Andrea
>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> ==
>>>> GeoServer Professional Services from the experts! Visit
>>>> http://goo.gl/NWWaa2 for more information.
>>>> ==
>>>>
>>>> Ing. Andrea Aime
>>>> @geowolf
>>>> Technical Lead
>>>>
>>>> GeoSolutions S.A.S.
>>>> Via Poggio alle Viti 1187
>>>> 55054 Massarosa (LU)
>>>> Italy
>>>> phone: +39 0584 962313
>>>> fax: +39 0584 1660272
>>>> mob: +39 339 8844549
>>>>
>>>> http://www.geo-solutions.it
>>>> http://twitter.com/geosolutions_it
>>>>
>>>> *AVVERTENZE AI SENSI DEL D.Lgs. 196/2003*
>>>>
>>>> Le informazioni contenute in questo messaggio di posta elettronica e/o
>>>> nel/i file/s allegato/i sono da considerarsi strettamente riservate. Il
>>>> loro utilizzo è consentito esclusivamente al destinatario del messaggio,
>>>> per le finalità indicate nel messaggio stesso. Qualora riceviate questo
>>>> messaggio senza esserne il destinatario, Vi preghiamo cortesemente di
>>>> darcene notizia via e-mail e di procedere alla distruzione del messaggio
>>>> stesso, cancellandolo dal Vostro sistema. Conservare il messaggio stesso,
>>>> divulgarlo anche in parte, distribuirlo ad altri soggetti, copiarlo, od
>>>> utilizzarlo per finalità diverse, costituisce comportamento contrario ai
>>>> principi dettati dal D.Lgs. 196/2003.
>>>>
>>>>
>>>>
>>>> The information in this message and/or attachments, is intended solely
>>>> for the attention and use of the named addressee(s) and may be confidential
>>>> or proprietary in nature or covered by the provisions of privacy act
>>>> (Legislative Decree June, 30 2003, no.196 - Italy's New Data Protection
>>>> Code).Any use not in accord with its purpose, any disclosure, reproduction,
>>>> copying, distribution, or either dissemination, either whole or partial, is
>>>> strictly forbidden except previous formal approval of the named
>>>> addressee(s). If you are not the intended recipient, please contact
>>>> immediately the sender by telephone, fax or e-mail and delete the
>>>> information in this message that has been received in error. The sender
>>>> does not give any warranty or accept liability as the content, accuracy or
>>>> completeness of sent messages and accepts no responsibility for changes
>>>> made after they were sent or for other risks which arise as a result of
>>>> e-mail transmission, viruses, etc.
>>>>
>>>> -------------------------------------------------------
>>>>
>>>
>>>
>>>
>>> ------------------------------------------------------------------------------
>>> One dashboard for servers and applications across Physical-Virtual-Cloud
>>> Widest out-of-the-box monitoring support with 50+ applications
>>> Performance metrics, stats and reports that give you Actionable Insights
>>> Deep dive visibility with transaction tracing using APM Insight.
>>> http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
>>
>>
>>> _______________________________________________
>>> GeoTools-Devel mailing list
>>> GeoTools-Devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>>
>>>
------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel
