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

sets.json: File is 'dark720p_xxx' not 'dark70p_xxx' #113

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xiphmont
Copy link
Contributor

@xiphmont xiphmont commented Nov 4, 2021

Looks like this typo has always been there and folks were either renaming the file in the set, or the file in the set was misnamed back then too. These days, the test sets up for download at xiph have the correct filename, and this caused a bit of a headscratcher for a while (because typos are hard to spot, especially when there's a second bug there that looks identical).

@xiphmont
Copy link
Contributor Author

xiphmont commented Nov 5, 2021

BTW-- the merge and revert were because of a github misfire on my part; I hadn't intended to override review and merge unilaterally.

Copy link
Collaborator

@vibhoothi vibhoothi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given that lot of old runs are obselote/not there in beta instance it makes sense to have this change IMHO, nevertheless I want to hear views from @tdaede before landing this

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.

None yet

2 participants