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 literature.py to optionally create tex/bib output #908

Merged
merged 10 commits into from
Mar 14, 2023

Conversation

rieder
Copy link
Member

@rieder rieder commented Nov 24, 2022

This would allow AMUSE to recognise "--bibtex" to automatically create a .tex and .bib file, making it easier to include all relevant references in an article.
Ideally we'd have some string available for each AMUSE package, generated from a bibkey, that can simply be included here.

@rieder rieder changed the title Some initial work to optionally create tex/bib files Update literature.py to optionally create tex/bib output Nov 24, 2022
@rieder rieder requested a review from spzwart November 24, 2022 18:03
@rieder rieder self-assigned this Nov 24, 2022
@rieder
Copy link
Member Author

rieder commented Nov 24, 2022

References need to be standardised for this, in the following

  • Each literature line must start with either "ADS:(ADS bibcode)" or "DOI:(DOI code)"
  • After this, it doesn't matter, but ideally it would just be the authors' names, year of publishing etc. (not the title)

@rieder rieder removed the request for review from silviatoonen November 24, 2022 20:08
@rieder
Copy link
Member Author

rieder commented Nov 24, 2022

Mostly done. Ideally we should include a bibfile for all codes in AMUSE, probably just one big (not that big) file would do here.

@rieder
Copy link
Member Author

rieder commented Nov 25, 2022

I think this works and can be merged.

@rieder rieder requested review from a team as code owners February 17, 2023 09:37
# Conflicts:
#	src/amuse/community/secularmultiple/interface.py
@rieder rieder merged commit 5666436 into amusecode:main Mar 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant