c172p-team / c172p

A high detailed version of the Cessna 172P aircraft for FlightGear
GNU General Public License v2.0
80 stars 43 forks source link

Jagged lines in reflection in glass #1022

Open legoboyvdlp opened 6 years ago

legoboyvdlp commented 6 years ago

image

image

On branch bug-1015

gilbertohasnofb commented 6 years ago

What's wrong with that? Just some reflection distorted by the glass, isn't it?

legoboyvdlp commented 6 years ago

I'm a little concerned about the jagged lines. What exactly is it reflecting there? But perhaps the new reflection cubemap might reduce the severity of those lines?

gilbertohasnofb commented 6 years ago

What exactly is it reflecting there?

Some random image that is not a true reflection of the world outside.

But perhaps the new reflection cubemap might reduce the severity of those lines?

I don't think that's worth it to be honest. @wlbragg?

wlbragg commented 6 years ago

I don't remember exactly where I pointed it out, but I am not totally happy with the reflection cubes we are using, Problem is I don't know that we can "easily" improve upon them. They are Blender generated and are reflecting what you see in Blenders Model view, not real life (or in this case in sim). Because of the way models were historically put together they also don't contain any of the instruments. So the reflections aren't really even close but a distant facsimile. My suggestion is someone try to create new maps from in the sim VS in Blender. The issue there is the scale is not square. But if you could start the sim in the most square resolution you can, and basically crop the screen shot square, maybe we could achieve satisfactory results. I think they would look better with missing data (because of the dimensions and having to square crop) than full data cubes but using a facsimile of the rendered visual. I could be wrong, but I think it would look more realistic. As far as this glass distortion were seeing now, I don't know that that will change much because it is being viewed at such an extreme angle, I think.

gilbertohasnofb commented 6 years ago

They are Blender generated and are reflecting what you see in Blenders Model view

Are you sure about that? I took a look at them and they seemed to be using FG textures and everything. There are a bunch of cube maps in that FGDATA folder, but all of them look equally bad. :cry:

wlbragg commented 6 years ago

No. you right @gilbertohasnofb I was looking at the backup colored maps, specifically the red one... red_nx However the ones were using do appear to be rendered from in the sim. reflection_nx

wlbragg commented 6 years ago

Although they do appear to be too dark in my opinion. I think we may be losing some detail. Also why the gray windows VS white? This was the original grayscale version that wasn't proportionally correct. grayscale_nx

wlbragg commented 6 years ago

The first image are the old wrong proportioned gray scale versions.. fgfs-screen-006 fgfs-screen-007 These are what we are using now... fgfs-screen-004 fgfs-screen-005

gilbertohasnofb commented 6 years ago

@wlbragg I think the reason we are using grey scale cockpit maps is because we have many different liveries for the cockpit but a single map, and back in the day we decided that a BW map would fit better with all of them. I still think that's an all right decision as the reflection is rarely that noticeable (in the sense of being able to really see cockpit parts and other things in the reflected image)

wlbragg commented 6 years ago

@gilbertohasnofb

I think the reason we are using grey scale cockpit maps is

I'm not questioning that, I agree.

Now my only "potential" issue is the gray scale maps we are using now. They have a gray (or darker) window area that I think should really be white? The area that is window is actually gray in color and I don't think that is right.

Also, as a whole they appear to be a little darker than the original "gray scale" ones we used to use. Those we replaced because they were distorted proportionally.

gilbertohasnofb commented 6 years ago

I see, I will take a look at that.

gilbertohasnofb commented 6 years ago

@wlbragg done, reflection maps should have a white background, could you please confirm if that improves the issue?

legoboyvdlp commented 6 years ago

@wlbragg

image

Is that line meant to be half green and half white on the back window?

gilbertohasnofb commented 6 years ago

Is that line meant to be half green and half white on the back window?

No, but that's a livery mistake. I will fix that.

gilbertohasnofb commented 6 years ago

@legoboyvdlp I fixed this in the opened damage model PR branch refactor-damage-model

legoboyvdlp commented 6 years ago

Further problem: image

gilbertohasnofb commented 6 years ago

Looks to me that the ice formation is mapped just frontally and so it looks terrible on the sides.

@legoboyvdlp turn off that HUD and fly this beauty using its own instruments!! :rofl:

onox commented 6 years ago

We should totally ban the HUD :stuck_out_tongue_winking_eye:

wlbragg commented 6 years ago

Can we verify the ice is not working on any sides? Also be aware of the fact that the front windshield is one piece and so if the front windshield has ice then the side is also included in the effect. This may have to do with the effects switch used to distinguished between the two different angles of glass. The more horizontal VS vertical glass.

We need to revisit this whole fog, rain, reflection effect and make sure we didn't inadvertently screw it up with the changing of the windshield.

I would like to see some good imaged of ice on all the windows.

gilbertohasnofb commented 6 years ago

We should totally ban the HUD stuck_out_tongue_winking_eye

Someone please make a PR for the main FG project removing that thing! :rofl:

gilbertohasnofb commented 6 years ago

@wlbragg Sorry for my ignorance, but how can I force the ice to appear on the glass?

But it does look the modifications to the glass made the effects a bit strange, the rain also looks distorted on the sides:

fg

wlbragg commented 6 years ago

But it does look the modifications to the glass made the effects a bit strange, the rain also looks distorted on the sides:

Yes, that is a fact and there is no good solution to correct that. The reason is we have two modes to do the rain in the effect

<surface-mapping-scheme type="int">1</surface-mapping-scheme>

and

<surface-mapping-scheme type="int">2</surface-mapping-scheme>

Mode 1 is for more horizontal glass faces and mode 2 is for the vertical glass faces. Because our front window is one object now we can only assign mode 1. We used to assign mode 2 to the side objects. We still assign mode 2 to the door and aback seat side windows.

By the way, I just pushed the Blend and .ac turning on some objects I had off while doing the livery work. I forgot to turn them back on.

As far as the ice is concerned, I'll have to experiment before I can suggest a method to easily see it.

wlbragg commented 6 years ago

@gilbertohasnofb I just started from KEQA, I had the aircraft started up and sitting on the runway at startup. It was sitting there for a little while and the I ctrl+U up to maybe 10-12 thousand feet. I opened the heater slider all the way for a minute or two, then opened the air slider all the way and it started to show the ice pretty quickly. I then shut both sliders and paused the simulation. fgfs-screen-008

As you can see it is all working but indeed the windshield side glass is not looking good because of it being only one object and having to be assigned mode 1. The only solution is to separate the sides to their own objects like it was before so we can assign mode 2 to them (and put up with the hard edge) .

gilbertohasnofb commented 6 years ago

@wlbragg I see, thanks for the detailed post above.

The only solution is to separate the sides to their own objects so we can assign mode 2 to them.

What would be the side effect of doing that? Would that reinstate the issue of the glass looking a bit weird in the cockpit, which is what we had before?

gilbertohasnofb commented 6 years ago

Meaning, would it look again like this? https://github.com/c172p-team/c172p-detailed/issues/1010

wlbragg commented 6 years ago

Yes!

gilbertohasnofb commented 6 years ago

Hmm, so we have to choose between pretty glass vs pretty effects. What would be your preference? I am really unsure myself. The effects do not look good, but those are visible on 1% of flights while the front glass is always there. On the other, the divided glass does not look that bad and one can argue that that's totally acceptable in order to have the effects working well. What would you pick?

wlbragg commented 6 years ago

Here it is as before fgfs-screen-009 fgfs-screen-011

gilbertohasnofb commented 6 years ago

The effect certainly looks better, but the glass does look a bit worse. We need a choice: what would you pick?

wlbragg commented 6 years ago

For completeness, here it is now. fgfs-screen-014 fgfs-screen-013

I guess with the hard edge? I would like @onox and @legoboyvdlp to weight in!

gilbertohasnofb commented 6 years ago

Thanks for the images. Yes, the effects do look terrible right now and the hard edge is not that much worse than we have, so I am fine reverting the glass into separate pieces. Let's see what others say.

dany93 commented 6 years ago

We should totally ban the HUD

I know that people use it, at least on other aircraft (to see their controls or the heading). I sometimes happen to be glad to find it, would it be only to find the direction of my starting airport. Cheating, yes... But during tests, cheating often is useful (like Ctrl-U, etc...). Anyway, it only appears if you ask for it. Otherwise I don't see how it can be annoying.

With the UFO, it is essential.

legoboyvdlp commented 6 years ago

I'm happy for the glass to be reverted if it solves the effects.

gilbertohasnofb commented 6 years ago

Anyway, it only appears if you ask for it. Otherwise I don't see how it can be annoying.

I know, @dany93, I am just poking fun. There are many screenshots with the HUD out there and they break my heart, but that's just me being annoying. I promise I won't sent the PR removing it from the source code :stuck_out_tongue_winking_eye:

dany93 commented 6 years ago

There are many screenshots with the HUD out there and they break my heart

In this case, if you are as much an expert in image processing as in sound processing (and patient) that might be the solution :wink:

Sometimes, they also are here to show the GPS coordinates.

I promise I won't sent the PR removing it from the source code

(Relieved) Thank you for your tolerance! :smile:

I thought of sending an image, just for you, with the HUD and the sky alone, without one of these ugly dashboards, but I don't want to pollute the Internet. Only an idea...

wkitty42 commented 6 years ago

On 03/28/2018 10:08 AM, dany93 wrote:

I thought of sending an image, just for you, with the HUD and the sky alone, without one of these ugly dashboards, but I don't want to pollute the Internet. Only an idea...

you mean like flying around and hitting the 'c' key to toggle off the whole craft and cockpit dash stuff so it looks like you're just floating along in the air but with somewhat louder engine noise? i've done that... i've even flown entire flights without the craft or cockpit showing at all :) :) :)

but yeah, please leave the HUD alone... i definitely use it for coordinates amongst other things... heck, i've even been thinking of how to have a HUD projected on the windscreen of some craft because moving the view is too much to deal with and especially on a ""small"" monitor that is not HD or even widescreen...

gilbertohasnofb commented 6 years ago

In this case, if you are as much an expert in image processing as in sound processing (and patient) that might be the solution wink

@dany93 Challenge accepted. His is @legoboyvdlp's screenshot before:

And here is after:

image

Much better isn't it? :rofl:

gilbertohasnofb commented 6 years ago

It took me nearly 30 minutes to retouch @legoboyvdlp's image above for the joke, so someone please post a "hahaha" so that I don't feel that bad, please. :wink:

wlbragg commented 6 years ago

I was expecting it the other way around, for you to only leave the hud and get rid of the ugly panels!

gilbertohasnofb commented 6 years ago

@wlbragg There we go;

image

And this one only took 30 seconds!

legoboyvdlp commented 6 years ago

:rofl:

wkitty42 commented 6 years ago

On 03/28/2018 11:12 AM, Gilberto Agostinho wrote:

Much better isn't it? 🤣

hahaha

:)

-- NOTE: No off-list assistance is given without prior approval. Please keep mailing list traffic on the list unless a signed and pre-paid contract is in effect with us.

gilbertohasnofb commented 6 years ago

Thanks guys :smile:

legoboyvdlp commented 6 years ago

Should this be closed? Or is it open pending regeneration of cubemaps?