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

Updated Spectogram.py #135

Merged
merged 2 commits into from
Jun 17, 2024
Merged

Conversation

manno198
Copy link
Contributor

@manno198 manno198 commented Jun 16, 2024

Updated Spectogram.py

I have added a "Save" button to the layout in the PySimpleGUI window.
The button is initially disabled and is enabled only when the "Listen" button is active.

A new function save_spectrogram was added. This function opens a file save dialog, allowing the user to select the location and filename to save the spectrogram image.
The function uses fig.savefig(file_path) to save the current spectrogram figure to the chosen location.

Checklist
️✅I have gone through the contributing guide
️✅I have updated my branch and synced it with the project main branch before making this PR

closes #125

I have added the "Save" button for saving the screenshot of spectrogram .

Also corrected spelling error in "Spectrogram" which was "Spectogram" previously.
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Great job, @HarshitaSingh198! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.

We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

I changed the spelling "Spectrogram" back to "Spectogram" ,the spelling typo mistake I did previously.
Copy link
Collaborator

@dinxsh dinxsh left a comment

Choose a reason for hiding this comment

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

LGTM, should be ready for merge

@Soumya-Kushwaha Soumya-Kushwaha merged commit 9e5e0ea into Soumya-Kushwaha:main Jun 17, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request gssoc level1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feature addition in Spectrogram.py for saving the screenshot of spectrogram
3 participants