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

Change to /api/feed/v2, fix new format with stems, typos #5

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kuhnchris
Copy link

@kuhnchris kuhnchris commented Jul 26, 2024

Hello!

Since I currently play around with Suno, I noticed that they recently added the data for generating STEMS (pro) into their UI, breaking the predefined classes. So here is a fix for that.
Since I also run into troubles with the pagination / amount of "songs" returned, I changed the endpoint to the /api/feed/v2 endpoint, giving us "num_of_songs" which helps us iterate through the "pages".
I added example code for just this into the README.md

(edit: 🙈 forgot to mention:
Also added the MetadataHistory Model, so you can identify where a song is coming from (song 1 -> song 2 -> song 3, etc.), and added new example data for the v3.5 metadata/clips structure)

And lastly, i fixed a typo in check_error.

If there is anything missing let me know and I'll be glad to fix it. :-)

Thanks for your effort making this API (publicly available),
Chris

@Malith-Rukshan
Copy link
Owner

Hello @kuhnchris,

Thanks for your detailed contributions and fixes! 🤝 The updates for generating STEMS, pagination, and the MetadataHistory Model are much appreciated. I'll review and merge these changes in the next update. If anything else is needed, I'll let you know.

🫶 Thanks for improving the API!

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

Successfully merging this pull request may close these issues.

2 participants