Skip to content

Resolve BNG2.pl from $PATH via $BNGPATH env var #72

@wshlavacek

Description

@wshlavacek

Location: bionetgen/core/utils/utils.py:651

Original marker:

# TODO: Figure out how to use the BNG2.pl if it's set
# in the PATH variable. Solution: set os.environ BNGPATH
# and make everything use that route

test_bngexec resolves BNG2.pl from explicit candidate paths today. Users who already manage BNG2.pl via $PATH (the conventional Unix install pattern) still have to set the path explicitly.

What would unblock this: the original author already proposed the path forward — read $BNGPATH from os.environ, and route all BNG2.pl invocations through that. Concretely, add a fallback chain to the resolver: explicit candidate → $BNGPATHshutil.which("BNG2.pl") → error.

Discovered during: non-Atomizer TODO/FIXME triage sweep, 2026-05-08.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions