Skip to content

Added branching logic to ww init in nullable types util#4369

Merged
MichaelFu512 merged 5 commits intomainfrom
nightly-test
Dec 15, 2023
Merged

Added branching logic to ww init in nullable types util#4369
MichaelFu512 merged 5 commits intomainfrom
nightly-test

Conversation

@MichaelFu512
Copy link
Copy Markdown
Contributor

@MichaelFu512 MichaelFu512 commented Dec 13, 2023

Pull Request Description

Resolves #4370
Fixes nightly test by adding branching logic in nullable types util (_downcast_nullable_y).


After creating the pull request: in order to pass the release_notes_updated check you will need to update the "Future Release" section of docs/source/release_notes.rst to include this pull request by adding :pr:123.

@MichaelFu512 MichaelFu512 changed the title Added branching logic to ww init in Added branching logic to ww init in nullable types util Dec 13, 2023
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 13, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a0bea2f) 99.7% compared to head (546123a) 99.7%.

Additional details and impacted files
@@           Coverage Diff           @@
##            main   #4369     +/-   ##
=======================================
+ Coverage   99.7%   99.7%   +0.1%     
=======================================
  Files        357     357             
  Lines      39919   39922      +3     
=======================================
+ Hits       39793   39796      +3     
  Misses       126     126             

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@MichaelFu512 MichaelFu512 marked this pull request as ready for review December 14, 2023 19:56
Copy link
Copy Markdown
Collaborator

@jeremyliweishih jeremyliweishih left a comment

Choose a reason for hiding this comment

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

LGTM thanks!

@MichaelFu512 MichaelFu512 merged commit 703f258 into main Dec 15, 2023
@MichaelFu512 MichaelFu512 deleted the nightly-test branch December 15, 2023 21:12
@MichaelFu512 MichaelFu512 mentioned this pull request Feb 1, 2024
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.

ww initialization fails during _downcast_nullable_types

3 participants