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.

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

Magic spaces #1521

Closed
rpspringuel opened this issue Feb 10, 2021 · 5 comments · Fixed by #1526
Closed

Magic spaces #1521

rpspringuel opened this issue Feb 10, 2021 · 5 comments · Fixed by #1526
Assignees

Comments

@rpspringuel
Copy link
Contributor

rpspringuel commented Feb 10, 2021

While working on #1491 I found a magic space and while looking to fix that, I've found a couple of more:

gregoriotex-signs.tex:1213: \gre@dimen@temp@three=\dimexpr(3700sp * \gre@factor)\relax %
gregoriotex-signs.tex:1215: \gre@dimen@temp@three=\dimexpr(4500sp * \gre@factor)\relax%
gregoriotex-signs.tex:1218: \gre@dimen@temp@three=\dimexpr(2500sp * \gre@factor)\relax %
gregoriotex-spaces.tex:173: \global\gre@dimen@stafflineheight=\dimexpr(1500 sp * \gre@stafflinefactor)\relax %
gregoriotex-spaces.tex:182: \global\gre@dimen@interstafflinespace=\dimexpr((31500sp * \gre@factor) %
gregoriotex-spaces.tex:220: \global\gre@dimen@constantglyphraise = \dimexpr((-22000 sp * \gre@factor) %
gregoriotex-spaces.tex:815: \global\gre@dimen@glyphraisevalue = \dimexpr(15750 sp * \gre@factor * \gre@count@temp@three)\relax %
gregoriotex-spaces.tex:999: \global\gre@dimen@additionaltopspace=\dimexpr(15750 sp * \gre@count@temp@one * \gre@factor)\relax %
gregoriotex-spaces.tex:1006: \global\gre@dimen@additionaltopspacealt=\dimexpr(15750 sp * \gre@count@temp@one * \gre@factor)\relax %
gregoriotex-spaces.tex:1013: \global\gre@dimen@additionaltopspacenabc=\dimexpr(15750 sp * \gre@count@temp@one * \gre@factor)\relax %

The ones in gregoriotex-spaces.tex all appear to be related to the staff lines (the distance between them, their thickness, the vertical distance between note positions, etc.). The ones in gregoriotex-signs.tex seem to be related to positioning the mora after a punctum inclinatum.

Should I convert all these to configurable dimensions, or should they remain as hard-coded internal values? In either case, I'm going to try to work out the sensical relations between the numbers and store them in registers so that they can be referred to by name.

@henryso
Copy link
Contributor

henryso commented Feb 10, 2021

I think some of these, at least, are properties of assumptions on the font size. In my opinion, those should not be configured via gsp-default. Font related spaces should be modified along with selecting such a font. I don't think we are in a good position to support this generically yet. Work has already been done to support the Solesmes fonts which is the IP of the monastery, and such a mechanism should probably build on that.

@rpspringuel
Copy link
Contributor Author

That makes sense. Is the existing work to support the Solesmes fonts in any of the repositories?

@henryso
Copy link
Contributor

henryso commented Feb 11, 2021

Yes, the work was already integrated by Elie. Basically, the Solesmes fonts have a different scale than the gregorio fonts. The adjustments to make everything sane are already in, at least enough to support the Solesmes fonts.

@rpspringuel
Copy link
Contributor Author

Okay, then what I'm going to do is name the magic distances, put their definitions near the font selection code, and identify the distances as "font calibration distances." That should make it clear what they're for and put them into the place where someone who is working on making a new font compatible with GregorioTeX is likely to look for them.

@henryso
Copy link
Contributor

henryso commented Feb 12, 2021

Sounds like a plan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants