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

Reference to undefined IMOS.templates.dimensions.adcp_remapped #773

Closed
sspagnol opened this issue Feb 15, 2022 · 2 comments
Closed

Reference to undefined IMOS.templates.dimensions.adcp_remapped #773

sspagnol opened this issue Feb 15, 2022 · 2 comments
Labels
Type:bug Something is missing or wrong
Milestone

Comments

@sspagnol
Copy link
Contributor

In Parser/OceanContour/OceanContour.m line 667 there is a code

dimensions = IMOS.templates.dimensions.adcp_remapped;

but this does not exist in Util/+IMOS/+templates/dimensions.m, so not sure how oceanContourParser ever worked.

@sspagnol sspagnol mentioned this issue Feb 16, 2022
@lbesnard
Copy link
Contributor

@sspagnol I found the same issue while trying to debug #775

I emailed @ocehugo to understand how this parser is meant to work.

At L440 (https://github.com/aodn/imos-toolbox/blob/master/Parser/OceanContour/OceanContour.m#L440 ), the following file should automatically be tested by the docstrings unittests written by hugo:

% file = [toolboxRootPath 'data/testfiles/netcdf/Nortek/OceanContour/Signature/s500_enu_avg.nc'];

However, running the toolbox with this file fails at L647 with the following error:

The class IMOS.templates.dimensions has no Constant property or Static method named 'adcp_remapped'.

Error in OceanContour.readOceanContourFile (line 647)
                dimensions = IMOS.templates.dimensions.adcp_remapped;

The only other reference is here:

% mode [str] - ['timeSeries','profile','adcp_raw','adcp_remapped'].

@lbesnard lbesnard added the Type:bug Something is missing or wrong label May 26, 2022
@lbesnard lbesnard added this to the Q4 - FY21/22 milestone May 26, 2022
@lbesnard
Copy link
Contributor

fixed by #774

@lbesnard lbesnard closed this as completed Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type:bug Something is missing or wrong
Projects
None yet
Development

No branches or pull requests

2 participants