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

Issue 186 xml copy not working #1511

Merged
merged 4 commits into from
Sep 2, 2015
Merged

Conversation

doutriaux1
Copy link
Contributor

No description provided.

@doutriaux1
Copy link
Contributor Author

@aashish24 @dnadeau4

@durack1
Copy link
Member

durack1 commented Aug 31, 2015

@doutriaux1 @dnadeau4 looks like changes to source/testing/cdms2/CMakeLists.txt are causing the build errors above..

@@ -16,7 +16,10 @@ cdat_add_test("test_dim_unlimited"
cdat_add_test("CDMS_Test_Zonal_regrid_Switch_to_Regrid2"
"${PYTHON_EXECUTABLE}"
${cdat_SOURCE_DIR}/testing/cdms2/test_regrid_zonal_switch_to_regrid2.py)

cdat_add_test("cdms2_test_createcopy_lose_dtype
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing quote "

@aashish24
Copy link
Contributor

@doutriaux1 please fix the missing double quote

@durack1
Copy link
Member

durack1 commented Aug 31, 2015

@doutriaux1 @dnadeau4 @aashish24 I just edited this in the browser and added the missing double quote back in, plus corrected some indentation quirks.. It's triggered a rebuild..

@durack1
Copy link
Member

durack1 commented Aug 31, 2015

@doutriaux1 should vcs errors still be happening on oceanonly?

The following tests FAILED:
     45 - vcs_test_antialiasing (Failed)
    296 - vcs_test_isoline_labels_multi_label_input_types (Failed)
    297 - vcs_test_isoline_labels (Failed)
    298 - vcs_test_isoline_labelskipdistance (Failed)
    299 - vcs_test_isofill_isoline_labels (Failed)
    321 - vcs_test_animate_isoline_text_labels (Failed)
    322 - vcs_test_animate_isoline_text_labels_colored (Failed)

outvar = outfile['tas']
outtype= outvar.dtype # dtype(float64)

os.remove("newfile.nc")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@doutriaux1 doesn't this need a outfile.close() so that the file handle doesn't prevent deletion?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aashish24
Copy link
Contributor

@doutriaux1 @dnadeau4 @aashish24 I just edited this in the browser and added the missing double quote back in, plus corrected some indentation quirks.. It's triggered a rebuild..

thanks @durack1

@aashish24
Copy link
Contributor

The following tests FAILED:
45 - vcs_test_antialiasing (Failed)
296 - vcs_test_isoline_labels_multi_label_input_types (Failed)
297 - vcs_test_isoline_labels (Failed)
298 - vcs_test_isoline_labelskipdistance (Failed)
299 - vcs_test_isofill_isoline_labels (Failed)
321 - vcs_test_animate_isoline_text_labels (Failed)
322 - vcs_test_animate_isoline_text_labels_colored (Failed)
this is known system issue.

@doutriaux1
Copy link
Contributor Author

@aashish24 can we merge this then?

thanks @durack1 for catching this, it's cleaner now

##bot##skip_commit
@doutriaux1
Copy link
Contributor Author

@aashish24 the skip_commit bit didn't get parsed by the bot server... anyhow ready to merge in when you want.

@aashish24
Copy link
Contributor

thanks @doutriaux1 lets buildbot finish running and then if all looks good, merge it

@doutriaux1
Copy link
Contributor Author

looks like garant lost internet or something, can't see github, not worried about that failure.

aashish24 added a commit that referenced this pull request Sep 2, 2015
@aashish24 aashish24 merged commit 0903f7d into master Sep 2, 2015
@aashish24 aashish24 deleted the issue_186_xml_copy_not_working branch September 2, 2015 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants