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

Option to only get citations of packages in Project.toml and not in Manifest.toml #18

Closed
Datseris opened this issue Sep 13, 2021 · 6 comments

Comments

@Datseris
Copy link

I just used get_tool_citation() which gave me citation for 50 more names that I've ever seen in my life. But that's not really how citing works; you cite things you used directly and you have seen directly. If e.g., you cite a scientific paper from the 80s, you don't also cite all the papers that paper cites.

An option of only collecting the packages explicitly in the Project.toml file would resolve this.

@Datseris
Copy link
Author

E.g., even though I do not use Measurements.jl in my work, aparrently some package has some deep dependency on Measurements.jl and its included in my "citation clipboard".

@SebastianM-C
Copy link
Owner

SebastianM-C commented Sep 13, 2021

You can use only_direct=true in this case (see https://sebastianm-c.github.io/PkgCite.jl/stable/#PkgCite.get_citations). Does that solve the issue?

@Datseris
Copy link
Author

Oh, sorry I have not realized there is an online hosted docs, I was reading the README. Yes this is a good solution, but would be also nice to have in the get_tool_citation() which copies this convenient string into clipboard.

@Datseris
Copy link
Author

Oh that's already possible. Thanks!

@Datseris
Copy link
Author

(p.s.: I think only_direct = true should be the default argument. It doesn't seem natural to reference any package in the Manifest)

@SebastianM-C
Copy link
Owner

There was a discussion about this on slack and there were people arguing for both sides, so that's why I left it like this.

The only_direct option should also work with get_tool_citation, but I think there is an omission in the docs. Thanks for bringing it to my attention.

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

No branches or pull requests

2 participants