Skip to content
This repository has been archived by the owner on Dec 21, 2023. It is now read-only.

Style Transfer stylize docstring contradicts itself #2789

Closed
TobyRoseman opened this issue Dec 4, 2019 · 1 comment · Fixed by #2864
Closed

Style Transfer stylize docstring contradicts itself #2789

TobyRoseman opened this issue Dec 4, 2019 · 1 comment · Fixed by #2864
Assignees
@TobyRoseman
Labels
docs style transfer toolkits
Milestone
6.1

Comments

@TobyRoseman
Copy link
Collaborator

The parameter section of the stylize docstring says the image parameter should be an SFrame or an image. However the Returns section of the docstring seems to say it could also be an SArray, but it's really not clear. Looking at the code it does seem like an SArray should be an accepted type for image.
@TobyRoseman TobyRoseman added this to the 6.1 milestone Dec 10, 2019
@TobyRoseman TobyRoseman self-assigned this Dec 16, 2019
@TobyRoseman
Copy link
Collaborator Author

SArray is definitely supported here. We even have a unit test for it.

@TobyRoseman TobyRoseman mentioned this issue Dec 16, 2019
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@TobyRoseman TobyRoseman

Labels
docs style transfer toolkits
Projects
None yet
Milestone
6.1
Development

Successfully merging a pull request may close this issue.

Minor Fixes for Style Transfer TobyRoseman/turicreate
1 participant
@TobyRoseman