Skip to content

Conversation

HoriaMercan
Copy link
Collaborator

@HoriaMercan HoriaMercan commented Jun 30, 2025

  • I have fixed paths and dockerfiles for proper building of cpp benchmarks

  • I have implemented the 210-thumbnailer benchmark in cpp for AWS

    • Modified the deployment package for cpp to link OpenCV for build
    • Updated storage.hpp & storage.cpp aws headers for easier use of the S3 API
  • Implemented 501-graph-pagerank benchmark CPP-AWS

  • Implemented 503-graph-bfs benchmark CPP-AWS

  • Added OpenCV docker dependencies for AWS C++

  • Added igraph docker dependencies

  • Added torch/torchvision dependencies

  • cpp_dependencies config for building CMakeList.txt

  • updated ./tools/build_docker_images.py to be compliant with config/systems.json

TODO:

  • Build a docker image to run image-recognition benchmark with AWS ECR

Copy link

coderabbitai bot commented Jun 30, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@HoriaMercan HoriaMercan marked this pull request as draft July 8, 2025 19:25
@HoriaMercan HoriaMercan marked this pull request as ready for review July 8, 2025 21:37
@HoriaMercan HoriaMercan changed the title [aws] Add support for thumbnailer benchmark in cpp [aws] Add support for benchmarks in cpp Jul 9, 2025
…n benchmark logic. Add namespace for storage.hpp to avoid name collisions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants