Fix hatchling packages configuration for src layout #36
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
7 tasks
Co-authored-by: GiggleLiu <6257240+GiggleLiu@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Update to address feedback from PR #35 for v0.1 release
Fix hatchling packages configuration for src layout
Jan 20, 2026
Copilot stopped work on behalf of
GiggleLiu due to an error
January 20, 2026 09:48
GiggleLiu
added a commit
that referenced
this pull request
Jan 20, 2026
* refactor: Professional restructuring for v0.1 release Transform BPDecoderPlus from training project to production-ready package. Resolves #28 ## Added - LICENSE file (MIT License) - CONTRIBUTING.md with development guidelines and setup instructions - CHANGELOG.md following Keep a Changelog format - .pre-commit-config.yaml with ruff, security checks, and file validation - .editorconfig for consistent coding styles across editors - Project URLs in pyproject.toml (Homepage, Documentation, Repository, Issues) - Maintainers field in package metadata - Ruff configuration for linting and formatting (line-length: 100) - Development dependencies: ruff, pre-commit, mypy - New Makefile targets: format, lint, lint-fix, type-check, pre-commit, check, dev-setup ## Changed - Reorganized dataset structure to single-source flat layout in datasets/ - Moved all dataset files from subdirectories to datasets/ root - Moved visualization PNGs to datasets/visualizations/ - Updated CLI default dataset path from datasets/noisy_circuits to datasets - Updated README.md to focus on Python implementation - Updated dataset documentation to reflect new structure - Fixed pyproject.toml structure (dependencies before project.urls) - Migrated tool.uv.dev-dependencies to dependency-groups.dev - Updated Makefile dataset generation paths - Moved lecture notes from docs/ to note/ directory - Updated .gitignore with development tools and binary file patterns ## Removed - Duplicate dataset files from subdirectories (circuits/, dems/, uais/, syndromes/, noisy_circuits/) - Julia code examples and references from README - Winter school training references from project description ## Testing - All 71 tests pass - Package builds and imports correctly - Dataset structure validated 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> * [WIP] Update to address feedback from PR #35 for v0.1 release (#36) * Initial plan * fix: Correct package path configuration to src/bpdecoderplus Co-authored-by: GiggleLiu <6257240+GiggleLiu@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: GiggleLiu <6257240+GiggleLiu@users.noreply.github.com> --------- Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com> Co-authored-by: GiggleLiu <6257240+GiggleLiu@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The
packagesfield in[tool.hatch.build.targets.wheel]pointed tobpdecoderplusinstead ofsrc/bpdecoderplus, causing the build system to look for the package in the wrong location.Changes:
pyproject.tomlline 62:packages = ["src/bpdecoderplus"]This aligns with the src-layout structure where the package source is at
src/bpdecoderplus/.✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.