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
Insert the actual parameter names in the documentation #19
Comments
Not sure what you mean, can you provide an example? |
For example in the teamyearbyyearstats endpoint, the Parameters are named LeagueID, PerMode, SeasonType and TeamID, but in python the actual variables are called league_id, per_mode_simple, season_type_all_star and team_id. |
I agree. This is something that definitely needs to be added in. |
Right now, the only identifying pattern is the regex that can be cross referenced to parameters documentation. However, this one is should have been added in the original documentation because no wonder people seem to be having problems using their own parameters. Should be a relatively easy fix in the Endpoint documentation tool. If anyone wants to take this on, feel free. Otherwise, I will get to it sometime in the coming days since I believe this should be a priority. |
Yes, I think this is important. As of now is there anyway to find out what parameter names we can pass in? I'm getting unexpected arguments errors when I do this: I know 'player_id' is right but I'm just guessing on the others. |
Ok I found them in the files, but it would be nice to have them in the docs. Also, I don't understand why "Season" is season_all and SeasonType is "season_type_all_star"? for PlayerGameLog.py |
@mchauta https://github.com/swar/nba_api/blob/master/docs/nba_api/stats/library/parameters.md |
Also with the holiday season here, I am hoping to find a good break to catch up with some of these issues and knock a few of them out. So this is still pending and will be added. |
Changes have been implemented as of v1.0.7 I made more changes to the docs to make it a bit easier to read, specifically the endpoint page. Changes include:
Let me know your thoughts and changes you want to see. Definitely open to adding information and changing formatting if needed. |
Adding the actual variable name to the url name in the documentation of the different endpoints would facilitate the usage of the package.
The text was updated successfully, but these errors were encountered: