Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add ColumnWiseEncoder #232

Open
yiweny opened this issue Nov 20, 2023 · 1 comment
Open

Add ColumnWiseEncoder #232

yiweny opened this issue Nov 20, 2023 · 1 comment
Assignees
@weihua916
Labels
1 - Priority P1 feature

Comments

@yiweny
Copy link
Contributor

yiweny commented Nov 20, 2023

So we can assign a different encoder for different columns.
@weihua916 weihua916 self-assigned this Nov 21, 2023
@weihua916
Copy link
Contributor

Thanks for raising this! Let me take on this. I think a promising approach is to add an argument, col_encoder_dict, in StypeWiseEncoder so that it uses the given StypeEncoder for specific columns while using the default StypeEncoder (specified in stype_encoder_dict) for the rest.

We can support ColumnWiseEncoder, but it sounds a bit overkill to specify the encoder for each column name.

@weihua916 weihua916 mentioned this issue Nov 28, 2023
Merged
weihua916 added a commit that referenced this issue Nov 28, 2023
@weihua916
Add get_col_feat to TensorFrame (#244)
d2b10df 
Towards column-wise encoder:
#232

Also fixed `concat`, `save`, `load` utils for `MultiEmbeddingTensor`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@weihua916 weihua916

Labels
1 - Priority P1 feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@weihua916 @yiweny