Optimize the RegisterFromNodeAnnotations code to make it clearer. Enhance its readability.

[chaunceyjiang]

1. Increase the commonts of the code.
2. Merge duplicate codes into functions.
3. Reduce the nesting level of if statements.

[archlitchi]

thanks for refining these code, have you tested?

[haitwang-cloud]

As a user for the k8s-vgpu-scheduler , I would suggest that @chaunceyjiang could add more UT for your code change to ensure that the new code is fully tested.

[chaunceyjiang]

have you tested?

I am currently testing. I will post the test results later.

[chaunceyjiang]

As a user for the k8s-vgpu-scheduler , I would suggest that @chaunceyjiang could add more UT for your code change to ensure that the new code is fully tested.

Good idea.

However, this project currently lacks the corresponding GitHub action( golang-lint, UT, E2E, etc.), but I will supplement the UT later.

[haitwang-cloud]

As a user for the k8s-vgpu-scheduler , I would suggest that @chaunceyjiang could add more UT for your code change to ensure that the new code is fully tested.

Good idea.

However, this project currently lacks the corresponding GitHub action( golang-lint, UT, E2E, etc.), but I will supplement the UT later.
Good to know that ,I am working on build the UTs and E2E test case for the code 2.

[chaunceyjiang]

Good to know that ,I am working on build the UTs and E2E test case for the code 2.

Amazing!

[archlitchi]

please resolve this conflict first :)

[chaunceyjiang]

Use the latest dev image.

The GPU pod can be scheduled normally.

The scheduler log is also normal.

[chaunceyjiang]

/cc @archlitchi PTAL.