cgat-developers / cgat-apps

cgat-apps repository
Other
33 stars 14 forks source link

Debugging print statement left in #113

Closed IanSudbery closed 4 months ago

IanSudbery commented 2 years ago

There is what appears to be a debugging statement left in IndexedFasta, which is leading to multi gigabyte log files.

@Acribbs Git blames says it's you? Is it okay to remove this?

https://github.com/cgat-developers/cgat-apps/blob/4de381418e80cfd085b17fc8a3048a48f55e57cf/cgat/IndexedFasta.py#L490

Acribbs commented 2 years ago

Oops, bad debugging from me, definitely, sorry.

Get Outlook for iOShttps://aka.ms/o0ukef


From: Ian Sudbery @.> Sent: Wednesday, February 2, 2022 5:49:37 PM To: cgat-developers/cgat-apps @.> Cc: Adam Cribbs @.>; Mention @.> Subject: [cgat-developers/cgat-apps] Debugging print statement left in (Issue #113)

There is what appears to be a debugging statement left in IndexedFasta, which is leading to multi gigabyte log files.

@Acribbshttps://github.com/Acribbs Git blames says it's you? Is it okay to remove this?

https://github.com/cgat-developers/cgat-apps/blob/4de381418e80cfd085b17fc8a3048a48f55e57cf/cgat/IndexedFasta.py#L490

— Reply to this email directly, view it on GitHubhttps://github.com/cgat-developers/cgat-apps/issues/113, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AC4CNZ3WQQVEBXFXAKXLHY3UZFVDDANCNFSM5NMXAT4Q. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you were mentioned.Message ID: @.***>

Acribbs commented 4 months ago

Fixed with #120