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 :select-drawmode method to camera-model #309

Merged
merged 2 commits into from
Nov 28, 2015

Conversation

YoheiKakiuchi
Copy link
Member

This PR is camera-model version of #308

@k-okada
Copy link
Member

k-okada commented Nov 23, 2015

Please add sample code

◉ Kei Okada

2015/11/23 21:10、Yohei Kakiuchi notifications@github.com のメッセージ:

This PR is camera-model version of #308

You can view, comment on, or merge this pull request online at:

#309

Commit Summary

add :select-drawmode method to camera-model
File Changes

M irteus/irtsensor.l (15)
Patch Links:

https://github.com/euslisp/jskeus/pull/309.patch
https://github.com/euslisp/jskeus/pull/309.diff

Reply to this email directly or view it on GitHub.

@YoheiKakiuchi
Copy link
Member Author

Add code for generating hidden-line image to demo program.

hidden-line-camera-image

@@ -356,6 +356,21 @@
)))
img
))
(:select-drawmode (mode objs)
Copy link
Member

Choose a reason for hiding this comment

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

please add documentation

Copy link
Member

Choose a reason for hiding this comment

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

if this is 'public' method

Copy link
Member Author

Choose a reason for hiding this comment

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

add document

YoheiKakiuchi added a commit that referenced this pull request Nov 28, 2015
add :select-drawmode method to camera-model
@YoheiKakiuchi YoheiKakiuchi merged commit 0e455a8 into euslisp:master Nov 28, 2015
@YoheiKakiuchi YoheiKakiuchi deleted the add_drawmode_camera branch December 15, 2015 05:27
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