Skip to content

Commit

Permalink
feat(a380x/fcu): Fix visual issues with FCU light test and missing la…
Browse files Browse the repository at this point in the history
…bels (#9570)

* fix(A380X/FCU): fix FCU light test to display correct test values

* fix(A380X/FCU): Update FCU font to fix decimal point

* changed width of decimal point in FCU which didn't match the real aircraft

* updated font also fixes issue of overlapping text during light test

* feat(A380X/FCU): Add TRUE label and corrected position of FPA label in FCU

* Add TRUE label to FCU

* Move FPA label to the correct position (left of V/S)

* Move degree symbol closer to numbers based on reference photo

* docs: update changelog

* docs: update changelog

* docs: clean up changelog

* feat(a380x/fcu): update VS light test with correct second char

* feat(A380X/FCU): Add light test patterns for EFIS CP labels

* chore(A380X/FCU): minor refactor VS labels

* feat(A380X/FCU): Add light test for pre-select baro setting

* add light test for pre-select baro setting

* removed QFE label as QFE systems are not implemented and in the wrong position

* feat(A380X/FCU): Change display of FPA value to use the correct digits display

* chore: code cleanup

* docs: update changelog order

* clean up subscription function for light test.
  • Loading branch information
heclak authored Dec 8, 2024
1 parent 5c8071b commit be06c3f
Show file tree
Hide file tree
Showing 13 changed files with 343 additions and 44 deletions.
2 changes: 1 addition & 1 deletion .github/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@
1. [A380X/LIGHTS] Fix function of FCU brightness knobs - @heclak (Heclak)
1. [A380X/FWS] Fix "NO ZFW OR ZFWCG DATA" ECAM alert after landing - @flogross89 (floridude)
1. [A380X/SD] Add brake temperature color change to amber when brakes are hot - @heclak (Heclak)

1. [A380X/FCU] Fix display of values on FCU during light test - @heclak (Heclak)

## 0.12.0

Expand Down
Binary file not shown.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
195 changes: 182 additions & 13 deletions fbw-a380x/src/fonts/A380X_FCU.sfd
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ OS2Version: 4
OS2_WeightWidthSlopeOnly: 0
OS2_UseTypoMetrics: 1
CreationTime: 1603675803
ModificationTime: 1706370840
ModificationTime: 1732268502
PfmFamily: 17
TTFWeight: 400
TTFWidth: 5
Expand Down Expand Up @@ -85,8 +85,8 @@ NameList: AGL For New Fonts
DisplaySize: -48
AntiAlias: 1
FitToEm: 0
WinInfo: 58 29 11
BeginChars: 65539 36
WinInfo: 0 29 11
BeginChars: 65539 37

StartChar: .notdef
Encoding: 65536 -1 0
Expand Down Expand Up @@ -218,21 +218,21 @@ EndChar

StartChar: period
Encoding: 46 46 5
Width: 180
Width: 0
GlyphClass: 1
Flags: W
LayerCount: 2
Fore
SplineSet
28 -100 m 128,-1,1
28 -71 28 -71 49 -50 c 128,-1,2
70 -29 70 -29 99 -29 c 128,-1,3
128 -29 128 -29 149 -50 c 128,-1,4
170 -71 170 -71 170 -100 c 128,-1,5
170 -129 170 -129 149 -150 c 128,-1,6
128 -171 128 -171 99 -171 c 128,-1,7
70 -171 70 -171 49 -150 c 128,-1,0
28 -129 28 -129 28 -100 c 128,-1,1
-70 -100 m 128,-1,1
-70 -71 -70 -71 -49 -50 c 128,-1,2
-28 -29 -28 -29 1 -29 c 128,-1,3
30 -29 30 -29 51 -50 c 128,-1,4
72 -71 72 -71 72 -100 c 128,-1,5
72 -129 72 -129 51 -150 c 128,-1,6
30 -171 30 -171 1 -171 c 128,-1,7
-28 -171 -28 -171 -49 -150 c 128,-1,0
-70 -129 -70 -129 -70 -100 c 128,-1,1
EndSplineSet
Validated: 1
EndChar
Expand Down Expand Up @@ -1993,6 +1993,7 @@ SplineSet
159 798 159 798 191 766 c 128,-1,8
223 734 223 734 223 689 c 128,-1,9
EndSplineSet
Validated: 1
EndChar

StartChar: d
Expand Down Expand Up @@ -2208,5 +2209,173 @@ LayerCount: 2
Fore
Validated: 1
EndChar

StartChar: asterisk
Encoding: 42 42 36
Width: 800
Flags: W
LayerCount: 2
Fore
SplineSet
451 403 m 4,0,1
447 396 447 396 440 396 c 6,2,-1
354 396 l 6,3,4
347 396 347 396 343 403 c 4,5,6
332 421 332 421 323 437 c 4,7,8
318 446 318 446 318 457 c 132,-1,9
318 468 318 468 323 477 c 4,10,11
371 557 371 557 384 578 c 4,12,13
388 585 388 585 397 585 c 132,-1,14
406 585 406 585 410 578 c 4,15,16
417 567 417 567 440 528.5 c 132,-1,17
463 490 463 490 471 477 c 4,18,19
477 468 477 468 477 457.5 c 132,-1,20
477 447 477 447 471 437 c 4,21,22
458 415 458 415 451 403 c 4,0,1
264 366 m 4,23,24
282 377 282 377 304 377 c 6,25,-1
524 377 l 6,26,27
531 377 531 377 537 372 c 4,28,29
552 355 552 355 584 321 c 4,30,31
593 311 593 311 593 298 c 4,32,33
593 284 593 284 583 274 c 4,34,35
558 248 558 248 539 230 c 4,36,37
532 222 532 222 520 222 c 6,38,-1
301 222 l 6,39,40
282 222 282 222 266 231 c 4,41,42
261 234 261 234 190 275 c 4,43,44
177 282 177 282 177 298 c 4,45,46
177 313 177 313 190 321 c 4,47,48
197 325 197 325 224 341.5 c 132,-1,49
251 358 251 358 264 366 c 4,23,24
451 196 m 4,50,51
458 184 458 184 471 162 c 4,52,53
477 152 477 152 477 141.5 c 132,-1,54
477 131 477 131 471 122 c 4,55,56
427 48 427 48 410 21 c 4,57,58
406 14 406 14 397 14 c 132,-1,59
388 14 388 14 384 21 c 4,60,61
377 33 377 33 354 71 c 132,-1,62
331 109 331 109 323 122 c 4,63,64
318 131 318 131 318 142 c 132,-1,65
318 153 318 153 323 162 c 4,66,67
339 189 339 189 343 196 c 4,68,69
347 202 347 202 354 202 c 6,70,-1
440 202 l 6,71,72
447 202 447 202 451 196 c 4,50,51
695 319 m 0,73,74
685 308 685 308 666 307 c 0,75,76
665 307 665 307 659 306.5 c 128,-1,77
653 306 653 306 649 306 c 0,78,79
627 306 627 306 611 317 c 0,80,81
592 329 592 329 548 382 c 0,82,83
541 390 541 390 541 401 c 0,84,85
541 430 541 430 541 453 c 0,86,87
541 481 541 481 541 502.5 c 128,-1,88
541 524 541 524 541 551.5 c 128,-1,89
541 579 541 579 541 612 c 0,90,91
541 625 541 625 547 637 c 0,92,93
580 710 580 710 602 758 c 0,94,95
608 771 608 771 623 771 c 0,96,97
631 771 631 771 638 767 c 0,98,99
655 753 655 753 668 735 c 0,100,101
682 716 682 716 692 694 c 0,102,103
703 669 703 669 703 641 c 2,104,-1
703 346 l 2,105,106
703 339 703 339 702 332 c 0,107,108
700 324 700 324 695 319 c 0,73,74
114 325 m 0,109,110
110 327 110 327 107 330 c 0,111,112
97 339 97 339 97 352 c 2,113,-1
97 645 l 2,114,115
97 681 97 681 115 711 c 0,116,117
115 712 115 712 121 723 c 0,118,119
127 733 127 733 139 733 c 0,120,121
147 733 147 733 152 727 c 0,122,123
158 722 158 722 196 684 c 128,-1,124
234 646 234 646 247 633 c 0,125,126
259 621 259 621 259 605 c 2,127,-1
259 388 l 2,128,129
259 377 259 377 249 372 c 0,130,131
238 365 238 365 204.5 346 c 128,-1,132
171 327 171 327 162 322 c 0,133,134
152 317 152 317 141 317 c 0,135,136
132 317 132 317 124 320 c 0,137,138
117 323 117 323 114 325 c 0,109,110
153 743 m 0,139,140
151 751 151 751 153 754.5 c 128,-1,141
155 758 155 758 164 767 c 0,142,143
169 772 169 772 179 778 c 0,144,145
200 793 200 793 226 793 c 2,146,-1
555 793 l 1,147,-1
572 792 l 2,148,149
573 792 573 792 576 791.5 c 128,-1,150
579 791 579 791 581 791 c 0,151,152
589 790 589 790 594 785 c 0,153,154
600 779 600 779 598 771 c 0,155,156
593 761 593 761 542 647 c 0,157,158
536 633 536 633 521 633 c 2,159,-1
272 633 l 2,160,161
260 633 260 633 252 642 c 0,162,163
157 735 157 735 153 743 c 0,139,140
695 278 m 0,164,165
700 273 700 273 702 265 c 0,166,167
703 258 703 258 703 251 c 2,168,-1
703 -45 l 2,169,170
703 -72 703 -72 692 -97 c 0,171,172
682 -119 682 -119 668 -138 c 0,173,174
655 -156 655 -156 638 -170 c 0,175,176
631 -175 631 -175 623 -175 c 0,177,178
608 -175 608 -175 602 -161 c 0,179,180
600 -157 600 -157 577.5 -107.5 c 128,-1,181
555 -58 555 -58 547 -40 c 0,182,183
541 -28 541 -28 541 -15 c 0,184,185
541 18 541 18 541 45.5 c 128,-1,186
541 73 541 73 541 94.5 c 128,-1,187
541 116 541 116 541 144 c 0,188,189
541 167 541 167 541 196 c 0,190,191
541 206 541 206 548 214 c 0,192,193
592 268 592 268 611 280 c 0,194,195
627 291 627 291 649 291 c 0,196,197
653 291 653 291 659 290.5 c 128,-1,198
665 290 665 290 666 290 c 0,199,200
685 289 685 289 695 278 c 0,164,165
153 -143 m 0,201,202
157 -135 157 -135 252 -42 c 0,203,204
260 -34 260 -34 272 -34 c 2,205,-1
521 -34 l 2,206,207
536 -34 536 -34 542 -47 c 0,208,209
593 -161 593 -161 598 -172 c 0,210,211
600 -179 600 -179 594 -185 c 0,212,213
589 -190 589 -190 581 -191 c 0,214,215
580 -191 580 -191 576.5 -191.5 c 128,-1,216
573 -192 573 -192 572 -192 c 2,217,-1
555 -193 l 1,218,-1
226 -193 l 2,219,220
200 -193 200 -193 179 -178 c 0,221,222
169 -172 169 -172 164 -167 c 0,223,224
155 -158 155 -158 153 -154.5 c 128,-1,225
151 -151 151 -151 153 -143 c 0,201,202
114 275 m 0,226,227
117 277 117 277 124 280 c 0,228,229
132 283 132 283 141 283 c 0,230,231
152 283 152 283 162 278 c 0,232,233
171 273 171 273 204.5 254 c 128,-1,234
238 235 238 235 249 228 c 0,235,236
259 223 259 223 259 212 c 2,237,-1
259 -6 l 2,238,239
259 -22 259 -22 247 -33 c 0,240,241
234 -46 234 -46 196 -84 c 128,-1,242
158 -122 158 -122 152 -127 c 0,243,244
147 -133 147 -133 139 -133 c 0,245,246
127 -133 127 -133 121 -123 c 256,247,248
115 -113 115 -113 115 -112 c 0,249,250
97 -81 97 -81 97 -45 c 2,251,-1
97 247 l 2,252,253
97 261 97 261 107 269 c 0,254,255
110 272 110 272 114 275 c 0,226,227
EndSplineSet
Validated: 2049
EndChar
EndChars
EndSplineFont
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ export class Altitude extends DisplayComponent<AltitudeProps> {
private readonly altText = MappedSubject.create(
([isLightTest, altitude]) => {
if (isLightTest) {
return '88888';
return '8.8.8.8.8';
}
const value = Math.floor(Math.max(altitude, 100));
return value.toString().padStart(5, '0');
Expand Down
19 changes: 14 additions & 5 deletions fbw-a380x/src/systems/instruments/src/FCU/Components/Baro.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ export class Baro extends DisplayComponent<BaroProps> {
private readonly baroText = MappedSubject.create(
([mode, correction, isLightTest]) => {
if (isLightTest) {
return '88.88';
return '8.8.8.8';
}
switch (mode) {
case 'STD':
Expand All @@ -66,10 +66,18 @@ export class Baro extends DisplayComponent<BaroProps> {
);

private readonly preSelBaroText = MappedSubject.create(
([correction, isVisible]) =>
isVisible ? (correction < 100 ? correction.toFixed(2) : correction.toFixed(0).padStart(4, '0')) : '',
([correction, isVisible, isLightTest]) => {
if (isLightTest) {
return '8.8.8.8';
} else if (isVisible) {
return correction < 100 ? correction.toFixed(2) : correction.toFixed(0).padStart(4, '0');
} else {
return '';
}
},
this.correction,
this.isPreSelVisible,
this.isLightTestActive,
);

onAfterRender(_node: VNode): void {
Expand Down Expand Up @@ -97,7 +105,8 @@ export class Baro extends DisplayComponent<BaroProps> {
>
QNH
</text>
<text
{/* Removed QFE label as QFE systems are INOP */}
{/* <text
id="QFE"
x="40%"
y="26%"
Expand All @@ -108,7 +117,7 @@ export class Baro extends DisplayComponent<BaroProps> {
}}
>
QFE
</text>
</text> */}
<text id="PreSelBaroValue" class="Common Active" x="100%" y="26%" text-anchor="end">
{this.preSelBaroText}
</text>
Expand Down
11 changes: 7 additions & 4 deletions fbw-a380x/src/systems/instruments/src/FCU/Components/Heading.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -10,16 +10,19 @@ export class Heading extends DisplayComponent<HeadingProps> {
return (
<div id="Heading">
<svg width="100%" height="100%">
<text id="HDG" class="Common Active" x="13%" y="20%">
<text id="TRUE" class="Common Active " x="23%" y="20%">
TRUE
</text>
<text id="HDG" class="Common Active" x="48%" y="20%">
HDG
</text>
<text id="TRK" class="Common Inactive" x="47%" y="20%">
<text id="TRK" class="Common Inactive" x="68%" y="20%">
TRK
</text>
<text id="Value" class="Common Value" x="11%" y="86%">
<text id="Value" class="Common Value" x="47%" y="86%">
---
</text>
<text id="DEGREES" class="Common Value" x="86%" y="86%">
<text id="DEGREES" class="Common Value" x="89%" y="86%">
&#176;
</text>
</svg>
Expand Down
Loading

0 comments on commit be06c3f

Please sign in to comment.