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

Main axis recognition #342

Merged
merged 9 commits into from
Mar 30, 2019
Merged

Main axis recognition #342

merged 9 commits into from
Mar 30, 2019

Conversation

copyme
Copy link
Member

@copyme copyme commented Aug 30, 2018

Add an option for showing main axis of a digital point (text and point color if in the viewer mode). The axes are found by checking for a digital point axes of the DSSes passing thru the point.

@copyme
Copy link
Member Author

copyme commented Aug 30, 2018

The point of a stable direction are RED (X), GREEN (Y), BLUE (Z). Then, if a point is not stable e.g., its axes are X and Z then the color of that point is (255,0,255).

image

/cc @JacquesOlivierLachaud

@copyme
Copy link
Member Author

copyme commented Aug 30, 2018

The text output looks like this

....
(35, 19, 6); MAIN_AXES = (Z)
(34, 19, 7); MAIN_AXES = (Z)
(34, 20, 8); MAIN_AXES = (Z)
(33, 20, 9); MAIN_AXES = (Z)
(32, 20, 10); MAIN_AXES = (Z)
(32, 21, 11); MAIN_AXES = (X, Z)
(31, 21, 12); MAIN_AXES = (X, Z)
(30, 21, 13); MAIN_AXES = (X, Z)
(29, 21, 14); MAIN_AXES = (X, Z)
(28, 21, 15); MAIN_AXES = (X, Z)
(27, 22, 15); MAIN_AXES = (X)
(26, 22, 16); MAIN_AXES = (X)
(25, 22, 17); MAIN_AXES = (X)
(24, 22, 17); MAIN_AXES = (X)
(23, 21, 18); MAIN_AXES = (X)
(22, 21, 18); MAIN_AXES = (X)
....

@dcoeurjo
Copy link
Member

can you please merge back the master ?

@dcoeurjo
Copy link
Member

Changelog entry ?

@copyme
Copy link
Member Author

copyme commented Mar 22, 2019

@dcoeurjo done

@kerautret
Copy link
Member

Thanks @copyme , I just fix the conflict if @dcoeurjo is ok we can merge for being in 1.0 ;) 🎉

@copyme
Copy link
Member Author

copyme commented Mar 29, 2019

@kerautret cool :-)

@kerautret kerautret merged commit f0b91ae into DGtal-team:master Mar 30, 2019
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.

None yet

3 participants