Codecov Report

❌ Patch coverage is 97.11111% with 13 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (gd/structured3@b24c181). Learn more about missing BASE report.

@@                Coverage Diff                @@
##             gd/structured3     #316   +/-   ##
=================================================
  Coverage                  ?   99.25%           
=================================================
  Files                     ?       24           
  Lines                     ?     2282           
  Branches                  ?        0           
=================================================
  Hits                      ?     2265           
  Misses                    ?       17           
  Partials                  ?        0           

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

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Any idea why codecov is lacking? In any case it's not a big deal yet, what matters is what you want to merge into main at the end.
By the way, you should probably work in a branch of your fork that starts from here, so that you can push stuff to it freely

Any idea why codecov is lacking? In any case it's not a big deal yet, what matters is what you want to merge into main at the end.

codecov is werid here, this PR is simply a merge from main branch, which already covers all the functionalities in JuMP extension, not sure why it's warning.

By the way, you should probably work in a branch of your fork that starts from here, so that you can push stuff to it freely.

Yeah, I just open PRs from my fork to the gd/structured3 branch, that can allow a flexible testing from my side.

Alright, let's merge this and keep going! Thanks for your enthusiasm :)