Page MenuHome
Differential D13605

Geometry Nodes: Remove "normal" from the spreadsheet
Needs ReviewPublic
Actions

Authored by Hans Goudey (HooglyBoogly) on Dec 17 2021, 6:01 AM.

Details

Reviewers
Jacques Lucke (JacquesLucke)
Summary

The normal attribute is not really an attribute, since it's read-only,
it's left over from when we didn't have a better way to expose the
feature. We'll likely remove the attribute when we remove the legacy
nodes.

This commit removes the column from the spreadsheet. It can still
be viewed with the normal input node and a viewer.

Diff Detail

Repository
rB Blender
Branch
spreadsheet-normal-remove (branched from master)
Build Status
Buildable 19503
Build 19503: arc lint + arc unit

Event Timeline

Hans Goudey (HooglyBoogly) requested review of this revision.Dec 17 2021, 6:01 AM
Hans Goudey (HooglyBoogly) created this revision.
Harbormaster completed remote builds in B19503: Diff 46147.Dec 17 2021, 6:01 AM
Hans Goudey (HooglyBoogly) retitled this revision from Geometry Nodes: Remove "normal" from the spreadsheet by default to Geometry Nodes: Remove "normal" from the spreadsheet.Dec 17 2021, 6:06 AM
Hans Goudey (HooglyBoogly) added a reviewer: Jacques Lucke (JacquesLucke).
Hans Goudey (HooglyBoogly) added a project: Geometry Nodes.
Jacques Lucke (JacquesLucke) added a comment.Dec 17 2021, 10:46 AM

I do wonder if there is another way to indicate when a column contains an attribute or derived data. Seeing derived data in the spreadsheet can be useful. Especially seeing the normal can be good for teaching purposes.

Hans Goudey (HooglyBoogly) mentioned this in D13601: Spreadsheet: Add material column for meshes.Dec 18 2021, 1:01 AM

Revision Contents
Changeset List

PathSize
source/
blender/
editors/
space_spreadsheet/
4 lines

Diff 46147

View Options

source/blender/editors/space_spreadsheet/spreadsheet_data_source_geometry.cc

Loading...