Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-72838

SimulinkCheckParser error on fingerprinting files

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Minor Minor
    • analysis-model
    • operting system of runner is windows
      Analysis Model API Plugin Version 12.1.0
      Warnings Plugin Version 11.2.0

      The SimulinkCheckParser expects the id of the div.ReportContent block to be the filename of the model and passes it to the issuebuilder.

      issueBuilder.setFileName(system)

      but the id is only the model name and not the filename so the .slx extension is missing. What leads to fingerprint errors.

      [-ERROR-] Can't create fingerprints for some files:
      [Simulink Check Tool] [-ERROR-] - 'sldemo_mdladv' file not found

      or am I missing something?
      I tried to call the recordIssues from within the same directory the slx file is located.
      I tried it with the result file located next to the slx file.

          [JENKINS-72838] SimulinkCheckParser error on fingerprinting files

          Jayanth added a comment -

          Would hardcoding ".slx" in the argument be a valid solution or does it also support other model types such as .mdl?

          Jayanth added a comment - Would hardcoding " .slx" in the argument be a valid solution or does it also support other model types such as .mdl?

          I'm not really firm with matlab but I think only hardcoding .slx is not a good solution as you already pointed out .mdl file endings and there surely are even more.

          Kevin Broselge added a comment - I'm not really firm with matlab but I think only hardcoding .slx is not a good solution as you already pointed out .mdl file endings and there surely are even more.

          Jayanth added a comment -

          I have a couple of questions :

          1. Does the generated html report follow a standard template?
          2. Even if it does follow some standard, are there cases where content such as 'Model Advisor Report' are not populated? 

          (The screenshot below is from 'Simulink.html' , I targeted the Model Advisor Report content to extract the model name which can be used for fingerprinting the files ) 

           

           

          Jayanth added a comment - I have a couple of questions : Does the generated html report follow a standard template? Even if it does follow some standard, are there cases where content such as 'Model Advisor Report' are not populated?  (The screenshot below is from ' Simulink.html ' , I targeted the Model Advisor Report content to extract the model name which can be used for fingerprinting the files )     

          We do nothing special.

          We only run the standard matlab ModelAdvisor.

          For us this results in a result directory per model that has been checked which each contain a report.html file which we pass to the parser.

          e.g. 

          Kevin Broselge added a comment - We do nothing special. We only run the standard matlab ModelAdvisor. For us this results in a result directory per model that has been checked which each contain a report.html file which we pass to the parser. e.g. 

          Jayanth added a comment - - edited

          Ok thank you for the clarifications , so im guessing targeting the content of the ReportContent div would solve this problem. I will create a PR soon after testing it on my side . 

          Jayanth added a comment - - edited Ok thank you for the clarifications , so im guessing targeting the content of the ReportContent div would solve this problem. I will create a PR soon after testing it on my side . 

          Jayanth added a comment -

          Hey this issue has now been resolved . Link to the PR : https://github.com/jenkinsci/analysis-model/pull/1121

          Jayanth added a comment - Hey this issue has now been resolved . Link to the PR : https://github.com/jenkinsci/analysis-model/pull/1121

            jay_vts Jayanth
            thebro Kevin Broselge
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: