Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#139 closed defect (fixed)

increase efficiency for new XYZ/Geo datasets

Reported by: Kevin Milner Owned by: Kevin Milner
Priority: major Milestone: OpenSHA 1.2
Component: commons Version:
Keywords: Cc:

Description (last modified by Kevin Milner)

Here are numbers for the current implementations of the GeoDataSet? interface. Each number is for 100,000 invocations of each method. For large datasets, this is unacceptable.

Change History (4)

comment:1 Changed 13 years ago by Kevin Milner

Description: modified (diff)

comment:2 Changed 13 years ago by Kevin Milner

Done in [7413]...basically the culprit was an ArrayList?.contains() call, which has now been replaced by HashMap?.containsKey(). Put operations are now trivially fast.

I think this change is worth merging into the 1.1 release, and releasing 1.1.1. For example, ShakeMaps? could be INCREDIBLY slow without this fix (if they have >40,000 points).

comment:3 Changed 13 years ago by Kevin Milner

Resolution: fixed
Status: newclosed

comment:4 Changed 13 years ago by Kevin Milner

just re-released 1.1 (as version 1.1.1) with this fix

Note: See TracTickets for help on using tickets.