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.

Sign up for GitHub

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

Jump to bottom

[Feat] r.univar: implement OpenMP parallelization also for extended statistics #2564

Closed
petrasovaa opened this issue Aug 30, 2022 · 3 comments
Closed

[Feat] r.univar: implement OpenMP parallelization also for extended statistics #2564

petrasovaa opened this issue Aug 30, 2022 · 3 comments
Labels
C Related code is in C enhancement New feature or request
Milestone
8.4.0

Comments

@petrasovaa
Copy link
Contributor

r.univar uses OpenMP parallelization, but only without -e (extended statistics). It would be good to add the parallelization support for this option as well. See also #1634.
@petrasovaa petrasovaa added enhancement New feature or request C Related code is in C labels Aug 30, 2022
@cwhite911
Copy link
Contributor

cwhite911 commented Sep 1, 2022 •
edited
Loading

@petrasovaa I'm working on adding json output for r.univar. Do you want me to add the parallelization support too or have you already started?

@petrasovaa
Copy link
Contributor Author

That may not be that simple, I am not planning to work on this anytime soon.

@aaronsms aaronsms mentioned this issue Dec 5, 2022
Merged
@petrasovaa petrasovaa added this to the 8.4.0 milestone Feb 5, 2024
@petrasovaa
Copy link
Contributor Author

Fixed by #2683.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C Related code is in C enhancement New feature or request
Projects
None yet
Milestone
8.4.0
Development

No branches or pull requests
2 participants
@cwhite911 @petrasovaa