Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added all missing VisualServer bindings #15892

Merged
merged 1 commit into from
Feb 14, 2018

Conversation

Zylann
Copy link
Contributor

@Zylann Zylann commented Jan 20, 2018

  • Added bindings for multimesh, immediate, skeleton, light, reflection probe, gi probe, lightmap, particles, camera, environment, scenario, instance
  • Removed draw and sync, were duplicates of force_* equivalents
  • Bumped binders max arguments from 11 to 13
  • Wrote some wrappers as not all methods were variant-friendly

closes #15879

- Added bindings for multimesh, immediate, skeleton, light, reflection probe, gi probe, lightmap, particles, camera, environment, scenario, instance
- Removed draw and sync, were duplicates of force_* equivalents
- Bumped binders max arguments from 11 to 13
- Wrote some wrappers as not all methods were variant-friendly
@akien-mga akien-mga modified the milestones: 3.0, 3.1 Jan 20, 2018
@akien-mga akien-mga merged commit 574133e into godotengine:master Feb 14, 2018
@akien-mga
Copy link
Member

akien-mga commented Feb 14, 2018

Merged a bit fast and noticed this only afterwards:

Removed draw and sync, were duplicates of force_* equivalents

That's an API breakage, so:

  • this part should not be cherry-picked for 3.0.x
  • draw and sync should be readded in the master branch, with a deprecation warning if we want to remove them later on

@akien-mga
Copy link
Member

@hpvb Be sure to cherry-pick fd92e57 too which I just pushed to fix the above-mentioned API breakage.

akien-mga added a commit that referenced this pull request Feb 19, 2018
Those are deprecated as VS.force_sync and VS.force_draw do the same and more explicitly,
but we cannot remove them without marking them as deprecated before that.

Fixes issue introduced in #15892.
@hpvb hpvb removed the cherrypick label Feb 19, 2018
hpvb pushed a commit that referenced this pull request Feb 19, 2018
Those are deprecated as VS.force_sync and VS.force_draw do the same and more explicitly,
but we cannot remove them without marking them as deprecated before that.

Fixes issue introduced in #15892.

(cherry picked from commit fd92e57)
malcolmhoward pushed a commit to malcolmhoward/godot that referenced this pull request Jul 31, 2018
Those are deprecated as VS.force_sync and VS.force_draw do the same and more explicitly,
but we cannot remove them without marking them as deprecated before that.

Fixes issue introduced in godotengine#15892.

(cherry picked from commit fd92e57)
malcolmhoward pushed a commit to malcolmhoward/godot that referenced this pull request Jul 31, 2018
Those are deprecated as VS.force_sync and VS.force_draw do the same and more explicitly,
but we cannot remove them without marking them as deprecated before that.

Fixes issue introduced in godotengine#15892.

(cherry picked from commit fd92e57)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

VisualServer methods are not bound
3 participants