-
Notifications
You must be signed in to change notification settings - Fork 2
V1.1.0 #9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
nttg8100
commented
Dec 1, 2025
- Release for paper Ampiciline
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
revised menuscript v1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR prepares version 1.1.0 for release, including code cleanup, build system migration, and reproducibility improvements. The changes include migrating from Poetry to Hatchling for package management, adding a seed parameter for reproducible experiments, removing unused code files, and applying code formatting improvements.
- Migrated build system from Poetry to Hatchling with Pixi workspace configuration
- Added seed parameter to model pipeline for reproducibility
- Removed unused files (partition.py, formula.py, pixi.toml) and improved code formatting
Reviewed changes
Copilot reviewed 10 out of 12 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| smartsensor/process/normalize.py | Fixed type conversion bug by adding .astype(np.float64) and reformatted long lines |
| smartsensor/model/split_data.py | Removed unused variable in loop, updated docstring, and improved comment formatting |
| smartsensor/model/partition.py | Deleted unused train_test_split_by_conv function |
| smartsensor/model/metric.py | Updated type annotation to modern syntax and reformatted long lines |
| smartsensor/model/formula.py | Deleted unused custom_predict function |
| smartsensor/main.py | Added seed parameter for reproducibility and reformatted long lines |
| smartsensor/e2e.py | Added numpy import and seed parameter with np.random.seed() call |
| pyproject.toml | Migrated from Poetry to Hatchling build system with Pixi workspace configuration |
| pixi.toml | Removed standalone pixi.toml (functionality moved to pyproject.toml) |
| .gitignore | Added pixi environment exclusions |
| .gitattributes | Added pixi.lock merge configuration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>