wmo-im / tt-nwpmd

2 stars 2 forks source link

Browsable topic hierarchy in the repo #14

Closed yhe-wmo closed 4 months ago

yhe-wmo commented 1 year ago

TT-NWPMD meeting on 2023.06.13:

@alexandreleroux demonstrated browsable topic hierarchy, which has been generated automatically by a short script. Domonstration

TT meeting considers the browsable topic hierarchy more user friendly than raw CSVs; and appreciates Alex and his colleagues on developing the concept and the script. TT meeting decides to adopt the browsable topic hierarchy.

alexandreleroux commented 1 year ago

Thanks @yhe-wmo ,

Next steps ;

Does that make sense to you? Thanks!

amilan17 commented 1 year ago

@tomkralidis I added this to the TT-WISMD project for discussion at our next meeting

amilan17 commented 1 year ago

@alexandreleroux I can test this in my forked repo, but I'll need some guidance on how to implement.

tomkralidis commented 1 year ago

TT-WISMD 2023-06-22:

PhilippeTh commented 1 year ago

I have completed the code and the workflow on my branch: https://github.com/PhilippeTh/tt-nwpmd/tree/PT_Markdown_Restructure_Enhancement

The github workflow is in .github/workflows/main.yml, the script itself along with the README to explain the workflow and how to run the script can be found in https://github.com/PhilippeTh/tt-nwpmd/tree/PT_Markdown_Restructure_Enhancement/tree/script

If you have any comments, feedback or questions feel free to reply and if we see no problem with the way I've done it I will then make a merge request.

Thanks!

tomkralidis commented 1 year ago

TT-WISMD 2023-07-07:

yhonda21 commented 1 year ago

TT-NWPMD 2023-07-05: The new tool was tested and implemented by an ECCC colleague, Phillippe. He also created a document on how to use it. Currently, our proposed TH is only available. Once other TH is developed, it will be automatically integrated by the tool. Who will take care of this tool will be discussed at the next TT-WISMD meeting.

yhonda21 commented 10 months ago

TT-NWPMD meeting on 2023.09.14

alexandreleroux commented 10 months ago

Thanks @yhonda21 for the followup.

Can you ask clarifications to TT-WISMD how this potential improvement would be a heavy process for the complicated TH?

Of course if it's not this simple script, the need for visualizing TH beyond the raw CSV files remain.

Thanks for the additional feedback Yuki and anyone else!

josusky commented 10 months ago

Hi @alexandreleroux, when we tried to use it in the 8 level TH where some levels had hundreds of possible values (especially center ID and country), and the script generated hundreds of thousands of files (thus hundreds of MBs). That is what Tom calls "heavy". It will be a bit less heavy if the center ID and country are joined (https://github.com/wmo-im/wis2-topic-hierarchy/issues/58). So in TT-WISMD, we are not against visual aids, we were just afraid of flooding the git repository with tons of almost identical files. Maybe it helps if you look at the TH (or at least its first 8-ish levels) as as set of key-value pairs that are put together, in a defined order. Each CSV file defines values for one key. Below that it starts to be a hierarchy, a tree, but the beginning is a multidimensional matrix.

alexandreleroux commented 4 months ago

Thanks @josusky for your feedback.

Now that https://codes.wmo.int/wis/topic-hierarchy is alive and is browsable in a similar way that what this ticket proposed last year, I'm closing this ticket.

Thanks and regards!