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.

Sign up for GitHub

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

Jump to bottom

Issue 188 #197

Open
wants to merge 2 commits into
base: issue-170
Choose a base branch
from
Open

Issue 188 #197

wants to merge 2 commits into from

Conversation

icui
Copy link
Collaborator

@icui icui commented Nov 27, 2024 •
edited
Loading

Description

Call base specfem workflow from Python
Input of the function is argv, e.g.

from specfempp._core import _run
 _run(['', '-p','specfem_config.yaml'])

The function is named _run because it's not intended to be used directly. Instead, the Python library will call _run internally.

Issue Number

#188

Checklist

Please make sure to check developer documentation on specfem docs.

  • I ran the code through pre-commit to check style
  • My code passes all the integration tests
  • I have added sufficient unittests to test my changes
  • I have added/updated documentation for the changes I am proposing
  • I have updated CMakeLists to ensure my code builds
  • My code builds across all platforms

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lsawade lsawade lsawade approved these changes

@Rohit-Kakodkar Rohit-Kakodkar Awaiting requested review from Rohit-Kakodkar

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@icui @lsawade