Skip to content

Package for PyPI and support BCL Convert (v2) sample sheets#2

Merged
cokelaer merged 1 commit into
mainfrom
packaging-bclconvert
Jul 2, 2026
Merged

Package for PyPI and support BCL Convert (v2) sample sheets#2
cokelaer merged 1 commit into
mainfrom
packaging-bclconvert

Conversation

@cokelaer

@cokelaer cokelaer commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Package the Streamlit app into a check_my_sample_sheet/ package with a check-my-sample-sheet console entry point (launches streamlit run).
  • Replace the pytest-only pyproject.toml with full poetry metadata + build-system so a wheel/sdist can be published to PyPI.
  • Auto-detect v1 (bcl2fastq) vs v2 (BCL Convert) sample sheets via sequana SampleSheetFactory / get_sample_sheet_version; add a v2 example + load button.
  • Reset the file_uploader when an example is loaded, so a previously uploaded file no longer silently takes precedence.
  • Rename UI title to "Check My Sample Sheet".
  • Require sequana>=0.23.0 (ships BCLConvert); add requests dependency.
  • Update tests for the new layout and README with install/usage.

Layout

Assets (examples/, imgs/) moved into the package and resolved via __file__ (cwd-independent). Verified the wheel ships all examples + logo + console script.

Tests

pytest — 13 passed. poetry build produces sdist + wheel.

Follow-ups (not in this PR)

  • Publish to PyPI.
  • Update the Streamlit Cloud main file path to check_my_sample_sheet/app.py (was app.py).

🤖 Generated with Claude Code

- Move app.py, examples/, imgs/ into a check_my_sample_sheet package
- Resolve packaged assets (logo, examples) via __file__ instead of cwd
- Add __main__.py console entry point (check-my-sample-sheet) launching streamlit
- Replace pytest-only pyproject.toml with full poetry metadata + build-system
  and a check-my-sample-sheet script entry point
- Auto-detect v1 (bcl2fastq) vs v2 (BCL Convert) sheets via sequana
  SampleSheetFactory/get_sample_sheet_version; add a v2 example and load button
- Reset the file_uploader when an example is loaded so a previously uploaded
  file no longer takes precedence
- Rename UI title to "Check My Sample Sheet"
- Require sequana>=0.23.0 (ships BCLConvert); add requests dependency
- Update tests for the new package layout and README with install/usage

🤖 Generated with [Claude Code](https://claude.com/claude-code)
@cokelaer cokelaer merged commit 23f849c into main Jul 2, 2026
3 checks passed
cokelaer added a commit that referenced this pull request Jul 3, 2026
Package for PyPI and support BCL Convert (v2) sample sheets
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