OpenTrafficCam / OTAnalytics

Perform traffic analysis on trajectories of road users
https://opentrafficcam.org/OTAnalytics/
GNU General Public License v3.0
18 stars 3 forks source link

bug/4735-event-file-is-bigger-after-updating-event-creation #490

Closed briemla closed 7 months ago

briemla commented 7 months ago

OP#4735

briemla commented 7 months ago

The event creation has slowed down, I added some changes to speed up removing duplicate events from the event repository. It should be faster now. The benchmark is also running as of writing this comment. @briemla Can you review my changes. I'll approve this PR when I get your ok.

Looks good to me. I switched the invocation order in add. Duplicates will now be discarded before sorting as it already was done in add_all. This is to be consistent between the two add-methods. @randy-seng it is ready to be merged from my side.