-
Notifications
You must be signed in to change notification settings - Fork 127
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
$COBAYA_MODULES env variable? #32
Comments
Already there https://github.com/CobayaSampler/cobaya/blob/master/cobaya/run.py#L136 But I see now that it's not documented. Will add to the docs, thanks! |
Great! |
Could
|
Added and both documented. I hope you don't mind that it will come with 2.0: cherry-picking is a little dangerous at the moment (quite a few changes). It'll be in <2 weeks anyway. |
Just another suggestion as you move toward 2.0-- have you considered using a $COBAYA_MODULES environment variable as the default modules directory? That way, if
modules
isn't passed in an info string, then you can just default toos.getenv('COBAYA_MODULES', None)
or something like that as the modules path. This would also make it easier to share.yaml
files between machines (e.g., would avoid having to hard-code paths).If you're interested, I'm happy to give a shot at a PR along these lines.
The text was updated successfully, but these errors were encountered: