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

add fbx export for static mesh #75

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Conversation

xoyojank
Copy link

No description provided.

@gildor2
Copy link
Owner

gildor2 commented Jun 19, 2018

Hi,
Is this occasional pull request, or you intended to do that?

@xoyojank
Copy link
Author

intended

@gildor2
Copy link
Owner

gildor2 commented Jun 19, 2018

Thank you for contribution, and sorry - I'm not going to merge this. There are many reasons for that, the main are:

  1. I'm not going to include fbx sdk into umodel's source code, it's huge (especially if including platforms other than Windows), entirely proprietary.
  2. Your code is copy-paste from Unreal engine 4. It is great that it can be compiled in UModel's environment with minimal changes, but I can't use that code.
  3. I do not like fbx format and sdk, it is terrible,

Other comments to this PR:

  • You broke my build system, I'm not going to migrate to Visual Studio build - this won't work with Linux, and it is not so user-friendly (for me) as my system.
  • There's a lot of place where you changed my coding style with no reason.

Personally I'd like to say the following. I worked a lot with Fbx sdk in past 2 years, so it would not be hard for me to write StaticMesh exporter by myself, from scratch (without use of UE4 sources). It is much much harder to work with SkeletalMesh and animation. Also, it is not possible to support just statics here - it is pointless (there are many better formats which supports static meshes), and users will demand more. In the future I'm going to support glTF format, I like it much more than fbx. It's probably a few hours of work to implement export of static meshes into glTF.

@xoyojank
Copy link
Author

I see, thanks for your reply.

@PaulNevada
Copy link

And how can I get exported content move to my UE4 project?

@gildor2
Copy link
Owner

gildor2 commented Jun 16, 2019

Two ways.

  1. Use UModel's gltf export. You may import static meshes with built-in plugin (it imports only static meshes). Also, there's a third-party plugin which imports all kinds of gltf available.
  2. As other people do, use 3ds Max or Blender as intermediate product, which imports psk/psa files and converts them to fbx.

@PaulNevada
Copy link

Thanks!
Another question is why does umodel often miss the needed textures and replace them with the null-ones so to speak?

@xoyojank
Copy link
Author

xoyojank commented Jun 24, 2019

Two ways.

  1. Use UModel's gltf export. You may import static meshes with built-in plugin (it imports only static meshes). Also, there's a third-party plugin which imports all kinds of gltf available.
  2. As other people do, use 3ds Max or Blender as intermediate product, which imports psk/psa files and converts them to fbx.

there are some bugs of gltf plugin for UE4, so I think gltf is not ready for use in production pipeline now.

@gildor2
Copy link
Owner

gildor2 commented Jun 24, 2019

Another question is why does umodel often miss the needed textures and replace them with the null-ones so to speak?

This is a FAQ question, not a "github issue".

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.

3 participants