This PR makes the WandBLogger wrapper compatible with pre-existing WandB sessions. Additionally, some related details have been improved in the tool. For more information, see the changelog.
Motivation and Context
We've noticed that Sinergym can be used in scripts where a WandB session has already been started. The wrapper was incompatible with this use case, such as when using sweeps.
Now, the wrapper can be used with a pre-existing WandB session, without the need to specify the entity or project (which, if provided, will be ignored). It still allows specifying other parameters during construction, maintaining full functionality of the wrapper.
Types of changes
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
Description
This PR makes the
WandBLoggerwrapper compatible with pre-existing WandB sessions. Additionally, some related details have been improved in the tool. For more information, see the changelog.Motivation and Context
We've noticed that Sinergym can be used in scripts where a WandB session has already been started. The wrapper was incompatible with this use case, such as when using sweeps.
Now, the wrapper can be used with a pre-existing WandB session, without the need to specify the entity or project (which, if provided, will be ignored). It still allows specifying other parameters during construction, maintaining full functionality of the wrapper.
Types of changes
Checklist:
autopep8second level aggressive.isort.cd docs && make spelling && make htmlpass (required if documentation has been updated.)pytest tests/ -vvpass. (required).pytype -d import-error sinergym/pass. (required)Changelog: