Skip to content

Sourcery refactored main branch - #1

Open
sourcery-ai[bot] wants to merge 1 commit into
mainfrom
sourcery/main
Open

sourcery-ai[bot] wants to merge 1 commit into
mainfrom
sourcery/main

Conversation

@sourcery-ai

@sourcery-ai sourcery-ai Bot commented Nov 11, 2023

Copy link
Copy Markdown

Branch main refactored by Sourcery.

If you're happy with these changes, merge this Pull Request using the Squash and merge strategy.

See our documentation here.

Run Sourcery locally

Reduce the feedback loop during development by using the Sourcery editor plugin:

Review changes via command line

To manually merge these changes, make sure you're on the main branch, then run:

git fetch origin sourcery/main
git merge --ff-only FETCH_HEAD
git reset HEAD^

Help us improve this pull request!

@sourcery-ai
sourcery-ai Bot requested a review from deburky November 11, 2023 12:43
Comment thread temperature_scaling.py
Comment on lines 63 to +64

y_pred_calibrated = y_pred / self.scalar_
return y_pred_calibrated No newline at end of file
return y_pred / self.scalar_ No newline at end of file

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Function TemperatureScaling.predict_proba refactored with the following changes:

y_pred = self._sigmoid(linear_model)
y_pred_class = [1 if p >= 0.5 else 0 for p in y_pred]
return y_pred_class
return [1 if p >= 0.5 else 0 for p in y_pred]

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Function LogisticRegressionCustomLoss.predict refactored with the following changes:

Comment on lines -118 to +119
y_pred = self._sigmoid(y_logit)
return y_pred No newline at end of file
return self._sigmoid(y_logit) No newline at end of file

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Function LogisticRegressionCustomLoss.predict_proba refactored with the following changes:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants