-
Notifications
You must be signed in to change notification settings - Fork 57
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
Feature Request: n_components=1 #82
Comments
Thank you for your interest in PaCMAP. I think that the algorithm will be able to run in the case when |
Hi! Thanks a lot for the quick reply. I can attest that so far simply removing the line works without crash. As I can't properly evaluate the quality of the dimension reduction in this scenario, what do you think of replacing the line by a warning that this has not been thoroughly evaluated / at the user's own risk? If you then released the new version that would simplify my reproduction instruction for my projects. The alternative is to instruct to users how to patch the lib. Thanks! |
hi @thiswillbeyourgithub and @hyhuang00. I have a particularly good test-scenario @thiswillbeyourgithub: sample RGB colorspace, dimension reduce to 1, and compare to the ordering (invariant to start position) that arises from the H (hue) channel of HSV ( I'm evaluating the source code and paper as well, to think through the implications of trying this. I'm really glad to know that code-wise, it worked. I'll try this on my branch. feature/unit-dimension |
Hi,
It seems setting n_components to 1 errors out because PaCMAP does not support dimensions less than 2.
I think it would be nice, as I was planning on testing it on repeng which is a small library that allows to nudge an LLM by injecting a 1D vectors that was created by dimension reduction of the internal representation of a dataset of pairs of good vs bad texts.
Is that something that will never happen or is there a way?
Thanks!
The text was updated successfully, but these errors were encountered: