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.
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
Desparsified lasso(1/4): add comments and docstring of the functions #127
base: main
Are you sure you want to change the base?
Desparsified lasso(1/4): add comments and docstring of the functions #127
Changes from 26 commits
99fa174
3eebe97
ffbb6e9
f92ea4d
d465c36
eeae34b
d23e3d6
4ad785c
4787c27
97ff04f
4cdaf90
931e52f
d3d91e8
92c7f57
8366e70
e81e3db
9052326
7e4435a
8068e89
b4055a3
109dcd2
0ca0ec1
977cd30
f25cc8c
f304a5e
aa44640
ab0ebd5
9081811
e2dd939
aea49d6
4b376c7
6c9989a
0511cc6
942a508
6001f9b
ef5ec38
c5a070f
9a099b9
e9ba724
44985a1
367d0df
0d09b9b
7c6ada3
9dbebc4
91c8b67
1b381a5
f9f6f0f
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
why do you remove the memory argument ?
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.
This argument is for optimisation of the calculation by memorising the results of a call of a function with the same arguments. I don't think the basic user requires it and I don't take the time to look in detail if it's very efficient.
I think that it's interesting when the function is run multiple times on the same data but I don't think that it's important to keep it for the moment because it should be the case.
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.
Same comment: why get rid of memory ?
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.
see above