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

Optional model name #662

Merged
merged 56 commits into from
Jul 6, 2023
Merged

Optional model name #662

merged 56 commits into from
Jul 6, 2023

Conversation

afard
Copy link
Collaborator

@afard afard commented Jul 4, 2023

Please review this PR after the one titled "Overwrite model parameter" is merged (#660).

This PR resolves #618.

afard and others added 30 commits August 5, 2020 16:55
@afard afard requested review from oualib and mail4umar July 4, 2023 02:59
@afard afard self-assigned this Jul 4, 2023
@oualib oualib added the Machine Learning - System Anything related to ML init or system... label Jul 5, 2023
…el_name

Resolved Conflicts:
	verticapy/machine_learning/model_selection/kmeans.py
	verticapy/machine_learning/vertica/automl/dataprep.py
	verticapy/machine_learning/vertica/base.py
	verticapy/machine_learning/vertica/cluster.py
	verticapy/machine_learning/vertica/decomposition.py
	verticapy/machine_learning/vertica/ensemble.py
	verticapy/machine_learning/vertica/linear_model.py
	verticapy/machine_learning/vertica/naive_bayes.py
	verticapy/machine_learning/vertica/neighbors.py
	verticapy/machine_learning/vertica/preprocessing.py
	verticapy/machine_learning/vertica/svm.py
	verticapy/machine_learning/vertica/tree.py
	verticapy/tests/vModel/test_countvectorizer.py
	verticapy/tests/vModel/test_dbscan.py
@oualib oualib merged commit d03b250 into vertica:master Jul 6, 2023
8 checks passed
@afard afard deleted the optional_model_name branch September 13, 2023 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Machine Learning - System Anything related to ML init or system...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Making the name parameter optional for declaring a model object
2 participants