-
Notifications
You must be signed in to change notification settings - Fork 16
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
ENH: Add AVA models and datasets #674
Comments
Discussed this with @marisbasha and @yardencsGitHub today. Updating here with some thoughts I've had
|
I don't think we need this for the initial implementation but noting for future work:
|
Tentative / rough to-do list for @marisbasha after our meeting today
|
@NickleDave I am having trouble with Just to clarify, I should use my own "toy data" or does running |
Hey @marisbasha! Sorry you're running into this issue. It's probably something we haven't explained clearly enough.
Just checking, did you already download the "source" test data as described here? To do that you would run nox -s test-data-download-source
You are right that these are basically "toy" datasets, that are as small as possible. I tried to define the two different types in that section on the development set-up page but just in case it's not clear: the "source" data is inputs to vak, like audio and annotation files. You create the other type, the "generated" test data, when you run You don't actually need to generate this test data to be able to develop. I just suggested it as a fairly painless way to check that you were able to set up the environment correctly. The script that generates the test data should be able to run to completion without any errors. I am almost finished with that feature branch that will fix the unit tests so you can run them to test what you are developing. That branch will also speed up the script that generates the test data considerable and reduce the size of the generated test data. Does that help? |
Everything fine now. Thanks! |
🙌 awesome, glad to hear it! Will ping you here as soon as I get that branched merged, it does fix a couple minor bugs so you'll probably want to |
@NickleDave I have pushed again to my fork the parts divided by file. |
Ah whoops, sorry I missed this @marisbasha. What you have so far looks great. I am reading through your code now to make sure I understand where you're at. We can definitely discuss what to do with the trainer when we meet tomorrow. |
https://autoencoded-vocal-analysis.readthedocs.io/en/latest/index.html
https://elifesciences.org/articles/67855
https://github.com/pearsonlab/autoencoded-vocal-analysis/tree/master
The text was updated successfully, but these errors were encountered: