OpenFAST / matlab-toolbox

Collection of Matlab tools developed for use with OpenFAST
Apache License 2.0
50 stars 67 forks source link

Feature/Add MoorDyn capability #20

Closed pschuenemann closed 3 years ago

pschuenemann commented 3 years ago

Hallo,

I realized that the matlab-toolbox, especially Fast2Matlab and Matlab2Fast, could not properly handle MoorDyn input files. Thus, I did some copy and paste and adjustments within the mentioned functions and added the missing capability.

I thought that it may help other users in future and thus request to include it in the NREL's master repo. If you have any questions or reservations regarding the proposed changes, please let me know.

Best regards, Paul Schünemann

ebranlard commented 3 years ago

Thanks for doing that. In your first commit it seems like the entire file Fast2Matlab file was rewritten, which makes it harder to see the changes. I'm guessing the line endings were replaced. I don't know of an easy way to revert this..

There are other options, using a git rebase but I think the options above are easier. Could you maybe give it a try.

Sorry for the extra work! Let us know if you can try that.

pschuenemann commented 3 years ago

Hey @ebranlard,

you're right. In my first commit my changes are hard to see. Sorry for this. I checked it, but to my mind the line endings are the same. However, I saw that I somehow changed the name of the function to FAST2Matlab (to be the same as the file name, as MATLAB likes to have it). Due to this I guess git/github thought, that I have rewritten the whole function. Thus, I changed the function name back to the original one (Fast2Matlab) and committed this change, but this doesn't help. It looks still like I have rewritten the whole function.

To this end, I followed your last approach and restart everything from scratch, which I will pull request in a minute.

Cheers Paul