search

dkriegner / xrayutilities

xrayutilities - a package with useful scripts for X-ray diffraction physicists
http://xrayutilities.sourceforge.io
GNU General Public License v2.0
83 stars 29 forks source link

allow float for cch1 and cch2 #141

Closed trdd closed 2 years ago

trdd commented 2 years ago

Hi Dominik,

mir ist aufgefallen, dass cch1 und cch2 auf int gesetzt werden obwohl die c-Routine auch float nehmen würde. Bei kleinen Winkeln machen 0.5 Pixel schon manchmal ganz schön was aus. Wäre cool wenn du das mit upstream nehmen könntest.

Viele Grüße, Tobias (Ritschel)

blokhin commented 2 years ago

For those who don’t read German:

it seems cch1 and cch2 are integers, but c-Routine also accepts floats. With the smaller angles even 0.5 matters.

dkriegner commented 2 years ago

thanks to both of you. This clearly is a bug. Even the docstring of the python-function says it can be a float. I will merge it.