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

Another pull request from johnoel/stock-synthesis #530

Closed
wants to merge 82 commits into from
Closed

Another pull request from johnoel/stock-synthesis #530

wants to merge 82 commits into from

Conversation

johnoel
Copy link
Contributor

@johnoel johnoel commented Nov 14, 2023

Concisely describe what has been changed/addressed in the pull request.

  • Resolves issue #

What tests have been done?

Where are the relevant files?

What tests/review still need to be done?

Is there an input change for users to Stock Synthesis?

Additional information (optional).

Copy link
Collaborator

@Rick-Methot-NOAA Rick-Methot-NOAA left a comment

Choose a reason for hiding this comment

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

Hi Jon,
Elizabeth has been the team member most involved with the docker project and she will be out for a couple of weeks. Please ping me if you must have this PR reviewed sooner.
Rick

@johnoel
Copy link
Contributor Author

johnoel commented Nov 15, 2023

Thanks Rick for letting me know. I will just wait for Elizabeth and work on other Docker improvements.

@e-perl-NOAA
Copy link
Collaborator

@johnoel Thank you for working on this! I'm finally back after being in and out for the past 2 weeks. I'll try this out this week and get back to you with any problems I have (hopefully I won't have any).

@e-perl-NOAA
Copy link
Collaborator

@johnoel, I added a comment to the code, but the reason that the actions are passing is because build-ss3.yml still contains the original code to get the admb release from GitHub. If you take those lines out, build.ss3 will not pass with the changes that you have made.

@e-perl-NOAA
Copy link
Collaborator

@johnoel, have you been able to dive into this more recently to see what the issue is? I would like to get this mostly solved this month so we are sure to have it wrapped up before March. Thank you!

@johnoel
Copy link
Contributor Author

johnoel commented Jan 9, 2024

Back from vacation, I will look at this today.

@johnoel
Copy link
Contributor Author

johnoel commented Jan 9, 2024

Hi Elizabeth (@e-perl-NOAA), can you review changes?

@e-perl-NOAA
Copy link
Collaborator

@johnoel please see my draft PR #550, there are some weird c++ errors now and the windows docker image still isn't working.

@johnoel
Copy link
Contributor Author

johnoel commented Jan 12, 2024

See Pull Request #552

@johnoel johnoel closed this Jan 12, 2024
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