M
Mudlet•3y ago
Mike C.

It may wait one second on some devices

It may wait one second on some devices, tough to say. There is a hook in the programming that when the position changes it will calculate the volume change https://github.com/Mudlet/Mudlet/blob/4655c93b2f936e603c15a25bf536ce05efb066c4/src/TMedia.cpp#L719
23 Replies
Mike C.
Mike C.OP•3y ago
Looks like we could play with this setNotifyInterval() perhaps https://forum.qt.io/topic/61680/qmediaplayer-positionchanged-qint64/5 @Talon It is possible to make it much more precise, for a cost I assume 🙂 It defaults to 1000ms, we could take it to 16ms in the example they had. @Talon if you make an issue on GitHub then I will take a look and ask you to test it. !issue
Carl-bot
Carl-bot•3y ago
You can let us know any feedback or doubts you may have in a github issue: https://github.com/Mudlet/Mudlet/issues/new
Unknown User
Unknown User•3y ago
Message Not Public
Sign In & Join Server To View
Mike C.
Mike C.OP•3y ago
No - let's say you have ambience in one room, then move to another room and it restarted the ambience. Continue helps keep the ambience playing if it receives a new call for that same ambience from the client. So you can move 10 rooms in a zone and have the same music - perhaps when you switch zones it plays something different. Does that read well? There is a cheap version of pan with start https://wiki.mudlet.org/w/Manual:Scripting#start if that helps. You can start a track so many seconds in. And yes you could use Client.Media.Play with continue to adjust volume.
Unknown User
Unknown User•3y ago
Message Not Public
Sign In & Join Server To View
Mike C.
Mike C.OP•3y ago
Trying it out
Unknown User
Unknown User•3y ago
Message Not Public
Sign In & Join Server To View
Mike C.
Mike C.OP•3y ago
Small bug. If you take out the "loops": -1, for now, or change it to "loops": 1, you will be good. I will resolve it.
Unknown User
Unknown User•3y ago
Message Not Public
Sign In & Join Server To View
Mike C.
Mike C.OP•3y ago
This is true. And it should, so I am investigating root cause 🙂 Hey @Talon - would you mind trying this out in a development version of Mudlet and let me know if your problem has disappeared? Please pick the version of Mudlet from here: https://github.com/Mudlet/Mudlet/pull/6264#issuecomment-1229690467 and please reply here or in the comments of that GitHub request as to the success/failure.
Unknown User
Unknown User•3y ago
Message Not Public
Sign In & Join Server To View
Mike C.
Mike C.OP•3y ago
Hi @Talon, please try out a fix for the scenario you described with the Mudlet version found here: https://github.com/Mudlet/Mudlet/pull/6274#issuecomment-1233635210 and let me know how it goes. You may hear a very slight pause when you start the successive commands sometimes. This seems barely noticable and infrequent. I will consider if there is something more could be done in the future.
Unknown User
Unknown User•3y ago
Message Not Public
Sign In & Join Server To View
Mike C.
Mike C.OP•3y ago
Yes. The fix will be available in the next release. You could download the Mudlet PTB and use it up until the next release to get around it if you like.
Unknown User
Unknown User•3y ago
Message Not Public
Sign In & Join Server To View
Mike C.
Mike C.OP•3y ago
Yes, possibly. Anything where continue with loops could be impacted. You could give it a try with the public test build to ensure it is gone: https://make.mudlet.org/snapshots/?platform=all&source=ptb
Unknown User
Unknown User•3y ago
Message Not Public
Sign In & Join Server To View
Mike C.
Mike C.OP•3y ago
Yes, Linux needs some help to play sound. Have to compile in some other code. I know there is some guidance here on the Discord on that subject.
Unknown User
Unknown User•3y ago
Message Not Public
Sign In & Join Server To View
Unknown User
Unknown User•3y ago
Message Not Public
Sign In & Join Server To View
Mike C.
Mike C.OP•3y ago
OK, found the issue there I believe and will send a fix. Still working on this and will keep you updated. Burned at least 20h this weekend trying to figure it out 🙂 Classic programming stuff. It will probably be one line to change in the end or something. Hi @Blizzard , when you get a chance, could you install and try a fixed version of Mudlet to demonstrate the fix is complete? You could follow the example I have in the pull request, or try the scenario you created to help show success. Here is the link: https://github.com/Mudlet/Mudlet/pull/6389#issuecomment-1288210934. If you could comment here or in the PR that this is resolved, that will be most appreciated! Hi @Blizzard did you have a chance to give it a try yet? Thanks!
Unknown User
Unknown User•3y ago
Message Not Public
Sign In & Join Server To View
Want results from more Discord servers?
Add your server