Skip to content

Merge gsl/develop to RRFS_dev#136

Closed
SamuelTrahanNOAA wants to merge 221 commits into
RRFS_devfrom
gsl/develop
Closed

Merge gsl/develop to RRFS_dev#136
SamuelTrahanNOAA wants to merge 221 commits into
RRFS_devfrom
gsl/develop

Conversation

@SamuelTrahanNOAA
Copy link
Copy Markdown
Collaborator

@SamuelTrahanNOAA SamuelTrahanNOAA commented Mar 14, 2022

See title for details.

There are conflicts where code was commented out, but that code changed.

SMoorthi-emc and others added 30 commits October 1, 2021 16:45
@SamuelTrahanNOAA
Copy link
Copy Markdown
Collaborator Author

There are conflicts where code was commented out, but that code changed.

@SamuelTrahanNOAA
Copy link
Copy Markdown
Collaborator Author

Developer @mdtoyNOAA made the conflicting changes in RRFS_dev, and he's gone this week. He commented out some code, and the code he commented out has changed in gsl/develop. I'm pretty sure we can resolve the conflicts by commenting out the new code. It looks like that will achieve his intended effect. Does anyone object?

@SamuelTrahanNOAA
Copy link
Copy Markdown
Collaborator Author

Developer @mdtoyNOAA made the conflicting changes in RRFS_dev, and he's gone this week. He commented out some code, and the code he commented out has changed in gsl/develop. I'm pretty sure we can resolve the conflicts by commenting out the new code. It looks like that will achieve his intended effect. Does anyone object?

I'm hoping for a response from @tanyasmirnova or @joeolson42 here.

@tanyasmirnova
Copy link
Copy Markdown

@SamuelTrahanNOAA It is ok with me to comment out the new code.

@joeolson42
Copy link
Copy Markdown
Collaborator

It's OK with me too.

@SamuelTrahanNOAA
Copy link
Copy Markdown
Collaborator Author

I'm going to generate new baselines before merging this.

@SamuelTrahanNOAA
Copy link
Copy Markdown
Collaborator Author

I'm going to have to make a branch for this merge due to the regression test logs. I'll open another PR shortly.

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.