Skip to content

Conversation

@furszy
Copy link

@furszy furszy commented Jul 6, 2021

The random seed value, used in boost for shuffling the unit tests order, is interpreted as an unsigned int inside the boost test framework and the current value is overflowing the type range, thus why GA is failing in every PR and commit.

Solution: use the framework functionality to select a random seed based on the current time (can check it inside boost documentation link).

The random seed value is interpreted as an unsigned int inside boost test and github's run_id value is overflowing the type range.
@furszy furszy self-assigned this Jul 6, 2021
@furszy furszy force-pushed the 2021_fix_GA_boost_test branch from e5c4f47 to c3d0d65 Compare July 6, 2021 14:28
@furszy furszy added this to the 6.0.0 milestone Jul 6, 2021
Copy link

@random-zebra random-zebra left a comment

Choose a reason for hiding this comment

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

ACK c3d0d65

@furszy furszy requested a review from Fuzzbawls July 6, 2021 18:54
Copy link
Collaborator

@Fuzzbawls Fuzzbawls left a comment

Choose a reason for hiding this comment

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

ACK 5e6200b57a06e1ed2f0d0c522da5333cfb5c2786

@furszy furszy merged commit dfe6d4c into PIVX-Project:master Jul 6, 2021
@furszy furszy deleted the 2021_fix_GA_boost_test branch November 29, 2022 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants