Skip to content

Conversation

Alvinleenh
Copy link

Updated syntax change:
sp.import_script_from_url -> sp.io.import_script_from_url
sp.import_template -> sp.io.import_template
sp.bytes_of_string -> sp.utils.bytes_of_string

Fixed typo:
assume_consecutive_tokens -> assume_consecutive_token_ids

Please refer to issue #358

Updated syntax change:
sp.import_script_from_url -> sp.io.import_script_from_url
sp.import_template -> sp.io.import_template
sp.bytes_of_string -> sp.utils.bytes_of_string

Fixed typo:
assume_consecutive_tokens -> assume_consecutive_token_ids
@Alvinleenh
Copy link
Author

Hi @bhaskarSingh, I would like to request for review on this change. Thanks

@Alvinleenh Alvinleenh marked this pull request as draft November 27, 2021 17:19
@Alvinleenh Alvinleenh marked this pull request as ready for review November 27, 2021 17:20
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.

1 participant