Skip to content

Add mass to MDAnalysis base units#5439

Open
sici17 wants to merge 2 commits into
MDAnalysis:developfrom
sici17:patch-1
Open

Add mass to MDAnalysis base units#5439
sici17 wants to merge 2 commits into
MDAnalysis:developfrom
sici17:patch-1

Conversation

@sici17

@sici17 sici17 commented Jul 17, 2026

Copy link
Copy Markdown

Fixes #3944

Changes made in this Pull Request:

  • Added mass to the MDAnalysis.units base-units documentation table.
  • Added "mass": "u" to MDANALYSIS_BASE_UNITS.
  • Updated the corresponding base-units test.

I did not update package/CHANGELOG because this is a small documentation/base-units table fix.

The SI value used for u is 1.66053892103219 × 10^-27 kg, which is consistent with the CODATA 2010 constants already used in MDAnalysis.units.

Note that the User Guide currently lists a newer value for u (1.66053906660(50) × 10^-27 kg). This PR intentionally keeps the MDAnalysis.units API documentation consistent with the existing CODATA 2010 values used by the module, following the maintainer guidance in the issue.

LLM / AI generated code disclosure

LLMs or other AI-powered tools (beyond simple IDE use cases) were used in this contribution: yes

PR Checklist

  • Issue raised/referenced?
  • Tests updated/added?
  • Documentation updated/added?
  • package/CHANGELOG file updated?
  • Is your name in package/AUTHORS? (If it is not, add it!)
  • LLM/AI disclosure was updated.

Developers Certificate of Origin

I certify that I can submit this code contribution as described in the Developer Certificate of Origin, under the MDAnalysis LICENSE.

@read-the-docs-community

Copy link
Copy Markdown

Documentation build overview

📚 MDAnalysis | 🛠️ Build #33636596 | 📁 Comparing 6fe19af against latest (3d3a53e)

  🔍 Preview build  

4 files changed
± index.html
± documentation_pages/units.html
± _modules/MDAnalysis/units.html
± documentation_pages/analysis/wbridge_analysis.html

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.

mass unit not listed in 'Base units' table

1 participant