Git Product home page Git Product logo

Comments (14)

anentropic avatar anentropic commented on May 23, 2024 1

for kicks I pasted in the traceback above and asked the new Llama 3 for a solution...

it actually gave a nice explanation and something that worked (!)

amending the update content script to:

POETRY_DYNAMIC_VERSIONING=false python3 -m pip install -e '.[clickhouse,duckdb,clickhouse,examples,geospatial]'

got around the dunamai issue

So the two parts of the fix are probably:

  1. add a Dockerfile under .devcontainer something like:
FROM mcr.microsoft.com/vscode/devcontainers/python:3.11

RUN apt-get update && apt-get install -y --no-install-recommends libgdal-dev
  1. prepend POETRY_DYNAMIC_VERSIONING=false to the pip install -e

from ibis.

ncclementi avatar ncclementi commented on May 23, 2024

@anentropic Thanks for reporting this.

I'm not quite sure what's going on, because the geospatial dependencies were added recently, see #8837 and it looks like you are running on main.

@cpcloud any idea what could be going on here?

from ibis.

cpcloud avatar cpcloud commented on May 23, 2024

A local vs code environment a codespace are not the same. If a package attempting to call gdal-config that means it's being built from source, which means the wheel for whatever platform + python version you're using doesn't exist.

from ibis.

cpcloud avatar cpcloud commented on May 23, 2024

I don't think it's a local versus remote issue.

I also don't really understand how this works. Does the codespace run on GitHub infrastructure and VS code just "points" at that?

from ibis.

cpcloud avatar cpcloud commented on May 23, 2024

I don't think anyone's used the devcontainer setup locally, but it works on GitHub codespaces. I use it from time to time to try things that I don't want to run locally.

from ibis.

anentropic avatar anentropic commented on May 23, 2024

I believe it's running a docker container locally on my machine

from the terminal I get:

vscode ➜ /workspaces/ibis (main) $ arch
aarch64
vscode ➜ /workspaces/ibis (main) $ uname -a
Linux b66611250c8b 6.6.22-linuxkit #1 SMP Fri Mar 29 12:21:27 UTC 2024 aarch64 GNU/Linux

which makes sense as I'm on an M2 macbook

from ibis.

cpcloud avatar cpcloud commented on May 23, 2024

RuntimeError: Detected Git repository, but failed because of dubious ownership

Not entirely sure whether this will be possible to deal with. Speculating here, but I think this is probably related to how volumes are being mounted inside the container.

from ibis.

cpcloud avatar cpcloud commented on May 23, 2024

That error is also happening in dunamai which is a dependency of many many things.

I think we probably need someone who develops using a Mac to debug this

from ibis.

gforsyth avatar gforsyth commented on May 23, 2024

Hey @anentropic -- want to give it another shot now and see if it works?

from ibis.

amoeba avatar amoeba commented on May 23, 2024

Hi @gforsyth, I had to make a few tweaks to get the devcontainer to run and those changes are in #9019.

from ibis.

anentropic avatar anentropic commented on May 23, 2024

Hey @anentropic -- want to give it another shot now and see if it works?

Not working yet but looks like #9019 will fix it (needs the change in devcontainer.json to actually use the Dockerfile)

from ibis.

cpcloud avatar cpcloud commented on May 23, 2024

Just merged that, give it another go if you can!

from ibis.

anentropic avatar anentropic commented on May 23, 2024

Apologies, but it seems POETRY_DYNAMIC_VERSIONING=false was a hallucination by the LLM

I tried it and it appeared to work (i.e. by entering that line manually in terminal of a failed container launch)

but it doesn't work when (re)building the container from scratch ...I tried again just now and same thing. (And I can't find evidence for it via Google or GitHub search)

I'm not sure what devcontainers is doing to make the two scenarios behave differently

According to poetry-dynamic-versioning plugin docs a correct env var to disable active versioning would be POETRY_DYNAMIC_VERSIONING_BYPASS='8.0.0' i.e. specifying a static version value

I tried this and took care to make devcontainers rebuild the image with it and it seems to work

from ibis.

cpcloud avatar cpcloud commented on May 23, 2024

PR to fix the hallucination: #9031

from ibis.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.