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

fix Spring for single node graph #32

Merged
merged 1 commit into from
Aug 5, 2021
Merged

fix Spring for single node graph #32

merged 1 commit into from
Aug 5, 2021

Conversation

hexaeder
Copy link
Collaborator

@hexaeder hexaeder commented Aug 5, 2021

Returned NaN before. See MakieOrg/GraphMakie.jl#28

Note: This will still do all the iterations which is kinda pointless. But i think a some point we might want to stop the iteration after some tolerance is reached anyway (similar to SFDP).

@codecov
Copy link

codecov bot commented Aug 5, 2021

Codecov Report

Merging #32 (240a24e) into master (e80e36a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #32   +/-   ##
=======================================
  Coverage   98.14%   98.14%           
=======================================
  Files           8        8           
  Lines         431      432    +1     
=======================================
+ Hits          423      424    +1     
  Misses          8        8           
Impacted Files Coverage Δ
src/spring.jl 97.77% <100.00%> (+0.05%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e80e36a...240a24e. Read the comment docs.

@hexaeder hexaeder requested a review from SimonDanisch August 5, 2021 15:34
@hexaeder hexaeder merged commit a2f0fb1 into master Aug 5, 2021
@hexaeder hexaeder deleted the hw/fixspring branch August 5, 2021 16:25
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.

2 participants