Skip to content

System dependent outline#2070

Merged
doutriaux1 merged 3 commits intomasterfrom
system-dependent-outline
Aug 2, 2016
Merged

System dependent outline#2070
doutriaux1 merged 3 commits intomasterfrom
system-dependent-outline

Conversation

@danlipsa
Copy link
Copy Markdown
Contributor

@danlipsa danlipsa commented Aug 1, 2016

No description provided.

danlipsa added 3 commits July 29, 2016 09:32
We increase the parallel projection parallelepiped with 1/1000 so that
it does not overlap with the outline of the dataset. This resulted in
system dependent display of the outline.
@kwrobot
Copy link
Copy Markdown
Member

kwrobot commented Aug 1, 2016

Basic content checks passed!

Branch-at: 337bb70
Acked-by: @kwrobot

@danlipsa
Copy link
Copy Markdown
Contributor Author

danlipsa commented Aug 1, 2016

Baselines changes in:
CDAT/uvcdat-testdata#150

@danlipsa
Copy link
Copy Markdown
Contributor Author

danlipsa commented Aug 1, 2016

@doutriaux1 @aashish24 This fixes the system dependent showing of the top/bottom line for the outline. This was caused by the parallel projection parallelepiped having the same coordinates as the dataset outline. There is still system dependent label placement for isofill/isoline. Please review.

@danlipsa
Copy link
Copy Markdown
Contributor Author

danlipsa commented Aug 1, 2016

@doutriaux1 @aashish24 The following tests fail on mac and linux - I have alternate baselines only for isofill/isoline + labels.
Linux
28 tests failed

The following tests FAILED:
13 - testWrite (Failed)
62 - test_vcs_read_old_scr_2 (Failed)
73 - test_vcs_verify_boxfill_basics (Failed)
121 - test_vcs_boxfill_decreasing_latitude (Failed)
122 - test_vcs_meshfill_no_wrapping (Failed)
413 - test_vcs_isofill_level0 (Failed)
414 - test_vcs_isofill_level1 (Failed)
415 - test_vcs_isofill_level2 (Failed)
469 - test_vcs_matplotlib_colormap (Failed)
549 - dv3d_vector_test (Failed)
550 - dv3d_slider_test (Failed)
551 - Hovmoller_volume_test (Failed)
552 - dv3d_volume_test (Failed)
553 - dv3d_constituents_test (Failed)
554 - dv3d_surface_test (Failed)
555 - dv3d_hovmoller_test (Failed)
580 - cdms2_test_createcopy_lose_dtype (Failed)
581 - CDMS_Test_01 (Failed)
582 - CDMS_Test_02 (Failed)
583 - CDMS_Test_03 (Failed)
584 - CDMS_Test_04 (Failed)
590 - CDMS_Test_10 (Failed)
598 - CDMS_Test_18 (Failed)
599 - CDMS_Test_20 (Failed)
602 - CDMS_Test_write_compressed (Failed)
606 - CDMS_Test_pack (Failed)
609 - CDMS_Test_write_extended_dimension (Failed)
610 - CDMS_Test_del_attributes (Failed)

@aashish24
Copy link
Copy Markdown
Contributor

@danlipsa the changes looks reasonable to me and looke like you deleted the extra baselines as they are no longer needed?

@danlipsa
Copy link
Copy Markdown
Contributor Author

danlipsa commented Aug 1, 2016

@aashish24 @doutriaux1 Indeed, the extra baseline is no longer needed for that extra line on the top. We still need an extra baseline for isoline/isofill + labels. I am not sure why we get different label positions on different platform - this is a different bug.

@danlipsa
Copy link
Copy Markdown
Contributor Author

danlipsa commented Aug 1, 2016

These are the mac os x failed tests:
26 tests failed

The following tests FAILED:
3 - testMPIGrid (Failed)
4 - testAccuracy (Failed)
5 - testMarvel (Failed)
6 - testUnits (Failed)
7 - testRaynaud (Failed)
8 - testDateline (Failed)
9 - testEsmfInit4Pes (Failed)
17 - testMpi10Pes (Failed)
18 - testRemoteGet8Pes (Failed)
20 - testConserv (Failed)
21 - testEsmfVsLibcf (Failed)
22 - testEsmfInterface1 (Failed)
23 - testEsmfInterface2Pes (Failed)
24 - testDistArray6Pes (Failed)
25 - testGhostedDistArray6Pes (Failed)
26 - testEsmfGridToMeshRegridCsrv (Failed)
28 - testEsmf3DSmallesmf2Pes (Failed)
31 - testEsmfSmall2Pes (Failed)
32 - testEsmfSmall4Pes (Failed)
33 - testEsmfSmall8Pes (Failed)
35 - testTasRegrid2Pes (Failed)
46 - testEsmfSalinity2Pes (Failed)
47 - testEsmfSalinity4Pes (Failed)
48 - testEsmfSalinity8Pes (Failed)
52 - testDistSrc4Pes (Failed)
88 - test_vcs_verify_boxfill_basics (Failed)

@danlipsa
Copy link
Copy Markdown
Contributor Author

danlipsa commented Aug 1, 2016

Do: test

@kwrobot
Copy link
Copy Markdown
Member

kwrobot commented Aug 1, 2016

Testing commands handed to buildbot.

Branch-at: 337bb70

@kwrobot
Copy link
Copy Markdown
Member

kwrobot commented Aug 1, 2016

Your merge request has been queued for testing. You may view the test results on CDash or Buildbot.

Branch-at: 337bb70

@doutriaux1 doutriaux1 merged commit 54f2834 into master Aug 2, 2016
@doutriaux1 doutriaux1 deleted the system-dependent-outline branch August 2, 2016 15:41
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.

4 participants