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

feat: Add a grid feature #1082

Merged
merged 1 commit into from
Aug 13, 2021
Merged

feat: Add a grid feature #1082

merged 1 commit into from
Aug 13, 2021

Conversation

manthey
Copy link
Contributor

@manthey manthey commented Apr 22, 2021

This refactors the contourFeature to avoid code duplication, as the grid and contour features are nearly the same. Specifically, the contour feature has data values at each vertex of a mesh and the grid feature has data values on the elements of the mesh. The grid feature has solid colors for each mesh element, while the contour feature has interpolated values between the vertices.

This adds a small tutorial.

This closes #545.

@aashish24
Copy link
Member

@manthey build issues - holding review until it passes

@manthey
Copy link
Contributor Author

manthey commented Apr 22, 2021

It passes locally (and in a docker build environment). I'm not sure what is failing in travis. This is built on top of PR #1079, so if that gets merged, I'll rebase and see if kicking off travis again fixes it.

@manthey manthey force-pushed the grid-feature branch 2 times, most recently from db5554a to f2bb8a8 Compare April 27, 2021 18:10
Base automatically changed from countour-ranges to master April 28, 2021 15:02
@aashish24
Copy link
Member

@manthey I have started the review process on this PR

@manthey
Copy link
Contributor Author

manthey commented May 10, 2021

@manthey I have started the review process on this PR

Great. I just rebased it against master, since otherwise it wouldn't merge on its own (since #1084 was merged).

@manthey
Copy link
Contributor Author

manthey commented Jun 8, 2021

I rebased this to use the github actions.

@manthey manthey changed the title Add a grid feature feat: Add a grid feature Jun 15, 2021
This refactors the contourFeature to avoid code duplication, as the grid
and contour features are nearly the same.  Specifically, the contour
feature has data values at each vertex of a mesh and the grid feature
has data values on the elements of the mesh.  The grid feature has solid
colors for each mesh element, while the contour feature has interpolated
values between the vertices.

This adds a small tutorial.

This closes #545.
Copy link
Member

@aashish24 aashish24 left a comment

Choose a reason for hiding this comment

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

Earlier approved these changes and recently we updated it to use current master

@manthey manthey merged commit b5a6d9a into master Aug 13, 2021
@manthey manthey deleted the grid-feature branch August 13, 2021 17:05
@github-actions
Copy link

🎉 This PR is included in version 1.4.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement Grid Feature
2 participants