Skip to content

v2.0.1 - #153

Merged
thusser merged 4 commits into
mainfrom
develop
Aug 28, 2026
Merged

v2.0.1#153
thusser merged 4 commits into
mainfrom
develop

Conversation

@thusser

@thusser thusser commented Aug 28, 2026

Copy link
Copy Markdown
Member

version 2.0.1

dependabot Bot and others added 4 commits August 26, 2026 11:52
Bumps [pyside6](https://pyside.org) from 6.11.1 to 6.11.2.

---
updated-dependencies:
- dependency-name: pyside6
  dependency-version: 6.11.2
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
chore(deps): bump pyside6 from 6.11.1 to 6.11.2
…151) (#152)

The wrapper installed by patch_generic_unit_parser() adds one stack frame,
shifting astropy's frame-level bookkeeping: astropy.utils.parsing calls
get_caller_module_dict(2) directly and its _patch_ply_module wrapper adds two
more levels, so the extra frame made the walk land on astropy's parsing wrapper
instead of the grammar-defining frame. The fallback then fired on plain CPython
and corrupted unit/coordinate parsing when running from source.

- Only install the patch in the Nuitka-compiled standalone binary
  ('__compiled__' in globals()), evaluated in pyobs_gui/__init__.py which is
  guaranteed compiled; otherwise it is a strict no-op.
- Step one level deeper (levels + 1) in the patched walks to restore the
  original frame targeting, so the fallback only fires when the walk genuinely
  lacks the grammar (compiled frames without f_locals).
- Register the vendored grammar namespace in sys.modules and materialize its
  __file__ stub so PLY's inspect.getmodule()/getsourcefile() validation works
  when the grammar has to be rebuilt.
- Scope the fallback to astropy.units.format.generic only: angle/cds/ogip
  grammars are not vendored, so feed the broken-locals pdict back unchanged
  instead of silently parsing with the wrong grammar.
@thusser
thusser merged commit 24f9533 into main Aug 28, 2026
2 checks passed
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