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

Typos in blurred segment equation #1561

Merged
merged 4 commits into from
Apr 29, 2021
Merged

Conversation

ngophuc
Copy link
Contributor

@ngophuc ngophuc commented Apr 15, 2021

PR Description

your description here

Checklist

  • Unit-test of your feature with Catch.
  • Doxygen documentation of the code completed (classes, methods, types, members...)
  • Documentation module page added or updated.
  • New entry in the ChangeLog.md added.
  • No warning raised in Debug cmake mode (otherwise, Travis C.I. will fail).
  • All continuous integration tests pass (Travis & appveyor)

@kerautret
Copy link
Member

Thanks ;)

@kerautret kerautret self-assigned this Apr 15, 2021
Copy link
Member

@kerautret kerautret left a comment

Choose a reason for hiding this comment

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

👍🏻

@dcoeurjo
Copy link
Member

thanks a lot. Could you please add a new entry to the Changelog ?

@ngophuc
Copy link
Contributor Author

ngophuc commented Apr 28, 2021

Sorry, I dont understand. What I have to do exactly for the Changelog?

@kerautret
Copy link
Member

@ngophuc
Copy link
Contributor Author

ngophuc commented Apr 28, 2021

Ok, done ! Thank Bertrand.

@kerautret
Copy link
Member

Ok, done ! Thank Bertrand.

You can edit it in this branch no ? I see another PR ;)

@ngophuc
Copy link
Contributor Author

ngophuc commented Apr 28, 2021

Ok, done ! Thank Bertrand.

You can edit it in this branch no ? I see another PR ;)

👍🏻

@dcoeurjo
Copy link
Member

Thanks @ngophuc
Merging

@dcoeurjo dcoeurjo merged commit aaef06c into DGtal-team:master Apr 29, 2021
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.

3 participants