Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

DOC : document exact freetype versions for tests#4345

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

tacaswell
Copy link
Member

Document what version of freetype was used to generate baseline
images in PR#4031 / commit
005cfde and which version is
(currently) being run on travis.ci

@WeatherGod
Copy link
Member

How do we plan to keep this information up to date?

@tacaswell
Copy link
MemberAuthor

Developer good will?

If we are very clever we might be able to extract the Travis version as
part of the automated build process.

On Thu, Apr 16, 2015, 15:42 Benjamin Rootnotifications@github.com wrote:

How do we plan to keep this information up to date?


Reply to this email directly or view it on GitHub
#4345 (comment)
.

@WeatherGod
Copy link
Member

eh, good will or no, I see this going south pretty quickly. Also, we don't package images generated from Travis anyway. Doesn't our unit tests record which version of freetype is used in order to mark some tests as skipped?

@tacaswell
Copy link
MemberAuthor

The case that prompted this was a question in#4342 which is having un-explained failures on travis (maybe due to freetype version issues). Having this documented someplace other than an issue commit is a good thing, even if we risk it getting out of date.


Due to subtle differences in the font rendering under different
version of freetype some care must be taken when generating the
baseline images. Currently, almost all of the images were generated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

"Currently (2015)"

It might be a good idea to document when currently is? To make it more obvious if this is out of date

Document what version of freetype was used to generate baselineimages in PRmatplotlib#4031 / commit005cfde and which version is(currently) being run on travis.ci
@tacaswelltacaswellforce-pushed thedoc_freetype_img_test_version branch from6641d25 tobc4cf9aCompareApril 18, 2015 15:50
@tacaswell
Copy link
MemberAuthor

Updated and squashed.

WeatherGod added a commit that referenced this pull requestApr 20, 2015
DOC : document exact freetype versions for tests
@WeatherGodWeatherGod merged commitdba7626 intomatplotlib:masterApr 20, 2015
@tacaswelltacaswell deleted the doc_freetype_img_test_version branchApril 20, 2015 16:01
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@tacaswell@WeatherGod@jenshnielsen

[8]ページ先頭

©2009-2025 Movatter.jp