CIRDLES / Topsoil

Community-driven replacement for Isoplot
Apache License 2.0
28 stars 35 forks source link

SVG for the concordia line + multiple samples #25

Closed spenchristoph closed 10 years ago

spenchristoph commented 10 years ago

Hi all, Well done on getting this out. It looks very promising! It would be great if the concordia line was in vector format to facilitate easy editing in Illustrator. Might I also recommend plotting multiple samples in a similar to redux, i.e. a blank line separates different samples which would plot in different colors. Cheers, Chris

pfif commented 10 years ago

Hi Chris ! First of all, thank you!

  1. Concordia line in Vector Format : I assume you've already exported a chart to SVG, and opened it in Illustrator. What are you able to do in illustrator : see the concordia line or just edit it ?
  2. Multiple samples : We've already talked about that feature and have planned to integrate it down the road ! :-)

Cheers, Florent.

spenchristoph commented 10 years ago

Hi Florent, I am able to view the svg (although you must add .svg to the end), but get the error "The operation cannot complete because of an unknown error. [CANT]" when I try to open it in Illustrator. This may be a Mac problem as others have said this problem is related to the font-families used in the visualisations. Chris

On Thu, May 22, 2014 at 3:06 PM, pfif notifications@github.com wrote:

Hi Chris ! First of all, thank you!

  1. Concordia line in Vector Format : I assume you've already exported a chart to SVG, and opened it in Illustrator. What are you able to do in illustrator : see the concordia line or just edit it ?
  2. Multiple samples : We've already talked about that feature and have planned to integrate it down the road ! :-)

Cheers, Florent.

— Reply to this email directly or view it on GitHubhttps://github.com/CIRDLES/topsoil/issues/25#issuecomment-43892423 .

pfif commented 10 years ago

Yeah, Illustrator's giving us that error too. We were hoping that it was just because we're a few updates behind. We're trying to fix this in the issue #26.

noahmclean commented 10 years ago

On a PC here -- I can open the .svg in Illustrator when I append the right extension, but the fonts are rasterized.

Chris is my office mate, and we were discussing this earlier. By 'the concordia line in vector format', Chris meant that he wanted topsoil to draw it using Bezier curves, like in Redux, instead of line segments. It looks like you're already using Bezier curves for the ellipses.

johnzeringue commented 10 years ago

Noah,

Glad to hear that it opens in Illustrator for Windows. We had verified that our SVGs were valid, but were unable to try opening them on a PC. I also just released a patch (v0.0.2) that sets sans-serif as a backup font which might fix your font issue and (if I'm really lucky) the CANT error on Mac.

Chris, if you could try to open a SVG from the new release in Illustrator and let me know how that goes, I'd appreciate it. Also, a copy of an SVG that does open in Illustrator for Mac would probably help me get this fixed.

Thanks for the clarification on the concordia line. Redux currently does not use Bezier curves for either the Wetherill or the Tera-Wasserburg concordias. This can be a new feature to both, however, if you think that the Bezier curve would be better. Additionally, we can add more line segments to the curve approximation, making it smoother.

Thanks for all the feedback! This is exactly what we need! I'm going to split this conversation up into multiple issues though, since it makes the issue tracker more effective. If you could submit suggests as individual issues in the future, it would be much appreciated.

CANT Illustrator Error: Issue #26 Bezier Concordia Line: Issue #27 Multiple Samples: Issue #28

I'm going to close this issue because it's not specific, but I hope these conversations will continue in the appropriate issues.

Noah, if you can create a new issue for your font problem that includes a screenshot (you should be able to paste it right into the issue), it would be much appreciated.

Thanks, John

spenchristoph commented 10 years ago

John, Same problem with the svg on Mac Illustrator ([CANT] error) using v0.0.2. This svg works fine: http://en.wikipedia.org/wiki/File:Bitmap_VS_SVG.svg Chris

On Thu, May 22, 2014 at 4:24 PM, John Zeringue notifications@github.comwrote:

Noah,

Glad to hear that it opens in Illustrator for Windows. We had verified that our SVGs were valid, but were unable to try opening them on a PC. I also just released a patch (v0.0.2) that sets sans-serif as a backup font which might fix your font issue and (if I'm really lucky) the CANT error on Mac.

Chris, if you could try to open a SVG from the new release in Illustrator and let me know how that goes, I'd appreciate it. Also, a copy of an SVG that does open in Illustrator for Mac would probably help me get this fixed.

Thanks for the clarification on the concordia line. Redux currently does not use Bezier curves for either the Wetherill or the Tera-Wasserburg concordias. This can be a new feature to both, however, if you think that the Bezier curve would be better. Additionally, we can add more line segments to the curve approximation, making it smoother.

Thanks for all the feedback! This is exactly what we need! I'm going to split this conversation up into multiple issues though, since it makes the issue tracker more effective. If you could submit suggests as individual issues in the future, it would be much appreciated.

CANT Illustrator Error: Issue #26https://github.com/CIRDLES/topsoil/issues/26 Bezier Concordia Line: Issue #27https://github.com/CIRDLES/topsoil/issues/27 Multiple Samples: Issue #28 https://github.com/CIRDLES/topsoil/issues/28

I'm going to close this issue because it's not specific, but I hope these conversations will continue in the appropriate issues.

Noah, if you can create a new issue for your font problem that includes a screenshot (you should be able to paste it right into the issue), it would be much appreciated.

Thanks, John

— Reply to this email directly or view it on GitHubhttps://github.com/CIRDLES/topsoil/issues/25#issuecomment-43902945 .

bowring commented 10 years ago

Note that Redux draws parametric line segments for Concordia using the Path2D.curveTo function as follows:

curvedP.curveTo(// (float) mapX(myWorkingSeg.minX()), (float) mapY(myWorkingSeg.minY()), (float) mapX(myWorkingSeg.controlX()), (float) mapY(myWorkingSeg.controlY()), (float) mapX(myWorkingSeg.maxX()), (float) mapY(myWorkingSeg.maxY()));

in ConcordiaGraphPanel.drawConcordiaLineSegments

On Thu, May 22, 2014 at 11:24 AM, John Zeringue notifications@github.comwrote:

Noah,

Glad to hear that it opens in Illustrator for Windows. We had verified that our SVGs were valid, but were unable to try opening them on a PC. I also just released a patch (v0.0.2) that sets sans-serif as a backup font which might fix your font issue and (if I'm really lucky) the CANT error on Mac.

Chris, if you could try to open a SVG from the new release in Illustrator and let me know how that goes, I'd appreciate it. Also, a copy of an SVG that does open in Illustrator for Mac would probably help me get this fixed.

Thanks for the clarification on the concordia line. Redux currently does not use Bezier curves for either the Wetherill or the Tera-Wasserburg concordias. This can be a new feature to both, however, if you think that the Bezier curve would be better. Additionally, we can add more line segments to the curve approximation, making it smoother.

Thanks for all the feedback! This is exactly what we need! I'm going to split this conversation up into multiple issues though, since it makes the issue tracker more effective. If you could submit suggests as individual issues in the future, it would be much appreciated.

CANT Illustrator Error: Issue #26https://github.com/CIRDLES/topsoil/issues/26 Bezier Concordia Line: Issue #27https://github.com/CIRDLES/topsoil/issues/27 Multiple Samples: Issue #28 https://github.com/CIRDLES/topsoil/issues/28

I'm going to close this issue because it's not specific, but I hope these conversations will continue in the appropriate issues.

Noah, if you can create a new issue for your font problem that includes a screenshot (you should be able to paste it right into the issue), it would be much appreciated.

Thanks, John

— Reply to this email directly or view it on GitHubhttps://github.com/CIRDLES/topsoil/issues/25#issuecomment-43902945 .

Jim Bowring Principal Investigator, www.CIRDLES.org http://www.CIRDLES.org%20

Computer Science College of Charleston 66 George Street Charleston, SC 29424

Google Voice: 843.608.1399 (preferred) Google Email: bowring@gmail.com

Office: JC Long room 222 843.953.0805 http://stono.cs.cofc.edu/~bowring/ bowringj@cofc.edu

R. Buckminster Fuller (1972): If humanity is to survive aboard our planet, it must become universally literate and preoccupied with inherently cooperative Comprehensive Anticipatory Design Science in which every human is concerned with accomplishing the comfortably sustainable well-faring of all other humans.

johnzeringue commented 10 years ago

Okay, sorry for the misunderstanding. I'll make note of this in issue #27 and add implementing that particular solution in Topsoil to an upcoming milestone.