jaakkopasanen / AutoEq

Automatic headphone equalization from frequency responses
MIT License
12.87k stars 2.47k forks source link

Some entries of INDEX.md contain wrong case making reconstructed URL not accessible (404) #740

Closed dmitrykos closed 7 months ago

dmitrykos commented 7 months ago

Some entries in results/INDEX.md contain entries with wrong case, for example:

https://github.com/jaakkopasanen/AutoEq/blob/0fad4b2ce4b3ceb97f0086c3a0940b4c2473c1f3/results/INDEX.md?plain=1#L309

here problem is in Soundcore where core starts with low case letter.

When INDEX.md is parsed the reconstructed URL gives 404: Not Found: https://raw.githubusercontent.com/jaakkopasanen/AutoEq/master/results/Rtings/in-ear/Anker%20Soundcore%20Liberty%204%20NC/Anker%20Soundcore%20Liberty%204%20NC%20ParametricEQ.txt

Working URL can be obtained if going through the Web interface for the same preset: https://raw.githubusercontent.com/jaakkopasanen/AutoEq/master/results/Rtings/in-ear/Anker%20SoundCore%20Liberty%204%20NC/Anker%20SoundCore%20Liberty%204%20NC%20ParametricEQ.txt

Note that working URL has SoundCore where Core starts with capital letter.

jaakkopasanen commented 7 months ago

Window's case insensitive paths are the bane of my existence! Thanks for reporting, will fix.

jaakkopasanen commented 7 months ago

These should be fixed now.

dmitrykos commented 7 months ago

@jaakkopasanen thank you for kindly fixing the issue! Reparsed the database after your fix and there are no more such issues, except for the one which is nor related to this issue and it also gives 404 in the web interface: https://github.com/jaakkopasanen/AutoEq/blob/master/results/crinacle/711%20in-ear/Samsung%20Galaxy%20Buds2%20Pro%20(passive%20mode