-
Notifications
You must be signed in to change notification settings - Fork 3
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
Fixes on correlation of PCA and UMAP components to computed_feature script #159
Conversation
@Soorya19Pradeep do you expect more work on this branch? |
Yes @ziw-liu . I am going through, correcting the feature extraction methods |
@ziw-liu, what is the best way to combine the |
You can put the fluorescence features behind a condition that only executes when there is a second channel. |
@Soorya19Pradeep we decided to not use UMAP for feature correlation. Can you update this PR to reflect that? |
I am adding small bug fixes on the script for computing the correlation between PCA and UMAP features and the computed features on this branch.