Skip to content

Sketchmap#284

Draft
sofiia-chorna wants to merge 18 commits into
scikit-learn-contrib:mainfrom
sofiia-chorna:sketchmap
Draft

Sketchmap#284
sofiia-chorna wants to merge 18 commits into
scikit-learn-contrib:mainfrom
sofiia-chorna:sketchmap

Conversation

@sofiia-chorna

@sofiia-chorna sofiia-chorna commented Jan 14, 2026

Copy link
Copy Markdown
Contributor

chatty-based reimplementation of sketchmap
tested with the MAD test with original smap, the result is quite similar

for now, there is only fit_transform method, for out-of-distribution data user needs to either do out-of-sample embedding or train a MLP (as done in MAD paper!).

also, i tried to do the function that suggests parameters from https://sketchmap.org/index.html?page=tuts&psub=analysis but should be refined!

TODO:

  • Check similarity with original sketchmap results
  • Clean up code
  • Clean up example
  • Add tests?

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--284.org.readthedocs.build/en/284/

Comment thread tests/test_sketchmap.py

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should at least add a regression test maybe based on a random dataset.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice idea!

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.

2 participants