Skip to content

Conversation

@TomNicholas
Copy link
Member

Addresses @elyall's comment in #1463 (comment) by updating the icechunk dependency URL in the bug report template.

Update the icechunk dependency URL in the bug report template.
@ianhi
Copy link
Collaborator

ianhi commented Dec 10, 2025

Can we just point to the nightly wheel instead? I think that would be a better user experience. With something like this:
https://docs.astral.sh/uv/concepts/projects/dependencies/#index

it may end up being uv specific and fall back to pypi for other script runners. but that's fine with me for the convenience of not needing to compile icechunk to make a bug report.

Same for zarr I guess, though that's less important

@ianhi
Copy link
Collaborator

ianhi commented Dec 10, 2025

This is the magic set of settings:

        # /// script
        # requires-python = ">=3.11"
        # dependencies = [
        #   "icechunk",
        #   "zarr",
        # ]
        #
        # [[tool.uv.index]]
        # name = "scientific-python-nightly-wheels"
        # url = "https://pypi.anaconda.org/scientific-python-nightly-wheels/simple/"
        #
        # [tool.uv.sources]
        # icechunk = { index = "scientific-python-nightly-wheels" }
        # zarr = { index = "scientific-python-nightly-wheels" }
        #
        # [tool.uv]
        # prerelease = "allow"
        # ///

which does grab icechunk 2.0.0a. So we might want to constrain icechunk to be <2 for now

@TomNicholas
Copy link
Member Author

Looks like you finished this @ianhi ? (Thank you!) I can't self-approve, so if you want to approve it then either of us can merge it. (Or maybe @paraseba can just give me god privileges)

Co-authored-by: Ian Hunt-Isaak <[email protected]>
@ianhi ianhi merged commit adf1034 into main Dec 15, 2025
17 checks passed
@ianhi ianhi deleted the fix-ic-url-bug-report-template branch December 15, 2025 15:49
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.

3 participants