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

Streamline array encoders #840

Open
paxcema opened this issue Feb 11, 2022 · 0 comments
Open

Streamline array encoders #840

paxcema opened this issue Feb 11, 2022 · 0 comments
Assignees
@paxcema
Labels
enhancement New feature or request help wanted Extra attention is needed refactor Some part of the code is working ok for now, but for various reasons should be refactored

Comments

@paxcema
Copy link
Member

paxcema commented Feb 11, 2022 •
edited

Once #833 is merged, We should look into simplifying the family of array-TS encoders. In particular, TsCatArrayEncoder and TsNumArrayEncoder should be refactored to inherit from ArrayEncoder (or the other two existing classes) instead of BaseEncoder.
@paxcema paxcema added the enhancement New feature or request label Feb 11, 2022
@paxcema paxcema added help wanted Extra attention is needed refactor Some part of the code is working ok for now, but for various reasons should be refactored labels Jan 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@paxcema paxcema

Labels
enhancement New feature or request help wanted Extra attention is needed refactor Some part of the code is working ok for now, but for various reasons should be refactored
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@paxcema