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

changed 'compiler_config' to 'compile_config' to match parameter in n=3 places, otherwise compile fails #39

Open

aurotripathy wants to merge 1 commit into furiosa-ai:main from aurotripathy:fix-yolox-compile-param

Contributor

aurotripathy commented Mar 2, 2023

Problem

model compile fails due to one parameter having a typo; 'compiler' instead of 'compile'

Solution

make a consistent change everywhere

Testing

compile passes and the notebook completes

Checklist

Will this be part of a product update? If yes, please update Changelog.
Changes are less than 1000 lines (refer to Pull Request 크기 줄이는 법)
My code has passed black/isort.
I have performed a self-review of my code.
All CI checks have passed.


          changed 'compiler_config' to compile_config to match parameter

hyunsik force-pushed the main branch from 7a30eb6 to 85fb4c7 Compare

April 25, 2023 21:03

stale bot commented Jul 24, 2023

This issue has been automatically marked as stale because it has not had recent activity.

stale bot added the stale label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels