Skip to content

Uniformed server side stdout prints and parseCLI#238

Merged
GlassOfWhiskey merged 12 commits into
masterfrom
clean-print-update-parsing
May 18, 2026
Merged

Uniformed server side stdout prints and parseCLI#238
GlassOfWhiskey merged 12 commits into
masterfrom
clean-print-update-parsing

Conversation

@marcoSanti
Copy link
Copy Markdown
Member

@marcoSanti marcoSanti commented Apr 27, 2026

This commit cleans up and standardizes the various message prints that the server performs during its execution. It also updates the parseCLI method; instead of instantiating objects directly, it now populates a structure that is returned with the parsed parameters. This structure is then used to instantiate objects from the CAPIO server main function.

This commit cleans up and uniforms the varius messge prints that the server
performs during it's execution.
@marcoSanti marcoSanti force-pushed the clean-print-update-parsing branch from eddbf52 to e80650d Compare April 27, 2026 11:06
@marcoSanti marcoSanti changed the title Uniformed server side stdout prints Uniformed server side stdout prints and parseCLI Apr 27, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 27, 2026

Codecov Report

❌ Patch coverage is 54.97382% with 86 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.43%. Comparing base (7c13f82) to head (bd8cb2e).
⚠️ Report is 5 commits behind head on master.

Files with missing lines Patch % Lines
capio/server/src/cli_parser.cpp 42.52% 14 Missing and 36 partials ⚠️
capio/server/include/remote/listener.hpp 47.05% 3 Missing and 6 partials ⚠️
capio/server/src/mpi_backend.cpp 0.00% 6 Missing and 2 partials ⚠️
capio/server/capio_server.cpp 68.18% 1 Missing and 6 partials ⚠️
capio/common/shm.hpp 28.57% 3 Missing and 2 partials ⚠️
capio/server/include/utils/signals.hpp 60.00% 2 Missing and 2 partials ⚠️
capio/server/include/utils/server_println.hpp 78.57% 1 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #238      +/-   ##
==========================================
- Coverage   63.84%   63.43%   -0.42%     
==========================================
  Files          77       79       +2     
  Lines        3214     3268      +54     
  Branches     1997     2008      +11     
==========================================
+ Hits         2052     2073      +21     
- Misses        381      398      +17     
- Partials      781      797      +16     

☔ 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.

@marcoSanti marcoSanti force-pushed the clean-print-update-parsing branch 3 times, most recently from e4a515e to 15d092f Compare April 27, 2026 14:59
@marcoSanti marcoSanti force-pushed the clean-print-update-parsing branch from 15d092f to eed6199 Compare April 27, 2026 15:01
@marcoSanti marcoSanti force-pushed the clean-print-update-parsing branch from db2376c to 8904bc6 Compare April 28, 2026 22:03
Comment thread capio/server/include/utils/server_println.hpp Outdated
Comment thread capio/server/include/utils/server_println.hpp Outdated
Comment thread capio/server/include/utils/server_println.hpp Outdated
Comment thread capio/server/capio_server.cpp Outdated
Comment thread capio/server/include/utils/server_println.hpp Outdated
marcoSanti and others added 4 commits May 18, 2026 12:11
Co-authored-by: Iacopo Colonnelli <iacopo.c92@gmail.com>
Co-authored-by: Iacopo Colonnelli <iacopo.c92@gmail.com>
Co-authored-by: Iacopo Colonnelli <iacopo.c92@gmail.com>
@marcoSanti marcoSanti force-pushed the clean-print-update-parsing branch from ac842a9 to d2e1f93 Compare May 18, 2026 10:42
@marcoSanti marcoSanti force-pushed the clean-print-update-parsing branch from d2e1f93 to bd8cb2e Compare May 18, 2026 10:44
Copy link
Copy Markdown
Collaborator

@GlassOfWhiskey GlassOfWhiskey left a comment

Choose a reason for hiding this comment

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

LGTM

@GlassOfWhiskey GlassOfWhiskey merged commit 6b14036 into master May 18, 2026
40 checks passed
@GlassOfWhiskey GlassOfWhiskey deleted the clean-print-update-parsing branch May 18, 2026 12:25
@GlassOfWhiskey GlassOfWhiskey changed the title Uniformed server side stdout prints and parseCLI Uniformed server side stdout prints and parseCLI May 18, 2026
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.

2 participants