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 publication lists #355

Merged
merged 2 commits into from
Apr 1, 2024
Merged

Update publication lists #355

merged 2 commits into from
Apr 1, 2024

Conversation

abachma2
Copy link
Member

@abachma2 abachma2 commented Apr 1, 2024

This PR updates some of the publications listed on the website. Currently, the only change is to add the source/cite/citecyc.bib file to the bibtext_bibfiles list in source/conf.py so that the publications about difference Cyclus versions appears. This fixes one of the items identified in #337.

@nuclearkatie also mentioned in #337 an item to update this list. Were there any specific publications you had in mind? The lists on the website seem to be focused on the core of Cyclus, and not as much on applications. Do we have any newer publications that fit this bill? I think expanding this list to publications that just use Cyclus will be burdensome to maintain, but I'm open to ideas.

@abachma2 abachma2 self-assigned this Apr 1, 2024
@abachma2 abachma2 added this to the v1.6 milestone Apr 1, 2024
Copy link
Member

@gonuke gonuke left a comment

Choose a reason for hiding this comment

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

While you are at it, can you add others. I found some in the newsletters directory

@abachma2
Copy link
Member Author

abachma2 commented Apr 1, 2024

I really like the idea in #356, so let's not merge this until that is completed, and it can be a part of this PR.

@gonuke
Copy link
Member

gonuke commented Apr 1, 2024

It may take some time to implement #356. Might be better to go ahead with this until we sort that out properly.

Copy link
Member

@gonuke gonuke left a comment

Choose a reason for hiding this comment

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

I'm happy to approve this now, and make #356 a separate PR. This makes everything work immediately until we can sort out how to do #356.

@abachma2
Copy link
Member Author

abachma2 commented Apr 1, 2024

That works for me. Merge away

@gonuke gonuke merged commit 6ad98b6 into cyclus:source Apr 1, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants