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

some improvements on handling linked cameras #3

Merged
merged 5 commits into from
Aug 16, 2012
Merged

Conversation

tinitron
Copy link
Contributor

create atknob only when linking cameras
no autokeyframe
changed to Int_Knob like other frame knobs in Nuke
added button to set to current frame like in the Tracker node
added inpanel option for camera creation so that the propertybin does not get filled up when creating projection alley

unfortunately contains also commit of my attempt on bugfixing

Michael Habenicht added 2 commits August 16, 2012 11:54
project3d node creation with nuke.createNode instead of nuke.nodes, fixes bug that all project3d nodes are connected to the last camera created
no autokeyframe
changed to Int_Knob like other frame knobs in Nuke
added button to set to current frame like in the Tracker node
added inpanel option for camera creation so that the propertybin does not get filled up when creating projection alley
@julik
Copy link
Owner

julik commented Aug 16, 2012

Hey this is awesome, but can you first merge my master in and then reapply this on top? I also removed the all_nodes thing like you did in 8332d3d

I don't really like the "Inpanel" thing but I can handle it later.

TIA

Michael Habenicht and others added 3 commits August 16, 2012 15:16
no autokeyframe
changed to Int_Knob like other frame knobs in Nuke
added button to set to current frame like in the Tracker node
inpanel option removed as it only makes sense when using nuke.createNode
no autokeyframe
changed to Int_Knob like other frame knobs in Nuke
added button to set to current frame like in the Tracker node
inpanel option removed as it only makes sense when using nuke.createNode
@tinitron
Copy link
Contributor Author

Should now work with the new commits

julik added a commit that referenced this pull request Aug 16, 2012
@julik julik merged commit 53f4c96 into julik:master Aug 16, 2012
@julik
Copy link
Owner

julik commented Aug 16, 2012

Hey Michael, thanks for your contribs, all merged!

Please consider switching your editor to UNIX linebreaks. I know projectionist uses tabs now which is against the recommendations but I will change that soon.

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

Successfully merging this pull request may close these issues.

2 participants