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

TimeSeries Slider & Play button not working #189

Closed
SHaringT opened this issue Jul 26, 2023 · 5 comments
Closed

TimeSeries Slider & Play button not working #189

SHaringT opened this issue Jul 26, 2023 · 5 comments

Comments

@SHaringT
Copy link

Hi,

I tried to use the web app to make a manifest. When I load a soundfile into the create timeseries ui and extract the amplitude/pitch I just can see the start. I can't move the blue slider to select another timeframe to be shown. Sometimes it moves but jumps right back.

Also the play button is not working. It shows "Pause" for a blink of an eye and then is back to play.

Tried it with Brave & Chrome browser.

Can I add further information? Would be glad to find a solution

Greetings

@rewbs
Copy link
Owner

rewbs commented Jul 27, 2023

Thanks for the report, you're absolutely right. Fixing now, hold tight.

rewbs added a commit that referenced this issue Jul 27, 2023
@rewbs
Copy link
Owner

rewbs commented Jul 27, 2023

This is resolved in v0.1.94, which is now deployed. Thanks again for the report and apologies for the issue!

@rewbs rewbs closed this as completed Jul 27, 2023
@SHaringT
Copy link
Author

Thanks for resolving it so quickly! Just an additional question if that is okay here: Is it intended to only update the positon when extracting once more? If not I could also try to take a look.

Greetings

@rewbs
Copy link
Owner

rewbs commented Jul 29, 2023

Hi! Yes that is my intent: only the selected segment is analysed, so if you select another segment you have to reanalyze, aka extract.

The alternative would be to analyse the whole track upfront and store the result in memory, and then allow the user to pan along the full analysed segment. I worry this would be too slow on long audio files.

So I have a slight preference for keeping the functionality as is. That being said I agree the UX is not very clear so if you have any ideas to make it more intuitive that would be awesome.

@SHaringT
Copy link
Author

Relatable. Do you have an Idea of where the perfomance bottleneck is? I only slightly touched JS/React and no typescript at all. From a Python/C# perspective the information and processing amount seems to be manageable. From The usage it seems like UI perfomance is falling behind?

I'm not trying to speak down your code, It's a great piece I just merely dipped into using and am quite interested in, not just from the user perspective. I just took a few glimpses into the code, but as the computing is done locally anyways maybe it would be an idea to outsorce the saving and processing to another backend/process/mini-api-server so the UI may just do it's UI thing? Apologies if something like this already is happening.

For the intuitiveness part; Maybe rename the button to "process snippet" and/or put an explanation next to it that only this part is processed? I need to say I'm also a madman at things like that and just skip through documentations when something is not working and gradually test through UI's :D Maybe also adding "calculation range from ms x to ms y" input box so everyone can test which range is still working good and you don't need to cut the piece in audacity to create seamless processed snippets.

Thanks for being so responsive and greetings!

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

No branches or pull requests

2 participants