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

Show the git commit version in screenshots/videos [Jak 2] #2677

Merged
merged 5 commits into from
Jun 4, 2023

Conversation

Zedb0T
Copy link
Contributor

@Zedb0T Zedb0T commented May 30, 2023

With issues/screenshots becoming more common its probably time to turn this on for debugging reasons.

@Zedb0T
Copy link
Contributor Author

Zedb0T commented May 30, 2023

(Can set display-sha to false in the startup.gc file to disable it for main devs)

…nner mode one

Before this it just didnt turn on but gave zero feedback so it was kind of bad...
@Zedb0T
Copy link
Contributor Author

Zedb0T commented May 30, 2023

2023-05-30.08-29-20.mp4

@Zedb0T Zedb0T changed the title Show the git commit version in screenshots/videos Show the git commit version in screenshots/videos [Jak 2] May 30, 2023
@@ -374,7 +374,6 @@
(define *display-heap-status* #f)
(define *display-actor-counts* #f)
(define *display-text-box* #f)
(define *display-sha* *debug-segment*)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why remove this? pckernel-h is loaded for all games.

@@ -256,7 +258,9 @@
`(begin
(cpad-clear! 0 r1)
(logxor! ,cheats-var (pc-cheats ,cheat))
(cheats-sound-play (logtest? ,cheats-var (pc-cheats ,cheat)))
(if (-> *pc-settings* speedrunner-mode?)
(sound-play "caught-eel")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sound effect is not in Jak 2! (what's the reason for this change anyway?) Also, fix the indenting please.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The reasoning for the change is in the commit


Before this it just didnt turn on but gave zero feedback so it was kind of bad...```

Then there is a video that is supplied as well. Do you have any suggestions for a sound that is present in both games? 

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, well it wasn't apparent that speedrun mode locked out cheats. Why not just not set the cheat variable instead of this, then?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you mean by that exactly?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This code still lets the cheat code go through. If you input the cheat for blue eco, you will still get blue eco even with speedrun mode enabled.
image

I dont like this sound as a negative feedback in this case open to suggestions
@water111 water111 merged commit d41858c into open-goal:master Jun 4, 2023
8 checks passed
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

Successfully merging this pull request may close these issues.

None yet

3 participants