Page MenuHome

Michael Kowalski (makowalski)
User

Projects (10)

User Details

User Since
Sep 17 2020, 8:06 PM (92 w, 4 d)

Recent Activity

Fri, Jun 17

Michael Kowalski (makowalski) accepted D15222: USD: speed up large USD imports by not rebuilding material name map for each object.

This looks good. Thank you for this optimization, Aras!

Fri, Jun 17, 4:54 PM · Performance, Pipeline, Assets & I/O, Import/Export, BF Blender

Fri, Jun 10

Michael Kowalski (makowalski) added a comment to T97362: USD image assets paths are not resolved in a cross platform manner.

@Jamie Cope (copeg) Hi Jamie. The fix is committed. Please let me know if there are remaining issues with regard to the texture paths.

Fri, Jun 10, 9:15 PM · Pipeline, Assets & I/O, USD, BF Blender
Michael Kowalski (makowalski) closed T97362: USD image assets paths are not resolved in a cross platform manner as Resolved by committing rBadd1da52ad78: Fix T97362: forward slashes in USD texture paths.
Fri, Jun 10, 9:12 PM · Pipeline, Assets & I/O, USD, BF Blender
Michael Kowalski (makowalski) committed rBadd1da52ad78: Fix T97362: forward slashes in USD texture paths (authored by Michael Kowalski (makowalski)).
Fix T97362: forward slashes in USD texture paths
Fri, Jun 10, 9:12 PM
Michael Kowalski (makowalski) committed rBf6268f921ab0: USD import: Handle material name collisions (authored by Michael Kowalski (makowalski)).
USD import: Handle material name collisions
Fri, Jun 10, 4:20 PM
Michael Kowalski (makowalski) closed D14869: T90535: Handle material name collisions on USD import.
Fri, Jun 10, 4:20 PM · USD

Thu, Jun 9

Michael Kowalski (makowalski) added a comment to D14869: T90535: Handle material name collisions on USD import.

Thanks for the review, Sybren! I've updated the comment, per your suggestion.

Thu, Jun 9, 6:41 PM · USD
Michael Kowalski (makowalski) updated the diff for D14869: T90535: Handle material name collisions on USD import.
  • USD import: update comment.
Thu, Jun 9, 6:38 PM · USD

Wed, Jun 8

Michael Kowalski (makowalski) claimed T97362: USD image assets paths are not resolved in a cross platform manner.
Wed, Jun 8, 4:31 PM · Pipeline, Assets & I/O, USD, BF Blender
Michael Kowalski (makowalski) added a comment to T97362: USD image assets paths are not resolved in a cross platform manner.

Thanks for investigating this, @Omar Emara (OmarSquircleArt). I believe the solution is to ensure that relative paths have forward slashes when exporting USD from Blender. I will take this task and will work on a fix in the next couple of days.

Wed, Jun 8, 4:31 PM · Pipeline, Assets & I/O, USD, BF Blender

Fri, Jun 3

Michael Kowalski (makowalski) changed the status of T98572: UsdPreviewSurface import is missing UV transformations from Needs Information from Developers to Confirmed.
Fri, Jun 3, 7:34 PM · USD, Pipeline, Assets & I/O, BF Blender
Michael Kowalski (makowalski) claimed T98572: UsdPreviewSurface import is missing UV transformations.
Fri, Jun 3, 7:34 PM · USD, Pipeline, Assets & I/O, BF Blender
Michael Kowalski (makowalski) added a comment to T98572: UsdPreviewSurface import is missing UV transformations.

The current USD Preview Surface IO doesn't handle the UsdTransform2d node, so this is a known limitation. But this should be addressed and I'll extend the logic to support texture coordinate transforms like this.

Fri, Jun 3, 7:33 PM · USD, Pipeline, Assets & I/O, BF Blender

Thu, Jun 2

Michael Kowalski (makowalski) accepted D11592: Alembic/USD: use geometry sets to import data.

The USD related changes look good to me.

Thu, Jun 2, 8:44 AM · Pipeline, Assets & I/O, USD, Alembic

May 24 2022

Michael Kowalski (makowalski) added a comment to D14853: Re-fix T97366: Support single-file UDIMs if given a tokenized path.

I think Blender guessing what users mean is fine in the case of opening a single image. These kinds of heuristics can be nice quality of life improvements, and we have already done this for image sequences for a long time.

In the context of loading USD files relying on such heuristics is not going to work reliably though. Don't USD files already have file paths with <UDIM> tokens? I think that token is all the USD importer should be looking at, none of the other heuristics should be used for USD, probably not even as an option.

May 24 2022, 5:09 PM

May 23 2022

Michael Kowalski (makowalski) added a comment to T97195: Import USD materials that are not assigned to any object.

Thank you so much for the feedback, @Lennart Demes (struffel). Indeed, importing unassigned materials is a desirable feature, as I noted above, and I'm about to start work on this. Your files will serve as a good test case, so thank you for providing them!

May 23 2022, 4:36 AM · Pipeline, Assets & I/O, USD, BF Blender

May 11 2022

Michael Kowalski (makowalski) added a comment to D14895: Fix T97947: USD will fail to export without file extension.

This looks good to me, as well, and works nicely in my quick test. Thank you, @Philipp Oeser (lichtwerk)!

May 11 2022, 3:45 PM · USD, Pipeline, Assets & I/O

May 7 2022

Michael Kowalski (makowalski) updated the summary of D14869: T90535: Handle material name collisions on USD import.
May 7 2022, 1:29 AM · USD
Michael Kowalski (makowalski) added a comment to D14869: T90535: Handle material name collisions on USD import.

Thanks for the review, @Sybren A. Stüvel (sybren). I made the changes.

May 7 2022, 1:26 AM · USD
Michael Kowalski (makowalski) updated the diff for D14869: T90535: Handle material name collisions on USD import.
  • USD import: rename material collision option.
  • USD import: material collision code fixes.
May 7 2022, 1:11 AM · USD

May 6 2022

Michael Kowalski (makowalski) added a comment to D14853: Re-fix T97366: Support single-file UDIMs if given a tokenized path.

Thank you for the explanation, @Jesse Yurkovich (deadpin).

May 6 2022, 11:45 PM
Michael Kowalski (makowalski) added a comment to D14853: Re-fix T97366: Support single-file UDIMs if given a tokenized path.

As I noted in D13297#399515, I don't agree with the "two consecutive frames means UDIM" heuristic.

May 6 2022, 4:39 PM
Michael Kowalski (makowalski) added a comment to D13297: Fix USD Preview Surface import bugs reported in T90535 (missing materials in Animal Logic's ALab)..

I split out the material name collision fix into patch D14869. I will likewise separate the UDIM import and material purpose fallback logic into separate commits.

May 6 2022, 6:12 AM
Michael Kowalski (makowalski) retitled D14869: T90535: Handle material name collisions on USD import from T90535: Handle USD material import name collisions. to T90535: Handle material name collisions on USD import.
May 6 2022, 5:59 AM · USD
Michael Kowalski (makowalski) added a reviewer for D14869: T90535: Handle material name collisions on USD import: Sybren A. Stüvel (sybren).
May 6 2022, 5:58 AM · USD
Michael Kowalski (makowalski) updated the summary of D14869: T90535: Handle material name collisions on USD import.
May 6 2022, 5:57 AM · USD
Michael Kowalski (makowalski) retitled D14869: T90535: Handle material name collisions on USD import from USD Import: Handle material name collision. to T90535: Handle USD material import name collisions..
May 6 2022, 5:39 AM · USD
Michael Kowalski (makowalski) requested review of D14869: T90535: Handle material name collisions on USD import.
May 6 2022, 5:35 AM · USD

May 5 2022

Michael Kowalski (makowalski) accepted D14853: Re-fix T97366: Support single-file UDIMs if given a tokenized path.

The changes look good to me, and I confirmed that this fixes the issue loading UDIM textures that have only one tile (as described in D13297). Thank you for the quick fix!

May 5 2022, 6:41 PM

May 4 2022

Michael Kowalski (makowalski) added a comment to D13297: Fix USD Preview Surface import bugs reported in T90535 (missing materials in Animal Logic's ALab)..

Hmm, ok, let me try getting this fixed up for you. This is a tricky one as we don't want files like frame-1042.png to be UDIM'd when the user attempts to load them normally. Tile sets do not have to start at 1001 either so the detection can't make the determination based on that.

Would an additional boolean parameter to BKE_image_get_tile_info to allow this case be ok?

  • If True, a tokenized filepath must be provided and the tile count can be 1 or more -- this signifies that the caller is aware of what they're doing (providing the tokenized path) and willing to handle any range of tiles returned
  • If False, any filepath could be provided but the tile count must be 2 or more -- this signifies that the caller is not aware of what the path is, it might be a UDIM or might not, be safe and search for at least 2
May 4 2022, 11:40 PM
Michael Kowalski (makowalski) added a comment to D13297: Fix USD Preview Surface import bugs reported in T90535 (missing materials in Animal Logic's ALab)..

The issue with textures not loading seems to have been caused by the change introduced in

May 4 2022, 10:58 PM

Apr 28 2022

Michael Kowalski (makowalski) added a comment to D13297: Fix USD Preview Surface import bugs reported in T90535 (missing materials in Animal Logic's ALab)..

I tried the patch on the latest master with the new USD libraries, and there are textures missing now.

About splitting up the patch, I can see your point. I still think it's better to separate some things out, such that for example UDIM support is added separately from material name collision handling.

Apr 28 2022, 5:14 PM
Michael Kowalski (makowalski) added a comment to D14792: CMake: Reduce dependencies of USD to what's actually needed by Blender.

yeah but by that logic, we're using none of the imaging module right now, you want to disable the whole thing?

I'd like to see @Michael Kowalski (makowalski) chime in here and determine "is it useful to us and may we use it in the near future" lets not base it on if it's currently being used.

I've updated the description to read:

However, since [the Imaging] module was included for its support for converting primitive shapes (sphere, cube, etc.) to geometry, OSL is not necessary.

Earlier it read "..., it's not necessary", where it was ambiguous what "it" referred to. Support for USD primitives is a concrete use case.

Apr 28 2022, 4:23 PM · Platforms, Builds, Tests & Devices, USD, BF Blender (3.2)
Michael Kowalski (makowalski) accepted D14193: USD volume export.

Your changes look good to me, @Piotr Makal (pmakal), and I like your solution of having the File References option in its own box in the UI.

Apr 28 2022, 4:05 AM · USD

Apr 15 2022

Michael Kowalski (makowalski) added a comment to T93052: USD Import: No texture coordinates imported.

This is due to the "varying" primvar interpolation type not being handled in the UV conversion code. I'm addressing this.

Apr 15 2022, 12:44 AM · Pipeline, Assets & I/O, USD, BF Blender

Apr 14 2022

Michael Kowalski (makowalski) added a comment to T96458: USD Importer - not respecting output channels.

I can generate a test case based on the description and will investigate.

Apr 14 2022, 5:21 PM · Pipeline, Assets & I/O, USD, BF Blender
Michael Kowalski (makowalski) added a comment to T93052: USD Import: No texture coordinates imported.

I was able to reproduce the issue with the test case and am investigating.

Apr 14 2022, 4:56 PM · Pipeline, Assets & I/O, USD, BF Blender

Apr 12 2022

Michael Kowalski (makowalski) claimed T97195: Import USD materials that are not assigned to any object.
Apr 12 2022, 3:48 AM · Pipeline, Assets & I/O, USD, BF Blender
Michael Kowalski (makowalski) added a comment to T97195: Import USD materials that are not assigned to any object.

Thanks for reporting this, @Zhen Dai (zhendai). I agree that this is a good feature request and will investigate. We can probably add an option to import unassigned materials, or something similar.

Apr 12 2022, 3:48 AM · Pipeline, Assets & I/O, USD, BF Blender

Apr 7 2022

Michael Kowalski (makowalski) added a comment to D11592: Alembic/USD: use geometry sets to import data.

The changes look very good! (I primarily focused on the changes in the USD code.) @Kévin Dietrich (kevindietrich), would I be able to test the code in this patch in its current form, or should I wait for the changes in D14277 to be completed?

Apr 7 2022, 3:51 AM · Pipeline, Assets & I/O, USD, Alembic

Mar 28 2022

Michael Kowalski (makowalski) added a comment to T96747: Support for USD PointInstancer.

I'm guessing here a bit, but one approach would be creating a point cloud with a geometry nodes modifier that uses the Instance on Points node. All of the instances could go in a collection, and the point cloud could have an instance_index attribute used to choose the correct instance.

Mar 28 2022, 8:04 PM · USD, Pipeline, Assets & I/O, BF Blender

Mar 25 2022

Michael Kowalski (makowalski) accepted D14394: USD: Fix whole archive linking on windows..

This looks good to me, as well. (With the caveat that I don't have a lot of specific experience building the USD libs.) Good catch on an interesting issue!

Mar 25 2022, 5:14 PM · Platform: Windows, USD
Michael Kowalski (makowalski) added a comment to T96747: Support for USD PointInstancer.

it would be great to have the option that the PointInstancers be imported as particle systems

Since the old particle system is deprecated, I don't think this is the right approach. Without more knowledge of USD I'm not sure what to suggest instead, but I would think something using the same type of instances generated by geometry nodes would be preferable (see InstancesComponent). That has support for arbitrary generic attributes on instances.

Mar 25 2022, 4:04 PM · USD, Pipeline, Assets & I/O, BF Blender

Mar 24 2022

Michael Kowalski (makowalski) closed D14204: Fix: Volume grid duplication after importing USD file.
Mar 24 2022, 5:44 PM · USD
Michael Kowalski (makowalski) committed rBef228d92f2e7: Fix: Volume grid duplication on USD import. (authored by Michael Kowalski (makowalski)).
Fix: Volume grid duplication on USD import.
Mar 24 2022, 5:44 PM
Michael Kowalski (makowalski) accepted D14204: Fix: Volume grid duplication after importing USD file.

I'm approving these changes and will commit the fix.

Mar 24 2022, 4:31 PM · USD
Michael Kowalski (makowalski) changed the status of T96747: Support for USD PointInstancer from Needs Information from Developers to Confirmed.
Mar 24 2022, 3:57 PM · USD, Pipeline, Assets & I/O, BF Blender
Michael Kowalski (makowalski) added a comment to T96747: Support for USD PointInstancer.

Thank you for the feedback @Sam Wong (anthropoy). I am investigating this option and will post progress in this task.

Mar 24 2022, 3:56 PM · USD, Pipeline, Assets & I/O, BF Blender
Michael Kowalski (makowalski) added a comment to T96747: Support for USD PointInstancer.

@Michael Kowalski (makowalski) : havent gotten to building USD to get access to utils that let me examine a.usd the same way I can examine an alembic .abc (so I cannot really tell how the USD looks from the inside), but I assume this is not implemented yet?
For me, a single empty is imported here (under another empty for the particle system under another empty as the root).
Also not sure if the plans are to do it like alembic in blender [which imports verts with meshsequence cache modifier for plain particles or objects with transform caches for instanced particles afaict], or if this will somehow end up on the pointcloud type?

CC @Sybren A. Stüvel (sybren)

Mar 24 2022, 3:36 PM · USD, Pipeline, Assets & I/O, BF Blender

Mar 23 2022

Michael Kowalski (makowalski) added a comment to D13297: Fix USD Preview Surface import bugs reported in T90535 (missing materials in Animal Logic's ALab)..

This patch is introducing too many new things all at once. Support for importing UDIM should be its own patch. Material name collision handling as well. Falling back to another "material purpose" is yet another thing, and cleanups should also be separate from functional changes.

Even with this patch, there seem to be missing textures. At least, I think there are, as the house/shed itself isn't textured:

Mar 23 2022, 5:05 AM

Mar 22 2022

Michael Kowalski (makowalski) added a comment to D13297: Fix USD Preview Surface import bugs reported in T90535 (missing materials in Animal Logic's ALab)..

This patch is introducing too many new things all at once. Support for importing UDIM should be its own patch. Material name collision handling as well. Falling back to another "material purpose" is yet another thing, and cleanups should also be separate from functional changes.

Even with this patch, there seem to be missing textures. At least, I think there are, as the house/shed itself isn't textured:

Mar 22 2022, 4:49 PM
Michael Kowalski (makowalski) added inline comments to D14193: USD volume export.
Mar 22 2022, 4:09 PM · USD

Mar 19 2022

Michael Kowalski (makowalski) added a comment to D14204: Fix: Volume grid duplication after importing USD file.

This change looks good to me! Again, thanks for discovering this @Piotr Makal (pmakal). Unless @Sybren A. Stüvel (sybren) has additional remarks or wants to review further, I think this can be committed. We can confirm whether additional review is needed this week.

Mar 19 2022, 6:23 PM · USD

Mar 15 2022

Michael Kowalski (makowalski) added a comment to D14193: USD volume export.

Thanks for the latest round of changes, @Piotr Makal (pmakal)! The patch looks good to me, but I'd like to wait for any additional comments from @Sybren A. Stüvel (sybren) based on his review.

Mar 15 2022, 2:23 AM · USD

Mar 14 2022

Michael Kowalski (makowalski) added a comment to T93904: USD import: investigate texture color space conversion issue.

I'm uploading the notebook.7z archive originally provided by @BRUNO RANTIN PO (brurpo) that contains the USDs and textures that exhibit the behavior. Note that this example is for Windows only, as it references textures in the c: drive. To test on Windows, extract the contents under the c: root for the texture paths to resolve correctly.

Mar 14 2022, 11:21 PM · Pipeline, Assets & I/O, BF Blender, USD
Michael Kowalski (makowalski) changed the status of T93904: USD import: investigate texture color space conversion issue from Needs Information from Developers to Confirmed.
Mar 14 2022, 6:29 PM · Pipeline, Assets & I/O, BF Blender, USD
Michael Kowalski (makowalski) added a comment to T93904: USD import: investigate texture color space conversion issue.

Hi @Philipp Oeser (lichtwerk). Thanks for following up on this. Yes, this bug can be confirmed, and I'll provide a USD example, as you suggested. This is a relatively simple change, and I will most likely make the fix after the latest USD Preview Surface import patch (https://developer.blender.org/D13297) has been approved and merged into master.

Mar 14 2022, 6:29 PM · Pipeline, Assets & I/O, BF Blender, USD

Mar 11 2022

Roger B (rboxman) awarded D13297: Fix USD Preview Surface import bugs reported in T90535 (missing materials in Animal Logic's ALab). a Love token.
Mar 11 2022, 11:05 AM
Michael Kowalski (makowalski) added a comment to D14193: USD volume export.

Thanks for making the changes, @Piotr Makal (pmakal)!

Mar 11 2022, 1:29 AM · USD

Mar 5 2022

Michael Kowalski (makowalski) added inline comments to D14193: USD volume export.
Mar 5 2022, 7:01 PM · USD

Mar 2 2022

Michael Kowalski (makowalski) added a comment to D14193: USD volume export.

Some more comments. Further review to follow.

Mar 2 2022, 5:18 AM · USD

Mar 1 2022

Satish Goda (satishgoda) awarded rBc85c52f2ce47: USD Preview Surface material export. a 100 token.
Mar 1 2022, 7:04 PM
Michael Kowalski (makowalski) added a comment to D14193: USD volume export.

Just a couple of minor points. Additional review to follow.

Mar 1 2022, 4:11 AM · USD
Michael Kowalski (makowalski) added a comment to D14193: USD volume export.

I started the review, and the code looks very good so far!

Mar 1 2022, 12:07 AM · USD

Feb 28 2022

Michael Kowalski (makowalski) added a comment to T94396: USD animation exported from Audio2face can't play after opening saved scene.

This fix is available in the 3.2 alpha builds.

Feb 28 2022, 2:11 AM · USD, Pipeline, Assets & I/O, BF Blender

Feb 27 2022

Michael Kowalski (makowalski) added a comment to D14204: Fix: Volume grid duplication after importing USD file.

Good catch, @Piotr Makal (pmakal)! I will test this functionality this week.

Feb 27 2022, 11:44 PM · USD

Feb 25 2022

Michael Kowalski (makowalski) abandoned D14066: Fix T94396: USD errors opening saved scenes.

As per discussion with @Sybren A. Stüvel (sybren) at the pipeline/assets/IO meeting yesterday, this revision doesn't require a review, so I'm closing it. I've submitted the changes in https://developer.blender.org/rB0a6a74bac41558af0ee6d3031d923470d32e767e.

Feb 25 2022, 3:57 PM · USD
Michael Kowalski (makowalski) closed T94396: USD animation exported from Audio2face can't play after opening saved scene as Resolved by committing rB0a6a74bac415: Fix T94396: USD errors opening saved scenes..
Feb 25 2022, 3:28 PM · USD, Pipeline, Assets & I/O, BF Blender
Michael Kowalski (makowalski) committed rB0a6a74bac415: Fix T94396: USD errors opening saved scenes. (authored by Michael Kowalski (makowalski)).
Fix T94396: USD errors opening saved scenes.
Feb 25 2022, 3:28 PM

Feb 23 2022

Michael Kowalski (makowalski) added a comment to D14184: USD: Support building against USD 21.11+.

@Ray molenkamp (LazyDodo) Thanks for creating the patch. To clarify, the one new TODO I added for myself is to be addressed later. It's to add missing functionality (setting the distant light angle on export) which is unrelated to the USD upgrade.

Feb 23 2022, 10:28 PM

Feb 18 2022

Michael Kowalski (makowalski) added a comment to D13297: Fix USD Preview Surface import bugs reported in T90535 (missing materials in Animal Logic's ALab)..

Is there a way to test this without having to download >15 GB of data?


... and this is at the Blender office with our 500 MBit/sec fibre connection. It takes 2 hours to just download the first 1 GB file, so the other 14.5 GB of textures is going to be a struggle.

Feb 18 2022, 9:48 PM

Feb 11 2022

Michael Kowalski (makowalski) added a comment to rBc85c52f2ce47: USD Preview Surface material export..

I'm curious to know why the UDS file becomes so good. My blender test file is 2.7 megs, textures included. The alembic export is 1.7m but the USD is 24.2 meg. Usually alembic and USD are about the same size. This one is more than 14 times bigger. Here's the clean test file.

Feb 11 2022, 10:17 PM
Michael Kowalski (makowalski) updated subscribers of D14087: Fix T95203: USD mesh sequence cache modifier imports incorrect shaders..

@Kévin Dietrich (kevindietrich) Hi Kevin. I just wanted you to be aware that in this patch I'm adding a new mesh sequence cache modifier read flag for material face sets that apples to USD only. I would make it clear in the property description and in the documentation that this does not affect Alembic. However, I don't feel this is an ideal solution, because it can make the UI confusing when some properties apply to one format and not the other. This is probably inevitable to some extent since the USD and Alembic mesh readers behave differently in some ways. In any case, please let me know if you have any concerns about this change or can think of better ways to handle this. Thanks!

Feb 11 2022, 4:44 PM · USD
Michael Kowalski (makowalski) updated the summary of D14087: Fix T95203: USD mesh sequence cache modifier imports incorrect shaders..
Feb 11 2022, 4:36 PM · USD
Michael Kowalski (makowalski) retitled D14087: Fix T95203: USD mesh sequence cache modifier imports incorrect shaders. from Fix T95203: USD mesh sequence modifier imports incorrect shaders. to Fix T95203: USD mesh sequence cache modifier imports incorrect shaders..
Feb 11 2022, 3:41 AM · USD
Michael Kowalski (makowalski) added a revision to T95203: Shaders not displayed properly with USD: D14087: Fix T95203: USD mesh sequence cache modifier imports incorrect shaders..
Feb 11 2022, 3:37 AM · BF Blender
Michael Kowalski (makowalski) requested review of D14087: Fix T95203: USD mesh sequence cache modifier imports incorrect shaders..
Feb 11 2022, 3:37 AM · USD

Feb 9 2022

Michael Kowalski (makowalski) added a comment to T94396: USD animation exported from Audio2face can't play after opening saved scene.

My apologies. This simple fix has been delayed unnecessarily due to miscommunication on my part. Hopefully we can submit this very soon once revision D14066 has been accepted.

Feb 9 2022, 8:27 PM · USD, Pipeline, Assets & I/O, BF Blender
Mariano Basti (marianobasti) awarded D14066: Fix T94396: USD errors opening saved scenes a Love token.
Feb 9 2022, 8:21 PM · USD
Michael Kowalski (makowalski) added a reviewer for D14066: Fix T94396: USD errors opening saved scenes: Sybren A. Stüvel (sybren).
Feb 9 2022, 8:17 PM · USD
Michael Kowalski (makowalski) requested review of D14066: Fix T94396: USD errors opening saved scenes.
Feb 9 2022, 8:15 PM · USD
Michael Kowalski (makowalski) added a revision to T94396: USD animation exported from Audio2face can't play after opening saved scene: D14066: Fix T94396: USD errors opening saved scenes.
Feb 9 2022, 8:15 PM · USD, Pipeline, Assets & I/O, BF Blender

Feb 8 2022

Michael Kowalski (makowalski) added a comment to T95203: Shaders not displayed properly with USD.

@Robert Rioux (riouxr) My apologies for the delay looking into this. I've identified the source of the bug in the logic for setting material face sets when the mesh sequence cache modifier is applied. I have a solution which I'm currently testing and will update this thread when the fix is ready.

Feb 8 2022, 4:39 AM · BF Blender

Feb 7 2022

Michael Kowalski (makowalski) added a comment to rBc85c52f2ce47: USD Preview Surface material export..

I see that subsurface, subsurface color, specular, specular tint, anamorphic, transmission and emission are not supported. Displacement too but that's not connected to the shader itself. Do you think this could be supported eventually? The reason I ask is that it would be absolutely wonderful because Clarisse's Disney shader has the same settings as Principal BSDF and it would make it so much more easier to bring things from Blender to Clarisse.

Oh! UDIM support too :-)

I assume the USDPreviewSurface Shader handles things like transmission differently to the Principled Shader. Apparently, if the opacity value is below 1, it is treated as translucency and if it is zero, it becomes transparent. there is no dedicated transmission value in the USDpreviewshader

It'd be amazing to have that work properly though. Glass materials don't seem to work right now..

Feb 7 2022, 4:00 PM

Feb 2 2022

Michael Kowalski (makowalski) added a comment to rBc85c52f2ce47: USD Preview Surface material export..

@Robert Rioux (riouxr) To follow up further, are any Blender material nodes being created at all when you re-import the USD? If so, is there a specific problem in the shader import you are noticing (e.g., perhaps the texture paths are not coming in correctly, etc.)?

Feb 2 2022, 5:22 AM
Michael Kowalski (makowalski) added a comment to rBc85c52f2ce47: USD Preview Surface material export..

That is something I have been waiting for for a long time. :-)
That being said, I tried to export a model with some maps on the color, metallic, roughness and normal. I turned on the export USD preview, textures and all.
I reimported that UDS in Blender with the option for USD preview but the shaders didn't translate. Am I missing something or it's still too early? I'm using alpha 3.2

Feb 2 2022, 3:34 AM
Robert Rioux (riouxr) awarded rBc85c52f2ce47: USD Preview Surface material export. a Love token.
Feb 2 2022, 2:41 AM
Vyacheslav (hitrpr) awarded rBc85c52f2ce47: USD Preview Surface material export. a Party Time token.
Feb 2 2022, 12:43 AM

Feb 1 2022

Michael Kowalski (makowalski) changed the status of T95407: USD volume export from Needs Triage to Confirmed.
Feb 1 2022, 9:30 PM · USD

Jan 27 2022

Michael Kowalski (makowalski) added inline comments to D13297: Fix USD Preview Surface import bugs reported in T90535 (missing materials in Animal Logic's ALab)..
Jan 27 2022, 9:18 PM
Deen (Bit) awarded D13647: USD Preview Surface material export. a Love token.
Jan 27 2022, 5:00 PM
Michael Kowalski (makowalski) added a comment to T95203: Shaders not displayed properly with USD.

Hi @Philipp Oeser (lichtwerk): Yes, I can reproduce this, but not always as originally described. One way I can reliably reproduce it is to import that USD directly and attempt to change the material colors, which has no effect unless in edit mode. I haven't had a chance to investigate this yet but will do so in the next day or so.

Jan 27 2022, 4:51 AM · BF Blender

Jan 26 2022

Michael Kowalski (makowalski) added a comment to T95203: Shaders not displayed properly with USD.

Thanks for reporting this. I'll take a look.

Jan 26 2022, 3:59 AM · BF Blender

Jan 25 2022

Michael Kowalski (makowalski) added a comment to D13647: USD Preview Surface material export..

Another test blend file, this one with an external textures directory:

Jan 25 2022, 6:25 PM
Michael Kowalski (makowalski) added a comment to T94396: USD animation exported from Audio2face can't play after opening saved scene.

Here is another workaround: when you first launch Blender, before opening the .blend file in question, do a simple export to USD (for example, save the default cube to a throw-away USD file somewhere). This will force the USD plugins to load. You should then be able to load the blend file with the mesh sequence cache modifiers with no errors. Please let me know if this does not work for you.

Jan 25 2022, 5:53 PM · USD, Pipeline, Assets & I/O, BF Blender
Michael Kowalski (makowalski) added a comment to D13647: USD Preview Surface material export..

Hi @Sybren A. Stüvel (sybren). I've used the more complex "Junk Shop" scene for testing:

Jan 25 2022, 5:12 PM

Jan 24 2022

Michael Kowalski (makowalski) added a comment to D13647: USD Preview Surface material export..

Thanks for the review, @Hans Goudey (HooglyBoogly)!

Jan 24 2022, 1:44 AM
Michael Kowalski (makowalski) updated the diff for D13647: USD Preview Surface material export..

Update to address Hans Goudey's latest remark.

Jan 24 2022, 1:42 AM

Jan 20 2022

Michael Kowalski (makowalski) added a comment to D13647: USD Preview Surface material export..

@Sybren A. Stüvel (sybren) , I'm also working on the unit tests you requested. Can I submit the tests in a new patch, or do you want me to include them in this one?

Jan 20 2022, 2:21 AM

Jan 15 2022

Michael Kowalski (makowalski) added a comment to D13647: USD Preview Surface material export..

I believe I've addressed the remaining comments, but please let me know if I missed anything.

Jan 15 2022, 3:08 AM