Skip to content
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

Update who feature selection example #212

Merged
merged 1 commit into from
Aug 24, 2023
Merged

Update who feature selection example #212

merged 1 commit into from
Aug 24, 2023

Conversation

agoscinski
Copy link
Collaborator

@agoscinski agoscinski commented Aug 22, 2023

The plot has been updated with the revision of the ore paper. See corresponding plot in the ore repo https://github.com/victorprincipe/skmatter-ore/blob/main/paper/WhoDataset-Selection.ipynb

  • change kernel ridge to linear regression

  • add correlation plot that shows similarity between the selector methods


📚 Documentation preview 📚: https://scikit-matter--212.org.readthedocs.build/en/212/

@agoscinski agoscinski changed the title Ppdate who feature selection example Update who feature selection example Aug 22, 2023
The plot has been updated with the revision of the ore paper

* change kernel ridge to linear regression

* add correlation plot that shows similarity between the selector
  methods
@agoscinski
Copy link
Collaborator Author

@ceriottm
Copy link
Collaborator

Observation for both the notebook and the paper: I would only show the lower-left corner, and also not show the diagonal. Those are redundant.

@agoscinski
Copy link
Collaborator Author

agoscinski commented Aug 23, 2023

Updated it in the paper but I would keep this redundancy for the example. Redundancy can help with understanding. Also making such a plot with matplotlib would make the code more complicated which also reduces readability.

@ceriottm
Copy link
Collaborator

Updated it in the paper but I would could this redundancy for the example. Redundancy can help with understanding. Also making such a plot with matplotlib would make the code more complicated which also reduces readability.

OK I see the point of keeping the code clean in the example (although in many examples we have pretty sophisticated matplotlib layouts). Good with me then

Copy link
Collaborator

@ceriottm ceriottm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good with me.

@ceriottm ceriottm merged commit 2a8ffdc into main Aug 24, 2023
8 checks passed
@ceriottm ceriottm deleted the update-who-example branch August 24, 2023 05:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants