【mlflow 系列8】向 mlflow 提交pr(pull request)

简介: 【mlflow 系列8】向 mlflow 提交pr(pull request)

背景


因为之前在帮助算法同事进行mlflow 模型调优的过程中,涉及到mlflow一项优化点,所以给mlflow提交了一个pr


操作


参照官网提供的python contributing,所以按照他的提示一步一步来就可以,

但是在提交pr的过程中还是得注意几点,


描述清楚问题

进行单元测试

进行代码格式化

在代码格式化这个问题上,我就花了一天去搞定,因为之前不太熟悉python,所以有点着急(其实也是没有踏踏实实按照官网的操作来),导致花了一天的时间,在此记录一下过程


写完代码后,进行black --line-length=100 --exclude=mlflow/protos .

这一步是用black格式话代码

提交pr前进行单元测试以及linter pass

./lint.sh
./dev/run-small-python-tests.sh
./dev/run-large-python-tests.sh

其实在单元测试的时候, 也不一定要全部测试,如果你的没标志类似@pytest.mark.large这种注解的话,只需要运行./dev/run-small-python-tests.sh就行,当然我们也可以更加严谨一点

提交pr,就是git的操作 git commit/git push等等

其实最想说的一点是python的格式化和单元测试工具,因为对于java来说,还得自己配置maven插件,而对于python来说是有现成的工具的

打开mlflow下的 lint.sh文件,我们就能看到几种工具


black 代码格式化工具,按照通用的代码格式进行格式化,

pycodestyle 代码风格检查工具

pylint 原代码分析器,可以分析代码错误以及代码风格

这三种工具都可以通过pip install 安装


相关文章
|
3月前
|
监控
【Azure Event Hub】关于Event Hub指标 ConsumerLag 的解释
在使用Azure Event Hub的过程中,需要监控消费端是否正常消费数据?而常规的指标只有 Incoming Message, Outgoing Message,是否指标能表明当前Event Hub消费滞后,即 Incoming 数量远远大于 Outgoing呢?
|
4月前
|
jenkins 持续交付
Jenkins Pipeline 流水线 - Parameters 参数化构建
Jenkins Pipeline 流水线 - Parameters 参数化构建
127 0
|
6月前
flowable流程部署只生成了act_re_deployment,未生成act_re_procdef
flowable流程部署只生成了act_re_deployment,未生成act_re_procdef
|
7月前
|
开发工具 git 开发者
【GitHub】如何在github上提交PR(Pull Request) + 多个pr同时提交、互不干扰
【GitHub】如何在github上提交PR(Pull Request) + 多个pr同时提交、互不干扰
1160 6
|
7月前
|
运维 Kubernetes Devops
云效产品使用报错问题之flow step publish 新的镜像提示: step sign 已被占用。如何解决
本合集将整理呈现用户在使用过程中遇到的报错及其对应的解决办法,包括但不限于账户权限设置错误、项目配置不正确、代码提交冲突、构建任务执行失败、测试环境异常、需求流转阻塞等问题。阿里云云效是一站式企业级研发协同和DevOps平台,为企业提供从需求规划、开发、测试、发布到运维、运营的全流程端到端服务和工具支撑,致力于提升企业的研发效能和创新能力。
|
JavaScript Java Go
Pipeline as Code 轻松管理你的流水线
云效 Flow 基于 Pipeline as Code 支持以 YAML 方式编排流水线,帮助客户解决多条流水线快速创建、批量管理等问题,满足跳过/分支等复杂流程编排场景。今天我们跟着云效流水线产品经理一起体验和感受云效流水线 YAML 化的最新能力。
592 0
|
存储
gitlab--运行流水线、设置 tags、设置 pipeline 状态、添加徽章
gitlab--运行流水线、设置 tags、设置 pipeline 状态、添加徽章
jira学习案例17-hook+ts实现useArray小作业
jira学习案例17-hook+ts实现useArray小作业
68 0
jira学习案例17-hook+ts实现useArray小作业
|
Java
Github参与jdk8u,提交的PR提示:Testing is not configured
Github参与jdk8u,提交的PR提示:Testing is not configured
115 0
Github参与jdk8u,提交的PR提示:Testing is not configured
|
Kubernetes API 调度
开发 k8s 管理平台 - k8sailor 17. Pod 的阶段(phase)与状态(status)
开发 k8s 管理平台 - k8sailor 17. Pod 的阶段(phase)与状态(status)
352 0
开发 k8s 管理平台 - k8sailor 17. Pod 的阶段(phase)与状态(status)