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

updated glossary #47

Merged
merged 9 commits into from
Apr 25, 2024

Conversation

HassanAkbar
Copy link
Member

@HassanAkbar HassanAkbar commented Feb 26, 2024

Generated glossary from 20231214 Multi-Lingual Glossary – Published unlocked with Math.xlsx using tc211-termbase

@HassanAkbar
Copy link
Member Author

I've updated the glossary by generating it using the updated tc211-termbase gem which now uses glossarist to generate concepts.

@ReesePlews Can you help verify these changes?

Here are the other related merged PRs for the other gems that will use glossarist to read these concepts.

cc: @ronaldtse

@ReesePlews
Copy link
Contributor

@HassanAkbar thank you very much for your work on this! where can i review the changes, are they on a the public site or the staging site? please provide a URL so there is no confusion.

@ronaldtse
Copy link
Member

Maybe we should deploy a staging site at a subsite like how it’s done in fontist/formulas ?

@ReesePlews
Copy link
Contributor

@ronaldtse i agree about using the staging site. i think it would be best at this time, at least until i have the other documents ready for translation purposes. just send me the link when you are ready and i will check it. i think we have a few weeks for checking. thanks

@ReesePlews
Copy link
Contributor

hello @ronaldtse can you give an update on the issue about using stem:[...] notation to support the math scripts? if i read correctly you and @HassanAkbar are moving forward with geolexica using the stem:[...] notation. i just wanted to confirm that. thanks

@HassanAkbar
Copy link
Member Author

@ReesePlews @ronaldtse The staging site for updated glossary with stem:[...] support is deployed at https://www.geolexica.org/isotc211-staging/

@ReesePlews
Copy link
Contributor

@HassanAkbar thank you very much for the update. i confirm i can see the staging site. the math on a couple of items i checked looks really nice. please give me some time for checking and i will submit any issues i find to you @ronaldtse . appreciate your diligent work on this project. thank you!

@ronaldtse
Copy link
Member

Term 2455 (https://www.geolexica.org/isotc211-staging/concepts/2455/) contains the "latexmath:..." syntax which needs to be parsed as math in LaTeX math format by Plurimath. @HassanAkbar

@HassanAkbar
Copy link
Member Author

@ronaldtse I've updated the staging site and the latexmath:[...] syntax is now working.

@ReesePlews
Copy link
Contributor

hello @HassanAkbar i see that now in the barycentric coordinates entry. please ask @ronaldtse as he helped was working with another member on the math processing. was that the only latex instance? that should probably be changed to stem:[ ... ] as the others. i dont think there is a reason we would need the latex code, but please ask Ron. thank you

@ronaldtse
Copy link
Member

@HassanAkbar @ReesePlews so the reason why we had latexmath:... was because Asciimath did not have the backepsilon symbol. Do we now support it in Plurimath? @HassanAkbar can you please help check? Thanks!

@HassanAkbar
Copy link
Member Author

@ronaldtse currently we do not support backepsilon in Plurimath.

@HassanAkbar
Copy link
Member Author

@ronaldtse we can add support for latex symbols in Plurimath for asciimath. It shouldn't take that long for that.
Should we add that?

@ronaldtse
Copy link
Member

ronaldtse commented Apr 1, 2024

@ronaldtse we can add support for latex symbols in Plurimath for asciimath. It shouldn't take that long for that. Should we add that?

I agree. Let's do that.

Actually, it already exists here:

@ronaldtse
Copy link
Member

@HassanAkbar please heal emerge this when ready, thanks.

@HassanAkbar HassanAkbar force-pushed the generated_concepts_from_20231214_glossary_with_math branch from d19df13 to 449dca1 Compare April 24, 2024 10:31
@HassanAkbar
Copy link
Member Author

@ronaldtse This branch contains the concepts generated from the 20231214_Multi-Lingual_Glossary–Published__unlocked__withMath.xlsx file, and we are updating this branch whenever @ReesePlews creates a new issue related to the isotc211 staging site.

Can I merge this branch or wait until all the concepts are finalized?

@ronaldtse
Copy link
Member

Let's merge this. I don't want to wait anymore.

@ronaldtse ronaldtse merged commit df49fd5 into main Apr 25, 2024
3 checks passed
@ronaldtse ronaldtse deleted the generated_concepts_from_20231214_glossary_with_math branch April 25, 2024 04:42
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.

3 participants