GitLab

JOSS Submission

See merge request tpurcell/cpp_sisso!33

Use inner_product of Standardized Values instead of r for calculating overlap

See merge request tpurcell/cpp_sisso!37

Unit test now works

Prevent similar errors to what was seen with remove duplicates

removing duplicates did not remove every duplicate now it does
Also standarized the function somewhat

This is now explicitly tested

cause failure in tests

The bug shown was argsort on a vector of size 0

Hopefully will remove the data overwrite issue

Based on Luca's comments

May fix the possible overwrite error

Not appearing on docker image

The test failed because SIS was too large

Correct bugs when attempting to allow additional features when max_cor < 1

They don't show up on the docker image

use stand_val everywhere
correct condition for pearson

Something is wrong with the comparision functions

Fails when tested thought

Will be used for comp_feats

Will be used to make calculation of cross-correlation faster

Also add the relevant ciatations

Makes some of the comparisions easier

Modularize SISSOSolver

See merge request tpurcell/cpp_sisso!36

Use one fit/l0_regularization but add helper functions for the differences

Minor changes to LossFunctionConvexHull as a result (score for inds *= n_class * n_samp)

gnu compilers should now pass checks for featur space size
Corrected where phi.txt is outputted to

If max_rung==0 is not None

Postfix was done correctly, issue was sorting of the phi_temp from the file