S-101-Portrayal-subWG / Working-Documents

16 stars 5 forks source link

PC rules - References to 'Converter' and 'FC' versions #143

Closed alvarosanuy closed 2 months ago

alvarosanuy commented 1 year ago

It has been noted that comments in the header of the rules' files include the following text that could be misleading to 'new users': image Some may read the Feature Catalogue version as the version the content of the rule is compliant with.

@mikan66 provided the following comment to this issue: the rule file header comment was from the initial automation conversion from S-52 DAI to S-101 Lua. The conversion tool inserted the catalogue used for the conversion process. Most rules will have this same comment if they were converted over. Users can disregard this header comment on each converted rule. There is some usefulness for us to know what edition was used for the conversion during development. I suppose the WG will need to decide if we need to clean up all rules for the production release and standardize the header comment across all files. We should keep comments for specific issues resolved in each rule file like: "-- PSWG #109, PC #131".

Should we look at removing these entries from PC 2.0.0 onwards or maybe replace them with something else?

alvarosanuy commented 9 months ago

Decisions made at Portrayal subWG meeting on 19/10/23

alvarosanuy commented 3 months ago

Decisions made at Portrayal subWG meeting on 09/04/24

alvarosanuy commented 2 months ago

I noticed you closed your Issue # 340. I'm closing this issue as well

alvarosanuy commented 2 months ago

Implemented in PC 1.3.0

DavidGrant-NIWC commented 2 months ago

Implemented in PC 1.3 by https://github.com/iho-ohi/S-101_Portrayal-Catalogue/issues/340