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

Unrecognized segment command: e #55

Closed
emeeks opened this issue Apr 16, 2018 · 4 comments
Closed

Unrecognized segment command: e #55

emeeks opened this issue Apr 16, 2018 · 4 comments
Labels

Comments

@emeeks
Copy link

emeeks commented Apr 16, 2018

roughGenerator.path will error out when given coordinates that are using scientific notation. Such as:

"M0,14.285714285714334L6.666666666666667,15.645165893991958L13.333333333333334,13.621434467236043L20,13.666889665817564L26.666666666666668,12.023412687602018L33.33333333333333,13.243774216451186L40,16.13238259981665L46.666666666666664,14.213536271906634L53.333333333333336,10.717681905478344L60,9.380249589210877L66.66666666666666,6.711179246747719L73.33333333333333,4.461618721950487L80,1.4708837451115357L86.66666666666667,3.359501689846894L93.33333333333333,0.03474343373119382L100,5.684341886080802e-14L106.66666666666667,2.603415833149171L113.33333333333333,6.052247452057372L120,9.12101436682056L126.66666666666666,9.141835233343272L133.33333333333331,9.0660892978415L140,9.953911817145155L146.66666666666666,12.450879362054053L153.33333333333334,9.649799726633688L160,13.035117833272778L166.66666666666669,9.896493775623355L173.33333333333334,11.033623453666053L180,9.70015386837514L186.66666666666666,8.748002714353476L193.33333333333334,10.960013271080328L200,12.351601607315104L200,21.645476831474497L193.33333333333334,18.538159084791687L186.66666666666666,19.765878642824674L180,18.524637635901854L173.33333333333334,23.13767925589468L166.66666666666669,20.0561877117795L160,23.922978272291886L153.33333333333334,21.327285732462798L146.66666666666666,22.835220857805837L140,19.464872113830666L133.33333333333331,21.48062185638247L126.66666666666666,24.949540935783574L120,22.804503808048736L113.33333333333333,22.62795094018361L106.66666666666667,16.31183693402673L100,13.068864406281222L93.33333333333333,12.896195068125735L86.66666666666667,15.41399692966391L80,16.000620863852532L73.33333333333333,18.38334497905032L66.66666666666666,19.79949753439365L60,19.95550516109813L53.333333333333336,22.212629025892227L46.666666666666664,24.86703498591848L40,29.122837164608313L33.33333333333333,23.293803174076174L26.666666666666668,21.881592853666803L20,20.690654271435704L13.333333333333334,19.552547736496933L6.666666666666667,20.740111759007306L0,21.428571428571473Z"
@pshihn
Copy link
Collaborator

pshihn commented Apr 16, 2018

Yes. I wrote a rudimentary parser for paths that does not take this into account. Oops.

@emeeks
Copy link
Author

emeeks commented Apr 16, 2018

I'm not sure why SVG insists on floating point precision in its output. I mean, I know SVGs can be zoomed but yeesh.

@pshihn
Copy link
Collaborator

pshihn commented Apr 16, 2018

Shouldn't be too hard to fix, hopefully. Will take a stab at it tonight.

@pshihn
Copy link
Collaborator

pshihn commented Apr 16, 2018

Turns out it was a faulty regex cleanup. Was easy to fix. Should work in v2.1.2

@pshihn pshihn closed this as completed Apr 16, 2018
@pshihn pshihn added the bug label Apr 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants