-
Notifications
You must be signed in to change notification settings - Fork 2
Artn and flare #144
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
Artn and flare #144
Conversation
…repaint adding information dicts with sample makers
active_environment_indices=active_environment_indices, | ||
) | ||
|
||
logger.info(" Fitting the FLARE hyperparameters...") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note that we should make this logging more agnostic to the method. Maybe not in this PR, but something to keep in mind as a TODO
logger.info(f" sigma = {sigma: 12.8f}") | ||
logger.info(f" sigma_e = {sigma_e: 12.8f}") | ||
logger.info(f" sigma_f = {sigma_f: 12.8f}") | ||
logger.info(f" sigma_s = {sigma_s: 12.8f}") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all the parameters above are FLARE dependent. Maybe we could hide this logging in a flare_trainer method ? Or something like that. Could be done in another PR
A first tracer bullet for active learning with FLARE.