Skip to content

75 soleposthoc integration#76

Merged
PasoStudio73 merged 18 commits intomainfrom
75-soleposthoc-integration
Aug 13, 2025
Merged

75 soleposthoc integration#76
PasoStudio73 merged 18 commits intomainfrom
75-soleposthoc-integration

Conversation

@PasoStudio73
Copy link
Member

Now that Lumen algorhitm in SolePostHoc package is almost ready, we should rething its integration in SoleXplorer.
How SolePostHoc is linked to SoleXplorer is actually a little hacked: you can find that hack in devPaso branch of SolePostHoc repo.

Asking for SolePostHoc mods to satisfy SoleXplorer user interface it's a little bit tricky and out of context, I think.
Better to refactor SoleXplorer->SolePostHoc interface to allow an easy user experience interface and mainain the complexity that a weapon like SolePostHoc demand.

@PasoStudio73 PasoStudio73 self-assigned this Aug 9, 2025
@PasoStudio73 PasoStudio73 added the enhancement New feature or request label Aug 9, 2025
@PasoStudio73 PasoStudio73 added this to SOLE Aug 9, 2025
@PasoStudio73 PasoStudio73 linked an issue Aug 9, 2025 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Aug 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.85%. Comparing base (9f21e65) to head (37dc893).
⚠️ Report is 25 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #76      +/-   ##
==========================================
+ Coverage   95.67%   95.85%   +0.18%     
==========================================
  Files           8        8              
  Lines         439      531      +92     
==========================================
+ Hits          420      509      +89     
- Misses         19       22       +3     

☔ View full report in Codecov by Sentry.
📢 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.

@PasoStudio73 PasoStudio73 merged commit 5b67ae6 into main Aug 13, 2025
3 of 4 checks passed
@github-project-automation github-project-automation bot moved this to Done in SOLE Aug 13, 2025
@PasoStudio73 PasoStudio73 deleted the 75-soleposthoc-integration branch August 13, 2025 19:52
Copy link
Member

@giopaglia giopaglia Aug 14, 2025

Choose a reason for hiding this comment

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

(Is it the case that this is not needed anymore because there are CI automations for building the documentation?)

Copy link
Member

Choose a reason for hiding this comment

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

I've updated the comment, maybe it was not so clear

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

SolePostHoc integration

2 participants