-
Notifications
You must be signed in to change notification settings - Fork 29
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
Update CN-faults.gmt from the official data #48
Conversation
related to #40 |
参考阮一峰的博客了解回车和换行 参考解决方案:
|
@liuzhumei 当前你是从 forked 的仓库提交。如果把你设置为 contributors,就具有读写 GMT-china 仓库的权限。这样做的好处是你可以直接提交 PR 至 GMT-china 的仓库,特别是中文手册可以预览提交的 PR 实际网页效果,如 gmt-china/GMT_docs#444 (comment). 是否可以把你加入 contributors 团队,方便你常常提交 PR 至社区仓库? |
那太好了,我最头疼没法预览了。那我在本地的提交步骤并没有区别吧,还是git push origin --> PR的过程? origin和upstream就是同一个仓库了。 |
在这个PR中我上传的是以CRLF为行尾的GB2312编码文件,是windows用户需要的。编译的时候再生成一个LF结尾的UTF-8文件就行了吧? |
最好是传 LF 结尾的文件。参考这个 #48 (comment).
yes. 那时候,origin 是 gmt-china 的仓库.
当前 #493 还是原先的模式,使用的是你自己 forked 的仓库。 下次可以直接提交修改至 gmt-china 的仓库:
跟修改自己的仓库一样,也没有所谓的 upstream 远程了。 已经邀请了,请接受:https://github.com/orgs/gmt-china/people/pending_invitations |
本仓库里的文件都是Linux/macOS用户所使用的,所以是LF结尾、UTF8编码。在发布新版本时,会有workflow自动将其转换为Windows可以使用的GB2312编码。 |
还有一个问题,这个PR里的数据看上去是比master分支里已经有的数据要少不少行的,是数据采样率不同?还是数据处理过程中数据被阶段了?我之前用某个命令转换的时候会遇到某个字符编码出错,导致转换失败进而数据被截断。如何确认数据完整性呢? |
@liuzhumei 我对文件的准确性没有什么疑问了。目前依然存在的几个问题:
|
|
很奇怪,我可以看到是 UTF8 编码,但是我打开文件依然是乱码。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
@core-man OK to merge? |
shp格式的 白泥-沙湾断裂 是一段,但是 gmt 格式就是以下两段且只有第一段有属性信息,是这个意思吗?
因此,目前 PR 的 gmt 格式数据跟官方 shp 格式数据的唯一差异就是数据段数以及多段数据的头段信息的位置,原因是使用 |
是的。
如果不算对官方数据属性字段名称的纠错以外(“分段魂龄” 改为 “分段年龄”, "地震地_"改为 “地震地表破裂带时间” 等等。 官方数据应该是从tab格式转成的shp,所以本身就有错误), 数据分段是PR数据的唯一差异了。 |
所以此 PR 里全部数据处理流程?
|
是的,原则上这三个步骤就够了。实际上,第二个步骤shp转gmt后,我生成的实际是ANSI编码的数据,所以第一次commit后, @seisman 看到的是乱码。我后来手工转成utf编码并最后一次commit的。 我再研究研究ogr2ogr命令,应该可以在脚本中解决,并更新到 GMT_docs#493 中。 |
提交文件的时候总是提示"LF will be replaced by CRLF" 应该怎么处理?
Fixes #40.