Port Scrolling Texture Interpolation over from 2Ship#6224
Open
leggettc18 wants to merge 16 commits intoHarbourMasters:developfrom
Open
Port Scrolling Texture Interpolation over from 2Ship#6224leggettc18 wants to merge 16 commits intoHarbourMasters:developfrom
leggettc18 wants to merge 16 commits intoHarbourMasters:developfrom
Conversation
Mainly fixes some of the lingering water temple related things, but also apparently affects some textures in Tower Escape and/or the cutscene afterwards.
Contributor
Author
|
Removing do-not-merge as I think the "issues" that are left are either fixed or not actually issues, just me having trouble actually verifying if they are correct visually. Playtesting and feedback encouraged! |
It looks like they have a scrolling texture that just doesn't actually move, or maybe this makes it so the texture doesn't actually rotate with the gem? Not sure but it looks more correct to me this way.
Not 100% sure if it's correct but it looks right.
Contributor
Author
|
Found some minor issues with Phantom Ganon and the fire in the cutscene with the deku tree sprout afterwards, have PG already fixed locally but the fire in the cutscene is giving me a hard time. |
Contributor
Author
|
On closer inspection I think the fire might have been fine, everything I've tried to do to fix it has looked worse. I think it looking choppy to me before might have been an optical illusion since there is a low enough offscreen that I only see the points of the animation where flames are higher. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Big thanks to @Eblo and @NEstelami, this is porting over the implementation by @NEstelami from 2ship and applying to Ship's code.
Known issues with the current implementation, some of these may just be errors on my part, some are deeper issues:
Deku Tree kind of "shimmers" best I can tell he has a scrolling texture for his death animation and it keeps trying to interpolate to the next frame, but in game the texture doesn't actually start scrolling until later.Like Likes have a similar thing going on when they are stunned.Great Fairy Fountain walls need to be double checked. They might be fine but they are visually busy so it's hard to tell.(Now that I've had a chance to directly compare these, they are interpolating fine after all)Same for Spiritual Stones, they might be fine but I can't even really tell where the scrolling texture even is.Water Temple Waterfall room before Dark Link, water does not appear to be interpolated correctly.Water Spouts (like the one covering the map chest in Water Temple) aren't interpolating correctly.Logo Screens (LUS and N64 logos) have weird flickering happening when you soft-reset the game.I either accidentally fixed this or it is Debug build exclusive, need to do more investigation.Blue Warps need to be double checked, they might be fine but something seemed off when I last looked at one.I copied what 2ship did. It looks fine in game as far as I can tell.Morpha's inner core looks fine but the outer shell looks off.(Did a direct comparison, Morpha just always looked like that)Boss Souls may not look right, I haven't checked them yet but I have a feeling I made a mistake there somewhere.Might be more too, haven't finished my vanilla playthrough for playtesting. Overall my playtest playthrough has been going well, and seeing all the smoothed out scrolling textures is very pleasing on the eyes. Areas like Zora's Domain and Water temple are quite stunning with the scrolling textures interpolated. Wanted to go ahead and get the PR opened for this so other people could have a gander and see if they can find any issues I haven't noticed.
Build Artifacts