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

16 error occurs in the simple fwi example #17

Merged
merged 18 commits into from
Nov 5, 2024

Conversation

AmirMardan
Copy link
Owner

This pull request includes multiple changes to the PyFWI project to enhance documentation, update dependencies, and improve code readability and functionality. The most important changes include adding a GitHub Actions workflow for CI, updating documentation, and refactoring code for better readability.

CI/CD Improvements:

  • Added a GitHub Actions workflow to install Python dependencies, run tests, and lint with multiple Python versions (.github/workflows/pr.yml).

Documentation Updates:

  • Added a link to the documentation in the README.md file.
  • Added a CNAME file for ReadTheDocs configuration (docs/CNAME).
  • Updated the release version in docs/conf.py from 0.1.9 to 0.1.10.
  • Enhanced the index documentation with additional links and contact information (docs/index.rst).

Code Refactoring:

  • Refactored the FWI class initialization for better readability (src/PyFWI/fwi.py).
  • Refactored the TimeLapse class and its methods for improved clarity (src/PyFWI/tl_fwi.py). [1] [2] [3]
  • Removed unused set_env_variable parameter from WavePreparation and WavePropagator classes (src/PyFWI/wave_propagation.py). [1] [2] [3]

Dependency Updates:

  • Removed unnecessary dependencies segyio and logging from pyproject.toml.
  • Updated the version in setup.py from 0.1.9 to 0.1.10.

Example and Test Code:

  • Added a new example script for surface FWI (example/fwi_example_surface.py).

@AmirMardan AmirMardan linked an issue Nov 5, 2024 that may be closed by this pull request
@AmirMardan AmirMardan added bug Something isn't working enhancement New feature or request labels Nov 5, 2024
@AmirMardan AmirMardan merged commit 93e6b88 into main Nov 5, 2024
9 checks passed
@AmirMardan AmirMardan deleted the 16-error-occurs-in-the-simple-fwi-example branch November 5, 2024 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

error occurs in the “Simple FWI Example”
1 participant