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

.plotit public #8

Closed
mebrooks opened this issue Dec 5, 2016 · 3 comments
Closed

.plotit public #8

mebrooks opened this issue Dec 5, 2016 · 3 comments

Comments

@mebrooks
Copy link
Collaborator

mebrooks commented Dec 5, 2016

Would it make sense for the .plotit funciton to be public so that users can see the arguments (e.g. add, ci, cicol)? Alternatively, we may want to document these arguments for each of the other functions they can be used with (e.g. ssbplot)

@mebrooks
Copy link
Collaborator Author

mebrooks commented Dec 5, 2016

It seems that there are problems when passing these extra arguments to the multiple-plot functions (see example below), so I think I'll just document them with the functions where they work.

In this example, all the plots in the 2nd command get added to the recruitment plot.

plot(fit, ci=FALSE)
plot(fit, add=TRUE, cicol="red")

I don't want to get into the details of these plotting functions. I just wanted to document the features that I found helpful while writing a vignette.

@mebrooks mebrooks closed this as completed Dec 5, 2016
@fishfollower
Copy link
Owner

Need to look at that more. OK to make .plotit public, but with a different name.

@fishfollower
Copy link
Owner

fishfollower commented Dec 5, 2016 via email

fishfollower pushed a commit that referenced this issue Feb 18, 2019
* ...

* mvnorm by trans

* mix for f

* ...

* Added example with list of matrices.

* Added start of attempt too include covariance matrices from SToX

* working for F now

* added example to work from

* added the R part of the list processing

* unfinished ... times up

* ...

* added output options for forecast

* docs

* added option to hit landings not catch

* landval  in forecast

* modification

* Small changes regarding data input.

* working version

* simulation code and such

* more flexible

* more options to specify covariance

* bump version

* removed unused

* binary merged

* simulation for MVMIX

* now works for F

* changed default for patch in loadConfig

* reverted to simpler option

* made backward compatible (w.r.t. last decimals og gradient)

* keep argument to mvmix

* now for obs also

* robustified calculation

* bump version

* remove blank line

* ver. nr. in md docs

* changed from t_1 to t_3 in mixture distribution

* ...

* version

* added NEWS file

* Allowed retro to work on a fit where a map is given

* bubble legend refinement

* Allow selecting R used in Makefile from commandline

* Test dataSet and confSet constructors from SEXP

* Use listMatrixFromR in dataSet constructor

* In revdep test, do not check CRAN incoming

* fixed keep bug in mvmix

* Add functions to table/plot survey catchabilities

* qtable specific for sam fits

* throw warning when one or more retro or leaveout runs did not converge

* fix qtable doc and add Rd files

* current changes before constructing plotFix branch

* Include map from original fit in process error residual routine

* Add fSelectivityPlot

* minor change plot F-selectivity

* minor change plot F-selectivity

* minor change fselectivityplot

* minor change fselectivityplot
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

No branches or pull requests

2 participants