Skip to content

Interface + base class structure for data module#803

Merged
ndem0 merged 7 commits into
mathLab:0.3from
GiovanniCanali:data_structure
May 26, 2026
Merged

Interface + base class structure for data module#803
ndem0 merged 7 commits into
mathLab:0.3from
GiovanniCanali:data_structure

Conversation

@GiovanniCanali

@GiovanniCanali GiovanniCanali commented May 25, 2026

Copy link
Copy Markdown
Collaborator

This PR fixes #798

In this PR, the following updates for the data module have been introduced:

  • Removed deprecated dataset.py
  • Enhanced and extended documentation to improve clarity and usability.
  • Enhanced test suite to improve coverage.

The core logic underpinning the entire data module has remained unchanged from a user perspective.

@GiovanniCanali GiovanniCanali self-assigned this May 25, 2026
@GiovanniCanali GiovanniCanali added enhancement New feature or request pr-to-fix Label for PR that needs modification 0.3 labels May 25, 2026
@GiovanniCanali
GiovanniCanali force-pushed the data_structure branch 3 times, most recently from fccbcfe to 800fefe Compare May 25, 2026 13:03
@GiovanniCanali
GiovanniCanali force-pushed the data_structure branch 4 times, most recently from 7d256af to a7227b8 Compare May 26, 2026 08:23
@GiovanniCanali
GiovanniCanali marked this pull request as ready for review May 26, 2026 08:31
@GiovanniCanali
GiovanniCanali requested review from a team and dario-coscia as code owners May 26, 2026 08:31
@GiovanniCanali GiovanniCanali added pr-to-review Label for PR that are ready to been reviewed and removed pr-to-fix Label for PR that needs modification labels May 26, 2026
@ndem0
ndem0 merged commit 990030d into mathLab:0.3 May 26, 2026
8 of 19 checks passed
@GiovanniCanali
GiovanniCanali deleted the data_structure branch May 26, 2026 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request pr-to-review Label for PR that are ready to been reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants