Skip to content

Allow multiplots in matplotlib#94

Merged
bchretien merged 1 commit into
roboptim:masterfrom
fdarricau:master
Apr 30, 2015
Merged

Allow multiplots in matplotlib#94
bchretien merged 1 commit into
roboptim:masterfrom
fdarricau:master

Conversation

@fdarricau
Copy link
Copy Markdown
Contributor

Adding tools to be able to do cleaner plots using matplotlib tools, plus a test to show it.

In detail, this commit allow users to:

  • print well-defined titles for their plots (beware of calling command after the actual plot, it is a matplotlib mandatory and it won't work otherwise)
  • print several curves on the same figure in a multiplot style, instead of merging all into a single plot, which leads to poor readability

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This should follow the naming conventions of RobOptim: isPlot_.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think a std::pair<unsigned int, unsigned int> would be better to specify the layout, rather than forcing the plot on a single column.

@bchretien bchretien added this to the 3.1 release milestone Apr 30, 2015
@fdarricau fdarricau force-pushed the master branch 2 times, most recently from 3826a03 to 70fe5c7 Compare April 30, 2015 07:49
Take a layout instead of just a plot number
@bchretien
Copy link
Copy Markdown
Member

👍

bchretien added a commit that referenced this pull request Apr 30, 2015
Allow multiplots in matplotlib
@bchretien bchretien merged commit f097ef1 into roboptim:master Apr 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants