Skip to content
This repository was archived by the owner on Sep 9, 2020. It is now read-only.

PRED-2194 Use old reasonCodesPredictions route to support 4.3.x installs #147

Merged
merged 5 commits into from
Jan 9, 2019
Merged

PRED-2194 Use old reasonCodesPredictions route to support 4.3.x installs #147

merged 5 commits into from
Jan 9, 2019

Conversation

pprett
Copy link
Contributor

@pprett pprett commented Jan 7, 2019

BATCH SCORING PULL REQUEST

DataRobot 4.3.x does not yet provide a /predictionExplanations route.
We should use the not yet deprecated /reasonCodesPredictions route for non-deployment aware routes.

RATIONALE

DataRobot 4.3.x does not yet provide a /predictionExplanations route.

CHANGES

TESTING

@devexp-slackbot
Copy link

JIRA

@pprett pprett requested a review from Axik January 9, 2019 13:50
Axik
Axik approved these changes Jan 9, 2019
View reviewed changes
Copy link
Contributor

@Axik Axik left a comment

Choose a reason for hiding this comment

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

LGTM

@coveralls
Copy link

coveralls commented Jan 9, 2019 •
edited
Loading

Coverage Status

Coverage increased (+0.006%) to 83.955% when pulling 64e7f1c on peter/PRED-2194 into c17ca96 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.006%) to 83.955% when pulling 64e7f1c on peter/PRED-2194 into c17ca96 on master.

@pprett pprett merged commit 50847c9 into master Jan 9, 2019
@pprett pprett deleted the peter/PRED-2194 branch January 9, 2019 19:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
1 more reviewer

@Axik Axik Axik approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pprett @coveralls @Axik