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

BUGFIX: Desalination Application #28

Merged
merged 2 commits into from
May 17, 2023
Merged

Conversation

kmruehl
Copy link
Contributor

@kmruehl kmruehl commented Jul 27, 2022

Resolving WEC-Sim/WEC-Sim#918

Tests pass locally, not sure why they don't pass on the runner

@kmruehl
Copy link
Contributor Author

kmruehl commented Feb 8, 2023

@H0R5E any thoughts on what could be causing this issue?

@H0R5E
Copy link
Contributor

H0R5E commented Feb 9, 2023

@kmruehl, I can't test this locally, as I don't have a Simscape Fluids license. Can you retrigger the workflow? It was such a long time since it was last run that maybe something has changed with the MATLAB runners.

@H0R5E
Copy link
Contributor

H0R5E commented Feb 9, 2023

OK, getting a more informative error now:

  ================================================================================
  Error occurred in TestDesalination/testDesalination and it did not run to completion.
      ---------
      Error ID:
      ---------
      'SL_SERVICES:utils:STD_EXCEPTION'
      --------------
      Error Details:
      --------------
      Error using wecSim
      STD exception 'pmf::PreFormattedMessageException': 'Could not open file
      /home/runner/.matlab/R2022b/thisMatlab.pem. No such file or directory.' was
      caught.
      
      Error in TestDesalination/testDesalination (line 54)
                  wecSim
  ================================================================================

This looks like a permissions problem, so it's something to do with the MATLAB runner. I'll open an issue about it tomorrow.

@kmruehl kmruehl self-assigned this May 17, 2023
@kmruehl
Copy link
Contributor Author

kmruehl commented May 17, 2023

The desal test case was failing locally, so I updated the slx case file. The desal test now passes locally, so let's see if that resolves the issue remotely.

@kmruehl
Copy link
Contributor Author

kmruehl commented May 17, 2023

refer to discussion here: WEC-Sim/WEC-Sim#918

The desal tests pass locally, but I was unable to resolve the issue on the runner. So, I went ahead and removed the desal test from the runner. It'll only be running locally from now on.

@kmruehl kmruehl merged commit d416f7b into WEC-Sim:dev May 17, 2023
@kmruehl kmruehl deleted the desal-bugfix branch May 17, 2023 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants