Skip to content

Added Gaussian blurring method to augmentations - #41

Open
sirish-gambhira wants to merge 4 commits into
mlpack:masterfrom
sirish-gambhira:master
Open

Added Gaussian blurring method to augmentations#41
sirish-gambhira wants to merge 4 commits into
mlpack:masterfrom
sirish-gambhira:master

Conversation

@sirish-gambhira

Copy link
Copy Markdown

Hey @kartikdutt18, kindly review this PR and let me know your suggestions/ comments on the method. I would be happy to work on them and improve the code.

@mlpack-bot

Copy link
Copy Markdown

Thanks for opening your first pull request in this repository! Someone will review it when they have a chance. In the mean time, please be sure that you've handled the following things, to make the review process quicker and easier:

  • All code should follow the style guide
  • Documentation added for any new functionality
  • Tests added for any new functionality
  • Tests that are added follow the testing guide
  • Headers and license information added to the top of any new code files
  • HISTORY.md updated if the changes are big or user-facing
  • All CI checks should be passing

Thank you again for your contributions! 👍

@mlpack-bot

Copy link
Copy Markdown

This issue has been automatically marked as stale because it has not had any recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions! 👍

@@ -1,6 +1,6 @@
/**
/**Adding support for more data types to mlpack, it would be preferable to add the support upstream to Armadillo instead, so that may be a better direction to go first. Then very little code modification for mlpack will be necessary./**

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is a pretty long comment

@birm
birm self-requested a review May 21, 2021 17:45
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@sirish-gambhira@birm@kartikdutt18