Test Get started page with test data set

sfregosi / agate

agate: Acoustic Glider Analysis Tools and Environment. MATLAB-based tools for working with passive acoustic underwater gliders.

https://sfregosi.github.io/agate/

GNU General Public License v3.0

2 stars 2 forks source link

Test Get started page with test data set #62

Open sfregosi opened 3 months ago

sfregosi commented 3 months ago

https://sfregosi.github.io/agate/get-started.html https://drive.google.com/drive/folders/1kNDH56g4NjOzGDHeGncIkjUXXUoPj8Gd?usp=drive_link

[ ] Installation - Try Option 3: Forking the repository
[ ] Download test data set and set up folder structure as suggested
[ ] Modify configuration file given with test data with proper paths for downloaded test data set (can skip basestation configuration file step)
[ ] Initialize agate with the test data set's configuration file (after making the necessary modifications to map to your local paths) (can skip the remaining 'download' steps)

sfregosi commented 3 months ago

@megwood2012 - I can't assign this task to you, but see if you can check things off. I think once you interact with it I may be able to assign you.

megwood2012 commented 3 months ago

@sfregosi My work email doesn't have access to the linked google drive folder, which is where I assume the test data lives. I've sent an access request to you

megwood2012 commented 3 months ago

@sfregosi I followed the steps in getting started, and got through initializing agate, which seemed to work fine. However, when I tried the final extract piloting parameters and flight metrics step, I got the "Unrecognized function or variable 'CONFIG'" error message again. I've attached a screenshot of that screen here for you. agate CONFIG error

sfregosi commented 3 months ago

@megwood2012 Can you show me what line you entered just before the 'struct' part of this error message? Where you initialized agate. It is looking for something called CONFIG that you have to define when you initialize. This step changed so before you didn't have to define it as explicitly (it was a global variable...but those are not recommended so I got rid of those) but now you do. I've got the correct approach outlined here but let me know if you were pulling the code from elsewhere as I likely missed updating it somewhere. Your first initialization step should look something like

CONFIG = agate('configurationfile.cnf');

megwood2012 commented 3 months ago

@sfregosi Pulling the code from the top part of the 'download all baseStation files' section did solve my 'CONFIG' issue. I just skipped that part when I initially ran it, since I knew I couldn't download baseStation files. Maybe worth having a separate 'Configure agate' section before the 'download all baseStation files' section, in case other people get impatient like me?

megwood2012 commented 3 months ago

@sfregosi Just realized that the final 'ExtractPilotingParams (CONFIG, path_bsLocal, path_status) code requires access to the baseStation, so the problem I'm running into where it's asking to open a piloting params table I don't have is probably an expected error, correct?

sfregosi commented 3 months ago

Created issue #66 to make that initialization code clearer (this might be a duplicate issue I swear I started that yesterday, but I can't find it anywhere so just made #66).

Added a comment to the google doc but no, the error you are getting about a missing piloting params table wasn't expected due to not being able to download from the basestation, but it should have been expected now that I'm looking at the documentation's example code that is incorrect and missing a 'catch' option in the function itself. Created issue #67 for that.