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

Companion import of a 216 eeprom is broken #2378

Closed
kilrah opened this issue Jun 22, 2015 · 14 comments
Closed

Companion import of a 216 eeprom is broken #2378

kilrah opened this issue Jun 22, 2015 · 14 comments
Assignees
Milestone

Comments

@kilrah
Copy link
Member

kilrah commented Jun 22, 2015

Invalid data + crashes

@kilrah kilrah added this to the OpenTX 2.1.0 milestone Jun 22, 2015
@bsongis bsongis self-assigned this Jun 22, 2015
@bsongis
Copy link
Member

bsongis commented Jun 22, 2015

Would you attach an eeprom?

@kilrah
Copy link
Member Author

kilrah commented Jun 22, 2015

http://www.andrebernet.ch/divers/eeprom-good-2015-06-19-16-42-47.zip

I was looking at P-51, flight modes tab crashes

@projectkk2glider
Copy link
Member

The switch import is still wrong, the added SF- is causing problems. It probably should be fixed in class SwitchesConversionTable but it is hard for me to decipher what to do there.

@fraternl
Copy link
Contributor

My EEPE's can't be imported either. I'm assuming it's the same bug...
Should I post something or just wait and test after some fix is made?

@bsongis
Copy link
Member

bsongis commented Jun 28, 2015

Yes please attach your eeprom here and tell us what goes wrong, thanks!

@projectkk2glider
Copy link
Member

@fraternl be sure to use the latest nightly build before posting new problems.

@bsongis bsongis reopened this Jun 28, 2015
@bsongis
Copy link
Member

bsongis commented Jun 28, 2015

I will wait for some tests before closing this one.

@projectkk2glider
Copy link
Member

Still some problems with switches

In Companion 2.0.17:
switches2 0

In latest next:
switches2 1

L1, L7, L11 and L12 have problems.

@bsongis
Copy link
Member

bsongis commented Jun 28, 2015

I knew it wouldn't be so simple :)

bsongis added a commit that referenced this issue Jun 28, 2015
@projectkk2glider
Copy link
Member

It is good now!

@kilrah
Copy link
Member Author

kilrah commented Jun 28, 2015

Discussed via chat, but for the record still module settings broken and input names missing.

@bsongis
Copy link
Member

bsongis commented Jun 28, 2015

Input names are now good, is there anything else missing?

@kilrah
Copy link
Member Author

kilrah commented Jun 28, 2015

Looks all good now!

@bsongis
Copy link
Member

bsongis commented Jun 28, 2015

Cool!

@bsongis bsongis closed this as completed Jun 28, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants