-
Notifications
You must be signed in to change notification settings - Fork 164
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
Implement hstack and vstack #506
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@simmplecoder I just made a few tiny comments. Apart from that, for the purpose of stacking image views it looks fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should cover all new functions/functionality with tests
I just realised these are examples!
Is this PR related to any of the GSoC 2020 work?
@mloskot it is a convenience tool to make putting input and result in perspective easier. It could use a lot of work, but I thought we could do by it for GSoC. Anyway, this is not really needed, but would be a good addition until a better way is found. |
@simmplecoder Sounds good. Let's consider it as part of GSoC as part of the whole toolkit. Could you address @codejaeger 's suggestions? |
@mloskot , sure, will ping you back when ready. |
Width can be different in hstack, height can be different in vstack
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Allow non-equal dims along stack dim Width can be different in hstack, height can be different in vstack
Allow non-equal dims along stack dim Width can be different in hstack, height can be different in vstack
Description
Adds ability to stack images either horizontally or vertically. Provides output view argument form and image returning form for convenience. Does full input checking. This is rough version only, not intended for usage outside writing examples.
References
https://cpplang.slack.com/archives/CSVT0STV2/p1594414134118400
Tasklist