Fix: Small changes to USD and Alembic curves import #118829

Merged
Hans Goudey merged 3 commits from HooglyBoogly/blender:cleanup-usd-abc-curves-import into main 2024-02-28 05:06:02 +01:00

3 Commits

Author SHA1 Message Date
Hans Goudey ded394b4a6 Pass Span argument by value 2024-02-27 22:59:46 -05:00
Hans Goudey 2c01509810 More float literals 2024-02-27 22:01:19 -05:00
Hans Goudey 2adbc2afc0 Fix: Small changes to USD and Alembic curves import
Make both importers work the same way: first building an array
of the new offsets, then comparing the offsets to check if the
topology changed. Previously the USD importer incorrectly
compared curve offsets to USD point counts.

Also:
- Don't unnecessarily set the resolution to the incorrect default.
  The default for the curves system is 12. But it's unnecessary to
  fill the attribute to the default value anyway.
- Use some helper methods to iterate over all curves.
- Use float literal instead of integer.
2024-02-27 21:52:03 -05:00