profile
viewpoint
Chris Simpkins chrissimpkins @source-foundry https://github.com/sponsors/chrissimpkins @googlefonts, @rust-lang, @rust-lang RDG Working Group, @source-foundry

chrissimpkins/codeface 5349

Typefaces for source code beautification

all-contributors/all-contributors 4754

✨ Recognize all contributors, not just the ones who push code ✨

chrissimpkins/Crunch 2918

Insane(ly slow but wicked good) PNG image optimization

chrissimpkins/cinder 135

A clean, responsive MkDocs theme

chrissimpkins/crypto 31

Simple symmetric GPG file encryption and decryption

chrissimpkins/chug 21

Simple, clickable Homebrew package management for macOS

chrissimpkins/commandlines 12

A Python command line argument to object parsing library for command line application development

chrissimpkins/dsh 11

A command line search interface to the programming language/framework docsets in the Mac OSX application, Dash. Includes language specific documentation filters.

chrissimpkins/awesome 5

A curated list of awesome lists

chrissimpkins/center-of-mass-diff 1

Analysis of center of mass and area calculations for use in diff of font glyphs between builds

issue commentgooglefonts/gftools

gftools unable to handle folder names with spaces?

I don’t know if related, but I am also getting a permission error. This is running the latest merge of the build script from @vv-monsalve on Mac OS.

= Post procesing TTF
Traceback (most recent call last):
  File "../venv/bin/gftools", line 99, in <module>
    p = subprocess.Popen(args, stdout=sys.stdout,
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/subprocess.py", line 854, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/subprocess.py", line 1702, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
PermissionError: [Errno 13] Permission denied: '../venv/bin/gftools-fix-hinting.py' 

Ugh... @vv-monsalve @tiroj I suggest trying a change of all of the python [path]/gftools [sub-command] calls to python [path]/[gftools script that is called by gftools executable]. These names are formed by replacing the space between gftools and [SUB-COMMAND] with a hyphen and adding a .py. e.g.,

python ../venv/bin/gftools fix-hinting [FONT]

becomes

python ../venv/bin/gftools-fix-hinting.py [FONT]

You'll find the full list of script names in https://github.com/googlefonts/gftools/tree/master/bin

tiroj

comment created time in 4 days

issue commentgooglefonts/gftools

gftools unable to handle folder names with spaces?

Seems related: pypa/virtualenv#53

😆 From that thread. Life in 2020...

2020-10-16_13-39-17

tiroj

comment created time in 4 days

Pull request review commentgooglefonts/fontbakery

Ignore zero advance-width glyphs at com.google.fonts/check/monospace

 def glyph_metrics_stats(ttFont):     ascii_glyph_names = [ttFont.getBestCmap()[c] for c in range(32, 128)                          if c in ttFont.getBestCmap()]     ascii_widths = [adv for name, (adv, lsb) in glyph_metrics.items()-                    if name in ascii_glyph_names]+                    if name in ascii_glyph_names and adv != 0]

I'm not sure that this influences the issue report if I understand the change correctly Felipe. This looks like it is a "is it a monospaced font" side definition change. The fonts that I tested were designed as monospaced fonts. We successfully passed that test and the result was correct. The issue that I found was that when it is a monospaced font, there are glyphs outside of the ASCII set that are detected as inappropriately defined in a monospaced font b/c they have zero advance widths. These were in the combining mark glyph set which are compiled with zero advance widths. Please let me know if I can provide any additional information

felipesanches

comment created time in 5 days

PullRequestReviewEvent

pull request commentchrissimpkins/cinder

Display permalinks on hover

All set to go here after https://github.com/chrissimpkins/cinder/pull/95/commits/92bedf98e1e84a4f51b70c5b45ffb739742bf630 @anlau?

anlau

comment created time in 5 days

issue commentgooglefonts/gftools

gftools unable to handle folder names with spaces?

I can totally reproduce this. Seems to me venv related.

Which venv did you use? A locally generated one? I changed the build following Chris workaround. Under my local venv both, the previous and the current build worked for me locally,

Users should always be creating their venv environment locally. You don't want to commit your venv directory to your git history.

tiroj

comment created time in 5 days

issue commentgooglefonts/gftools

gftools unable to handle folder names with spaces?

Thanks, Chris, for the detailed answer.

in a clone of your repo that includes a venv directory path with spaces

Do you mean a venv included in the repository? I'm curious since there was one, but I deleted it as it is not the suggested practise to include it in the repository.

You can add venv to your .gitignore. It should not be pushed to the remote. But it is ok to include it in repo locally IMO. Not aware of guidance about the practice out there.

It definitely looks like the venv installed executables do not like spaces in directory paths

So, for clarification, the main source of conflict in this case would be the spaces in directory paths. It wouldn't matter if a new venv is created for the directory it would still have those, that's why in this case including the path to ../venv would be the best solution. Is that it?

It appears to be the spaces in the absolute directory path that John was using above the level of the repository directory. Any space in that path. I tested it by adding a space in the repository directory name and, in a separate test, the venv directory name, it caused issues in both cases.

A workaround for absolute paths with spaces is to use explicit relative file paths to the venv installed executable sources.

Thanks, I'll try that and let you know the outcome :)

G/L!

tiroj

comment created time in 5 days

issue commentgooglefonts/gftools

gftools unable to handle folder names with spaces?

I reproduced this with macOS 10.15.5 / cPython 3.8.6 in a clone of your repo that includes a venv directory path with spaces. It definitely looks like the venv installed executables do not like spaces in directory paths. You might check the fontmake that you are using within the venv too. In my activated venv it looks like this is the system installed version of fontmake and not the venv installed version. That could be an important issue if you intend to fix the compiler version in your build workflow.

A workaround for absolute paths with spaces is to use explicit relative file paths to the venv installed executable sources. This works for me and should support anyone who builds from a virtual environment directory that is named venv at the root of your repository:

src/build.sh

#!/bin/sh
set -e

mkdir -p ../fonts/otf ../fonts/ttf

echo = "Generating TTFs"
../venv/bin/fontmake -u ./UFO/Castoro-Regular.ufo -o ttf --output-dir ../fonts/ttf -a
../venv/bin/fontmake -u ./UFO/Castoro-Italic.ufo -o ttf --output-dir ../fonts/ttf -a
# fontmake -m ./UFO/Castoro_Roman.designspace -o ttf --output-dir ../fonts/ttf -a
# fontmake -m ./UFO/Castoro_Italic.designspace -o ttf --output-dir ../fonts/ttf -a

echo = "Post procesing TTF"
ttfs=$(ls ../fonts/ttf/*.ttf)
for ttf in $ttfs
do
    python ../venv/bin/gftools-fix-hinting.py $ttf
    mv "$ttf.fix" $ttf
    python ../venv/bin/gftools-fix-dsig.py -f $ttf;
    python3 castoro_stat_table.py $ttfs
done

echo = "Generating OTFs"
../venv/bin/fontmake -u ./UFO/Castoro-Regular.ufo -o otf --output-dir ../fonts/otf
../venv/bin/fontmake -u ./UFO/Castoro-Italic.ufo -o otf --output-dir ../fonts/otf
# fontmake -m ./UFO/Castoro_Roman.designspace -o otf --output-dir ../fonts/otf
# fontmake -m ./UFO/Castoro_Italic.designspace -o otf --output-dir ../fonts/otf

echo "Post processing static OTFs"
otf=$(ls ../fonts/otf/*.otf)
for otf in $otf
do
	python ../venv/bin/gftools-fix-dsig.py -f $otf;
    psautohint $otf;
done

echo "Done!"
tiroj

comment created time in 6 days

issue commentfigma/community

Variable Font Support

+1

rememberlenny

comment created time in 6 days

pull request commentchrissimpkins/hl

Bump regex from 1.4.0 to 1.4.1

@dependabot merge

dependabot-preview[bot]

comment created time in 7 days

pull request commentchrissimpkins/cinder

Display permalinks on hover

Sounds good to me. Let's go with that approach.

anlau

comment created time in 7 days

pull request commentsource-foundry/fdiff

Bump yarl from 1.6.0 to 1.6.2

@dependabot merge

dependabot[bot]

comment created time in 8 days

pull request commentsource-foundry/fdiff

Bump multidict from 4.7.6 to 5.0.0

@dependabot merge

dependabot[bot]

comment created time in 8 days

pull request commentsource-foundry/fdiff

Bump aiohttp from 3.6.2 to 3.6.3

@dependabot merge

dependabot[bot]

comment created time in 8 days

pull request commentchrissimpkins/cinder

Display permalinks on hover

So, which one should I go for? Style-wise I have a small preference for the symbol.

If it can help, MkDocs default themes use a Font Awesome icon and Material theme a symbol.

On a side note, after this PR is completed we should update MkDocs themes page (update screenshot and add badge: https://img.shields.io/pypi/dm/mkdocs-cinder).

Which one is the symbol? The pilcrow?

anlau

comment created time in 8 days

issue commentgooglefonts/fontbakery

Does Travis / tox use the dependency versions declared at requirements.txt? Or does it install the latest versions of each package instead?

I believe that tox installs from setup.py defs by default. It looks like you can specify a requirements.txt file with the install_command configuration. The most straightforward approach may be to abandon tox in the CI and just use the Travis Python runner env with pip install and pytest commands.

felipesanches

comment created time in 8 days

pull request commentf-actions/font-bakery

Bump jest from 26.5.2 to 26.5.3

@dependabot merge

dependabot[bot]

comment created time in 9 days

pull request commentpy-actions/flake8

Bump jest from 26.5.2 to 26.5.3

@dependabot merge

dependabot[bot]

comment created time in 9 days

pull request commentpy-actions/py-dependency-install

Bump jest from 26.5.2 to 26.5.3

@dependabot merge

dependabot[bot]

comment created time in 9 days

pull request commentchrissimpkins/recurse

Bump structopt from 0.3.18 to 0.3.19

@dependabot merge

dependabot[bot]

comment created time in 9 days

pull request commentchrissimpkins/recurse

Bump regex from 1.3.9 to 1.4.0

@dependabot merge

dependabot[bot]

comment created time in 9 days

pull request commentchrissimpkins/recurse

Bump anyhow from 1.0.32 to 1.0.33

@dependabot merge

dependabot[bot]

comment created time in 9 days

pull request commentchrissimpkins/hl

Bump regex from 1.3.9 to 1.4.0

@dependabot merge

dependabot-preview[bot]

comment created time in 9 days

pull request commentpy-actions/flake8

Bump eslint-plugin-react from 7.21.3 to 7.21.4

@dependabot merge

dependabot-preview[bot]

comment created time in 10 days

pull request commentpy-actions/py-dependency-install

Bump eslint-plugin-react from 7.21.3 to 7.21.4

@dependabot merge

dependabot-preview[bot]

comment created time in 10 days

pull request commentf-actions/font-bakery

Bump eslint-plugin-react from 7.21.3 to 7.21.4

@dependabot merge

dependabot-preview[bot]

comment created time in 10 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 98510c72cbaec69c5a1616ca201fd2aa838d6882

v0.5.0 -> v0.5.1-dev0

view details

Chris Simpkins

commit sha 4dd97869f62d668fd5580e51206e57464015a3fb

remove unnecessary import in test module

view details

push time in 12 days

issue commentsource-foundry/path-inspector

Add segments sub-command

Released in v0.5.0

chrissimpkins

comment created time in 12 days

issue closedsource-foundry/path-inspector

Add segments sub-command

with support for line distance and quadratic curve arc length reporting

closed time in 12 days

chrissimpkins

created tagsource-foundry/path-inspector

tagv0.5.0

TTF font curve path inspector

created time in 12 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha b1a381998dc06511121c5a09c9e3348851929434

fixed segments module unit tests for line endings on win

view details

push time in 12 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 7a11e2cf7aad274e58a01e26f8d6470e1286a257

v0.5.0

view details

push time in 12 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 81dd68769b61c73f7ca32be34fecf51856ae32a1

[stringbuilder] add segment_total_distance function

view details

Chris Simpkins

commit sha c986e98dad8239a3998a59e00f82c8c6888ccd60

add segment_total_distance unit tests

view details

Chris Simpkins

commit sha ce2ada67cec7477d5719bcf9dd9f74759672b011

[segments] refactor distance total string formatting approach

view details

Chris Simpkins

commit sha d19a6c6bc9d23042caae8bd306b55a5320ab10f0

add segments module unit tests

view details

push time in 12 days

pull request commentchrissimpkins/cinder

Display permalinks on hover

@mmatyas any thoughts about the two approaches that Antoine posted above?

anlau

comment created time in 12 days

pull request commentchrissimpkins/cinder

Display permalinks on hover

Configuration is on the user side. Without permalink, headerlink a is not generated. This is the default for a fresh install (mkdocs new myproject). One user can then set permalink to true to use the default permalink character (¶). Or any other character (or a FA icon if it's supported by his theme).

I'm not sure if I prefer the icon or of the symbol.

This approach sounds good to me. Willing to merge this.

anlau

comment created time in 12 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 1774ae657bbd94ec5222b2390e6ab54e6cd7b5ef

[stringbuilder] fix spacing in segment strings removes extra space between words

view details

Chris Simpkins

commit sha deac5fbf49b19f3a58c9acdec517f952ed711b96

add segment_quadratic_curve unit tests

view details

push time in 12 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha b208a363f1b29724b25e0228c5173cda93164c59

add segment_line unit tests

view details

push time in 12 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 3cdc04ca67ce0728755c9877df4eed78af7e787f

remove profiler from VC

view details

Chris Simpkins

commit sha f954302d1b03420c0f211b113abe382431e9a15d

[.gitignore] add profiler.py

view details

push time in 12 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 1bbb5096e15981d12346932f5be54a0c4edbbf3d

[stringbuilder] segment_line and segment_quadratic_curve use color by default

view details

push time in 12 days

pull request commentchrissimpkins/cinder

Change "overflow-y" to "auto" for sidebar

Released in v1.0.6. Thank you very much for these changes!

anlau

comment created time in 12 days

push eventsf-web/sf-web.github.io

Chris Simpkins

commit sha 258e461cdc7439ced34448b8957ca35511d1f814

Cinder v1.0.6

view details

push time in 12 days

release chrissimpkins/cinder

v1.0.6

released time in 12 days

created tagchrissimpkins/cinder

tagv1.0.6

A clean, responsive MkDocs theme

created time in 12 days

push eventchrissimpkins/cinder

Chris Simpkins

commit sha fd07b001da8def16064c886430d442873f25dd00

v1.0.6

view details

push time in 12 days

push eventchrissimpkins/cinder

Chris Simpkins

commit sha 4289b4146e0508fc2cfa018b88eb95e163f4d73d

minified CSS with fix in b0c2fd742b7a75db1b862ec7aad4b2ae8ff100d7

view details

push time in 12 days

startedmozilla/uniffi-rs

started time in 12 days

startedPomax/Font.js

started time in 13 days

pull request commentchrissimpkins/cinder

Display permalinks on hover

My 2 cents is that the link icon more intuitively indicates that this is a link to the header. I'm not opposed to the pilcrow symbol. It is likely used frequently enough out there for this purpose.

As I mentioned before, I would really prefer not to show an anchor icon by default and force users to explicitly edit configuration files to use something else. If the markdown_extensions > toc > permalink setting is not present in a user configuration file, does this mean that they get no permalinks? Or is the new behavior always on hover permalinks that can be modified to a different set of icons if you specify the permalink setting (True or icon name) in the yaml config file?

anlau

comment created time in 13 days

push eventchrissimpkins/cpython

Miss Skeleton (bot)

commit sha 5b1fdcacfc36dfe36693f4c350c958df6600de14

bpo-41376: Fix the documentation of `site.getusersitepackages()` (GH-21602) `site.getusersitepackages()` returns the location of the user-specific site-packages directory even when the user-specific site-packages is disabled. ``` $ python -s -m site sys.path = [ '/home/user/conda/lib/python37.zip', '/home/user/conda/lib/python3.7', '/home/user/conda/lib/python3.7/lib-dynload', '/home/user/conda/lib/python3.7/site-packages', ] USER_BASE: '/home/user/.local' (exists) USER_SITE: '/home/user/.local/lib/python3.7/site-packages' (doesn't exist) ENABLE_USER_SITE: False ``` It was not practical to prevent the function from returning None if user-specific site-packages are disabled, since there are other uses of the function which are relying on this behaviour (e.g. `python -m site`). (cherry picked from commit 35f041dd0171f575fc3adce1709b31fdf45a5ff6) Co-authored-by: Phil Elson <pelson.pub@gmail.com>

view details

push time in 13 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 7a470409fa00a09c16322f58f5853dde182cb1e3

add stringbuilder tests

view details

push time in 13 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 3b8c3b2f3d8e0d1ab743c9aaad791bfcf1b8dedf

add cross-platform newline support in tests

view details

push time in 13 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 656915ba869a2fe5d5e3af6261d4f1731fa39b0e

[stringbuilder] comment out, keep overlap code

view details

Chris Simpkins

commit sha a975aaef51d307b971eb3a82e63dc8ab0bee1ff5

initial batch of stringbuilder unit tests

view details

push time in 13 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 830d5c29c204b7b5ffa6c0f785264e0a2a4bc884

put overlap source on ice needs work

view details

push time in 13 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 7fb8b59926f99f4ed449e4301f9ada89dfc13bb3

remove unused import statement

view details

push time in 13 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha c011e580db89d4b17aeed17bf212de250a15fce5

[stringbuilder] performance: remove std.stdout.isatty call in loop based on profiling of segments sub-command

view details

Chris Simpkins

commit sha a947e62c6ca65890d5414ffa3684afc3d73e87d5

update unit tests mock for new approach to TTY testing for ANSI color use

view details

push time in 13 days

pull request commentpy-actions/py-dependency-install

Bump jest from 26.5.0 to 26.5.2

@dependabot merge

dependabot[bot]

comment created time in 14 days

pull request commentf-actions/font-bakery

Bump jest from 26.5.0 to 26.5.2

@dependabot merge

dependabot[bot]

comment created time in 14 days

pull request commentpy-actions/flake8

Bump jest from 26.5.0 to 26.5.2

@dependabot merge

dependabot[bot]

comment created time in 14 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha b80aed5b7c92709701fb7840d9be9aefdbed8dea

add math module unit tests

view details

push time in 14 days

push eventchrissimpkins/cpython

Miss Islington (bot)

commit sha e3ce3bba9277a7c4cfde5aaf6269b6c68f334176

bpo-40847: Consider a line with only a LINECONT a blank line (GH-20769) A line with only a line continuation character should be considered a blank line at tokenizer level so that only a single NEWLINE token gets emitted. The old parser was working around the issue, but the new parser threw a `SyntaxError` for valid input. For example, an empty line following a line continuation character was interpreted as a `SyntaxError`. Co-authored-by: Pablo Galindo <Pablogsal@gmail.com> (cherry picked from commit 896f4cf63f9ab93e30572d879a5719d5aa2499fb) Co-authored-by: Lysandros Nikolaou <lisandrosnik@gmail.com>

view details

Miss Islington (bot)

commit sha f6428babb1b9172cc01145d4c091b91dc601d969

Collections module reformatting and minor code refactoring (GH-20772) (GH-20798)

view details

Miss Islington (bot)

commit sha 961edf7979ca34d6fe104a1cce005aa8cac35821

bpo-40939: Generate keyword.py using the new parser (GH-20800) (cherry picked from commit 9727694f08cad4b019d2939224e3416312b1c0e1) Co-authored-by: Lysandros Nikolaou <lisandrosnik@gmail.com>

view details

Miss Islington (bot)

commit sha 2f46c65071c5528ce143bd02786f8fe94fe65d1f

bpo-34401: Fix test_gdb for HP GDB version string (GH-20804) The GDB provided by HPE on HP-UX contains a modified version string. Therefore the tests fail. Adapt the regex to match that string. Patch by Michael Osipov. Co-Authored-by: Michael Osipov <michael.osipov@siemens.com> (cherry picked from commit b2dca49ca3769cb60713f5c2b43e5d5bbdc1f9c7) Co-authored-by: Victor Stinner <vstinner@python.org>

view details

Victor Stinner

commit sha dc40105c88b968a50c3458e10e1d732e957ef0a3

bpo-40947: getpath.c uses PyConfig.platlibdir (GH-20807) (GH-20813) Followup of bpo-40854, there is one remaining usage of PLATLIBDIR which should be replaced by config->platlibdir. test_sys checks that sys.platlibdir attribute exists and is a string. Update Makefile: getpath.c and sysmodule.c no longer need PLATLIBDIR macro, PyConfig.platlibdir member is used instead. Co-authored-by: Sandro Mani <manisandro@gmail.com> (cherry picked from commit d72b9644a3e6eec83be48b1ebc2ec6ca776134d3)

view details

Pablo Galindo

commit sha 3782497cc22e70b41e32ac09cb06d3948074d8a7

[3.9] bpo-40939: Fix test_keyword for the old parser (GH-20814)

view details

Miss Islington (bot)

commit sha 2c0a59bb3d2fb83312c965ea9d5ef9d0028a53bb

Update lexical_analysis.rst (GH-17508) Use Sphinx role markup for `str.format`. Automerge-Triggered-By: @csabella (cherry picked from commit e2fb8a2c42ee60c72a40d93da69e9efc4e359023) Co-authored-by: Géry Ogam <gery.ogam@gmail.com>

view details

Miss Islington (bot)

commit sha ee3f7fee94915ab4f34c92d6c89b211de476660d

bpo-40955: Fix memory leak in subprocess module (GH-20825) ``` Direct leak of 8 byte(s) in 1 object(s) allocated from: GH-0 0x7f008bf19667 in __interceptor_malloc (/lib64/libasan.so.6+0xb0667) GH-1 0x7f007a0bee4a in subprocess_fork_exec /home/heimes/dev/python/cpython/Modules/_posixsubprocess.c:774 GH-2 0xe0305b in cfunction_call Objects/methodobject.c:546 ``` Signed-off-by: Christian Heimes <christian@python.org> (cherry picked from commit 0d3350daa8123a3e16d4a534b6e873eb12c10d7c) Co-authored-by: Christian Heimes <christian@python.org>

view details

Christian Heimes

commit sha f91917908749cb595113f8d8428ae1d6241f0392

bpo-40964: disable remote IMAP tests (GH-20836) Remote host cyrus.andrew.cmu.edu is blocking incoming connections and is causing test suite to fail. Signed-off-by: Christian Heimes <christian@python.org>

view details

Miss Islington (bot)

commit sha 166d7234b5ae07f78feb5ddfb3026fbd2a1a36e2

bpo-40164: Update Windows OpenSSL to 1.1.1g (GH-20834) (cherry picked from commit 80d827c3cb041ae72b9b0572981c50bdd1fe2cab) Co-authored-by: Srinivas Reddy Thatiparthy (శ్రీనివాస్ రెడ్డి తాటిపర్తి) <thatiparthysreenivas@gmail.com>

view details

Miss Islington (bot)

commit sha 94bb4b7db2a4eadd299acede204b087aca30c9a2

bpo-40834: Fix truncate when sending str object with channel (GH-20555) (cherry picked from commit 29c117202e386bad1d66ae336e2fefa1a1809ee0) Co-authored-by: An Long <aisk@users.noreply.github.com>

view details

Miss Islington (bot)

commit sha c932f5c1e531f5a3cbf0ce664e4bcd22aa04994d

bpo-40957: Fix refleak in _Py_fopen_obj() (GH-20827) Signed-off-by: Christian Heimes <christian@python.org> (cherry picked from commit 9672912e8f90374fd31b37ca0fb7cefbc6f4c555) Co-authored-by: Christian Heimes <christian@python.org>

view details

Miss Islington (bot)

commit sha f1534d084085033e3cb3d9bd6e4ac42b0b23b21d

Minor code clean-ups (GH-20838) (GH-20858)

view details

Miss Islington (bot)

commit sha bda4cc82de06490d982e199e5e49aaed390461ba

[3.9] bpo-37674: Tweak imp module deprecation note in the docs (GH-20480) (GH-20859) (cherry picked from commit dea3223740127ac13f984c1d38f127ab6701af44) Co-authored-by: Zackery Spytz <zspytz@gmail.com> Automerge-Triggered-By: @brettcannon

view details

Miss Islington (bot)

commit sha 55c1d21761e2e5feda5665065ea9e2280fa76113

bpo-40855: Fix ignored mu and xbar parameters (GH-20835) (#GH-20862)

view details

Miss Islington (bot)

commit sha 37c9fc5baa29d28c6809807b8f06444126760928

Some reformatting (suggested by Black) and minor factoring. (GH-20865) (GH-20866)

view details

Miss Islington (bot)

commit sha 83cd968f5cc59ad60b665080087f50fefac17a07

bpo-30064: Fix slow asyncio sock test (GH-20868) Using a log2n way to fill a much smaller buffer, and receiving in a cleaner way with EOF. The failing test was reproducible using the following command thanks to @aeros : ```bash ./python -m test test_asyncio.test_sock_lowlevel --match test_sock_client_racing -j100 -F -v ``` According to test results, we may still need to bump the timeout: https://github.com/python/cpython/blob/5aad027db9618f22f6fa2274e05dd50f928d2ed7/Lib/test/test_asyncio/test_sock_lowlevel.pyGH-L256-L257 (cherry picked from commit 8f04a84755babe516ebb5304904ea7c15b865c80) Co-authored-by: Fantix King <fantix.king@gmail.com>

view details

Jason R. Coombs

commit sha 8a3469047c3c7b68f434ed244ef3ae4292dd8cbc

[3.9] bpo-40924: Remove protocol for supplying Traversable objects from loaders (GH-20820) Remove protocol that loaders can present a files method as found in importlib_resources 1.3-1.6 (added in Python 3.8b1).

view details

Pablo Galindo

commit sha 145fa86598b850e9ee0efed413dec9ec5e4431b1

[3.9] Include soft keywords in keyword.py (GH-20877). (GH-20880) (cherry picked from commit 78319e373d57cd4da67660f888aa7092efbd6f24) Co-authored-by: Pablo Galindo <Pablogsal@gmail.com>

view details

Pablo Galindo

commit sha 30b59fd7cf946f7163283eef7d4389bd4e582492

[3.9] Improve readability and style in parser files (GH-20884) (GH-20885) (cherry picked from commit fb61c42) Co-authored-by: Pablo Galindo <Pablogsal@gmail.com>

view details

push time in 14 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 2b21f22653494a0725a0202da8583b771a9869a5

[datastructures] add Coordinate object equality testing methods

view details

Chris Simpkins

commit sha 9596a2fbbcf616b6f6bb3c581940cb875a93582c

add datastructures unit tests

view details

push time in 14 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 8d3652d0e0a2ee83bc10850d9bd14d4a32cb83ac

[segments] fmt

view details

push time in 14 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha de8e6cd87bb819957dcd4ed1eab3c61e647f2e68

[Makefile] set line length = 90 on import statement formatting target

view details

push time in 14 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 7f88d3ac1c40da0e857198317db2ee6530d44b3a

[bezier] add docstring

view details

Chris Simpkins

commit sha f5dc731fbcb04cad5c15a438ab7fcfe8a20d82f1

add bezier module unit tests

view details

push time in 14 days

delete branch googlefonts/fontelemetry

delete branch : dependabot/pip/notebook-5.7.8

delete time in 15 days

push eventgooglefonts/fontelemetry

dependabot[bot]

commit sha fd57bdbc3a2ed918800f0de91f50d296376aa185

Bump notebook from 5.7.0 to 5.7.8 Bumps [notebook](https://github.com/jupyter/jupyterhub) from 5.7.0 to 5.7.8. - [Release notes](https://github.com/jupyter/jupyterhub/releases) - [Changelog](https://github.com/jupyterhub/jupyterhub/blob/master/CHECKLIST-Release.md) - [Commits](https://github.com/jupyter/jupyterhub/commits) Signed-off-by: dependabot[bot] <support@github.com>

view details

Chris Simpkins

commit sha b4d77d1beebd94b55e5fc5545fb1d7ea33c29d0b

Merge pull request #22 from googlefonts/dependabot/pip/notebook-5.7.8 Bump notebook from 5.7.0 to 5.7.8

view details

push time in 15 days

PR merged googlefonts/fontelemetry

Bump notebook from 5.7.0 to 5.7.8 dependencies

Bumps notebook from 5.7.0 to 5.7.8. <details> <summary>Commits</summary> <ul> <li>See full diff in <a href="https://github.com/jupyter/jupyterhub/commits">compare view</a></li> </ul> </details> <br />

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


<details> <summary>Dependabot commands and options</summary> <br />

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the Security Alerts page.

</details>

+2 -2

0 comment

2 changed files

dependabot[bot]

pr closed time in 15 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 410daf38244087b1dbe24012b4e92a0e73079625

import fmt test files

view details

push time in 15 days

pull request commentsource-foundry/font-line

Bump fonttools from 4.16.0 to 4.16.1

@dependabot merge

dependabot[bot]

comment created time in 15 days

pull request commentsource-foundry/ufolint

Bump fonttools[ufo] from 4.16.0 to 4.16.1

@dependabot merge

dependabot[bot]

comment created time in 15 days

pull request commentsource-foundry/fdiff

Bump fonttools from 4.16.0 to 4.16.1

@dependabot merge

dependabot[bot]

comment created time in 15 days

pull request commentsource-foundry/font-v

Bump fonttools from 4.16.0 to 4.16.1

@dependabot merge

dependabot[bot]

comment created time in 15 days

pull request commentsource-foundry/dehinter

Bump fonttools from 4.16.0 to 4.16.1

@dependabot merge

dependabot[bot]

comment created time in 15 days

pull request commentsource-foundry/path-inspector

Bump fonttools from 4.16.0 to 4.16.1

@dependabot merge

dependabot[bot]

comment created time in 15 days

pull request commentf-actions/cldr-coverage

Bump fonttools from 4.16.0 to 4.16.1

@dependabot merge

dependabot[bot]

comment created time in 15 days

issue commentsource-foundry/path-inspector

Add segments sub-command

added total distance summary as of 4a539e0d69785cf7adc43285cd3e490ed122eb83

chrissimpkins

comment created time in 15 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 073d538f5c0b55a9b711e598ebc81cd84657d9cc

[stringbuilder] add bold_text function

view details

Chris Simpkins

commit sha 0e54b4bffbf7b2c5ed9ccde9bfe3eda1e82be976

[segments] bold "Total" in summary line

view details

push time in 15 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 4a539e0d69785cf7adc43285cd3e490ed122eb83

[segments] add total distance summaries

view details

push time in 15 days

issue commentsource-foundry/path-inspector

Add segments sub-command

initial implementation as of 5babd06080b96831bab1328e7ac70767bcb1d134

2020-10-05_23-34-13

chrissimpkins

comment created time in 15 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha b32b1003d165979ba25f028509337137d6aa2ff8

add bezier module

view details

Chris Simpkins

commit sha ee0829a09b03f7b61abdfd9baf10e761ada7f16a

add datastructures module

view details

Chris Simpkins

commit sha ababaa06eb00497b8a1716878d23cc75ff4603f0

[Makefile] add pretty toggle to mypy testing

view details

Chris Simpkins

commit sha 9891b2e0a4df653880294f7e4720aa773412b956

[math] add linear distance between coords function

view details

Chris Simpkins

commit sha 5babd06080b96831bab1328e7ac70767bcb1d134

initial implementation of segments sub-command

view details

Chris Simpkins

commit sha 7c42e027d07115fdbcf7f35cb5094c9dc8c568c3

import statement formatting isort default

view details

push time in 15 days

issue openedsource-foundry/path-inspector

Add segments sub-command

with support for line distance and quadratic curve arc length reporting

created time in 15 days

pull request commentpy-actions/flake8

Bump jest from 26.4.2 to 26.5.0

@dependabot merge

dependabot-preview[bot]

comment created time in 15 days

issue openedgooglefonts/fontbakery

Possibly incorrect WARN on monospaced build width consistency - some marks have zero advance width definitions

Glyphs source file -> static instance TTF font build with fontmake v2.2.1 pipeline. This is a weight axis monospaced typeface.

Observed behaviour

fontbakery raises a WARN level report that the *comb glyphs in our glyph set have inconsistent widths for a monospaced typeface.

The report is:

* WARN: Font is monospaced but 9 glyphs (3.91304347826087%) have a different width. You should check the widths of: ['uni0308', 'gravecomb', 'acutecomb', 'uni0302', 'uni030C', 'uni030A', 'tildecomb', 'uni0304', 'uni0327'] [code: mono-outliers]

and this appears to come from this block of the fontbakery source:

https://github.com/googlefonts/fontbakery/blob/42e872c92947569f5972a76bb52175e95666ed08/Lib/fontbakery/profiles/shared_conditions.py#L152-L173

I reviewed the Glyphs source files and the advance width for all of the reported glyphs is set at the expected value for the typeface (600).

Here is what I see in the binaries:

>>> for glyphname, metrics in glyph_metrics.items():
...     if metrics[0] != 600:
...         print(f"{glyphname}: {metrics}")
...
uni0308: (0, 140)
gravecomb: (0, 213)
acutecomb: (0, 213)
uni0302: (0, 146)
uni030C: (0, 146)
uni030A: (0, 190)
tildecomb: (0, 126)
uni0304: (0, 161)
uni0327: (0, 200)

And a visual of dieresiscomb:

2020-10-05_18-07-38

It looks like a subset of the marks are compiled with a zero advance width value and should be excluded from this check.

Expected behaviour

This is not reported as a warning for monospaced typefaces if these metrics are valid.

created time in 15 days

pull request commentpy-actions/py-dependency-install

Bump jest from 26.4.2 to 26.5.0

@dependabot merge

dependabot-preview[bot]

comment created time in 15 days

pull request commentf-actions/font-bakery

Bump jest from 26.4.2 to 26.5.0

@dependabot merge

dependabot-preview[bot]

comment created time in 15 days

pull request commentchrissimpkins/cinder

Display permalinks on hover

Is it possible to change the default that is used in this project without the yml configuration edit by a user? Do you know if we can add it to the https://github.com/chrissimpkins/cinder/blob/master/cinder/mkdocs_theme.yml?

anlau

comment created time in 16 days

push eventchrissimpkins/cinder

Antoine Lauzon

commit sha b0c2fd742b7a75db1b862ec7aad4b2ae8ff100d7

Change "overflow-y" to "auto" for sidebar Also modified height so it doesn't overlap footer.

view details

Chris Simpkins

commit sha 885e51e661dd878c2ba8050c640a050640401e9f

Merge pull request #96 from anlau/sidebar-overflow-auto Change "overflow-y" to "auto" for sidebar

view details

push time in 16 days

PR merged chrissimpkins/cinder

Change "overflow-y" to "auto" for sidebar

With overflow "auto", scrollbar is not shown if not needed. Otherwise I think it's equivalent to "scroll".

Also modified height so it doesn't overlap footer.

+2 -2

2 comments

1 changed file

anlau

pr closed time in 16 days

pull request commentchrissimpkins/cinder

Change "overflow-y" to "auto" for sidebar

Let's go with this approach until someone complains... :)

I'll push a new release this week.

anlau

comment created time in 16 days

pull request commentf-actions/font-bakery

Bump eslint-plugin-react from 7.21.2 to 7.21.3

@dependabot merge

dependabot[bot]

comment created time in 16 days

pull request commentpy-actions/py-dependency-install

Bump eslint-plugin-react from 7.21.2 to 7.21.3

@dependabot merge

dependabot[bot]

comment created time in 16 days

pull request commentpy-actions/flake8

Bump eslint-plugin-react from 7.21.2 to 7.21.3

@dependabot merge

dependabot[bot]

comment created time in 16 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha ca668132062843ed741f63f6bc7137206ad3e394

fix: do not use ANSI escape code color when not tty, cyan_text, cyan_bright_text, green_text, red_text

view details

push time in 16 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha bc2967573ea98208af4573d7c0666c53b5cad482

[math] formatting

view details

Chris Simpkins

commit sha cabc7ce37632241b1f190103200e7431ad19905e

[coordinates] remove unncessary coordinates_report function parameter

view details

push time in 16 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 762ac5e04405c8008785ebf8531bcf0eca9ba81c

add math module with cartesian coordinate midpoint and OT spec flavored rounding functions

view details

push time in 17 days

issue closedsource-foundry/path-inspector

Add contours sub-command

with contour count reporting

closed time in 17 days

chrissimpkins

issue commentsource-foundry/path-inspector

Add contours sub-command

released in v0.4.0

chrissimpkins

comment created time in 17 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 65a6bd6e80aad9a870db9b81db9da9a789bd138f

[direction] add documentation comment

view details

push time in 17 days

push eventsource-foundry/path-inspector

Chris Simpkins

commit sha 29a959adfbb27e875e31611b0fb1303d7fbcd798

v0.4.0 -> v0.4.1-dev0

view details

push time in 17 days

created tagsource-foundry/path-inspector

tagv0.4.0

TTF font curve path inspector

created time in 18 days

more