Modify

Opened 3 years ago

Closed 3 years ago

#21669 closed defect (fixed)

[PATCH] Move plugin ejml to ivy, update ejml from 0.34 to 0.41

Reported by: taylor.smock Owned by: Don-vip
Priority: normal Milestone:
Component: Plugin Version:
Keywords: ejml Cc:

Description (last modified by taylor.smock)

I'm going to be uploading two patches. The first patch should be binary compatible with the current suite of plugins (it keeps the current ejml version). I am not updating the ejml version until I get the dependent plugins onto ivy.

The second patch updates ejml from 0.34 to 0.41.

See https://github.com/lessthanoptimal/ejml/blob/SNAPSHOT/change.txt for the full changelog for 0.34 -> 0.41.

ejml is used by the following plugins:

  • areaselector
  • cadastre-fr
    • Also depends upon geotools
  • geotools (non-ivy)
    • ImportImagePlugin (doesn't appear to have libs in tree)
    • Tracer-testing (tracer/tracer2 didn't appear to have libs in tree)
    • cadastre-fr (nothing needed)
    • matsim (gradle)
    • opendata (non-ivy)

Attachments (2)

21669.patch (7.1 KB ) - added by taylor.smock 3 years ago.
21669.0.41.patch (801 bytes ) - added by taylor.smock 3 years ago.
Update ejml from 0.34 to 0.41 (depends upon attachment:21669.patch)

Download all attachments as: .zip

Change History (12)

by taylor.smock, 3 years ago

Attachment: 21669.patch added

by taylor.smock, 3 years ago

Attachment: 21669.0.41.patch added

Update ejml from 0.34 to 0.41 (depends upon attachment:21669.patch)

comment:1 by taylor.smock, 3 years ago

Description: modified (diff)
Summary: [PATCH] Move plugin ejml to ivy[PATCH] Move plugin ejml to ivy, update ejml from 0.34 to 0.41

comment:2 by Don-vip, 3 years ago

Owner: changed from team to Don-vip
Status: newassigned

comment:3 by Don-vip, 3 years ago

In 35883/osm:

see #21669 - Move plugin ejml to ivy

comment:4 by Don-vip, 3 years ago

In 35885/osm:

see #21669 - Upgrade to EJML 0.41

comment:5 by Mkyral, 3 years ago

Hi,
Tracer-testing author there ;-)

After latest update of all plugins, LPIS module of tracer-testing stopped working. It generates no exception, action just silently die on this row:
https://github.com/mkyral/josm-tracer/blob/development/src/org/openstreetmap/josm/plugins/tracer/modules/lpis/krovak.java#L128

JOSM still works, as well as other tracer-testing modules. Only this LPIS module, where I need transformation from/to krovak CRS, does nothing.

I have no idea what is wrong and if it could be connected with this change, or it is something else.

comment:6 by Mkyral, 3 years ago

Update,
when I use old ejml.jar from backup, everything works as expected. Could this be an incompatibility with older geotools?

comment:7 by Mkyral, 3 years ago

Last working version is EJML 0.39.

in reply to:  6 comment:8 by taylor.smock, 3 years ago

Replying to Mkyral:

Update,
when I use old ejml.jar from backup, everything works as expected. Could this be an incompatibility with older geotools?

Definitely a possibility. I haven't gotten around to updating/touching geotools, but recompiling it and tracer2 might fix the problem.

comment:9 by Mkyral, 3 years ago

I've compiled latest geotools (26.1) and it does not work either. Also I've found this open issue:
https://osgeo-org.atlassian.net/browse/GEOT-6384 (Upgrade EJML to 0.38) - so it was probably not upgraded to v0.38 yet.

In ejml changelog is this in 0.40 part:

  • Strictly enforce usage of @Nullable to public API


  • CommonOps_DDRM and CommonOps_DSCC
    • Updating functions to return output if null and use common input sanity check functions
  • CommonOps_DSCC
    • duplicatesAdd() is a function which will add elements that are duplicated, turning it into a valid matrix again
    • Removed the functions below since they had a runtime complexity of O(N2) relative to matrix size instead of O(N)
      • multTransA(S,S,S), multTransB(S,S,S), innerProductLower(S,S,S)
      • Thanks Florentin Dörre for first noticing the performance issue

comment:10 by Don-vip, 3 years ago

Resolution: fixed
Status: assignedclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Don-vip.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.