Skip to content

Commit

Permalink
comment/sorting fix, and uv for learntools
Browse files Browse the repository at this point in the history
  • Loading branch information
djherbis committed Dec 2, 2024
1 parent aa56836 commit e27d6ee
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
7 changes: 4 additions & 3 deletions Dockerfile.tmpl
Original file line number Diff line number Diff line change
Expand Up @@ -21,10 +21,11 @@ ENV PATH="~/.local/bin:${PATH}"
# b/183041606#comment5: the Kaggle data proxy doesn't support these APIs. If the library is missing, it falls back to using a regular BigQuery query to fetch data.
RUN uv pip uninstall --system google-cloud-bigquery-storage

# NOTE(herbison): uv fails to install this for some reason
RUN pip install git+https://github.com/Kaggle/learntools
# uv cannot install this in requirements.txt without --no-build-isolation
# to avoid affecting the larger build, we'll post-install it.
RUN uv pip install --no-build-isolation --system "git+https://github.com/Kaggle/learntools"

# We install an incompatible pair of libs (shapely<, libpysal==4.9.2) so we can't put this one in the requirements.txt
# b/328788268 We install an incompatible pair of libs (shapely<2, libpysal==4.9.2) so we can't put this one in the requirements.txt
RUN uv pip install --system "libpysal==4.9.2"

# Adding non-package dependencies:
Expand Down
5 changes: 3 additions & 2 deletions kaggle_requirements.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
altair>=5.4.0
# Please keep this in alphabetical order
Altair>=5.4.0
Babel
Boruta
Cartopy
Expand All @@ -11,7 +12,7 @@ Rtree
shapely<2
SimpleITK
# b/302136621: Fix eli5 import for learntools, newer version require scikit-learn > 1.3
tpot==0.12.1
TPOT==0.12.1
Theano
Wand
annoy
Expand Down

0 comments on commit e27d6ee

Please sign in to comment.