SCIPpp icon indicating copy to clipboard operation
SCIPpp copied to clipboard

Add new c'tors to LinExpr

Open hedtke opened this issue 1 year ago • 3 comments

Add new c'tors to LinExpr to speed-up the creation of expressions with multiple variables.

Fixes #32

This is an alternative implementation of #34

hedtke avatar Jan 27 '25 11:01 hedtke

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 96.13%. Comparing base (d9061e8) to head (f9678e5). :warning: Report is 14 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
+ Coverage   95.62%   96.13%   +0.50%     
==========================================
  Files          10       10              
  Lines         183      207      +24     
  Branches       48       60      +12     
==========================================
+ Hits          175      199      +24     
  Misses          3        3              
  Partials        5        5              
Files with missing lines Coverage Δ
include/scippp/lin_expr.hpp 100.00% <100.00%> (ø)
source/lin_expr.cpp 100.00% <100.00%> (ø)
:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Jan 27 '25 12:01 codecov[bot]

@TonyCongqianWang you might want to have a look into this

hedtke avatar Jan 27 '25 14:01 hedtke

I'll try to do it tomorrow!

TonyCongqianWang avatar Jan 27 '25 19:01 TonyCongqianWang

@erogelio18 ready for next review round

hedtke avatar Aug 26 '25 12:08 hedtke

Mac CI fails because Conan center is currently offline

hedtke avatar Aug 26 '25 12:08 hedtke