Skip to content

This pulls in the latest data_readers (and related code) to allow#2515

Open
clessig wants to merge 2 commits into
develop-sslfrom
clessig/develop-ssl/update_data_reading
Open

This pulls in the latest data_readers (and related code) to allow#2515
clessig wants to merge 2 commits into
develop-sslfrom
clessig/develop-ssl/update_data_reading

Conversation

@clessig

@clessig clessig commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Description

This pulls in the latest data_readers (and related code) to allow training with operan offset and IASI-PCs

Issue Number

Checklist before asking for review

  • I have performed a self-review of my code
  • My changes comply with basic sanity checks:
    • I have fixed formatting issues with ./scripts/actions.sh lint
    • I have run unit tests with ./scripts/actions.sh unit-test
    • I have documented my code and I have updated the docstrings.
    • I have added unit tests, if relevant
  • I have tried my changes with data and code:
    • I have run the integration tests with ./scripts/actions.sh integration-test
    • (bigger changes) I have run a full training and I have written in the comment the run_id(s): launch-slurm.py --time 60
    • (bigger changes and experiments) I have shared a hegdedoc in the github issue with all the configurations and runs for this experiments
  • I have informed and aligned with people impacted by my change:
    • for config changes: the MatterMost channels and/or a design doc
    • for changes of dependencies: the MatterMost software development channel

@clessig

clessig commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator Author

@shmh40 @sophie-xhonneux : not sure where you have the operan offset but this PR updates the data reading to be consistent with develop

@github-actions github-actions Bot added the data Anything related to the datasets used in the project label Jun 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data Anything related to the datasets used in the project

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant