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.

Sign up for GitHub

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

Jump to bottom

CLI11 to replace Boost program option #51

Merged
merged 24 commits into from Oct 6, 2020
Merged

CLI11 to replace Boost program option #51

merged 24 commits into from Oct 6, 2020

Conversation

kerautret
Copy link
Member

@kerautret kerautret commented Sep 28, 2020 •
edited

PR Description

Progression:

  • geometry2d
    • rosinThreshold
    • meaningFullThickness
    • houghLineDetect
    • distanceTransform
      • LUTBasedNSDistanceTransform
  • geometry3d
    • basicEditMesh
    • basicMorphoFilter
    • computeMeshDistances
    • off2obj
    • off2sdp
    • volLocalMax
    • xyzScale
  • visualisation
    • displayLineSegments
    • displaySet2dPts
    • displayTgtCoverAlphaTS
    • graphViewer
    • meshViewerEdit

Checklist

  • Doxygen documentation of the code completed (classes, methods, types, members...)
  • Check if it follows the tools structure described in CONTRIBUTING.md
  • New entry in the ChangeLog.md added.
  • Update the readme with potentially a screenshot of the tools if it applies.
  • No warning raised in Debug cmake mode (otherwise, Travis C.I. will fail).

@kerautret kerautret changed the title CLI11 to replace Boost program option [WIP] CLI11 to replace Boost program option Sep 28, 2020
kerautret and others added 21 commits September 29, 2020 11:26
@kerautret
CLI11 in meaningFullThickness
5c178e7
@kerautret
CLI11 in HoughTransform + OpenCV issue + CMakeList issues
ec4194f
@kerautret
CLI11 in LUTBasedNSDistanceTransform
072305e
@kerautret
CLI11 in basicEditMesh
af977c8
@kerautret
CLI11 in basicMorphoFilter
60b2e97
@kerautret
CLI11 in computeMeshDistances
cd0a2f9
@kerautret
CLI11 in off2obj
9ddfe76
@ngophuc
displaySet2dPts
28dee21
@ngophuc
displayTgtCoverAlphaTS
56f755b
@kerautret
Merge pull request #2 from ngophuc/CLI11
f2c90d5 
displaySet2dPts
@ngophuc
graphViewer
e8ddb22
@ngophuc
meshViewerEdit
1c6c75d
@ngophuc
xyzScale
ca16436
@kerautret
CLI11 in off2sdp
cb7067e
@kerautret
Merge pull request #3 from ngophuc/CLI11
e00650d 
graphViewer
@kerautret
Merge branch 'CLI11' of github.com:kerautret/DGtalTools-contrib into …
58ef571 
…CLI11
@ngophuc
volLocalMax
497997d
@kerautret
CLI11 in displayLineSegments
ae3bbc0
@kerautret
Merge pull request #4 from ngophuc/CLI11
b9c039b 
Thanks!
@kerautret
removing prog opt in cmake
3bc2f8e
@kerautret
changelog
96a7f91
@kerautret kerautret changed the title [WIP] CLI11 to replace Boost program option CLI11 to replace Boost program option Sep 29, 2020
@kerautret
Copy link
Member Author

Note Travis is fine with PR see test merged PR

@dcoeurjo
Copy link
Member

dcoeurjo commented Oct 6, 2020

ok merging this one, we'll see later for travis, thanks

@dcoeurjo dcoeurjo merged commit 66f007b into DGtal-team:master Oct 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dcoeurjo dcoeurjo Awaiting requested review from dcoeurjo

Assignees
No one assigned
Labels
ready for review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@kerautret @dcoeurjo @ngophuc