-
Notifications
You must be signed in to change notification settings - Fork 3
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
[WIP] Update dependencies #2
Conversation
Temporary workaround by installing |
Resolved compile errors related to |
let two = T::from(2.0).unwrap(); | ||
let three = T::from(3.0).unwrap(); | ||
let six = T::from(6.0).unwrap(); | ||
let two = T::from::<f64>(2.0).unwrap(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This cast seems necessary
src/umv/make.rs
Outdated
let yi = y.slice(s![.., 0]).insert_axis(Axis(1)); | ||
let coeffs = stack![Axis(1), dydx, yi]; | ||
let bla = s![.., 0 as usize]; | ||
let yi = y.slice(s![.., 0 as usize]).insert_axis(Axis(1)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@espdev , do you have an idea why this explicit cast is necessary?
706d87d
to
fc06f40
Compare
|
||
// cannot multiply `&CsMatBase<T, usize, Vec<usize>, Vec<usize>, Vec<T>>` by `T` | ||
// the trait `Mul<T>` is not implemented for `&CsMatBase<T, usize, Vec<usize>, Vec<usize>, Vec<T>>` | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The last issue blocking compilation: The Mul trait is not implemented for the CsMatBase I am not sure whether it can be implemented outside the sprs crate (I tried at the bottom of trait.rs
-- commented out. @espdev , any ideas?
Fixed all but one issue connected to sprs |
Hi @StefanUlbrich, Thank you for your work and participation in csaps-rs! Do I understand correctly that sprs version 0.11 does not work with csaps (the last issue)? |
is an external package mandatory? we only have band matrices so maybe this can be exploited? |
This compiles at least let a1 = qtwq.map(|el| s1 * *el );
let a2 = r.map(|el| *el * smooth); |
@StefanUlbrich |
Thanks! Yes, there is some movement in sprs. I still have an compiler error that I could not figure out on my own. I doubt that this is resolved. Could we have a look together, @espdev (my rust experience is still limited)? |
Sorry guys. Now there is not enough time for the project. I won't be able to participate for a couple of weeks. |
No worries @espdev . It seems that upgrading to sprs is blocked by sparsemat/sprs#278 ... using generic floats is an unresolved issue. It would not be a problem if we were dealing with |
@espdev , I managed to find a way around the issue with generic floats without the need to fix anything sprs (see sparsemat/sprs#278 (comment)). The package builds (!) and the unit tests pass except for one (and this seems to be a dimension issue, see below). There is still tons of comments and cleanup left, but it seems the issue is close to be finished.
|
@StefanUlbrich great news! |
Any idea on why the one unit test might fail (i.e., did it run on main)? |
@StefanUlbrich |
Fixed tests except test_evaluate_3d.
I have no idea why the test has failed. Array memory layout and strides do not match. This test has definitely worked before (with old ndarray/sprs versions). |
Open issues:
make.rs
,evaluate.rs
)