Skip to content

Conversation

rvasav26
Copy link
Collaborator

@rvasav26 rvasav26 commented Apr 10, 2025

Adding PCovC code, examples, and testing suite.

Contributor (creator of PR) checklist

  • Tests updated (for new features and bugfixes)?
  • Documentation updated (for new features)?
  • Issue referenced (for PRs that solve an issue)?

For Reviewer

  • CHANGELOG updated if important change?

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

@rvasav26
Copy link
Collaborator Author

Reopening pull request due to errors when pulling/pushing.

)

* Fixed `_ndim_coords_from_arrays` import

* Adding `validate_data` calls and tags for CUR, FPS, PCovCUR, and PCovFPS

* Adding `validate_data` calls and updated tags to Ridge2FoldCV

* Added `validate_data` calls to PCovR

* Added `validate_data` calls to KPCovR

* Fixing linting

* Fix rendering issues and depencies

* update pyproject.toml

* fix some tests

* update runners

* fix docs

* lint plus warnings as errors

* update infra

* Updates to fix sklearn branch (scikit-learn-contrib#241)

* replacing `self._validate_data` with `validate_data`

* Additional fixes

* Fixing scikit-learn warnings

* Fixing PCovR to work with 1D column vectors

* Fixing examples

* Changing shape of y in pcovr tests

* Fixing PCovR to not mess with data shape

---------

Co-authored-by: Christian Jorgensen <[email protected]>

* Update pyproject.toml

Co-authored-by: Rose K. Cersonsky <[email protected]>

* Removing print statement in tests/test_kernel_pcovr.py

Co-authored-by: Rose K. Cersonsky <[email protected]>

* Fixing artifact in WHO example

* Adding test for incorrect score threshold type

* Updating test

* Fixing validation checks in CUR + FPS

* Fix linting

* Update coverage collection

* Update changelog

---------

Co-authored-by: cajchristian <[email protected]>
Co-authored-by: Christian Jorgensen <[email protected]>
Co-authored-by: Rose K. Cersonsky <[email protected]>
@rosecers
Copy link
Collaborator

rosecers commented May 5, 2025

@rvasav26 #239 has been merged, so you can rebase on main and work from there.

@rvasav26 rvasav26 closed this May 14, 2025
@rvasav26 rvasav26 deleted the adding-pcovc branch May 14, 2025 01:19
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.

4 participants