Skip to content

Update tests and make github workflow checkout MDANSE examples and runs scripts. - #1278

Merged
ChiCheng45 merged 5 commits into
protosfrom
chi/update-unit-tests
Aug 12, 2026
Merged

Update tests and make github workflow checkout MDANSE examples and runs scripts.#1278
ChiCheng45 merged 5 commits into
protosfrom
chi/update-unit-tests

Conversation

@ChiCheng45

@ChiCheng45 ChiCheng45 commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Description of work
Added SSF3D regression tests and made the GitHub workflow check out MDANSE examples, run the scripts, and check that the outputs exist.

Closes ISISNeutronMuon/MDANSE-Examples#6

Skips script2_vhfd.py in tutorial 2 as it can take quite some time.

To test
Tests should pass.

@ChiCheng45
ChiCheng45 requested a review from oerc0122 August 12, 2026 13:46
@ChiCheng45 ChiCheng45 changed the title Update test and make github workflow checkout MDANSE examples and runs scripts. Update tests and make github workflow checkout MDANSE examples and runs scripts. Aug 12, 2026

@oerc0122 oerc0122 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

Comment thread .github/workflows/tests.yml Outdated
Comment thread .github/workflows/tests.yml Outdated
path: MDANSE-Examples

- name: Run tutorial 1 scripts
working-directory: MDANSE-Examples/tutorials/tutorial1-phase-transition/mdanse_inputs

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it be easier to add a marimo notebook to the examples and run that at some pint?

@ChiCheng45 ChiCheng45 Aug 12, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've never used Marimo notebooks before, but they look nice, could be a possibility for the future.

@ChiCheng45
ChiCheng45 merged commit f39f695 into protos Aug 12, 2026
48 checks passed
@ChiCheng45
ChiCheng45 deleted the chi/update-unit-tests branch August 12, 2026 15:47
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.

Added nightly GitHub actions tests.

2 participants