Unused of parameter full_rank

[lionelkusch]

In the file hidimstat/adaptive_permutation_threshold.py, the function _manual_inversion has a parameter full_rank. This parameter is not used in the main function ada_svr (line 39).

I propose to remove because it's actually a dead code. Can you check if there are not too many assertions and if I miss some assertions?

[lionelkusch]

I remove the VallueError because it's not possible to access it due to the operation of np.dot(X, X.T) on line 39. The only line using this function.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.15%. Comparing base (7571832) to head (3a920ea).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #58      +/-   ##
==========================================
+ Coverage   77.11%   77.15%   +0.03%     
==========================================
  Files          46       46              
  Lines        2465     2460       -5     
==========================================
- Hits         1901     1898       -3     
+ Misses        564      562       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[bthirion]

LGTM.
Is thsis part of the code base tested ?

[lionelkusch]

There were not tests for this part of the code. I found the dead code by trying to test this code through the main function ada_svr.

[bthirion]

I suggest to take this opportunity to add a test. best,

[lionelkusch]

Do you want to remove it or keep it and add some tests for it?

[bthirion]

My feeling is that we should add a test.

[lionelkusch]

The pull request removes lines that cannot be tested via the main function.
Consequently, testing these lines makes no sense.
On the other hand, if we keep them, I can test the private function.