Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Declare ngram variable in select_ngrams #3959

Closed
wants to merge 3 commits into from
Closed

Conversation

JonoYang
Copy link
Member

  • This is to avoid an undeclared variable issue when running the last part of the function

Signed-off-by: Jono Yang jyang@nexb.com

    * This is to avoid an undeclared variable issue when running the last part of the function

Signed-off-by: Jono Yang <jyang@nexb.com>
Signed-off-by: Jono Yang <jyang@nexb.com>
@JonoYang JonoYang force-pushed the update-select_ngrams branch 2 times, most recently from 3152dc7 to 55a75f5 Compare October 24, 2024 00:47
    * Support Python 3.13.0

Signed-off-by: Jono Yang <jyang@nexb.com>
@JonoYang
Copy link
Member Author

JonoYang commented Nov 7, 2024

Closing this in favor of another pr

@JonoYang JonoYang closed this Nov 7, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant