In the current implementation, the definition of the hydra.run.dir leads to an error if a different logger than wandb is used. Therefore, put the definition of the special wandb run.dir into the wandb logger config and use a default one in the hydra config instead.
Moreover, added a very basic hydra tutorial notebook.
[x] I have raised an issue to propose this change (required for new features and bug fixes)
Types of changes
What types of changes does your code introduce? Remove all that do not apply:
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds core functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (update in the documentation)
[x] Example (update in the folder of examples)
Checklist
Go over all the following points, and put an x in all the boxes that apply.
If you are unsure about any of these, don't hesitate to ask. We are here to help!
[ ] My change requires a change to the documentation.
[ ] I have updated the tests accordingly (required for a bug fix or a new feature).
Description
Closes #132
In the current implementation, the definition of the hydra.run.dir leads to an error if a different logger than wandb is used. Therefore, put the definition of the special wandb run.dir into the wandb logger config and use a default one in the hydra config instead.
Moreover, added a very basic hydra tutorial notebook.
Motivation and Context
Changes to hydra config as discussed with @fedebotu in https://github.com/ai4co/rl4co/discussions/119
Types of changes
What types of changes does your code introduce? Remove all that do not apply:
Checklist
Go over all the following points, and put an
xin all the boxes that apply. If you are unsure about any of these, don't hesitate to ask. We are here to help!