API Key in train.py - Githubissues

fabienmorgan commented 6 months ago

I would suggest you to remove the API Key of Weights & Biases in the train.py file. This key should not be known to the public.

1716757342 commented 6 months ago

Thank you for your reminder, we have taken care of it. Wishing you a pleasant life.

fabienmorgan commented 6 months ago

I don't think the issue has been. Firstly, the API Key still can be found in the git history train.py. Secondly, you need to remove or comment out the whole line and can't just remove the string. Else os.environ["WANDB_API_KEY"] and os.environ["WANDB_MODE"] get assigned to nothing, which returns this error: Traceback (most recent call last):

File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/fmg/repos/FMg_SR_Bachelor_Thesis/Original_MMSR/MMSR/__main__.py", line 10, in <module>
    from .train import train as train
  File "/home/fmg/repos/FMg_SR_Bachelor_Thesis/Original_MMSR/MMSR/train.py", line 4
    os.environ["WANDB_API_KEY"]  = 
                                  ^
SyntaxError: invalid syntax

1716757342 / MMSR

API Key in train.py #2