组里的某senior DS不会用git, 怎么说服他呢?

楼主在非tech 传统行业做data science engineer.


组里一个senior DS不会用git 做 version control, 他什么改动都在master branch. 重点是他的改动自己没测过, 经常出现iden错位或者typo (import pandas 写成 import andas) 之类的问题, 这就给pull master造成很大的困扰.  
还有一个问题是由于他只在master branch改动, 他没有用PR的习惯, 可能改动只有一行 或者是改零散几个点. 而由于他不习惯比较moduler的写法, 他把所有的逻辑都写到一个script ( > 700 lines for a script)

楼主试图和他结束为什么PR是很重要的方法去track changes (这个project大概有4-5个人) , 但是他ego很强 - result important 所以code review不重要是huge burden.

楼主的确没想到现在还有人不用version control ( 他是operation research的PhD ... 他们paper都不需要合作的吗....) , 也和manager提出来希望能get some help, manager也试图提出来master改动的重要性, 但是这个大哥不愿改动 ....

这算不算是很不错的 solve colleague confict的一个例子 ? ( 但是楼主似乎没解决好... ) 求各位大佬提提意见?

谢谢!

很多DS在之前不是非常注重product的公司，他们是不需要写git的，甚至没有code review的环节。所以虽然极端些，但是我觉得在接下来一段时间内还是会遇到很多git用的少的DS。

我的建议是这样：
1. 向你的manager汇报这位DS不太会git，不要用埋怨的口气，简单的阐述事实。
2. 表明这样会拖慢组内的进度（不要说你的进度，说组内的进度），但是自己愿意花时间向那个DS简单go through 一下git的流程，说git不是那么难，稍微帮他过一遍相信就没有问题了。
3. 如果manager说没关系不用，那到这里你就不用再管了。manager已经知道了这个问题，到时候真的项目有问题也不需要你承担责任了。
4. 但是大概率manager会说好啊，麻烦你了。这时候你顺便说一句其他ds 或者其他stakeholder要不要也一起过一下？来一个小seminar刚好帮大家一起省时间。如果manager能找到不止一个人跟你学习，那同样也不会伤害那个DS的面子。
5. 你大概花半个小时到一个小时的时间给他们简单说一下git的基本信息，强调一下那个DS在工作中出现的问题就可以了。这样以后再出现问题，也不会是你的责任了。

不给他push to master的权限。。。

你们都不code review就能直接merge，这不明摆着会出事吗。这是公司管理上的问题，如果管理层连这些基本的software engineering practice都不去enforce的话，你就自己决定自己是不是和公司的文化相匹配吧。
有些规则enforce了会造成阵痛，但是明白人都应该去做。你不能说西藏农奴制几千年了运转得也很好改起来挺麻烦，就要一直保持下去

PhD 不用git真的很正常。平时研究都习惯了自己代码存本地单干，多半都是one man army。我觉得是说服不了的，如果问题很严重，并且影响了多个组员，可以大家一起开的会？之后还不行就找老板。

为啥不愿意改？强制改，只有benefit没有坏处啊

麻烦大家给我加点米🥹🥹， 没米看面筋，谢谢🙏

让他break一次背个锅，以后你们就有例子来enforce他了，或者他就被开了。：)

匿名用户 发表于 2023-04-28 12:54:32
PhD 不用git真的很正常。平时研究都习惯了自己代码存本地单干，多半都是one man army。我觉得是说服不了的，如果问题很严重，并且影响了多个组员，可以大家一起开的会？之后还不行就找老板。

倒也不一定啦……我和我身边的CS PhD表示不用git很离谱

ChenLiu1996 发表于 2023-4-28 16:38
倒也不一定啦……我和我身边的CS PhD表示不用git很离谱

会一点git然后瞎搞的更可怕，搞错了还要给擦屁股就很烦

看中国的狗哥 发表于 2023-04-28 13:49:29
会一点git然后瞎搞的更可怕，搞错了还要给擦屁股就很烦

那只能说你们的设置有问题…一般需要code review，git push上传的东西不会直接merge的，很安全