Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

fix: overfit test #137

Merged
merged 4 commits into from
Oct 18, 2021
Merged

fix: overfit test #137

merged 4 commits into from
Oct 18, 2021

Conversation

tadejsv
Copy link
Contributor

@tadejsv tadejsv commented Oct 18, 2021

I realized I put the wrong sign in the overfit test.
I will also change the margin in it to 0.1

I realized I put the wrong sign in the overfit test.
I will also change the margin in it to 0.5
@github-actions github-actions bot added size/s area/testing This issue/PR affects testing labels Oct 18, 2021
@codecov
Copy link

codecov bot commented Oct 18, 2021

The author of this PR, tadejsv, is not an activated member of this organization on Codecov.
Please activate this user on Codecov to display this PR comment.
Coverage data is still being uploaded to Codecov.io for purposes of overall coverage calculations.
Please don't hesitate to email us at success@codecov.io with any questions.

@tadejsv tadejsv marked this pull request as ready for review October 18, 2021 15:00
Copy link
Member

@bwanglzu bwanglzu left a comment

Choose a reason for hiding this comment

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

LGTM!

@tadejsv tadejsv merged commit 0285280 into main Oct 18, 2021
@tadejsv tadejsv deleted the fix-overfit-test branch October 18, 2021 15:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants