Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • M metaseq
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 95
    • Issues 95
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 41
    • Merge requests 41
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Administrator
  • metaseq
  • Issues
  • #435
Closed
Open
Issue created Oct 19, 2022 by Administrator@rootOwner

Remove defaults for arch - force users to specify

Created by: suchenzang

There are lots of defaults set for all the transformer_* arches - we should remove as much of these as possible and noisily fail if users do not specify these values.

https://github.com/facebookresearch/metaseq/pull/429#discussion_r998848249 is an example where learned position still defaults to False, even though all of our runs now default this to True in our sweep scripts. These shouldn't be set at all and left to users to define each time (until we sort out our config pains).

Assignee
Assign to
Time tracking