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

Test data pull from stockeff #47

Open
andybeet opened this issue Sep 20, 2023 · 5 comments
Open

Test data pull from stockeff #47

andybeet opened this issue Sep 20, 2023 · 5 comments
Assignees

Comments

@andybeet
Copy link
Member

stockeff.mv_cf_landings table now have species value included and pull from CAMS from 2020+. Prior to this AA tables are used.

We should pull data from this table and compare to current comlandr implementation.

We should also monitor the CAMS portion of the data through time to see how frequently they are updated, since they are now live tables.

@andybeet andybeet self-assigned this Sep 20, 2023
@andybeet
Copy link
Member Author

andybeet commented Nov 2, 2023

Looks like SPPVALUE is pretty consistent post 1981 (post WOLAND era)
This plot show the areas in which landings have the most value
top20byareaValue

@andybeet
Copy link
Member Author

WOLAND data not added by StockeEff folks
"We did not include WOLAN because there were concerns as to the validity of the data. We have no way of validating it and had heard a few years ago that this field was not populated properly in this series of objects. Since Population Dynamics does not use this field we did not continue paying attention to it (did somebody validate it/were there updates)."

@sgaichas
Copy link
Member

Are Atlantic herring landings already integrated in stockEff?

When I use get_comland_raw_data from the stockEFF tables, I get 20481 rows with NESPP3==168 (Atlantic herring) (1964-2023)

Then in get_herring_data, when it goes to the GARFO table to get herring landings and removes those that were included in the first function, I get 13264 rows with NESPP3==168 (Atlantic herring). Fewer.

I have not yet checked the data so this may be ok, just noting here before I forget

Sorry if its a red herring haha

@andybeet
Copy link
Member Author

So when we were pulling from the AA tables, herring was being replaced by herring that is pulled from its own table (maine_herring_catch table). But to answer your question as to whether STOCKEFF handles this already, i don't know. I will reach out.

@andybeet
Copy link
Member Author

andybeet commented Jul 1, 2024

STOCKEFF are not hadling this, they do not replace herring with state of maine data. so we ned to look into the code for a potential bug and make sure we are pulling from the correct tables, since i believe ITD moved them/ renamed them

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