Skip to content

Add is_bipartite(::Graph)#4499

Merged
benlorenz merged 12 commits intooscar-system:masterfrom
Sequenzer:master
Jan 27, 2025
Merged

Add is_bipartite(::Graph)#4499
benlorenz merged 12 commits intooscar-system:masterfrom
Sequenzer:master

Conversation

@Sequenzer
Copy link
Copy Markdown
Collaborator

The title says it all.

Comment thread src/Combinatorics/Graphs/functions.jl Outdated
Comment thread src/Combinatorics/Graphs/functions.jl Outdated
@lgoettgens lgoettgens changed the title Added Function is_bipartite Add is_bipartite(::Graph) Jan 24, 2025
@lgoettgens lgoettgens added topic: combinatorics release notes: use title For PRs: the title of this PR is suitable for direct use in the release notes labels Jan 24, 2025
@lgoettgens lgoettgens requested a review from benlorenz January 24, 2025 17:27
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.55%. Comparing base (f329ec4) to head (fba384a).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4499      +/-   ##
==========================================
- Coverage   84.55%   84.55%   -0.01%     
==========================================
  Files         672      672              
  Lines       88888    88833      -55     
==========================================
- Hits        75158    75111      -47     
+ Misses      13730    13722       -8     
Files with missing lines Coverage Δ
src/Combinatorics/Graphs/functions.jl 96.74% <100.00%> (+0.03%) ⬆️

... and 28 files with indirect coverage changes

Copy link
Copy Markdown
Member

@benlorenz benlorenz left a comment

Choose a reason for hiding this comment

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

A few suggestions, and please add the function to docs/src/Combinatorics/graphs.md.

Comment thread src/Combinatorics/Graphs/functions.jl Outdated
Comment thread src/Combinatorics/Graphs/functions.jl Outdated
Comment thread src/Combinatorics/Graphs/functions.jl Outdated
Comment thread src/Combinatorics/Graphs/functions.jl Outdated
Comment thread src/Combinatorics/Graphs/functions.jl Outdated
Comment thread src/Combinatorics/Graphs/functions.jl Outdated
Sequenzer and others added 6 commits January 25, 2025 10:27
Co-authored-by: Benjamin Lorenz <benlorenz@users.noreply.github.com>
Co-authored-by: Benjamin Lorenz <benlorenz@users.noreply.github.com>
Co-authored-by: Benjamin Lorenz <benlorenz@users.noreply.github.com>
Co-authored-by: Benjamin Lorenz <benlorenz@users.noreply.github.com>
Comment thread docs/src/Combinatorics/graphs.md Outdated
@Sequenzer Sequenzer marked this pull request as ready for review January 27, 2025 09:32
@benlorenz benlorenz merged commit e645614 into oscar-system:master Jan 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release notes: use title For PRs: the title of this PR is suitable for direct use in the release notes topic: combinatorics

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants