Spotify playlist generator #140

sandyjswl · 2020-10-02T05:53:41Z

Spotify Playlist Generator

Generate multiple playlists of a given size from a large Spotify playlist.

Fixes (#140 )

Type of change

Choosing one or more options from the following as per the nature of your Pull request.

NOTE: These boxes can be checked using [X]
Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Documentation Update

Checklist:

Please tick all the boxes that are fulfilled by your Pull Request.

I have named my files and folder, according to this project's guidelines.
My code follows the style guidelines of this project.
My Pull Request has a descriptive title. (not a vague title like Update index.md)
I have commented on my code, particularly in hard-to-understand areas.
I have created a helpful and easy to understand README.md, according to the given README_TEMPLATE.
I have included a requirements.txt file (if external libraries are required.)
My changes do not produce any warnings.
I have added a working sample/screenshot of the script.

sandyjswl · 2020-10-02T05:58:34Z

@AdityaJ7 Can you please review this?

AdityaJ7

Please comment your code everything else looks good to me

sandyjswl · 2020-10-02T10:48:54Z

Please comment your code everything else looks good to me

@AdityaJ7 Added comments, can you please merge the PR?

AdityaJ7 · 2020-10-02T10:50:30Z

Please comment your code everything else looks good to me

@AdityaJ7 Added comments, can you please merge the PR?

Will be merged automatically when two approved reviews are received

tusharnankani

Can you remove unnecessary blank lines from the code?

Also, add id, a playlist for testing purposes.

tusharnankani · 2020-10-02T10:55:50Z

Scripts/Miscellaneous/Spotify_Playlist_Generator/README.md

+1. [Create Spotify Developers Account and create a new app](https://developer.spotify.com/)
+2. Export Client Id, Client Secret and Redirect URI
+    * `export SPOTIPY_CLIENT_ID='your-spotify-client-id'`
+    * `export SPOTIPY_CLIENT_SECRET='your-spotify-client-secret'`
+    * `export SPOTIPY_REDIRECT_URI='your-app-redirect-url'`
+3. `python main.py -p [playlist_id] -l [size_of_each_playlist]`


Hi! Can you add a test id and playlist? For viewers to test.

Hi! I don't have a test account on Spotify, i tested these on my personal account.

chavarera · 2020-10-02T11:03:35Z

Scripts/Miscellaneous/Spotify_Playlist_Generator/README.md

+
+### Screenshot/GIF showing the sample use of the script
+<!--Remove the below lines and add yours -->
+![alt text](/screenshots/screenshot.png)


remove /

Suggested change

![alt text](/screenshots/screenshot.png)

![alt text](screenshots/screenshot.png)

chavarera · 2020-10-02T11:05:59Z

Scripts/Miscellaneous/Spotify_Playlist_Generator/spotify_playlist_generator.py

+spotipy_instance = spotipy.Spotify(auth_manager=SpotifyOAuth(scope=scope, cache_path=os.getcwd()))
+
+
+# Method to read the command line arguments and parse them


instead of adding comments outside from functions make doc string inside in functions

Suggested change

# Method to read the command line arguments and parse them

def get_args():

"""Method to read the command line arguments and parse them

Input:

Output:

"""

chavarera · 2020-10-02T11:07:34Z

Scripts/Miscellaneous/Spotify_Playlist_Generator/spotify_playlist_generator.py

+def get_track_ids_for_playlist(playlist):
+    res = []
+    for song in playlist:
+        #Extract the ID of the track


when you are writing single line command use after you statement

Suggested change

#Extract the ID of the track

res.append(song['track']['id']) # Extract the ID of the track

chavarera · 2020-10-02T11:10:36Z

Scripts/Miscellaneous/Spotify_Playlist_Generator/spotify_playlist_generator.py

+# This method generates smaller playlists from the input playlist
+def generate_playlists(playlist_size, playlist_songs, user_id):
+    # Create the smaller playlists from the given large playlist 
+    smaller_playlists = [playlist_songs[x:x + playlist_size] for x in range(0, len(playlist_songs), playlist_size)]


Limit all lines to a maximum of 79 characters.

Suggested change

smaller_playlists = [playlist_songs[x:x + playlist_size] for x in range(0, len(playlist_songs), playlist_size)]

smaller_playlists = [playlist_songs[x:x + playlist_size] for x in

range(0, len(playlist_songs), playlist_size)]

tusharnankani · 2020-10-03T18:28:44Z

Are you working on it? @sandyjswl

sandyjswl · 2020-10-03T18:31:15Z

Hi, I'll fix the changes tommorow. I won't be able to the ID and Playlist because I don't to expose my personal playlist. But I can make the other changes.

sandyjswl · 2020-10-15T08:24:18Z

@tusharnankani @AdityaJ7 Can you merge this now?

AdityaJ7 · 2020-10-26T14:39:19Z

@allcontributors please add @sandyjswl for code

allcontributors · 2020-10-26T14:39:30Z

@AdityaJ7

I've put up a pull request to add @sandyjswl! 🎉

github-actions · 2022-06-10T10:13:14Z

Python World does not accept pull requests on this repository anymore.

sandyjswl added 2 commits October 2, 2020 11:19

Add Spotify Playlist Generator script

f4d913e

Add Spotify Playlist Generator scripts

d58e2f1

AdityaJ7 requested review from AdityaJ7 and tusharnankani October 2, 2020 10:08

AdityaJ7 suggested changes Oct 2, 2020

View reviewed changes

Added comments

366692e

AdityaJ7 approved these changes Oct 2, 2020

View reviewed changes

tusharnankani suggested changes Oct 2, 2020

View reviewed changes

chavarera suggested changes Oct 2, 2020

View reviewed changes

sandyjswl and others added 4 commits October 5, 2020 07:40

PR Changes

5ccad2b

Add deepsource

1d10ebb

Edit deep source

742441c

Delete .deepsource.toml

1d6d275

AdityaJ7 requested review from tusharnankani and chavarera October 26, 2020 11:59

tusharnankani approved these changes Oct 26, 2020

View reviewed changes

mergify bot merged commit 251d79f into Python-World:master Oct 26, 2020

mergify bot added the hacktoberfest-accepted label Oct 26, 2020

allcontributors bot mentioned this pull request Oct 26, 2020

docs: add sandyjswl as a contributor #607

Merged

github-actions bot locked and limited conversation to collaborators Jun 10, 2022

mergify bot added the nice-job label Jun 10, 2022

	![alt text](/screenshots/screenshot.png)
	![alt text](screenshots/screenshot.png)

		spotipy_instance = spotipy.Spotify(auth_manager=SpotifyOAuth(scope=scope, cache_path=os.getcwd()))


		# Method to read the command line arguments and parse them

-# Method to read the command line arguments and parse them
+def get_args():
+     """Method to read the command line arguments and parse them
+     Input:
+     Output:
+     """

	#Extract the ID of the track
	res.append(song['track']['id']) # Extract the ID of the track

	smaller_playlists = [playlist_songs[x:x + playlist_size] for x in range(0, len(playlist_songs), playlist_size)]
	smaller_playlists = [playlist_songs[x:x + playlist_size] for x in
	range(0, len(playlist_songs), playlist_size)]

Spotify playlist generator #140

Spotify playlist generator #140

Uh oh!

Conversation

sandyjswl commented Oct 2, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Spotify Playlist Generator

Type of change

Checklist:

Uh oh!

sandyjswl commented Oct 2, 2020

Uh oh!

AdityaJ7 left a comment

Choose a reason for hiding this comment

Uh oh!

sandyjswl commented Oct 2, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

AdityaJ7 commented Oct 2, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

tusharnankani left a comment

Choose a reason for hiding this comment

Uh oh!

tusharnankani Oct 2, 2020

Choose a reason for hiding this comment

Uh oh!

sandyjswl Oct 5, 2020

Choose a reason for hiding this comment

Uh oh!

chavarera Oct 2, 2020

Choose a reason for hiding this comment

Uh oh!

chavarera Oct 2, 2020

Choose a reason for hiding this comment

Uh oh!

chavarera Oct 2, 2020

Choose a reason for hiding this comment

Uh oh!

chavarera Oct 2, 2020

Choose a reason for hiding this comment

Uh oh!

tusharnankani commented Oct 3, 2020

Uh oh!

sandyjswl commented Oct 3, 2020

Uh oh!

sandyjswl commented Oct 15, 2020

Uh oh!

AdityaJ7 commented Oct 26, 2020

Uh oh!

allcontributors bot commented Oct 26, 2020

Uh oh!

github-actions bot commented Jun 10, 2022

Uh oh!

Uh oh!

sandyjswl commented Oct 2, 2020 •

edited

Loading

sandyjswl commented Oct 2, 2020 •

edited

Loading

AdityaJ7 commented Oct 2, 2020 •

edited

Loading