dlr-gtlab / python-module

GTlab Module for embadding Python into GTlab
0 stars 0 forks source link

Resolve "Decorator to set object link properties has problems with heritage" - [merged] #558

Closed rainman110 closed 4 months ago

rainman110 commented 5 months ago

In GitLab by @jensschmeink on May 29, 2024, 10:24

Merges 279-decorator-to-set-object-link-properties-has-problems-with-heritage -> master

Closes #279

rainman110 commented 5 months ago

In GitLab by @jensschmeink on May 29, 2024, 10:46

added 1 commit

Compare with previous version

rainman110 commented 5 months ago

In GitLab by @jensschmeink on May 29, 2024, 10:56

Commented on CMakeLists.txt line 27

The find package is only needed for GTlab version < 2.0

rainman110 commented 5 months ago

In GitLab by @jensschmeink on May 29, 2024, 11:00

added 1 commit

Compare with previous version

rainman110 commented 5 months ago

In GitLab by @jensschmeink on Jun 4, 2024, 11:19

requested review from @rainman110

rainman110 commented 5 months ago

In GitLab by @jensschmeink on Jun 4, 2024, 11:19

marked this merge request as ready

rainman110 commented 5 months ago

In GitLab by @jensschmeink on Jun 4, 2024, 11:20

@rainman110 this MR is necessary as the python decorator decided which objects to accept in an object link property by its own and not by the latest impementation

rainman110 commented 5 months ago

approved this merge request

rainman110 commented 5 months ago

added 8 commits

Compare with previous version

rainman110 commented 5 months ago

reset approvals from @rainman110 by pushing to the branch

rainman110 commented 5 months ago

added 1 commit

Compare with previous version

rainman110 commented 5 months ago

added 1 commit

Compare with previous version

rainman110 commented 5 months ago

@jensschmeink Irgendwie baut das nicht mehr mit 1.7. Kannst du mal schecken? ich habe eigentlich nur die Cmake Änderungen Rückgängig gemacht (die sind schon im master) und gerebased.

rainman110 commented 5 months ago

In GitLab by @jensschmeink on Jun 5, 2024, 08:55

@rainman110 You have removed the ifdef for 2-0 and 1-7. There had been some extensions in the objectlink property which are not accessable in GTlab 1-7

rainman110 commented 5 months ago

Oh, I removed the commit regarding CMake. So I suppose the ifdefs were part of this commit :face_palm: :laughing: ?

rainman110 commented 5 months ago

In GitLab by @jensschmeink on Jun 5, 2024, 09:05

added 3 commits

Compare with previous version

rainman110 commented 5 months ago

In GitLab by @jensschmeink on Jun 5, 2024, 09:09

added 4 commits

Compare with previous version

rainman110 commented 5 months ago

In GitLab by @jensschmeink on Jun 5, 2024, 09:12

@rainman110 It's running now. Please check for approval.

rainman110 commented 5 months ago

approved this merge request

rainman110 commented 5 months ago

mentioned in commit a15cb9774f957d8034330ac8b4b40dcad0bc02df