-
Notifications
You must be signed in to change notification settings - Fork 3
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
[WIP] Don't rely on configs in prepare_for_gfn
#62
base: uncertainty
Are you sure you want to change the base?
Conversation
Make loading the data optional ; MAKE SURE this does not break GflowNet validation procedure which could expect the dataloaders |
Implemented a first version refactoring config["prevent_load"] = {
"logger": True,
"loss": True,
"datasets": True,
"optimizer": True,
"extras": True,
} |
On |
Not tested yet, feel free to do it yourself in the mean time.
This would be my preferred direction if we don't hit more configs-related obstacles.