Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP padLinear #213

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

WIP padLinear #213

wants to merge 3 commits into from

Conversation

curran
Copy link
Contributor

@curran curran commented Jun 17, 2020

Closes #150

Please let me know if this looks like a good direction. With some encouragement I'm happy to flesh out the PR with README updates - but first I want to confirm this direction is acceptable. Thanks for reviewing!

@curran curran changed the title WIP Expose pad from linear. WIP padLinear Jun 17, 2020
@curran curran mentioned this pull request Jun 17, 2020
@Fil
Copy link
Member

Fil commented Jul 27, 2020

It would be nice to see how this would work for other types of scales (log, sqrt…)

@curran
Copy link
Contributor Author

curran commented Feb 13, 2021

An idea from https://observablehq.com/@mbostock/inequality-in-american-cities :

function padLog(x, k) {
  return padLinear(x.map(Math.log), k).map(Math.exp);
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Pad domain addition?
2 participants