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

Add Detailed Information for game_id Parameter in parameters.md #457

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gmortensen
Copy link

This pull request adds detailed information for the game_id parameter in the parameters.md file. The game_id parameter is frequently used throughout the nba_api library, but it previously lacked detailed documentation. The following updates have been made:

Pattern for game_id:

  • Added regex pattern ^\d{10}$ to validate the format of game_id.

Explained the significance of the third digit in the game_id, which functions as a flag indicating the type of season:
1: Preseason game
2: Regular season game
3: All-Star weekend game
4: Playoff game
5: Play-in game

Included examples to illustrate the format and the significance of the third digit:

  • 0022200147: Regular season game
    
  • 0012200147: Preseason game
    

Version:
This update is for version 1.5.0.

Please let me know if there are any additional changes or improvements needed for this documentation update.

Thank you for your time and consideration in reviewing this PR.

@rsforbes
Copy link
Collaborator

@gmortensen - Apologies for not getting to this sooner.

The parameters.md file is auto-generated via generator.py. The generator then looks to mapping.py. If you're able to incorporate your changes there, I think you'll be set to go.

@rsforbes rsforbes self-assigned this Aug 27, 2024
@rsforbes rsforbes self-requested a review August 27, 2024 23:53
@rsforbes rsforbes marked this pull request as draft August 28, 2024 00:34
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.

2 participants