NikolaiVChr / flightgear-saab-ja-37-viggen

Saab 37 Viggen for Flightgear flight simulator
http://wiki.flightgear.org/Saab_37_Viggen
GNU General Public License v2.0
27 stars 15 forks source link

Feature request: Implement high pressure cutoff #101

Closed colingeniet closed 4 years ago

colingeniet commented 4 years ago

The high pressure fuel cock is controlled by the thrust lever, see JA37 english manual sec. 9 (engine) par. 2.2.3.1 and 4. It is distinct from the low pressure fuel cock controlled by the cutoff switch.

To do:

NikolaiVChr commented 4 years ago

Is that what this image shows, throttle positions vs. engine response: So either 0-3 deg or 3-9 deg throttle position?

image

NikolaiVChr commented 4 years ago

That image is from the "Short RM8A engine description" document. I posted it in discord yesterday.

colingeniet commented 4 years ago

Cross checking your picture with JA37 english manual section 9.4:

For the cutoff catch, I see two ways:

Which would you prefer?

NikolaiVChr commented 4 years ago

Not sure, they both sound good. I will leave that to you. :)

colingeniet commented 4 years ago

By the way the positions of the AB indents on the pictures do not quite match the ones we have (rounded: 70,84,98, versus 70,82.5,95) so I will also change that if you see no issue.

NikolaiVChr commented 4 years ago

I see no issue, feel free to improve it. This is what I modeled what we have now from:

IMG_0948

NikolaiVChr commented 4 years ago

I think I took IDLE as 0 and scaled it that way, don't remember.

colingeniet commented 4 years ago

Oh, that's perfect with the two together. I mistook the 13-16 zone for the flight idle indent, but it actually is the ground idle catch. I guess 7-9 is the actual cutoff position, but does not have a physical marker on the throttle scale. I should have everything to do it correctly now.

NikolaiVChr commented 4 years ago

I could not click on the handle, but change 2 lines in this component and then you can click on handle:

<switch name="names/fcs/throttle-cutoff-catch">
            <default value="fcs/throttle-cutoff-catch"/>
            <!-- In the cutoff zone, always go back to resting position -->
            <test value="0">
                fcs/throttle-pos-deg-limited lt 13
            </test>
            <!-- If resting and beyond the idle position, engage -->
            <test logic="AND" value="-1">
                fcs/throttle-cutoff-catch == 0
                fcs/throttle-pos-deg-limited ge 15
            </test>
            <!-- If beyond flight idle, do not hold the lifted position.
                 This is purely for gameplay (IRL the lifted position would not hold anyway).
            -->
            <test logic="AND" value="-1">
                fcs/throttle-pos-deg-limited ge 24
            </test>
            <output>fcs/throttle-cutoff-catch</output>
        </switch>

Furthermore I can move the throttle in the entire range when it is both below the latch and above it. Thats not intended is it?

NikolaiVChr commented 4 years ago

Plus the launcher states dont work properly after these commits.