Limiting mini_magick memory usage about refile-mini_magick HOT 4 CLOSED

Not sure if there's a global config option in MiniMagick to set this kind of stuff. There isn't anything in Refile at least.

from refile-mini_magick.

If your VPS has certain memory/disk limits, it seems reasonable for me to set it per-VPS, that is, in the policy.xml file. Is there a reason why you would want to set it per-application/attribute?

I think there is no pretty way to add it to Refile. I was thinking we could write the processing methods like this:

def fit(img, width, height)
  img.combine_options do |cmd|
    # ...
    yield cmd if block_given?
    # ...
  end
end

And then you could override that method with

Refile::MiniMagick.prepend Module.new {
  def fit(img, width, height) do
    super do |cmd|
      cmd.limit "memory", "400MiB"
      cmd.limit "map", "400MiB"
      cmd.limit "disk", "1MiB"
    end
  end
}

from refile-mini_magick.

Part of the problem is, as far as I can tell, policy.xml isn't multi-instance aware. If I have 8 files being thumbnailed (which I very well could, from a single page render, depending on browser setup) the memory preferences would be much lower per file than they would be if I could force them to process serially.

from refile-mini_magick.

In #1 there was a similar request to make Refile::MiniMagick extendable using a dedicated API, but in #1 (comment) I explained why I don't think it would be a good idea. However, I will probably add an (undocumented) yield I proposed above, hopefully it should be good enough for both cases.

from refile-mini_magick.