Skip to content

Comments

Allow user to do one of the split/preprocess steps#88

Merged
charlotte-avery merged 2 commits intodevelopfrom
separate-preprocess-steps
Jan 15, 2026
Merged

Allow user to do one of the split/preprocess steps#88
charlotte-avery merged 2 commits intodevelopfrom
separate-preprocess-steps

Conversation

@charlotte-avery
Copy link
Collaborator

@charlotte-avery charlotte-avery commented Dec 19, 2025

  • Separated out the split and preprocess data preparation steps
  • Added licence headers to scripts created in the previous PR
  • Fixed line length of doc strings

@charlotte-avery charlotte-avery force-pushed the separate-preprocess-steps branch from 0d7982d to 17c9ad8 Compare December 19, 2025 09:26
Signed-off-by: Charlotte Avery <charlotte.avery@centrefornetzero.org>
Signed-off-by: Charlotte Avery <charlotte.avery@centrefornetzero.org>
@charlotte-avery charlotte-avery force-pushed the separate-preprocess-steps branch from 4897d70 to 958bfdb Compare January 15, 2026 15:23
@charlotte-avery charlotte-avery marked this pull request as ready for review January 15, 2026 15:25
Copy link
Collaborator

@Ianlmgoddard Ianlmgoddard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@charlotte-avery charlotte-avery merged commit 7da3e2c into develop Jan 15, 2026
3 of 4 checks passed
@charlotte-avery charlotte-avery deleted the separate-preprocess-steps branch January 15, 2026 16:11
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.

2 participants