Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Vessels mask #717

Open
wants to merge 302 commits into
base: dev
Choose a base branch
from
Open

Vessels mask #717

wants to merge 302 commits into from

Conversation

TitouanGragnic
Copy link
Collaborator

@TitouanGragnic TitouanGragnic commented Dec 5, 2024

This adds a new analysis panel, now for use to compute Vessels Masks, but can also be used to add analysis algorithms.
The new panel is initialized in analysis_init.cc and most its computing is done in analysis.cc
A new folder analysis has been created in sources and includes folder to contain all its related files
The computations done to compute the artery and vein masks are the same than in Pulsewave, results quality vary greatly depending on the input.

Baschtien and others added 30 commits October 31, 2024 11:36
Co-authored-by: Simon Riou <simon-riou@users.noreply.github.com>
@GustaveHerve GustaveHerve marked this pull request as ready for review January 6, 2025 14:10
@GustaveHerve GustaveHerve changed the title draft : Vessels mask Vessels mask Jan 7, 2025
@GustaveHerve GustaveHerve marked this pull request as draft January 7, 2025 08:49
@GustaveHerve GustaveHerve marked this pull request as ready for review January 13, 2025 13:45
@GustaveHerve GustaveHerve added the good first issue Good for newcomers label Jan 14, 2025
@Hpn4 Hpn4 linked an issue Jan 15, 2025 that may be closed by this pull request
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New feature : vessels mask
7 participants