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

feat: Added Exponential Probability Distribution #2780

Merged
merged 31 commits into from
Oct 12, 2024
Merged

feat: Added Exponential Probability Distribution #2780

merged 31 commits into from
Oct 12, 2024

Conversation

HarshilShah1804
Copy link
Contributor

Description of Change

Added Exponential distribution in probability.

Checklist

  • Added description of change
  • Added file name matches File name guidelines
  • Added tests and example, test must pass
  • Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
  • Relevant documentation/comments is changed or added
  • PR title follows semantic commit guidelines
  • Search previous suggestions before making a new one, as yours may be a duplicate.
  • I acknowledge that all my contributions will be made under the project's license.

Notes:

realstealthninja
realstealthninja requested changes Oct 7, 2024
View reviewed changes
DIRECTORY.md Outdated Show resolved Hide resolved
probability/exponential_dist.cpp Outdated Show resolved Hide resolved
probability/exponential_dist.cpp Outdated Show resolved Hide resolved
probability/exponential_dist.cpp Outdated Show resolved Hide resolved
probability/exponential_dist.cpp Outdated Show resolved Hide resolved
probability/exponential_dist.cpp Outdated Show resolved Hide resolved
probability/exponential_dist.cpp Outdated Show resolved Hide resolved
probability/exponential_dist.cpp Outdated Show resolved Hide resolved
@realstealthninja realstealthninja added hacktoberfest awaiting modification Do not merge until modifications are made labels Oct 7, 2024
HarshilShah1804 and others added 7 commits October 8, 2024 12:51
@HarshilShah1804 @realstealthninja
Update probability/exponential_dist.cpp
a994abb 
Co-authored-by: realstealthninja <68815218+realstealthninja@users.noreply.github.com>
@HarshilShah1804 @realstealthninja
Update probability/exponential_dist.cpp
05de7d0 
Co-authored-by: realstealthninja <68815218+realstealthninja@users.noreply.github.com>
@HarshilShah1804 @realstealthninja
Update probability/exponential_dist.cpp
478ba41 
Co-authored-by: realstealthninja <68815218+realstealthninja@users.noreply.github.com>
@HarshilShah1804
Update the files
c340e01
@HarshilShah1804
Merge branch 'Harshil' of https://github.com/HarshilShah1804/C-Plus-Plus
55ea280 
 into Harshil
@HarshilShah1804
Removed the link from documentation
6ca8991
@HarshilShah1804
Merge branch 'master' into Harshil
9f165ca
realstealthninja
realstealthninja requested changes Oct 8, 2024
View reviewed changes
Copy link
Collaborator

@realstealthninja realstealthninja left a comment

Choose a reason for hiding this comment

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

replace ( with \f$ for it to render
\f[ makes a big equation
while the other is inline
examples can be found here

probability/exponential_dist.cpp Outdated Show resolved Hide resolved
probability/exponential_dist.cpp Outdated Show resolved Hide resolved
probability/exponential_dist.cpp Outdated Show resolved Hide resolved
realstealthninja and others added 6 commits October 8, 2024 19:27
@realstealthninja
docs: remove the second brief tag
489cac5
@HarshilShah1804 @realstealthninja
Update latex notation
39981b0 
Co-authored-by: realstealthninja <68815218+realstealthninja@users.noreply.github.com>
@HarshilShah1804 @realstealthninja
Update latex notation
ea42824 
Co-authored-by: realstealthninja <68815218+realstealthninja@users.noreply.github.com>
@HarshilShah1804
Merge branch 'master' into Harshil
f1ea41c
@HarshilShah1804
Corrected format issues
c4244c8
@HarshilShah1804
Merge branch 'Harshil' of https://github.com/HarshilShah1804/C-Plus-Plus
3b7a06b 
 into Harshil
@HarshilShah1804 @realstealthninja
Update probability/exponential_dist.cpp
e1b2e73 
Co-authored-by: realstealthninja <68815218+realstealthninja@users.noreply.github.com>
realstealthninja
realstealthninja requested changes Oct 9, 2024
View reviewed changes
Copy link
Collaborator

@realstealthninja realstealthninja left a comment

Choose a reason for hiding this comment

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

please lint your file with clang-tidy / clang-format

probability/exponential_dist.cpp Outdated Show resolved Hide resolved
probability/exponential_dist.cpp Outdated Show resolved Hide resolved
probability/exponential_dist.cpp Outdated Show resolved Hide resolved
HarshilShah1804 and others added 6 commits October 9, 2024 19:35
@HarshilShah1804
Merge branch 'master' into Harshil
3429c90
@HarshilShah1804 @realstealthninja
Update probability/exponential_dist.cpp
a2bcdd4 
Co-authored-by: realstealthninja <68815218+realstealthninja@users.noreply.github.com>
@HarshilShah1804 @realstealthninja
Update probability/exponential_dist.cpp
1fd6081 
Co-authored-by: realstealthninja <68815218+realstealthninja@users.noreply.github.com>
@HarshilShah1804
Added more test, formatted with clang-format
c6c23dc
@HarshilShah1804
Update files
06ca03a
@HarshilShah1804
Merge branch 'master' into Harshil
9850db6
@HarshilShah1804
Copy link
Contributor Author

@realstealthninja All changes are made. Please review

@HarshilShah1804
Copy link
Contributor Author

@realstealthninja I have modified the code according to the suggestions. Please review and tell me if anything else is required. Thank you

realstealthninja
realstealthninja approved these changes Oct 12, 2024
View reviewed changes
Copy link
Collaborator

@realstealthninja realstealthninja left a comment

Choose a reason for hiding this comment

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

Lgtm, thank you for your contribution. Happy hacktoberfest ♥️

@realstealthninja realstealthninja added approved Approved; waiting for merge hacktoberfest-accepted Accepted to be counted towards Hacktoberfest and removed awaiting modification Do not merge until modifications are made labels Oct 12, 2024
@HarshilShah1804 HarshilShah1804 deleted the Harshil branch October 12, 2024 10:46
@HarshilShah1804 HarshilShah1804 restored the Harshil branch October 12, 2024 14:18
@realstealthninja realstealthninja merged commit c2adc3b into TheAlgorithms:master Oct 12, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@realstealthninja realstealthninja realstealthninja approved these changes

@Panquesito7 Panquesito7 Awaiting requested review from Panquesito7 Panquesito7 is a code owner

Assignees
No one assigned
Labels
approved Approved; waiting for merge hacktoberfest hacktoberfest-accepted Accepted to be counted towards Hacktoberfest
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@HarshilShah1804 @realstealthninja