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

chore: Bumpenvs for NGC+ images #8975

Merged
merged 9 commits into from
Mar 11, 2024
Merged

chore: Bumpenvs for NGC+ images #8975

merged 9 commits into from
Mar 11, 2024

Conversation

MikhailKardash
Copy link
Contributor

Description

Bumpenvs for NGC+ Images (and tf2.8 removal)
determined-ai/environments#235

Test Plan

CI doesn't fail

Checklist

  • Changes have been manually QA'd
  • User-facing API changes need the "User-facing API Change" label.
  • Release notes should be added as a separate file under docs/release-notes/.
    See Release Note for details.
  • Licenses should be included for new code which was copied and/or modified from any external code.

Ticket

MD-270

@cla-bot cla-bot bot added the cla-signed label Mar 8, 2024
@determined-ci determined-ci requested a review from a team March 8, 2024 00:49
@determined-ci determined-ci added the documentation Improvements or additions to documentation label Mar 8, 2024
Copy link

netlify bot commented Mar 8, 2024

Deploy Preview for determined-ui canceled.

Name Link
🔨 Latest commit 5f90c6d
🔍 Latest deploy log https://app.netlify.com/sites/determined-ui/deploys/65ef32cab1874f0008e32b36

Copy link

codecov bot commented Mar 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.55%. Comparing base (68017dd) to head (5f90c6d).
Report is 14 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8975   +/-   ##
=======================================
  Coverage   47.55%   47.55%           
=======================================
  Files        1168     1168           
  Lines      176706   176706           
  Branches     2356     2354    -2     
=======================================
  Hits        84026    84026           
  Misses      92522    92522           
  Partials      158      158           
Flag Coverage Δ
backend 42.75% <100.00%> (ø)
harness 63.94% <100.00%> (ø)
web 42.89% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
harness/determined/deploy/gcp/constants.py 100.00% <100.00%> (ø)
master/internal/config/provconfig/aws_config.go 11.20% <ø> (ø)
master/internal/config/provconfig/gcp_config.go 26.76% <100.00%> (ø)

... and 3 files with indirect coverage changes

@MikhailKardash MikhailKardash marked this pull request as ready for review March 8, 2024 21:33
@MikhailKardash MikhailKardash requested review from a team as code owners March 8, 2024 21:33
Users can build their own images from a given NGC container version by using the
`build-pytorch-ngc` or `build-tensorflow-ngc` workflows found in our environments `MakeFile`. Our
environments repo is located at `https://github.com/determined-ai/environments`.

Copy link
Member

@tara-det-ai tara-det-ai Mar 11, 2024

Choose a reason for hiding this comment

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

attempted to apply the following edits (hard to show restructured text formatting in a markdown comment)

:orphan:

New Features

  • Include early-access NVIDIA NGC-based images in our environment offerings. These images are
    accessible from pytorch-ngc <https://hub.docker.com/r/determinedai/pytorch-ngc>_ or
    tensorflow-ngc <https://hub.docker.com/r/determinedai/tensorflow-ngc>. By downloading and
    using these images, users acknowledge and agree to the terms and conditions of all third-party
    software licenses contained within, including the NVIDIA Deep Learning Container License <https://developer.download.nvidia.com/licenses/NVIDIA_Deep_Learning_Container_License.pdf>__.
    Users can build their own images from a specified NGC container version by using the
    build-pytorch-ngc or build-tensorflow-ngc workflows located in our environments
    MakeFile in the environments repository <https://github.com/determined-ai/environments>
    .

Copy link
Member

Choose a reason for hiding this comment

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

applied a commit that fixes the formatting and modifies some word choices

@determined-ci determined-ci requested a review from a team March 11, 2024 15:54
@MikhailKardash MikhailKardash merged commit cc2e9b4 into main Mar 11, 2024
98 of 100 checks passed
@MikhailKardash MikhailKardash deleted the ngc_bumpenvs_3 branch March 11, 2024 19:43
maxrussell pushed a commit that referenced this pull request Mar 21, 2024
* chore: Bumpenvs for NGC+ images

* linting

* this didn't need to be in here

* update release notes

* release notes

* oops

* rst is hard

* Update ngc-images.rst

* run rstfmt

---------

Co-authored-by: Tara <tara.charter@hpe.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants