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

Hyundai: Synchronize flag names #2101

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sunnyhaibin
Copy link
Contributor

@sunnyhaibin sunnyhaibin commented Dec 10, 2024

Description

  • Refactor flag names in Hyundai safety to make them more descriptive and consistent.
  • Variable names starting with HYUNDAI_PARAM_ have been updated to start with FLAG_HYUNDAI_.
  • The functionality remains unchanged; only the naming has been updated for improved clarity and readability.
  • This change helps maintain a standardized naming convention across the Python and C++ codebase, making future development and debugging easier.
  • No logic or functional behavior of the code is altered in this PR.

panda/python/__init__.py

Lines 197 to 204 in bedee40

FLAG_HYUNDAI_EV_GAS = 1
FLAG_HYUNDAI_HYBRID_GAS = 2
FLAG_HYUNDAI_LONG = 4
FLAG_HYUNDAI_CAMERA_SCC = 8
FLAG_HYUNDAI_CANFD_HDA2 = 16
FLAG_HYUNDAI_CANFD_ALT_BUTTONS = 32
FLAG_HYUNDAI_ALT_LIMITS = 64
FLAG_HYUNDAI_CANFD_HDA2_ALT_STEERING = 128

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.

1 participant