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

Magnetic Compass enhancements #1462

Merged
merged 2 commits into from Nov 16, 2023
Merged

Conversation

hbeni
Copy link
Contributor

@hbeni hbeni commented Nov 14, 2023

Magnetic Compass enhanced

  • simulated pivot mount
  • can get stuck when outside the pivots pitch limits
  • avionics have electrical influence on the bearing (compass correction is the same as it was before with all avionics on; by coincidence its the same)

(Ported over from the c182s)

- simulated pivot mount
- can get stuck when outside the pivots pitch limits
- avionics have electrical influence on the bearing
  (compass correction is the same as it was before with all avionics on)
@wlbragg
Copy link
Collaborator

wlbragg commented Nov 15, 2023

Wow. thanks @hbeni you've been busy. I take a look at everything you've done and get it merged when I get a free moment.

@hbeni
Copy link
Contributor Author

hbeni commented Nov 15, 2023

Cool,
maybe the animation and stuck-conditions need revision, but i think the current values are reasonable.
And these are easily to be changed just by parameters.
Currently enjoying the changes at Grenchen/GA-Wednesday and it looks good to me :)

@hbeni
Copy link
Contributor Author

hbeni commented Nov 15, 2023

Wow. thanks @hbeni you've been busy. I take a look at everything you've done and get it merged when I get a free moment.

When you try it, notice the following things:

  • When switching avionics on, the compass will move off course for some degrees.
  • When pitching up/pitching down, the compass disc will pitch/bank accordingly to "keep it straight to the earth surface
    (actually the magnetic field lines, but we assume perfect calibration to the location where you are currently flying; thus perfect calibration factory sided to the location)
  • When pitching up/down or banking off-limits the compass disc will hit the casing and be unable to move.
    Once out of that condition (currently set to +-9° pitch and +-25° bank), the compass will be free and rotate to the magnetic field again
  • In cruise flight, the compass is expected to move freely; in climb and descend quickly hit limits

@hbeni
Copy link
Contributor Author

hbeni commented Nov 15, 2023

Also, you really should consider adding the Compass correction card card to the cockpit somewhere (or the compass itfself); the values are the same as for the c182 (is i took the initial deviation table from the 172 :)
Notice: the version in master differs from the one in the compassDeviation branch (there was a correction at the easterly setting))

@wlbragg
Copy link
Collaborator

wlbragg commented Nov 16, 2023

So is the compass deviation card something a pilot simply refers to so they know how much offset to apply to the compass readout? The compass has no calibration dial to set it to read accurately?

@wlbragg
Copy link
Collaborator

wlbragg commented Nov 16, 2023

I also read the mailing list thread related to this. I guess we are not going to do this in c++ for now with

config/has-pivot and config/max-pivot-angle to select this behaviour on / off

@wlbragg
Copy link
Collaborator

wlbragg commented Nov 16, 2023

@hbeni

I'm a bit confused, I saw the compass deviation card in the c182. You don't have a magnetic compass like the 172 does (on top of the dash) I see how that applies to that compass. If you want to fly 90 then you set coarse to 92. But what about the compass on the panel, that has a calibration knob, yet looking at the card, it is different deviations for different headings, so one calibration change wouldn't apply to all?

How do you use that card in the c182?

@hbeni
Copy link
Contributor Author

hbeni commented Nov 16, 2023

So is the compass deviation card something a pilot simply refers to so they know how much offset to apply to the compass readout? The compass has no calibration dial to set it to read accurately?

No, it is a simple magnetic compass.
This gets calibrated from a technican (by adjudting the compendator magnets), and the remaining error is noted on the deviation card.
The Card shows the error that can not be corrected away.

@hbeni
Copy link
Contributor Author

hbeni commented Nov 16, 2023

I also read the mailing list thread related to this. I guess we are not going to do this in c++ for now with

config/has-pivot and config/max-pivot-angle to select this behaviour on / off

I also don‘t think it will be added shortly. Hence this addition.

@hbeni
Copy link
Contributor Author

hbeni commented Nov 16, 2023

@hbeni

I'm a bit confused, I saw the compass deviation card in the c182. You don't have a magnetic compass like the 172 does (on top of the dash) I see how that applies to that compass. If you want to fly 90 then you set coarse to 92. But what about the compass on the panel, that has a calibration knob, yet looking at the card, it is different deviations for different headings, so one calibration change wouldn't apply to all?

Are you refering to the Directional gyro?
That must be calibrated against the magnetic Compass; and its not the instrument in question here.
Every plane needs a simple magnetic compass by regulation.
The 172p has it mounted to the top of the panel and the 182 has it mounted to the windshield.

How do you use that card in the c182?

The card applies to the mag compass only.
It basicly tells you: „when the plane flies a magnetic heading of 90, the compass will show 94 because of all the metal of the plane and the avionics electricity“.
So, „to fly heading 90, steer 94 (on the mag compass)“.

@wlbragg wlbragg merged commit a8bc436 into c172p-team:master Nov 16, 2023
@wlbragg
Copy link
Collaborator

wlbragg commented Nov 16, 2023

OK, thanks for the explanation. Didn't see the 182 mag compass.

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

2 participants