Skip to content
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

Submitting a Pull Request #606

Closed
JWLee89 opened this issue Aug 10, 2021 · 2 comments
Closed

Submitting a Pull Request #606

JWLee89 opened this issue Aug 10, 2021 · 2 comments

Comments

@JWLee89
Copy link

JWLee89 commented Aug 10, 2021

Hi,

I want to make a pull request for some tensorrt_converter layers that I wrote but was having a few issues with permissions.

Below are the steps that I took.

  1. Checked out from master branch
  2. Created a new local branch
  3. Added new converter.
  4. Pushed new branch to upstream (error occurs here)

I am assuming this is because I don't have permission to push to the torch2trt repository, so I was wondering:

What are the proper procedures for making a pull request?

FYI I read through CONTRIBUTING.md - link

Thank you for your time.

Best regards,
Jay

P.S. I am uncertain as to whether this is the recommended approach for asking a question, but just in case my question flies under the radar.

@jaybdub
Copy link
Contributor

jaybdub commented Aug 10, 2021

Hi @JWLee89 ,

Thanks for the question.

Pushing directly to NVIDIA-AI-IOT/torch2trt isn't permitted.

I'd recommend making a personal fork of torch2trt (JWLee89/torch2trt), and then you can make the pull request from any branch in your fork.

Perhaps we should update the contributing guide with these instructions.

Please let me know if this helps or you run into any issues / questions.

Best,
John

@JWLee89
Copy link
Author

JWLee89 commented Aug 11, 2021

Hi @jaybdub John,

Thank you for the answer.
It makes perfect sense to make a personal fork and making the pull request from there.

I think it may help out some people like myself who are not used to contributing to open source.

I managed to make the pull request under PR #607.

Thank you for the help once again John!

Best regards,
Jay

@JWLee89 JWLee89 closed this as completed Aug 16, 2021
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

No branches or pull requests

2 participants