News for hackers. Stuff that matters.
Re:Use pytorch2+cu118 with ADA hardware for 50%+ speedup by voz_ on Wednesday July 19, 2023 @01:13PM (#36789849)
This comment brings a tear to my eye.
Re:Use pytorch2+cu118 with ADA hardware for 50%+ speedup by doctorpangloss on Wednesday July 19, 2023 @03:46PM (#36792126)
The underlying problem is the community's decision to make users manage this in the first place.

This is an example of a setup.py that correctly installs the accelerated PyTorch for your platform:

https://github.com/comfyanonymous/ComfyUI/blob/9aeaac4af5e19...

As you can see, never merged. For philosophical reasons I believe. The author wanted to merge it earlier and changed his mind.

Like why make end users deal with this at all? The ROI from a layperson choosing these details is very low.

Python has a packaging problem, this is well known. Fixing setuptools would be highest yield. Other package tooling can't install PyTorch, for example: https://github.com/python-poetry/poetry/issues/6409#issuecom....

PyTorch itself is wonkily packaged. But I'm sure they have a good reason for this. Anyway, it goes to show that you can put a huge amount of effort into fixing this particular problem that everyone touching this technology has, and the maintainers everywhere will go nowhere with it. And I don't think this is a "me" problem, because there is so much demand for packaging PyTorch correctly - all the easy UIs, etc.

Re:Use pytorch2+cu118 with ADA hardware for 50%+ speedup by brucethemoose2 on Wednesday July 19, 2023 @04:02PM (#36792366)
> But I'm sure they have a good reason for this.

CUDA and ROCM make this an intractable problem. Basically there is no way to sanely package everything users need, and the absolutely enormous, cude/rocm versioned pytorch packages with missing libs are already a compromise.

TBH the whole ecosystem is not meant to be for end user inference anyway.

Re:Use pytorch2+cu118 with ADA hardware for 50%+ speedup by voz_ on Wednesday July 19, 2023 @04:29PM (#36792756)
Sorry, no idea what you are talking about.

I am talking about dynamic shapes in torch.compile.

You seem to be talking about software packaging. You also make heavy use of the word "this" without it being clear what "this" is.

Re:Use pytorch2+cu118 with ADA hardware for 50%+ speedup by brucethemoose2 on Wednesday July 19, 2023 @04:46PM (#36793057)
The two most popular stable diffusion UIs (automatic1111 and comfy) have longstanding issues with a few known but poorly documented bugs, like the ADA performance issue.

For instance, the torch.compile thing we are talking about is (last I checked) totally irrelevant for those UIs because they are still using the Stability AI implementation, not diffusers package that Huggingface checks for graph breaks. This may extend to SDXL.