Skip to content

Commit

Permalink
Merge branch 'jean-pierreBoth:master' into master
Browse files Browse the repository at this point in the history
  • Loading branch information
jianshu93 authored Oct 23, 2024
2 parents c2199bf + 78fe65d commit 9b82388
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
5 changes: 4 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
# A data embedding tool and related data analysis or clustering

![](Annembed_logo-small.jpg)

<div align="center">
<img width="50%" src ="Annembed_logo-small.jpg">
</div>

The crate provides mainly in the form of a library (*See documentation of the binary embed for a small executable embedding data in csv files*):

Expand Down
4 changes: 2 additions & 2 deletions src/diffmaps.rs
Original file line number Diff line number Diff line change
Expand Up @@ -130,9 +130,9 @@ where
u.ncols()
);
}
let real_dim = asked_dim.min(u.ncols()) - 1;
let real_dim = asked_dim.min(u.ncols());
// we can get svd from approx range so that nrows and ncols can be number of nodes!
let mut embedded = Array2::<F>::zeros((u.nrows(), asked_dim));
let mut embedded = Array2::<F>::zeros((u.nrows(), real_dim));
// according to theory (See Luxburg or Lafon-Keller diffusion maps) we must go back to eigen vectors of rw laplacian.
// Appendix A of Coifman-Lafon Diffusion Maps. Applied Comput Harmonical Analysis 2006.
// moreover we must get back to type F
Expand Down

0 comments on commit 9b82388

Please sign in to comment.