Skip to content

Update plot.jl to account for line style - #176

Open
etiennedeg wants to merge 14 commits into
JuliaGraphs:masterfrom
etiennedeg:rebase-131
Open

Update plot.jl to account for line style#176
etiennedeg wants to merge 14 commits into
JuliaGraphs:masterfrom
etiennedeg:rebase-131

Conversation

@etiennedeg

Copy link
Copy Markdown
Member

This is a rebase of #131, after I messed it up.
I added some documentation

davide-fand others added 14 commits October 5, 2020 17:56
Proposal to:
1) add a custom line dash style of the edges
2) specify whether to resize the box as a unit square or preserve the proportions (x and y axes) among the different stations
This reverts commit 921d54d.
Proposal to:
1) add a custom line dash style of the edges
2) specify whether to resize the box as a unit square or preserve the proportions (x and y axes) among the different stations
This reverts commit 921d54d.
@codecov

codecovBot commented Jul 22, 2022

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 6.06061% with 31 lines in your changes missing coverage. Please review.
✅ Project coverage is 33.03%. Comparing base (2ce59f9) to head (f108307).
⚠️ Report is 39 commits behind head on master.

Files with missing linesPatch %Lines
src/deprecations.jl0.00%27 Missing ⚠️
src/collapse_plot.jl0.00%3 Missing ⚠️
src/layout.jl0.00%1 Missing ⚠️
Additional details and impacted files
@@ Coverage Diff @@## master #176 +/- ##
==========================================
- Coverage 33.26% 33.03% -0.24% 
==========================================
Files 9 9 Lines 526 566 +40 ==========================================
+ Hits 175 187 +12 - Misses 351 379 +28 

☔ 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.

@hdavid16

Copy link
Copy Markdown
Contributor

@etiennedeg, thanks for moving things with this old PR. what is the status of this PR?

@etiennedeg

Copy link
Copy Markdown
MemberAuthor

I don't know what is the policy on this package for testing, maybe we need some kind of tests, but I don't see many tests here. Otherwise, I think we are good. @simonschoelly, a quick review maybe ?

Comment threadREADME.md
## Add dash style to edges
```julia
using Measures
gplot(g, edgedashstyle=[5mm, 2mm])

@hdavid16hdavid16Aug 5, 2022

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The use of this kwarg should be properly documented. In this case, one edge will have 5mm dashes and the next will have 2mm dashes

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

No, all edges will have the same style, that is alternating 5mm solid line and 2mm blank line.

Comment threadREADME.md
gplot(g, edgedashstyle=[5mm, 2mm])
# vary edge styles
using Compose
gplot(g, EDGELINEWIDTH=0.7, edgedashstyle=[[10px, 5px], [10px, 5px, 3px, 5px], [4px, 4px], []])

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The use of this kwarg should be properly documented. When vectors are passed for each edge, the values define the sequence of dashes and spaces. For example, the first edge will have 10px solid dash with 5px spacing in between; the second edge will have the following alternating sequence: 10px solid - 5 px space - 3px solid - 5 px space - etc...

Comment threadsrc/plot.jl
Optional. Color for the edge strokes, can be a Vector. Default: `colorant"lightgray"`

`edgedashstyle`
Optional. Dash style for the edge, can be a Vector. Default: no dashed line.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The default here is []

@hdavid16

Copy link
Copy Markdown
Contributor

If it helps, I can add this to #186 so that it is consistent with straight and curved lines with the new fixes that enable curved self-loops. And we can have one PR with all these feature additions.

Sign up for freeto 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.

3 participants

@etiennedeg@hdavid16@davide-f