Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • Umami Umami
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 34
    • Issues 34
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Merge requests 5
    • Merge requests 5
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Issue
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • atlas-flavor-tagging-tools
  • algorithms
  • UmamiUmami
  • Merge requests
  • !501

Extend flexibility in input var plotting functions

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Joschka Birk requested to merge birk-input-var-plots into master Apr 04, 2022
  • Overview 24
  • Commits 33
  • Pipelines 45
  • Changes 15

Summary

This MR introduces the following changes

  • using **kwargs in the plotting functions, allowing to specify any parameter that is understood in the plotting API (in this case the histogram_plot class) in the config file for plotting
  • removed the check_kwargs_var_plots function
  • moved the "kwargs translation" (for backwards compatibility) to the high-level functions in plot_input_vars.py (--> kwargs are translated to new naming right away, such that the new naming is used consistently afterwards)
  • making histogram_plot functions less verbose (logger.info -> logger.debug), to reduce stdout in large plotting scripts
  • adding a script that converts the "Parameters" section of certain docstrings to markdown tables. The resulting files can then be used in the docs (using our placeholder syntax we introduced in !476 (merged))

Example

With this change, you can now e.g. remove the ATLAS branding from the input var plots by simply adding atlas_brand: "" to the plot_settings in the config:

--> -->

Relates to the following issues

  • #147 (closed)

Conformity

  • Changelog entry
  • Documentation
  • Development guidelines
  • Style guides
Edited Apr 05, 2022 by Joschka Birk
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: birk-input-var-plots