Skip to content

/ cdms

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

Jump to bottom

missing_value attribute in file not respected when reading in #18

Closed
doutriaux1 opened this issue Oct 12, 2016 · 4 comments
Closed

missing_value attribute in file not respected when reading in #18

doutriaux1 opened this issue Oct 12, 2016 · 4 comments
Assignees
@dnadeau4
Labels
bug important
Milestone
2.10

Comments

@doutriaux1
Copy link
Contributor

@doutriaux1 doutriaux1 commented Oct 12, 2016

test_tvariable.py line 50
@dnadeau4
Copy link
Contributor

@dnadeau4 dnadeau4 commented Oct 31, 2016

Where is test_tvariable.py? I cannot find it.
@doutriaux1
Copy link
Contributor Author

@doutriaux1 doutriaux1 commented Oct 31, 2016

hum... not sure...
@dnadeau4
Copy link
Contributor

@dnadeau4 dnadeau4 commented Oct 31, 2016

It seems that when we create a variable with subRegion, the missing_value is reset to 1.e20

(Pdb) v.subRegion((366., 731., 'ccn'), (-42., 42., 'ccn'), (90., 270.)).getMissing()
1e+20
(Pdb) v.getMissing()
-99.9
@dnadeau4
Copy link
Contributor

@dnadeau4 dnadeau4 commented Oct 31, 2016

BTW, this is a new test created by @chaosphere2112 the file is here:
https://github.com/UV-CDAT/cdms/blob/master/Test/test_tvariable.py
dnadeau4 added a commit that referenced this issue Oct 31, 2016
@dnadeau4
fix #18 tvariable missingvalue copy issue in subRegion
9f4f48a
dnadeau4 added a commit that referenced this issue Oct 31, 2016
@dnadeau4
fix #18 tvariable missingvalue copy issue in subRegion
b465d21
@doutriaux1 doutriaux1 modified the milestone: 2.10 May 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dnadeau4 dnadeau4

Labels
bug important
Projects
None yet
Milestone
2.10
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@doutriaux1 @dnadeau4