Closed karlicoss closed 1 year ago
Oh, another bunch of warnings I noticed is latitudeE7
missing on about 10 semantic entries historically. Happy to make it a bit more defensive too and just ignore if coordinates are missing if you think it makes sense
Thanks, yeah making optional sounds good
yeah, if its missing latlon I think its worth just sending a logger.debug
and continue
ing, I dont think its worth parsing info that doesnt have the location
done, ended up even a bit cleaner I'm using an editable local setup and might PR a few more minor patches later so no need to rush in releasing after merging!