Edits to gui to add buttons to clear plot and clear parameters, remov…

[bkuebel]

…e unwanted parameters from list, add yellow/red lines for suspect/fail data, and add button to Find Gaps, print out longest data gap, and add an annotation if the time period of the data gap isn't already annotated

[cwingard]

I had to download the mlapp to my computer to play with it. It works with my data, looks nice!

[cwingard]

@bkuebel I played around with this some more with a phsen data set. The new sorting method you have for the variables removes several other ones that are needed/desired. Your function removes everything that isn't named raw*, practical*, density* or seawater*. That means I can't see most of the variables I need to look at for the phsen. I suggest changing the function to:

% Button pushed function: LoadDataButton
function LoadDataButtonPushed(app, event)
    persistent lastpath
    if isempty(lastpath) 
        lastpath = 0;
    end

    if lastpath == 0
        [filename, pathname] = uigetfile('*.nc', 'Select NetCDF File');
    else
        [filename, pathname] = uigetfile('*.nc', 'Select NetCDF File', lastpath);
    end

    if pathname ~= 0
        lastpath = pathname;
        data = nc_reader(fullfile(pathname, filename));

        % Remove the *_qc_executed and *_qartod_executed string parameters
        ind = ~endsWith(data.Properties.VariableNames, {'executed'});
        ind2keep = ind == 1;
        app.ParametersListBox.Items = data.Properties.VariableNames(ind2keep);
        setappdata(app.LeftPanel, 'ncdata', data);
        setappdata(app.LeftPanel, 'ncfile', filename);
        app.PlotButton.Enable = true;
    end
end

Also, in your find gapsand theplotfunctions, the date/time string needs to include theTbetween the date and the time. That format is how the API delivers the date/time string and howit expects the string to be formatted (e.g.2021-01-19T16:16:00`). If you can update the data_review.mlapp to include these changes and push the commit to the gui_edits branch, we can give it another check.

[cwingard]

@bkuebel @snwhite Updated the pull request with changes to both the nc_reader function:

• address some bugs in the NetCDF library used by Matlab
• properly supports string variables like the *_qartod_executed variables we will now be using
• changes how the date/time was being calculated
  Also made a few changes to the data_review app:
• addresses bug identified by Sheri White when saving a downloaded file (didn't auto-generate the file name on the second save)
• adds ability to append deployments (must be the same data record with the same variables) to allow for cross-deployment assessments.
• removes string variables from the list of variables (can't support those at this time, and they throw an error).

[cwingard]

@bkuebel Lets cancel this PR. The code has been moved into the ooi-data-explorations repo and has long since been updated past this particular PR.