Skip to content

Issue/711/jk cov kmeans - #712

Draft
Kabelo-T wants to merge 2 commits into
mainfrom
issue/711/jk_cov_kmeans
Draft

Issue/711/jk cov kmeans#712
Kabelo-T wants to merge 2 commits into
mainfrom
issue/711/jk_cov_kmeans

Conversation

@Kabelo-T

@Kabelo-T Kabelo-T commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

Description

fixes #711

Main changes

  • implement kmeans clustering to partition sky for jackknife covariance calculation

Checklist

Besides passing all CI checks and coverage is at 100%, make sure you also checked the following items
(check details in CONTRIBUTING).

For developers

  • Notebooks: notebooks related to this PR have been updated and all notebooks can run correctly.
  • Build the documentation: All documentation builds correctly.

For reviewers

  • Notebooks: notebooks related to this PR have been updated and all notebooks can run correctly.
  • Build the documentation: All documentation builds correctly.

For developers (part 2)

After the PR has been approved by two reviewers:

  • Update the code version in clmm/__ini__.py.
  • Keep only relevant points in the squash and merge commit message.
  • If any dependencies have been altered, update environment.yml, pyproject.toml, INSTALL.md, and README.md. A maintainer should also be notified to change the requirements on conda-forge.
  • Update clmm/theory/_ccl_supported_versions.py and clmm/theory/ccl.py if pyccl's version constraints have been altered.

@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 90.432% (-9.6%) from 100.0% — issue/711/jk_cov_kmeans into main

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.

CLMM jk covariance partitions sky using healpix subdivisions, want to compare to k-means clustering

2 participants