Remove some warnings from Mimi #481
Conversation
There was a problem hiding this comment.
Unless we're considering restoring these @warn calls, there's no need to keep the test_log stuff. Also, if we're not issuing warnings on things like redefinitions, there's little point in keeping the calls. @warn is user-oriented, while @debug is developer-oriented. For this reason I wonder if it ever makes sense to change @warn to @debug. @davidanthoff -- what do you think?
There was a problem hiding this comment.
I'm in favor of merging as is for now.
We can revisit the issue of verbose mode later. I looked a bit into the logging infrastructure, and I think it would be fairly simple to use that for our needs. But after the workshop :)
@ckingdon95 do you want to merge this one here once you are sure that it is ready?
#480