Skip to content
This repository has been archived by the owner on Oct 24, 2023. It is now read-only.

SKLEARN: Add TRAIN_TEST_SPLIT #246

Closed
wants to merge 3 commits into from
Closed

SKLEARN: Add TRAIN_TEST_SPLIT #246

wants to merge 3 commits into from

Conversation

jjerphan
Copy link
Contributor

Just a initial node to setup the usage of scikit-learn.

Description

  • I've included a concise description of what each node does

Styleguide

  • My node adheres to the styleguide for Flojoy nodes

Docs

  • I've submitted a PR for a documentation page for the new node(s) that contains usage examples (see docs.flojoy.io)

Testing

  • This PR includes a unit test (example here and/or ideally a screenshot of the node's output on an example app.

@jjerphan
SKLEARN: Add TRAIN_TEST_SPLIT
e864fbd 
Just a initial node to setup the usage of scikit-learn.

Signed-off-by: Julien Jerphanion <git@jjerphan.xyz>
@jjerphan
fix: Format with black
365fde9 
Signed-off-by: Julien Jerphanion <git@jjerphan.xyz>
@jjerphan
Add type annotation
218b6fa 
Signed-off-by: Julien Jerphanion <git@jjerphan.xyz>
@jjerphan jjerphan changed the base branch from main to develop August 29, 2023 14:52
@jjerphan
Copy link
Contributor Author

jjerphan commented Sep 7, 2023 •
edited
Loading

Closing as TRAIN_TEST_SPLIT is already present as a Node and as flojoy-ai/studio#823 is not a priority at the moment.

@jjerphan jjerphan closed this Sep 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jjerphan