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

Merge upstream changes and fix DependencyControl feed #4

Open
wants to merge 19 commits into
base: master
Choose a base branch
from

Conversation

seproDev
Copy link

This PR aims to cleanup the currently quite messy DependencyControl situation in this repo. This includes the following changes:

  • Merges upstream changes from unanimated/luaegisub
  • Merges NecrosCopy: Assign mark before using it unanimated/luaegisub#15
  • Fixes script_namespace of FadeWorkS and Colourise which caused the macros to show up twice in DependencyControl interfaces after installing
  • Merges Move to ttools repo #1, which will prevent some invalid hash issues. Currently the feed from this repo is in DependencyControl, but the files get downloaded from unanimated/luaegisub. This currently causes a hash mismatch when installing Relocator.
  • Remove UTF-8 BOM in depctrl feed (invalid JSON per specification)
  • Bump minor version of all macros and update hashes in depctrl feed

petzku and others added 19 commits August 22, 2020 01:16
Relocator: Support for subpixel clips (4.5.1)
Relocator: Fix hash in DependencyControl.json
Following user defined rounding rules for vector clips
`koko_da` determines the \ko duration by dividing the "Fade in" value by
the number of word characters and punctuation if the "Fade in" value is
at least 40.

`len` contains the number of word characters and punctuation, but 1 is
erroneously subtracted from it before the division, resulting in
durations that are too long to fit the "Fade in" duration. The fix is
to remove the subtraction and simply divide by `len`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants