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

status of the project? #2

Open
mikapfl opened this issue May 12, 2017 · 8 comments
Open

status of the project? #2

mikapfl opened this issue May 12, 2017 · 8 comments

Comments

@mikapfl
Copy link

mikapfl commented May 12, 2017

Hi,

I'm very interested in using pyFAI to transform GISAXS and GIWAXS images, so I found pygix. However, I'm not quite sure what the status of the project is. As far as I can see, only the angular and q_z vs. q_xy mapping works at the moment (and using transform_image directly, also q_z vs. q_y), but I'm interested in q_x vs q_y mapping, which is useful for highly ordered samples like gratings where q_x=0 is particularily interesting. At the same time, this transformation is highly non-linear and would benefit a lot from pixel splitting (vs stupid interpolation), so pyFAI should be a good fit.

Maybe I'd find the time to implement the appropriate transformations myself, but I haven't yet figured out what would be necessary to provide to the pyFAI functions to do the q_x mapping.

I guess my questions are: Is pygix still in development or maintenance? Are there resources which explain how to use the 2d-histogramming functions of pyFAI so I could try to implement it myself?

Cheers

Mika

@mrakitin
Copy link
Collaborator

mrakitin commented Oct 6, 2021

Hello @tgdane, I wonder how open you are to review/merge potential pull requests? There were some efforts to update the code for the Python 3+ compatibility - see master...ronpandolfi:master for the changes.

Also, the repository is missing the license file. Do you mind adding the file, so that the fork from your repo could be used and developed by others? Our interest at NSLS-II is to package the code with conda, and PyPI release is normally the best source for that package. Do you mind if we maintain the package there?

Interested parties: @gfreychet, @ronpandolfi

Thank you!

@tgdane
Copy link
Owner

tgdane commented Oct 8, 2021

Hey @mikapfl! Sorry that I missed this issue a few years ago! I saw @mrakitin's PR for the python3 changes and have merged. Did you manage to find a way to use the library for your use case?

In terms of the status of the project, I'm no longer working in research and am not actively maintaining the repo but after putting a lot of work, I'd be really happy for someone to take over maintenance! I can contribute where needed from time to time. I have added @mrakitin as a contributor for now but happy to add anyone else.

A long time ago @kif (author of pyFAI - the library which this was based on) and I discussed incorporating pygix into pyFAI. Jerome, I don't know if there's been any interest in grazing incidence data with pyFAI in recent years? I imagine pyFAI has probably changed a lot since I originally wrote pygix!

@mikapfl
Copy link
Author

mikapfl commented Oct 8, 2021

@tgdane Thanks for asking, I managed without pygix since my problems weren't so time-critical.

I'm also not working with grazing incidence scattering anymore, so I'm not the right person to take over maintenance, unfortunately. Because I think it could be really useful for some former colleagues, I would be willing to review some changes or such if that helps. (-:

@kif
Copy link
Contributor

kif commented Oct 8, 2021

Hi Thomas,
Nice to read from you. I have rarely requests about grazing incidence scattering, and especially not from ESRF. That said, the internals from pyFAI have evolved and now, most rebinning engines are really separated from the geometry, which is useful for your project since you use a separated geometry.
I consider tagging pyFAI 1.0 when all rebinning engines are squared out, i.e. all cut corners are properly handled.

I always considered to have a pluggable geometry but this is a project for pyFAI2, on which I did not start working yet.

Cheers,

Jerome

@ctakacs
Copy link

ctakacs commented Jan 27, 2022

Hi Jerome, @kif

Incorporating grazing incidence scattering natively into pyFAI would be absolutely fantastic in my opinion. I forked the original repository and crudely updated the code to work with python 3 (and some newer versions of pyFAI).

I use it for a number of grazing incidents experiments here at SSRL and have a number of users/collaborators using it as well. Getting this to all work is a bit of an endeavor and having it be natively supported into pyFAI would be a real help.

Cheers,
Chris

@mrakitin
Copy link
Collaborator

@tgdane, we just met with @gfreychet and other colleagues to fix the packaging issue (numpy.distutils deprecation). We wonder if you would be willing to transfer the package to the https://github.com/NSLS-II org, so that we can have an ownership of it? We already have a fork in a beamline specific organization, https://github.com/NSLS-II-SMI, but it probably will more visible in the facility-wide org. Please let us know.

Our goal would be to merge @kif's PR #6 and apply the new structure to the project based on https://github.com/scientific-python/cookie/.

Please see the list of tasks we have in mind for now at NSLS-II-SMI#2.

@kif
Copy link
Contributor

kif commented Nov 14, 2024

Hi, Edgar is working on the integration of pygix features into pyFAI... You can give it a try.

@ctakacs
Copy link

ctakacs commented Nov 14, 2024 via email

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

No branches or pull requests

5 participants