Comments & organization of app code

cct-datascience / AFS_database_code

Code for app to display standardized fish data in North America

https://viz.datascience.arizona.edu/afs-standard-fish-data/

BSD 2-Clause "Simplified" License

1 stars 3 forks source link

Comments & organization of app code #20

Open diazrenata opened 11 months ago

diazrenata commented 11 months ago

See https://github.com/diazrenata/AFS_database_code/pull/1

KristinaRiemer commented 11 months ago

Other than my one comment about reading in location data, I think this looks good! Having all of these comments and structure is super useful. I am going to wait until you implement that change @diazrenata and then I'll test the code locally to make sure everything works and merge this.

diazrenata commented 11 months ago

@KristinaRiemer I toggled the toy_locs data and resolved the merge conflicts (mostly related to whitespace/indentation). I also fixed the typo referenced in #18, so that PR is no longer needed. This should be ready for you to test!