search

fluviofx / fluviofx

Fluid dynamics for Unity's VFX graph
Other
697 stars 49 forks source link

Fluvio & HDRP 10+ #42

Open melMass opened 3 years ago

melMass commented 3 years ago

Hi,

I'm just discovering Fluvio FX, it looks amazing! Thanks for making it open for everyone.

I wanted to test it in Unity 2020+ using HDRP 10+. For some reason (I'm about to look more into it), the initialization of the "Install", is not rerun after each compilation and I have to do it manually. As I'm not embedding the visual effect package, the editor invalidates the access modifications made by the install, I'm not sure if this is a new behavior of the UPM:

The package cache was invalidated and rebuilt because the following immutable asset(s) were unexpectedly altered:

Well, I'm actually now embedding every related package (HDRP, VFX Graph, ShaderGraph), making them editable and removes the need to rerun the install after each compilation (I also removed each of them from the manifest obv).

image

The last error I've got was:

Packages/com.fluvio.fx@d30cc8ba92/Editor/ShaderPostprocessor/ShaderPostprocessor.cs(72,52): error CS0122: 'VFXDataParticle.capacity' is inaccessible due to its protection level

As I'm now embedding the packages, I just changed the access level of capacity in visual effect sources, effectively removing the error.

So now I don't have anymore compilation errors, but I think to many changes occurred in VFX graph since June 20 as this is how a blank FluvioGraph looks like:

image

Did anyone started an update of Fluvio already?

Thanks

melMass commented 3 years ago

I was not looking in the proper menu, the block are there, so I tried to recreate the graph based on the stickies:

image image

But the compute shader fails with the following errors:

image

So that makes me wonder, is Fluvio supported on Metal in the first place?

Thanks

lilyvelour commented 3 years ago

See #29, where we talk about various compatibility fixes for current Unity versions. I'll keep this open specifically though since I haven't verified how FluvioFX works with Metal at this time.