Skip to content

refactor(math): move impl-flag resolver to polismath/utils/env_flags.py#2641

Draft
jucor wants to merge 1 commit into
spr/edge/18c7048ffrom
spr/edge/bc0518d7
Draft

refactor(math): move impl-flag resolver to polismath/utils/env_flags.py#2641
jucor wants to merge 1 commit into
spr/edge/18c7048ffrom
spr/edge/bc0518d7

Conversation

@jucor

@jucor jucor commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Move the generic legacy-vs-improved env-switch resolver out of pca.py into
polismath/utils/env_flags.py (public name resolve_impl_flag). pca.py and
utils/engine_mode.py both import it from there, so reading
POLISMATH_ENGINE_MODE no longer drags the numpy/pandas pca import chain,
and resolution warnings log under polismath.utils.env_flags instead of the
pca logger. Resolution rules unchanged (strip+lowercase, unknown -> default
with warning, read at call time). tests/test_env_flags.py pins the rules,
the shared-resolver identity, and the light engine_mode import (RED observed
on ModuleNotFoundError before the move; the light-import test fails on the
old code by construction).

commit-id:bc0518d7


Stack:


⚠️ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

Move the generic legacy-vs-improved env-switch resolver out of pca.py into
polismath/utils/env_flags.py (public name resolve_impl_flag). pca.py and
utils/engine_mode.py both import it from there, so reading
POLISMATH_ENGINE_MODE no longer drags the numpy/pandas pca import chain,
and resolution warnings log under polismath.utils.env_flags instead of the
pca logger. Resolution rules unchanged (strip+lowercase, unknown -> default
with warning, read at call time). tests/test_env_flags.py pins the rules,
the shared-resolver identity, and the light engine_mode import (RED observed
on ModuleNotFoundError before the move; the light-import test fails on the
old code by construction).

commit-id:bc0518d7
This was referenced Jul 22, 2026
@jucor
jucor marked this pull request as draft July 22, 2026 00:51
@jucor
jucor changed the base branch from spr/edge/18c7048f to edge July 22, 2026 02:01
@jucor
jucor changed the base branch from edge to spr/edge/18c7048f July 22, 2026 02:02
@jucor
jucor changed the base branch from spr/edge/18c7048f to edge July 22, 2026 02:10
@jucor
jucor changed the base branch from edge to spr/edge/18c7048f July 22, 2026 02:10
@jucor
jucor changed the base branch from spr/edge/18c7048f to edge July 22, 2026 03:54
@jucor
jucor changed the base branch from edge to spr/edge/18c7048f July 22, 2026 03:54
@jucor
jucor changed the base branch from spr/edge/18c7048f to edge July 22, 2026 06:16
@jucor
jucor changed the base branch from edge to spr/edge/18c7048f July 22, 2026 06:16
@jucor
jucor changed the base branch from spr/edge/18c7048f to edge July 22, 2026 06:50
@jucor
jucor changed the base branch from edge to spr/edge/18c7048f July 22, 2026 06:50
@jucor
jucor changed the base branch from spr/edge/18c7048f to edge July 22, 2026 08:11
@jucor
jucor changed the base branch from edge to spr/edge/18c7048f July 22, 2026 08:11
@jucor
jucor changed the base branch from spr/edge/18c7048f to edge July 22, 2026 09:58
@jucor
jucor changed the base branch from edge to spr/edge/18c7048f July 22, 2026 09:58
@jucor
jucor changed the base branch from spr/edge/18c7048f to edge July 22, 2026 11:21
@jucor
jucor changed the base branch from edge to spr/edge/18c7048f July 22, 2026 11:21
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.

1 participant