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 pressure and temperature calculations for lattice parameters #593

Merged
merged 26 commits into from
Jan 4, 2024

Conversation

psavery
Copy link
Collaborator

@psavery psavery commented Dec 19, 2023

The pressure and temperature parameters are set by reading in a JCPDS file, or by the user manually setting them.

These are then able to be used to compute lattice parameters at higher temperatures and pressures than at ambient.

saransh13 and others added 22 commits December 18, 2023 19:34
There are still some compatibility issues, but not as many.

Signed-off-by: Patrick Avery <[email protected]>
Signed-off-by: Patrick Avery <[email protected]>
This is a time consuming function, its input is simple, and its
output doesn't take up too much memory. It's the perfect candidate
for memoization.

Signed-off-by: Patrick Avery <[email protected]>
"np" is the standard, so let's use that.

Signed-off-by: Patrick Avery <[email protected]>
This is so that we can run black on files without specifying the
options manually.

Signed-off-by: Patrick Avery <[email protected]>
Signed-off-by: Patrick Avery <[email protected]>
It is better to initialize it here instead of in the newUnitCell
method, because it should only need to happen once: when the
material is initialized.

Signed-off-by: Patrick Avery <[email protected]>
Signed-off-by: Patrick Avery <[email protected]>
These can be computed and don't need to be stored.

Signed-off-by: Patrick Avery <[email protected]>
@psavery psavery requested a review from saransh13 December 19, 2023 22:05
@pep8speaks
Copy link

pep8speaks commented Dec 19, 2023

Hello @psavery! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2024-01-03 11:41:17 UTC

Signed-off-by: Patrick Avery <[email protected]>
Signed-off-by: Patrick Avery <[email protected]>
This property no longer exists, and we should just set the lattice parameters
instead.

Signed-off-by: Patrick Avery <[email protected]>
These import path aliases should keep old scripts from breaking.

For example, you can now still do all of the following:

```python
from hexrd.crystallography import PlaneData
from hexrd import crystallography
crystallography.PlaneData
```

And using the new import paths, you can do the following:
```python
from hexrd.material.crystallography import PlaneData
from hexrd.material import crystallography
crystallography.PlaneData
```

We also added a check to verify that the import path alias is not
a real file path. If it is, then an exception is raised. This will
keep us from having real file paths overlapping with the alias import
paths.

Signed-off-by: Patrick Avery <[email protected]>
Copy link
Member

@saransh13 saransh13 left a comment

Choose a reason for hiding this comment

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

LGTM!

@saransh13 saransh13 merged commit 3553b51 into master Jan 4, 2024
6 checks passed
@psavery psavery deleted the jcpds-reader branch February 12, 2024 21:40
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.

3 participants