-
Notifications
You must be signed in to change notification settings - Fork 16
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
feat: add Hex2Vec embedder #236
Conversation
… fragmentation, fix different dtypes in hex2vec transform
1164f94
to
34c7d9e
Compare
Codecov Report
@@ Coverage Diff @@
## main #236 +/- ##
==========================================
- Coverage 97.54% 96.28% -1.27%
==========================================
Files 55 60 +5
Lines 1632 1829 +197
==========================================
+ Hits 1592 1761 +169
- Misses 40 68 +28
Flags with carried forward coverage won't be shown. Click here to find out more.
|
… API, add test case generation code, test case files, and test for hex2vec embedder
…ransforming without fitting the model and using default encoder_sizes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really like current variables naming. It's really clear to follow 🥇
Add an example with some clustering or other visualizations (RGB) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add Hex2Vec to tests/miscellaneous/test_optional_dependencies.py
-> test_torch()
No description provided.