Running Stochastic Benchmark on Simualted Annealing results#30
Running Stochastic Benchmark on Simualted Annealing results#30bernalde merged 29 commits intobernalde:mainfrom
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
bernalde
left a comment
There was a problem hiding this comment.
Some changes were introduced to other folders that were irrelevant to this PR. I will review the notebook separately
bernalde
left a comment
There was a problem hiding this comment.
There are some comments still unresolved. I will review the notebook once they are sorted out.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #30 +/- ##
=======================================
Coverage ? 48.68%
=======================================
Files ? 14
Lines ? 2280
Branches ? 0
=======================================
Hits ? 1110
Misses ? 1170
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
The following changes were made:
|
bernalde
left a comment
There was a problem hiding this comment.
Approved. Please run the notebook from the head to the bottom and let's wait for @anurag-r20 's revision to merge
anurag-r20
left a comment
There was a problem hiding this comment.
Everything looks goof! I think there is an issue with the include_groups but I have fixed that in my PR. So once that is merged, this warning should go away when you re-run.
This runs SB with:
I'm open this PR to keep track of progress made, but Azain and I decided to keep working on his PR. I will be adding the parts where our code differs to his PR.