Trainer mode (PPM_IN) made system-wide instead of model mixers

[GoogleCodeExporter]

What Version of the firmware are you running?

Repository UUID: f6e3ec8c-4711-3139-51d9-7b347ef5a272
Revision: 526
Node Kind: directory
Schedule: normal
Last Changed Author: gruvin
Last Changed Rev: 511
Last Changed Date: 2011-01-19 13:34:45 -0600 (Wed, 19 Jan 2011)

What steps will reproduce the problem?
Simple 4 channel set up.  TEAR

1.  Program channel 1 with switch THR with value -100 using EEP
 BTW EEP ver is below.

Revision: 246
Node Kind: directory
Schedule: normal
Last Changed Author: erezraviv
Last Changed Rev: 246
Last Changed Date: 2011-03-09 22:48:48 -0600 (Wed, 09 Mar 2011)

2.  Test in simulator.  This works.  hit the switch and the throttle output 
goes to -100  :)
3.  Upload to TX.  

What is the expected output? What do you see instead?

No throttle output in either switch position.

The safety switch menu on the TX does not show the programming that I did under 
eep.  :(

Before I programmed the throttle safety cutoff switch the throttle was 
functional.

Is your forked version compatible with eep?  Have you changed the structures 
stored on the eeprom?  Do you have a branch of eep that I should use?

What version of the product are you using? On what operating system?

Please provide any additional information below.

Original issue reported on code.google.com by wle...@gmail.com on 24 Mar 2011 at 12:48

[GoogleCodeExporter]

eeprom image from eep.

Original comment by wle...@gmail.com on 24 Mar 2011 at 12:50

Attachments:

• SkyTrainer.eepe

[GoogleCodeExporter]

So maybe this isn't a bug at all.  I don't believe your branch has this 
feature.  Do you have any plans to sync your tree up with er9x's to pick up his 
changes?

Original comment by wle...@gmail.com on 25 Mar 2011 at 2:31

[GoogleCodeExporter]

No changes to EEPROM format, only additional file format fir Fr-Sky settings 
storage.

There's no practical way I can make all the same changes as ER9X. The code 
tress are diverging significantly such that all such feature additions have to 
be manually coded/inserted and separately tested.

I do not understand what this missing feature even is though. What exactly is 
it supposed to do? THR mixing works for me in every way I can think of to use 
it. Is it merely an EEP compatibility issue? Has ER changed the EEPROM format 
since my fork?

Original comment by gru...@gmail.com on 26 Mar 2011 at 2:50

[GoogleCodeExporter]

Oh -- I misread. OK. Definitely an issue between EEP and Gruvinx9X. EEP is not 
formally supported by gruvin9x as this stage. I simply don't have the time or 
resources to keep up with ER's many ongoing changes. Sorry. (I did have 
intentions of adding the Fr-Sky EEPROM file format to EEP. But I haven't done 
so, and probably won't be doing so any time soon -- especially if EEP keeps 
adding new formats (is it?))

You might be better off simply sticking with the ER9X tree. SI there something 
g9x is providing that er9x is not?

Original comment by gru...@gmail.com on 26 Mar 2011 at 2:52

[GoogleCodeExporter]

Yes,  He added a safety switch menu which can be programmed to command an 
output to a fixed value.  For instance you can set the THR switch to override a 
channel output to a set value.  Say -100.  

I am beginning to understand things better so I suppose you could use the mixer 
to do the same thing.  (I just got my TX on Monday from HK.  :)

Since I wanted the switch cutoff feature I went back to er9x and ported your SW 
change for the PPM compatibility fix to his code base.  (Changed bit 6 on 
TCCRB3)  I also did the HW mod. I still need to test it out with a Spektrum TX 
though.  

I am curious to know what you had in mind to overlay the input PPM signal on 
top of the POTS on the TX.  I want that feature!  I am a novice pilot myself 
and I intend to use the PPM input at my club.

Original comment by wle...@gmail.com on 26 Mar 2011 at 12:42

[GoogleCodeExporter]

The latter PPM input (trainer) settings have now been implemented by bsongis in 
the branches/frsky development code. Basically, PPM_IN signal can be made to 
override (or add to) the physical sticks on the master unit. Each of the four 
input channels may be set to different weights (-100 to +100) and have a 
different (or no) activation switch assigned.

In the case of normal trainer operation, all channels would be set to := (input 
override), +/-100, with the TRN switch for activation.

Only RUD,ELE,THR and AIL are supported. These relate to PPM_OUT channels 1,2,3 
and 4, in the same order. 

(I'm not sure why that is the order. For me, THR, AIL, ELE, RUD would be better 
-- and Futaba users would be different again. But no matter. It works. We can 
address this as an option somehow, later.)

Original comment by gru...@gmail.com on 28 Jun 2011 at 6:08

• Changed state: Started

[GoogleCodeExporter]

Original comment by gru...@gmail.com on 28 Jun 2011 at 6:10

• Changed title: Trainer mode (PPM_IN) made system-wide instead of model mixers
• Added labels: Type-Enhancement
• Removed labels: Type-Defect

[GoogleCodeExporter]

Implementation completed by bsongis (r650). Great stuff. 

Tested working perfectly by gruvin, who then added code to have the TRAINER 
screen show the output channel names (RUD/ELE/THR/AIL) in the order they appear 
on the receiver, according to the "Rx Channel Order" setting in RADIO SETUP 
(was in MODE/TEMPLATES).

Original comment by gru...@gmail.com on 3 Jul 2011 at 8:50

• Changed state: Verified