Conversation
multiple updates for 1D inversion/performance/new example/usability etc.
There was a problem hiding this comment.
Pull Request Overview
This PR updates the yaml-cpp submodule to a new commit and expands the README with a structured list of publications and new citations.
- Bump
yaml-cppsubmodule to commit2f86d13775d119edbb69af52e5f566fd65c6953b - Add TomoATT software package citation and reorganize publication list under bold headings
- Introduce additional references grouped by tomography type
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| external_libs/yaml-cpp | Updated submodule commit hash |
| README.md | Inserted new citation for TomoATT, added headings for each tomography category, and added multiple new paper references |
Comments suppressed due to low confidence (1)
README.md:13
- [nitpick] To maintain consistent list formatting, prefix this heading with a hyphen (e.g.,
- **The TomoATT software package**) or adjust indentation to match surrounding list items.
- **The TomoATT software package**
README.md
Outdated
There was a problem hiding this comment.
[nitpick] For grammatical consistency, change "Spherical coordinate" to plural "Spherical coordinates".
| - **Regional tomography in Spherical coordinate** | |
| - Chen, J., Chen, G., Nagaso, M., & Tong, P. (2023). Adjoint-state traveltime tomography for azimuthally anisotropic media in spherical coordinates. Geophysical Journal International, 234(1), 712-736. [DOI](https://doi.org/10.1093/gji/ggad093). | |
| - **Teleseismic tomography in Spherical coordinate** | |
| - **Regional tomography in Spherical coordinates** | |
| - Chen, J., Chen, G., Nagaso, M., & Tong, P. (2023). Adjoint-state traveltime tomography for azimuthally anisotropic media in spherical coordinates. Geophysical Journal International, 234(1), 712-736. [DOI](https://doi.org/10.1093/gji/ggad093). | |
| - **Teleseismic tomography in Spherical coordinates** |
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
JingChen-Thu
left a comment
There was a problem hiding this comment.
Thank you very much for updating the source of newer YMAL branch and also the citation list.
|
@JingChen-Thu The yaml-cpp has been updated. Please take time to test whether all parameters are read correctly. |
Got it, I will test it later and give feedback to you. |
No description provided.