-
Notifications
You must be signed in to change notification settings - Fork 10
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
Handle entering an artist without a song #5
Comments
Yeah, I see its having a problem because you entered an artist and not an artist+song. The program is capturing similar artists when it thinks its capturing similar tracks. I will try to fix this in the future, but for the present try this:
|
Ok. I created the issue because I just expected a human readable error instead of a backtrace :) It's not a problem as for me and thank you for this utility. I just reported a thing that I considered as an UX-problem :) |
@schollz I see there is an argument num to control the length of the generated playlist with default value 3, so I do not get the same error. |
@syc728 Actually the problem is that sometimes a user might enter an artist and not a song. I think what needs to happen is that the closest match to the user input should be relayed back to the user. For instance:
This can be the default behavior, actually, so that if someone does type a song it will still ask if that is the one that they meant:
If the user says "y" then it proceeds normally. If the user says "n" then it asks the user to retype the artist name + song name. The functionality for finding the first seed song is already done, so it just needs to prompt the user to check to make sure it is correct. |
@schollz Oh I got it! I will work on it. |
Hello. I'm getting error "IndexError: list index out of range"
Best regards, Dmitry.
The text was updated successfully, but these errors were encountered: