Skip to content

fix: Apache-2.0 license text and stdout I/O for CSV/FASTA/PDB - #12

Merged
andrewcstewart merged 2 commits into
mainfrom
fix/license-and-stdout-io
Jul 15, 2026
Merged

fix: Apache-2.0 license text and stdout I/O for CSV/FASTA/PDB#12
andrewcstewart merged 2 commits into
mainfrom
fix/license-and-stdout-io

Conversation

@andrewcstewart

Copy link
Copy Markdown
Member

Summary

  • Replace the short Apache notice with the full official Apache License 2.0 text, add a NOTICE copyright file, and align packaging with SPDX / PEP 639 (license = "Apache-2.0").
  • Fix to_csv / to_fasta / to_pdb so file_path="-" writes to stdout (matching to_json), so biolm model run ... --format csv actually prints instead of creating a literal - file.

Test plan

  • Confirm sdist includes LICENSE / NOTICE and License-Expression: Apache-2.0
  • pytest tests/test_io.py (stdout - cases for csv/fasta/pdb)
  • biolm model run esm2-8m encode -i <file>.json --format csv prints CSV to stdout (no - file in cwd)

Made with Cursor

andrewcstewart and others added 2 commits July 14, 2026 21:22
Replace the short license notice with the official Apache License 2.0,
preserve copyright attribution in NOTICE, and align packaging with PEP 639.

Co-authored-by: Cursor <cursoragent@cursor.com>
Match to_json so biolm model run --format csv (and fasta/pdb) emit
payload to stdout instead of creating a literal '-' file.

Co-authored-by: Cursor <cursoragent@cursor.com>
@andrewcstewart
andrewcstewart merged commit c7f6488 into main Jul 15, 2026
8 checks passed
@andrewcstewart
andrewcstewart deleted the fix/license-and-stdout-io branch July 15, 2026 04:33
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