Closed Alfred-Onuada closed 9 months ago
@microsoft-github-policy-service agree
@Alfred-Onuada
Please can you check the code, I am getting the error
@Alfred-Onuada I think the code works for some plot like scatter plot but not histogram
I will have a look now, thanks for mentioning
@Alfred-Onuada I was already looking into it so I was able to spot the error.
In your scaffold template, you need to add "fig.show()" before returning chart. Please add that
@trojrobert take a look again I pushed a new commit to make sure visualize for plotly returns the raster and chart, to prevent errors.
I also tested with histograms across 3 datasets and it worked fine, let me know if you face any errors.
It is still not working. I actually already have the code to solve this but I wanted you to also contribute.
Please show me a test
@trojrobert
Below is the cli test using pytest
what test command are you using?
@Alfred-Onuada You code has a new dependency, we need to add that to the requirements. What version of kaleido are you using?
@Alfred-Onuada I commented on this before. You need to add fig.show() to the template
@Alfred-Onuada Please check my comment on scaffold.py.
Please also test with multiple dataset
@trojrobert kaleido@0.2.1, I have also included the fig.show() in the template
@victordibia I think this PR should be done.
Thanks @Alfred-Onuada and @trojrobert I'll review this PR and merge soon!
-V.
This pull request adds support for the missing Plotly visualization library.
so the following code won't throw an error anymore
lida.visualize(summary=summary, goal=goals[3], textgen_config=textgen_config, library='plotly')