Skip to content

Pr 3110 with julia updates - #3114

Merged
galabovaa merged 17 commits into
latestfrom
pr-3110
Jun 30, 2026
Merged

Pr 3110 with julia updates#3114
galabovaa merged 17 commits into
latestfrom
pr-3110

Conversation

@galabovaa

Copy link
Copy Markdown
Contributor

Updates to the julia workflow prompted by failing tests at #3110

galabovaa and others added 11 commits June 27, 2026 23:47
Latest in master for release
The usage of std::result_of within the header makes it impossible for this code to compile when the user uses C++20 but has HiGHS as dependency.
Added a new guard to change the function signature for be compatible with C++20 and later
@galabovaa
galabovaa requested review from mathgeekcoder and odow June 29, 2026 16:00
@codecov

codecov Bot commented Jun 29, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 73.15%. Comparing base (9713e3d) to head (ff64efa).
⚠️ Report is 3 commits behind head on latest.

Files with missing lines Patch % Lines
highs/util/HighsDynamicLibrary.h 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           latest    #3114   +/-   ##
=======================================
  Coverage   73.15%   73.15%           
=======================================
  Files         430      430           
  Lines      103937   103937           
  Branches    16717    16717           
=======================================
  Hits        76039    76039           
  Misses      27622    27622           
  Partials      276      276           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread highs/util/HighsDynamicLibrary.h Outdated
@mathgeekcoder

Copy link
Copy Markdown
Collaborator

@galabovaa I believe these latest build errors are resolved by #3112?

@galabovaa

Copy link
Copy Markdown
Contributor Author

@galabovaa I believe these latest build errors are resolved by #3112?

Yes, apologies for merging the wrong one!

@odow odow left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

There's a few different things happening here, but if the Julia tests pass I'm happy.

@galabovaa
galabovaa merged commit 1e3d101 into latest Jun 30, 2026
492 of 499 checks passed
@galabovaa
galabovaa deleted the pr-3110 branch July 27, 2026 08:37
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.

4 participants