Skip to content

Conversation

@Ian-Ganz
Copy link
Collaborator

@Ian-Ganz Ian-Ganz commented Nov 3, 2025

CI integration tests for verilog-beta.

Copy link
Collaborator

@murphe67 murphe67 left a comment

Choose a reason for hiding this comment

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

:)

RecordProperty("cycles", std::to_string(config.simTime));
}

TEST_P(VerilogSharingFixture, verilog_CI) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove

@@ -191,6 +194,70 @@ TEST_P(SpecFixture, spec_NoCI) {
RecordProperty("cycles", std::to_string(simTime));
}

// Verilog
TEST_P(VerilogFixture, verilog_CI) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove suffix

@Ian-Ganz Ian-Ganz force-pushed the verilog_beta_tests branch 5 times, most recently from 6f6f653 to 771568e Compare November 9, 2025 17:37
Copy link
Collaborator

@murphe67 murphe67 left a comment

Choose a reason for hiding this comment

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

Looks great, almost there :)

# e.g. by running 'cmake -DPARALLEL_FACTOR=<n> -B build'
# before 'ninja -C build <target>'
set(PARALLEL_FACTOR 8 CACHE STRING "Parallelism for integration test targets")
set(PARALLEL_FACTOR 1 CACHE STRING "Parallelism for integration test targets")
Copy link
Collaborator

Choose a reason for hiding this comment

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

instead of reducing CI speed, lets remove from CI for now and solve later

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