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

Allow for load in MaterialX documents from memory #45

Closed
kwokcb opened this issue Nov 24, 2023 · 0 comments · Fixed by #46
Closed

Allow for load in MaterialX documents from memory #45

kwokcb opened this issue Nov 24, 2023 · 0 comments · Fixed by #46
Labels
enhancement New feature or request

Comments

@kwokcb
Copy link
Contributor

kwokcb commented Nov 24, 2023

Proposal

This is a proposal to split up the current load from filename into 2 parts:

  1. Load from in memory document
  2. Load from filename which calls 1.

Requirement

This allows for an integration which loads in MaterialX documents on it's own to route this to QuiltIX. In my use case this is to load glTF files into MTLX documents and then updateing the QuiltiX graph. (See https://github.com/kwokcb/QuiltiX/blob/gltf_plugin/plugins/pluginQuiltX.py#L36 ).

Changes

The changes are relatively minor to split out the file read part of load_graph_from_mx_file into a load_graph_from_mx_doc (load from in memory document). The file handling part would stay in load_graph_from_mx_file which would call load_graph_from_mx_doc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants