add pdf renderings of testfiles #752

Merged
merged 1 commit into from Mar 12, 2014

Projects

None yet

3 participants

@lvinken
Contributor
lvinken commented Mar 9, 2014

For easy reference, add PDF renderings (by MuseScore 1.3) of test files that have not been changed since 1.3 (with the possible exception of changed line breaks).

@lasconic
Member

Hi Leon,
I understand the point of having visual references. I wonder why PDF though? It seems PNG would be easier since they could be displayed in a webpage in the future. A bit like we do for visual tests http://vtest.musescore.org/3310bf22/vtest.html
See http://dev-list.musescore.org/Visual-test-suite-td7578380.html

What do you think?

@lvinken
Contributor
lvinken commented Mar 10, 2014

Hi Nicolas,

of course I did not consider use in a webpage at all, I simply find it
convenient to have rendered versions of all test files available.

A number of test files have had PDF renderings included for a long time,
which worked well for me, so I did not even consider changing image file
format.

I find PDF easy and flexible to work with: small file size and allowing
zoom in without image quality degradation. If you want reasonable quality
with PNG, you have to render at least at about 100 dpi, get significantly
bigger files than the PDF files and you still cannot zoom in.

Note that if you insist, I have no problem with rendering all files again
as PNG and removing the existing PDFs.

Regards, Leon.

On Mon, Mar 10, 2014 at 8:53 AM, Nicolas Froment
notifications@github.comwrote:

Hi Leon,
I understand the point of having visual references. I wonder why PDF
though? It seems PNG would be easier since they could be displayed in a
webpage in the future. A bit like we do for visual tests
http://vtest.musescore.org/3310bf22/vtest.html
See http://dev-list.musescore.org/Visual-test-suite-td7578380.html

What do you think?

Reply to this email directly or view it on GitHubhttps://github.com/musescore/MuseScore/pull/752#issuecomment-37159344
.

@wschweer wschweer merged commit 2fdc4be into musescore:master Mar 12, 2014

1 check passed

default The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment