Skip to content
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

Update overview.md #948

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update overview.md #948

wants to merge 1 commit into from

Conversation

aaronjohnson
Copy link

Fixed a broken link to an external PDF.

Perhaps there is a more canonical location.

Fixed a broken link to an external PDF.

Perhaps there is a more canonical location.
@CLAassistant
Copy link

CLAassistant commented Dec 3, 2024

CLA assistant check
All committers have signed the CLA.

@coveralls
Copy link

Coverage Status

coverage: 98.783%. remained the same
when pulling 202a597 on aaronjohnson:patch-1
into e45b999 on uber:master.

@ajfriend
Copy link
Contributor

ajfriend commented Dec 4, 2024

I think, technically, we should be pointing to the publisher's website here: https://www.tandfonline.com/doi/abs/10.1559/152304003100011090

Unfortunately, the PDF is paywalled at that link, but that's the proper way to handle this. But hopefully it isn't too hard for an enterprising person to find legitimate access to the PDF via other sites (including Dr. Sahr's website). I just don't think we can link it directly here.

@sahrk
Copy link
Collaborator

sahrk commented Dec 4, 2024

I think, technically, we should be pointing to the publisher's website

Agreed. The paper is owned by the publisher, so the only legal link to it is the publisher's website. I note that the paper is available on LibGen.

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.

5 participants