Repository navigation
docs: use NET-LINE for EU Hydro examples and drop it from the tree - #78
Merged
Merged
Conversation
- Rename the EU Hydro example filenames to the NET-LINE form (dropping the descriptive _S2026_R10m_ tokens) in the design-principles doc - Remove the EU Hydro (EUHYDRO) branch from the filename tree so EGMS L2 is the terminal node - Fix alignment spacing on the SWF (HRL - 5m) line
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 411950dcb9
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ├── L2a-A, L2a-D, L2b-A, L2b-D EGMS calibrated (EGMS L2) | ||
| │ | ||
| └── NET, DIR, SUB EU Hydro (EUHYDRO) | ||
| └── L2a-A, L2a-D, L2b-A, L2b-D EGMS calibrated (EGMS L2) |
There was a problem hiding this comment.
Removing this variable branch does not fully drop EU Hydro from the filename tree: the By Product Code section at lines 49–50 still advertises EUHYDRO-NET_S{YYYY}_R10m_EUROPE_3035_V02-R00.gpkg. Readers therefore still see the obsolete pattern that this commit replaces with NET-LINE; remove or update that earlier branch as well.
Useful? React with 👍 / 👎.
| **Rule:** Omit production date from filenames. Include it only when multiple production runs for the same temporal period coexist in the same distribution channel. When present: `{YYYYMMDD}` as the last token before the extension. | ||
|
|
||
| **Compliant:** `CLMS_EUHYDRO-NET_S2026_R10m_EUROPE_3035_V02-R00_20261015.gpkg` | ||
| **Compliant:** `CLMS_EUHYDRO-NET-LINE_EUROPE_3035_V02-R00_20261015.gpkg` |
There was a problem hiding this comment.
For new EU Hydro deliveries, this is now labeled a compliant full filename even though it omits the temporal field required by the canonical template and token-count invariant in Principle 5, while Principle 8 defines no timeless/optional temporal case. Because dropping _S2026_ is intentional, add an explicit atemporal exception and reflect it in the filename tree/schema; otherwise implementers following this document cannot determine whether EUROPE is an extent or a shifted temporal token.
Useful? React with 👍 / 👎.
mckeea
added a commit
that referenced
this pull request
Oct 2, 2026
* Update LLM cache and version history [skip ci] * ci: prepare workflows for branch protection rulesets * ci: pin ruff rule set so upstream default changes don't break CI * docs: correct required YAML fields and document type in editor guide * chore: drop editor guide cache entry to force regeneration * chore: drop one image cache entry to force a cache write * Update LLM cache and version history [skip ci] * docs: unify CLMS filenaming doc names and title pattern Rename both docs to the shared clms-filenaming_<topic>_v1.qmd pattern, align both titles to 'CLMS Product Filenaming - <Topic>', and replace em dashes with plain hyphens throughout. * Update LLM cache and version history [skip ci] * feat: add parsEO dashboard (filename builder + schema tools) * fix: use relative API URL, add fetch error handling, debounce, XSS escaping * fix: empty API base for same-origin (SWAG proxy) * fix: remove API URL field, auto-connect on same-origin * feat: clickable chip-tree for filename assembly * feat: add parsEO dashboard to DOCS/tools (Quarto site) * feat: add parsEO dashboard to DOCS/tools with index.qmd landing page * feat: parsEO dashboard as OJS Quarto dashboard (replaces standalone HTML) * corrected type SFW * fix: remove bar labels and default to weekly view in CDSE migration dashboard Remove the +N number labels from the onboarding history bar chart (values are still available via tooltip on hover). Switch the default aggregation mode from daily to weekly for a cleaner overview. * fix: plot negative onboarding deltas below zero and key weeks by ISO week-year * fix: make the parsEO dashboard actually run * Update LLM cache and version history [skip ci] * Update non-browsable doc map [skip ci] * docs: use NET-LINE for EU Hydro examples and drop it from the tree (#78) * docs: correct product descriptions in CLMS filenaming guidelines (#77) Apply Lorenzo's feedback: - Attribute IW burst to Sentinel-1 swath, not EGMS InSAR - Rephrase EGMS L3 grid and L2 orbit descriptions with their official labels * chore(ci): merge DOCS/clcplus-core from publish-clcplus-core-20260910-094246 Triggered by: CI Workflow Branch: publish-clcplus-core-20260910-094246 Commit: f658ece Repo: f658ece * Update LLM cache and version history [skip ci] * chore(ci): merge DOCS/clcplus-core from publish-clcplus-core-20260910-095249 Triggered by: CI Workflow Branch: publish-clcplus-core-20260910-095249 Commit: 382cc06 Repo: 382cc06 * chore(ci): merge DOCS/clcplus-core from publish-clcplus-core-20260910-102206 Triggered by: CI Workflow Branch: publish-clcplus-core-20260910-102206 Commit: cd1b8ca Repo: cd1b8ca * Update LLM cache and version history [skip ci] * fix(build): realign drifted grid tables and guard dangling links in Typst PDFs * chore(ci): merge DOCS/clcplus-core from publish-clcplus-core-20260914-100407 Triggered by: CI Workflow Branch: publish-clcplus-core-20260914-100407 Commit: 1215deb Repo: 1215deb * fix(docs): unblock Typst build for CLC+ Core User Manual [skip ci] * fix(docs): realign drifted grid tables in CLC+ Core User Manual [skip ci] * fix(preview): apply dangling-link guard to local Typst renders [skip ci] * feat(build): add BUILD_ONLY filter to build-docs.sh [skip ci] * docs(agents): require suggestions before implementation [skip ci] * docs: add Protected Areas 2021/2024 nomenclature and mapping guideline (#79) * docs: add Protected Areas 2021/2024 nomenclature and mapping guideline Add the Priority Area Monitoring - Protected Areas 2021 and 2024 Nomenclature and Mapping Guideline, converted from the source PDF. Includes the full LC/LU class nomenclature and its 377 figures. `validate_qmd_files.py`: 38/38 valid; all media references resolve. * docs: restore Table 2 colors and fix fenced-div caption closures - Restore the LC/LU nomenclature legend colours in Table 2 by sampling each code cell's fill from the source PDF (pages 10-11) and injecting it as an inline background-color across the three code-level columns (88 cells). - Close the tbl-caption divs properly (blank line before the closing fence) so the caption markers no longer leak into the rendered table text. * Update LLM cache and version history [skip ci] * feat(build): compress media at build time, keeping originals in git * fix(build): compress only document media and repoint references safely * fix(build): detect transparency portably and flatten onto white * fix(build): use %[opaque] for transparency, portable across ImageMagick 6 and 7 * chore: sync published version state from main * chore: hold back unpublished docs (products batch, CDSE and parsEO dashboards) * chore: remove orphan media and cache of withdrawn HRL SLF PUM draft * chore: take MRVPP cache entries from main --------- Co-authored-by: GitHub Actions <actions@github.com> Co-authored-by: Matteo Mattiuzzi <matteo@mattiuzzi.com> Co-authored-by: ci_docker_builder <ci_docker_builder@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates to the CLMS filenaming guidelines:
_S2026_R10m_tokens) in the design-principles docvalidate_qmd_files.py: 36/36 valid.