-
-
Notifications
You must be signed in to change notification settings - Fork 29
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
Better descriptions for A/V file links on publication pages #1704
Comments
We have a solution and it is to put in a display name. Otherwise it uses a fallback. What would suggest the fallback be? We do not have any intelligent way to know what it is |
What's a display name? Can you give an example? |
Your username is "moozooh" but maybe you want the display name to be "hoozoom" for April 1st or something. The username is what is used internally for things, the display name is what is shown in the fora or elsewhere. |
I must've missed something, but how are usernames related to links to video downloads? Which part of the link text are they substituting? |
@Invariel I think moozooh didn't ask about the concept of display names, just how they apply here. @moozooh We currently already have a display name field for publication URLs. An example would be https://tasvideos.org/3570M where it says "CamHack via Mirror". |
We use it for exclusive extra encodes so there's a way to distinguish them, but having to use them for all files would be a problem. I think we can get away with simply calling them |
@Masterjun3 I see. the screenshot I took was from this publication where I can now see it wasn't used, so that resulted in the inadequate fallback adelikat mentioned. Considering we've had a fixed system of filenames in place since the 00s, a simple regex code should be able to parse for everything we need without much intelligence required of it. I only know a bit of Python, but here's something that's hopefully a working example: def parse_url(url):
mkv_found = bool(re.search(r'\.mkv', url))
mp4_found = bool(re.search(r'512kb\.mp4', url))
camhack_found = bool(re.search(r'camhack', url))
if mkv_found and not mp4_found:
return 'text1' if not camhack_found else 'text3'
elif mp4_found and not mkv_found:
return 'text2' if not camhack_found else 'text4'
elif not mkv_found and not mp4_found:
raise ValueError('Neither ".mkv" nor "512kb.mp4" found in the URL.') That's a simple heuristic for the four different types of URLs: MKV, MP4, MKV camhack, MP4 camhack, which automatically gives each of them the names defined in @vadosnaprimer That's much better because it makes the descriptions different from each other at least, which is good. But it would be even better if we called them by their function (as in the mockup I posted) rather than the technical details only video encode enthusiasts would properly understand. So rather than "MKV (10bit444)", it would be "High quality (MKV)". Consider that users unfamiliar with our encoding policies (realistically, most of them) have no idea whether the "MKV (10bit444)" would be bigger or smaller, or better or worse than the "MP4 (512kb)". This at least gives them some idea on which file they should go for depending on how they're planning to use it or what device to play it on. (Instead of "Streamable" we could also go with "Compatibility" for MP4.) |
The problem with replacing suffixes with specific words is that the former are not consistent. At first we were marking 444 encodes with the 10bit444 suffix, then we dropped the suffix and 10bit444 became the primary downloadable. MP4 should always be streamable, but its format can also be different. I think it would be wise to scan all currently linked encode files and check their suffixes, and based on that data decide how to present files that have them. BTW there's only one archive.org encode now, and it's streamable, downloadable, and compatible. |
I agree with this. |
I got a full dump thanks to this script by Dacicus, and really 512kb and 10bit444 are the only ones used a lot, everything else is either a name or a featured encode for something like camhack or slower speed. I don't know if showing 512kb and 10bit444 to the user would help in any way, and trying to be more helpful breaks against past inconsistency. |
switching back to HQ MKV/Compat MP4 like old site had, except new pubs only have the latter fix #1704
* better way to display video downloads switching back to HQ MKV/Compat MP4 like old site had, except new pubs only have the latter fix #1704 * var
Here's another unfortunate artifact carried over from the old site:
There's no way to tell the difference between these four links based on these descriptions because they aren't describing anything. I bet half the people looking at this aren't even sure what "A/V" stands for, let alone what this enigmatic Mirror is, considering it isn't mirroring anything anymore. It used to—back when we considered BitTorrent the primary source of video captures—but that was years ago. So this is both confusing and factually incorrect.
Here's what we could (and should) do instead, in the interest of clarity:
The text was updated successfully, but these errors were encountered: