-
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
modification on raylib, rf2depth #80
Conversation
rf2depth的一点优化,使用namedtuple改善了Station类的使用 depmodel.py 中的ccp_model太繁琐,未来会拆分重写。
rf2depth的一点优化,使用namedtuple改善了Station类的使用 depmodel.py 中的ccp_model太繁琐,未来会拆分重写。
rf2depth的一点优化,使用namedtuple改善了Station类的使用 depmodel.py 中的ccp_model太繁琐,未来会拆分重写。
我还想了解这个Test怎么在我的仓库里进行部署,这样我可以在pr之前先做一些测试。 |
rf2depth的一点优化,使用namedtuple改善了Station类的使用 depmodel.py 中的ccp_model太繁琐,未来会拆分重写。
rf2depth的一点优化,使用namedtuple改善了Station类的使用 depmodel.py 中的ccp_model太繁琐,未来会拆分重写。
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## dev #80 +/- ##
==========================================
+ Coverage 45.58% 45.67% +0.08%
==========================================
Files 40 40
Lines 5997 6016 +19
==========================================
+ Hits 2734 2748 +14
- Misses 3263 3268 +5 ☔ View full report in Codecov by Sentry. |
非常感谢,
|
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.
在这个修改中,我想知道为什么用Logger替代setuplog,以及为什么raytracing3d用大写?
Q1, Logger 的替代 |
我还是不能理解为什么这样做,比如为什么忽视setuplog的具体实现,为什么要在
在python语言规范中全大写变量名通常表示全局变量或常量,这不符合python规范。 |
我们并不需要一个大的更改,可以创建一个新的接口,seispy接口的几个层次还是比较明显的。 我觉得您对于finalist和cfg问题的描述也很到位。有几点我认为可以补充, |
|
|
首先作为python module必须保证pythonic语言规范,你可以用特定的前缀来表示这些变量。因为很多用户都在使用 |
关于raylib的内容,我有一些不太清楚的地方。即seispy里rayp 的单位。 obspy.taup和matlab taup输出的似乎都是s/rad, 这里我转换到了s/deg,与教程网站里给出文件的数量级达到一致。但是在程序内部,似乎会将s/deg的rayp 转换为s/rad的结果进行计算? 我还没细看过test?.py 。里面有很多不给出参数的调用,我感觉很奇怪就没细看。程序还是要一个数值上的检验比较靠谱些,这也是我选择doctest 的原因之一。 |
非常赞同您对规范的支持,我对python 规范还只有一个比较模糊的概念。为写出更优质的代码努力! 想了解您对其他改动的态度。比如namedtuple的使用 和 Station类的修改 yaml文件我用过,是我看错了。 之前配置过一个用xml配置参数的测地学软件,印象太过深刻。 |
我个人的习惯是用s/km并且在
这也是个历史遗留问题,对于函数适合用doctest,但对于一个完整的计算流程,很难定义一个数值上的比较。 |
是seispy主页上下载的预先计算的rayplib。 关于rayplib.py 未来可能有一个功能,即导入自定义速度模型。我还没考虑好怎么写接口,tvel 和DepModel使用的文本格式还是有一些区别。 |
rayplib的优化,提供了命令行选项
rf2depth的一点优化,使用namedtuple改善了Station类的使用
depmodel.py 中的ccp_model太繁琐,未来会拆分重写。
几个失败的尝试:
a. 使用namedtuple存储RF2depth 的结果
d. classmethod depth_model.ccp_xxx,设计得太过繁琐,未来考虑拆分
几个未来的变化想法:
b. finalist.dat 格式的改进。
c. 批量生成剖面,
a. namedtuple,相比于使用dict.add有速度和规范上的优势,但经过numpy.save后会拆解为tuple,只保留变量顺序。改动太大,跳过先。
b. finalist.dat细化:
saclst
)去生成这些文件。c. 批量生成剖面的细化