HEXRD / hexrd

A cross-platform, open-source library for the analysis of X-ray diffraction data.
Other
55 stars 26 forks source link

no inverse_distortion hexrd.extensions #586

Closed donald-e-boyce closed 9 months ago

donald-e-boyce commented 9 months ago

hexrd.extensions.inverse_distortion is referenced below but does not exist. https://github.com/HEXRD/hexrd/blob/5f742ea99f403bb2a7b5079d0497ca284c801143/hexrd/distortion/ge_41rt.py#L12

ZackAttack614 commented 9 months ago

@donald-e-boyce How are you running this? Have you pulled the repo, or did you install this via conda? If you pulled the repo, did you run the setup.py file, or run the make file located in the CPP subdirectory in the transforms module?

donald-e-boyce commented 9 months ago

Oh thanks. I built it recently (yesterday I think) but before that pull request. So I have to rebuild. I'll do that.

On Sat, Nov 25, 2023 at 11:56 AM Zack @.***> wrote:

@donald-e-boyce https://github.com/donald-e-boyce How are you running this? Have you pulled the repo, or did you install this via conda? If you pulled the repo, did you run the setup.py file, or run the make file located in the CPP subdirectory in the transforms module?

— Reply to this email directly, view it on GitHub https://github.com/HEXRD/hexrd/issues/586#issuecomment-1826376679, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIWT6MCDNLNOJ7RH4NCLPDYGIPNHAVCNFSM6AAAAAA72GVQPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRWGM3TMNRXHE . You are receiving this because you were mentioned.Message ID: @.***>

donald-e-boyce commented 9 months ago

Building from scratch, it failed due to not finding pybind11. I am running the development version in the instructions. It failed on the command:

CONDA_BUILD=1 pip install --no-build-isolation --no-deps -U -e hexrd-dev/

Here is the error:

          exec(code, locals())
        File "<string>", line 10, in <module>
      ModuleNotFoundError: No module named 'pybind11'
      [end of output]

After I installed pybind11 with conda, it built successfully and the hexrd command line comes up.

psavery commented 9 months ago

I can fix this later