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

Adding some accessors to allow deeper model equality checks #183

Merged
merged 1 commit into from
Oct 18, 2021

Conversation

Craigacp
Copy link
Member

Description

Added accessors for TreeModel to get the root node(s) so they can be compared for equality using their equals implementations. Added an equality method for two svm_models as there isn't one on svm_model. Also fixed a bug in LibLinearRegressionModel.getTopFeatures where it tried to lookup the bias and threw an exception.

Motivation

We use the expanded model equality checks when testing the reproducibility work.

…ed a bug in LibLinearRegressionModel.getTopFeatures where it tried to lookup the bias.
Copy link
Member

@JackSullivan JackSullivan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@Craigacp Craigacp merged commit ab0b42d into main Oct 18, 2021
@Craigacp Craigacp deleted the more-model-equality branch October 18, 2021 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Oracle employee This PR is from an Oracle employee squash-commits Squash the commits when merging this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants