Skip to content

Add ML enabled CGE for Mobile#455

Merged
longshuicy merged 11 commits into
developfrom
454-add-mlcge-enabled-documentation-for-mobile
Jul 29, 2025
Merged

Add ML enabled CGE for Mobile#455
longshuicy merged 11 commits into
developfrom
454-add-mlcge-enabled-documentation-for-mobile

Conversation

@ywkim312

@ywkim312 ywkim312 commented Jun 4, 2025

Copy link
Copy Markdown
Member

No description provided.

@ywkim312 ywkim312 linked an issue Jun 4, 2025 that may be closed by this pull request
@ywkim312
ywkim312 marked this pull request as draft June 4, 2025 18:22
@ywkim312 ywkim312 self-assigned this Jun 10, 2025
@ywkim312
ywkim312 marked this pull request as ready for review June 10, 2025 19:58
@Rashmil-1999

Copy link
Copy Markdown
Contributor

Can we remove the unused imports from the notebook?

@Rashmil-1999

Copy link
Copy Markdown
Contributor

everything else looks good :D

@navarroc

Copy link
Copy Markdown
Member

Be careful merging in develop. What you will likely want to do is keep my list of analyses.md and add your one line to mine. I fixed case issues and removed deprecated analyses in my PR. I am happy to do this for you if you'd prefer I handle the merge conflict.

@ywkim312

Copy link
Copy Markdown
Member Author

Be careful merging in develop. What you will likely want to do is keep my list of analyses.md and add your one line to mine. I fixed case issues and removed deprecated analyses in my PR. I am happy to do this for you if you'd prefer I handle the merge conflict.

Yes could you please do it? I am not sure what to do.

@navarroc

Copy link
Copy Markdown
Member

This should be good to merge once we get the green light from the CGE team.

@longshuicy longshuicy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

@longshuicy
longshuicy merged commit 112ad4c into develop Jul 29, 2025
1 check 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.

Add MLCGE enabled documentation for Mobile

4 participants