We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
n_components
Harmony has a n_components parameter, which, according to the docstring:
:param pc_components: Minimum number of principal components to use. Specify `None` to use pre-computed components
That value is used for utils.run_pca, but it's not passed over to scanpy's neighboor computation, see
utils.run_pca
Harmony/src/harmony/core.py
Line 67 in eca0771
So I wonder what the significance of that parameter actually is?
Also, I find the default value of 1000 a bit high, as scanpy's default here is much smaller, 50 I believe
1000
50
The text was updated successfully, but these errors were encountered:
The n_components parameter is used below
Line 48 in eca0771
In line 67, temp is already pre-computed principal components and thus the parameter is not passed for nearest neighbor computation.
temp
Sorry, something went wrong.
No branches or pull requests
Harmony has a
n_components
parameter, which, according to the docstring::param pc_components: Minimum number of principal components to use. Specify `None` to use pre-computed components
That value is used for
utils.run_pca
, but it's not passed over to scanpy's neighboor computation, seeHarmony/src/harmony/core.py
Line 67 in eca0771
So I wonder what the significance of that parameter actually is?
Also, I find the default value of
1000
a bit high, as scanpy's default here is much smaller,50
I believeThe text was updated successfully, but these errors were encountered: