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

CLDR Ticket #7119(accepted data)

Opened 3 years ago

Last modified 21 months ago

Some duplicate minimal pairs

Reported by: emmons Owned by: mark
Component: plurals Data Locale:
Phase: dsub Review:
Weeks: Data Xpath:
Xref:

Description

In reviewing the known issues, ran across the following in the plural unit tests:

  TestPluralCompleteness {
    Error: ar	cardinal	 duplicate samples: [zero, other] => «{0} كتاب»
    Error: bn	cardinal	 duplicate samples: [one, other] => «সসে {0}টি আপেল খেল»
    Error: vi	ordinal	 duplicate samples: [one, other] => «Rẽ vào lối rẽ thứ {0} bên phải.»
  } (0.029s) FAILED (3 failures)

Attachments

Change History

comment:1 Changed 3 years ago by mark

Pull the logKnownIssue when this is done.

comment:2 Changed 3 years ago by mark

  • Owner changed from anybody to mark
  • Priority changed from assess to major
  • Status changed from new to assigned
  • Component changed from unknown to data-plurals
  • Milestone changed from UNSCH to 26dsub

comment:3 Changed 3 years ago by mark

  • Milestone changed from 26dsub to 26rc

comment:4 Changed 3 years ago by tomzhang

Here are my finding and suggestion on the ticket:

2 possible reasons (fix 1 will work):

  1. The data is duplicate (e.g. sample uses same data "1" for "one", 1 for "others")
  2. 1 is right, but the way we check is wrong (e.g. we only accept 1 distint pattern for "one" & "others" separately, which in 1 they have the same pattern "1")

2 files are involved:

  1. PluralRulesFactory.java
  2. TestSupplementalInfo.java

2 possible suggestion(fix 1 will work):
Solution 1: file 1:

change 1 of line 405/407

1 of line 414/415
2 of line 416/418/419
1 of line 775/776

Solution 2: file 2 change check condition on line 1311 from (!= 1) to (<1)?

comment:5 Changed 3 years ago by mark

No, we can't do that. The problem is that the data collides. We need at least one new sample in each case that *doesn't* collide, so we need to get native language users to supply.

comment:6 Changed 3 years ago by mark

  • Milestone changed from 26rc to 27dsub

These need to be sync'ed with the data collection, so postponing to 27, before data submission.

comment:7 Changed 3 years ago by markus

  • Phase set to dsub
  • Milestone changed from 27dsub to 27

comment:8 Changed 2 years ago by mark

  • Milestone changed from 27 to 28

comment:9 Changed 2 years ago by markus

  • Type changed from unknown to data

comment:10 Changed 2 years ago by srl

  • Status changed from assigned to accepted

comment:11 Changed 22 months ago by mark

  • Milestone changed from 28 to 29

comment:12 Changed 21 months ago by emmons

  • Milestone changed from 29 to upcoming

Automatic move of all 29 -> upcoming

View

Add a comment

Modify Ticket

Action
as accepted
Author


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

 
Note: See TracTickets for help on using tickets.