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

Hkl utils #76

Merged
merged 62 commits into from
Nov 13, 2023
Merged

Hkl utils #76

merged 62 commits into from
Nov 13, 2023

Conversation

strempfer
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Oct 18, 2023

Codecov Report

Merging #76 (be121ff) into main (8f68560) will decrease coverage by 3.08%.
Report is 9 commits behind head on main.
The diff coverage is 0.00%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             main      #76      +/-   ##
==========================================
- Coverage   35.95%   32.88%   -3.08%     
==========================================
  Files          11       11              
  Lines        1777     1943     +166     
==========================================
  Hits          639      639              
- Misses       1138     1304     +166     
Files Coverage Δ
polartools/hkl_utils.py 0.00% <0.00%> (ø)

add function to remove reflection from list
change order of arguments in setor0 and setor1
implement functions write_config and read_config
Modify write_config
changes in write_config
implement function functions() to list available functions
update functions()
polartools/hkl_utils.py Fixed Show resolved Hide resolved
)


from inspect import getmembers, isfunction
Copy link
Contributor

Choose a reason for hiding this comment

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

The error in flake8 is a style one and comes from these imports in the middle of the file. You need to move them to the top.

from hklpy.hkl import user, util


def functions(select="all"):
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure what this function does.

polartools/hkl_utils.py Outdated Show resolved Hide resolved
polartools/hkl_utils.py Fixed Show resolved Hide resolved
remove blanks
use black -l 80 instead of -l 79
update functions()
help added to new functions
modify function_list in functions()
simplify code for inputs
fix error in string
fix print in sampleList
fix error in set_orienting
implement e4c in sampleList
fix indentation, black
fix index for e4c
implement fourc
add function set_experiment()
fix bug in setor0/1
remove commented lines
polartools/hkl_utils.py Fixed Show fixed Hide fixed
black formatting
fix bug in or1()
add function update_lattice()
black formatting
polartools/hkl_utils.py Fixed Show fixed Hide fixed
replace 'diffract and 'fourc' by check for number of axes. Remove list_functions() and set_experiment()
fix issues in update_lattice
remove comments
omega=float(th),
chi=float(chi),
phi=float(phi),
gamma=float(gamma),

Check failure

Code scanning / CodeQL

Potentially uninitialized local variable Error

Local variable 'gamma' may be used before it is initialized.
chi=float(chi),
phi=float(phi),
gamma=float(gamma),
mu=float(mu),

Check failure

Code scanning / CodeQL

Potentially uninitialized local variable Error

Local variable 'mu' may be used before it is initialized.
omega=float(th),
chi=float(chi),
phi=float(phi),
gamma=float(gamma),

Check failure

Code scanning / CodeQL

Potentially uninitialized local variable Error

Local variable 'gamma' may be used before it is initialized.
chi=float(chi),
phi=float(phi),
gamma=float(gamma),
mu=float(mu),

Check failure

Code scanning / CodeQL

Potentially uninitialized local variable Error

Local variable 'mu' may be used before it is initialized.
change Theta to Eta and Gamma to Nu
black formatting
polartools/hkl_utils.py Fixed Show fixed Hide fixed
polartools/hkl_utils.py Fixed Show fixed Hide fixed
polartools/hkl_utils.py Fixed Show fixed Hide fixed
add output text to read_config
fix bug in write_config
Add _geom_=current_diffractometer() to all functions.
change _geom_.forward to first orienting reflection and add UB-calc into read_config()
add compute_UB in functions
add else statements
@strempfer strempfer merged commit 20b174f into main Nov 13, 2023
7 of 8 checks passed
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