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

Improve the prediction technique #222

Merged
merged 5 commits into from
Mar 25, 2022

Conversation

geoffreydstewart
Copy link
Member

Improve the prediction technique by identifying points far away from exemplars as noise, rather than requiring such points to be close to noise point exemplars.

In addition to the unit tests, the quality of these prediction technique changes have been verified using existing notebooks available in this repo/branch.

…exemplars as noise, rather than requiring points to be close to noise point exemplars.

Signed-off-by: Geoffrey Stewart <[email protected]>
@Craigacp Craigacp added the Oracle employee This PR is from an Oracle employee label Mar 16, 2022
Copy link
Member

@Craigacp Craigacp left a comment

Choose a reason for hiding this comment

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

Can you add a test loading in a 4.2 model and asserting that the predictions are the same as they are in 4.2? We've done this before in cases where the on-disk format changed and it's a useful check to make sure nothing broke. Otherwise it looks good.

Copy link
Member

@Craigacp Craigacp 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 apart from the small note about keeping the comments up to date.

Copy link
Member

@Craigacp Craigacp left a comment

Choose a reason for hiding this comment

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

LGTM.

@Craigacp Craigacp merged commit bee1d75 into oracle:main Mar 25, 2022
@geoffreydstewart geoffreydstewart deleted the predictions-update branch March 27, 2022 18:23
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants