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

Update test results for me cost model changes #1955

Closed
wants to merge 1 commit into from

Conversation

mjprilliman
Copy link
Collaborator

-Update results for changes in NREL/ssc#1249

@mjprilliman mjprilliman added the dev bug Internal bug that should not be included in release notes label Dec 6, 2024
@mjprilliman mjprilliman added this to the SAM Fall 2024 Release milestone Dec 6, 2024
@mjprilliman mjprilliman requested a review from sjanzou December 6, 2024 20:06
@mjprilliman mjprilliman self-assigned this Dec 6, 2024
@mjprilliman mjprilliman closed this Dec 6, 2024
@mjprilliman mjprilliman deleted the me-cost-updates branch December 6, 2024 22:34
@mjprilliman
Copy link
Collaborator Author

Closed as changes are already included in #1958

@sjanzou
Copy link
Collaborator

sjanzou commented Dec 8, 2024

Closed as changes are already included in #1958

We should come up with a better process of updating the default results and the api files; especially when there is a flurry of pull requests and failing GitHub Actions...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev bug Internal bug that should not be included in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants