Add Road Network and Obstacle components to SUMO Reader#1128
Open
timothy-glover wants to merge 42 commits intodstl:mainfrom
Open
Add Road Network and Obstacle components to SUMO Reader#1128timothy-glover wants to merge 42 commits intodstl:mainfrom
timothy-glover wants to merge 42 commits intodstl:mainfrom
Conversation
…ertance for RadarBearing and RadarBearingRange
…ertance for RadarBearing and RadarBearingRange
…and relative edge caching
Co-authored-by: Henry Pritchett <87075245+hpritchett-dstl@users.noreply.github.com>
…s_clutter_detectable return
Co-authored-by: James Wright <69153443+jswright-dstl@users.noreply.github.com>
…arge obstacle numbers. Add in_obstacle method and removed obstacle_check flag
…ptional dependency. Modify VIBPF to handle multiple sensors
ae33df9 to
80a54ad
Compare
80a54ad to
08acf83
Compare
08acf83 to
dae75d9
Compare
Closed
…required, update tests and docs
dae75d9 to
a76c52b
Compare
a76c52b to
0929db5
Compare
…ved mapping from obstacle plotting, removed redundant intersection check and improved condition statements
0929db5 to
e35a91d
Compare
…rdinates==True. Requires network to be geo-referenced.
…ehicle types for filtering roads to import
e35a91d to
faa3839
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
NOTE: This pull request is dependent on the changes in #1096 to enable use of
Obstacleplatforms when processing buildings from SUMO. The first 21 commits are comparable to those in #1096, but rebased onto a more current version of main.This pull request introduces new components for the SUMO reader, initially introduced by @hpritchett-dstl in #827.
obstacle_genis a new method for reading polygons from the SUMO config file, filtering for polygons representing buildings, and constructing a list ofObstacleplatforms to represent them. Adjacent buildings that share a face are merged into a single obstacle.The second new component is the
road_network_genmethod which reads road network information from the SUMO config and returns a dictionary of edges representing the network. The user can provide an optional list of enumerators (VehicleTypeEnum) for specifying the roads which should be processed, those only allowing passenger vehicles for example. Edges are represented by straight line segments with each edge in the returned dictionary containing a sequence of coordinates representing the segments. The method attempts to combine multi lane roads into a single edge to reduce the size of the output. Tests are not provided due to the difficulty in simulatingSUMOGroundTruthReaderwithout an installation of SUMO and a suitable config file.