BGE 2.50: replace all material properties (strings) by datablock pointers (Material *) - (may affect ext. blend libraries) #22482

New Issue

Dalai Felinto · 2010-06-02T08:57:56+02:00

Dalai Felinto commented

2010-06-02 08:57:56 +02:00

%%%Hello there,
I just (finally) finished a patch to have materials - [x] and actuators

properties in Logic to use datablocks pointers.

There are two main advantages of this:
a) we can have nice lookups for the existent materials/actuators.
b) if you change a material name, everything get updated.

as a reference this is how Touch Sensor works already (since 2.49)

The downside of this is that I have no idea how well this will work
with external .blend "library" files. I tested with simple linked data
and it works fine.
However as soon as a material is no longer in the library the
sensor/actuator will not detect it anymore (no material == no
pointer).
I don't have experience using external library files with BGE, so I
would like some help to test it or at least to think if this is a real
drawback or not.

If this kind of flexibility is necessary, the alternative I see if to
add datablock lookups stored as strings (as we do for game properties,
bone and pose channel). And on top of that make the Touch Sensor to
use strings instead of pointers (for consistency sake).

- Ray Sensor, Collision Sensor and Constraint Actuator
- Actuator Sensor

Posted on bf-committers, but after talking with Campbell I agreed that it could wait until we have more games ported to 2.50 (to avoid new bugs).
The patch went through exhaustive tests though, it should be working fine.%%%

%%%Hello there, I just (finally) finished a patch to have materials - [x] and actuators - [x] properties in Logic to use datablocks pointers. There are two main advantages of this: a) we can have nice lookups for the existent materials/actuators. b) if you change a material name, everything get updated. * as a reference this is how Touch Sensor works already (since 2.49) The downside of this is that I have no idea how well this will work with external .blend "library" files. I tested with simple linked data and it works fine. However as soon as a material is no longer in the library the sensor/actuator will not detect it anymore (no material == no pointer). I don't have experience using external library files with BGE, so I would like some help to test it or at least to think if this is a real drawback or not. If this kind of flexibility is necessary, the alternative I see if to add datablock lookups stored as strings (as we do for game properties, bone and pose channel). And on top of that make the Touch Sensor to use strings instead of pointers (for consistency sake). - [x] - Ray Sensor, Collision Sensor and Constraint Actuator - [x] - Actuator Sensor **** Posted on bf-committers, but after talking with Campbell I agreed that it could wait until we have more games ported to 2.50 (to avoid new bugs). The patch went through exhaustive tests though, it should be working fine.%%%

Dalai Felinto commented

2010-06-02 08:57:56 +02:00

Changed status to: 'Open'

Mitchell Stokes commented

2014-07-04 10:27:30 +02:00

Added subscriber: @Moguri

Mitchell Stokes commented

2014-07-04 10:27:30 +02:00

Working on cleaning out the patch tracker:

Is this something still being worked on, or should this report be closed? If there is no response in a week, I will close this report. Active patches should start being migrated to Differential.

Working on cleaning out the patch tracker: Is this something still being worked on, or should this report be closed? If there is no response in a week, I will close this report. Active patches should start being migrated to Differential.

Dalai Felinto commented

2014-07-07 16:37:34 +02:00

Added subscriber: @brita

Dalai Felinto commented

2014-07-07 16:37:34 +02:00

This was an internal patch, no new functionality to the user. @brita can tackle this if she feels like, but I don't even know if there is a real benefit to the user.

Inês Almeida commented

2014-07-11 23:06:44 +02:00

Added subscriber: @kupoman

Inês Almeida commented

2014-07-11 23:06:44 +02:00

@dfelinto you mention 2 advantages no? The task looks like a good learning opportunity... @kupoman?

Aaron Carlisle commented

2018-09-29 15:47:19 +02:00

Changed status from 'Open' to: 'Archived'

Aaron Carlisle closed this issue

2018-09-29 15:47:19 +02:00

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

BGE 2.50: replace all material properties (strings) by datablock pointers (Material *) - (may affect ext. blend libraries) #22482