fix: asyncio import, removed model revision pinning and table grouping arg
🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!
-- conventional-commit-lint bot https://conventionalcommits.org/
Check out this pull request on ![]()
See visual diffs & provide feedback on Jupyter Notebooks.
Powered by ReviewNB
@polong-lin @alokpattani Seems owlbot post processor keeps eating the asyncio import line
@polong-lin @alokpattani Seems owlbot post processor keeps eating the asyncio import line
Autoflake is removing the import because it's not being used in the notebook. Why is asyncio needed?
@polong-lin @alokpattani Seems owlbot post processor keeps eating the asyncio import line
Autoflake is removing the import because it's not being used in the notebook. Why is asyncio needed?
@holtskinner the second half of the notebook is about async execution to process multiple videos. It's a problem with the linter not understanding the import is necessary to execute L1012 multiple_video_extraction_responses = await asyncio.gather(*get_responses)
@polong-lin @alokpattani Seems owlbot post processor keeps eating the asyncio import line
Autoflake is removing the import because it's not being used in the notebook. Why is asyncio needed?
@holtskinner the second half of the notebook is about async execution to process multiple videos. It's a problem with the linter not understanding the import is necessary to execute L1012
multiple_video_extraction_responses = await asyncio.gather(*get_responses)
Thanks for that context, I missed the usage. Not sure what's going on here. I'll do some experimentation