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.

Sign up for GitHub

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

Jump to bottom

Dekko: repo cleaned #7

Merged
merged 8 commits into from
Jan 18, 2023
Merged

Dekko: repo cleaned #7

merged 8 commits into from
Jan 18, 2023

Conversation

emmamarichal
Copy link
Contributor

@emmamarichal emmamarichal commented Jan 18, 2023 •
edited
Loading

Hi @EbenSorkin,

I cleaned the Dekko repo, according to our usual template.
The file seems to be much less damaged than Asar as far as devanagari is concerned.

I made a list of things to work on for the next upgrade:

I also noticed a bug in features, I don't know what it means since it concerns the devanagari. For now I just removed it to export the font.
Capture d’écran 2023-01-18 à 12 45 17

I'm going to open issues in this repo, I'll link them here.
Cheers!

Emma

@emmamarichal emmamarichal mentioned this pull request Jan 18, 2023
Open
@emmamarichal
Copy link
Contributor Author

Fontbakery report

Fontbakery version: 0.8.10

[1] Family checks
ℹ INFO: Check axis ordering on the STAT table. (com.google.fonts/check/STAT/axis_order)

  • ℹ INFO From a total of 1 font files, 1 of them (100.00%) lack a STAT table.

    And these are the most common STAT axis orderings:
    [code: summary]


[16] Dekko-Regular.ttf
🔥 FAIL: Check Google Fonts glyph coverage. (com.google.fonts/check/glyph_coverage)

  • 🔥 FAIL Missing required codepoints:

    • 0x0312 (COMBINING TURNED COMMA ABOVE)
      [code: missing-codepoints]
⚠ WARN: Glyphs are similiar to Google Fonts version? (com.google.fonts/check/production_glyphs_similarity)
  • ⚠ WARN Following glyphs differ greatly from Google Fonts version:
    • Ccedilla
    • Scedilla
    • Uogonek
    • ccedilla
    • eogonek
    • hookabovecomb
    • iogonek
    • scedilla
    • uni0162
    • uni0163 and 36 more.

Use -F or --full-lists to disable shortening of long lists.

⚠ WARN: Ensure fonts have ScriptLangTags declared on the 'meta' table. (com.google.fonts/check/meta/script_lang_tags)
  • ⚠ WARN This font file does not have a 'meta' table. [code: lacks-meta-table]
⚠ WARN: Check font contains no unreachable glyphs (com.google.fonts/check/unreachable_glyphs)

  • ⚠ WARN The following glyphs could not be reached by codepoint or substitution rules:

    • Gtilde

    • I.uc

    • asterisk.001

    • bha_radeva

    • ca_cadeva

    • ca_radeva

    • dha_radeva

    • f_f_i_dflt

    • f_i_trk

    • firsttonechinesecomb

    • And 44 more.

Use -F or --full-lists to disable shortening of long lists.
[code: unreachable-glyphs]

⚠ WARN: Check if each glyph has the recommended amount of contours. (com.google.fonts/check/contour_count)
  • ⚠ WARN This check inspects the glyph outlines and detects the total number of contours in each of them. The expected values are infered from the typical ammounts of contours observed in a large collection of reference font families. The divergences listed below may simply indicate a significantly different design on some of your glyphs. On the other hand, some of these may flag actual bugs in the font such as glyphs mapped to an incorrect codepoint. Please consider reviewing the design and codepoint assignment of these to make sure they are correct.

The following glyphs do not have the recommended number of contours:

- Glyph name: dollar	Contours detected: 0	Expected: 1, 3 or 5

- Glyph name: question	Contours detected: 0	Expected: 2

- Glyph name: bracketleft	Contours detected: 0	Expected: 1

- Glyph name: bracketright	Contours detected: 0	Expected: 1

- Glyph name: braceleft	Contours detected: 0	Expected: 1

- Glyph name: bar	Contours detected: 0	Expected: 1

- Glyph name: braceright	Contours detected: 0	Expected: 1

- Glyph name: asciitilde	Contours detected: 0	Expected: 1

- Glyph name: exclamdown	Contours detected: 0	Expected: 2

- Glyph name: cent	Contours detected: 0	Expected: 1 or 2 

- And 160 more.

Use -F or --full-lists to disable shortening of long lists.
[code: contour-count]

⚠ WARN: Does GPOS table have kerning information? This check skips monospaced fonts as defined by post.isFixedPitch value (com.google.fonts/check/gpos_kerning_info)
  • ⚠ WARN GPOS table lacks kerning information. [code: lacks-kern-info]
⚠ WARN: Do any segments have colinear vectors? (com.google.fonts/check/outline_colinear_vectors)

  • ⚠ WARN The following glyphs have colinear vectors:

    • AE (U+00C6): L<<766.0,653.0>--<767.0,688.0>> -> L<<767.0,688.0>--<767.0,716.0>>

    • AEacute (U+01FC): L<<766.0,653.0>--<767.0,688.0>> -> L<<767.0,688.0>--<767.0,716.0>>

    • K (U+004B): L<<315.0,1126.0>--<303.0,792.0>> -> L<<303.0,792.0>--<303.0,729.0>>

    • T (U+0054): L<<177.0,1192.0>--<598.0,1196.0>> -> L<<598.0,1196.0>--<757.0,1206.0>>

    • Tbar (U+0166): L<<182.0,1192.0>--<603.0,1196.0>> -> L<<603.0,1196.0>--<762.0,1206.0>>

    • Tcaron (U+0164): L<<177.0,1192.0>--<598.0,1196.0>> -> L<<598.0,1196.0>--<757.0,1206.0>>

    • at (U+0040): L<<1385.0,-198.0>--<1386.0,-198.0>> -> L<<1386.0,-198.0>--<1386.0,-198.0>>

    • b (U+0062): L<<105.0,198.0>--<105.0,284.0>> -> L<<105.0,284.0>--<122.0,1372.0>>

    • f (U+0066): L<<103.0,913.0>--<247.0,907.0>> -> L<<247.0,907.0>--<256.0,907.0>>

    • florin (U+0192): L<<105.0,913.0>--<253.0,907.0>> -> L<<253.0,907.0>--<258.0,907.0>>

    • And 24 more.

Use -F or --full-lists to disable shortening of long lists. [code: found-colinear-vectors]

⚠ WARN: Do outlines contain any jaggy segments? (com.google.fonts/check/outline_jaggy_segments)

  • ⚠ WARN The following glyphs have jaggy segments:

    • Ldot (U+013F): B<<734.0,212.0>-<737.0,213.0>-<739.0,213.0>>/L<<739.0,213.0>--<734.0,212.0>> = 11.309932474020195

    • aogonek (U+0105): B<<823.0,212.0>-<833.0,203.0>-<840.0,192.0>>/L<<840.0,192.0>--<827.0,217.0>> = 4.996760664571314

    • aogonek (U+0105): L<<849.0,175.0>--<846.0,181.0>>/B<<846.0,181.0>-<853.0,164.0>-<853.0,144.0>> = 4.184916125118319

    • d (U+0064): B<<689.5,912.5>-<760.0,862.0>-<769.0,779.0>>/B<<769.0,779.0>-<769.0,808.0>-<769.5,865.5>> = 6.1886159632415545

    • dcaron (U+010F): B<<689.5,912.5>-<760.0,862.0>-<769.0,779.0>>/B<<769.0,779.0>-<769.0,808.0>-<769.5,865.5>> = 6.1886159632415545

    • dcroat (U+0111): B<<689.5,912.5>-<760.0,862.0>-<769.0,779.0>>/B<<769.0,779.0>-<769.0,821.0>-<770.0,919.5>> = 6.1886159632415545

    • dong (U+20AB): B<<801.5,1217.5>-<872.0,1167.0>-<881.0,1084.0>>/B<<881.0,1084.0>-<881.0,1123.0>-<881.5,1214.5>> = 6.1886159632415545

    • g (U+0067): B<<686.5,-2.0>-<690.0,69.0>-<697.0,129.0>>/B<<697.0,129.0>-<686.0,81.0>-<642.0,47.0>> = 6.252983625259244

    • gbreve (U+011F): B<<686.5,-2.0>-<690.0,69.0>-<697.0,129.0>>/B<<697.0,129.0>-<686.0,81.0>-<642.0,47.0>> = 6.252983625259244

    • gcaron (U+01E7): B<<686.5,-2.0>-<690.0,69.0>-<697.0,129.0>>/B<<697.0,129.0>-<686.0,81.0>-<642.0,47.0>> = 6.252983625259244

    • And 50 more.

Use -F or --full-lists to disable shortening of long lists. [code: found-jaggy-segments]

⚠ WARN: Do outlines contain any semi-vertical or semi-horizontal lines? (com.google.fonts/check/outline_semi_vertical)

  • ⚠ WARN The following glyphs have semi-vertical/semi-horizontal lines:

    • AE (U+00C6): L<<1491.0,994.0>--<899.0,999.0>>

    • AEacute (U+01FC): L<<1491.0,994.0>--<899.0,999.0>>

    • Thorn (U+00DE): L<<305.0,1216.0>--<303.0,955.0>>

    • ij (U+0133): L<<544.0,329.0>--<546.0,775.0>>

    • j (U+006A): L<<144.0,329.0>--<146.0,775.0>>

    • jcircumflex (U+0135): L<<144.0,329.0>--<146.0,775.0>>

    • ordfeminine (U+00AA): L<<755.0,1146.0>--<754.0,842.0>>

    • product (U+220F): L<<119.0,-57.0>--<129.0,1093.0>>

    • product (U+220F): L<<760.0,985.0>--<263.0,987.0>>

    • uni01C8 (U+01C8): L<<1035.0,329.0>--<1037.0,775.0>>

    • And 13 more.

Use -F or --full-lists to disable shortening of long lists. [code: found-semi-vertical]

ℹ INFO: Show hinting filesize impact. (com.google.fonts/check/hinting_impact)
  • ℹ INFO Hinting filesize impact:
Dekko-Regular.ttf
Dehinted Size 180.9kb
Hinted Size 291.4kb
Increase 110.5kb
Change 61.1 %
[code: size-impact]
ℹ INFO: EPAR table present in font? (com.google.fonts/check/epar)
ℹ INFO: Is the Grid-fitting and Scan-conversion Procedure ('gasp') table set to optimize rendering? (com.google.fonts/check/gasp)
  • ℹ INFO These are the ppm ranges declared on the gasp table:

PPM <= 65535:
flag = 0x0F
- Use grid-fitting
- Use grayscale rendering
- Use gridfitting with ClearType symmetric smoothing
- Use smoothing along multiple axes with ClearType®
[code: ranges]

ℹ INFO: Familyname must be unique according to namecheck.fontdata.com (com.google.fonts/check/fontdata_namecheck)
ℹ INFO: Check for font-v versioning. (com.google.fonts/check/fontv)
  • ℹ INFO Version string is: "Version 2.002; ttfautohint (v1.8.4.7-5d5b)"
    The version string must ideally include a git commit hash and either a "dev" or a "release" suffix such as in the example below:
    "Version 1.3; git-0d08353-release" [code: bad-format]
ℹ INFO: Font contains all required tables? (com.google.fonts/check/required_tables)

  • ℹ INFO This font contains the following optional tables:

    • cvt

    • fpgm

    • loca

    • prep

    • GPOS

    • GSUB

    • And gasp [code: optional-tables]

ℹ INFO: List all superfamily filepaths (com.google.fonts/check/superfamily/list)
  • ℹ INFO . [code: family-path]

### Summary
💔 ERROR 🔥 FAIL ⚠ WARN 💤 SKIP ℹ INFO 🍞 PASS 🔎 DEBUG
0 1 8 109 8 101 0
0% 0% 4% 48% 4% 44% 0%

Note: The following loglevels were omitted in this report:

  • SKIP
  • PASS
  • DEBUG

@EbenSorkin
Copy link
Collaborator

EbenSorkin commented Jan 18, 2023 via email

@EbenSorkin EbenSorkin merged commit 929c5f2 into SorkinType:master Jan 18, 2023
@emmamarichal emmamarichal mentioned this pull request Feb 28, 2023
Closed
20 tasks
@emmamarichal emmamarichal mentioned this pull request Mar 9, 2023
Open
Sign up for free to 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.
2 participants
@emmamarichal @EbenSorkin