Skip to content

[fix] convert_output() with updated SS3 report file#272

Open
Schiano-NOAA wants to merge 4 commits into
mainfrom
fix-ss3-conout
Open

[fix] convert_output() with updated SS3 report file#272
Schiano-NOAA wants to merge 4 commits into
mainfrom
fix-ss3-conout

Conversation

@Schiano-NOAA
Copy link
Copy Markdown
Collaborator

This PR fixes the issues users were encountering when convert_output() was failing with the most recent version of SS3.

This should be now fixed and still work as previously expected.

@Schiano-NOAA Schiano-NOAA added the bug Something isn't working label May 29, 2026
@github-actions
Copy link
Copy Markdown
Contributor

New version checklist

  • Package version in DESCRIPTION has been updated
  • Release notes have been drafted/published
  • Cheatsheet content has been updated (if applicable)
  • Cheatsheet version has been updated

@github-actions
Copy link
Copy Markdown
Contributor

Code Metrics Report

Coverage Code to Test Ratio Test Execution Time
66.3% 1:0.1 6m43s

Code coverage of files in pull request scope (50.0%)

Files Coverage
R/convert_output.R 50.5%
R/utils.R 43.3%

Reported by octocov

@Schiano-NOAA
Copy link
Copy Markdown
Collaborator Author

@iantaylor-NOAA could test out convert_output() on this branch with the newest versions of SS3? I tested it out with an example Jason sent me and it works not, but doesn't hurt to test on other stocks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant