Skip to content
This repository was archived by the owner on Jul 4, 2025. It is now read-only.

Conversation

@marknguyen1302
Copy link
Contributor

Describe Your Changes

  • Fix undefined engine name
  • Update engine product name, description

Fixes Issues

  • Closes #
  • Closes #

Self Checklist

  • Added relevant comments, esp in complex areas
  • Updated docs (for bug fixes / features)
  • Created issues for follow-up changes or refactoring needed

@marknguyen1302 marknguyen1302 requested a review from louis-jan July 17, 2024 10:30

@Injectable()
export default class Onnxprovider extends CortexProvider {
productName = 'Onnx Inference Engine';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If so, please set name here

@marknguyen1302 marknguyen1302 merged commit 4d16714 into dev Jul 17, 2024
@marknguyen1302 marknguyen1302 deleted the fix/wrong-engine-list branch July 17, 2024 10:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants