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

Higher Quality Music #120

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

Higher Quality Music #120

wants to merge 17 commits into from

Conversation

vt-idiot
Copy link

@vt-idiot vt-idiot commented Jun 2, 2024

I replaced several tracks with much higher quality (and full length - don't know if that will be an issue) versions, but I don't think I'll be able to help much more with the Trammell Starks tracks. TWCClassics has the ~2 minute versions (or shorter) that TWC used, and the names they used for all of them are arbitrary. I was only able to match two tracks.

Check out the audio section for clips of every Trammell song. Please note that Trammell did not name any of these songs. I came up with my own titles in the late 90s.

https://twcclassics.com/information/trammell-starks.html

They are however likely all available...

https://archive.org/details/lfset1/
https://archive.org/details/lfset2/
https://archive.org/details/lfset3/

@vt-idiot
Copy link
Author

vt-idiot commented Jun 2, 2024

If anyone knows how to (locally) fingerprint music and then try and match the two sets of tracks I could probably replace all of them 👀

lol this worked https://www.codeproject.com/Articles/206507/Duplicates-detector-via-audio-fingerprinting

hang on, more commits

@vt-idiot
Copy link
Author

vt-idiot commented Jun 3, 2024

All done. Ish. There were some tracks I could not find.

@vbguyny
Copy link
Owner

vbguyny commented Jun 3, 2024

Awesome 😎 I'll need to find some time to pull your changes locally and test before merging your PR with the main branch.

@vt-idiot
Copy link
Author

vt-idiot commented Jun 3, 2024

Awesome 😎 I'll need to find some time to pull your changes locally and test before merging your PR with the main branch.

No worries. Testing it locally myself was how I realized those ' apostrophes in the filenames would be an issue unless I URL encoded them, hence 315e1de

Unrelated: now I have to figure out why I can access it just fine directly on my server but not from any other machines on the LAN 😕 already edited CorsMiddleware.cs to have the server's LAN IP
image
geocode.arcgis is working fine and reachable but everything else loads forever
image

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.

2 participants