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

Adding Audio Routing #1172

Merged
merged 3 commits into from
May 8, 2024
Merged

Adding Audio Routing #1172

merged 3 commits into from
May 8, 2024

Conversation

codingwithsurya
Copy link
Contributor

Adding Audio Routing

Github Issue Number Here: <#1162>
What user problem are we solving?
We are extending the Deep Learning Playground's capabilities to include audio data processing, allowing users to interact with, upload, and train deep learning models on audio files.

What solution does this PR provide?
This PR introduces a new route for audio processing similar to the existing image and tabular data routes. It includes the creation of schemas.py, audio.py, and init.py under the routes/audio directory. Additionally, it integrates this new route into the main application via urls.py.

Testing Methodology

How did you test your changes and verify that existing functionality is not broken

Any other considerations

@karkir0003
Copy link
Member

@codingwithsurya how did you test that this PR works?

@karkir0003
Copy link
Member

What are the next steps after this PR?

Copy link
Member

@karkir0003 karkir0003 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

left comments. please address

Copy link

sonarcloud bot commented May 4, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@codingwithsurya
Copy link
Contributor Author

@codingwithsurya how did you test that this PR works?

i didn't run any unit tests or anything since this was just getting the folder and routing setup

@codingwithsurya
Copy link
Contributor Author

codingwithsurya commented May 4, 2024

What are the next steps after this PR?

just filling out the files with necessary things for audio training. check out issues #1161 and #1156 for greater detail

@codingwithsurya
Copy link
Contributor Author

@karkir0003 i can only go ahead and get the schemas.py set up in this pr itself if u would like

@karkir0003
Copy link
Member

@karkir0003 i can only go ahead and get the schemas.py set up in this pr itself if u would like

sounds good

@codingwithsurya
Copy link
Contributor Author

@karkir0003 i can only go ahead and get the schemas.py set up in this pr itself if u would like

sounds good

let me do some research on how this will look like. we're employing kind of a different strategy here with our custom architecture. i'll get back to u on this.

@codingwithsurya
Copy link
Contributor Author

schemas.py will be handled in #1156.

@codingwithsurya codingwithsurya added this pull request to the merge queue May 8, 2024
Merged via the queue into nextjs with commit 9cb69e4 May 8, 2024
4 checks passed
@codingwithsurya codingwithsurya deleted the audio-route branch May 8, 2024 02:24
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.

[FEATURE]: Create Audio Route for Deep Learning Playground and Implement Audio Routing in urls.py
2 participants