pepkit / peppy

Project metadata manager for PEPs in Python
https://pep.databio.org/peppy
BSD 2-Clause "Simplified" License
37 stars 13 forks source link

Allow null subsample tables in `peppy.Project` #393

Closed nleroy917 closed 1 month ago

nleroy917 commented 2 years ago

I was attempting to create a new peppy.Project object with the following config:

name: GSE96155
pep_version: 2.1.0
sample_table: GSE96155_annotation.csv
subsample_table: null
...

And peppy threw the following error: TypeError: Provided argument has to be a List[str] or a str, got 'NoneType'

Based on the discussion over at geofetch: https://github.com/pepkit/geofetch/issues/80, peppy should allow null sub sample tables.

nleroy917 commented 1 year ago

Low priority, but just saying this still exists as a bug.