Skip to content

feedsim: reuse existing conda install when available#748

Open
NikhilTarte-Arm wants to merge 1 commit into
facebookresearch:v2-betafrom
NikhilTarte-Arm:feedsim-conda-change
Open

feedsim: reuse existing conda install when available#748
NikhilTarte-Arm wants to merge 1 commit into
facebookresearch:v2-betafrom
NikhilTarte-Arm:feedsim-conda-change

Conversation

@NikhilTarte-Arm

Copy link
Copy Markdown

The Ubuntu aarch64 FeedSim installer always attempted to download and bootstrap Miniconda before installing the CPU-only torch wheel used to provide LibTorch.

Reuse an existing conda installation when one is already available by creating the benchmark-local conda prefix from that conda instead of always downloading Miniconda. Keep the existing Miniconda download path as the fallback when conda is not present.

This preserves the local benchmark environment layout, avoids unnecessary Miniconda bootstrapping when conda is already provisioned, and only accepts Anaconda TOS when the installer actually bootstraps Miniconda itself.

The Ubuntu aarch64 FeedSim installer always attempted to download and bootstrap Miniconda before installing the CPU-only torch wheel used to provide LibTorch.

Reuse an existing conda installation when one is already available by creating the benchmark-local conda prefix from that conda instead of always downloading Miniconda. Keep the existing Miniconda download path as the fallback when conda is not present.

This preserves the local benchmark environment layout, avoids unnecessary Miniconda bootstrapping when conda is already provisioned, and only accepts Anaconda TOS when the installer actually bootstraps Miniconda itself.
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant