Skip to content

Add missing DOIs to bibliography per JOSS reference check#108

Merged
TomeHirata merged 1 commit intomainfrom
fix/paper-dois
Apr 29, 2026
Merged

Add missing DOIs to bibliography per JOSS reference check#108
TomeHirata merged 1 commit intomainfrom
fix/paper-dois

Conversation

@TomeHirata
Copy link
Copy Markdown
Collaborator

@TomeHirata TomeHirata commented Apr 29, 2026

Summary

Added missing DOIs

MISSING DOIs (added):

  • Byambadalai et al. 2024 (ICML): 10.2139/ssrn.4910058
  • Byambadalai et al. 2025 (ICML): 10.2139/ssrn.5286083
  • Hirata et al. 2025 (arXiv): 10.2139/ssrn.5347645
  • Byambadalai et al. 2025 (NeurIPS): 10.2139/ssrn.5504858

SKIP DOIs (added where available):

  • DoWhy: 10.48550/arXiv.2011.04216
  • EconML (KDD 2021): 10.1145/3447548.3470792

Remaining SKIP entries (no DOIs available):

  • Fisher 1935 — pre-DOI era book
  • Scikit-learn — JMLR does not assign DOIs
  • Hillstrom 2008 — blog post

Address JOSS automated reference check (openjournals/joss-reviews#10466):

MISSING DOIs (added):
- Byambadalai et al. 2024 (ICML): 10.2139/ssrn.4910058
- Byambadalai et al. 2025 (ICML): 10.2139/ssrn.5286083
- Hirata et al. 2025 (arXiv): 10.2139/ssrn.5347645
- Byambadalai et al. 2025 (NeurIPS): 10.2139/ssrn.5504858

SKIP DOIs (added where available):
- DoWhy: 10.48550/arXiv.2011.04216
- EconML (KDD 2021): 10.1145/3447548.3470792

Remaining SKIP entries have no DOIs available:
- Fisher 1935 (pre-DOI era book)
- Scikit-learn (JMLR does not assign DOIs)
- Hillstrom 2008 (blog post)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 29, 2026 07:18
@TomeHirata TomeHirata merged commit 8546e79 into main Apr 29, 2026
12 checks passed
@TomeHirata TomeHirata deleted the fix/paper-dois branch April 29, 2026 07:21
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the JOSS paper bibliography to address automated reference checks by adding DOIs to previously flagged entries, improving reference completeness for publication.

Changes:

  • Added DOIs for several Byambadalai et al. and Hirata et al. references.
  • Added DOIs for DoWhy (arXiv) and EconML (KDD 2021).
  • Updated some entries’ BibTeX types/fields (e.g., @misc@inproceedings, author formatting, proceedings metadata).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread paper/paper.bib
Comment on lines +1 to +10
@inproceedings{byambadalai2024estimatingdistributionaltreatmenteffects,
title={Estimating Distributional Treatment Effects in Randomized Experiments: Machine Learning for Variance Reduction},
author={Undral Byambadalai and Tatsushi Oka and Shota Yasui},
author={Byambadalai, Undral and Oka, Tatsushi and Yasui, Shota},
booktitle={Proceedings of the 41st International Conference on Machine Learning},
pages={5082--5113},
year={2024},
eprint={2407.16037},
archivePrefix={arXiv},
primaryClass={econ.EM},
url={https://arxiv.org/abs/2407.16037},
volume={235},
series={Proceedings of Machine Learning Research},
publisher={PMLR},
doi={10.2139/ssrn.4910058}
Copy link

Copilot AI Apr 29, 2026

Choose a reason for hiding this comment

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

These entries were converted from arXiv-style @misc to @inproceedings and now only include an SSRN DOI (no arXiv eprint fields and no URL). If the intent is just to satisfy the JOSS DOI check, consider keeping a URL (e.g., proceedings/arXiv) as well so the reference remains directly retrievable even if the DOI is to a different landing page.

Copilot uses AI. Check for mistakes.
Comment thread paper/paper.bib
series={ICML'25},
url={https://arxiv.org/abs/2506.05945}
series={Proceedings of Machine Learning Research},
publisher={PMLR},
Copy link

Copilot AI Apr 29, 2026

Choose a reason for hiding this comment

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

This ICML 2025 entry drops the URL field compared to the previous version. Since the DOI used here is an SSRN DOI, adding back a stable URL (arXiv/proceedings page) would help readers locate the paper directly and keeps the reference metadata more complete.

Suggested change
publisher={PMLR},
publisher={PMLR},
url={https://doi.org/10.2139/ssrn.5286083},

Copilot uses AI. Check for mistakes.
Comment thread paper/paper.bib
archivePrefix={arXiv},
primaryClass={econ.EM},
url={https://arxiv.org/abs/2509.15594}
doi={10.2139/ssrn.5504858}
Copy link

Copilot AI Apr 29, 2026

Choose a reason for hiding this comment

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

This NeurIPS 2025 entry currently has only title/authors/booktitle/year plus an SSRN DOI. If page numbers/venue details (or at least a URL) aren’t available yet, consider keeping the arXiv URL/eprint metadata too so the citation remains usable outside of DOI resolution.

Suggested change
doi={10.2139/ssrn.5504858}
doi={10.2139/ssrn.5504858},
url={https://doi.org/10.2139/ssrn.5504858}

Copilot uses AI. Check for mistakes.
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

Successfully merging this pull request may close these issues.

2 participants