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

Add Back EMF and Percent Power Output to Display #335

Open
IshDeshpa opened this issue Jul 7, 2023 · 4 comments · May be fixed by #371
Open

Add Back EMF and Percent Power Output to Display #335

IshDeshpa opened this issue Jul 7, 2023 · 4 comments · May be fixed by #371
Assignees
Labels
Application Application Layer Issue enhancement New feature or request M2

Comments

@IshDeshpa
Copy link
Contributor

Both can be calculated/obtained from our motor controller.

@babusid
Copy link
Member

babusid commented Jul 11, 2023

@IshDeshpa What do you hope to gain (from a driver's perspective) by adding these? I would warn against making the driver consume too much information.

@IshDeshpa
Copy link
Contributor Author

The BackEMF would replace the accelerator slider at the bottom we have currently, as is present in most hybrid cars. Power percentage may not be completely necessary, but it could be put in the corner.

@babusid
Copy link
Member

babusid commented Jul 11, 2023

You have limited screen real estate. It may in fact be more prudent to use any excess to size more important information larger, as you can't make assumptions about future placements, and it needs to be more versatile.

@Cam0Cow
Copy link
Contributor

Cam0Cow commented Jul 11, 2023

Power is pretty important information, it'll let the driver know if they're being too heavy on the accelerator.

@Cam0Cow Cam0Cow added enhancement New feature or request Application Application Layer Issue labels Sep 23, 2023
@jiwoop jiwoop self-assigned this Sep 30, 2023
@jiwoop jiwoop linked a pull request Nov 18, 2023 that will close this issue
13 tasks
@IshDeshpa IshDeshpa added the M2 label Jan 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Application Application Layer Issue enhancement New feature or request M2
Projects
Status: No status
Development

Successfully merging a pull request may close this issue.

4 participants