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

Polish rendering on Adventure Map #5716

Open
6 of 10 tasks
ihhub opened this issue Aug 8, 2022 · 33 comments
Open
6 of 10 tasks

Polish rendering on Adventure Map #5716

ihhub opened this issue Aug 8, 2022 · 33 comments
Assignees
Labels
bug Something doesn't work improvement New feature, request or improvement ui UI/GUI related stuff
Milestone

Comments

@ihhub
Copy link
Owner

ihhub commented Aug 8, 2022

Preliminary checks

Describe the problem requiring a solution

After merging #5692 we still have multiple tasks to be done in order to polish rendering on Adventure Map:

  • standartise object fading animation including heroes
  • remove old save formats (before 0.9.16? release)
  • do not store image parts but rather image regions and flip flag and use them while rendering
  • rendering of monsters over Dragon City (added a TODO comment for it)
  • rendering heroes / boats outside map while they are in fog
  • overlapping of boats during movement (froth should be rendered under another boat for some cases; perspective of boats)
  • path icon should be removed after a hero makes a turn, not before
  • some objects like flying ghosts over mines are not rendered if the main tile is below the rendering area

Good to have:

  • shadows of heroes must be drawn over other objects (tricky one. The original version does this but it breaks a lot of other places. For now we stick with common logic for all objects)

All issues related to rendering on Adventure Map should be reported here to make them easier to track.

Describe the possible solution

Fix everything.

Additional info

No response

@ihhub ihhub added bug Something doesn't work improvement New feature, request or improvement ui UI/GUI related stuff labels Aug 8, 2022
@ihhub ihhub added this to the 0.9.18 milestone Aug 8, 2022
@ihhub ihhub self-assigned this Aug 8, 2022
ihhub added a commit that referenced this issue Aug 8, 2022
ihhub added a commit that referenced this issue Aug 8, 2022
ihhub added a commit that referenced this issue Aug 8, 2022
A boat or hero or a monster standing above or on the left side of the screen were not rendered properly. More specifically a part of them.

relates to #5716
ihhub added a commit that referenced this issue Aug 9, 2022
relates to #5716

Also removed useless checks to speed things up.
@ihhub ihhub modified the milestones: 0.9.18, 0.9.19 Aug 10, 2022
ihhub added a commit that referenced this issue Aug 11, 2022
relates to #5716

Also small code cleanup
@Branikolog
Copy link
Collaborator

An issue with wide tree:
image
image
Tree.zip

@ihhub
Copy link
Owner Author

ihhub commented Aug 20, 2022

Hi @Branikolog , I think it works exactly the same way as in the original game due to object info.

@oleg-derevenetz
Copy link
Collaborator

It seems that Mine guards are no longer drawn (map "Skirmish"):

Screenshot1

Screenshot2

@ihhub
Copy link
Owner Author

ihhub commented Aug 22, 2022

Oh, no. I will fix it.

@Branikolog
Copy link
Collaborator

Hi @Branikolog , I think it works exactly the same way as in the original game due to object info.

Hi, @ihhub !
I suppose you're right. But nevertheless this bug should be fixed. ;)

@Branikolog
Copy link
Collaborator

Branikolog commented Aug 26, 2022

{SOLVED}
AI hero priority rendering bug:
image
Decisions_Hero bug.zip

Edit:
Fixed by #5787

ihhub added a commit that referenced this issue Aug 27, 2022
ihhub added a commit that referenced this issue Aug 27, 2022
@Branikolog
Copy link
Collaborator

Hi, @ihhub
I've noticed that AI hero sprite is partially visible through mountains while moving.
image
Horse tail.zip

@Branikolog
Copy link
Collaborator

Branikolog commented Sep 7, 2022

{SOLVED}
One more issue:
Alchemist lab flag appears only when the roof is visible on the screen:
image
image

@ihhub
Copy link
Owner Author

ihhub commented Sep 8, 2022

Hi, @ihhub I've noticed that AI hero sprite is partially visible through mountains while moving.

Are you testing the latest version with my fix for AI heroes? If yes is it the same for player's heroes?

@LeHerosInconnu
Copy link

The red AI player's hero is displayed over the top of the Necromancer's castle.

In fheroes2:

AI hero over top castle 01

Here is the save file (1.0.5 7020):
AI hero over top castle 01.zip

@LeHerosInconnu
Copy link

Graphic problem with boat near sirens.

In fheroes2:

2023-06-16.11-16-46.mp4

Here is the save file (1.0.5 7020):
Hero path near sirens 01.zip

@LeHerosInconnu
Copy link

The blue player's hero is displayed over the top of the water wheel.

In fheroes2:

Hero over water wheel 01

Here is the save file (1.0.5 7020):
Hero over water wheel 01.zip

@LeHerosInconnu
Copy link

The blue player's hero is displayed over the top of the castle.

In fheroes2:

Hero over top castle 02

Here is the save file (1.0.5 7308):
Hero over top castle 02.zip

@LeHerosInconnu
Copy link

The shadow of the tree is displayed over the hero Crack Hack.

In fheroes2:

Tree shadow 01

Here is the save file (1.0.5 7308):
Tree shadow 01.zip

@Districh-ru
Copy link
Collaborator

Districh-ru commented Jun 21, 2023

The right part of the Abandoned mine is not updated after the capture (bug found by apocalypse on Discord):

изображение

Save file: ghostmine_capture_sprit_bug.zip

Edit:
Fixed by #7351

@LeHerosInconnu
Copy link

A more recent save file for this.
The horse's head is hidden by a tree.

In fheroes2:

Hero sprite cut 1000

Here is the save file (1.0.8 7767):
Hero sprite cut 1000.zip

@LeHerosInconnu
Copy link

Another save file for this.

In fheroes2:

vlcsnap-2023-10-08-14h08m13s499

2023-10-08.14-06-26.mp4

Here is the save file (1.0.8 7846):
Before hero crackland 01.zip

@ihhub ihhub modified the milestones: 1.0.9, 1.0.10 Oct 11, 2023
@ihhub ihhub modified the milestones: 1.0.10, 1.0.11 Nov 18, 2023
@ihhub ihhub modified the milestones: 1.0.11, 1.1.0 Dec 23, 2023
@LeHerosInconnu
Copy link

There's a priority problem between the ghost animation display, the hero flag sprite, and the hero sprite.

In fheroes2:

2024-02-15.13-30-01.mp4

vlcsnap-2024-02-15-13h38m40s792

Here is the save file (1.0.12 8369):
Priorite ghost display 01.zip

@LeHerosInconnu
Copy link

Display priority problem between boat and sirens.

In fheroes2:

vlcsnap-2024-02-21-14h33m49s388

2024-02-21.14-30-56.mp4

Here is the save file (1.0.12 8369):
Ai sirens graphic glitch 01.zip

@ihhub ihhub modified the milestones: 1.1.0, 1.1.1 May 22, 2024
@LeHerosInconnu
Copy link

Display priority problem between hero and stables.

In fheroes2:

vlcsnap-2024-06-01-21h35m48s290

2024-06-01.21-35-18.mp4

Here is the save file (1.1.0 8733):
Display priority stables 01.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something doesn't work improvement New feature, request or improvement ui UI/GUI related stuff
Projects
None yet
Development

No branches or pull requests

5 participants