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

WIP : adding in shaply values #221

Closed
wants to merge 1 commit into from
Closed

Conversation

mortonjt
Copy link

@mortonjt mortonjt commented Oct 31, 2022

This is a preliminary pull request to add shapely values #219
Things will probably break for now.

To install this branch, run pip install git+https://github.com/mortonjt/q2-sample-classifier.git
Of course, make sure to run qiime dev refresh-cache after installation.

@mortonjt mortonjt changed the title adding in shaply values WIP : adding in shaply values Oct 31, 2022
@lizgehret lizgehret linked an issue Nov 1, 2022 that may be closed by this pull request
@lizgehret
Copy link
Member

Hi @mortonjt, thanks for your work on this! Is this something you'd like to see in the 2022.11 release? Our PR submission cutoff is this Friday, so we can get this added to the board if you think it'll be ready for review by Nov. 10th (our PR merge cutoff date).

@lizgehret lizgehret added the type:improvement Making something better. label Nov 1, 2022
@mortonjt
Copy link
Author

mortonjt commented Nov 2, 2022 via email

@lizgehret
Copy link
Member

@mortonjt sounds good! I'll get this added to our future release board. Let us know when it's ready for review. 🙂

@jindongmin
Copy link

jindongmin commented Nov 7, 2022

Hi, @mortonjt and I made this work, it's available here https://github.com/jindongmin/q2-sample-classifier.git. Thanks!

@lizgehret
Copy link
Member

Hi @jindongmin thanks for the update! Can you either add your changes to this PR or open up a new one for our team to review?

@mortonjt just FYI we ended up pushing back our 2022.11 release until December 15th, so we could still add this to that release if things are almost ready here!

@lizgehret
Copy link
Member

Hey @mortonjt @adamovanja we're going to go ahead and close this PR as well, in conjunction with this PR that we closed recently. Feel free to reach out if either of you have the bandwidth and interest to pick them back up though!

@lizgehret lizgehret closed this Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:improvement Making something better.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for SHAP
3 participants