-
Notifications
You must be signed in to change notification settings - Fork 742
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
Slate sequencer upgrade pr #445
Open
ikrima
wants to merge
93
commits into
20tab:master
Choose a base branch
from
kitelightning:SlateSequencerUpgrade-PR
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…iptstructs which are mutable/reference structs -Without it, slate argument binding has errors bc it tries to bind delegates to the uscriptstructs
# Conflicts: # Source/UnrealEnginePython/Public/UnrealEnginePython.h
FIX: Rename variable used in PY_RETURN_UOBJECT to __ret to avoid name collision
…PyEngine and removed it from UEPyEditor so that it's usable in standalone scripts. Fix various build ordering/ifdef issues for shipping Add clear_event function for multicasting
…r, not value. Accidentally making copies and not updating the actual delegate
…that it's enumerable as a USTRUCT -Adding py_ue_ftransform_get() helper to retrieve FTransform from ue_PyUScriptStruct as we're not storing it anymore directly -Adding new_ftransform_ptr variant that's meant to point to original struct to allow ref struct syntax -Adding global support for returning original data ptr for ref struct or not. Can disable it globally now by just changing py_ue_uscriptstruct_get_data() to return the new data NOTE: Had to explicitly specify ue_PyFTransformType.tp_call; was not picking it up from the base ue_PyUScriptStructType. Not sure if this means the subclassing wasn't set up properly and that other important things like basae.tp_dealloc won't get called
-Handles intiailization of ustruct, copying from existing data, and whether to hold on to ref or not -Needed to allow other explicit types such as FVector/FRotator/etc to use same code path
…r & FTransform NOTE: We just wrote out to original pointer block; now we need to update our local shadow copy This might be unnecessary
-Also adding TBaseStructure<FQuat>
…_init() like with ftransform/fvector FIX: PyFRotator not properly initializing it's member variables when passed in a single parameter
# Conflicts: # Source/UnrealEnginePython/Private/Slate/UEPySPythonMultiColumnTableRow.h # Source/UnrealEnginePython/Private/UEPyModule.cpp
…id struct by name Add ability to pass flags when creating new uobject using class name e.g. AnimMetaData('name', packge, outer, RF_PUBLIC|RF_STANDALONE)
# Conflicts: # Source/UnrealEnginePython/Private/Slate/UEPySPythonTreeView.cpp # Source/UnrealEnginePython/Private/Slate/UEPySlate.cpp # Source/UnrealEnginePython/Private/UEPyUScriptStruct.cpp
-Add enum type for native enums mostly used in slate widgets -Added codegen to enable quickly adding new ones -FIX: Memory corruption with incorrect delete in ListView -SPythonCombobox: Allow updating to options source list -Move Combobox to use same structure as SPythonListView (ue_PySPythonComboBox owns ref to the combobox TArray list) -Add ComboBoxStyle -Fixing SSplitter's slot's parameters & on_slot_resized -Making EUserInterfaceActionType a native enum in python Python Plugin: Add Set Folder Path function
added actor_get_level_sequence and class_has_any_flags added class_has_any_flags to check for flags on UClass added py_ue_static_mesh_get_num_triangles to UEPyStaticMesh added py_ue_sequencer_get_all_spawnables tp UEPySequencer added actor_component_set_can_ever_affect_navigation to UEPyActor
-Added EDataValidationResult shim -Exposed py_ue_is_data_valid
# Conflicts: # Source/UnrealEnginePython/Private/Slate/UEPyFMenuBuilder.cpp # Source/UnrealEnginePython/Private/Slate/UEPyFToolBarBuilder.cpp # Source/UnrealEnginePython/Private/Slate/UEPySCheckBox.cpp # Source/UnrealEnginePython/Private/Slate/UEPySDockTab.cpp # Source/UnrealEnginePython/Private/Slate/UEPySPythonComboBox.cpp # Source/UnrealEnginePython/Private/Slate/UEPySPythonListView.cpp # Source/UnrealEnginePython/Private/Slate/UEPySPythonMultiColumnTableRow.cpp # Source/UnrealEnginePython/Private/Slate/UEPySPythonMultiColumnTableRow.h # Source/UnrealEnginePython/Private/Slate/UEPySPythonTreeView.cpp # Source/UnrealEnginePython/Private/Slate/UEPySSplitter.cpp # Source/UnrealEnginePython/Private/Slate/UEPySTextBlock.cpp # Source/UnrealEnginePython/Private/Slate/UEPySWidget.cpp # Source/UnrealEnginePython/Private/Slate/UEPySlate.cpp # Source/UnrealEnginePython/Private/Slate/UEPySlate.h
slate memory corruption: ue_PySWidget::weakreflist member was not being initialized to nullptr in py_ue_new_swidget() add_viewport_widget_content: Pass in the dropped z_order param widget: re-introduce py_dict; useful to have base python widget be a dictionary as it allows for easy extensions to slate widgets in python with new attributes py_ue_swindow_set_on_window_closed: Use NewSlateDelegate vs NewStaticSlateDelegate so delegate tracker gets GC'ed py_ue_sdock_tab_set_on_tab_closed: same as above
… a context object
…gic of using DeferredSlateDelegate tracker -FIX: Added ue_py_slate_track_delegates() macro to manually add deferred delegates to tracker Other cosmetic code fixes as well (wrapping single line if statements with braces)
# Conflicts: # Source/UnrealEnginePython/Private/Fbx/UEPyFbxManager.cpp # Source/UnrealEnginePython/Private/Fbx/UEPyFbxObject.cpp # Source/UnrealEnginePython/Private/Slate/UEPyFSlateStyleSet.cpp # Source/UnrealEnginePython/Private/Slate/UEPySDockTab.cpp # Source/UnrealEnginePython/Private/Slate/UEPySPythonShelf.cpp # Source/UnrealEnginePython/Private/Slate/UEPySlate.h # Source/UnrealEnginePython/Private/Slate/UEPySlateDelegate.h # Source/UnrealEnginePython/Private/UEPySubclassing.cpp # Source/UnrealEnginePython/Private/UObject/UEPyStaticMesh.cpp # Source/UnrealEnginePython/Private/UnrealEnginePythonPrivatePCH.h # Source/UnrealEnginePython/Private/Wrappers/UEPyESlateEnums.cpp # Source/UnrealEnginePython/Private/Wrappers/UEPyFQuat.cpp
# Conflicts: # Source/UnrealEnginePython/Private/Slate/UEPySNumericEntryBox.cpp # Source/UnrealEnginePython/Private/Slate/UEPySPythonListView.cpp # Source/UnrealEnginePython/Private/Slate/UEPySPythonListView.h # Source/UnrealEnginePython/Private/Slate/UEPySPythonMultiColumnTableRow.cpp # Source/UnrealEnginePython/Private/Slate/UEPySWidget.h # Source/UnrealEnginePython/Private/Slate/UEPySlate.h # Source/UnrealEnginePython/Private/UEPyModule.cpp # Source/UnrealEnginePython/Private/UEPySubclassing.cpp # Source/UnrealEnginePython/Private/Wrappers/UEPyFAssetData.cpp # Source/UnrealEnginePython/Private/Wrappers/UEPyFQuat.h # Source/UnrealEnginePython/Private/Wrappers/UEPyFVector.h
# Conflicts: # Source/UnrealEnginePython/Private/Slate/UEPyIStructureDetailsView.cpp # Source/UnrealEnginePython/Private/Slate/UEPySlateDelegate.h # Source/UnrealEnginePython/Private/UEPyEngine.cpp # Source/UnrealEnginePython/Private/Wrappers/UEPyFEditorViewportClient.cpp
# Conflicts: # Source/UnrealEnginePython/Private/Slate/UEPySWidget.h # Source/UnrealEnginePython/Public/PythonBlueprintFunctionLibrary.h
# Conflicts: # Source/UnrealEnginePython/Private/Slate/UEPySHorizontalBox.cpp # Source/UnrealEnginePython/Private/Slate/UEPySVerticalBox.cpp # Source/UnrealEnginePython/Private/Slate/UEPySlate.h
# Conflicts: # Source/UnrealEnginePython/Private/UEPyModule.cpp # Source/UnrealEnginePython/Private/Wrappers/UEPyESlateEnums.cpp
# Conflicts: # UnrealEnginePython.uplugin
-Adding missing includes for standalone builds -Moving native enum shims into editor only
-FVector2D() does not initialize the struct
…criptstructs -Extend properties() to allow for EFieldIteratorFlags
# Conflicts: # Source/UnrealEnginePython/Private/UnrealEnginePython.cpp
Python bug fixes on slate attribute macro Add editor_refresh_all_browsers
# Conflicts: # Source/UnrealEnginePython/UnrealEnginePython.Build.cs
rdeioris
pushed a commit
that referenced
this pull request
Jun 30, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Features: Sequencer & Slate widgets/extensions
This is a painful PR so to make it easier, I made several merge points from at logical points to chunk the integration and help track changes. This doesn't include the latest merge with the threading refactor because it deadlocks easily (from mainbranch) so couldn't verify my merged changes work correctly. I left it up though here: https://github.com/kitelightning/UnrealEnginePython/tree/buggy-threading
Specific questions/feedback request
UEPySPythonComboBox.cpp: 170
//TODO: ikrimae: #ThirdParty-Python: #BUG: We are on purpose not doing Py_DECREF(values) because we're stealing the reference from _GetIter
//But we never decref values in the dealloc function. We should store a py_ref to the python list
//Ask roberto for the new refactored way for this
UEPySPythonListView.cpp: 155
//TODO: ikrimae: #ThirdParty-Python: #BUG: We are on purpose not doing Py_DECREF(values) because we're stealing the reference from _GetIter
//But we never decref values in the dealloc function. We should store a py_ref to the python list
//Ask roberto for the new refactored way for this
UEPyModule.cpp:2590:
//TODO: ikrimae: #ThirdParty-Python: Not sure if this will auto cleanup when the function parameters are destroyed or if the GWorld owner will force it to keep alive
UEPyModule.cpp:2608:
//TODO: ikrimae: #ThirdParty-Python: Not sure if this will auto cleanup when the function parameters are destroyed or if the GWorld owner will force it to keep alive
PythonHouseKeeper.h:275
//TODO: ikrimae: #ThirdParty-Python: #BUG: This implementation memory leaks. These delegates never get cleaned up