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.

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

minor bug fix #431

Merged
merged 2 commits into from
Apr 2, 2020
Merged

minor bug fix #431

merged 2 commits into from
Apr 2, 2020

Conversation

lapadron
Copy link
Contributor

Complete this sentence
THIS PULL REQUEST IS READY TO MERGE

Feature or improvement description
Minor bug fix in SubDyn_Driver. The first dimension of array SDin was begin accessed from index 0 and the do loop had one step too many.

Impacted areas of the software
SubDyn_Driver

@andrew-platt
Copy link
Collaborator

Can we change this to a PR to the dev branch instead of master?

@rafmudaf
Copy link
Collaborator

Well actually it looks like @lapadron left off the commit making the change. This pull request only contains the commits already on dev.

@andrew-platt
Copy link
Collaborator

@lapadron did you push your update to your branch on GitHub?

@lapadron
Copy link
Contributor Author

Sorry. I hadn't done the 'git add'. Now I did it and repeated steps 5 and 6. Do I need to do a new pull request now?

@rafmudaf rafmudaf changed the base branch from master to dev March 27, 2020 18:06
Copy link
Collaborator

@rafmudaf rafmudaf left a comment

Choose a reason for hiding this comment

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

What about changing the loop from starting at 0 to start at 1?

This is more consistent with the Fortran convention and generally a simpler change.

@andrew-platt
Copy link
Collaborator

n should probably start at 0 so that line 241 gives T=0.

@rafmudaf
Copy link
Collaborator

Ah good point @andrew-platt

…re/MinorBugFix

Minor bug fix in SubDyn_Driver. The first dimension of array SDin was being accessed from index 0
@rafmudaf rafmudaf merged commit 36a6ae8 into OpenFAST:dev Apr 2, 2020
@rafmudaf rafmudaf mentioned this pull request Apr 2, 2020
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.

3 participants