-
Notifications
You must be signed in to change notification settings - Fork 70
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
Inconsistent code commenting styles #172
Comments
👋 @TINGWEIJING |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Style Inconsistency
Just a suggestion for improvement in code commenting.
Evidence
cdle-traininglabs/dl4j-labs/src/main/java/ai/certifai/training/classification/BirdClassification.java
Lines 54 to 61 in 725a6e4
cdle-traininglabs/dl4j-labs/src/main/java/ai/certifai/training/classification/HeartDiseaseBinaryClassification.java
Lines 57 to 76 in 725a6e4
cdle-traininglabs/dl4j-labs/src/main/java/ai/certifai/training/datavec/kfold/SimpleKFoldDemo.java
Lines 37 to 45 in 725a6e4
cdle-traininglabs/dl4j-labs/src/main/java/ai/certifai/training/classification/Multiclass.java
Lines 93 to 98 in 725a6e4
cdle-traininglabs/dl4j-labs/src/main/java/ai/certifai/training/classification/OverfittingGermanCreditClassification.java
Lines 121 to 128 in 725a6e4
cdle-traininglabs/dl4j-labs/src/main/java/ai/certifai/training/earlyStopping/EarlyStopping.java
Lines 99 to 103 in 725a6e4
cdle-traininglabs/dl4j-labs/src/main/java/ai/certifai/training/classification/OverfittingGermanCreditClassification.java
Lines 199 to 208 in 725a6e4
cdle-traininglabs/dl4j-labs/src/main/java/ai/certifai/training/generative/MnistGAN.java
Lines 151 to 155 in 725a6e4
Possible solution
Just follow one of the styles and make it all consistent.
Styling references:
The text was updated successfully, but these errors were encountered: