Skip to content

Update checklist for adding training data#377

Merged
Damonamajor merged 11 commits intomasterfrom
update-checklist-for-adding-input-data
May 30, 2025
Merged

Update checklist for adding training data#377
Damonamajor merged 11 commits intomasterfrom
update-checklist-for-adding-input-data

Conversation

@Damonamajor
Copy link
Contributor

@Damonamajor Damonamajor commented May 5, 2025

Helper PR for the following issue in data architecture to update the finalize model checklist.

@Damonamajor Damonamajor changed the title Update checklist for adding input data Update checklist for adding training data May 5, 2025
Copy link
Member

@jeancochrane jeancochrane left a comment

Choose a reason for hiding this comment

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

Thanks for this chore! Just one small nitpick. Let's wait to merge until ccao-data/data-architecture#804 comes in, in case we end up changing anything there that we need to reflect here.

Co-authored-by: Jean Cochrane <jeancochrane@users.noreply.github.com>
@jeancochrane
Copy link
Member

@Damonamajor Now that ccao-data/data-architecture#804 is ready to merge, can we bring this PR up to date with the new instructions?

@Damonamajor Damonamajor requested a review from jeancochrane May 30, 2025 14:49
Copy link
Member

@jeancochrane jeancochrane left a comment

Choose a reason for hiding this comment

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

Thanks for this!

@Damonamajor Damonamajor merged commit 9045e7c into master May 30, 2025
5 checks passed
@Damonamajor Damonamajor deleted the update-checklist-for-adding-input-data branch May 30, 2025 15:36
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