Skip to content

Task298361 20260422 adding economic analysis documentation#4

Merged
dgorrichoT merged 8 commits intomainfrom
Task298361_20260422_adding_economic_analysis_documentation
May 8, 2026
Merged

Task298361 20260422 adding economic analysis documentation#4
dgorrichoT merged 8 commits intomainfrom
Task298361_20260422_adding_economic_analysis_documentation

Conversation

@Ramya-Bandi
Copy link
Copy Markdown
Collaborator

Hi David,

The final versions of the Measures and Economic Analysis files have been
committed and are ready to push, along with the recent change to separate
the Mermaid diagrams into a dedicated mmd folder.

Best regards,
Ramya

@Ramya-Bandi Ramya-Bandi requested a review from dgorrichoT April 23, 2026 19:46
@dgorrichoT
Copy link
Copy Markdown
Collaborator

Hi @Ramya-Bandi, in general, can you please use numref instead of fix numbers in the figures? Thanks! https://github.com/eeadata/WISE.WFD.Documentation/blob/main/docs/DataModelReview/WFDMeasures.md?plain=1#L21

Copy link
Copy Markdown
Collaborator

@dgorrichoT dgorrichoT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @Ramya-Bandi
Please, review those changes and ping me if needed for any clarifications 🙂

Comment thread docs/DataModelReview/img/Thumbs.db
Comment thread docs/conf.py
Comment thread docs/_static/customTable.css
Comment thread docs/DataModelReview/tables/Measures_MeasureType_ListTable
Comment thread docs/DataModelReview/tables/Measures_SubsetOfCEPAClasses_ListTable
Comment thread docs/DataModelReview/tables/Measures_MainKeyTypeOfMeasure_ListTable
Comment thread docs/DataModelReview/mmd/ProtectedAreasClassdiagram.mmd
Copy link
Copy Markdown
Collaborator

@dgorrichoT dgorrichoT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @Ramya-Bandi, I'm merging these changes

@dgorrichoT dgorrichoT merged commit ea95a22 into main May 8, 2026
2 checks passed
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.

2 participants