Skip to content

Refactor dataset_preparation directory using git mv to preserve file … - #125

Merged
chani0343 merged 4 commits into
mainfrom
refactor-dataset-preparation-with-git-mv
Nov 10, 2025
Merged

chani0343 merged 4 commits into
mainfrom
refactor-dataset-preparation-with-git-mv

Conversation

@chani0343

Copy link
Copy Markdown
Collaborator

Summary

Refactored dataset_preparation/ directory into organized dataset/ structure as per issue #105.

Changes

  • Renamed dataset_preparation/dataset/ with logical subdirectories
  • Moved classification_dataset.zipdataset/archives/
  • Moved src/optimal_class_mapping.pydataset/
  • Updated imports in evaluation scripts
  • Updated documentation and .gitignore

Testing

  • All scripts maintain identical functionality
  • Imports updated and verified
  • Documentation reflects new structure

…history

- Moved dataset_preparation/ to dataset/preparation/ using git mv
- Moved src/optimal_class_mapping.py to dataset/ using git mv
- Moved utilities/ to dataset/utilities/ using git mv
- Updated imports in evaluation scripts to use new paths
- All file history preserved as requested by @lyuzinmaxim

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

please add a newline

@lyuzinmaxim lyuzinmaxim left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

please run linting (isort, flake8 and black)

- Applied black formatting with line-length 119
- Applied isort with black profile
- Only modified files related to dataset preparation task

Addresses reviewer feedback
@chani0343
chani0343 force-pushed the refactor-dataset-preparation-with-git-mv branch from b35e15f to 0158a5f Compare November 10, 2025 15:01
Remove .gitkeep complexity and make dataset exclusions consistent with other entries
@chani0343
chani0343 merged commit a919005 into main Nov 10, 2025
1 check failed
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