Skip to content

Add multiset_permutations method that doesn't require length#164

Merged
inkydragon merged 6 commits intomasterfrom
os/multiset_permutations-method
May 10, 2025
Merged

Add multiset_permutations method that doesn't require length#164
inkydragon merged 6 commits intomasterfrom
os/multiset_permutations-method

Conversation

@oscardssmith
Copy link
Member

in analogy to the same case for regular permuations

in analogy to the same case for regular `permuations`
@codecov
Copy link

codecov bot commented Apr 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.21%. Comparing base (2ad84aa) to head (5a65c72).
Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #164   +/-   ##
=======================================
  Coverage   97.21%   97.21%           
=======================================
  Files           8        8           
  Lines         826      827    +1     
=======================================
+ Hits          803      804    +1     
  Misses         23       23           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@inkydragon
Copy link
Member

Maybe add some tests or doctests

@oscardssmith
Copy link
Member Author

test added.

@oscardssmith
Copy link
Member Author

should this be merged?

@inkydragon inkydragon added the feature Indicates new feature / enhancement requests label May 2, 2025
@inkydragon inkydragon added this to the v1.1.0 milestone May 2, 2025
Co-authored-by: Chengyu Han <cyhan.dev@outlook.com>
@inkydragon inkydragon merged commit 9380656 into master May 10, 2025
12 checks passed
@inkydragon inkydragon deleted the os/multiset_permutations-method branch May 10, 2025 04:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Indicates new feature / enhancement requests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants