facebookresearch/ParlAI

Unify names of some options

Open

#2,170 创建于 2019年11月18日

在 GitHub 查看
 (1 评论) (0 反应) (0 负责人)Python (10,371 star) (2,089 fork)batch import
Code QualityH1 2020Help WantedMinordonotreap

描述

Now that we have upgrade_opt we can do some resolution of the embedding_size and embeddingsize madness we have in seq2seq vs transformer vs memnn etc.

贡献者指南

技术栈
pythonpytorch
领域
developer experience
议题类型
refactor
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
half day
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
stale
清晰度议题是否清楚说明期望改动、验收标准和下一步。
mostly clear
前置要求
PythonParlAI codebase familiarityunderstanding of option naming conventions
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
65
研究方向
Search the codebase for all occurrences of 'embedding size' and 'embeddingsize'. These appear in seq2seq, transformer, and memnn agents. Use the existing 'upgrade opt' mechanism (likely in options.py or similar) to alias the old inconsistent names to a single unified name. Ensure that any existing configuration files still work by mapping the old names. Consult the issue comments for any additional guidance on the preferred unified name.