Skip to content

Conversation

@fcdl94
Copy link
Member

@fcdl94 fcdl94 commented Oct 3, 2025

remove __MACOSX directory and validate train/validation splits

  • Added functionality to remove the __MACOSX directory after extraction.
  • Implemented checks to ensure the presence of 'train' and 'validation' directories in the extracted content, raising errors if not found.

…ry and validating train/validation splits

- Added functionality to remove the __MACOSX directory after extraction.
- Implemented checks to ensure the presence of 'train' and 'validation' directories in the extracted content, raising errors if not found.
@github-actions
Copy link

github-actions bot commented Oct 3, 2025

Coverage

Tests Skipped Failures Errors Time
228 0 💤 0 ❌ 0 🔥 45.513s ⏱️

@fcdl94 fcdl94 changed the title feat(system): enhance archive extraction b feat(system): enhance archive extraction Oct 3, 2025
…directory validation

- Renamed the function `list_dir` to `list_directories` for clarity.
- Updated the extraction logic to use the new function name.
- Improved validation checks for 'train' and 'validation' directories by using `Path` objects for better consistency and readability.
@fcdl94 fcdl94 merged commit cfeebc7 into main Oct 3, 2025
10 checks passed
@fcdl94 fcdl94 deleted the fix/handle-macosx-in-zip-folder branch October 3, 2025 14:55
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.

1 participant