Mconti1527/materials mconti material overhaul v2
Redid this with git ssh to get around web ide bug that was causing a bunch of the pictures to mess up
Merge request reports
Activity
Okay @michael.migliore 18th time is the charm haha. It seems like there was a bug with using the web ide that was messing up a bunch of the images. I think this one MAY finally be good. Let me know.
Thanks for the help.
Perfect! Looks good now.
I suppose you have tested each material successfully?Hey Michael, yes I have tested each one thoroughly but It would be good for someone else to download and go through each one. A couple notes:
• The metals are the main selling point of this. I have refined them over the past year and the selection is very good
• All of the textured materials are CC0 with sources cited
• For the textures, I have included a range of general ones, but most of them I generated for my use and are a bit more specific. They could definitely be expanded at some point to include more general ones
• I have renamed the old materials to fit in this the name convention to keep the list clean
• The old wood material was removed as it wasn’t a true seamless texture
I’m guessing if all is good this will get into 5.9?
Let me know if you all have any questions, and thanks again for the help with this.
Edited by Marston@cory.quammen I made sure to keep all textures as small as possible. They are all scaled down to 1024x1024 or 512x512 to reduce size. At that size they still look fine for most applications, but aren't huge files like 4k would be
@demarle If you want to check out the new version, I totally redid this to be CC0 and updated all of the textures
Fantastic! Thanks Marston, I'll give it a whirl today. @cory.quammen I think we should aim to get this into 5.9 rc1.
Well that's embarrasing, I'm not the owner here anymore. :)
A little help @cory.quammen or @michael.migliore ?
Actually looks like @utkarsh.ayachit @ben.boeckel and @brad.king have the power. Could one of you please merge this? Thanks!
Hey Brad!
mentioned in commit a4af0292
Thanks Ben!
Just to remind ya, this repo is upstream for https://gitlab.kitware.com/paraview/paraview-superbuild/-/blob/master/versions.cmake#L107. In ParaView it shows up as the default set before you even do File -> Load Path Tracer Materials. Marston's just provided this much more expansive, and still license clean, set than the six or so we've had since PV 5.5.
@cory.quammen ping, the tarball (Currently
https://www.paraview.org/files/data/OSPRayMaterials-0.2.tar.gz
) needs to be updated to get included in release.mentioned in issue paraview-superbuild#174 (closed)