search

aidenlab / Juicebox

Visualization and analysis software for Hi-C data -
https://aidenlab.org/juicebox
MIT License
235 stars 57 forks source link

We created directory for logging, this can cause problems #533

Open theaidenlab opened 7 years ago

theaidenlab commented 7 years ago

We create directory for logging when we run Pre. It's not clear we need to do this. I'm not sure we write to the log file at all; if we do, it's very sporadic.

I've gotten errors on Voltron from the pre hic and pre hic30 job trying to access the Preferences at the same time and one of them dying. If this is not necessary, i.e. if we're not logging during pre, then we shouldn't do it.

sa501428 commented 7 years ago

Basically delete all calls/instances of the Logger class. Any call similar to log.error() should be replaced with System.err.println()

nchernia commented 7 years ago

This is high priority, this is the cause of the Voltron problem.

On Thu, Jun 22, 2017 at 8:58 PM, Muhammad Saad Shamim < notifications@github.com> wrote:

Basically delete all calls/instances of the Logger class. Any call similar to log.error() should be replaced with System.err.println()

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/theaidenlab/JuiceboxDev/issues/533#issuecomment-310471516, or mute the thread https://github.com/notifications/unsubscribe-auth/ABWiW54WlmHKjtCxHrYWCIaNBL4PorD9ks5sGrlvgaJpZM4N8fyR .

-- Neva Cherniavsky Durand, Ph.D. Staff Scientist, Aiden Lab www.aidenlab.org