-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Knockoffs(1/4): add comments and docstring of the functions #128
Open
lionelkusch
wants to merge
47
commits into
mind-inria:main
Choose a base branch
from
lionelkusch:PR_knockoffs
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
e202b84
Remove unecesarry argument
lionelkusch 82b829c
Change the a bit the behavior
lionelkusch d4f80c2
Update the variables
lionelkusch 47dbe24
Put all the knockoff test together
lionelkusch 81e64c2
Fix a bug
lionelkusch d6ded88
remove the function for estimating covariance
lionelkusch 801c5f9
Remove unnecessary file
lionelkusch 4c61dd7
Remove a function
lionelkusch b890cba
comparison with original code
lionelkusch b6fe948
improve docstring and function
lionelkusch 43a3e9a
Merge file for knockoff together
lionelkusch a116b53
Add function for repeat the gaussian knockoff
lionelkusch 1699675
Put all the test for knockoff in one file
lionelkusch db6098f
Include the new function in the init
lionelkusch e38f376
Fix bug
lionelkusch 8861480
Fix bugs
lionelkusch a0d53b0
Fix test for new signature of the function
lionelkusch 4bb9eb4
Improve the docstring
lionelkusch d21f46a
Improve docstring knockoff
lionelkusch f752f9e
/bin/bash: line 1: :wq: command not found
lionelkusch 071ea6d
Remove the begining of the file
lionelkusch 68dd7e6
Add equations
lionelkusch 727ee7d
Change reference for paper
lionelkusch efbe49a
add a reference
lionelkusch bfe111a
a a new tests
lionelkusch 211599e
rename function adn remove warning for test
lionelkusch beeda0a
Add parameters of knockoff
lionelkusch 3736661
Merge branch 'main' into PR_knockoffs
lionelkusch d066c5a
Format files
lionelkusch f6c699f
format file
lionelkusch bf621d2
Fix bugs
lionelkusch 779288b
Fix bug in utils
lionelkusch 20d0567
Update example knockoff
lionelkusch c71cc22
Formating
lionelkusch bbd3252
Update function
lionelkusch 9af787f
Apply suggestions from code review
lionelkusch 548e283
Fix name variables
lionelkusch 0ad893e
Fix name variables
lionelkusch 252a012
Fix test and name variables
lionelkusch b34cf1f
Add tests and fix bugs
lionelkusch 2ddfd9d
Add a tests and formating file
lionelkusch df45c82
Undo delete of the tests
lionelkusch e3eeb72
Improve coverage and test
lionelkusch 4df5d90
Format
lionelkusch 8aee168
Group the function agregation and not aggregation together
lionelkusch bfe6346
formating
lionelkusch 4e3f4d2
Replace lambda by alpha
lionelkusch File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are all these functions meant to be public ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, they should be public.