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

BUGFIX - gold+iron smelters sound problem #478

Open
p1plp1 opened this issue Sep 1, 2020 · 20 comments
Open

BUGFIX - gold+iron smelters sound problem #478

p1plp1 opened this issue Sep 1, 2020 · 20 comments

Comments

@p1plp1
Copy link

p1plp1 commented Sep 1, 2020

DataSourceDOS::get_sound(size_t index) { needs to be bugfixed for sound with if (index == 62),
when this resource is played it bubles only once and then high tone beeeep noise only ... any idea how to load this resource and cut only real data and play it correctly?

@sternenzaehler
Copy link

I will try to look into it.

@tlongstretch
Copy link

I had tried applying p1plp1's personal fix in my fork but it didn't seem to work so I reverted it.

@p1plp1
Copy link
Author

p1plp1 commented Apr 18, 2022

that was not fix :) that was only problem finding, for iron smelter sound with index 62 is there still problem, i tried to cut off and force sound lenght for index 62 with some constant value but without success, any progress anyone?

@tlongstretch
Copy link

My work-around is to play the Amiga sound instead. This could take a ton of time to solve the root cause, it doesn't seem worth the effort to me

@p1plp1
Copy link
Author

p1plp1 commented Jan 1, 2023

hello tlongstretch can i ask u for actual bugfixed whole compiled +sources packed into zip? i have difficulties to compile and run your version ... :( actually i cant get it working and play amiga music, when i use amiga data files also color palette is bad

@tlongstretch
Copy link

tlongstretch commented Jan 5, 2023

Of Forkserf or Freeserf? On Forkserf "Releases" area I have pre-compiled binaries for linux and windows, though I've only tested the legacy glibc versions personally. I can probably compile Freeserf also if you're having issues compiling it, though I thought it was already published here on Freeserf project.

https://github.com/forkserf/forkserf/releases

Freeserf has always had compiled binaries herE: https://github.com/freeserf/freeserf/releases/tag/v0.3

@tlongstretch
Copy link

When you say Amiga color palette is bad, what are you expecting it to look like? I had always assumed this is what it originally looked like as I never saw the Amiga version. Now, looking at screenshots if I google for "Settlers Amiga" it looks much like the DOS version. I think you are correct that the terrain is corrupted, but I'm not sure if there's anything that can be done to fix it. It looks same to me in Freeserf or Forkserf
Screenshot from 2023-01-04 22-12-56

@tlongstretch
Copy link

Regarding the DOS boils sound, I tried experimenting with editing it chunk by chunk as it loads and no success.

@jacefuse
Copy link

jacefuse commented Jan 5, 2023 via email

@p1plp1
Copy link
Author

p1plp1 commented Jan 5, 2023

yes, looks like that mess on that picture, pallete must be fixed prior of displaying any sprites etc.
i have two ideas:
-maybe saving pc pallete and trying to apply it with amiga files can solve this
-or try to redefine upper 32 collors with some constant (compare with original pc pallete) and only redefine
maybe it is pallete_color(index+32)=pallete_color(index+32)*4 for example, dunno "maybe some inspiration for some formula is based in default 256color pallette:
256colors

@tlongstretch
Copy link

tlongstretch commented Jan 5, 2023

you might be correct about the fix. Is there a significant different in the (correct) Amiga graphics versus DOS? The non-terrain sprites look extremely similar, and I nearly all cases I think the DOS ones look better. The only ones where Amiga looks arguably better are the Castle, Stock, and Garrison sprites, where Amiga has a brown roof while DOS has red.

amiga_vs_dos_buildings

@GraionDilach
Copy link

The star overlay on the ingame bottom bar's buttons when they are active is only animated in Amiga AFAIK.

@tlongstretch
Copy link

You are right, that spinning star does look slick. I just watched some youtube amiga settlers gameplay. I don't see anything else that looks significant, though. I think implementing the amiga spinning star and otherwise using DOS graphics is the way to go!

@tlongstretch
Copy link

it seems Freeserf does not implement the spinning star even if using Amiga graphics, unfortunately

@p1plp1
Copy link
Author

p1plp1 commented Jan 7, 2023

about boiling sound - i have just unconfirmed idea that problem can be in sound extraction algorythm, maybe i try to expand data and save it as playable wav sound and test it in audio player, if problem happends just during loading sound from file due to wrong algorythm.

@p1plp1
Copy link
Author

p1plp1 commented Jan 7, 2023

about castle, storage - hmmm that brown roof + rock wall looks better but if u look at road at front of the storage it looks dirty in amiga version, that red / brown roof is maybe rooot of that pallete problem --> what about shifting brown to be more red? maybe it fixes whole game and that terrain + desert textures too ...

@p1plp1
Copy link
Author

p1plp1 commented Jan 7, 2023

now im sure i have problems with mingw64 compiler and threads on Windows x64 ... dunno exactly why but i get SDL_CreateMutex errror hm, i updated mingw during time ...

@tlongstretch
Copy link

maybe try VSCode? That is what I am using on linux, and I was using VisualStudio Community Edition when I was developing on Windows

@tlongstretch
Copy link

I got the amiga spinning star working as a prototype. I had to extract the spinning star frame by frame from a youtube video, if anybody knows how to extract it from the amiga data files that would be better, but this still looks pretty good

amiga_spinning_star_example.mp4

@zdechlak
Copy link

https://thecompany.pl/gra/Settlers%7C+The

here you can play settlers via amiga emulator

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

6 participants