Wykres commitów

784 Commity (8ea8f90c55086c8dddc8dcf0fbb0e92ccfa8bf22)

Autor SHA1 Wiadomość Data
Taylor Smock 8ea8f90c55
Only run release code on code compiled for specific JOSM revision
Signed-off-by: Taylor Smock <tsmock@meta.com>
2023-02-22 12:38:00 -07:00
Taylor Smock d5e455b9f7
Move release code to JOSMPluginAction repo
Signed-off-by: Taylor Smock <tsmock@meta.com>
2023-02-22 09:19:31 -07:00
Taylor Smock c88b05021c
Fix #22760: IAE when adding a node inside two buildings
Signed-off-by: Taylor Smock <tsmock@meta.com>
2023-02-22 07:22:46 -07:00
Taylor Smock 68907f543c
Release on every push to the main branch
Signed-off-by: Taylor Smock <tsmock@meta.com>
2023-02-21 15:06:53 -07:00
Taylor Smock 968d61171a
Fix node deduplication occasionally merging nodes that should not be merged
Signed-off-by: Taylor Smock <tsmock@meta.com>
2023-02-21 15:01:43 -07:00
Taylor Smock e5b228bdba
Fix #22728: No URL for provider
Signed-off-by: Taylor Smock <tsmock@meta.com>
2023-02-14 08:27:57 -07:00
Taylor Smock e70f64e534
Fix #22683: IllegalArgumentException: Listener was already added
Signed-off-by: Taylor Smock <tsmock@meta.com>
2023-01-30 07:18:24 -07:00
Taylor Smock 00835859c5
Fix #22649: IAE: node is already deleted
Signed-off-by: Taylor Smock <tsmock@meta.com>
2023-01-16 07:08:53 -07:00
Taylor Smock 1c4b249284
StreetAddressTest: Handle upload check not looking at roads during visit phase
Signed-off-by: Taylor Smock <tsmock@meta.com>
2023-01-03 14:23:40 -07:00
Taylor Smock 1e774df4df
See #22624: Improperly added resultOffset to URLs sent to MapWithAI servers
Add non-regression test.

Signed-off-by: Taylor Smock <tsmock@meta.com>
2023-01-03 08:20:23 -07:00
Taylor Smock f4c7761d33
Fix #22624: Improperly added resultOffset to URLs sent to MapWithAI servers
Signed-off-by: Taylor Smock <tsmock@meta.com>
2023-01-03 07:32:45 -07:00
Taylor Smock 070e4fb9fe
MergeBuildingAddress: Don't delete duplicate addresses when added together
It is possible that the originating source didn't have addr:unit tags,
so other applications such as StreetComplete should detect duplicate
addresses and ask users if there are addr:unit tags, or which one is the
"main" building.

Furthermore, JOSM has a validator for duplicate addresses, so hopefully
the user will see the duplicate addresses and deal with them
appropriately, if it is obvious that one of the buildings is an
outbuilding.

Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-12-20 07:17:10 -07:00
Taylor Smock f251a0428e
MergeBuildingAddress: Only add ChangePropertyCommand if something will change
Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-12-20 06:49:05 -07:00
Taylor Smock de47e9c589
Significantly reduce memory usage by MergeBuildingAddress#getAddressPoints
This reduces CPU usage by 98.6% and memory allocations by 99.6%.
The time for adding all addresses in the
`39.69427;-86.104252;39.72361;-86.082427` download area went from ~1.25
minutes to ~10 seconds (~86% reduction).

Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-12-20 06:42:52 -07:00
Taylor Smock 84682ba8c6
StreetAddressTest: Significantly improve performance
This reduces CPU cycles by ~99% and memory allocations by ~74%.

Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-12-15 07:20:31 -07:00
Taylor Smock c453368368
StreetAddressOrder: Improve performance
This reduces CPU time in getNearbyAddresses by ~85% and memory
allocations by ~97%.

Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-12-15 07:20:31 -07:00
Taylor Smock f633e40322
Fix JavaDocs
Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-12-14 09:18:09 -07:00
Taylor Smock 9507670753
Paginate Esri sources
Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-12-14 09:05:47 -07:00
Taylor Smock b893f8a09e
Fix TM bound generation
Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-12-13 15:22:46 -07:00
Taylor Smock 01ce992d24
Update to non-deprecated methods
Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-12-07 07:23:42 -07:00
Taylor Smock 61d5b62689
MapWithAISourceReader#getBounds *must* return a modifiable list
Also, add @Territories annotation to a test that needed it

Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-12-07 06:02:37 -07:00
Taylor Smock 9d685d9baa
Workflows: Add report publishing
Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-12-06 14:32:52 -07:00
Taylor Smock 9a61f34d7f
Fix many SonarLint issues, mostly documentation
Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-12-06 14:32:30 -07:00
Taylor Smock 0fcc9e8ee0
Dependency updates
Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-12-06 08:52:26 -07:00
Taylor Smock ee0c576ee7
Fix #22509: stale URLs may cause downloads to fail
Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-11-17 10:32:48 -07:00
Taylor Smock 6d1ab18335
Attempt to fix #22511: Deadlock in DataSet#beginUpdate
This race occurs when
* MapWithAI is downloading and cleaning data *and*
* JOSM is performing a map paint

This attempts to fix the issue by getting a write lock
*prior to* entering any synchronized methods in DataSet.

Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-11-17 09:58:52 -07:00
Taylor Smock ed1af91aef
Move from Apache HTTP 4 to Apache HTTP 5
Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-11-07 16:14:24 -07:00
Taylor Smock 88a48a5fdb
Fix #22440: NPE in MapWithAIInfo#getConflationUrl
Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-10-11 10:46:43 -06:00
Taylor Smock ff3a9d38c3
Remove ivy (no longer necessary)
Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-10-10 12:26:49 -06:00
Taylor Smock 976f861309
Update GH Actions
Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-10-10 10:29:49 -06:00
Taylor Smock 96503e69d0
Update URL locations
Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-10-10 09:26:16 -06:00
Taylor Smock 207161b6c6
Fix #22316: CancellationException when cancelling after OSM download screen
Signed-off-by: Taylor Smock <tsmock@meta.com>
2022-10-06 15:24:06 -06:00
Taylor Smock 7380924ccc
UpdateProdTest: Use JOSM test rules to set revision (polluted following tests)
Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-10-05 15:31:08 -06:00
Taylor Smock 9419a57277
Add Plugin-Url to manifest generated by ant
Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-10-05 11:34:23 -06:00
Taylor Smock 26c0fda8d2
MapWithAICopyProhibitTest: Update testDestroyable for r18390
Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-10-05 11:34:23 -06:00
Taylor Smock cfb0c686c9
Initial GitHub actions
Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-10-05 11:34:23 -06:00
Taylor Smock 4d736c967f
Fix #22414: RuntimeException instead of OsmApiException is thrown from ForkJoinPool#join
Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-10-03 08:40:12 -06:00
Taylor Smock 9c8ebfe282
Dependency updates
Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-08-30 09:34:22 -06:00
Taylor Smock 5aa110b829
Update to gradle josm plugin 0.8.2 from 0.8.0
Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-08-30 06:57:50 -06:00
Taylor Smock 89c7d898ce
MapWithAILayerInfo: Fix an IllegalMonitorStateException
Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-08-30 06:57:49 -06:00
Taylor Smock 26d229fd3a
MapWithAIDataUtilsTest: Disable flaky test
Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-07-05 08:33:07 -06:00
Taylor Smock 7d7464f5da
Fix JOSM #22186: NPE in MapWithAIDataUtils#mergeDataSet
This is due to the localized message being null.

Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-07-05 07:23:03 -06:00
Taylor Smock 48751b16ec
ESRISourceReader: Account for `null` values for "accessInformation"
Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-06-28 12:43:44 -06:00
Taylor Smock 840cadb61b
Lint fixes
Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-06-22 12:26:24 -06:00
Taylor Smock b4a32e426c
GetDataRunnable: Move away from streams
Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-06-02 13:08:52 -06:00
Taylor Smock 9e00328d33
Decrease allocations in various locations
With the changes from
325c169468bce987bd93e78da81b92c12e52d86f..2c11de9e3ca95e5ea26ba08b053692ff013eb728,
this reduces allocations due to MapWithAI cleanup methods
(on download) from 1.7 GB to 265 MB.

Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-06-01 16:54:11 -06:00
Taylor Smock 7c3c0e72c2
MapWithAIDataUtils: Cleanup legacy methods
Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-06-01 10:29:25 -06:00
Taylor Smock 2c11de9e3c
GetDataRunnable: Significantly reduce allocations from merging nodes
Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-06-01 08:04:56 -06:00
Taylor Smock c9beb019ef
GetDataRunnable: Various cleanups
Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-06-01 08:04:56 -06:00
Taylor Smock 482ccffb8e
MapWithAIPreferenceHelper: Cache a property that is constantly hit
Signed-off-by: Taylor Smock <tsmock@fb.com>
2022-06-01 08:04:56 -06:00