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

Add link to figures and work directory on index.html #1254

Closed
bouweandela opened this issue Jul 23, 2021 · 6 comments · Fixed by #1256
Closed

Add link to figures and work directory on index.html #1254

bouweandela opened this issue Jul 23, 2021 · 6 comments · Fixed by #1256

Comments

@bouweandela
Copy link
Member

Hi! I recently used the bot for the first time and needed some help finding the output figures. Is the link to the figures something you could add to the output page? The output page looks really pretty btw.
"The figures are there: https://esmvaltool.cloud.dkrz.de/shared/esmvaltool/esmvalbot-output/pvcf_gmddraft-kr0c7llt/recipe_pv_capacity_factor_20210712_081803/plots/capacity_factor/main/
This really should be improved that this link is not given on the esmvalbot output page but one needs to put it together from its link and the main_log.txt"

Originally posted by @Emmadd in ESMValGroup/ESMValTool#1882 (comment)

@stefsmeets
Copy link
Contributor

It should link the images if they are provided in the provenance. Could this be an issue with the recipe? Did the authors declare the path to the output png as part of the provenance?

@bouweandela
Copy link
Member Author

Indeed this is only useful if the provenance is not quite right yet. Or maybe if you would want to download all figures?

@stefsmeets
Copy link
Contributor

Made a small pr with what this could look like. I don't have much time to test/debug it, so feel free to hi-jack in case this is useful, or otherwise close 😁

@Emmadd
Copy link

Emmadd commented Jul 26, 2021

I think the png path was included in the provenance. @katjaweigel should know for sure

Set provenance for output files

xprov <- list(
ancestors = list(
fullpath_filenames[i]
),
authors = list(
"cionni_irene"
),
projects = list("crescendo"),
caption = title,
statistics = list("other"),
realms = list("atmos"),
themes = list("phys"),
plot_file = filepng
)

provenance[[filencdf]] <- xprov

@bouweandela
Copy link
Member Author

bouweandela commented Jul 26, 2021

Ok, if that is the problem, this is partly my fault. Recording provenance for figures like this never quite worked as it should and we intend to completely drop support for it at some point (it is already buggy and needlessly complicated). The correct way to do it would be

xprov <- list(
    ancestors = list(
        fullpath_filenames[i]
    ),
    authors = list(
        "cionni_irene"
    ),
    projects = list("crescendo"),
    caption = title,
    statistics = list("other"),
    realms = list("atmos"),
    themes = list("phys"),
)
provenance[[filepng]] <- xprov
provenance[[filencdf]] <- xprov

i.e. write a new provenance record for every output file, regardless of the file type.

@katjaweigel
Copy link
Contributor

katjaweigel commented Jul 26, 2021

Ok, good to know, then I'll change that in the PR ESMValGroup/ESMValTool#2153 (when Mistral is working again)

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.

4 participants