cccl icon indicating copy to clipboard operation
cccl copied to clipboard

Update linear_congruential_generator with constexpr, tests and a fast discard

Open RAMitchell opened this issue 2 months ago • 10 comments

Description

closes Part of #5679

Update linear_congruential_generator with constexpr, stream operators, tests and a fast discard operator for common parameters.

RAMitchell avatar Oct 30 '25 16:10 RAMitchell

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

copy-pr-bot[bot] avatar Oct 30 '25 16:10 copy-pr-bot[bot]

/ok to test 4355e21

miscco avatar Nov 10 '25 14:11 miscco

@miscco test_engine has the constexpr evaluations inside it. Its already testing constexpr for c++20 - if I understand you correctly.

RAMitchell avatar Nov 10 '25 16:11 RAMitchell

😬 CI Workflow Results

🟥 Finished in 1h 57m: Pass: 91%/90 | Total: 22h 07m | Max: 1h 40m | Hits: 94%/186274

See results here.

github-actions[bot] avatar Nov 10 '25 16:11 github-actions[bot]

/ok to test 2cdb853a928a43f3e7b622918c85bfcfc41119c2

RAMitchell avatar Nov 10 '25 16:11 RAMitchell

😬 CI Workflow Results

🟥 Finished in 1h 09m: Pass: 98%/90 | Total: 21h 52m | Max: 1h 08m | Hits: 94%/215089

See results here.

github-actions[bot] avatar Nov 10 '25 17:11 github-actions[bot]

🥳 CI Workflow Results

🟩 Finished in 1h 50m: Pass: 100%/90 | Total: 22h 13m | Max: 1h 08m | Hits: 94%/219163

See results here.

github-actions[bot] avatar Nov 10 '25 18:11 github-actions[bot]

😬 CI Workflow Results

🟥 Finished in 3h 12m: Pass: 93%/90 | Total: 1d 00h | Max: 1h 20m | Hits: 94%/210897

See results here.

github-actions[bot] avatar Nov 12 '25 14:11 github-actions[bot]

😬 CI Workflow Results

🟥 Finished in 2h 37m: Pass: 95%/88 | Total: 1d 13h | Max: 1h 36m | Hits: 95%/211852

See results here.

github-actions[bot] avatar Nov 13 '25 12:11 github-actions[bot]

🥳 CI Workflow Results

🟩 Finished in 2h 07m: Pass: 100%/88 | Total: 22h 01m | Max: 1h 40m | Hits: 99%/212899

See results here.

github-actions[bot] avatar Nov 14 '25 13:11 github-actions[bot]

approved. One last suggestion, could you please change __A to single underscore _A for template parameters?

I am not in favor, single letter _A macros are frequently used by the compiler. Dont ask me why, but for example _T is used by MSVC to convert string literals

miscco avatar Nov 18 '25 07:11 miscco

Will merge as is according to @miscco `s comment.

RAMitchell avatar Nov 18 '25 08:11 RAMitchell