Conversation
This PR updates the BaseEnsemble constructor to allow the following: 1. Passing a string or path to the configuration 2. Not passing a configuration at all, which will automatically load the default configuration. This is now the default, since most users won't want to have to configure it (it should have reasonable defaults)
|
Hi! Thanks for your contribution. I think being able to use the I had some trouble running the code, in particular
@cthoyt Could you check if this is what you had in mind? |
|
@sfluegel05 thanks for taking a look! i'll try and resolve this ASAP and ping you here after |
|
@sfluegel05 just resolved the circular imports |
|
thanks for fixing this! |
Hello! I was just trying to run chebifier myself and thought there were a few ways it could be improved for quality of life.
This PR does the following:
BaseEnsembleBaseEnsemblefrom the top-level so it's easier to import