dosbox-staging icon indicating copy to clipboard operation
dosbox-staging copied to clipboard

Disknoise failed to find fdd_spinup.flac and fdd_spin.flac

Open weirddan455 opened this issue 10 months ago • 4 comments

Are you using the latest DOSBox Staging version?

  • [x] I have checked and I'm using the latest release.

Different version than latest?

No response

What operating system are you using?

Linux x86_64

If other OS or Linux, please describe

No response

Are you running DOSBox Staging on a native OS?

  • [x] I am running Staging on a native OS

Are you able to trigger the issue with our official DOSBox Staging release?

  • [x] I am running an official Staging release, not a release installed by my operating system's package manager, or a version repackaged by an external team.

Relevant hardware info

No response

Have you checked that no other similar issue already exists?

  • [x] I have searched and have not found similar issues.

A clear and concise description of what the bug is

When floppy_disk_noise = on, I see the following in the logs:

2025-06-10 18:55:50.394 | DISKNOISE: Failed to find FLAC file: 'fdd_spinup.flac'
2025-06-10 18:55:50.394 | DISKNOISE: Failed to find FLAC file: 'fdd_spin.flac'

I do not see either of these files in the repo.

Steps to reproduce the behaviour

Explain in detail how to reproduce the bug. Assume no familiarity with the software.

  1. Set floppy_disk_noise = on and watch the logs

Download URL of affected game or software

No response

Your configuration

[disknoise]
floppy_disk_noise = on

Provide a log

2025-06-10 18:55:50.394 | DISKNOISE: Failed to find FLAC file: 'fdd_spinup.flac'
2025-06-10 18:55:50.394 | DISKNOISE: Failed to find FLAC file: 'fdd_spin.flac'

Code of conduct and contributing guidelines

  • [x] Yes, I agree.

weirddan455 avatar Jun 10 '25 23:06 weirddan455

Tagging @dividebysandwich here as well 😄

weirddan455 avatar Jun 10 '25 23:06 weirddan455

I have not included such samples because I found none that sounded good and not super annoying. I could add empty samples to get rid of the warning, I was kinda hoping someone might be able to provide good samples at some point

dividebysandwich avatar Jun 11 '25 08:06 dividebysandwich

Floppies don't spin up like HDDs, so that's clearly not needed.

The spin sample is needed; I guess we can just take the spin sample from any emulator (WinUAE, VICE, etc.)

johnnovak avatar Jun 11 '25 08:06 johnnovak

I'll take a stab at it - I'll also add the seek-only mode and consolidate the whole thing.

dividebysandwich avatar Jun 12 '25 08:06 dividebysandwich