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 the option to set y-axis of raw value plot to the limits of the color map plot #484

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

stefangachter
Copy link
Contributor

The evo allows to set the color map scale limits of the errors by the options of plot_colormap_min and plot_colormap_max. In this case, the raw error plots differ from the colored map plots. The option plot_raw_with_same_limits_as_colormap allows to align limits of raw error and map color plots.

image
image

@@ -595,6 +595,7 @@ def error_array(ax: plt.Axes, err_array: ListOrArray,
threshold: float = None, cumulative: bool = False,
color: str = 'grey', name: str = "error", title: str = "",
xlabel: str = "index", ylabel: typing.Optional[str] = None,
xlim: typing.Tuple[float, float] = None, ylim: typing.Tuple[float, float] = None,
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The correct type is: typing.Optional[typing.Tuple[float, float]]

@@ -95,6 +95,10 @@ def parser() -> argparse.ArgumentParser:
help="percentile of the error distribution to be used "
"as the upper bound of the color map plot "
"(in %%, overrides --plot_colormap_max)")
output_opts.add_argument(
"--plot_raw_with_same_limits_as_colormap", type=bool,
help="set y-axis of raw value plot to the limits "
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one space too much

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 this pull request may close these issues.

3 participants