settlers-remake icon indicating copy to clipboard operation
settlers-remake copied to clipboard

Amazon Stock shadow seems to mix in windmill

Open HiranChaudhuri opened this issue 2 years ago • 8 comments

The Amazon stock shadow is not related to the building elements. It looks more like the windmill animations were reused as shadows.

Screenshot from 2022-06-15 16-47-47

Here is some data about my system:

OS: Linux 5.13.0-48-generic
JVM: OpenJDK 64-Bit Server VM 16.0.1+9-Ubuntu-120.04
Settlers Remake: Workflow Run 195

HiranChaudhuri avatar Jun 15 '22 14:06 HiranChaudhuri

Yes, it has the wrong shadow. There is a mechanism to correct the shadows. Just look at the ShadowMapping interface

paulwedeck avatar Jun 15 '22 15:06 paulwedeck

Apparently, the DAT File viewer is no help for finding shadows. So I had to have a look at the exported graphics.

It seems the file found in graphics/43/settlers/014/000_shadow.png is the best match for the Amazon stock. But I would not know how to perform a change similar to what you have done for the other mappings. So let's check on.

The shadows for the mill are also wrong. Currently they remind me of the tower?

Screenshot from 2022-06-16 08-32-09

And the tower - umm, yes.

Screenshot from 2022-06-16 08-33-39

The building editor is proving it's value. :-)

HiranChaudhuri avatar Jun 16 '22 06:06 HiranChaudhuri

With that patch it is better than before. Now it looks strange that

  • the mill's shadow does not follow the animation
  • the miss seems to spin backwards

How can that be fixed?

HiranChaudhuri avatar Jun 17 '22 12:06 HiranChaudhuri

search for MILL in MapObjectDrawer

paulwedeck avatar Jun 17 '22 19:06 paulwedeck

It seems like this line limits to displaying a single image as shadow.

Now regardless of what I tried to render the sequence images' shadows I still do not get a good result. Hence I believe I need to better control the sequence and shadow loading mechanism - can you provide some help there?

HiranChaudhuri avatar Jun 18 '22 17:06 HiranChaudhuri

Has every frame of the mill animation a proper shadow? If yes, then you should just draw the images with draw and remove the shadow part.

paulwedeck avatar Jun 18 '22 17:06 paulwedeck

No, just a few frames seem to have shadows attached. The animation is too fast for me to really see them. I took the effort to check the latest release 6.0.1 alpha. In there the mills do not spin their shadows anyway - across the civilizations. So we can consider that to be a different issue, and the solution I have so far is an improvement already.

HiranChaudhuri avatar Jun 18 '22 22:06 HiranChaudhuri

I have extended the BuildingCreator with functionality so it can (rudimentally) show an animation. Now it becomes evident that the mill shadows in the game do not rotate at all but they could/should.

For the roman mill it looks ok (although I am puzzled watching the show). The asian one seems to rotate in contradiction to it's shadow. The egyption is ok. The amazon one is lacking shadows.

HiranChaudhuri avatar Jun 19 '22 17:06 HiranChaudhuri