Skip to content
Snippets Groups Projects

add "unchecked" versions of `Matrix::{vandermonde,from_vec_vec}` and test both matrices

Merged STEVAN Antoine requested to merge matrices into main

changelog

  • replace Matrix::vandermonde with Matrix::vandermonde_unchecked
  • add a new Matrix::vandermonde which calls Matrix::vandermonde_unchecked after checking the seed points are distinct, otherwise, gives a KomodoError::InvalidVandermonde error
  • same with Matrix::from_vec_vec and Matrix::from_vec_vec_unchecked
  • add documentation tests for the two "checked" functions
  • run the main lib tests on both a random and a Vandermond matrix, just to be sure we do not take advantage of the Vandermonde structure
Edited by STEVAN Antoine

Merge request reports

Merge request pipeline #4802 passed

Merge request pipeline passed for 92562823

Merged by STEVAN AntoineSTEVAN Antoine 1 year ago (Apr 15, 2024 8:11am UTC)

Loading

Pipeline #4803 passed

Pipeline passed for 5dc7dd2c on main

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading