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

Amazon Stock shadow seems to mix in windmill #54

Open
HiranChaudhuri opened this issue Jun 15, 2022 · 8 comments
Open

Amazon Stock shadow seems to mix in windmill #54

HiranChaudhuri opened this issue Jun 15, 2022 · 8 comments

Comments

@HiranChaudhuri
Copy link

HiranChaudhuri commented Jun 15, 2022

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

@paulwedeck
Copy link
Owner

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

@HiranChaudhuri
Copy link
Author

HiranChaudhuri commented Jun 16, 2022

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
Copy link
Author

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?

@paulwedeck
Copy link
Owner

search for MILL in MapObjectDrawer

@HiranChaudhuri
Copy link
Author

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?

@paulwedeck
Copy link
Owner

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.

@HiranChaudhuri
Copy link
Author

HiranChaudhuri commented Jun 18, 2022

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
Copy link
Author

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.

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

No branches or pull requests

2 participants