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

Update for SAC 102.0 #149

Open
1 of 2 tasks
seisman opened this issue Sep 23, 2020 · 7 comments
Open
1 of 2 tasks

Update for SAC 102.0 #149

seisman opened this issue Sep 23, 2020 · 7 comments
Milestone

Comments

@seisman
Copy link
Owner

seisman commented Sep 23, 2020

SAC 102.0 was released recently. There are some big changes in the SAC file format, new features and bug-fixes. We need to add these new contents to the manual.

@sean0921
Copy link

sean0921 commented Dec 3, 2020

Related self-packaging script is updated:
https://github.com/sean0921/sac_debian_packager/tree/102.0-1%2Bsdp1.1

@core-man
Copy link
Collaborator

core-man commented Apr 19, 2021

Do we really want to update sac to 102? I think SAC101.6 is enough and we should use ObsPy first in the future (Originally posted in #155 (comment))? 我会建议老用户尽量使用 SAC 101.6,而不使用 SAC 102。

@wangliang1989
Copy link
Collaborator

Do we really want to update sac to 102? I think SAC101.6 is enough and we should use ObsPy first in the future (Originally posted in #155 (comment))? 我会建议老用户尽量使用 SAC 101.6,而不使用 SAC 102。

新老用户都应该用102

@core-man core-man pinned this issue Apr 22, 2021
@seisman seisman unpinned this issue Jun 9, 2021
@core-man core-man pinned this issue Jun 11, 2021
@seisman seisman added this to the 5.0 milestone Jun 11, 2021
@core-man
Copy link
Collaborator

core-man commented Jun 12, 2021

Shall we submit some PRs to dev branch so that the migration to SAC102 is smooth? Currently, dev only contains SAC102 installation. We might need to simply change 101.6 to 102.0 for the following files (grep 101.6 source/*/*rst):

  • https://github.com/seisman/SAC_Docs_zh/tree/master/source/install/*rst
  • https://github.com/seisman/SAC_Docs_zh/tree/master/source/basis/*rst
  • commands/cutim.rst
  • commands/merge.rst
  • commands/picks.rst
  • commands/plabel.rst
  • commands/plotpk.rst
  • commands/setdevice.rst
  • commands/sort.rst
  • commands/write.rst
  • macros/intro-to-sac-programming.rst
  • tools/sac-config.rst

Just want to cross-reference. @wangliang1989 is helping with SAC102 updating in #213 (comment) and #213 (comment), which need more work and revisions instead of simply changing 101.6 to 102.0. I may move some TODOs into the two comments.

@wangliang1989
Copy link
Collaborator

你的英文我不是完全看得懂

我觉得应该先让dev和master分支一致。然后我继续向dev分支发pr。这两步应该都会有冲突,分别解决。先让dev和master的合并是我希望102的内容在git历史中看上去是在最近发布的release的后面

@core-man
Copy link
Collaborator

core-man commented Jun 12, 2021

我觉得应该先让dev和master分支一致。然后我继续向dev分支发pr。这两步应该都会有冲突,分别解决。先让dev和master的合并是我希望102的内容在git历史中看上去是在最近发布的release的后面

是的,现在不能直接提交PR 并直接merge到master上。

我猜 @seisman 会很快在 #174 中让 dev 和 master 分支同步的。同步以后,#232#241 就可以逐步审稿merge到dev了。

@seisman
Copy link
Owner Author

seisman commented Jun 12, 2021

以后的 PR 会直接 merge 到 master 分支。

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

4 participants