Skip to content

use mol cache when tar archive missing#656

Open
tlitfin wants to merge 1 commit intojwohlwend:mainfrom
tlitfin:fix-cache
Open

use mol cache when tar archive missing#656
tlitfin wants to merge 1 commit intojwohlwend:mainfrom
tlitfin:fix-cache

Conversation

@tlitfin
Copy link

@tlitfin tlitfin commented Mar 13, 2026

The current cache expects to see the mol.tar archive rather than the mol directory. This means that the download is triggered even if the mol directory is in the cache.

This is a small change to check for either mol or mol.tar before re-initiating the download.

@volgin
Copy link

volgin commented Mar 21, 2026

boltz-community has a slightly more complete version of this fix — it checks for the extracted mols/ directory first, and only falls through to check for mols.tar if the directory is absent:

mols = cache / "mols"
tar_mols = cache / "mols.tar"
if not mols.exists():
    if not tar_mols.exists():
        # download mols.tar
    # extract mols.tar -> mols/

This handles both cases: mols/ already extracted (skip everything), and mols.tar present but not yet extracted (skip download, go straight to extraction). Might be worth adopting the same two-level check here.

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.

2 participants