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

Update testing #63

Closed
wants to merge 12 commits into from
Closed

Update testing #63

wants to merge 12 commits into from

Conversation

Simkern
Copy link
Collaborator

@Simkern Simkern commented Mar 20, 2024

  • preprocessor pass + macros added for TestExpm
  • Test_utils module created for simplified initialization and data handling in tests
  • print_mat subroutine added to utils for debugging purposes
  • all tests cleaned and test utils deployed.

@Simkern Simkern requested a review from loiseaujc March 20, 2024 14:26
src/Utils.f90 Show resolved Hide resolved
@loiseaujc
Copy link
Member

Everything looks good. Only issue is that you sent a PR to main rather than dev. Let's take the habit of sending PR to dev first and will do the whole PR into main afterward just to keep everything in sync.

@Simkern
Copy link
Collaborator Author

Simkern commented Mar 20, 2024

Sure, let's go to dev first, then to main.

@Simkern Simkern closed this Mar 20, 2024
@loiseaujc loiseaujc deleted the update_testing branch March 21, 2024 10:04
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