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 aims to fix #405.
Briefly, it introduces a new breaks function for the exponential transformation:
breaks_exp()
.It is also incorporated in
transform_exp()
as the breaks function.While it is not perfect, it most definitely is an improvement over the old situation. The way I came up with this break algorithm is just by eyeballing it. I found that if the range is greater than 3, the exponential transformation began to deform the space by a lot. When the range is smaller than 3, it wasn't too bad, so we're still using
breaks_extended()
there. When the range is larger than 3, we're labelling 0 and the lastn - 1
integers of the range.Some examples:
Small range reverts to
breaks_extended()
:Just after the cutoff point:
Even though the limits don't include 0, the expanded range does, so we still display 0 here:
Created on 2024-10-04 with reprex v2.1.1