Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Discussion about uploading programs to CEPC #12

Open
xshi opened this issue Sep 19, 2016 · 0 comments
Open

Discussion about uploading programs to CEPC #12

xshi opened this issue Sep 19, 2016 · 0 comments

Comments

@xshi
Copy link
Contributor

xshi commented Sep 19, 2016

大家好:

我是 github 上 cepc 组织账号的创建者,上次给过一个关于分布式分析的报告【1】,没有参加会议的同学欢迎参考。很高兴大家开始逐步把代码开始往上传。 有几点借此机会跟大家说明一下,也欢迎讨论:

  1. 如何开始上传?

如报告中所述,我们建议采用“Fork & Pull Reques” 模式开发。具体来说就是大家只需要告诉我们你所要做的项目(分析题目)的名字 (例如higgsrecoil), 我们就帮你创建一个只含有Readme的空项目。然后你对此项目进行“Fork”,在此基础上进行开发,具体步骤见报告。

  1. 上传哪些内容?

以物理分析程序为例,建议大家以”higgsrecoil” 项目【2】为模板,精简程序内容,尽量避免你的程序中有冗余代码,力争做到每一行都是有用的(至少是你明白为什么在那里)。 因为有了最简版本后再往上加就容易了。反之以后再做删减会非常痛苦。 另外就是最好只上传源代码以及文本文件。二进制数据等不要上传。

  1. 版权问题

如果程序是你原创的,或者借鉴了谁的工作,建议在文件头部注释部分写清楚,参考例子【3】。

  1. 如何才能“合并” 到CEPC的主账号里?

当你认为fork后的程序包可以合并,里面除了精简的程序外,还要在Readme中说明如何安装,编译,以及运行, 参考【2】中的三个bash 文件。 目的是另外一个人可以完全重复出来。 确认这些都没问题后,可以创建“pull reques” 简称“PR”, 在说明中@amazingzby (张冰洋), 或者相关的人员,只有安照你的说明能够重复出来了,我们再把代码进行合并。 这样保证一次提交至少有除作者外的另一个人测试过,保证代码质量。 类似的对话可以参考【4】

【1】http://indico.ihep.ac.cn/event/6253/session/7/contribution/30

【2】 https://github.com/cepc/higgsrecoil

【3】 https://github.com/besiii/jpsi2invi/blob/master/Analysis/Physics/PsiPrime/Jpsi2invi/Jpsi2invi-00-00-01/src/Jpsi2invi.cc#L6-L8
【4】 cms-sw/cmssw#4946

祝好!
史欣

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant