metadriverse / scenarionet

ScenarioNet: Scalable Traffic Scenario Management System for Autonomous Driving
Apache License 2.0
142 stars 21 forks source link

Use Waymo Protos Directly #38

Closed pimpale closed 7 months ago

pimpale commented 8 months ago

What changes do you make in this PR?

In this PR, we remove the dependency on waymo-open-dataset-tf-2-11-0 by including the Protobuf definitions inside the scenarionet repository itself.

This allows users to avoid having to install waymo-open-dataset-tf-2-11-0, which has many dependencies like numpy==1.5.1 which are outdated.

Checklist

pengzhenghao commented 8 months ago

@QuanyiLi Do we have license issue here?

pimpale commented 8 months ago

The protobuf definitions are under Apache 2.0, so it should hopefully be fine. I'm not sure under which license scenarionet is under though.

QuanyiLi commented 7 months ago

Like MetaDrive, ScenarioNet can use Apache 2.0 as well. So there won't be conflicts.