| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
- Rename map_margin to fig_margin in market_map.py and MarketMapModel.ts - Fix axis position by replacing this.width/this.height by this.plotareaWidth/this.plotareaHeight in MarketMap.ts
There was a problem hiding this comment.
Thanks for the PR! I'll try to make a patch release soon with this. It seems MarketMap is not tested at all on our pipeline
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Fixes a bug introduced after updating to 0.13.0 affecting the display of MarketMap figure and axis position.
Reproduce bug
On version 0.13.0, running cell 3 of notebook examples/Marks/Object Model/Market Map.ipynb renders blank. After fixing the rendering issue, cell 9 of the same notebook has axis mispositioned (see images below).
References
None
Code changes
Rename map_margin to fig_margin in market_map.py and MarketMapModel.ts to match getter in Figure.ts
Fix axis position by replacing this.width/this.height with this.plotareaWidth/this.plotareaHeight in MarketMap.ts
User-facing changes
Before
After
Before
After
Backwards-incompatible changes
None