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

[WIP] Unit test with random seed failure #909

Merged
merged 2 commits into from
Jul 6, 2023

Conversation

KexinFeng
Copy link
Contributor

@KexinFeng KexinFeng commented Jul 5, 2023

This PR tries to enable the unit test for the sampling autoregressive searching. The assertion is true when locally testing it. But in the PR continueous test, the assertion becomes false.

Update:

Now the assertion in the unit test passes the PR test. However, local test will fail at that assertion.

So the issue is, after setting the random seed, the outputs, even though fixed, are different between local testing and PR testing.

@KexinFeng KexinFeng requested review from zachgk, frankfliu and a team as code owners July 5, 2023 23:28
Copy link
Contributor

@lanking520 lanking520 left a comment

Choose a reason for hiding this comment

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

we removed those PT files since it violates the Jacoco test rules

@KexinFeng KexinFeng force-pushed the unit_test_random_seed branch from 6ade6c0 to 3c233c5 Compare July 6, 2023 04:46
@KexinFeng KexinFeng changed the title Unit test with random seed failure [wip] Unit test with random seed failure Jul 6, 2023
@KexinFeng KexinFeng changed the title [wip] Unit test with random seed failure Unit test with random seed failure Jul 6, 2023
@KexinFeng KexinFeng changed the title Unit test with random seed failure [WIP] Unit test with random seed failure Jul 6, 2023
@KexinFeng KexinFeng force-pushed the unit_test_random_seed branch from 31296e5 to b7c8f83 Compare July 6, 2023 18:00
@KexinFeng KexinFeng merged commit 86e9c9c into deepjavalibrary:master Jul 6, 2023
KexinFeng added a commit to KexinFeng/djl-serving-forked that referenced this pull request Aug 16, 2023
KexinFeng added a commit to KexinFeng/djl-serving-forked that referenced this pull request Aug 16, 2023
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.

3 participants