Walk navigation don't use units from scene #69934

Closed
opened 2019-09-16 10:29:15 +02:00 by Alberto Velázquez · 11 comments

System Information
Operating system: windows 10
Graphics card:

Blender Version
Broken: Blender2.81 16-9-2019

Short description of error
When I use walk navigation in a scene with units changed to cm and scale 0.1 the walk doesn't work correctly. Walk use same units 1, than default, not 0.1 If I change the walk aprameters in the preference windows it doesn't respect the units and 170cm is 17meters

**System Information** Operating system: windows 10 Graphics card: **Blender Version** Broken: Blender2.81 16-9-2019 **Short description of error** When I use walk navigation in a scene with units changed to cm and scale 0.1 the walk doesn't work correctly. Walk use same units 1, than default, not 0.1 If I change the walk aprameters in the preference windows it doesn't respect the units and 170cm is 17meters

Added subscriber: @AlbertoVelazquez

Added subscriber: @AlbertoVelazquez
Member
Added subscribers: @WilliamReynish, @JacquesLucke, @lichtwerk
Member

Not sure I understand correctly:

  • Preferences > Navigation > Fly & Walk > Walk > Walk Speed is a value specified in absolute m/s
  • this setting is of type B_UNIT_VELOCITY
  • so setting the scene units to meter or centimeter has no influence here (and should not?)
  • so setting the scene Unit Scale to 0.1 (would make the default cube 20cm instead of 200cm) has the influence of walking 10 times as fast past that cube [Walk Speed is still in m/s]
  • to compensate for that, I would expect the user to set this to a 10th of that speed manually?
  • BUT, I can see that View Height / Jump Height compensates automatically.... true... (but this is of type B_UNIT_LENGTH)
  • so BKE_scene_unit_scale compensates for length, but does not for velocity (I think this is on purpose though...)

@JacquesLucke : velocity is excluded on purpose here, right?
CC @WilliamReynish

Not sure I understand correctly: - `Preferences` > `Navigation` > `Fly & Walk` > `Walk` > `Walk Speed` is a value specified in absolute `m/s` - this setting is of type `B_UNIT_VELOCITY` - so setting the scene units to meter or centimeter has no influence here (and should not?) - so setting the scene Unit Scale to 0.1 (would make the default cube 20cm instead of 200cm) has the influence of walking 10 times as fast past that cube [`Walk Speed` is still in m/s] - to compensate for that, I would expect the user to set this to a 10th of that speed manually? - BUT, I can see that `View Height` / `Jump Height` compensates automatically.... true... (but this is of type `B_UNIT_LENGTH`) - so `BKE_scene_unit_scale` compensates for length, but does not for velocity (I think this is on purpose though...) @JacquesLucke : velocity is excluded on purpose here, right? CC @WilliamReynish

I will try to explain better.

I have created this scene to test the problem

Blender_3.jpg

If I use the default 1.0 units, in meters, use the walk navigation and activate gravity (160cm in preferences) with Tab my pov look like this.

Blender_2.jpg

If I use 0.01 units, in cm, and I use the walk navigation and activate gravity (1,6cm in preferences) my pov look like this when the solution must be the same because the height in preferences is the same.

Blender.jpg

Other problems are generated with the change, like wrongs teleports...

I will try to explain better. I have created this scene to test the problem ![Blender_3.jpg](https://archive.blender.org/developer/F7751418/Blender_3.jpg) If I use the default 1.0 units, in meters, use the walk navigation and activate gravity (160cm in preferences) with Tab my pov look like this. ![Blender_2.jpg](https://archive.blender.org/developer/F7751420/Blender_2.jpg) If I use 0.01 units, in cm, and I use the walk navigation and activate gravity (1,6cm in preferences) my pov look like this when the solution must be the same because the height in preferences is the same. ![Blender.jpg](https://archive.blender.org/developer/F7751423/Blender.jpg) Other problems are generated with the change, like wrongs teleports...

Added subscriber: @brecht

Added subscriber: @brecht

In #69934#777009, @lichtwerk wrote:

  • so BKE_scene_unit_scale compensates for length, but does not for velocity (I think this is on purpose though...)

I don't think that's on purpose, and if it is, it's wrong to show one thing in the UI and then do something else. Fixed that part in c25f5a3.

> In #69934#777009, @lichtwerk wrote: > - so `BKE_scene_unit_scale` compensates for length, but does not for velocity (I think this is on purpose though...) I don't think that's on purpose, and if it is, it's wrong to show one thing in the UI and then do something else. Fixed that part in c25f5a3.

Changed status from 'Needs Developer To Reproduce' to: 'Confirmed'

Changed status from 'Needs Developer To Reproduce' to: 'Confirmed'

Added subscriber: @Vyach

Added subscriber: @Vyach

please merge with #66713 (Walk Teleport and other movements do not take into account the unit scale of the Scene)

please merge with #66713 (Walk Teleport and other movements do not take into account the unit scale of the Scene)
Member

Valid point, that report is older and about the same thing, will merge these reports.

Valid point, that report is older and about the same thing, will merge these reports.
Member

Closed as duplicate of #66713

Closed as duplicate of #66713
Sign in to join this conversation.
No Label
Interest
Alembic
Interest
Animation & Rigging
Interest
Asset Browser
Interest
Asset Browser Project Overview
Interest
Audio
Interest
Automated Testing
Interest
Blender Asset Bundle
Interest
BlendFile
Interest
Collada
Interest
Compatibility
Interest
Compositing
Interest
Core
Interest
Cycles
Interest
Dependency Graph
Interest
Development Management
Interest
EEVEE
Interest
EEVEE & Viewport
Interest
Freestyle
Interest
Geometry Nodes
Interest
Grease Pencil
Interest
ID Management
Interest
Images & Movies
Interest
Import Export
Interest
Line Art
Interest
Masking
Interest
Metal
Interest
Modeling
Interest
Modifiers
Interest
Motion Tracking
Interest
Nodes & Physics
Interest
OpenGL
Interest
Overlay
Interest
Overrides
Interest
Performance
Interest
Physics
Interest
Pipeline, Assets & IO
Interest
Platforms, Builds & Tests
Interest
Python API
Interest
Render & Cycles
Interest
Render Pipeline
Interest
Sculpt, Paint & Texture
Interest
Text Editor
Interest
Translations
Interest
Triaging
Interest
Undo
Interest
USD
Interest
User Interface
Interest
UV Editing
Interest
VFX & Video
Interest
Video Sequencer
Interest
Virtual Reality
Interest
Vulkan
Interest
Wayland
Interest
Workbench
Interest: X11
Legacy
Blender 2.8 Project
Legacy
Milestone 1: Basic, Local Asset Browser
Legacy
OpenGL Error
Meta
Good First Issue
Meta
Papercut
Meta
Retrospective
Meta
Security
Module
Animation & Rigging
Module
Core
Module
Development Management
Module
EEVEE & Viewport
Module
Grease Pencil
Module
Modeling
Module
Nodes & Physics
Module
Pipeline, Assets & IO
Module
Platforms, Builds & Tests
Module
Python API
Module
Render & Cycles
Module
Sculpt, Paint & Texture
Module
Triaging
Module
User Interface
Module
VFX & Video
Platform
FreeBSD
Platform
Linux
Platform
macOS
Platform
Windows
Priority
High
Priority
Low
Priority
Normal
Priority
Unbreak Now!
Status
Archived
Status
Confirmed
Status
Duplicate
Status
Needs Info from Developers
Status
Needs Information from User
Status
Needs Triage
Status
Resolved
Type
Bug
Type
Design
Type
Known Issue
Type
Patch
Type
Report
Type
To Do
No Milestone
No project
No Assignees
5 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender#69934
No description provided.