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

检查102新版sac的命令的变化 #213

Open
wangliang1989 opened this issue Apr 23, 2021 · 11 comments
Open

检查102新版sac的命令的变化 #213

wangliang1989 opened this issue Apr 23, 2021 · 11 comments
Milestone

Comments

@wangliang1989
Copy link
Collaborator

102版的sac更新了一些命令,所以需要考察有哪些变化。主要有两类,一类是修正了一些bug,手册中一些例子做了规避bug的设计,现在已经没有必要了,应该修改。另一类是增加的新功能,我的计划是读取sac的官方介绍,逐一检查。向dev分支发PR。你们应该觉得可以吧?@seisman @core-man

@seisman
Copy link
Owner

seisman commented Apr 23, 2021

OK

@core-man
Copy link
Collaborator

good. hard work.

@wangliang1989
Copy link
Collaborator Author

wangliang1989 commented Jun 2, 2021

第一批检查的命令如下(就是我常用的命令),如果谁想做,先在这里at自己

@wangliang1989
Copy link
Collaborator Author

wangliang1989 commented Jun 7, 2021

新增的数据下载相关命令,如果谁想做,先在这里at自己

@core-man
Copy link
Collaborator

core-man commented Jun 7, 2021

打勾表示不需修改的吗?

@wangliang1989
Copy link
Collaborator Author

打勾表示不需修改的吗?

按照官方manual,没有变化。常用的基本就剩transfer了

@wangliang1989
Copy link
Collaborator Author

官方manual似乎没有记录bug修订,比如以前merge不支持通配符,现在支持了,官方manual没有说明。sac源代码的CHANGE文件有记录。@core-man

@core-man
Copy link
Collaborator

core-man commented Jun 7, 2021

那可以先解决官方手册里的改变,弄好后再回过头浏览change文件建立todo逐一检查。如何?

@wangliang1989
Copy link
Collaborator Author

那可以先解决官方手册里的改变,弄好后再回过头浏览change文件建立todo逐一检查。如何?

我差不多就是这样想的,但是在这之前,我想先解决那些常用的。常用的还剩4个,关键的就是transfer了。transfer内容很长,我还没看。

@wangliang1989
Copy link
Collaborator Author

我基础不行,你能不能处理transfer,我很怕我弄错@core-man

@core-man
Copy link
Collaborator

core-man commented Jun 7, 2021

我基础不行,你能不能处理transfer,我很怕我弄错@core-man

目前重点是101.6的发布,然后是102。对于102,我们先处理和审稿完你已经提交的 PR。

其实我也不太懂细节,都是在维护中学习,在学习中维护的。你可以的,而且还有大家一块审稿,不会错的。transfer 要是太复杂,我们可以先放放,回头再一块努力搞定。解决完已经提交的这几个 PR 后,可以先处理 change 文件,中途一块看看 transfer。等 SAC 101.6 和 GMT 6.2 发布后会更有精力看看 transfer。

话说最近 SAC, GMT 和 seismo-learn 文档最近进展太快了,审稿都要好久 😄

@core-man core-man pinned this issue Jun 11, 2021
@seisman seisman added this to the 5.0 milestone Jun 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants