Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Mismatch IN Rectifier and OUT Inverter #16

Open
florentLutz opened this issue Mar 10, 2023 · 0 comments
Open

Mismatch IN Rectifier and OUT Inverter #16

florentLutz opened this issue Mar 10, 2023 · 0 comments
Labels
bug Something isn't working Inverter Label for issues related to the inverter Rectifier Label for issues related to the rectifier

Comments

@florentLutz
Copy link
Owner

As of now the AC side of the rectifier does not correspond (in terme of what should be an input/output in the openMDAO sense of the term) to the AC side on an inverter. Should be investigated, but it might yield that it is not feasible, in which case, the current state will be chosen
@florentLutz florentLutz added bug Something isn't working Rectifier Label for issues related to the rectifier Inverter Label for issues related to the inverter labels Mar 10, 2023
florentLutz added a commit that referenced this issue Mar 10, 2023
@florentLutz
Folder setup and Rectifier correction
5270dee 
Set up of the folder for the generator.
Also renamed the inputs/outputs variables for the rectifier and their number to match the name at the input of the motor. Their type however, do not match, it should be investigated as mentioned in #16
Also moved the responsibility of the computation voltage_peak_in to the generator.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working Inverter Label for issues related to the inverter Rectifier Label for issues related to the rectifier
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@florentLutz