Skip to content

Citing SasModels and SasData#32

Merged
krzywon merged 3 commits intomasterfrom
citation-information
Apr 14, 2026
Merged

Citing SasModels and SasData#32
krzywon merged 3 commits intomasterfrom
citation-information

Conversation

@krzywon
Copy link
Copy Markdown
Contributor

@krzywon krzywon commented Apr 9, 2026

This updates our citation site with information on citing and/or acknowledging sasmodels and sasdata.

Copy link
Copy Markdown

@DrPaulSharp DrPaulSharp 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 to me.

Comment thread cite.md Outdated
Comment thread cite.md Outdated
Comment thread cite.md Outdated
Comment thread cite.md Outdated
Comment thread cite.md Outdated
Copy link
Copy Markdown
Contributor

@smk78 smk78 left a comment

Choose a reason for hiding this comment

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

Couple of minor comments, and agree with @butlerpd 's first comment. But othe rthan that LGTM.

@krzywon
Copy link
Copy Markdown
Contributor Author

krzywon commented Apr 13, 2026

I think I've addressed everyone's comments. Please have a second look and merge if ready.

Copy link
Copy Markdown
Member

@butlerpd butlerpd left a comment

Choose a reason for hiding this comment

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

I think this can be merged. The comment on the final acknowledgment is relatively trivial and maybe we can wait till somebody comes up with better verbiage than I.

I do however have another suggestions, also probably for the future. Basically I'm thinking of adding incentives rather than asking for extra work. (more carrot and less stick)

As new versions of each package are released, calculation methods may change or be corrected.
To account for these calculation differences, please cite the version or versions you used to analyze your data.

I think we could point out that accounting for this differences would be critical for reproducibility which in principle is something the user is interested in (or feels they must do somehow). So .... we are providing a way to make it easy for them :-)

Comment thread cite.md Outdated
@smk78
Copy link
Copy Markdown
Contributor

smk78 commented Apr 13, 2026

Suggested change to the citation works for me.

@butlerpd
Copy link
Copy Markdown
Member

I think this should be merged now. This can easily be updated at any time.

@krzywon krzywon merged commit e830277 into master Apr 14, 2026
@krzywon krzywon deleted the citation-information branch April 14, 2026 13:55
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.

4 participants