Skip to content

fix: add fallback for step metrics - #185

Closed
thehappydinoa wants to merge 1 commit into
openai:mainfrom
thehappydinoa:patch-1
Closed

thehappydinoa wants to merge 1 commit into
openai:mainfrom
thehappydinoa:patch-1

Conversation

@thehappydinoa

@thehappydinoa thehappydinoa commented Jan 15, 2023 •

Copy link
Copy Markdown

Fix for the following error:

Traceback (most recent call last):
  File "/Users/me/Desktop/Projects/project/.venv/bin/openai", line 8, in <module>
    sys.exit(main())
  File "/Users/me/Desktop/Projects/project/.venv/lib/python3.10/site-packages/openai/_openai_scripts.py", line 63, in main
    args.func(args)
  File "/Users/me/Desktop/Projects/project/.venv/lib/python3.10/site-packages/openai/cli.py", line 527, in sync
    resp = openai.wandb_logger.WandbLogger.sync(
  File "/Users/me/Desktop/Projects/project/.venv/lib/python3.10/site-packages/openai/wandb_logger.py", line 73, in sync
    fine_tune_logged = [
  File "/Users/me/Desktop/Projects/project/.venv/lib/python3.10/site-packages/openai/wandb_logger.py", line 74, in <listcomp>
    cls._log_fine_tune(
  File "/Users/me/Desktop/Projects/project/.venv/lib/python3.10/site-packages/openai/wandb_logger.py", line 162, in _log_fine_tune
    step = metrics.pop("step")
KeyError: 'step'

This was throwing errors locally.
@thejaminator

Copy link
Copy Markdown
Contributor

should be fixed in the latest version. could you try downloading the latest package?

@doramasma

doramasma commented Jan 25, 2023 •

Copy link
Copy Markdown

Hello! I have the same problem and I have just dowloaded the latest version. So, maybe the problem is still there!

@hallacy

hallacy commented Jan 25, 2023

Copy link
Copy Markdown
Contributor

We'll be releasing the latest patch today on pypi. In the meantime: if you'd like to test out the change, you'll need to git clone this repo and install it from source.

@hallacy

hallacy commented Jan 25, 2023

Copy link
Copy Markdown
Contributor

0.26.3 is live!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants