TUM-DAML / seml

SEML: Slurm Experiment Management Library
Other
165 stars 29 forks source link

Fix neptune api token argument, fixes #91 #92

Closed heborras closed 2 years ago

heborras commented 2 years ago

Reference issue

Closes #91

What does this implement/fix?

The fix changes how the API token is set depending on the function argument. And should fix issue #91.

Additional information

The PR also implements, that the example setting for the token in the settings.py is ignored. Otherwise the Neptune API token will never be read from the environment.

danielzuegner commented 2 years ago

Looks good to me. Thanks!