Merged
Conversation
Collaborator
kpchoi
commented
Mar 18, 2025
kpchoi
commented
- add oapve_param_parse() API newly.
Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com>
Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com>
Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com>
Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com>
Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com>
Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com>
Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com>
Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com>
…eFoundation/openapv into oapve_param_parse
- Fix compile error on mac env
Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com>
Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com>
Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com>
Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com>
- remove unused variable in *_neon.c Signed-off-by: Yangwoo Kim <yearly.kim@samsung.com>
dkozinski
reviewed
Mar 19, 2025
dkozinski
reviewed
Mar 19, 2025
Collaborator
dkozinski
left a comment
dkozinski
left a comment
There was a problem hiding this comment.
There is a lack of the ability to set the number of threads through the public API (opav.h). Could the option to configure the number of threads be added via the oapv_param_parse function?
Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com>
alejandro-arango-epicgames
pushed a commit
to alejandro-arango-epicgames/openapv
that referenced
this pull request
Sep 29, 2025
* modification of decoding tile * refactoring Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com> * refactoring Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com> * added xeve_param_parse() API Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com> * fix for arm buiding Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com> * fix typo Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com> * added 'preset' to oapve_param_parse() Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com> * added most parameters Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com> * added param.c and param.h and detection of num cpu core Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com> * modification to compile on window * removed wrong end of line Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com> * oapv_get_num_cpu_cores() fix for mac - Fix compile error on mac env * introduced ctx->threads Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com> * modification to select number of threads * fixed bug in tile calculation Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com> * removed compile warning message Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com> * fix to parse y4m header * Remove unused variable - remove unused variable in *_neon.c Signed-off-by: Yangwoo Kim <yearly.kim@samsung.com> * removed duplicated code Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com> --------- Signed-off-by: kp5.choi@samsung.com <kp5.choi@samsung.com> Signed-off-by: Yangwoo Kim <yearly.kim@samsung.com> Co-authored-by: Minsoo Park <mss.park@samsung.com> Co-authored-by: Yangwoo Kim <yearly.kim@samsung.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.