Opened 9 years ago

Closed 9 years ago

#12373 closed defect (fixed)

Possible typo in Units Module Avogadro constant

Reported by: emil Owned by: burcin
Priority: minor Milestone: sage-5.0
Component: symbolics Keywords: units Avogadro constants
Cc: wstein Merged in: sage-5.0.beta4
Authors: Sonseeahray Rucker, William Stein Reviewers: William Stein
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by burcin)

sage: units.amount_of_substance.mole
sage: b.convert(units.amount_of_substance.elementary_entity)
sage: 6.02214179000000e23*elementary_entity

Nist gives 6.022141 2 9000000e23, maybe a typo?
http://physics.nist.gov/cgi-bin/cuu/Value?na

Apply:

Attachments (2)

my.patch (573 bytes) - added by ruckers 9 years ago.
This patches fixes a typo in Avogadro's constant in symbolic.units.pyx
trac_12373-part2.patch (633 bytes) - added by was 9 years ago.
part 2 patch -- add example to illustrate that the fix works.

Download all attachments as: .zip

Change History (12)

comment:1 Changed 9 years ago by was

  • Component changed from PLEASE CHANGE to symbolics
  • Owner changed from tbd to burcin
  • Type changed from PLEASE CHANGE to defect

comment:2 Changed 9 years ago by was

A student (Sonseeahray Rucker) did this in my class, so a patch should appear here soon as a result.

Changed 9 years ago by ruckers

This patches fixes a typo in Avogadro's constant in symbolic.units.pyx

comment:3 Changed 9 years ago by ruckers

  • Status changed from new to needs_review

There's a typo in my typo patch description. The file being patched is symbol.units.py not .pyx.

comment:4 Changed 9 years ago by was

  • Description modified (diff)

Changed 9 years ago by was

part 2 patch -- add example to illustrate that the fix works.

comment:5 Changed 9 years ago by was

  • Status changed from needs_review to positive_review

I added a referee patch. This is a really simple little thing.

comment:6 Changed 9 years ago by burcin

  • Description modified (diff)
  • Reviewers set to W. Stein
  • Work issues check for typo deleted

comment:7 Changed 9 years ago by jdemeyer

Please write the full name including first name.

comment:8 Changed 9 years ago by was

  • Authors changed from D. Ackermann, W. Stein to Sonseeahray Rucke, William Stein
  • Reviewers changed from W. Stein to William Stein

comment:9 Changed 9 years ago by ruckers

  • Authors changed from Sonseeahray Rucke, William Stein to Sonseeahray Rucker, William Stein

Rucke correct to Rucker

comment:10 Changed 9 years ago by jdemeyer

  • Merged in set to sage-5.0.beta4
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.