specifysystems / lmpy

A library of tools useful for Lifemapper and related computations

GNU General Public License v3.0

2 stars 0 forks source link

wrangler logging #356

Closed zzeppozz closed 2 years ago

zzeppozz commented 2 years ago

title: 'Pull request template'

Pull Request Type

[ ] Bug Fix
[ ] New Feature
[ ] Documentation
[X ] Other

User tool feedback with logging and reporting.

Status

[ ] Ready
[X] In development
[ ] Hold

Description

Feedback for users of command-line tools so they know something is happening.

Link(s) to issue

342

Todos

[ X ] Tests
[ X ] Documentation
[ X ] Sample Data
[ ] Build Sphinx Documentation

Testing

Test command line tools with scripts in lmpy/tools/ directory and configuration/data files in biotaphy/tutorials repo

Sample Data

github biotaphy/tutorials repositiory, data/config and data/input directories

lgtm-com[bot] commented 2 years ago

This pull request fixes 1 alert when merging 45078a10f8bed6023fd9ff3809901e32d81f006f into 366129cd628bbcef9162dd01c8e0e830eaca51f5 - view on LGTM.com

fixed alerts:

1 for Unused import

lgtm-com[bot] commented 2 years ago

This pull request fixes 1 alert when merging c00184f44a4fe78b8c3b4f5941864deb268000cf into 366129cd628bbcef9162dd01c8e0e830eaca51f5 - view on LGTM.com

fixed alerts:

1 for Unused import

lgtm-com[bot] commented 2 years ago

This pull request fixes 1 alert when merging 70ea18f6a369d17af1b7ecf6e8bd00d1af8960aa into 366129cd628bbcef9162dd01c8e0e830eaca51f5 - view on LGTM.com

fixed alerts:

1 for Unused import

lgtm-com[bot] commented 2 years ago

This pull request fixes 1 alert when merging 5637151ad001fc3f4786f43360806ed4ce098c4b into 366129cd628bbcef9162dd01c8e0e830eaca51f5 - view on LGTM.com

fixed alerts:

1 for Unused import

lgtm-com[bot] commented 2 years ago

This pull request fixes 1 alert when merging d5bea7e03f78de2b2ded88fbe7e3d268bcb522f4 into 366129cd628bbcef9162dd01c8e0e830eaca51f5 - view on LGTM.com

fixed alerts:

1 for Unused import

lgtm-com[bot] commented 2 years ago

This pull request fixes 1 alert when merging 2495db2ac5744f8304f0fecaffadfd5169d174a3 into 366129cd628bbcef9162dd01c8e0e830eaca51f5 - view on LGTM.com

fixed alerts:

1 for Unused import

zzeppozz commented 2 years ago

Wrangler logging complete, more to be added in other tools

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert and fixes 1 when merging 62e645709fb192d21a25d3c0a238d597b2a9b965 into 366129cd628bbcef9162dd01c8e0e830eaca51f5 - view on LGTM.com

new alerts:

1 for Unused import

fixed alerts:

1 for Unused import