dfo-mar-mpas / MarConsNetAnalysis

https://dfo-mar-mpas.github.io/MarConsNetAnalysis/
0 stars 0 forks source link

Link app, meta analysis and PPT area names #7

Closed j-harbin closed 2 months ago

j-harbin commented 2 months ago

Right now the MarConsNetData data_objectives function (and therefore the app) use the following names as examples:

areas <- c("stAnnsBank", "musquash", "gully")

The metanalysis csv uses the following names: image

And the PPT uses key words: image

It is still possible to link all of these for the app, but it would make more sense if they were all the same names throughout the process. I'll do these changes, but I'm also tagging @remi-daigle just so we can discuss.

j-harbin commented 2 months ago

To add to this, the app actually uses a different naming convention based on the names opened from open data using data_CPCAD See example below

image

j-harbin commented 2 months ago

Some notes for myself:

I have now made the arguments for data_objectives and data_context to be in the following format st_Anns_Bank_MPA, gully_MPA etc. This will easily be tired to the PPT key words (after moving the _)

image

To connect the app naming convention "Gully Marine Protected Area" to data_objectives and the PPT I identify when there is the words "Marine Protected Area" and replace it with "MPA". I then do a match by adding a "_". E.g.

Gully Marine Protected Area would become Gully_MPA (note these changes will not be reflected in the app drop down menu and all happens in the back ground). One additional step that needs to happen is for the app "Musquash Estuary Marine Protected Area", which will have to have the word "Estuary" removed to match with the data_objectives argument musquash_MPA and the PPT tag: "Musquash MPA"

j-harbin commented 2 months ago

Updated: to link the meta analysis in to this I created my own csv in my sandbox and changed the names to be like the following: Gully_MPA_CO, st_anns_banks_MPA_CO, etc. I then remove the CO and replace the with a space to link it back to the ppt.

j-harbin commented 2 months ago

This has been done.