Tom Tucker

@nichamon I think we should consider having a base_sampler_term() function that has this function in it + some other base sampler cleanup. Same thing with setup, then we can "normalize"...

> > @nichamon I think we should consider having a base_sampler_term() function that has this function in it + some other base sampler cleanup. Same thing with setup, then we...

@nichamon, could you please review this?

@baallan the reason there is an order dependence is because the plugin decides what the log name is. I'm struggling with creating logs for plugins that are not loaded yet....

> > @baallan the reason there is an order dependence is because the plugin decides what the log name is. I'm struggling with creating logs for plugins that are not...

How do we feel about adding a log-level option to the load command? That way we could get what plugin writers want/need and not have the possibility of waif logs...

He's using at as a "sentinel" to determine when/if the log has been pre-registered or registered. I think this is something called "data coupling" which is, in general, highly discouraged....

@baallan, this is getting a bit far afield. All we want to do is log messages from init(), correct?

> > @baallan the reason there is an order dependence is because the plugin decides what the log name is. > > That seems like a problematic design in the...

Sorry, I meant re-factor...