pepkit / looper

A job submitter for Portable Encapsulated Projects
http://looper.databio.org
BSD 2-Clause "Simplified" License
20 stars 7 forks source link

Project name cannot be inferred when loading PEP from csv #484

Closed donaldcampbelljr closed 3 weeks ago

donaldcampbelljr commented 3 months ago
          > However, Looper goes on to later call `prj.name` and it fails with the exception: `Project name inference isn't supported on a project that lacks a config file.`

This would be a separate bug, then, because those are valid projects.

Originally posted by @nsheff in https://github.com/pepkit/looper/issues/456#issuecomment-1998076553

donaldcampbelljr commented 3 months ago

Currently, in Peppy, this will search for the name field in the project config. If none is given, it will default to project as the name. However, with loading from a csv, there is no project config and Peppy raises a NotImplementedError: https://github.com/pepkit/peppy/blame/6913e77f046b60d74681284ce4c22a133cca4f5f/peppy/project.py#L1048-L1052