Added global options for consort plot parameters#32
Merged
adayim merged 1 commit intoadayim:mainfrom Apr 2, 2026
Merged
Conversation
adayim
reviewed
Mar 18, 2026
| x = line_coords$x, | ||
| y = line_coords$y, | ||
| gp = gpar(fill = "black"), | ||
| gp = gpar(fill = col, col = col, lwd = lwd), |
Owner
There was a problem hiding this comment.
This is nice but I would change this to arrow_gp parameter and make sure users pass a gpar() object. What do you think?
Contributor
Author
There was a problem hiding this comment.
yes, we can change it.
adayim
reviewed
Apr 2, 2026
Owner
adayim
left a comment
adayim
left a comment
There was a problem hiding this comment.
I made some changes. I don't think you have added adsl data information in the data.R file. I have to delete it if there is no documentation about this data. I think we can remove the new vignette as I have updated the old vignette.
Contributor
Author
|
Thank you.
…On Thu, Apr 2, 2026 at 2:57 PM Alim Dayim ***@***.***> wrote:
***@***.**** commented on this pull request.
I made some changes. I don't think you have added adsl data information
in the data.R file. I have to delete it if there is no documentation
about this data. I think we can remove the new vignette as I have updated
the old vignette.
—
Reply to this email directly, view it on GitHub
<#32?email_source=notifications&email_token=AUOZLOLBDCJ2YKQTFP5PYVL4TZPU3A5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTIMBVGA2DQMRSGY2KM4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2L24DSL5ZGK5TJMV3V63TPORUWM2LDMF2GS33OONPWG3DJMNVQ#pullrequestreview-4050482264>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AUOZLOPSTFVINHWKXBX7LK34TZPU3AVCNFSM6AAAAACWUYUQFWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHM2DANJQGQ4DEMRWGQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
added global options for the below params.
Reset all to defaults
options(
consort_line_lwd = NULL, # Line width of connector arrows
consort_line_col = NULL, # Color of connector lines and arrow heads
consort_arrow_length = NULL, # Size of the arrow head(numeric=inches or pass grid::unit() 0.1
consort_arrow_type = NULL, # Arrow head style "closed" or "open"
consort_pad_u = NULL ,# Vertical and horizontal spacing between boxes
consort_bullet =NULL # Unicode character used for bullet points in side boxes
)