add art command and button to mged #29
Open
Conversation
This file contains hidden or 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
brlcad
reviewed
May 4, 2022
Member
brlcad
left a comment
brlcad
left a comment
There was a problem hiding this comment.
There's a couple calls to do_Raytrace that were not updated (one in rt.tcl and another in mouse.tcl) and that's a problem as that appears to break rt. Documentation is needed as this introduces a new command. On the whole, this looks good for stubbing in a new 'art' libged command (except file header is wrong).
zhuodannychen
added a commit
to zhuodannychen/brlcad
that referenced
this pull request
May 9, 2023
Visual cropping
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Added functionality to call "art" (appleseed rendering) command from mged and added a button on Raytrace Control Panel inside mged to call "art".
The code calls the command correctly but it does not render with art because there is no -M option for art current. I am working on adding the -M option to art.
This code allows to call art the same way as we call rt from mged.