Skip to content

[Feature] Structural Refactor for Execute#21

Open
MJC598 wants to merge 17 commits into
mainfrom
feature/matching-new-respond-api
Open

[Feature] Structural Refactor for Execute#21
MJC598 wants to merge 17 commits into
mainfrom
feature/matching-new-respond-api

Conversation

@MJC598

@MJC598 MJC598 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Quite a lot:

  • Restructured package layout to contain only one subpackage (respondpy.data)
  • Broke out exposed components in __init__.py more effectively to manage imports from language bindings
  • Added new handling for incomplete transition matrices and state vectors
  • Redid testing for new layout
  • Implemented a new Input class wrapper to help support SQLite and Config files
  • Implemented a wrapper Parameter class around the enum to better reduce redundant strings
  • Implemented better memory caching to avoid repeated I/O from the database
  • Bumped version to next minor patch (0.1.3 -> 0.2.0)

What Wrike task is this associated with?

Checklist before merging

  • If adding a core feature, I've added related tests.
  • This is part of a product update, and I've added an explanation of what is different to the changelog.

@MJC598 MJC598 requested review from ddbaptiste and ryondas June 10, 2026 21:27
@MJC598 MJC598 self-assigned this Jun 10, 2026
@MJC598 MJC598 added the enhancement New feature or request label Jun 10, 2026
@codecov

codecov Bot commented Jun 10, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 97.73585% with 12 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/respondpy/data/transition_matrices.py 90.74% 10 Missing ⚠️
src/respondpy/data/input.py 98.83% 2 Missing ⚠️
Files with missing lines Coverage Δ
src/respondpy/__init__.py 100.00% <100.00%> (+6.25%) ⬆️
src/respondpy/_utils.py 100.00% <100.00%> (ø)
src/respondpy/cost_effectiveness.py 100.00% <100.00%> (ø)
src/respondpy/data/__init__.py 100.00% <100.00%> (ø)
src/respondpy/data/database_helpers.py 100.00% <100.00%> (ø)
src/respondpy/data/logic_conditions.py 100.00% <100.00%> (ø)
src/respondpy/data/parameters.py 100.00% <100.00%> (ø)
src/respondpy/data/state_vectors.py 100.00% <100.00%> (ø)
src/respondpy/history.py 100.00% <100.00%> (ø)
src/respondpy/model.py 100.00% <100.00%> (ø)
... and 4 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant