-
Notifications
You must be signed in to change notification settings - Fork 60
union system updated for McXtrace 3 #2282
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
base: main
Are you sure you want to change the base?
Conversation
Including new Compton and Powder scattering process
|
also this description does not yet contain surface refraction |
copying over stucture and new developments from McStas:union-lib
|
@ebknudsen great, thanks! As discussed elsewhere I will see if I can merge / cut / paste / add some of the improvements from the neutron side later. (Will very likely not happen this side of new years...) |
|
@willend Thanks - this will come at its own pace. Remaining to be done:
In the longer term it could be desirable to split union-lib into a common and a neutron/X-ray specific description, to avoid falling out of sync, but that's not for now/this PR |
|
Agree on all of the above |
…come from McStas)
|
@ebknudsen |
|
@ebknudsen found it / worked around. It seems the default conda-solver is simply overwhelmed/hung. Have for now installed mamba and used that… I guess it is time for McCode to make use of something like https://github.com/mamba-org/setup-micromamba in the CI instead |
Including new Compton and Powder scattering process
Free-form text area
Please describe what your PR is adding in terms of features or bugfixes:
This PR adds work done for HALO-Xray in terms of implementing the Union system also for McXtrace 3 and include Compton and Powder scattering processes.
It is currently in a draft stage (i.e. not cleared for merge), using the old Union structure with Geometry_functions.c etc. in separate files. This is intended to be merged into a union-lib.c style description.
Development OS / boundary conditions
Please describe what OS you developed and tested your additions on, and if any special dependencies are required:
Debian buster
PR Checklist for contributing to McStas/McXtrace
For a coherent and useful contribution to McStas/McXtrace, please fill in relevant parts of the checklist:
My contribution includes patches to an existing component file
mcdocutility and rendered a reasonable documentation page for the component (please attach as screenshot in comments!)mctestutility to test one or more instruments making use of the component (please attachmcviewtestreport as screenshot in comments)mcrun --c-lint"linter" and followed advice to remove most / all warnings that are raisedMy contribution includes patches to an existing instrument file
mcdocutility and rendered a reasonable documentation page for the instrument (please attach as screenshot in comments!)mctestutility to test the instrument (please attachmcviewtestreport as screenshot in comments)mcrun --c-lint"linter" and followed advice to remove most / all warnings that are raisedMy contribution includes a new component file
mcdocutility and rendered a reasonable documentation page for the component (please attach as screenshot in comments!)contribcomponent categoryMy contribution includes a new instrument file
mcdocutility and rendered a reasonable documentation page for the instrument (please attach as screenshot in comments!)%Example:line to describe expected behaviourmcrun --c-lint"linter" and followed advice to remove most / all warnings that are raisedexampleshierarchy in a folder in the style ofexamples/ESS/New_stuff/New_stuff.instrexamplefolder, but if general use I have placed it in the globaldatafolder.My work touches the code-generator in mccode/src
My work touches / adds to the runtime lib code (.c,.h etc in multiple locations
My PR is meant to fix a specific, existing issue
My contribution contains something else