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.

Sign up for GitHub

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

Jump to bottom

320 introduce houdini generic loader hdas with examples for users #321

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

320 introduce houdini generic loader hdas with examples for users #321

wants to merge 10 commits into from
+243 −1

Conversation

MustafaJafar
Copy link
Contributor

@MustafaJafar MustafaJafar commented Dec 23, 2024 •
edited
Loading

Changelog Description

resolve #320
Add docs for AYON Loader HDAs as well as some examples.
TODO:

Testing notes:

  1. Go through docs, approve if changes make sense to you.

@MustafaJafar MustafaJafar self-assigned this Dec 23, 2024
@MustafaJafar MustafaJafar marked this pull request as draft December 23, 2024 13:47
@MustafaJafar MustafaJafar mentioned this pull request Dec 23, 2024
Open
8 tasks
@MustafaJafar MustafaJafar marked this pull request as ready for review January 10, 2025 00:11
@MustafaJafar MustafaJafar requested a review from moonyuet January 10, 2025 00:12
@moonyuet
Copy link
Member

@MustafaJafar Is this doc finished? I am asking this as I see there is still one item unchecked in your changelog,

@MustafaJafar
Copy link
Contributor Author

@MustafaJafar Is this doc finished? I am asking this as I see there is still one item unchecked in your changelog,

The last item is only about adding descriptions to mentioned node parameters. I'm going to push a commit to add them shortly.

@MustafaJafar MustafaJafar requested a review from moonyuet January 10, 2025 12:19
moonyuet
moonyuet approved these changes Jan 11, 2025
View reviewed changes
Copy link
Member

@moonyuet moonyuet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MustafaJafar
Copy link
Contributor Author

Note:
This PR should be merged after ynput/ayon-houdini#199

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@moonyuet moonyuet moonyuet approved these changes

@BigRoy BigRoy Awaiting requested review from BigRoy

Assignees

@MustafaJafar MustafaJafar

Labels
addon: Houdini Artist Docs type: enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Introduce Houdini Generic Loader HDAs with examples for users
2 participants
@MustafaJafar @moonyuet