EpistasisLab / EpistasisLab/Aliro

Add OneHotEncoder and OrdinalEncoder

Open
#588 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
JavaScript
Stars
238
Forks
60
PR merge metrics
No merged PRs in 30d

Description

Add a OneHotEncoder and OrdinalEncoder. In a previous commit (# 9edbbce) these were removed. We should put back the Encoders in the validateDataset step to allow uploading datasets with ordinal and categorical columns.

This functionality was part of Aliro prior to the 9edbbce commit. But, experiments could not be run. It seems the Encoders were never implemented when running experiments. So, this issue will require 2 steps:

1. Put back the encoders in the validateDataset step
2. Perform encoding when a new experiment is launched.

Testing of this feature can be done with the datasets in data/datasets/test/integration/

Once this is implemented, we'll need to put back the skipped unittests in learn_tests.py and test_validateDataset.py

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.