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.

Sign up for GitHub

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

Jump to bottom

cex vegan::ordiplot not respected by plot.(d)c(c)a #656

Closed
BertvanderVeen opened this issue Jun 1, 2024 · 1 comment
Closed

cex vegan::ordiplot not respected by plot.(d)c(c)a #656

BertvanderVeen opened this issue Jun 1, 2024 · 1 comment

Comments

@BertvanderVeen
Copy link

I sparingly use vegan::ordiplot, which has a "cex" argument. However, ordiplot calls plot.cca, which does not take a cex argument, so that the behavior of vegan::ordiplot is inconsistent for different methods. Is would be great if plot.cca would also receive a cex argument so that vegan::ordiplot is a bit more consistent across different methods.
@jarioksa
Copy link
Contributor

jarioksa commented Jun 1, 2024

Yes, verified. I'll see this happen. Thanks for your note.

jarioksa added a commit that referenced this issue Jun 4, 2024
@jarioksa
Merge branch 'issue-#656'
4a1e8ac
jarioksa added a commit that referenced this issue Jun 9, 2024
@jarioksa
ordiplot handles default cex via par(cex) if arg not given
ebec493 
We used to have argument cex=0.7 in ordiplot, but having cex in
call would require adding cex also to plot.cca which I don't want
to do. Still satisfies wish #656.
jarioksa added a commit that referenced this issue Jun 9, 2024
@jarioksa
plot.cca hardcoded cex: fixes #656
c19e57a 
plot.cca is poorly configurable, and most things cannot be fine
tuned. The whole functions should be redesigned along lines
of pipes plot(x, type="n") |> text("species", ...) |>
points("sites", ...)

(cherry picked from commit 6917ea2)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jarioksa @BertvanderVeen