Wrong Backgrounds #25

Closed
opened 2021-06-16 00:30:50 +00:00 by Ghost · 9 comments

At least one person is reporting Backgrounds that do not seem to match what the text says.
They seem to indicate that it's an issue mostly before the initial rooftop scene.
Listed were

  • Anon, Reed, and Trish in the bandroom, when they should be in the auditorium
  • Anon in the classroom instead of bandroom
  • Anon inside Fang's home before reaching it (pic related)

I have not personally double checked that these are errors, and will look into it tonight. Just wanted to create an issue when it got brought up in thread.

At least one person is reporting Backgrounds that do not seem to match what the text says. They seem to indicate that it's an issue mostly before the initial rooftop scene. Listed were * Anon, Reed, and Trish in the bandroom, when they should be in the auditorium * Anon in the classroom instead of bandroom * Anon inside Fang's home before reaching it (pic related) I have not personally double checked that these are errors, and will look into it tonight. Just wanted to create an issue when it got brought up in thread.
MichaelYick added the
bug
Low Priority
help wanted
labels 2021-06-16 00:33:13 +00:00
Author

Starting to dig into this now.

Edit: Looked into the initial 3 issues anon reported haven't gone through the whole game yet. Lot's of speculation ahead.

Lines ~4651 ~6643
Both times the texts supports you being in the Auditorium, but the background is the listed bandroom. Possibly used as a placeholder for an asset never created of the auditorium stage, should be an easy fix to change the file pointers

Line ~7773
Anon leaves his house with a fade to get on a bus and is instead inside Fang's house already. Again possibly a placeholder, or just needs removal to continue the fade or possibly use the sidewalk BG.

Starting to dig into this now. Edit: Looked into the initial 3 issues anon reported haven't gone through the whole game yet. Lot's of speculation ahead. Lines ~4651 ~6643 Both times the texts supports you being in the Auditorium, but the background is the listed bandroom. Possibly used as a placeholder for an asset never created of the auditorium stage, should be an easy fix to change the file pointers Line ~7773 Anon leaves his house with a fade to get on a bus and is instead inside Fang's house already. Again possibly a placeholder, or just needs removal to continue the fade or possibly use the sidewalk BG.
Author

@PrincipleSpears please let me know if I am wrong about any of this, this isn't a code issue as much as a story/knowledge issue. If the above assumptions are correct we can just roll the this into my intial PR, unless you would prefer them seperately for some reason.

@PrincipleSpears please let me know if I am wrong about any of this, this isn't a code issue as much as a story/knowledge issue. If the above assumptions are correct we can just roll the this into my intial PR, unless you would prefer them seperately for some reason.
Owner

Backgrounds not matching text is a code issue (after all they've been coded in) so this can stay. If you want to roll this into a PR, sure, but make sure that you note it down as solving this issue as well.

Backgrounds not matching text is a code issue (after all they've been coded in) so this can stay. If you want to roll this into a PR, sure, but make sure that you note it down as solving this issue as well.
Author

Better explanation. I understand it is literally a code issue (especially the fade scene), but as someone who's highschool auditorium always had band stuff on stage, I could see a wierd reality where it being the background for those scenes made sense.

But of course I'll mark it as such

Edit: PR is up and should fix this issue

Better explanation. I understand it is literally a code issue (especially the fade scene), but as someone who's highschool auditorium always had band stuff on stage, I could see a wierd reality where it being the background for those scenes made sense. But of course I'll mark it as such Edit: PR is up and should fix this issue
Owner

do you mind making a version of this PR for when https://git.snootgame.xyz/Cavemanon/SnootGame/pulls/17 gets merged? It'll be soon enough.

do you mind making a version of this PR for when https://git.snootgame.xyz/Cavemanon/SnootGame/pulls/17 gets merged? It'll be soon enough.
Author

Of course. Sort of assumed with the big boy almost being done that would be the case. I am currently on the road, but I will crank this out when I get to my destintation. As a heads up I will be unavailable tomorrow untill probably ~1800 CDT.

Of course. Sort of assumed with the big boy almost being done that would be the case. I am currently on the road, but I will crank this out when I get to my destintation. As a heads up I will be unavailable tomorrow untill probably ~1800 CDT.
Owner

sweet, can't wait to see it.

sweet, can't wait to see it.
Author

PR has been updated. Not tested. I'm doing this on a laptop in the back of a truck, using a mobile hotspot lol so apologies if something is borked.

PR has been updated. Not tested. I'm doing this on a laptop in the back of a truck, using a mobile hotspot lol so apologies if something is borked.
Author

Back of truck testing seems to work as intended.

Back of truck testing seems to work as intended.
MichaelYick added this to the Patchy-Patch5 Release milestone 2021-06-20 04:28:44 +00:00
MichaelYick removed the
help wanted
label 2021-06-20 04:31:41 +00:00
Sign in to join this conversation.
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Cavemanon/SnootGame#25
No description provided.