-
Notifications
You must be signed in to change notification settings - Fork 333
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
Use a max line length of 80 universally #1552
Conversation
…MaxLineLength � Conflicts: � keras_cv/layers/feature_pyramid.py � keras_cv/layers/object_detection/roi_align.py � keras_cv/layers/object_detection/rpn_label_encoder.py � keras_cv/layers/object_detection_3d/center_net_label_encoder.py � keras_cv/layers/object_detection_3d/heatmap_decoder.py � keras_cv/layers/preprocessing/base_image_augmentation_layer.py � keras_cv/layers/preprocessing/random_channel_shift.py � keras_cv/layers/preprocessing/random_translation.py � keras_cv/layers/preprocessing_3d/base_augmentation_layer_3d.py � keras_cv/layers/regularization/squeeze_excite.py � keras_cv/layers/spatial_pyramid.py � keras_cv/losses/penalty_reduced_focal_loss.py � keras_cv/models/convmixer.py � keras_cv/models/csp_darknet.py � keras_cv/models/efficientnet_lite.py � keras_cv/models/mobilenet_v3.py � keras_cv/models/object_detection/faster_rcnn.py � keras_cv/models/vit.py
…MaxLineLength � Conflicts: � keras_cv/utils/target_gather.py
…MaxLineLength � Conflicts: � keras_cv/models/backbones/resnet_v2/resnet_v2_backbone.py
…MaxLineLength � Conflicts: � keras_cv/layers/preprocessing/mix_up.py
…MaxLineLength � Conflicts: � keras_cv/callbacks/waymo_evaluation_callback.py � keras_cv/layers/preprocessing/base_image_augmentation_layer.py � keras_cv/layers/preprocessing/cut_mix.py � keras_cv/layers/preprocessing/mosaic.py � keras_cv/models/object_detection/retina_net/retina_net_coco_test.py
…MaxLineLength � Conflicts: � keras_cv/metrics/coco/mean_average_precision.py � keras_cv/metrics/coco/numerical_tests/mean_average_precision_test.py � keras_cv/metrics/coco/numerical_tests/recall_correctness_test.py � keras_cv/metrics/coco/recall.py � keras_cv/metrics/coco/utils.py � keras_cv/utils/fill_utils.py
Ping @ianstenbit @LukeWood |
Hi @ID6109, I have resolved the conflicts with master. Whenever, you have done it, please ping me. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is amazing! Thanks for the hard work!
I reviewed part of the PR, more reviews on the way.
examples/training/object_detection_3d/waymo/serialize_records.py
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
More reviews on the way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review finished.
Thanks again for the PR!
keras_cv/models/object_detection/retina_net/retina_net_label_encoder.py
Outdated
Show resolved
Hide resolved
Hey @haifeng-jin , thanks for the review! It seems there are loads of corrections required in the PR. I'll get back to you once I mitigate them. I'll also go through the codebase once again to see if any similar changes are necessary. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the update! a few more comments and we are ready to go.
Thanks, @haifeng-jin . I really appreciate you taking the time to review my code. Seems my understanding of how docstrings are interpreted required some much-needed improvement. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR! ready to be merged
/gcbrun |
Thanks for the amazing effort @ID6109! |
* Enforces line length 80 Part 1 * Enforces line length 80 Part 2 and fixes typos * Enforces line length 80 Part 3 and fixes typos * Enforces line length 80 Part 4 and fixes typos * Enforces line length 80 Part 5 and fixes typos * Minor change * Replaced double spaces with a single space * Merge Conflicts * Minor Changes * Resolves Merge Conflicts * Additional improvements + Changes requested * Merging * Minor improvements --------- Co-authored-by: Your Name <[email protected]> Co-authored-by: Haifeng Jin <[email protected]>
What does this PR do?
Fixes #1532 + a LOT of typos (including replacing double spaces after every sentence with a single space).
Do let me know if any changes are necessary. Thanks.
PS: I'll go ahead with #1500 after this gets merged. Might be too big of a PR or else.
Who can review?
@LukeWood @ianstenbit