Skip to content

Conversation

@nttg8100
Copy link
Member

@nttg8100 nttg8100 commented Dec 1, 2025

  • Release for paper Ampiciline

nttg8100 and others added 3 commits December 1, 2025 23:50
Copy link
Contributor

Copilot AI left a 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.

nttg8100 and others added 2 commits December 23, 2025 21:21
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>
@nttg8100 nttg8100 merged commit 432a511 into main Dec 23, 2025
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