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

The GI string light intensities in Metallica are too low when UseVPMModSol = 2 #312

Open
apophis79 opened this issue Aug 30, 2024 · 2 comments

Comments

@apophis79
Copy link

Describe the bug
The GI strings in Metallica are driven using lamp matrix indexes 130, 132, 134, and 136. When using new PWM functionality for the lamp matrix (UseVPMModSol = 2) the maximum intensity of the GI strings is very low (approx 20% of max). To solve this issue, I had to multiply the resulting PWM values coming from VPinMame for these lights by a factor of 5.

To Reproduce
Steps to reproduce the behavior:

  1. Download the table at this link (I will need to give you access, so please request access): https://drive.google.com/file/d/1SaIN1QB_dxxgdogCGAPziFIlqPQpP_jS/view?usp=drive_link
  2. Download the ROM at this link: https://drive.google.com/file/d/1wYtYMJVI4G-jAI-0W3ZdHTfp_MaNCQ_9/view?usp=sharing
  3. Search in the script for Const GIScale and set it to a value of 1.
  4. Notice the GI lights are always very dim. Setting the GIScale back to 5 makes use the full brightness range.

Expected behavior
The GI Strings PWM signals should not need to be manually manipulated in the script to work correctly.

Versions & Settings
VPinMame: 3.6.0.1038
VPX: v10.8.0 Beta Rev 1995 (a491317) windows directX 64bits
Windows 11

@vbousquet
Copy link
Contributor

vbousquet commented Aug 31, 2024

This issue is very likely linked to #311

Metallica uses the grinder nodeboard (part 520-5331-00) which is not fully emulated. We would need the firmware of this board or spend a significant amount of time reverse engineering the main CPU board to go further.

Metallica GI is wired to the coil outputs, using the coil PWM to drive intensity which (may) explain why the range is wrong as solenoid PWM is not yet supported in PinMame.

@apophis79
Copy link
Author

Note, I removed the table link in my original post above. You can just download the latest version of VPW's Metallica from VPUniverse.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants