Skip to content

ECO CorneringCounter and ExtremeBrakeCounter#11

Closed
zakharenkodmytro wants to merge 1 commit intomainfrom
extreme_brake
Closed

ECO CorneringCounter and ExtremeBrakeCounter#11
zakharenkodmytro wants to merge 1 commit intomainfrom
extreme_brake

Conversation

@zakharenkodmytro
Copy link
Copy Markdown

  • added CorneringCounter and ExtremeBrakeCounter

@zakharenkodmytro
Copy link
Copy Markdown
Author

@KevinJoiner for viz

Copy link
Copy Markdown

@KevinJoiner KevinJoiner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know if we want both extremeBrake and harshBrake can we double check with @zer0stars

@zer0stars
Copy link
Copy Markdown
Member

zer0stars commented Jun 24, 2025 via email

@zakharenkodmytro
Copy link
Copy Markdown
Author

zakharenkodmytro commented Jun 24, 2025

I feel like it’s 1 thing, which is harsh braking, with a severity value. Sent from my PhoneOn Jun 24, 2025, at 09:59, Kevin Joiner @.> wrote: @KevinJoiner commented on this pull request. I don't know if we want both extremeBrake and harshBrake can we double check with @zer0stars —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.>

Thought that we agreed on that - https://discord.com/channels/892438668453740634/980261367808421939/1385675578778193920. @zer0stars

@zakharenkodmytro
Copy link
Copy Markdown
Author

I feel like it’s 1 thing, which is harsh braking, with a severity value. Sent from my PhoneOn Jun 24, 2025, at 09:59, Kevin Joiner @.> wrote: @KevinJoiner commented on this pull request. I don't know if we want both extremeBrake and harshBrake can we double check with @zer0stars —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: _@**.**_>

Thought that we agreed on that - https://discord.com/channels/892438668453740634/980261367808421939/1385675578778193920. @zer0stars

@zer0stars, please elaborate more on the above

@zer0stars
Copy link
Copy Markdown
Member

Does VSS allow us to add 2 values? or we'd have to have both:
harsh_braking_counter
harsh_braking_severity

@zakharenkodmytro
Copy link
Copy Markdown
Author

zakharenkodmytro commented Jun 30, 2025

Does VSS allow us to add 2 values? or we'd have to have both:
harsh_braking_counter
harsh_braking_severity

You can define signals with different purposes, such as counters and severity levels, but VSS does not inherently support mathematical operations like adding two values directly within the specification. So If we want both harsh_braking_counter and harsh_braking_severity, we would define them as separate signals.

If we need to calculate a combined value (e.g., adding the two), this would need to be handled in the application logic outside of VSS, as VSS is a data modeling specification and does not perform computations.

Though there is new datatype struct introduced, it is not yet supported by vss tools, imo I would not use it unless we have very good reason to.

@JamesReate
Copy link
Copy Markdown
Member

if the only values coming in are 0 or 1, should these be booleans instead?

@LDMO-Dev
Copy link
Copy Markdown

LDMO-Dev commented Jul 3, 2025

if the only values coming in are 0 or 1, should these be booleans instead?

There has been one case where the counter showed up as 2 for the acceleration counter, but we have been unable to recreate that. Also, the harsh and extreme brake has 4 values, unless we split? We can ask Domas on Tuesday, once he is back in, to figure out if this signal should be acting as a counter or a Boolean.

@zakharenkodmytro
Copy link
Copy Markdown
Author

closing this, @LDMO-Dev feel free to reopen if needed

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.

5 participants