maliput / maliput_osm

BSD 3-Clause "New" or "Revised" License
2 stars 1 forks source link

Populates RoadNetwork with providers #36

Closed francocipollone closed 1 year ago

francocipollone commented 2 years ago

Summary

Populate providers for discrete, range value rules, and phases

Suggestions

State

Related to #12

francocipollone commented 2 years ago

[For the PhaseProvider]

I'd like to add a PhaseProviderBuilder in maliput::base. We already have one implementation in maliput_malidrive and it is quite basic so it could be part of maliput

https://github.com/maliput/maliput_malidrive/blob/main/src/maliput_malidrive/builder/phase_provider_builder.cc#L44-L68

That way we also avoid duplicating code. wdyt? @agalbachicar .

francocipollone commented 2 years ago

Same thing for hte Discrete and Range ValueRuleState providers:

agalbachicar commented 2 years ago

That way we also avoid duplicating code. wdyt? @agalbachicar .

Aye! Let's move it down to maliput-base

francocipollone commented 2 years ago

Tasks