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

Implement hstack and vstack #506

Merged
merged 2 commits into from
Jan 24, 2021
Merged

Conversation

simmplecoder
Copy link
Contributor

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

  • Ensure all CI builds pass
  • Review and approve

@simmplecoder simmplecoder requested review from lpranam and mloskot July 12, 2020 20:02
Copy link
Contributor

@codejaeger codejaeger left a 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.

Copy link
Member

@mloskot mloskot left a 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 mloskot added the cat/feature New feature or functionality label Aug 24, 2020
@simmplecoder
Copy link
Contributor Author

@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.

@mloskot
Copy link
Member

mloskot commented Aug 24, 2020

@simmplecoder Sounds good. Let's consider it as part of GSoC as part of the whole toolkit.

Could you address @codejaeger 's suggestions?
If this is something to consider for future improvements, then it may be worth to add them as // TODO: comments.

@simmplecoder
Copy link
Contributor Author

@mloskot , sure, will ping you back when ready.

Width can be different in hstack,
height can be different in vstack
@mloskot mloskot added this to the Boost 1.76+ milestone Jan 24, 2021
@mloskot mloskot added the google-summer-of-code All items related to GSoC activities label Jan 24, 2021
Copy link
Member

@mloskot mloskot left a comment

Choose a reason for hiding this comment

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

LGTM

@mloskot mloskot merged commit b82aed8 into boostorg:develop Jan 24, 2021
meshtag pushed a commit to meshtag/gil that referenced this pull request Apr 21, 2021
Allow non-equal dims along stack dim
Width can be different in hstack, height can be different in vstack
meshtag pushed a commit to meshtag/gil that referenced this pull request Apr 22, 2021
Allow non-equal dims along stack dim
Width can be different in hstack, height can be different in vstack
@mloskot mloskot mentioned this pull request May 12, 2022
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cat/feature New feature or functionality google-summer-of-code All items related to GSoC activities
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants