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

Ogg Audio Support #33

Closed
ryantheleach opened this issue Apr 1, 2021 · 8 comments
Closed

Ogg Audio Support #33

ryantheleach opened this issue Apr 1, 2021 · 8 comments
Assignees
Labels
⭐ enhancement An enhancement or new feature!
Milestone

Comments

@ryantheleach
Copy link

Please add support for the Ogg Audio format, NAudio appears to support it. <3

@GeekyEggo
Copy link
Owner

GeekyEggo commented Apr 4, 2021

@ryantheleach this is now available as part of a separate branch; it would be great if you could download it, have a play, and let me know if you experience any problems.

https://github.com/GeekyEggo/SoundDeck/raw/33-ogg/dist/com.geekyeggo.sounddeck.streamDeckPlugin

@ryantheleach
Copy link
Author

I'll try it today, thanks

GeekyEggo added a commit that referenced this issue Apr 4, 2021
@GeekyEggo GeekyEggo added this to the vNext milestone Apr 4, 2021
@ryantheleach
Copy link
Author

ryantheleach commented Apr 4, 2021

Running the file as-is without removing the old one (last version pushed via the StreamDeck Store) resulted in existing (Sampler Action) buttons giving 'alerts' when pressed.

restarting streamdeck did not fix this.

Opening task manager revealed many many SoundDeck instances installed (probably from my earlier testing of my own streamdeck plugin, it appears that the (previous) version of SoundDeck does not close when StreamDeck is restarted, or is not being opened as a child process of StreamDeck correctly)
image

Killing these now in task manager.

re-opening the plugin file you attached.

Sampler buttons still broken.

Existing sound effect buttons work fine.

New ogg support works for play sound effect button playlists.

Created a new Sampler button and configured it, worked fine. All other sampler buttons started working again.

@ryantheleach
Copy link
Author

ryantheleach commented Apr 4, 2021

So basically,

  1. Ogg support works.
  2. Sampler buttons don't upgrade cleanly in this version. but still work.
  3. SoundDeck no longer leaves background processes running in this new version.

@GeekyEggo
Copy link
Owner

GeekyEggo commented Apr 4, 2021

Can you elaborate on the Samplers not upgrading cleanly? Is it simply a case that pre-existing samplers don't work?

For local development I'd recommend using the npm run stop command; this is a helper script within the package.json file that kills both the Stream Deck, and the Sound Deck executable process tree.

@ryantheleach
Copy link
Author

I'd been using fritzandfriends streamdecktoolkit, but thanks.

The Samplers were pre-existing samplers, but they started working at one point, not sure if it was the multiple sounddeck's that I killed, or creating a new sampler someone re-inited them.

I don't have a backup, nor 100% knowledge which version I was using before testing this, so I apologise how awkward it will be to recreate it, and hope that it's a non issue / complicated with the other stuff I was doing.

@GeekyEggo
Copy link
Owner

GeekyEggo commented Apr 6, 2021

I've attempted to replicate the sampler issues by;

  1. Downgrading to the current version in the store.
  2. Creating a sample.
  3. Installing the beta.

I haven't been able to replicate any issues, and tested with multiple scenarios; one key change in this beta is the added "Default" devices, although this seemed to be okay. I'll be merging this branch back into main soon.

@GeekyEggo
Copy link
Owner

Looking at the NAudio.Wave.AudioFileReader class it appears that aif[f] format is also supported natively.

GeekyEggo added a commit that referenced this issue May 19, 2021
GeekyEggo added a commit that referenced this issue May 19, 2021
GeekyEggo added a commit that referenced this issue May 19, 2021
@GeekyEggo GeekyEggo added ⭐ enhancement An enhancement or new feature! and removed ⚡ enhancement labels Nov 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⭐ enhancement An enhancement or new feature!
Projects
None yet
Development

No branches or pull requests

2 participants