Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-1752

Missing images in Doxygen

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: lsstDoxygen
    • Labels:
      None
    • Team:
      SQuaRE

      Description

      Observe the broken image in the Doxygen documentation for meas_base.

      This image is present in the meas_base repository, and appears correctly when running scons doc directly. However, it doesn't make it on to the web version of the docs.

        Attachments

          Activity

          Hide
          swinbank John Swinbank added a comment -

          The problem here is due to lsstDoxygen, which creates an "aggregate" Doxyfile by parsing and combining the Doxygen configurations from all the various packages. In order to do that properly (If we assume it can be done properly, but that's another story...), it has to canonicalize relative paths to things like example files and (in this case) images. Currently, it doesn't. The fix looks straightforward.

          Show
          swinbank John Swinbank added a comment - The problem here is due to lsstDoxygen , which creates an "aggregate" Doxyfile by parsing and combining the Doxygen configurations from all the various packages. In order to do that properly (If we assume it can be done properly, but that's another story...), it has to canonicalize relative paths to things like example files and (in this case) images. Currently, it doesn't. The fix looks straightforward.
          Hide
          swinbank John Swinbank added a comment -

          Here's a tiny review for you. The last time I asked for a review of a change of just a few characters, I got tutted at – but since I'm not completely sure I understand how Doxygen is actually be invoked by Buildbot (based on the logs, it looks as though lsstDoxygen is cloned directly rather than being installed through eups, but I may be confused), I'd appreciate a sanity check if you have a moment.

          Anyway, change is on tickets/DM-1752 in meas_base.

          Show
          swinbank John Swinbank added a comment - Here's a tiny review for you. The last time I asked for a review of a change of just a few characters, I got tutted at – but since I'm not completely sure I understand how Doxygen is actually be invoked by Buildbot (based on the logs, it looks as though lsstDoxygen is cloned directly rather than being installed through eups, but I may be confused), I'd appreciate a sanity check if you have a moment. Anyway, change is on tickets/DM-1752 in meas_base .
          Hide
          frossie Frossie Economou added a comment -

          Sorry John, completely failed to notice this one in review.

          Show
          frossie Frossie Economou added a comment - Sorry John, completely failed to notice this one in review.

            People

            Assignee:
            Unassigned Unassigned
            Reporter:
            swinbank John Swinbank
            Reviewers:
            Frossie Economou
            Watchers:
            Frossie Economou, John Swinbank
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.