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

153 how should this be cited #268

Merged
merged 10 commits into from
Sep 28, 2023
Merged

153 how should this be cited #268

merged 10 commits into from
Sep 28, 2023

Conversation

dmdunla
Copy link
Collaborator

@dmdunla dmdunla commented Sep 27, 2023

This will add a new feature to get the APA or BibTeX citation for pyttb.

You will be able to click on "Cite this repository" on the "About" panel.

Closes #153.


📚 Documentation preview 📚: https://pyttb--268.org.readthedocs.build/en/268/

@dmdunla dmdunla linked an issue Sep 27, 2023 that may be closed by this pull request
@dmdunla
Copy link
Collaborator Author

dmdunla commented Sep 27, 2023

BTW, the BibTeX entry appears like this:

@misc{Dunlavy_pyttb_Python_Tensor_2023,
	author = {Dunlavy, Daniel M. and Johnson, Nicholas T. and Myers, Jeremy M. and Medina, Jordan S.},
	month = jun,
	title = {{pyttb: Python Tensor Toolbox, v1.6.2}},
	url = {https://github.com/sandialab/pyttb},
	year = {2023}
}

which renders with siamplain bibliographystype, for example, as

D. M. Dunlavy, N. T. Johnson, J. M. Myers, and J. S. Medina, pyttb: Python Tensor Toolbox, v1.6.2, June 2023, https://github.com/sandialab/pyttb.

@ntjohnson1
Copy link
Collaborator

Hmm, It looks like github magic works for a few other file formats. Thoughts on using a generic .bib file? I am not sure if anyone needs anything besides the LaTex. The benefit is that we can then link it to our read the docs as well/link to the source from our README. Ideally, we reduce the number of places you have to update things when we do a new release.

@dmdunla
Copy link
Collaborator Author

dmdunla commented Sep 27, 2023

I looked in that as you’ll see from my commits. It adds a link but does not parse the file unless it is .cff format. So I opted for that. The link takes you to a file page and renders the .bib text. The .cff file parses and renders APA and .bib text with a help copy button. I thought that would allow for a better flow. If you want to see that, let me know and I’ll show you.

I may be missing your point of using a bib file, too.

@dmdunla
Copy link
Collaborator Author

dmdunla commented Sep 28, 2023

I think I see how the .bib would be more useful now for other docs.

@dmdunla dmdunla added the documentation Improvements or additions to documentation label Sep 28, 2023
@dmdunla dmdunla self-assigned this Sep 28, 2023
Copy link
Collaborator

@ntjohnson1 ntjohnson1 left a comment

Choose a reason for hiding this comment

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

Looks good to me. Small comment is that it would be nice to add a link to the bib from the landing page on RTD and our README.

@dmdunla
Copy link
Collaborator Author

dmdunla commented Sep 28, 2023

Will add placement of the citation to the getting started Issue:

@dmdunla dmdunla merged commit 23b347e into main Sep 28, 2023
5 checks passed
@dmdunla dmdunla deleted the 153-how-should-this-be-cited branch September 28, 2023 11:37
ntjohnson1 pushed a commit to ntjohnson1/pyttb that referenced this pull request Sep 30, 2023
* Add citation file.

* FIxing names.

* Change to bibtex citation

* Revert "FIxing names."

This reverts commit 429c2c1.

* Remove .bib

* Changing type to misc

* Name fix

* Changing to .bib
dmdunla added a commit that referenced this pull request Oct 2, 2023
* CONFIG: Add nbstripout to [dev] and add pre-commit hook

* CONFIG: Test just adding test for pre-commit but not fixing failures

* CONFIG: Test adding notebook with outputs

* Revert "CONFIG: Test adding notebook with outputs"

This reverts commit 1a50093

* CONFIG: Add args to nbstripout

* 153 how should this be cited (#268)

* Add citation file.

* FIxing names.

* Change to bibtex citation

* Revert "FIxing names."

This reverts commit 429c2c1.

* Remove .bib

* Changing type to misc

* Name fix

* Changing to .bib

* Change names of tutorials that are for algorithms to start with algor… (#273)

* Change names of tutorials that are for algorithms to start with algorithm_

* Not an algorithm, so doesn't need algorithm_ prefix.

* Move partial reconstruction of ttensor from standalone tutorial and incorporate into class_ttensor tutorial.

---------

Co-authored-by: Jeremy Myers <jermyer@sandia.gov>

* CONFIG: Fix args and non-compliant notebook

* CONTRIBUTING_DOCS: Add pointer to pre-commit for instructions on running nbstripout

---------

Co-authored-by: Danny Dunlavy <dmdunla@sandia.gov>
Co-authored-by: Jeremy Myers <jmmyers01@wm.edu>
Co-authored-by: Jeremy Myers <jermyer@sandia.gov>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

How should this be cited?
2 participants