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

Add multiphase texture update API #200

Merged
merged 7 commits into from
Jun 21, 2024
Merged

Add multiphase texture update API #200

merged 7 commits into from
Jun 21, 2024

Conversation

adigitoleo
Copy link
Member

@adigitoleo adigitoleo commented Jun 17, 2024

feat: Add single-call API for multiphase texture updates

On line 638 of minerals.py.

Closes #198. Also rearranges definitions so that globals are above classes are above functions. This allows for mypy type hints of the multiphase function args.

fix: Update voigt averaging to use new StiffnessTensors API

This fix was brought to you by mypy static type linting.

Closes #198. Also rearranges definitions so that globals are above
classes are above functions. This allows for mypy type hints of the
multiphase function args.
This fix was brought to you by mypy static type linting.
@adigitoleo adigitoleo changed the title Add update multiphase Add multiphase texture update API Jun 17, 2024
@adigitoleo

This comment was marked as resolved.

Copy link
Member

@Patol75 Patol75 left a comment

Choose a reason for hiding this comment

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

Looking good. Just one suggestion. Also, what was the change for voigt_averages?

src/pydrex/minerals.py Outdated Show resolved Hide resolved
@Patol75 Patol75 merged commit f819fd3 into main Jun 21, 2024
10 checks passed
@Patol75 Patol75 deleted the add-update-multiphase branch June 21, 2024 03:48
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.

Add multiphase update_orientations
2 participants