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.

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

change: Implement encoder registry system #106

Open
mathysgrapotte opened this issue Feb 19, 2025 · 0 comments
Open

change: Implement encoder registry system #106

mathysgrapotte opened this issue Feb 19, 2025 · 0 comments

Comments

@mathysgrapotte
Copy link
Owner

Is your change request related to a problem? Please describe.

Encoders currently use hardcoded class names in YAML configs.

Describe the solution you'd like

  1. Create src/stimulus/data/encoding/registry.py
  2. Implement EncoderRegistry subclassing BaseRegistry
  3. Add decorators to existing encoders:
 @encoder_registry.register("text_onehot")
 class TextOneHotEncoder: ...
  1. Update YAML references from class names to registry keys
@mathysgrapotte mathysgrapotte moved this to Todo - mid issues in Stimulus v1.0 Feb 19, 2025
@mathysgrapotte mathysgrapotte moved this from Todo - mid issues to Todo - depend on other issues in Stimulus v1.0 Feb 19, 2025
@mathysgrapotte mathysgrapotte moved this from Todo - depend on other issues to Todo in Stimulus v1.0 Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

No branches or pull requests

1 participant