[Unicode]   Common Locale Data Repository : Bug Tracking Home | Site Map | Search
 
Modify

CLDR Ticket #11752(closed: fixed)

Opened 4 months ago

Last modified 6 weeks ago

add units to validity/unit.xml (separate bug), make list in spec just selected examples

Reported by: ftang@… Owned by: pedberg
Component: docs-spec Data Locale:
Phase: spec-beta Review: mark
Weeks: Data Xpath:
Xref:

ticket:11563

ticket:11467

ticket:11098

Description

I compare the list with the following
[B] ICU's implementation and the internal table gSubTypes
https://cs.chromium.org/chromium/src/third_party/icu/source/i18n/measunit.cpp?dr=CSs&g=0&l=110)
[C] The table in "6 Unit Elements" of CLDR spec http://unicode.org/reports/tr35/tr35-general.html#Unit_Elements
and also
[D] CLDR validation xml file https://unicode.org/repos/cldr/tags/latest/common/validity/unit.xml

and I found some inconsistency. and I have some questions:

Q1. Both [B] and [C] includes
atmosphere
petabyte
percent
permille
point
but it is not mentioned in [D]. Is this a CLDR bug between [C] and [D]?
Should CLDR

1) include them into the xml file (Change [D]) OR
2) remove them from the UTS35 (Change [C]?

Peter Edberg stated
"Regarding Q1, the right answer is 1, these should be included in the xml validity file. I think I should have done that when adding these units."

Attachments

Change History

comment:1 Changed 4 months ago by mark

  1. The following are all in trunk of validity/unit.xml, so are already handled.

pressure-atmosphere
digital-petabyte
concentr-percent
concentr-permille
length-point

  1. As far as ​http://unicode.org/reports/tr35/tr35-general.html#Unit_Elements goes, I strongly think we should abbreviate the table just a small set (dozen) of examples, and then point to the data file for validation.

comment:2 Changed 4 months ago by pedberg

  • Cc mark added
  • Phase changed from dsub to rc
  • Priority changed from assess to major
  • Status changed from new to accepted
  • Milestone changed from to-assess to 35

comment:3 Changed 3 months ago by pedberg

  • Phase changed from rc to spec-beta

comment:4 Changed 6 weeks ago by pedberg

  • Status changed from accepted to reviewing
  • Xref set to 11563 11467 11098
  • Review set to mark
  • Summary changed from atmosphere, petabyte, percent, permille, point should be added to https://unicode.org/repos/cldr/tags/latest/common/validity/unit.xml to add units to validity/unit.xml (separate bug), make list in spec just selected examples

OK, I eliminated some of the rows in the unit table. But I think it useful to retain rows for:
• Any new units introduced in a given cycle (these were added to the list in separate tickets (cldrbug 11563:, cldrbug 11467:, cldrbug 11098:)
• Any units for which there are special comments or about which there is special discussion or considerations in the body text.
• At least a couple of example units for every category.
So many rows are retained. However, at least it is clear that this is not a comprehensive list.

comment:5 Changed 6 weeks ago by mark

  • Status changed from reviewing to closed
  • Resolution set to fixed
View

Add a comment

Modify Ticket

Action
as closed
Next status will be 'new'
Next status will be 'closed'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.