Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
Closed #343 via #344. -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*event-6117012815__;Iw!!G2kpM7uM-TzIFchu!ip9TJLLpV_n9H6tOPvhntEGLBVsdsVivq4BIsWGGEKHJOX5HB8Os0ha5aKLlpiIUFqezrmtcqzg$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
Reminder set. If you have a problem, speak in the next 3w or forever hold your peace! -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1026967334__;Iw!!G2kpM7uM-TzIFchu!ipfYRUUTH1fdoKcSf9JhD5cj-6MogZeH0vSxAAGOprWSSZMSxE5z6tzL1wVB4JWjXon_zzne3U8$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
Thanks for the heads-up, @JonathanGregory. I have updated #344 accordingly. I have also addressed @ethanrd's suggestion of making the attribute name more explicit. Given the general support and the fact that no more comments seem to be outstanding, I think we can start the clock and merge the PR in three weeks, on 2022-02-22. -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1026914782__;Iw!!G2kpM7uM-TzIFchu!ndJZqwoCQXSF1JSMjRtBzb7e_SjQw3HeKG28kFIbJea8lvEXQWQfUbnFMLdyG0hmFmJXJVQ1Llo$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
Dear @JonathanGregory, such a clash is called a `conflict` in git lingo and indeed does occur. But it is also one of the most common problems in versioning and as such one of the core abilities and raison d'etre of git is to help with their resolution. So don't worry, this will be easy to address. Cheers Klaus -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1010099749__;Iw!!G2kpM7uM-TzIFchu!hA5hDuCghUqHKq87ObIWy4siQqjYnDB2KJ_GhR1BTT2DxoZtJNlmp7tVQdXSLKoAg2Fvz7Gw6Oc$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
Dear @zklaus I've done a [pull request](https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/pull/350__;!!G2kpM7uM-TzIFchu!h_4-L8njdnQuYrvIVsBWFfZZyiaEw7GB2tpt1NklqxCHzaGqwx_s6szUJC6W_hRzXiEzSDkGdyU$ ) to delete the `Conventions` attribute in the two examples, [as proposed](https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/349__;!!G2kpM7uM-TzIFchu!h_4-L8njdnQuYrvIVsBWFfZZyiaEw7GB2tpt1NklqxCHzaGqwx_s6szUJC6W_hRzXiEzc5yb6to$ ). I suppose that might lead to a clash when merging, if your change updates the attributes, would it? I am not a git expert (as is well-known). Best wishes Jonathan -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1009765855__;Iw!!G2kpM7uM-TzIFchu!h_4-L8njdnQuYrvIVsBWFfZZyiaEw7GB2tpt1NklqxCHzaGqwx_s6szUJC6W_hRzXiEzgJ2INvM$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
Thanks, @JonathanGregory, your points make sense to me. Let's take that discussion in the other issue and move forward here with the corrected attributes in place. -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1009038709__;Iw!!G2kpM7uM-TzIFchu!lNaPDF6X_YGUmSmazVRs6Qsb7PGAq_EDkFAlGYBmIWWVogVqeEzRZxonGj5yhXMdn5pNyO5WkCs$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
Dear @zklaus I don't think there are any "full examples" in the document. (This [new issue](https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/348__;!!G2kpM7uM-TzIFchu!lZI1e-2iPbokQ-LIbAFuEADT9eEzUL6k3p_RYXQdo1Ee5Nx1UkSEuM8EXIqqHr3JsxfhxSZhQl8$ ) is relevant to this.) That is probably because full examples are not what we need to illustrate particular points. I will propose the deletion of the `Conventions` attribute in this sole example as a separate issue. Best wishes Jonathan -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1008921207__;Iw!!G2kpM7uM-TzIFchu!lZI1e-2iPbokQ-LIbAFuEADT9eEzUL6k3p_RYXQdo1Ee5Nx1UkSEuM8EXIqqHr3JsxfhIjKOz9w$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
Re workflow, that was exactly my thinking. I have added this now to #344. Re removing the attribute from the examples, I am not so sure. I think we should probably consider categorizing examples in the conventions as either "full examples" or "simple examples"/"excerpts" and then rather add the attribute to all full examples and possibly some excerpts as appropriate. My reasoning is that I think many first time producers of data files will follow closely some examples and if we leave out this attribute, they might too. But perhaps this deserves a separate discussion. Re @ethanrd's comment on the naming of the attribute, I completely agree, `attribute-version` is not very clear. How about `current-version-in-attribute` or `current-version-for-attribute`? -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1007607710__;Iw!!G2kpM7uM-TzIFchu!mk8WbnPulon2XXFw6vehZDdcM_QxE8pWmekxh3EM_MPW-_B4pA9_jibp9QVkA2nzA29MeZ_adao$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
It took me awhile to understand the meaning of the term `attribute-version`. Perhaps `current-version-as-attribute`? I like the definitive nature of updating the version from `1.10-draft` to `1.10` for release and then immediately updating to `1.11-draft`. It leaves an artifact (source zip file) that is clearly marked as a full release. On the other hand, this is a documentation project not a software project and we distribute document artifacts not source artifacts. However, it might be more of an issue depending on where the CF DOI conversation about version specific DOIs goes. -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1007587182__;Iw!!G2kpM7uM-TzIFchu!itXafRG3RTx6TTruC04c6mYK8PUHq8wrIPEwQR7qCkUMJZS8G8Txe825ye7H3aq9q7X2kyP_juE$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
Dear @zklaus As @adigitoleo says, there are no examples in the document which contain the `Conventions` attribute except in sect 7.5 (examples 7.15 and 7.16). Therefore I'd suggest you _delete_ the `Conventions` attribute in those two examples, instead of correcting it. It's not necessary (now that 7.5 is an established part of the conventions). Best wishes Jonathan -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1007411080__;Iw!!G2kpM7uM-TzIFchu!icrPTrMgqI711cha1xI-Zs6DJ3I9IUM5dBVFP1WiraBinvCqQhcKQ57kPmbw5jw6ICgVwhdCXhk$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
That's clever, @zklaus. Thanks. I agree with @erget, and I am not an expert on the build workflow. -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1007399109__;Iw!!G2kpM7uM-TzIFchu!kCPBGs5G8fFI1lAeXRdUl839RH7nl6w8km2hYL9TQwP02-ABIkn5JWvsLq67IE1EKFScYy98k3Y$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
>From @zklaus in >https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/pull/344*issuecomment-1007108115__;Iw!!G2kpM7uM-TzIFchu!nzfFRuwavWqsiYAnVaPztGBBgkQDo_YadSDdBXX2OYtbfvHDKgQmoST4eJMEM8vFe2_WhuQ-6Z4$ > : > I have added a fancified version of the version handling. Let me know what > you think or if you want me to explain a bit more. > > PS: I am totally happy to roll back the last commit and to just add ` draft` > or `-draft` to the version. Just thought I'd show one other way that we could > go about this to improve automation. I think this is pretty elegant - IMO in order to make this perfect the way to go would be to change [the build workflow](https://urldefense.us/v3/__https://github.com/zklaus/cf-conventions/blob/single-source-version/.github/workflows/adoc_build.yml__;!!G2kpM7uM-TzIFchu!nzfFRuwavWqsiYAnVaPztGBBgkQDo_YadSDdBXX2OYtbfvHDKgQmoST4eJMEM8vFe2_WU-Yetuk$ ) to run with `-a final` as described [here](https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/pull/344/files*diff-5ff41050491a2b74889153ce01bd77a1b09e5acbd156063709f330a2ef0bcc8eR7__;Iw!!G2kpM7uM-TzIFchu!nzfFRuwavWqsiYAnVaPztGBBgkQDo_YadSDdBXX2OYtbfvHDKgQmoST4eJMEM8vFe2_WeJFdl0U$ ) on full tags - that sounds straightforward but I'm not sure if that's easy to do though, as I don't have a clear pathway to do that in mind. Any thoughts on that, and would anybody be interested in some extra credit for improving the build process? ;) -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1007265396__;Iw!!G2kpM7uM-TzIFchu!nzfFRuwavWqsiYAnVaPztGBBgkQDo_YadSDdBXX2OYtbfvHDKgQmoST4eJMEM8vFe2_WO91k0_s$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
I prefer the more terse formulation. Replacing the space with a dash, so `1.10-draft`, would be similar to a well known pattern for software versioning of appending `-SNAPSHOT` or `-mmddhhss` to a version number. Perhaps a bit more obvious than `1.10 draft` but still terse. -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1006698450__;Iw!!G2kpM7uM-TzIFchu!godn4COKgWP6Uv5u0oy0fJnLstAZU8MVnx0XCA2sh6VX-5VLiwvzSLH5NvE9zgc8JE6W3ZnuhB8$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
Personally, I'm more for the terser formulation of `draft` - IMO that makes it obvious enough. -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1006643482__;Iw!!G2kpM7uM-TzIFchu!jCiP3-VMGr2lbbobclyJ0fNi0G71NYMNysLZr1jZuNlc_B2U21ruK6EUOOqEeh6EpRGqEFQ-7aM$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
I agree with putting `draft` in the current-version. I think it would be a positive advantage if it turns up e.g. in `Conventions` (in the text of the draft document), because that makes the status obvious. In fact maybe we could be more explicit and say e.g. `:current-version: 1.10 draft (not yet released, not to be used)` or is that going too far? -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1006601253__;Iw!!G2kpM7uM-TzIFchu!gmDvuWnAHhcNEbiLrMd_YpBiOLpsaphXSMZLtQvPAfGl-qgZxQwkB48attuVS-y0Tyc0w65NRE0$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
@JonathanGregory thanks for bringing us back on track! > Might it make sense to do this?: > > :current-version: 1.10 draft > > And then remove draft pre-release? I think that would make sense. -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1006599840__;Iw!!G2kpM7uM-TzIFchu!jttTw2yUABTJgBdvfkzErMN9qDha-JO5EuRaIAReK6afQ4WKG02j12gvIEfQtYqQ9Y0A9UalC7s$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
Discussion in review comments of https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/pull/344__;!!G2kpM7uM-TzIFchu!lLnLFd19gThcoMUaRLU_pBH4VxrLNZCKKoySzJQnQfoNQoRM5jsViIxya-aghDFqq-QhUmVjGFI$ reproduced here for the record. (May I politely and respectfully remind everyone that the CF practice is to keep substantive discussions about text in the issue, not the PR, so we have a single place to look for the history. Thanks. :smile: ) @erget: Might it make sense to do this?: `:current-version: 1.10 draft` And then remove ` draft` pre-release? @zklaus: I had this locally at some point, but I removed it again because I felt that it looked odd in the `:Conventions` attribute both in the examples, which would read ``` // global attributes: :Conventions = "CF-1.10 draft" ; :featureType = "timeSeries" ; ``` and in the text where this attribute is described, for example ``` [...] attribute **`Conventions`** to a string value that contains "**`CF-1.10 draft`**". ``` Hence my suggestion to add a second `:status-indicator:` or similar that can be combined with the `:current-version:` where appropriate. But I am happy to use the simpler solution :) @erget: I don't have overly strong feelings here, but in the absence of Sturm und Drang from other corners my preference would be to go with using `current-version` for the whole shebang. Reasoning: - It's simpler than having 2 attributes - It doesn't result in invalid CDL, since space characters are allowed and thus it would be easier to maintain, and also more obvious that one is working with a draft. -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1006598333__;Iw!!G2kpM7uM-TzIFchu!lLnLFd19gThcoMUaRLU_pBH4VxrLNZCKKoySzJQnQfoNQoRM5jsViIxya-aghDFqq-QhhTqL448$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
@JonathanGregory I see the typo in the following places: - Section 1.4, par 2 - Section 2.6.1 par 1 - Section 7.5 in the example There are no other examples with the Convention attribute. Looking at the built artifacts from that PR, there are no moree version number typos that I can see. -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1005765742__;Iw!!G2kpM7uM-TzIFchu!hktTCoUa_9wrD8pPoBeTlRmeviEljaB7HV0oJ_aJz_8d-8XZ7td0gKL6cLsPHnc5-vt4QXIGQaw$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
Dear @zlaus, @erget, @davidhassell et al. Thanks for the [PR](https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/pull/344__;!!G2kpM7uM-TzIFchu!nCorpr1w3xsiQ29-ZyQWgmXIFudxCW5wd2GqSaOFhrXPhujPZREoH3m0_nr7RyAKYpj6ZWWJJgg$ ), Klaus. I agree that you've provided a neat solution, as Daniel says in other words. I agree with renaming the top-level anchor of the conformance document. It's probably more useful if generic anyway. I note that you have helpfully made this change in an example in ch7. Is this the _only_ example in the whole document which includes the `Conventions` attribute? If so, it would be easier to delete it in that example, and that would also be better for consistency. Daniel suggests defining `current-version` to `1.10 draft`, not just `1.10`, until it's released. I think that's a good idea. It means that `draft` would automatically appear in the titles of both conventions and conformance documents. This procedure should be added to the release checklist, if there is one. Is there a reason why we should not release 1.10 with this change as soon as we've agreed it, and any others agreed since 1.9, and then start preparing 1.11 for release in the summer? Cheers Jonathan -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1005741937__;Iw!!G2kpM7uM-TzIFchu!nCorpr1w3xsiQ29-ZyQWgmXIFudxCW5wd2GqSaOFhrXPhujPZREoH3m0_nr7RyAKYpj6Ta9f8_0$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
Thanks @zklaus, the PR looks pretty cool. Also yes, happy new year everybody! :) I've requested also the input of @davidhassell since he's more heavily involved in the release process. @JonathanGregory, I propose that we proceed as follows: - If we can agree on an approach for a longer-term fix, e.g. by accepting the changes proposed in #344, then we cherry-pick those on top of 1.9 and make a patch release to correct the defect. - If the long-term approach takes longer to clarify, i.e. more than a week or so, we manually correct the defect in 1.9. What do you think? Cheers, Daniel -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1005688209__;Iw!!G2kpM7uM-TzIFchu!gAL93PODqwDjFufg1W1CiFpJ10SK0I9_idYJFg-4LheaCKoGQtk4NwsRqBF8kKH60a_Yth8uzn8$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
I added a draft PR in #344 that addresses (2). If we decide to tackle (1) separately, the list of changes in that PR should give a good overview of the places that need changing. -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1005048728__;Iw!!G2kpM7uM-TzIFchu!m6nNeQYM3PAMCPSKO2OwquYU3KgUmQytZa_s3bpPq63GGGTU8wKmzpKLw-XWT3kOHho9XCe1l0A$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
Dear all Thanks for pointing it out, Leon @adigitoleo. I think @zklaus is right that we could add a "replacement" i.e. a sort of macro in AsciiDoc for the current version number, so that only one occurrence needed to be changed. I am not familiar with the build process so I won't volunteer to do that. Is there a checklist for making a new version, to which this could be added? I think @davidhassell must know. As Daniel @erget says, it would be easy to fix the typo, and I'm sure we would approve the change as a defect issue for it . But could we agree to modify the live 1.9 document _immediately_? There would be no point in agreeing a defect correction to 1.9 which wasn't going live until 1.10 in this case! Best wishes and happy 2022 to the CF community Jonathan -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1004637875__;Iw!!G2kpM7uM-TzIFchu!lcGrz1UD4c19TlIyQJvGIhD4Ap6e2doFs2H-l36Yi7Q90AaOtFLhEqnMyqW-ecl42OedBQ6f-QU$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
Hi all, so it looks like we actually have 2 things in this issue: 1. Fix typo (easy) 2. Potentially improve the document so it's more future-proof (1) sounds necessary to me; (2) sounds like a good idea. Is anybody willing to champion 1 or both of these? I would support it! -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1003915544__;Iw!!G2kpM7uM-TzIFchu!ksmqVVAYAC6__KIzSamNPIVTLQNJZ1EU7econf18MA7-_J4T-CZlQN3K7nleQ7DmXDqunx083pQ$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
Re: [CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
This should be possible using a "replacement" as described in [Section 26.9 of the asciidoc userguide](https://urldefense.us/v3/__https://asciidoc-py.github.io/userguide.html*X7__;Iw!!G2kpM7uM-TzIFchu!g9-eXsCt31CXN0toVTDJdMsAixwlWH5Sq5yhyLtaW7Ch9hVJLjPfXvzXF_Vg69HBpUAEbFj5dyY$ ). -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343*issuecomment-1003741003__;Iw!!G2kpM7uM-TzIFchu!g9-eXsCt31CXN0toVTDJdMsAixwlWH5Sq5yhyLtaW7Ch9hVJLjPfXvzXF_Vg69HBpUAESzn3pAk$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
[CF-metadata] [cf-convention/cf-conventions] Single-source the version number (Issue #343)
The current published standard (v1.9) contains a typo in section 1.4 Overview: > Files using this version of the CF Conventions must set the NUG defined > attribute Conventions to contain the string value "CF-1.8" to identify > datasets that conform to these conventions. So, the version number was not incremented in that paragraph. This could be avoided by somehow reading the version number from a central location when building the document, if that is possible. Just something I noticed when reading through again today. -- Reply to this email directly or view it on GitHub: https://urldefense.us/v3/__https://github.com/cf-convention/cf-conventions/issues/343__;!!G2kpM7uM-TzIFchu!nR8GTOvbXjW4DIkg9_k1Z_7A1xycOAUF-q8Kxiw_YDzDqxO6uNR9Sa95sH6JnikP_EbS7wtNiQA$ You are receiving this because you are subscribed to this thread. Message ID: This list forwards relevant notifications from Github. It is distinct from cf-metad...@cgd.ucar.edu, although if you do nothing, a subscription to the UCAR list will result in a subscription to this list. To unsubscribe from this list only, send a message to cf-metadata-unsubscribe-requ...@listserv.llnl.gov.
