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

List of channels isn't complete and doesn't represent the full list of output data from the ECU #109

Open
gslender opened this issue Oct 6, 2012 · 3 comments

Comments

@gslender
Copy link
Contributor

gslender commented Oct 6, 2012

I see we've enumerated the list of Guage Configurations, but this isn't the list of available Output Channels... so this is something that needs to be fixed. TS actually gives you the full list of [OutputChannels] in the ini to select from. This list also contains no duplicate, which is present in the [Guage Configurations] list.

@scudderfish
Copy link
Owner

That should be easy to create. Do you want to have a go or shall I?

@gslender
Copy link
Contributor Author

gslender commented Oct 7, 2012

This requires a major redo of the entire Gauges approach - are you sure you are ok for me to do this? I'll be ripping heaps out and changing a lot of code? ok?

G

@scudderfish
Copy link
Owner

Sure, go for it. The gauge thing worked when there were only gauges and
there weren't that many of them. We have too many Indicator types for it
to work well anymore.

On 7 October 2012 09:52, gslender [email protected] wrote:

This requires a major redo of the entire Gauges approach - are you sure
you are ok for me to do this? I'll be ripping heaps out and changing a lot
of code? ok?

G


Reply to this email directly or view it on GitHubhttps://github.com//issues/109#issuecomment-9206303.

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

No branches or pull requests

2 participants