Skip to content

Conversation

@tristan-f-r
Copy link
Collaborator

@tristan-f-r tristan-f-r commented Jul 22, 2025

Closes #5.

I'm a little wary of using the .bib files as artifacts (they are very easily reproducible, though, in a similar fashion to our other generate-inputs/parse-outputs tests) because it does feel redundant, but I did want to leave that in, in case this actually is very useful to look for DOI correctness in future algorithm implementations.


This conflicts with #292, but since it's a small change, it doesn't really matter which matter it's merged - just mentioning it for a heads up.

More importantly, though, this does need to change depending on when #320 is merged, since this needs to have attribution-{rev} attached to it.

Note

I don't know what the best practices are for bibtex? If there's formatting suggestions, I'm very open to them 👍

@tristan-f-r tristan-f-r added the enhancement New feature or request label Jul 22, 2025
@tristan-f-r tristan-f-r added the awaiting-author Author of the PR needs to fix something from a review / etc. label Sep 25, 2025
@read-the-docs-community
Copy link

Documentation build overview

📚 spras | 🛠️ Build #29714542 | 📁 Comparing 99790f7 against latest (d4cbe34)


🔍 Preview build

Show files changed (1 files in total): 📝 1 modified | ➕ 0 added | ➖ 0 deleted
File Status
contributing/index.html 📝 modified

@tristan-f-r tristan-f-r removed the awaiting-author Author of the PR needs to fix something from a review / etc. label Sep 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pathway reconstruction algorithm attribution

2 participants