Skip to content

Move latent-lens into a diffae extra - #23

Open
ahillsley wants to merge 1 commit into
mainfrom
hillsleya/latent-lens-extra
Open

Move latent-lens into a diffae extra#23
ahillsley wants to merge 1 commit into
mainfrom
hillsleya/latent-lens-extra

Conversation

@ahillsley

Copy link
Copy Markdown
Collaborator

czi-ai/latent-lens is a private repo (visibility: internal) and not on PyPI, so as a default dependency it broke uv sync for anyone without access. Its only consumers are the diffae interpretability viewer scripts, so it moves to a diffae extra here (a PEP 508 direct reference, so no [tool.uv.sources] needed), and build_umap_montage's top-level import becomes lazy like the three others in that directory. Paired with czbiohub-sf/ops_monorepo#26, which drops it from the workspace root — merge this one first.

czi-ai/latent-lens is a private repo (visibility: internal) and not on PyPI,
so as a default dependency it broke `uv sync` for anyone without access to it.
Its only consumers are the diffae interpretability viewer/figure scripts.

Declared as a PEP 508 direct reference so it needs no [tool.uv.sources]
(allow-direct-references is already set), and build_umap_montage's top-level
import is now lazy inside montage_from_cache, matching the three other
latent_lens imports in that directory. Install with `uv sync --extra diffae`
or `pip install ops_model[diffae]`.

Paired with czbiohub-sf/ops_monorepo#(pending), which drops it from the
workspace root.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.

1 participant