Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Update task args for wandb logic to be cleaner #4548

Merged
merged 2 commits into from
May 17, 2022

Conversation

moyapchen
Copy link
Contributor

@moyapchen moyapchen commented May 16, 2022

Way easier to reason about things to have the task arg bit be in a separate section; also add in bit to skip splitting for multitask args.

(Looking at my wandb and commit history, looks like what I ran that was working was a different fork than what I'd used to test on wandb? Not sure at allll how that happened.)

Test plan:

  • Ran locally both with and without multitasking + task args, verified it behaved properly

Way easier to reason about things to have the task arg bit be in a separate section
@moyapchen moyapchen merged commit a161633 into main May 17, 2022
@moyapchen moyapchen deleted the mpchen/fix_wandb_logging branch May 17, 2022 00:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants