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

PowerShell script BuildTestReleaseMATLAB throws an error #180

Closed
lmalenfant opened this issue Nov 7, 2024 · 3 comments · Fixed by #181
Closed

PowerShell script BuildTestReleaseMATLAB throws an error #180

lmalenfant opened this issue Nov 7, 2024 · 3 comments · Fixed by #181
Assignees

Comments

@lmalenfant
Copy link
Member

Describe the bug
I cloned a clean VTS copy and selected the master branch updated on Oct 2, 2024. When I used ./BuildTestReleaseMATLAB, the MATLAB release generation was stopped after opening the MATLAB. The MATLAB error was
Running unit tests for Monte Carlo simulations...
Unable to resolve the name 'Vts.MonteCarlo.Detectors.ROfRhoDetectorInput'.

Error in DetectorInput.ROfRho (line 175)
sampleInput = Vts.MonteCarlo.Detectors.ROfRhoDetectorInput;

Error in mc_tests (line 5)
si = SimulationInput();

Error in vts_tests (line 10)
mc_tests();

However, when I checked an earlier date (Sep 17,2024 Merge pull request #171...), I was able to generate a MATLAB release without any problems.

Application (MCCL, MATLAB or VTS code):
MATLAB

To Reproduce
Steps to reproduce the behavior:

  1. Clone the VTS
  2. In the VTS folder select the file BuildTestReleaseMATLAB.ps1
  3. Right-click and select Run with PowerShell
  4. See error

Expected behavior
The MATLAB tests should run and a zip file is created

@lmalenfant lmalenfant self-assigned this Nov 7, 2024
@lmalenfant
Copy link
Member Author

@janakarana this was what Carole described and what you alerted to us previously. The libraries were not being copied to the MATLAB folder because win-x64 is only created when running BuildTestRelease, if BuildTestReleaseMATLAB is run in isolation then it needs to pull those libraries from local. I made the change on the branch, could you pull this version and if it fixes you issue, I will merge into master.

@janakarana
Copy link
Member

@lmalenfant I pulled the code and ran BuildTestReleaseMATLAB. I created the release without any errors. Thank you

@lmalenfant
Copy link
Member Author

@janakarana if you approve the PR I can merge it.

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 a pull request may close this issue.

2 participants