Alembic export not respecting the renderable flag. #89594

New Issue

Beorn Leonard · 2021-07-02T09:41:22+02:00

Beorn Leonard commented

2021-07-02 09:41:22 +02:00

System Information
Operating system: Windows 10
Graphics card: nvidia Quadro K4000

Blender Version
Broken: 2.93.1

When exporting an alembic file, when the option to only export renderable objects is checked, the objects flagged as "disable in renders" in the outliner are still exported.
Effectively, the checkbox doesn't seem to do anything, or at least doesn't behave as expected.

Exact steps for others to reproduce the error
In the default scene, duplicate the cube, move it to the side and flag it "disable in renders" in the outliner.
Export an alembic file.
Delete all objects and import the alembic file you just exported.
Both cubes will be there in the alembic file, and both will be set as renderable.

This is important to fix for example in scenes which use low res geometry for animators to work with, but high res geo for the render.
We need to be able to only export the renderable geo reliably.

**System Information** Operating system: Windows 10 Graphics card: nvidia Quadro K4000 **Blender Version** Broken: 2.93.1 When exporting an alembic file, when the option to only export renderable objects is checked, the objects flagged as "disable in renders" in the outliner are still exported. Effectively, the checkbox doesn't seem to do anything, or at least doesn't behave as expected. **Exact steps for others to reproduce the error** In the default scene, duplicate the cube, move it to the side and flag it "disable in renders" in the outliner. Export an alembic file. Delete all objects and import the alembic file you just exported. Both cubes will be there in the alembic file, and both will be set as renderable. This is important to fix for example in scenes which use low res geometry for animators to work with, but high res geo for the render. We need to be able to only export the renderable geo reliably.

Beorn Leonard commented

2021-07-02 09:41:22 +02:00

Added subscriber: @BeornLeonard

Philipp Oeser commented

2021-07-02 10:39:59 +02:00

Added subscriber: @lichtwerk

Philipp Oeser commented

2021-07-02 10:39:59 +02:00

Changed status from 'Needs Triage' to: 'Confirmed'

Philipp Oeser self-assigned this 2021-07-02 10:39:59 +02:00

Philipp Oeser commented

2021-07-02 10:39:59 +02:00

Confirmed, will check

Pratik Borhade commented

2021-07-02 10:52:06 +02:00

Added subscriber: @PratikPB2123

Philipp Oeser commented

2021-07-02 11:29:46 +02:00

Looks like the option to only export renderable objects was broken for a long time.
If checked, it seemed to be working (objects marked non-renderable in the Outliner were pruned from the export), however unchecking that option did not include them in the export

a95f863596 made it so if "renderable_only" is set, it will still export (is now more dependent on the Visible Objects option

Looks like the option to only export renderable objects was broken for a long time. If checked, it seemed to be working (objects marked non-renderable in the Outliner were pruned from the export), however unchecking that option did not include them in the export a95f863596 made it so if "renderable_only" is set, it will still export (is now more dependent on the `Visible Objects` option

Sybren A. Stüvel commented

2021-07-02 11:35:29 +02:00

Added subscriber: @dr.sybren

Sybren A. Stüvel commented

2021-07-02 11:35:29 +02:00

@BeornLeonard when exporting to Alembic, objects are evaluated in "render" mode anyway, so "Visible Only" already actually means "Visible When Rendering". Would that solve your problem?

Philipp Oeser commented

2021-07-02 11:40:48 +02:00

Renderable Objects only and Visible Objects only are really the same thing, no?
If we find something that makes Visible Objects not sufficient in some usecase, and define how things should work, we can make the option work that way, otherwise removing it seems actually like the way to go I think.

`Renderable Objects` only and `Visible Objects` only are really the same thing, no? If we find something that makes `Visible Objects` not sufficient in some usecase, and define how things should work, we can make the option work that way, otherwise removing it seems actually like the way to go I think.

Philipp Oeser commented

2021-07-02 11:43:57 +02:00

Changed status from 'Confirmed' to: 'Needs User Info'

Philipp Oeser commented

2021-07-02 11:43:57 +02:00

@BeornLeonard: will wait for a possible usecase before taking further actions.

Beorn Leonard commented

2021-07-02 11:44:42 +02:00

@dr.sybren I'm not sure I understand, and perhaps there's something I'm missing.
When I click "Disable in Renders" in the outliner, and export an alembic with the option "Only Renderable Objects" checked, all the objects still get exported, regardless.

What am I missing?

@dr.sybren I'm not sure I understand, and perhaps there's something I'm missing. When I click "Disable in Renders" in the outliner, and export an alembic with the option "Only Renderable Objects" checked, all the objects still get exported, regardless. What am I missing?

Philipp Oeser commented

2021-07-02 11:49:04 +02:00

In #89594#1185927, @BeornLeonard wrote:
@dr.sybren I'm not sure I understand, and perhaps there's something I'm missing.
When I click "Disable in Renders" in the outliner, and export an alembic with the option "Only Renderable Objects" checked, all the objects still get exported, regardless.

What am I missing?

That option to only export renderable objects was broken for a long time.
If checked, it seemed to be working (objects marked non-renderable in the Outliner were pruned from the export), however unchecking that option did not include them in the export
But true, it changed - for the worse if you like - in a95f863596 which made it so if "renderable_only" is set, it will still export (is now more dependent on the Visible Objects option)
Atm. the Renderable Objects option does not do anything

Please try the Visible Objects option, does this work for you?

> In #89594#1185927, @BeornLeonard wrote: > @dr.sybren I'm not sure I understand, and perhaps there's something I'm missing. > When I click "Disable in Renders" in the outliner, and export an alembic with the option "Only Renderable Objects" checked, all the objects still get exported, regardless. > > What am I missing? That option to only export renderable objects was broken for a long time. If checked, it seemed to be working (objects marked non-renderable in the Outliner were pruned from the export), however unchecking that option did not include them in the export But true, it changed - for the worse if you like - in a95f863596 which made it so if "renderable_only" is set, it will still export (is now more dependent on the `Visible Objects` option) Atm. the `Renderable Objects` option **does not do anything** Please try the `Visible Objects` option, does this work for you?

Beorn Leonard commented

2021-07-02 11:52:54 +02:00

@lichtwerk
I tried to outline my use case in the original report, but here's some more detail:
In a production it is common to use low res geometry to give animators better performance.
In the outliner, the low res meshes are flagged as visible, but not renderable, and the full res meshes are often flagged in the opposite way.
(Actually the visibility is generally switchable as animators will want to see the high res mesh in the later stages of polish).
When it comes to exporting a mesh cache, we want only the high res meshes exported, and I would expect the exporter to ignore all meshes that are flagged as not renderable.

Does this clarify things?

@lichtwerk I tried to outline my use case in the original report, but here's some more detail: In a production it is common to use low res geometry to give animators better performance. In the outliner, the low res meshes are flagged as visible, but not renderable, and the full res meshes are often flagged in the opposite way. (Actually the visibility is generally switchable as animators will want to see the high res mesh in the later stages of polish). When it comes to exporting a mesh cache, we want only the high res meshes exported, and I would expect the exporter to ignore all meshes that are flagged as not renderable. Does this clarify things?

Philipp Oeser commented

2021-07-02 11:58:55 +02:00

In #89594#1185931, @BeornLeonard wrote:
@lichtwerk
I tried to outline my use case in the original report, but here's some more detail:
In a production it is common to use low res geometry to give animators better performance.
In the outliner, the low res meshes are flagged as visible, but not renderable, and the full res meshes are often flagged in the opposite way.
(Actually the visibility is generally switchable as animators will want to see the high res mesh in the later stages of polish).
When it comes to exporting a mesh cache, we want only the high res meshes exported, and I would expect the exporter to ignore all meshes that are flagged as not renderable.

Does this clarify things?

This can be achieved with the Visible Objects option.

> In #89594#1185931, @BeornLeonard wrote: > @lichtwerk > I tried to outline my use case in the original report, but here's some more detail: > In a production it is common to use low res geometry to give animators better performance. > In the outliner, the low res meshes are flagged as visible, but not renderable, and the full res meshes are often flagged in the opposite way. > (Actually the visibility is generally switchable as animators will want to see the high res mesh in the later stages of polish). > When it comes to exporting a mesh cache, we want only the high res meshes exported, and I would expect the exporter to ignore all meshes that are flagged as not renderable. > > Does this clarify things? This can be achieved with the `Visible Objects` option.

Beorn Leonard commented

2021-07-02 11:58:56 +02:00

@lichtwerk :
Ok, I see what you and @dr.sybren were getting at.
The "visible objects" flag does indeed stop the non renderable objects from exporting.

In that case, can I suggest changing those labels? Right now it doesn't match the labels in the outliner, and seems a bit anti-intuitive to the user.
But any way, this solves my issue for now. Thanks

@lichtwerk : Ok, I see what you and @dr.sybren were getting at. The "visible objects" flag does indeed stop the non renderable objects from exporting. In that case, can I suggest changing those labels? Right now it doesn't match the labels in the outliner, and seems a bit anti-intuitive to the user. But any way, this solves my issue for now. Thanks

Philipp Oeser commented

2021-07-02 12:03:08 +02:00

@dr.sybren: so plan of attack is:

remove Renderable Objects option
improve UI text of Visible Objects? replace "Export only objects in visible layers" with "Export only objects visible in rendering"?

@dr.sybren: so plan of attack is: - remove `Renderable Objects` option - improve UI text of `Visible Objects`? replace "Export only objects in visible layers" with "Export only objects visible in rendering"?

Sybren A. Stüvel commented

2021-07-02 12:31:55 +02:00

Yes, let's remove "Renderable Objects".

As for the "Visible Objects" option, what I think is a better approach is what I aready added to the USD exporter: a choice whether Render or Viewport evaluation is used. When that choice is explicit, it's also clear which visibility actually matters.

These could be done in two separate commits, as I think the removal of "Renderable Objects" is already a good start.

Yes, let's remove "Renderable Objects". As for the "Visible Objects" option, what I think is a better approach is what I aready added to the USD exporter: a choice whether Render or Viewport evaluation is used. When that choice is explicit, it's also clear which visibility actually matters. These could be done in two separate commits, as I think the removal of "Renderable Objects" is already a good start.

Beorn Leonard commented

2021-07-02 12:35:34 +02:00

@dr.sybren Just checked out the USD exporter, and indeed that is a much more clear way of presenting it, IMHO.