Skip to content

Add autogen for AMDC article in control with AMDC folder#151

Open
Daehoon-Sung wants to merge 115 commits into
mainfrom
feature/add-autogen-control-with-AMDC-article
Open

Add autogen for AMDC article in control with AMDC folder#151
Daehoon-Sung wants to merge 115 commits into
mainfrom
feature/add-autogen-control-with-AMDC-article

Conversation

@Daehoon-Sung
Copy link
Copy Markdown
Contributor

@Daehoon-Sung Daehoon-Sung commented Mar 19, 2026

This is to close #143 to close #149 to close https://github.com/Severson-Group/AMDC-Examples/issues/6.

Notes

This comment might be useful to refer to how we came up with the plan for those two ariticles.

Self-Review

In this section, please self-review (answer all questions) on a suitable review checklist prior to requesting review from others. Select a review checklist based on what content is being merged in; see the Review Checklists section.

Reviewer Instructions

Reviewers, please copy and paste a suitable review checklist into your review and answer all questions.

Appendix

This section should be the same for all PRs. Do not edit this section when creating a PR.

Review Checklists

Checklists maintained by the eLev lab for research repositories include:

Standard checklist

1. Are all files under 300 kB (if not, please carefully assess whether it is worth committing them)? **Yes or No**
2. Are all files named according to the appropriate [naming convention](https://github.com/Severson-Group/research-repo-template?tab=readme-ov-file#file-naming), i.e., dash-case, camelCase, snake case? **Yes or No**
3. Do all Markdown files follow the [CONTRIBUTING article template](https://github.com/Severson-Group/.github/blob/main/CONTRIBUTING.md#markdown-documentation-template)? **Yes or No**
4. Do all links work in the material that the PR is adding? **Yes or No**
5. Is the PR configured to close the correct issue(s)? **Yes or No**
6. Did the PR fully address the `Approach` section of the issue(s) it is closing? **Yes or No**

Please work on addressing any **No** items.

Daehoon-Sung and others added 4 commits April 17, 2026 16:20
Clarified the Simulink and AMDC workflow, emphasizing the separation of control development into design and execution domains. Added details about the subsystems in the Simulink model.
@noguchi-takahiro
Copy link
Copy Markdown
Contributor

Hello @elsevers, the AMDC article of Autogen is ready. See the following rendered articles:

I think we can have better block diagram for the article, but we can discuss.

@noguchi-takahiro
Copy link
Copy Markdown
Contributor

noguchi-takahiro commented May 2, 2026

Hello @elsevers, I updated the file organization for autogen folder. See the following rendered articles:

@elsevers
Copy link
Copy Markdown
Contributor

elsevers commented May 4, 2026

Thanks @noguchi-takahiro. I have it on my queue.

@elsevers
Copy link
Copy Markdown
Contributor

Following up on out 5/11 meeting:

@noguchi-takahiro
Copy link
Copy Markdown
Contributor

Hello @elsevers, I have addressed the item 1 in this comment. See the following rendered articles:

I also updated the index so that those contents show up on the left banner.

@Daehoon-Sung regarding the following comment, can you take a few minutes to review this discussion and make sure if we addressed all of these within the autogen article?

Take a look at the top two discussions here: https://github.com/Severson-Group/AMDC-Examples/discussions . Do we need to update out article to factor any of this in?

My takeaway is we already addressed these but I wanted to double check this with you.

Copy link
Copy Markdown

@mhmokhtarabadi mhmokhtarabadi left a comment

Choose a reason for hiding this comment

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

Thank you @Daehoon-Sung and @noguchi-takahiro for this nice article. I have some inline comments. Feel free to comment on what you think if you don't like my suggestions.

Comment thread source/getting-started/tutorials/autogen/index.md Outdated
Comment thread source/getting-started/tutorials/autogen/index.md Outdated
Comment thread source/getting-started/tutorials/autogen/index.md Outdated
Comment thread source/getting-started/tutorials/autogen/index.md
Comment on lines +52 to +59
```MATLAB
samplingFrequency_Hz = 10e3; % sampling frequency (Hz)
samplingTime_s = 1/samplingFrequency_Hz; % sampling time (s)
simulationTime_s = samplingTime_s/10; % simulation time (s)

omegaElec_radps = 377.0; % electrical angular velocity (rad/s)
dutyAmplitude = 0.8; % amplitude of the duty-ratio waveform (--)
```
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@noguchi-takahiro, do you think we should follow PR https://github.com/Severson-Group/KnowledgeBase/pull/366 as the reference for naming the variables here?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you @mhmokhtarabadi, can I request you to update naming convention for both setup.m and setup_model.slx and generate_duty.slx for me?

@noguchi-takahiro
Copy link
Copy Markdown
Contributor

Thank you @mhmokhtarabadi. I addressed your feedback. Can you kindly update naming convention based on this latest article here or https://github.com/Severson-Group/KnowledgeBase/pull/366?

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 AMDC article of Autogen in "Control with AMDC" section Add AMDC tutorial article using Simulink Autogen

4 participants