-
Notifications
You must be signed in to change notification settings - Fork 52
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
Adds option to configure video interpolation method #539
Conversation
Can you make this an option in SettingsClass and set the default to full linear? Maybe in the Video section? |
5ca1f10
to
6de609a
Compare
Yes I can. |
d85ea51
to
8f1fe66
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested the modes work as intended with a few cleanups required.
redalert/mapsel.cpp
Outdated
@@ -154,17 +154,15 @@ char const* Map_Selection(void) | |||
Animate_Frame(anim, *pseudoseenbuff, 1); | |||
for (int x = 0; x < 256; x++) | |||
memset(&PaletteInterpolationTable[x][0], x, 256); | |||
CopyType = 1; | |||
Interpolate_2X_Scale(pseudoseenbuff, &SeenBuff, 0); | |||
Settings.Video.InterpolationMode = 1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this assignment and use 1
in the call. Setting it will change the ini value to 1
after you quit the game.
@@ -1697,23 +1692,23 @@ void Multi_Score_Presentation(void) | |||
Animate_Frame(anim, *pseudoseenbuff, 1); | |||
for (int x = 0; x < 256; x++) | |||
memset(&PaletteInterpolationTable[x][0], x, 256); | |||
CopyType = 1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can either revert these changes because they are ifdeffed out or fix them to use the argument as well.
8f1fe66
to
6aa0b8d
Compare
Changes default interpolation from scanline to full linear. Adds option for conquer.ini/redalert.ini [Video] section, InterpolationMode=x 0 = Horizontal interpolation, vertical skip scanline. 1 = Horizontal interpolation, vertical doubling. 2 = Full interpolation in both axis.
6aa0b8d
to
7b2b1ad
Compare
Appearance on scaled resolutions that are not integer scaled is much improved.