Skip to content

Refactored datapoints load#359

Merged
javihern98 merged 12 commits intomainfrom
cr-350
Nov 18, 2025
Merged

Refactored datapoints load#359
javihern98 merged 12 commits intomainfrom
cr-350

Conversation

@albertohernandez1995
Copy link
Contributor

Major changes:

  • Refactored the way datapoints are load from Paths.
  • Refactored typing of load datapoints function.
  • Implemented validate dataset for S3 URIs
  • Added tests to ensure validate_dataset method works as intended.

Closes #350

Copy link
Contributor

@javihern98 javihern98 left a comment

Choose a reason for hiding this comment

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

Looks good, thanks! 😊

@javihern98 javihern98 merged commit 0071f7b into main Nov 18, 2025
20 checks passed
@javihern98 javihern98 deleted the cr-350 branch November 18, 2025 12:46
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.

Review validate_dataset function to fix datapoints Path handling

2 participants