Skip to content

refactor(bridge): extract scitex.bridge into standalone scitex-bridge package#248

Open
ywatanabe1989 wants to merge 1 commit intodevelopfrom
feat/extract-scitex-bridge
Open

refactor(bridge): extract scitex.bridge into standalone scitex-bridge package#248
ywatanabe1989 wants to merge 1 commit intodevelopfrom
feat/extract-scitex-bridge

Conversation

@ywatanabe1989
Copy link
Copy Markdown
Owner

Summary

Extracts `scitex.bridge` into scitex-bridge v0.1.0 (1,941 LOC).

  • Cross-module adapters (stats↔plt / stats↔vis / plt↔vis) preserved.
  • `[bridge]` extra → `scitex-bridge>=0.1.0`.
  • 32/51 tests pass (same passing count as upstream — 8 are xfailed in upstream because of unrelated scitex.schema deprecation + scitex.stats._utils missing).

Decoupling

  • `scitex.io.bundle.kinds._stats.Position` → tiny vendored dataclass; real Position preferred when scitex is installed.
  • Optional `scitex.io.bundle.kinds._plot._models` imports stay inside `try/except` (matching upstream).

🤖 Generated with Claude Code

… package

Cross-module adapters (stats↔plt / stats↔vis / plt↔vis) now live in the
standalone scitex-bridge package
(https://github.com/ywatanabe1989/scitex-bridge).

scitex.bridge/__init__.py becomes a sys.modules alias.
The [bridge] extra is updated to depend on scitex-bridge>=0.1.0.

Decoupling notes (handled in scitex-bridge's initial commit):
- scitex.io.bundle.kinds._stats.Position → vendored as a tiny dataclass.
  The real Position is preferred when scitex (umbrella) is installed so
  produced objects round-trip cleanly through bundle code paths.
- scitex.io.bundle.kinds._plot._models imports remain inside try/except.

32/51 tests pass (same as upstream — 8 are xfailed in upstream because of
unrelated scitex.schema deprecation + scitex.stats._utils missing; both are
pre-existing umbrella issues).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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