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

additional functionality for gridify.m #442

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

vtvivian
Copy link
Contributor

added squarify2 function, which allows step size to be defined explicitly (ebsdGrid.dx) instead of auto calculated
this sometimes leads to seam artefacts in ebsdGrid output, so all elements in ebsdgrid are assigned the nearest ebsd datapoint to remove these seams, with no interpolation of orientation data.

added squarify2 function, which allows step size to be defined explicitly (ebsdGrid.dx) instead of auto calculated
this sometimes leads to seam artefacts in ebsdGrid output, so all elements in ebsdgrid are assigned the nearest ebsd datapoint to remove these seams, with no interpolation of orientation data.
Copy link
Member

@ralfHielscher ralfHielscher left a comment

Choose a reason for hiding this comment

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

Hi Vivian,

thank you for contributing to MTEX. Your functions squarify2 shares many code lines with squarify. In general it is preferable to avoid code duplication as it make maintenance of the code more hard. Maybe you could add setting the unit cell as an option to quarify.

Ralf.

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.

2 participants