Skip to content

Feature/export docs #175

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.

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

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

DatSplit
Copy link

Description

Added a docs folder in the root of the repository with documentation about exporting trained models as suggested by @SkalskiP in #170:

rf-detr /
├── docs /
│   └── export.md
├── README.md
...

With the following subsections:

  • ONNX export
  • TensorRT conversion
  • Deploying on Jetson Orin Nano (8 GB developer kit)

Type of change

Documentation only

How has this change been tested, please provide a testcase or example of how you tested the change?

Ran the Python snippets on my Jetson Orin Nano 8 GB developer kit (Ubuntu: 22.04.4 LTS, TensorRT-10.0.1.6, CUDA 12.0, Python 3.10)

Any specific deployment considerations

No

Docs

This feature only adds docs!

@CLAassistant
Copy link

CLAassistant commented Apr 21, 2025

CLA assistant check
All committers have signed the CLA.

@DatSplit
Copy link
Author

DatSplit commented Apr 21, 2025

Good afternoon @SkalskiP,

As mentioned by you in #170 "the sooner the better. It can be really rough. I can help you out with structuring it properly. I mostly care about the steps to follow."

I believe there is still a bug in the newly created code snippet in (Deploying on Jetson Orin Nano (8 GB developer kit)).
Will work on this tonight or tomorrow (first check whether there is actually a bug), as I got to go now.

Questions:

  1. (ONNX export) more details on other possible arguments that can be passed to the export() method?
  2. (TensorRT conversion) More details required on how to set-up TensorRT correctly?
  3. Other: might it be benifical to add a tensorRT export option to model.export()?
  4. Should I explain the run_sync() method in more detail?

Any feedback is welcome!

@DatSplit
Copy link
Author

DatSplit commented May 2, 2025

I have read the CLA Document and I sign the CLA.

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