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

cubit learn #797

Draft
wants to merge 12 commits into
base: develop
Choose a base branch
from
Draft

cubit learn #797

wants to merge 12 commits into from

Conversation

shimwell
Copy link
Collaborator

@shimwell shimwell commented Apr 3, 2021

Proposed changes

This is a test PR to see if cubit learn can be built in the docker image

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Code refactoring
  • Documentation Update (if none of the other choices apply)
  • New tests

Checklist

  • Pep8 applied
  • Unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

@shimwell shimwell added the Draft label Apr 3, 2021
@shimwell shimwell marked this pull request as draft April 3, 2021 23:11
@codecov
Copy link

codecov bot commented Apr 3, 2021

Codecov Report

Merging #797 (d23cc04) into develop (3a9a910) will not change coverage.
The diff coverage is n/a.

❗ Current head d23cc04 differs from pull request most recent head 9b97d27. Consider uploading reports for the commit 9b97d27 to get more accurate results
Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #797   +/-   ##
========================================
  Coverage    95.34%   95.34%           
========================================
  Files           74       74           
  Lines         5182     5182           
========================================
  Hits          4941     4941           
  Misses         241      241           

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 3a9a910...9b97d27. Read the comment docs.

Dockerfile Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
paramak/neutronics_utils.py Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant