Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test bumping ref to see if it fails pipelines #2102

Closed
wants to merge 5 commits into from

Conversation

devtekve
Copy link
Contributor

No description provided.

@sunnyhaibin
Copy link
Contributor

sunnyhaibin commented Dec 11, 2024

Introduced in commaai/opendbc@551081b

#16 2.626 Traceback (most recent call last):
#16 2.626   File "/tmp/opendbc_repo/opendbc/dbc/generator/tesla/tesla_radar_continental.py", line 4, in <module>
#16 2.626     from opendbc.dbc.generator.tesla._radar_common import get_radar_point_definition, get_val_definition
#16 2.626 ModuleNotFoundError: No module named 'opendbc'

@sunnyhaibin
Copy link
Contributor

Broken if using this ref: commaai/opendbc@551081b
Working if using this ref: commaai/opendbc@fb6ad76

cc @sshane

@devtekve
Copy link
Contributor Author

devtekve commented Dec 11, 2024

Yeah can confirm commaai/opendbc@551081b is no longer compatible with Panda

@devtekve
Copy link
Contributor Author

resolved by #2103

@devtekve devtekve closed this Dec 11, 2024
@devtekve devtekve deleted the test_bump_ref_opendbc branch December 11, 2024 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants