Skip to content

Citation and contributors#200

Merged
krzywon merged 2 commits intomasterfrom
release-automation-script-rework
May 7, 2026
Merged

Citation and contributors#200
krzywon merged 2 commits intomasterfrom
release-automation-script-rework

Conversation

@krzywon
Copy link
Copy Markdown
Collaborator

@krzywon krzywon commented Apr 9, 2026

This adds a contributors list and citation information for the repo.

There are parallel PRs in sasmodels, sasview, and sasview.github.io for this work.

Fixes #189 (mostly)

Copy link
Copy Markdown

@codescene-delta-analysis codescene-delta-analysis Bot left a comment

Choose a reason for hiding this comment

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

No application code in the PR — skipped Code Health checks.

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 requested a review from jellybean2004 May 1, 2026 13:25
Copy link
Copy Markdown
Member

@jellybean2004 jellybean2004 left a comment

Choose a reason for hiding this comment

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

All looks good -- one comment, probably already discussed, but just checking.

Comment thread README.md
The latest instructions for citing and acknowledging are available on our website.

[Citing SasData](http://www.sasview.org/cite/#citing-sasdata)
[Acknowledging SasData](http://www.sasview.org/cite/#acknowledging-sasview)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should there be a separate acknowledgement to SasData (not SasView)?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I think the acknowledgement is general enough that it can be used for any of the packages. Maybe we can mention this at the meeting today.

@krzywon
Copy link
Copy Markdown
Collaborator Author

krzywon commented May 7, 2026

I'm going ahead and merging this for now. If we change to a repo-specific acknowledgement, we can always change the link.

@krzywon krzywon merged commit eae8899 into master May 7, 2026
11 checks passed
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.

provide a Zenodo DOI for sasdata

2 participants