godot icon indicating copy to clipboard operation
godot copied to clipboard

Update Node.xml: specify that normal processing happens in tree order

Open 0x53A opened this issue 1 year ago • 5 comments

For _ready it was already spelled out in a few places that it happens bottom to top (child first parent later). I found it hard to find information about the order of _process and _physics_process, the only place I found was here in the tutorial for SceneTree which someone linked me to.

0x53A avatar Oct 06 '24 23:10 0x53A

Thank you for fixing the typo!

I found that my suggested change is incomplete, for _process and _physics_process, there are the properties process_priority and process_physics_priority, respectively.

0x53A avatar Oct 13 '24 08:10 0x53A

Edit: When Priority is changed inside a process callback, the order will be wrong for this one iteration, then correct at the next iteration. That should be fine.

0x53A avatar Oct 13 '24 09:10 0x53A

I'd say this is a bit too complex to describe in the class docs here, and should be a detailed page in the docs instead, and linked to

Related:

  • https://github.com/godotengine/godot-docs/issues/9305

AThousandShips avatar Oct 13 '24 09:10 AThousandShips

I've removed it from the top paragraph since as you said that would get pretty long, mentioning both priority and tree order.

Personally I would prefer to still add it to the actual method documentation, if only for parity with _ready, but the call is obviously yours.

To be fully honest I don't have the knowledge, time, or motivation to create the full processing diagram requested in the issue.

0x53A avatar Oct 13 '24 09:10 0x53A

The commits need to be squashed into one before this can be merged. See here for one explanation on how to do do that.

Repiteo avatar Oct 16 '24 18:10 Repiteo

Thanks!

Repiteo avatar Oct 21 '24 21:10 Repiteo