run_full_lvs sources set_lvs_env.py which is absent from the package and repo (v1.3.1)

[dmjdxb]

cf-precheck 1.3.1 (pip install cf_precheck==1.3.1): the LVS check cannot run because be_checks/run_full_lvs (line ~40) does:

source <($LVS_ROOT/set_lvs_env.py -c $CONFIG_FILE $DESIGN_NAME)

but set_lvs_env.py is not shipped. find locates it neither in the installed site-packages/cf_precheck/be_checks/ nor in a fresh git clone --depth 1 of this repo.

Result:

run_full_lvs: line 40: .../be_checks/set_lvs_env.py: No such file or directory
ERROR: LVS environment problem

so the standalone lvs (and dependent oeb) checks never reach netgen — they fail on environment setup.

Reproduced on both a Linux pip install and a macOS install (both v1.3.1).

Could you either ship set_lvs_env.py in the package/repo, or document that the lvs gate requires the Caravel/Openframe MPW harness to provide it? Thanks.

[dmjdxb]

Root cause (traced across refs)

set_lvs_env.py is referenced from three call sites, all resolving to $LVS_ROOT = checks/be_checks/:

• checks/be_checks/run_full_lvs:40 — source <($LVS_ROOT/set_lvs_env.py -c $CONFIG_FILE $DESIGN_NAME)
• checks/be_checks/run_be_checks:61 — same
• checks/lvs_check/lvs.py → checks/utils/utils.py:204 — be_env['LVS_ROOT'] = f'{os.getcwd()}/checks/be_checks/'

But checks/be_checks/set_lvs_env.py is absent from the repo at every ref — main and all mpw-9x tags including mpw-9c (which the efabless/caravel_openframe_project template pins via MPW_TAG). raw.githubusercontent.com/.../<ref>/checks/be_checks/set_lvs_env.py → HTTP 404 for main, mpw-9c, mpw-9, mpw-8.

Consequence: the entire back-end LVS path (run_be_checks → run_full_lvs, and lvs_check/lvs.py) aborts at run_full_lvs:42 (if [[ ! -v EXTRACT_FLATGLOB ]]; then echo "ERROR: LVS environment problem."). Since the Docker image only provides tools and the scripts are mounted from the clone, make run-precheck hits the same gap — it is not a pip-packaging-only issue.

Workaround

set_lvs_env.py just translates the lvs_config JSON (resolving INCLUDE_CONFIGS + $VAR) into export KEY=VALUE lines for run_full_lvs to source. A faithful reconstruction restores the flow. Happy to open a PR re-adding it if that's useful.