Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Localisation of numbers #38

Closed
opoudjis opened this issue Oct 22, 2020 · 6 comments
Closed

Localisation of numbers #38

opoudjis opened this issue Oct 22, 2020 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@opoudjis
Copy link
Contributor

In order to enable localisation of number rendering in BIPM, will do the following.

  • All numbers to be localised in rendering must be marked up as stem:[]. Sorry @manuel489. So a number like 1337 must be marked up as stem:[1337].
  • Will create gem in plurimath to iterate through MathML, and change the rendering of any numbers in MathML in Presentation XML to localised versions of the numbers. BIPM's requirements are satisfied by twitter_cldr's .localize(:fr).to_s. twitter_cldr's localisations are configured in YAML, and we might need to inherit localisation from the gem and configure it likely for a flavour; that is not currently the case.
  • Any MathML in Presentation XML that consists only of a number (after localisation) is converted to the string of that number.
@opoudjis opoudjis added the enhancement New feature or request label Oct 22, 2020
@opoudjis opoudjis self-assigned this Oct 22, 2020
@ronaldtse
Copy link
Contributor

@opoudjis when you say stem:[] you mean number:[]? Are you saying all numbers have to be marked up as mathml?

@opoudjis
Copy link
Contributor Author

Yes I am. We agreed on that.

@opoudjis
Copy link
Contributor Author

twitter_cldr's localisations are configured in YAML, and we might need to inherit localisation from the gem and configure it likely for a flavour; that is not currently the case.

In fact, ITU wants apostrophe as the numeral group separator, so we will need to configure twitter_cldr after all to realise that.

@manuelfuenmayor
Copy link

manuelfuenmayor commented Oct 23, 2020

All numbers to be localised in rendering must be marked up as stem:[]. Sorry @manuel489. So a number like 1337 must be marked up as stem:[1337].

Does that include dates as well? What does "localisation of number rendering" mean? Is this for addressing the "group-by-three" issue in numbers with many digits (specified in #32)?

@ronaldtse
Copy link
Contributor

Does that include dates as well?

This is a good question that @opoudjis and I also discussed. In fact there should be a date:[] and/or year:[] command that allows specifying such data (and localizing them).

So stem:[] should not be used with dates.

What does "localisation of number rendering" mean?
Is this for addressing the "group-by-three" issue in numbers with many digits? (specified in #32)

Yes.

@opoudjis
Copy link
Contributor Author

@manuel489 if it's any consolation, I would not bother doing this for numbers that are less than four digits long. There is nothing to localise there.

opoudjis added a commit to metanorma/isodoc that referenced this issue Oct 26, 2020
opoudjis added a commit to metanorma/metanorma.org that referenced this issue Oct 26, 2020
opoudjis added a commit to metanorma/metanorma-itu that referenced this issue Oct 26, 2020
opoudjis added a commit that referenced this issue Oct 26, 2020
ronaldtse pushed a commit to metanorma/metanorma.org that referenced this issue Oct 27, 2020
opoudjis added a commit that referenced this issue Nov 9, 2020
opoudjis added a commit to metanorma/metanorma-itu that referenced this issue Nov 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants