Skip to content

Marine Tidal / LCOE Calculator default case opens with UI error #1983

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

Closed
cpaulgilman opened this issue Dec 17, 2024 · 8 comments
Closed

Marine Tidal / LCOE Calculator default case opens with UI error #1983

cpaulgilman opened this issue Dec 17, 2024 · 8 comments
Assignees

Comments

@cpaulgilman
Copy link
Collaborator

cpaulgilman commented Dec 17, 2024

Describe the bug

The default Marine Tidal / LCOE Calculator case displays a UI error when it opens.

image

To Reproduce

  1. Create a Marine Tidal / LCOE Calculator case.
  2. See error.

The error seems to be triggered by loading the Time Series option on the Tidal Resource page.

Operating System and Version

SAM 2024.12.12

Windows 11

@cpaulgilman cpaulgilman added this to the 2024 Release Patch 1 milestone Dec 17, 2024
@mjprilliman
Copy link
Collaborator

There's an issue with initialization of the tidal resource library on first open. Switching from Frequency of occurrence to Time series fixes this (you can then go back to Frequency of occurrence). Completely exiting SAM and reopening after doing this for tidal configs (LCOE and None) removes the error in subsequent runs. I can't replicate the initial error in the dev version (SAM or SAMOS) or the RC, @sjanzou any ideas on what's different for the release?

@sjanzou
Copy link
Collaborator

sjanzou commented Dec 18, 2024

There's an issue with initialization of the tidal resource library on first open. Switching from Frequency of occurrence to Time series fixes this (you can then go back to Frequency of occurrence). Completely exiting SAM and reopening after doing this for tidal configs (LCOE and None) removes the error in subsequent runs. I can't replicate the initial error in the dev version (SAM or SAMOS) or the RC, @sjanzou any ideas on what's different for the release?

Hmmm.... when I download and install the 2024.12.12 release and create and Marine Tidal / LCOE case, I receive no error on Windows:
image

@sjanzou
Copy link
Collaborator

sjanzou commented Dec 18, 2024

Also, when the release is downloaded and tested on MacOS, the Marine Tidal / LCOE configuration opens and runs without issue
image

@sjanzou
Copy link
Collaborator

sjanzou commented Dec 18, 2024

@mjprilliman, I checked the commit history and cannot find any changes between the release candidate and the full release in SAM-private...did any library folders change?

@cpaulgilman
Copy link
Collaborator Author

Yesterday I was able to replicate this error reliably (in Windows). Today, the case opens with no error, even after I uninstalled and reinstalled SAM 2024.12.12.

@janinefreeman janinefreeman modified the milestones: 2024 Release Patch 1, SAM Spring 2025 Release Mar 7, 2025
@sjanzou
Copy link
Collaborator

sjanzou commented Apr 7, 2025

With the 2025.4.7 Windows version, there is no error for me when opening ME Tidal /LCOE calculator:

Image

@mjprilliman, can you please confirm?

@sjanzou
Copy link
Collaborator

sjanzou commented Apr 7, 2025

Working with the sam-linux-2025-4-7.run on RHEL8.6:

Image

@sjanzou
Copy link
Collaborator

sjanzou commented Apr 7, 2025

Working with sam-macos-2025-4-7.pkg on MacOS 15.3.2

Image

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

No branches or pull requests

4 participants