yuriknsk / tu154b

Tupolev Tu-154b airliner model for FlightGear
13 stars 15 forks source link

Added "hijack" warning light and "2500kg fuel" alarm sound mute switch #27

Closed ghost closed 9 years ago

ghost commented 9 years ago

ССО опасн - warning light shows up when transponder is set to 7500. As mentioned below I got this from the "tu154eng.pdf" but I'm not 100% sure if this is how this warning works, because it's very hard to find something about the tu154 in english.

kroki commented 9 years ago

I can't check it in action, I assume you use red light ССО опасн ("threat notification system - danger"). But what are your sources? I.e. what makes you think that this light turns on when transponder set to 7500? Normally we follow Flight Manual very closely and do not invent things on our own. For lights there's a doc www.dream-air.ru/new/pilotam/.pdf (note that file has no name, only .pdf extension - I had trouble finding it in file system after download). It's in Russian, but there's little text and you may feed it to Google translate. Re ССО опасн it made me think that this alarm is initiated by fight attendants to warn cockpit crew about hijack threat. The description of yellow light ССО связь ("threat notification system - communication") is not of much help there.

Having said that, we may use one of this lights for 7500 because it's unlikely we'll use them for anything else.

kroki commented 9 years ago

Perhaps it's also worth noting that "Added hijack warning" is too vague (yup, I said message is good in e-mail, but I changed my mind :P). The exact form of such "warning" is not immediate from the patch, so it's better to use a description field above (and/or a commit message itself) to describe what exactly should happen so that everyone else won't have to look for unknown clues, and will be able to confirm that they indeed observe what they were supposed too.

ghost commented 9 years ago

My source for that change was the "tu154eng.pdf" which is delivered with the aircraft in the "Docs" folder. There it says "147 - Lights up when the transponder is set to "hijack" (not used the sim) " and 147 is the ССО опасн warning light. So the international Transponder Code for Hijack is 7500.

ghost commented 9 years ago

You wrote me in the mail, that I can add a description to the commit by leaving a blank line after the commits name and then adding the description. I uploaded this commit with the linux command line and I'm not sure how to add a blank line to this. Should i Use "\n" or is there any other way?

kroki commented 9 years ago

"147 - Lights up when the transponder is set to "hijack"

I'm pretty sure that's not how it works in real thing, but having it this way in the model won't harm and will at least make the light alive.

Should i Use "\n" or is there any other way?

For some reason I suspect you use Linux in a virtual machine from Windows (you didn't have to BTW). Still I assume that you installed Linux with GUI support. If so I suggest to install git gui utility (and also gitk). Linux distributions vary so I can't tell you the exact package name nor the exact command to use to install the package (googling the problem will give you the answer). Once installed, git gui command will open up a window that will allow you to select changes to commit, enter commit message in milti-line text edit field, and commit the changes with a button. gitk --all command will show you the history of all branches.

But if you installed Linux without GUI support, than running git commit (i.e. without -m/--message option) will start a text editor, which is likely to be vim. There you type i to enter "input" mode, enter the text (arrows to navigate, backspace/delete keys for editing), then press ESC key to leave "input" mode, and type :wq (which stands for "command-write-quit").

ghost commented 9 years ago

No, it's the other way round, I use Windows in a virtual machine. But never the less thanks for the git gui and gitk, I will install these.

DestroyFX commented 9 years ago

I have tested the patchs.

Everything work according the the patch description. Also, I have scanned the documentation for setting the transponder to 7500 but I might just have missed it (I'm not good a Russian and I got 130M of scanned doc that can't be OCRed nor google translated).

antage commented 9 years ago

I read this document - http://aviadocs.net/RLE/Tu-154B/CD1/TU/Tu-154B_TU_v14.pdf The document explains a procedure for checking "ССО опасно" alarm (a lamp and buzzer) . "ССО опасно" alarm can be activated by any of 3 fake sockets. Button "СБРОС СИГНАЛА ССО" turns off the alarm.

I didn't find any references to transponder in the document.

ghost commented 9 years ago

Alright so I guess the Documentation https://github.com/yuriknsk/tu154b/blob/master/Docs/tu154eng.pdf is wrong. So can anyone explain how I can undo this commit?

kroki commented 9 years ago

You don't have to remove any of the patches. Just close pull request and move this patch from your master branch to some other:

git checkout master
git branch hijack-7500-SSO-light
git reset --hard HEAD~2
git push --force --all

and this patch will be moved from master to hijack-7500-SSO-light branch. Maybe someone will pickup your work later.

Update: changed HEAD^ to HEAD~2 as you have two patches to move.

ghost commented 9 years ago

Ok, I put them both in a new branch.

I will now figure out if I can find any reliable documentation of the Tu-154B-2 in english and see if I can work on anything of that.

kroki commented 9 years ago

It's a voluntary work, please don't feel obliged to do anything beyond your comfortable reach. The work that doesn't require reading Russian and that most of us would like someone else to do :) is to finish 3-D model of engineer panel. It's possible to do most of it by placing existing elements onto their depicted positions. But since you added one switch and refused to add another I guess you learned how tedious the process is :)

Also, if you are good with 3-D modeling than we need to finish the transponder: we have this part but only a picture for this (the latter is BTW where power switch is located).

ghost commented 9 years ago

Yeah it was just a little frustrating that every single patch or fix I made was wrong for now. But I will not give up :) So I will try on the second part of the transponder you posted me. I guess its is located downright of the part already modeled. Is the location correct there? I will now start modeling the instrument and will look for the function of each button later on.

kroki commented 9 years ago

BTW, on Russian forum we have an artist who may paint the background and add labels once (some part of) engineer panel is ready (and he also speaks German BTW).

More tasks to consider (for you, for DestroyFX, for anyone else):

I guess its is located downright of the part already modeled. Is the location correct there?

No, it is more close to airplane's nose. It looks on the model's texture exactly as it looks on the picture I provided.

Update: note that if you view current transponder panel from the right seat you'll see that it actually hangs on the air, i.e. it is not in the plane of the model geometry. This is because the geometry's angle differs significantly from the texture's image apparent angle, and to align model borders with borders on texture you have to trick it like this.

kroki commented 9 years ago

For the reference: transponder-main I think "protocol" selector selects the format of a reply (what subset of altitude, speed, heading, fuel, etc. to transmit and in what form). In Russia УВД ("ATC") is used, in other contries normally set to R8S. I don't know what ВОЛНА does (literally "wave", may mean "band" or smth similar).

Update: according to docs this transponder can't report altitude in R8S mode unless pressure is set to 760mmhg. So you may implement test light condition as (test_button && power && (protocol != "R8S" || pressure == 760)).

Update2: РСП - set on landing, УВД - in Russia, R8S - in other countries (023М is not mentioned). ВОЛНА selects radio channel assigned before the flight (source, p.91).

Update3: Russian docs spell R8S, but label on the picture looks like RBS. It's possible that 8 in the docs is a result of a missed optical character recognition, and the actual char is B.

ghost commented 9 years ago

skawk

I know this is far from being finished, positioning this model is really pain in the ass. But you can see where I'm going to place it and the first 2 knobs (Protocol and Band, or whatever). So the placing there is correct? I just put it above the image of the instrument.

kroki commented 9 years ago

Yup, that's the right place.

ghost commented 9 years ago

transponder

This is how far I am with shaping the base of the new Transponder Panel. I'm not really happy with that, because it's getting pretty thin on the end. That's related to the shape of the console and otherwise the Transponder panel would go over the edge. What do you think?

kroki commented 9 years ago

The model of the base should not be distorted, i.e. it should be a rectangle (with cut ears). You don't have to place it exactly above the original trace on texture, our artist will paint the background as needed (if you look at the bottom of ef605873e page you'll notice that we also moved the other part: it used to be in place of a bottom metal plate, but that location is obscured by chair armrest). So just move it closer to the viewer and a bit closer to the window and it will fit (or rotate it more aggressively to align with the edge - whatever you choose).

I just found a description of a different transponder model. From it follows that for our purposes 023М is the same as УВД (basically, all protocols are the same for us). It also says that the test lamp blinks whenever transponder send the reply (we may do this by timer, say every 1 minute).

ghost commented 9 years ago

Ah well, if you have someone who could redraw the side console, I would place the panel like this: fgfs-screen-001

There it fits with the edge and has the same distance to the steel plate on the left like the other transponder panel and it is still parallel to the other transponder panel. What I would need now is someone, who could draw a "text.png" with the russian text on it in the size 231x76. What is the name of the artist in the russian forum so I could ask him for it.

kroki commented 9 years ago

What is the name of the artist in the russian forum so I could ask him for it.

git show cd19b000 will reveal his name and e-mail. As a temporal mean you may resize original transponder picture to use for texture and ask him to draw the same sizes.

P.S.: do write in German, he doesn't speak English IIRC.

DestroyFX commented 9 years ago

I can redraw the side console to remove the old transponder panel under your new panel and have it in the Orange branch of my git by this night. I did also clean the section for the 2500kg fuel mute but that one is incorporated with the panel overhaul i'm doing (currently busy making a three position switch based azs1)

DestroyFX commented 9 years ago

Ok. I made a change to the vc04 file : https://github.com/DestroyFX/tu154b/commit/591d43dad2a33c3eea70be173b8d43830e1240eb

I could not put it in the OrangeTiger branch because I have a ton of stuff not ready to commit so I put it in my devel branch. You should add my git in remote to your local tree in that way :

$ git remote add destroyfx https://github.com/DestroyFX/tu154b.git (add a remote named destroyfx, you can put the name you want) $ git fetch destroyfx devel (to download my devel branch, will not overwrite or apair in your stuff)

$ git cherry-pick 591d43dad2a33c3eea70be173b8d43830e1240eb

That way you get my texture and when you are ready to merge you can include my commit and it will be a separated patch by me in your pull request. Also I recommend you to cherry-pick my three whitespaces patchs (just don't include them in pull request) if you plan to play in the code more so a pile of conflict don't pile up on the Yurik side when he merge.

Edit : I did not have a proper green matter to sample because of the angles/light so it's not that perfect and can be use temporary.

kroki commented 9 years ago

At a risk of being offensive I'd say that our artist has a better sense or color tone ;). Borders are clearly visible on yours.

kroki commented 9 years ago

Edit : I did not have a proper green matter to sample because of the angles/light so it's not that perfect and can be use temporary.

We can live temporarily with the old trace showing through. Each image commit will add to repo size, so it's better not to add temporal images. The point is not that you picked the wrong color at a wrong place and used a wrong brush. The point is that true artist will do much more than picking and brushing. See how he drew the metal plate at the bottom of ef60587 - you can actually feel the rough surface and ambient light. That's why I wasn't particularly exited when you started drawing the panel instead of filling in a model: your dividing line is obviously thinner and clearly shows aliasing pattern close to where two lines converge into one.

I'm currently listening to the course titled "No excuses". From the description:

...every one of us, as an individual, is responsible—responsible for what we do, responsible for who we are, responsible for the way we face and deal with the world, responsible, ultimately, for the way the world is.

So lets do what we can take responsibility for without any excuses, and let the artist do his part. I'm particularly good at making other people hate me, and I expect everyone to follow me in doing their best :).

DestroyFX commented 9 years ago

@kroki I know and it's why I edited my initial post to mention "temporary". I did made another attempt here: https://github.com/DestroyFX/tu154b/commit/f2b292c6f0fe28bb345741d20e3e3b7fae43edb6 but I can't really do better than that patching it. to many different shade of green to manage :p

DestroyFX commented 9 years ago

For the other panel I began to do for the air conditioning section. I stopped to try to improve it and just use my "hole" mask to fit the switch in the proper positions.

ghost commented 9 years ago

I wrote the artist from the russian forum, but unfortunatly hes going on holidays tomorrow. So he will make the background image and the caption for the panel in mid September. I'm going to place the handles on the panel for now and position them excactly, when I have the images from him. The most handles are just copy and paste I guess, but I have to look how to do the Emergency switch with the cover. I think there is no switch like that before.

DestroyFX commented 9 years ago

That's not that far and it leave time for me to place the models on the other panel. The textures I made allowed me to see how it will not fit in the original position so I did not loose my time making it :p