search

ESCOMP / CISM

Community Ice Sheet Model
GNU Lesser General Public License v3.0
6 stars 11 forks source link

libglissade IO issues #32

Open jedwards4b opened 3 years ago

jedwards4b commented 3 years ago

l've been trying to clean up cesm IO issues and found that libglissade does not use the normal cesm logging mechanism and instead does print for log messages. print should be replaced with write(stdout, *) throughout this library.

gunterl commented 3 years ago

Hi Jim,

I'll talk to Bill about this and get back to you.

Thanks

Gunter

On Tue, Oct 6, 2020 at 11:27 AM jedwards4b notifications@github.com wrote:

l've been trying to clean up cesm IO issues and found that libglissade does not use the normal cesm logging mechanism and instead does print for log messages. print should be replaced with write(stdout, *) throughout this library.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ESCOMP/CISM/issues/32, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB6QV6BMAM5ZKJVMF4M2FVLSJNHO5ANCNFSM4SGKK4EA .

-- Gunter Leguy, Ph.D (he/him) Project Scientist National Center for Atmospheric Research cell: (575) 418 1021 desk: (303) 497 1790

whlipscomb commented 3 years ago

@jedwards4b, How soon do you need this to happen? This is a straightforward change, but it touches a lot of code and could create some conflicts between the main trunk and various development branches.

Adding @Katetc as well as @gunterl to the conversation.

jedwards4b commented 3 years ago

It's not urgent, but it would be a nice cleanup. Thanks for considering.

whlipscomb commented 3 years ago

@jedwards4b, This is definitely something we can do. @gunterl and I are wrapping up a couple of papers now. Once that's done, we're planning to do some cleanups and mergers, and we'll add this to the list.