Change media upload limits and remove client-side resizing

[Gargron]

For media attachments, the limits become:

• Images:
  • Upload size: 10 MB to 16 MB
  • Target dimensions: Full HD to 4K
  • Processing limit: 4K to 8K
• Videos:
  • Upload size: 40 MB to 99 MB
  • FPS limit: 60 FPS to 120 FPS
  • Processing limit: Full HD to 4K

For link preview cards, the limits become:

• Upload size: 1 MB to 2 MB

Client-side resizing is removed to avoid low-quality output. The max. RAM usage of ImageMagick becomes 847 MB.

---

Fixes MAS-47

[renchap]

If this goes through, you can also remove the resize_image.js files and the exif-js JS dependency.

[tabletcorry]

Can the media resizing process forks be niced to keep them from swamping the system as heavily?

Larger media sounds great, but I already see occasional media cause massive CPU spikes on my personal instance. Communicating the low priority of conversion to the cpu scheduler would help a lot.

[ClearlyClaire]

This will cause an increase in RAM, CPU, and storage usage on all servers, so we really need to be careful about such changes.

[ClearlyClaire]

If this goes through, you can also remove the resize_image.js files and the exif-js JS dependency.

True, although we could use exif-js to provide alt-text-from-image-metadata functionality (could also be done server-side I guess but it makes more sense to me as a client-side feature)

[shleeable]

Storage is cheap, so I'm fine with the increased image/movie size, but the effect Client-side resizing being removed will have on smaller instances needs to be considered.

I'd rather you set a standard and transcode on the iOS app for example to a known universal standard... Our CPUs are going to cry... but making 8 accounts and uploading a large 4K video will most than likely result in a DOS.

[anampses]

This is a welcome change for art instances such as mine. However, I noticed that this upcoming change increases the max video resolution to 4K, but image resolution is increasing to only 2K for some reason. Is there a reason for this? Why would video media get to have more pixels when previously both video and images had the same FHD limit image format?

Ideally the image size should be bumped up to at least 4K, just like video. Having more pixels is important for artists since it is more common for images to be entirely graphical (eg illustrations, comics) than it is with videos which is usually captured.

Also, since there is no more client-side resizing, a higher pixel limit will keep the server's CPU/RAM usage down since the server won't have to do resizing of images as often.

[ClearlyClaire]

Bumping the image pixel count limit is very RAM-costly due to the tools we use. Until we ditch ImageMagick, I don't think it's wise to increase the allowed input image pixel count at all. #23322 touches on this and actually allows larger output images than the current PR (that means a potential storage cost increase, but not a RAM or CPU one unlike the current PR)

[rrgeorge]

I think that the max size should only be 100MB, as that’s the upload limit on some CDNs.

[Gargron]

Changed video size limit to 99 MB, target dimensions for images to 4K (same as videos).

The changed image matrix limit means the max. ImageMagick memory usage will be about 847 MB.

[WhiskeyOmega]

Can I suggest we include slightly higher vertical resolution to include DSLR images Aspect Ratios

app/models/concerns/attachmentable.rb
MAX_MATRIX_LIMIT = 33_177_600 # 7680x4913px or approx. 847MB in RAM

app/models/media_attachment.rb
original: {
pixels: 9_434_880, # 3840x2457px
file_geometry_parser: FastGeometryParser,
}.freeze,

[floogulinc]

Any idea when we'll see this in a stable release?

[madrobby]

For whom it may concern, the standard size for many phone and DSLR photos is currently moving to (and often already is) 48MP (roughly 8,500 by 5,650 pixels, this is slightly more than 8K UHD).

It's likely fine to stay at the ~4K target as most phones convert 48MP to 12MP internally anyway, but it's definitely coming somewhere down the line.

[FediVideos]

Does anyone know if the audio upload limit has been increased too? Doesn't seem to be mentioned anywhere...?

[rrgeorge]

Does anyone know if the audio upload limit has been increased too? Doesn't seem to be mentioned anywhere...?

It appears that audio uses the same size limit as video (

mastodon/app/models/media_attachment.rb

Line 233 in 27e84c0

def larger_media_format?

), so it's now 99mb.