Fix initialize_tucker

[hello-fri-end]

#518

[codecov]

Codecov Report

Merging #519 (4f8a945) into main (a982cf9) will decrease coverage by 0.08%.
Report is 4 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #519      +/-   ##
==========================================
- Coverage   86.78%   86.70%   -0.08%     
==========================================
  Files         118      118              
  Lines        7515     7515              
==========================================
- Hits         6522     6516       -6     
- Misses        993      999       +6     

Files Changed	Coverage Δ
tensorly/decomposition/_tucker.py	87.91% <ø> (ø)
tensorly/decomposition/tests/test_tucker.py	100.00% <ø> (ø)

... and 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[JeanKossaifi]

Thanks @hello-fri-end! Can you just fix the formatting and looks good to merge.

[hello-fri-end]

Thanks @hello-fri-end! Can you just fix the formatting and looks good to merge.

Fixed the formatting. Thanks :)

[JeanKossaifi]

Great, thanks for the fix @hello-fri-end, merging!
As a side note, I wrote a better init in TensorLy-Torch) we should probably bring it in at some point, it allows the reconstruction to have zero mean and given standard deviation - I opened #521