Skip to content

Don't use the default multiplicity parameter from sasview - #751

Open
pkienzle wants to merge 1 commit into
masterfrom
sasview4085-multiplicity-ignored
Open

Don't use the default multiplicity parameter from sasview#751
pkienzle wants to merge 1 commit into
masterfrom
sasview4085-multiplicity-ignored

Conversation

@pkienzle

Copy link
Copy Markdown
Contributor

Fixes SasView/sasview#4085 (comment)

For multiplicity controls, sasview sets the multiplicity attribute in the sasview shim rather than multiplicity control parameter. This attribute is now used when evaluating the model, rather than the default multiplicity=1.

@codescene-access codescene-access Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No quality gates enabled for this code.

See analysis details in CodeScene

Quality Gate Profile: Custom Configuration
Install CodeScene MCP: safeguard and uplift AI-generated code. Catch issues early with our IDE extension and CLI tool.

@krzywon krzywon left a comment

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.

Code looks fine. Seems to work.

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.

Core multi shell model not properly calculating

2 participants