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

Switch to Colab's public image as the base. #1434

Closed
wants to merge 1 commit into from

Conversation

metrizable
Copy link

A first pass of using Colab's public image as the base image. This is a rough, minimalist/brute force, first attempt at making the switch. None of the libraries provided by Colab's are used. A few things of note:

  • Installing miniconda was needed as Colab doesn't provide it. This likely can be avoided to just using micromamba with a few tweaks.
  • A pip warning for BigQuery_Helper was resolved by passing in --use-pep517
  • Some hackery with nltk was performed, which with a little effort, should be resolved. It was likely an artifact of an earlier iteration
  • The entrypoint needed to be overridden, as Colab's image specifies a run script entrypoint, which clobbers testing

A few other rough edges and improvements should be made.

@djherbis
Copy link
Contributor

djherbis commented Jan 3, 2025

Thanks again for the initial PR for this, now that we're Colab based I'll close this one :)

@djherbis djherbis closed this Jan 3, 2025
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