You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Fixesadd symmetrical log scale #221 (Symlog Scale): Added a new axis type \symlog\ (symmetrical log). This uses an \�sinh\ transformation to allow log-like scaling that passes smoothly through zero and handles negative numbers.
Fixes[Feature Request] Support line styles in Scattermapbox #6791 (Line Dash): Added support for \line.dash\ in \scattermap\ and \scattermapbox\ traces, mapping Plotly dash strings (e.g. 'dot', 'dashdot') to Mapbox's \line-dasharray\ format.
Changes:
Modified \src/plots/map/layout_defaults.js\ for auto-bounds.
Modified \src/plots/cartesian/set_convert.js\ and \layout_attributes.js\ for \symlog.
Modified \�ttributes.js\ and \convert.js\ in \src/traces/scattermap\ and \src/traces/scattermapbox\ for dashed lines.
Could you split your work into separate PRs for each issue?
To address the individual issues:
Map zoom and center: I would recommend to hold off further work for now — another contributor is working on a fix in Dynamic map defaults #7692 which looks promising and I'd like to give them the chance to pursue it.
symlog axis type: Doesn't seem to be quite working yet but I encourage you to open a separate PR and keep working on it. You'll want to add an image test by adding a .json file under test/image/mocks/ which shows an example of the symlog axis type in action.
line styles in scattermapbox: Likewise doesn't seem to be working yet but feel free to open a separate PR and continue work.
I'm going to close this PR but please open separate PRs for either or both of the latter two issues if you'd like to keep working on them!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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.
This PR addresses three issues:
zoomandcenterforscattermaptraces does not show all points #7674 (Scattermap Zoom): Implemented auto-calculation for \zoom\ and \center\ in \scattermap\ traces when they are not provided by the user. It now computes the bounding box of the data and fits the map view accordingly.Changes: