Skip to content

Change model() example#3074

Merged
almchung merged 1 commit intoprocessing:masterfrom
TanviKumar:test-io
Jul 20, 2018
Merged

Change model() example#3074
almchung merged 1 commit intoprocessing:masterfrom
TanviKumar:test-io

Conversation

@TanviKumar
Copy link
Copy Markdown
Member

The current model() example has the same error the previous loadModel() example had. This PR fixes that issue by using the octahedron example instead.

@TanviKumar TanviKumar requested a review from almchung July 9, 2018 15:53
Copy link
Copy Markdown

@almchung almchung left a comment

Choose a reason for hiding this comment

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

Looks good. Do we want to keep the old example anyway, Kate? @kjhollen

@kjhollen
Copy link
Copy Markdown
Member

@almchung I think it's ok to just have one example here, assuming this links to loadModel anyway!

@almchung almchung merged commit 6bffb17 into processing:master Jul 20, 2018
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.

3 participants