Skip to content

Conversation

Rashmil-1999
Copy link
Contributor

No description provided.

@Rashmil-1999 Rashmil-1999 linked an issue May 21, 2025 that may be closed by this pull request
@Rashmil-1999 Rashmil-1999 marked this pull request as ready for review July 29, 2025 18:37
@Rashmil-1999 Rashmil-1999 changed the title [WIP] add ml enabled lumberton Add ml enabled lumberton Jul 30, 2025
Copy link
Member

@ywkim312 ywkim312 left a comment

Choose a reason for hiding this comment

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

Please update changlog

@@ -233,6 +233,11 @@ analyses/meandamage
.. autoclass:: meandamage.meandamage.MeanDamage
:members:

analyses/mlenabledcgelumberton
========================
Copy link
Member

Choose a reason for hiding this comment

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

Please fix this "====" to the same length as the line 236

Copy link
Member

@longshuicy longshuicy left a comment

Choose a reason for hiding this comment

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

Code runs well. Results look reasonable.

@longshuicy longshuicy merged commit feb41f4 into develop Jul 30, 2025
11 checks passed
@longshuicy longshuicy deleted the 656-implement-ml-enabled-cge---lumberton branch July 30, 2025 19:44
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.

Implement ML Enabled CGE - Lumberton
3 participants