BF: conditioning on too much in forward stepwise, c(mathematical bug …

selective-inference / R-software

R software for selective inference

GNU General Public License v2.0

34 stars 18 forks source link

Closed jonathan-taylor closed 8 years ago

jonathan-taylor commented 8 years ago

…rather than code bug); also added a comment about one-sided pvalues

Not sure that all pvalues are always onesided but the ones using polyhedral seem to be by default.

jonathan-taylor commented 8 years ago

Test comment so that Jon gets an email

jonathan-taylor commented 8 years ago

Changing the number of rows of Gamma will probably introduce bugs in the fsInf function. This has to be checked before merging.