fix issue when there is multiple legends #401
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Case 1. When there is only one legend:
In this case I just let the location of the Plotly legend be similar to the location of the Matlab legend. I attach an example screenshot.
Case 2: When there are multiple legends.
Before making changes to the code,
fig2plotlydoes the following (see screenshot). As we can see, the error appears because the philosophy of plotly for the legends is different from MATLAB. In MATLAB we can place a legend for each plot. While Plotly what it does is configure a single legend that handles the total group of plots.So, the current
fig2plotlycode places the Plotly legend according to the first MATLAB legend and there it places the information for all MATLAB legends.Updated code for this PR.
To solve the issue with multiple legends, I have updated the
fig2plotlycode and now it does the followinghere is the link to Chart-Studio https://chart-studio.plotly.com/~galvisgilberto/4574/mahalanobis/#/
Code example