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

unsupervised contrastive loss #522

Open
wants to merge 5 commits into
base: staging
Choose a base branch
from

Conversation

Dante-Basile
Copy link

@Dante-Basile Dante-Basile commented Dec 12, 2021

Reference issue

#520

Type of change

This contains the loss function and a basic ResNet50 demo demonstrating training and decreasing loss on CIFAR-10.

What does this implement/fix?

This demonstrates that the ResNet50 network can effectively optimize the loss of the unsupervised contrastive loss function. Future experiments will examine the accuracy of the trained network.

Additional information

NDD 2021

Copy link
Member

@PSSF23 PSSF23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Refer to file structure in #518

@codecov
Copy link

codecov bot commented Dec 12, 2021

Codecov Report

Merging #522 (8373149) into staging (634d4d1) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           staging     #522   +/-   ##
========================================
  Coverage    90.09%   90.09%           
========================================
  Files            7        7           
  Lines          404      404           
========================================
  Hits           364      364           
  Misses          40       40           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 634d4d1...8373149. Read the comment docs.

Copy link
Member

@PSSF23 PSSF23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Dante-Basile You need to black format the files.

@jdey4 do you think this is an ideal way to separate supervised & unsupervised? Or should there be separate folders in benchmarks?

@PSSF23 PSSF23 requested review from rflperry and jdey4 December 13, 2021 20:59
Copy link
Member

@PSSF23 PSSF23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Your notebooks have unnecessary outputs & empty cells. Remove them.
  • Remove commented lines of code

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