Skip to content

update scar scvi-tools version to 1.4.3#11651

Merged
Zethson merged 4 commits into
masterfrom
feature/scvi_updates
May 15, 2026
Merged

update scar scvi-tools version to 1.4.3#11651
Zethson merged 4 commits into
masterfrom
feature/scvi_updates

Conversation

@Zethson
Copy link
Copy Markdown
Member

@Zethson Zethson commented May 15, 2026

Follow up to #11648

Also updates scvi-tools for scar.
Further sets required environment variables. Otherwise, scvi-tools:py3.13-cu12-1.4.3- crashes inside torch._dynamo initialization when run with -u : and the UID isn't in /etc/passwd.
PyTorch 2.7's inductor cache-dir helper calls getpass.getuser(), which falls through to pwd.getpwuid() and raises KeyError for any UID not present in the container's passwd file. This is the default Docker/Singularity behaviour on most institutional setups (LDAP/AD-assigned UIDs).

We set the

os.environ.setdefault("TORCHINDUCTOR_CACHE_DIR", os.path.join(os.getcwd(), "torch_cache"))

to fix this.

Signed-off-by: Lukas Heumos <lukas.heumos@posteo.net>
Zethson added 2 commits May 15, 2026 16:18
Signed-off-by: Lukas Heumos <lukas.heumos@posteo.net>
Signed-off-by: Lukas Heumos <lukas.heumos@posteo.net>
@Zethson Zethson marked this pull request as ready for review May 15, 2026 14:29
@Zethson Zethson requested a review from nictru May 15, 2026 14:30
@Zethson Zethson changed the title update scar scvi-tools version update scar scvi-tools version to 1.4.3 May 15, 2026
@nictru
Copy link
Copy Markdown
Contributor

nictru commented May 15, 2026

I think it would be more elegant to set TORCHINDUCTOR_CACHE_DIR so that we do not have to alter USER and HOME which could affect other things that we do not think about currently

@Zethson
Copy link
Copy Markdown
Member Author

Zethson commented May 15, 2026

I think it would be more elegant to set TORCHINDUCTOR_CACHE_DIR so that we do not have to alter USER and HOME which could affect other things that we do not think about currently

Right, I was also worried about that but in the end thought "whatever". I'll change it.

Signed-off-by: Lukas Heumos <lukas.heumos@posteo.net>
@Zethson Zethson added this pull request to the merge queue May 15, 2026
Merged via the queue into master with commit 4b801f9 May 15, 2026
28 checks passed
@Zethson Zethson deleted the feature/scvi_updates branch May 15, 2026 14:53
manascripts pushed a commit to manascripts/modules that referenced this pull request May 21, 2026
* update scar scvi-tools version

Signed-off-by: Lukas Heumos <lukas.heumos@posteo.net>

* hash

Signed-off-by: Lukas Heumos <lukas.heumos@posteo.net>

* hash

Signed-off-by: Lukas Heumos <lukas.heumos@posteo.net>

* set torch cache

Signed-off-by: Lukas Heumos <lukas.heumos@posteo.net>

---------

Signed-off-by: Lukas Heumos <lukas.heumos@posteo.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants