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

CURA-11378 stress benchmarks #1988

Merged
merged 18 commits into from
Nov 28, 2023
Merged

CURA-11378 stress benchmarks #1988

merged 18 commits into from
Nov 28, 2023

Conversation

jellespijker
Copy link
Member

@jellespijker jellespijker commented Nov 23, 2023

Description

https://ultimaker.github.io/CuraEngineBenchmarks/dev/stress_bench/

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • Test A
  • Test B

Test Configuration:

  • Operating System:

Checklist:

  • My code follows the style guidelines of this project as described in UltiMaker Meta
  • I have read the Contribution guide
  • I have commented my code, particularly in hard-to-understand areas
  • I have uploaded any files required to test this change

casperlamboo and others added 8 commits November 20, 2023 15:26
Reads in a number of known problematic polygons and settings and counts
how many crashes occur in those shapes

Contributes to CURA-11378
Showing the failed test cases in the tooltip

Contributes to CURA-11378
Copy link
Contributor

github-actions bot commented Nov 23, 2023

Unit Test Results

26 tests  ±0   26 ✔️ ±0   6s ⏱️ ±0s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ±0 

Results for commit 686fc0b. ± Comparison against base commit 574c4b9.

♻️ This comment has been updated with latest results.

@casperlamboo casperlamboo mentioned this pull request Nov 23, 2023
7 tasks
@casperlamboo casperlamboo marked this pull request as ready for review November 23, 2023 15:26

std::vector<Resource> getResources()
{
auto resource_path = std::filesystem::path(std::source_location::current().file_name()).parent_path().append("resources");
Copy link
Contributor

Choose a reason for hiding this comment

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

std::source_location is not available on mac 😔

Copy link
Member

@rburema rburema left a comment

Choose a reason for hiding this comment

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

LGTM.

I like it when both of you are on a ticket, makes reviewing easy; when walking through the commits, each time when one of you introduces something I don't like the other removes it like 2 commits later :-p

I'll try to merge main into it, if that passes (enough), I'll merge it back to main.

stress_benchmark/stress_benchmark.cpp Show resolved Hide resolved
@rburema rburema merged commit 6a47349 into main Nov 28, 2023
20 checks passed
@jellespijker jellespijker deleted the CURA-11378_stress_benchmarks branch December 11, 2023 06:50
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