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

Vapor crashes on 2D NetCDF data #2985

Closed
sgpearse opened this issue Feb 1, 2022 · 0 comments · Fixed by #2987
Closed

Vapor crashes on 2D NetCDF data #2985

sgpearse opened this issue Feb 1, 2022 · 0 comments · Fixed by #2987
Labels
Bug Regression A bug that did not previously exist in released code

Comments

@sgpearse
Copy link
Collaborator

sgpearse commented Feb 1, 2022

I believe the attached 2D data file is CF compliant as per Vapor's standards, but the application crashes upon importing.

To reproduce:

  1. Download the NetCDF file below
  2. Import the file through the File->Import->NetCDF-CF menu option

Download File Here

(base) cisl-farmington:coen pearse$ ncdump -h foo.nc
netcdf foo {
dimensions:
	x = 120 ;
	y = 120 ;
	time = 1 ;
variables:
	double x(x) ;
		x:_FillValue = NaN ;
		x:axis = "X" ;
		x:units = "m" ;
	double y(y) ;
		y:_FillValue = NaN ;
		y:axis = "Y" ;
		y:units = "m" ;
	int64 time(time) ;
		time:axis = "T" ;
		time:units = "days since 2021-12-30 10:00:00" ;
		time:calendar = "proleptic_gregorian" ;
	float fuelsl(time, y, x) ;
		fuelsl:_FillValue = NaNf ;
}
@sgpearse sgpearse added the Bug label Feb 1, 2022
@clyne clyne added the Regression A bug that did not previously exist in released code label Feb 2, 2022
clyne added a commit that referenced this issue Feb 2, 2022
clyne added a commit that referenced this issue Feb 2, 2022
* Fixed #2985 - Vapor crashes on 2D NetCDF data

* Added unit test for ConstNodeIterator ConstNodeBegin(const CoordType &minu, const CoordType &maxu)

* clang-format pre-push hook
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Regression A bug that did not previously exist in released code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants