Skip to content

Add doc for function background - #55

Open
StephenNneji wants to merge 2 commits into
RascalSoftware:mainfrom
StephenNneji:update_docs
Open

StephenNneji wants to merge 2 commits into
RascalSoftware:mainfrom
StephenNneji:update_docs

Conversation

@StephenNneji

Copy link
Copy Markdown
Collaborator

#54

uses: actions/setup-python@v4
with:
python-version: '3.11'
python-version: '3.10'

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why has the python version been downgraded? Is this a good idea since version 3.10 has just gone end-of-life? https://devguide.python.org/versions/

Function Backgrounds
====================

RAT supports function backgrounds which use a custom function to describe the background for the experiment.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think this might read better with a comma, as: "RAT supports function backgrounds, which use a custom function to describe the background for the experiment. "


The signature for background functions is shown in the code snippet below. The first argument `xdata` is a vector containing the q points over the simulation range,
which should incorporate the q values of the supplied data and the second argument `params` is a vector containing the background parameters associated with the background
up to 5 can be related to a specific functoin background. The function should return an array with the background value for each of the input simulation points.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

"and the second argument params is a vector containing the background parameters associated with the background up to 5 can be related to a specific functoin background."

Typo in "function". Also it feels like there is some punctuation missing here.

This branch has not been deployed

No deployments
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.

2 participants