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

ndslice: concatenate/vstack/hstack [$15 awarded] #213

Closed
wilzbach opened this issue May 15, 2016 · 8 comments · Fixed by libmir/mir-algorithm#28
Closed

ndslice: concatenate/vstack/hstack [$15 awarded] #213

wilzbach opened this issue May 15, 2016 · 8 comments · Fixed by libmir/mir-algorithm#28

Comments

@wilzbach
Copy link
Member

wilzbach commented May 15, 2016

Idea: Join multiple ndslices to a bigger ndslice
Maybe sth. like:

auto a = iotaSlice(2, 4);
auto b = 2.repeat.sliced(2, 4);
a.concatenate(b);

(Forum discussion)


The $15 bounty on this issue has been claimed at Bountysource.

@9il
Copy link
Member

9il commented May 15, 2016

next week plan:

  • selection.elementsMap - shell with opIndex, length and (optional) save methods.
  • selection.concatenated - concatenation without allocation based on elementsMap
  • slice.elements / slice.makeElements - allocates a common array composed of slice elements.
  • slice.concatenation / ``slice.makeConcatenation` - multidimensional concatenation with allocation

@wilzbach
Copy link
Member Author

@9il
Copy link
Member

9il commented May 15, 2016

but maybe we can have this as template argument

yes

@Marenz
Copy link

Marenz commented Sep 17, 2016

Seeing that this was created May 15th and reading "next week plan" leads me to the suspicion that the issue might have been forgotten. So I figure a minor ping might be in order. ping

@9il
Copy link
Member

9il commented Sep 18, 2016

@Marenz GLAS and ndslice.algorithm just have higher priority.

@Marenz
Copy link

Marenz commented Sep 18, 2016

Sure, no pressure, I just pinged in case it was forgotten ;)
Thanks for the work!

@9il 9il changed the title ndslice: concatenate/vstack/hstack ndslice: concatenate/vstack/hstack [$15] Sep 18, 2016
@9il 9il added the bounty label Sep 18, 2016
@9il
Copy link
Member

9il commented Sep 18, 2016

@Marenz Thank you for the first Mir bounty! Currently we have issue with bountyscore, because Mir project is registered for DlangScience instead of Mir team. I hope we will fix it #345 soon

@9il 9il changed the title ndslice: concatenate/vstack/hstack [$15] ndslice: concatenate/vstack/hstack Sep 18, 2016
@9il 9il removed the bounty label Sep 18, 2016
@9il 9il changed the title ndslice: concatenate/vstack/hstack ndslice: concatenate/vstack/hstack [$15] Sep 18, 2016
@9il 9il added the bounty label Sep 18, 2016
@9il
Copy link
Member

9il commented Sep 18, 2016

Bounty is located in https://www.bountysource.com/teams/libmir now. I have sent query to get access for this bountysource team.

@9il 9il changed the title ndslice: concatenate/vstack/hstack [$15] ndslice: concatenate/vstack/hstack [$15 awarded] Mar 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants