-
Notifications
You must be signed in to change notification settings - Fork 386
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.
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
Turboelectric HTS Ducted Fan Network #535
Conversation
Hi, thanks for the PR! The issue is not with anything any of us did. Looks like there was an error installing some basic python packages way before anything SUAVE related. Here's a stack overflow link where someone had the same problem. I suspect this will be fixed soon. I tried to restart the build just now and it still errored. Instead of opening and closing the PR, just post a message here in again in a few days and I'll try to rebuild it manually. In general none of the SUAVE team members will look at a PR until green check mark appears. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi! Thanks for the PR.
Since you said you are new to using Github, I wanted to let you know what to expect. We'll have a set of reviews, from one or more developers to look over the code until we're satisfied. Then we'll approve your pull request. The automated tools (appeveyor and coveralls) must also approve and we have it set such that we cannot override them.
Just know that if any of our comments seem rude, they're not intended to be. We try to write concise comments. We sometimes leave hundreds of comments in a week.
Please ask us questions if you have any. We are here to help.
regression/scripts/turboelectric_HTS_ducted_fan_network/turboelectric_HTS_ducted_fan_network.py
Outdated
Show resolved
Hide resolved
regression/scripts/turboelectric_HTS_ducted_fan_network/turboelectric_HTS_ducted_fan_network.py
Outdated
Show resolved
Hide resolved
trunk/SUAVE/Components/Energy/Networks/Turboelectric_HTS_Ducted_Fan.py
Outdated
Show resolved
Hide resolved
trunk/SUAVE/Components/Energy/Networks/Turboelectric_HTS_Ducted_Fan.py
Outdated
Show resolved
Hide resolved
trunk/SUAVE/Components/Energy/Networks/Turboelectric_HTS_Ducted_Fan.py
Outdated
Show resolved
Hide resolved
trunk/SUAVE/Methods/Cooling/Cryocooler/Cooling/cryocooler_model.py
Outdated
Show resolved
Hide resolved
trunk/SUAVE/Methods/Cooling/Cryocooler/Cooling/cryocooler_model.py
Outdated
Show resolved
Hide resolved
trunk/SUAVE/Methods/Propulsion/serial_HTS_turboelectric_sizing.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple quick comments. I'll have another set of eyes take a look this week to approve.
Nice work!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few stylistic changes then I will approve. Great work!
Pull Request Checklist