pepkit / looper

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

Local variable used before defined in Checker #355

Closed vreuter closed 12 months ago

vreuter commented 1 year ago

https://github.com/pepkit/looper/blob/dev/looper/looper.py#L123

@nsheff do you know what should be going on here? I'm guessing maybe this isn't hit yet since it requires use of "new" Checker that's using pipestat?

donaldcampbelljr commented 1 year ago

The new Checker is working fine to generate a table if looper is configured to use pipestat and it hould not be able to access the new Checker if pipestat is not configured.

looper check ./hello_looper-master/project/project_config_pipestat_example.yaml
Looper version: 1.4.0-dev

     'count_lines' pipeline status summary     
┏━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
┃   Status   ┃     Jobs count/total jobs      ┃
┡━━━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┩
│   failed   │              1/5               │
└────────────┴────────────────────────────────┘

I believe this is not an issue?