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

Greciliae too low? #858

Closed
eroux opened this issue Feb 1, 2016 · 10 comments
Closed

Greciliae too low? #858

eroux opened this issue Feb 1, 2016 · 10 comments

Comments

@eroux
Copy link
Contributor

eroux commented Feb 1, 2016

Now that I'm taking a look at greciliae after modifying another font, there is something I'm really puzzled by: I think greciliae is way too low! It's partiularly obvious on porrectus: when one is used to another font, it's hard to read what not a porrectus is in greciliae... Comparing with the original caeciliae, it's quite obvious to me that there's been a mistake during the import... Before I'm making a few attempts at fixing it (after finishing #803 of course), I'd like to know what you think?

@henryso
Copy link
Contributor

henryso commented Feb 1, 2016

Can you post some images of greciliae vs caeciliae? I'm not entirely sure what you mean by "low".

@eroux
Copy link
Contributor Author

eroux commented Feb 1, 2016

Here is a porrectus in caeciliae:

porrectus-caeciliae

and the same in greciliae:

porrectus-greciliae

in caeciliae, the staff line is around the middle of the left of the stroke, while is greciliae, it's around the third...

@henryso
Copy link
Contributor

henryso commented Feb 1, 2016

Thank you. In that case, I agree, but some people may be used to greciliae...

@eroux
Copy link
Contributor Author

eroux commented Feb 1, 2016

Yes... How do you think we should proceed? Should we ask the user list? Maybe propositing between the two options:

  • just change greciliae
  • reimport caeciliae and rename the old one greciliae-old

?

@henryso
Copy link
Contributor

henryso commented Feb 2, 2016

If we are going to only package one font and make the rest available for download, I think it would be best to have "greciliae-old", "gregorio", "parmesan", and all the "-op" fonts be available that way, with "greciliae" (new) being the one that is distributed.

If we are not going to do that, then I would announce this to the list in order to help us decide whether we just replace greciliae or distribute both.

I am open to any other ideas, as well.

@eroux
Copy link
Contributor Author

eroux commented Feb 2, 2016

Making a few calculations, the caeciliae glyphs should be scaled by 1.26 and then shifted down by 391.42. I don't know what calculations I did during initial import, but it clearly wasn't this one... Maybe what's best is to finish #803 and then I'll work a few hours on this new import, make a sample pdf with simple glyphs, and propose the change to the list... What do you think? Unless someone would like to give it a try? (it's a bit repetitive, but it's also a bit fun!) @eschwab or @rpspringuel would you be interested and have a little time (I hope the merge for #803 can be tonight or tomorrow)?

@eroux
Copy link
Contributor Author

eroux commented Feb 3, 2016

I've made an attempt and sent it to the list, see this thread.

eroux added a commit that referenced this issue Feb 4, 2016
@eroux
Copy link
Contributor Author

eroux commented Feb 4, 2016

According to the only answer on the list, I guess it's no big deal to just change it... See branch fix-858 for an attempt, let's just wait a little more to see if other answers come.

@henryso
Copy link
Contributor

henryso commented Feb 4, 2016

I guess they answered you directly, because I haven't seen anything on the list. But I agree that we should wait a bit longer.

@eroux
Copy link
Contributor Author

eroux commented Feb 4, 2016

I didn't notice, but you're right, the answer was for me alone. There was only one answer, positive.

@henryso henryso closed this as completed Feb 7, 2016
rpspringuel added a commit to rpspringuel/gregorio that referenced this issue Feb 7, 2016
…miktex

* commit '63fffd12247217b4cd9ff203c41b406ba2c1f3dd':
  update doc
  more final adjustments (mainly reverting pes changes)
  final small adjustments
  fix documentation
  document upgrade
  document GrePunctumMora, fix bug
  more options, tune spaces
  make it actually work
  lower second note of porretus with ambitus one (closer to the Graduale)
  first attempt
  more adjustments
  more adaptations, mostly revert pes ambitus one
  document
  attempt to fix gregorio-project#858
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants