Skip to content

ESIGMAHM integration#251

Merged
md-arif-shaikh merged 5 commits into
vijayvarma392:mainfrom
Akash-Maurya-0899:esigma
Apr 15, 2026
Merged

ESIGMAHM integration#251
md-arif-shaikh merged 5 commits into
vijayvarma392:mainfrom
Akash-Maurya-0899:esigma

Conversation

@Akash-Maurya-0899
Copy link
Copy Markdown
Contributor

This PR adds:

  1. a demo notebook for using gw_eccentricity with ESIGMAHM
  2. gw_eccentricty posterior postprocessing scripts and notebooks for ESIGMAHM

@md-arif-shaikh
Copy link
Copy Markdown
Collaborator

The notebooks under examples (not under examples/postprocessing) are run with github actions. ESIGMAHM note book examples/egw_from_ESIGMAHM_demo.ipynb is failing because it can't find esigmapy. There are two ways to fix it. First one is harder, you have to install it in the workflow. See here for example. The second one is to make the workflow skip this particular notebook for now. It is still usefull to have it in the examples directory. To skip, add your notebook to this list here.

@md-arif-shaikh
Copy link
Copy Markdown
Collaborator

Also, please add your example notebook in the list of notebooks in the README file.

@Akash-Maurya-0899
Copy link
Copy Markdown
Contributor Author

The notebooks under examples (not under examples/postprocessing) are run with github actions. ESIGMAHM note book examples/egw_from_ESIGMAHM_demo.ipynb is failing because it can't find esigmapy. There are two ways to fix it. First one is harder, you have to install it in the workflow. See here for example. The second one is to make the workflow skip this particular notebook for now. It is still usefull to have it in the examples directory. To skip, add your notebook to this list here.

I have gone with the easier, second option for now.

@md-arif-shaikh md-arif-shaikh merged commit b90a896 into vijayvarma392:main Apr 15, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants