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

Add The Neverending Story FlexDMD #331

Merged
merged 5 commits into from
Nov 7, 2024

Conversation

evilwraith
Copy link
Contributor

@evilwraith evilwraith commented Oct 31, 2024

Add FlexDMD to Cabinet Mode for The Neverending Story

Original table here:

https://vpuniverse.com/files/file/15488-neverending-story-mod/

Add FlexDMD to Cabinet Mode
@francisdb
Copy link
Contributor

Mind renaming this pr to mention the table name?

@francisdb
Copy link
Contributor

francisdb commented Nov 1, 2024

Mind adding the table url to the PR description?

@evilwraith evilwraith changed the title Add FlexDMD Add The Neverending Story FlexDMD Nov 2, 2024
Changed filename to .vbs.dmd to meet naming convention.
Edit to fix error on standard res FlexDMD
@francisdb
Copy link
Contributor

francisdb commented Nov 2, 2024

Can you also rename the patch file to contain dmd ?

@francisdb
Copy link
Contributor

The downloaded file is Neverending Story (2021).vpx and not The Neverending Story(Original 2021) v1.vpx?

@francisdb
Copy link
Contributor

image

Looks like the embedded dmd no longer shows scores?

Changed names to reflect table download name.
Changed .vbs.patch to .vbs.dmd.patch
@evilwraith
Copy link
Contributor Author

Looks like the embedded dmd no longer shows scores?

I believe that's as intended. There's a change that can be made to the asset in vpx to fill that in. I can try to modify the script to address that but the solution is to change the asset.

@evilwraith
Copy link
Contributor Author

Asset is Wall54, backwall_noDMD

@francisdb
Copy link
Contributor

So #332 behaves the same? Found it strange that images do work.

+ DMDScene.AddActor FlexDMD.NewImage("Dig" & i, "VPX.dempty&dmd=2")
+ Digits(i).Visible = False
+ Next
+ digitgrid.Visible = False
Copy link
Contributor

Choose a reason for hiding this comment

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

Same fix required here?

@evilwraith
Copy link
Contributor Author

evilwraith commented Nov 4, 2024 via email

@francisdb
Copy link
Contributor

@jsm174 I think this is ok to go

@jsm174 jsm174 merged commit cb959fb into jsm174:master Nov 7, 2024
1 check passed
@jsm174
Copy link
Owner

jsm174 commented Nov 7, 2024

@evilwraith - awesome!!

@francisdb - thanks for working with @evilwraith on this!!

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.

3 participants