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

Move age_representativeness_plot() to {nwfscSurvey} #111

Open
7 tasks
kellijohnson-NOAA opened this issue May 1, 2023 · 4 comments
Open
7 tasks

Move age_representativeness_plot() to {nwfscSurvey} #111

kellijohnson-NOAA opened this issue May 1, 2023 · 4 comments
Assignees
Labels
priority: low The lowest level priority, i.e., not urgent. status: in progress Currently working on this issue topic: code Related to R code within this package type: refactor Changes to the codebase or documentation that do NOT alter the values returned, e.g., styling.

Comments

@kellijohnson-NOAA
Copy link
Contributor

kellijohnson-NOAA commented May 1, 2023

@chantelwetzel-noaa {nwfscSurvey} is the destination of choice right?

age_representativeness_plot <- function(bio,

Before doing so, finish the following tasks

  • allow for parameter-based column names for length, age, and year
  • convert to {ggplot2}
    • haven't added $\hat{b}$ yet
    • need a better theme
    • only want p value colored not the entire KS test line
  • remove some parameters that are not needed
  • rename function to something more concise with an action term to start (check, plot)

initial figure that I am working on (mid convert to {ggplot2} is (haven't added $\hat{b}$ yet)
test2

original figure was
test_3

@kellijohnson-NOAA kellijohnson-NOAA added topic: code Related to R code within this package status: in progress Currently working on this issue type: refactor Changes to the codebase or documentation that do NOT alter the values returned, e.g., styling. priority: low The lowest level priority, i.e., not urgent. labels May 1, 2023
@kellijohnson-NOAA kellijohnson-NOAA self-assigned this May 1, 2023
@chantelwetzel-noaa
Copy link
Contributor

chantelwetzel-noaa commented May 1, 2023

I think the {nwfscSurvey} is a good location for this. Could I add one more task to your list?

@kellijohnson-NOAA
Copy link
Contributor Author

You bet @chantelwetzel-noaa! I moved it to my list so I won't forget.

@kellijohnson-NOAA
Copy link
Contributor Author

I have a question ... should the K-S test and the distributions shown on the figure be from mutually exclusive data or should the comparison be all lengths vs lengths with ages? I am thinking that the comparison should be between lengths without ages versus lengths with ages such that no data point exists in the two data sets being compared.

@brianlangseth-NOAA
Copy link
Contributor

I dont see any reason it has to be one or the other. Originally I did it to compare lengths with aged-lengths distributions, and thus there was some overlap. I think this has undergone adjustments that I haven't been following. So long as the KS test compares the two distributions that are being plotted, I think it is intuitively doing what it is supposed to be doing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: low The lowest level priority, i.e., not urgent. status: in progress Currently working on this issue topic: code Related to R code within this package type: refactor Changes to the codebase or documentation that do NOT alter the values returned, e.g., styling.
Projects
None yet
Development

No branches or pull requests

3 participants