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

Castle screen, fully built castle, differences noticed #2059

Open
LeHerosInconnu opened this issue Oct 21, 2020 · 16 comments
Open

Castle screen, fully built castle, differences noticed #2059

LeHerosInconnu opened this issue Oct 21, 2020 · 16 comments
Assignees
Labels
assets Game data related stuff improvement New feature, request or improvement low priority Low priority item which could be addressed in the future

Comments

@LeHerosInconnu
Copy link

LeHerosInconnu commented Oct 21, 2020

In fheroes2, there are defects due to the display priority of sprites or/and sprites which are displayed differently.
In fheroes2, some defects have already been corrected compared to the original version of the game, they have not been reported here.

Sorceress' Castle.

  • Pixels at the bottom and to the left of the Captain's Quarters do not have the same color as in the original game.

In original game:

Castle screen H2 003

In fheroes2:

Castle screen fh2 003

Open the two images in different tabs in your browser, enlarge the images and switch between the two tabs alternately.
Heroes 2 first, fheroes2 second.

Castle screen H2 captain coupé 003 Castle screen fh2 captain coupé 003

Edit 2020/12/21.

Knight's Castle.

To comply with the creature buildings color code, red for unimproved buildings and blue for upgraded buildings.
Knight castle, inverted the sprites for the Archery Range and the Upg. Archery Range (red and blue).

TWNKDW_1.ICN
0000.bmp

  • replaced by 0000.bmp from TWNKUP_1.ICN

TWNKUP_1.ICN
0000.bmp

  • replaced by 0000.bmp from TWNKDW_1.ICN

CSTLKNGT.ICN
0020.bmp

  • renamed to 0025.bmp

0025.bmp

  • renamed to 0020.bmp

Necromancer's Castle.

There is a problem with the Laboratory road connection.

In fheroes2:
Laboratory road flaw fh2 01

In original game, corrected:
Laboratory road corrected H2 01

There is a problem with the Mage Guild road connection.

In fheroes2:
Mage Guild road flaw fh2 01

In original game:
Mage Guild H2 01

Necromancer castle screen, changes needed to avoid display "flaws" when switching between castles with Mausoleum and upgraded Mausoleum.
The position of the sprites must be adjusted.
Note: Graphic modifications have also been applied, they will be specified in another issue (changes to the half top of the sprite).

TWNNDW_4.ICN
0000.bmp

TWNNUP_4.ICN
0000.bmp

In fheroes2:
Dwelling Lich position fh2 01

In fheroes2:
Dwelling Lich position fh2 02

In original game, corrected:
Dwelling Lich position corrected H2 02

Open the three images in different tabs in your browser, enlarge the images and switch between the three tabs alternately.
fheroes2 unimproved dwelling first, fheroes2 upgraded dwelling second, Heroes 2 corrected position third.

Dwelling Lich position fh2 coupé 01 Dwelling Lich position fh2 coupé 02 Dwelling Lich position corrected H2 coupé 02

All changes applied (graphics change included).
Heroes 2 original first, Heroes 2 corrected second.
Original-0000 Change-0000

@Branikolog Branikolog added bug Something doesn't work Hacktoberfest help wanted Extra attention is needed ui UI/GUI related stuff labels Oct 21, 2020
@Branikolog Branikolog added this to the 0.9 milestone Oct 21, 2020
@ihhub ihhub self-assigned this Oct 24, 2020
ihhub added a commit that referenced this issue Oct 24, 2020
ihhub added a commit that referenced this issue Oct 24, 2020
@ihhub
Copy link
Owner

ihhub commented Oct 24, 2020

Hi @LeHerosInconnu , I've updated the order of building drawing. I didn't close this issue. Please check it again (once the compilation is done) and let me know if something is missing or simply close the issue if everything is fine. What I fixed:

  • Tavern drawing in Barbarian castle
  • Captain drawing in Necromancer castle
  • Statue drawing in Knight castle
  • Library drawing in Wizard castle

Of course not everything could be fixed as I noticed that the original game has even more issues than our.

@LeHerosInconnu
Copy link
Author

LeHerosInconnu commented Oct 24, 2020

@ihhub.

Hi @LeHerosInconnu , I've updated the order of building drawing. I didn't close this issue. Please check it again (once the compilation is done) and let me know if something is missing or simply close the issue if everything is fine. What I fixed:

  • Tavern drawing in Barbarian castle
  • Captain drawing in Necromancer castle
  • Statue drawing in Knight castle
  • Library drawing in Wizard castle

Of course not everything could be fixed as I noticed that the original game has even more issues than our.

All the changes you have made are working properly.
For other problems, it will certainly be necessary to slightly modify the graphics because a change in display priority will not necessarily solve the problem.

I suggest leaving the issue open as a reminder for the other problems.
If it's ok, I'll edit the first post to leave only the problems unfixed.

@ihhub
Copy link
Owner

ihhub commented Oct 25, 2020

Yes, please edit the first message. Maybe these pixel related issues because of another incorrect ICN decoding.

@LeHerosInconnu
Copy link
Author

@ihhub.

Yes, please edit the first message. Maybe these pixel related issues because of another incorrect ICN decoding.

I edited the first post.

@ihhub
Copy link
Owner

ihhub commented Oct 25, 2020

@ihhub.

Yes, please edit the first message. Maybe these pixel related issues because of another incorrect ICN decoding.

I edited the first post.

Mmm.. I don't see any changes :(

@LeHerosInconnu
Copy link
Author

@ihhub.

@ihhub.

Yes, please edit the first message. Maybe these pixel related issues because of another incorrect ICN decoding.

I edited the first post.

Mmm.. I don't see any changes :(

Maybe I hadn't updated the change. :)
It should be fine now.

@ihhub
Copy link
Owner

ihhub commented Oct 26, 2020

@LeHerosInconnu well, it happens. No worries :)

@ihhub ihhub added low priority Low priority item which could be addressed in the future and removed help wanted Extra attention is needed labels Oct 28, 2020
ihhub added a commit that referenced this issue Oct 29, 2020
also fixed crystal garden in Sorceress castle
relates to #2059
ihhub added a commit that referenced this issue Oct 29, 2020
also fixed crystal garden in Sorceress castle
relates to #2059
@ihhub
Copy link
Owner

ihhub commented Oct 29, 2020

Hi @LeHerosInconnu could you please verify castle drawings with latest commit? I made fixes for Barbarian castle roads.

Regarding different pixels in Knight and Sorceress castles: it might be that we miss some extra logic about image transcoding. I made some changes for images but it looks like it's not the case. So I don't know what's wrong with them.

@LeHerosInconnu
Copy link
Author

LeHerosInconnu commented Oct 29, 2020

Hello @ihhub,

Hi @LeHerosInconnu could you please verify castle drawings with latest commit? I made fixes for Barbarian castle roads.

Regarding different pixels in Knight and Sorceress castles: it might be that we miss some extra logic about image transcoding. I made some changes for images but it looks like it's not the case. So I don't know what's wrong with them.

Your corrections for the routes of the barbarian castle are working correctly.
It's perfect. :)
Did you modify the sprites "in the program", by cutting part of the sprites for example?

In these castles screens, there are also other flaws that have not been corrected in fheroes2, do you want me to report them in this thread, or it can wait?

I've modified the first post accordingly.

@ihhub
Copy link
Owner

ihhub commented Oct 31, 2020

Hi @LeHerosInconnu , I didn't modify any sprites but made a correct order of drawings.

If you spot some issues with sprites like incorrect pixels or colors, for example Calavry sprite which we fixed yesterday, please create a thread for this. We might able to fix it within an application without using modified assets of the game.

@LeHerosInconnu
Copy link
Author

@ihhub.

Hi @LeHerosInconnu , I didn't modify any sprites but made a correct order of drawings.

If you spot some issues with sprites like incorrect pixels or colors, for example Calavry sprite which we fixed yesterday, please create a thread for this. We might able to fix it within an application without using modified assets of the game.

Okay, I will add the other defects of the castles in this thread and create another thread for the other defects.

@LeHerosInconnu
Copy link
Author

First post updated with other flaws for Knight's Castle and Necromancer's Castle.

@ihhub ihhub modified the milestones: 0.9, 1.0 Jan 28, 2021
ihhub added a commit that referenced this issue Jul 11, 2021
- fix farm rendering in Knight castle (#2059)
- add missing farm sprite made by our artist
ihhub added a commit that referenced this issue Aug 19, 2021
- fix farm rendering in Knight castle (#2059)
- add missing farm sprite made by our artist
@ihhub
Copy link
Owner

ihhub commented Aug 25, 2021

Hi @LeHerosInconnu , could you please update the first post in regards to Knight castle since we fixed the issue with the Farm?

@LeHerosInconnu
Copy link
Author

Hello @ihhub,

Hi @LeHerosInconnu , could you please update the first post in regards to Knight castle since we fixed the issue with the Farm?

First post updated.

@Eldorias-Piano-Covers
Copy link

Eldorias-Piano-Covers commented May 3, 2022

Here is an animated gif concerning the Mosoleum pixel offset I was about to post (v 0.9.14) :
A

@ihhub ihhub added assets Game data related stuff and removed bug Something doesn't work ui UI/GUI related stuff labels Sep 17, 2022
@ihhub ihhub modified the milestones: 1.0, Beyond 1.0 Sep 17, 2022
@ihhub
Copy link
Owner

ihhub commented Sep 17, 2022

Reclassifying this issue as asset enhancement with target for beyond 1.0.

@ihhub ihhub added the improvement New feature, request or improvement label Sep 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assets Game data related stuff improvement New feature, request or improvement low priority Low priority item which could be addressed in the future
Projects
None yet
Development

No branches or pull requests

5 participants