-
Notifications
You must be signed in to change notification settings - Fork 2
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
Now using anti-aliasing on svg, updated NodeOS.ppm #3
base: master
Are you sure you want to change the base?
Conversation
@@ -3,6 +3,7 @@ | |||
xmlns="http://www.w3.org/2000/svg" | |||
width="56cm" | |||
height="60cm" | |||
shape-rendering="geometricPrecision" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does "geometricPrecission" give better results? Shouldn't it be "crispEdges"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It outputs a really nice image compared to what I've got before. I'll test it with crispEdges again, but I'm not use if crispEdges really improves the quality of the picture
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The improvements are somewhat of the same. CrispEdges produces a "color-true" version with a bit of "damage" to the borders while geometricPrecission produces a, well, geometrically precise image. It's up to you :P
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It should force to output borders with high contrast, reducing the numbers
of colours. If that's not enlught, then we'll need to reduce the palette of
the PNG image.
El 9/8/2016 16:30, "John Green" [email protected] escribió:
In NodeOS.svg
#3 (comment):@@ -3,6 +3,7 @@
xmlns="http://www.w3.org/2000/svg"
width="56cm"
height="60cm"
- shape-rendering="geometricPrecision"
It outputs a really nice image compared to what I've got before. I'll test
it with crispEdges again, but I'm not use if crispEdges really improves the
quality of the picture—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/NodeOS/media/pull/3/files/231d44d1dcfea0e3485b9048a4c589cd61cfc889#r74069206,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAgfvqVaB_n6MG84BSfgYiJ8wCidwXdZks5qeI70gaJpZM4JgHOI
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you show both to compare?
El 9/8/2016 16:35, "John Green" [email protected] escribió:
In NodeOS.svg
#3 (comment):@@ -3,6 +3,7 @@
xmlns="http://www.w3.org/2000/svg"
width="56cm"
height="60cm"
- shape-rendering="geometricPrecision"
The improvements are somewhat of the same. CrispEdges produces a
"color-true" version with a bit of "damage" to the borders while
geometricPrecission produces a, well, geometrically precise image. It's up
to you :P—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/NodeOS/media/pull/3/files/231d44d1dcfea0e3485b9048a4c589cd61cfc889#r74070287,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAgfvjQuYgggeq-P6WrRK6gM7aXX-L4-ks5qeJArgaJpZM4JgHOI
.
Give them a try to both, but I think crispEdge will generate a better I'm thinking that since it can be defined with CSS and would be nice to El 9/8/2016 16:42, "John Green" [email protected] escribió:
|
Ok so I'll use the CrispEdge and later we will do what ? I was thinking about hardcoding the logo via diffs (just create a new file) and not generating it at runtime... |
Yeah, include it on the diff patch, later we'll see what we do with the svg El 9/8/2016 16:52, "John Green" [email protected] escribió:
|
Ok, fine. |
I tested |
They are not the better, but for our use case it is ;-) In fact the O is El 10/8/2016 9:03, "John Green" [email protected] escribió:
|
The ppm is at my fork |
Reviewed ;-) Don't know how it looks but the ppm code don't have glitches, El 10/8/2016 9:29, "John Green" [email protected] escribió:
|
Sure, that's fine ! |
The quality of the ppm is better now but still not as good as the svg's !