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

Some fixes for comments during JOSS reviews #39

Merged
merged 4 commits into from
Feb 7, 2025
Merged

Some fixes for comments during JOSS reviews #39

merged 4 commits into from
Feb 7, 2025

Conversation

prashjha
Copy link
Owner

@prashjha prashjha commented Feb 7, 2025

Fixes #35 #36

Partially fixes #37

Proposed Changes

  • update example files. To avoid keeping two separate example scripts, one for test and other for example, we now have examples copied from test/test_data/peridem folder with minor changes in time steps and number of outputs.
  • update README.md and remove hyperlinks in section headings.

@prashjha prashjha merged commit 8599859 into main Feb 7, 2025
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.

My points for the JOSS review Failing Test: Two-particle with wall
1 participant