Skip to content

Use new matrix! macro in unit tests #54

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
Closed
AtheMathmo opened this issue Sep 26, 2016 · 1 comment
Closed

Use new matrix! macro in unit tests #54

AtheMathmo opened this issue Sep 26, 2016 · 1 comment
Labels
easy

Comments

@AtheMathmo
Copy link
Owner

With #47 merged we now have a macro for small matrices! This macro helps to enforce at compile time that our dimensions are correct for our input data and is stricter than the existing new constructor. Because of this we should use this for future tests and also convert existing tests to use this.
gcollura added a commit to gcollura/rulinalg that referenced this issue Oct 24, 2016
@gcollura
Use macro matrix! in tests. Solve issue AtheMathmo#54.
5ad08de
@gcollura gcollura mentioned this issue Oct 24, 2016
Merged
AtheMathmo pushed a commit that referenced this issue Oct 26, 2016
@gcollura @AtheMathmo
Use macro matrix! in tests. Solve issue #54. (#73)
62d28ce 
* Use macro matrix! in tests. Solve issue #54.

* Use new macro syntax whenever possible

* Use traditional constructor on given matrices
@AtheMathmo
Copy link
Owner Author

Thanks @gcollura !

theotherphil pushed a commit to theotherphil/rulinalg that referenced this issue Dec 4, 2016
@AtheMathmo
Implementing OpAssign overloading and tests (AtheMathmo#54)
de9d06e 
* Implementing OpAssign overloading and tests

* Fixing minor issue with test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
easy
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@AtheMathmo