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

Update rusefi_f407-discovery.ini with 3D maps #5944

Closed
wants to merge 4 commits into from

Conversation

zed65
Copy link
Contributor

@zed65 zed65 commented Feb 15, 2024

Added dedicated 3D maps

Added dedicated 3D maps
@zed65
Copy link
Contributor Author

zed65 commented Feb 15, 2024

3dTuning

@rusefillc
Copy link
Contributor

@zed65 ohh you mean all the way top level maps. I am hesitant to add, I am not convinced that it's worth the cost of screen "real estate". Somewhere on a narrow screen that new button would be a major problem for someone

@ElDominio do you have an opinion?

@zed65
Copy link
Contributor Author

zed65 commented Feb 15, 2024

3dTuning3
I understand, but what if we add them under "Fuel", my point is to get gauges and histogram to the left of the map

@rusefillc
Copy link
Contributor

my point is to get gauges and histogram to the left of the map

that I would happily merge! Still a bit confused what does that have to do with 3D since 3D and flat table seems to be separate unrelated switch? anyway please create new or update this PR?

@ElDominio
Copy link
Collaborator

But Tunerstudio generates 3d maps if you click the 3d map checkbox?

@zed65
Copy link
Contributor Author

zed65 commented Feb 15, 2024

Maybe I'm missing something but I'm used to be able to maximize the tune window and still have your favorite gauges there
Anyway I think I made the changes in this PR (I'm totally new to this fork and PR thing :))
3dTuning4

@rusefillc rusefillc closed this in 13ebd74 Feb 15, 2024
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.

None yet

3 participants