Skip to content

Compute BB of a projected dataset from all points.#410

Closed
danlipsa wants to merge 2 commits intoCDAT:masterfrom
danlipsa:projected_isofill_bounds
Closed

Compute BB of a projected dataset from all points.#410
danlipsa wants to merge 2 commits intoCDAT:masterfrom
danlipsa:projected_isofill_bounds

Conversation

@danlipsa
Copy link
Copy Markdown
Contributor

@danlipsa danlipsa commented Jun 4, 2019

The border in Cartesian space may no corespond to the border in
the projected space.

Baselines in
CDAT/uvcdat-testdata#212

@danlipsa
Copy link
Copy Markdown
Contributor Author

danlipsa commented Jun 4, 2019

@doutriaux1 @scottwittenburg Please review.

Copy link
Copy Markdown
Collaborator

@scottwittenburg scottwittenburg left a comment

Choose a reason for hiding this comment

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

Looks good to me if it's passing all the tests.

Comment thread vcs/vcs2vtk.py
@danlipsa
Copy link
Copy Markdown
Contributor Author

danlipsa commented Jun 4, 2019

@doutriaux1 Do you know what is going on with coveralls test failure? This is why the first CircleCI is failing.

@doutriaux1
Copy link
Copy Markdown
Contributor

@danlipsa don't worry about coverage, but could please add a test for this, so we can catch if it breaks again. Thanks.

@danlipsa
Copy link
Copy Markdown
Contributor Author

danlipsa commented Jun 4, 2019

@doutriaux1 I added a test for this. With master, this test gets the small globe we've seen in the animation example.

@danlipsa
Copy link
Copy Markdown
Contributor Author

danlipsa commented Jun 4, 2019

@doutriaux1 Do you need to merge uvdat-testdata before the tests pass? I added a branch with the same name but it seems that it is not enough. The test fails because it cannot find the baseline.

@danlipsa
Copy link
Copy Markdown
Contributor Author

danlipsa commented Jun 4, 2019

Oh, I think data it needs to be a branch to the origin it cannot be a branch on my fork. I'll try that.

danlipsa added 2 commits June 4, 2019 16:52
The border in Cartesian space may no corespond to the border in
the projected space.
@danlipsa danlipsa force-pushed the projected_isofill_bounds branch from 3ab7a3a to 7f302db Compare June 4, 2019 20:53
@danlipsa
Copy link
Copy Markdown
Contributor Author

danlipsa commented Jun 4, 2019

Try a a branch of origin in
#411

@danlipsa danlipsa closed this Jun 4, 2019
@downiec downiec added this to the 8.2 milestone Oct 21, 2019
@downiec downiec modified the milestones: 8.2, 8.2.1 Jul 27, 2020
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