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

Cherry-pick PRs 14703, 14637 #15020

Merged

Conversation

aparajit-pratap
Copy link
Contributor

@aparajit-pratap aparajit-pratap commented Mar 14, 2024

  • remove coreclr-ncalc references

  • add failing test for dropdown node

  • cleanup

  • update tests

  • attempt initial fix

  • cleanup

  • update test

  • review comments

  • add code comments

Purpose

Cherry-pick #14637 #14703

Declarations

Check these if you believe they are true

  • The codebase is in a better state after this PR
  • Is documented according to the standards
  • The level of testing this PR includes is appropriate
  • User facing strings, if any, are extracted into *.resx files
  • All tests pass using the self-service CI.
  • Snapshot of UI changes, if any.
  • Changes to the API follow Semantic Versioning and are documented in the API Changes document.
  • This PR modifies some build requirements and the readme is updated
  • This PR contains no files larger than 50 MB

Release Notes

(FILL ME IN) Brief description of the fix / enhancement. Mandatory section

Reviewers

(FILL ME IN) Reviewer 1 (If possible, assign the Reviewer for the PR)

(FILL ME IN, optional) Any additional notes to reviewers or testers.

FYIs

(FILL ME IN, Optional) Names of anyone else you wish to be notified of

…nitialized to null (DynamoDS#14703)

* remove coreclr-ncalc references

* add failing test for dropdown node

* cleanup

* update tests

* attempt initial fix

* cleanup

* update test

* review comments

* add code comments
@aparajit-pratap aparajit-pratap changed the title Cherry-pick DYN-6527: Fix graph update for primitive input nodes that are first i… Cherry-pick DYN-6527: Fix graph update for primitive input nodes that are first initialized to null (#14703) Mar 14, 2024
@aparajit-pratap aparajit-pratap changed the title Cherry-pick DYN-6527: Fix graph update for primitive input nodes that are first initialized to null (#14703) Cherry-pick PRs 14703), 14637 Mar 14, 2024
@aparajit-pratap aparajit-pratap changed the title Cherry-pick PRs 14703), 14637 Cherry-pick PRs 14703, 14637 Mar 14, 2024
@QilongTang
Copy link
Contributor

Is this PR related to #15019 in any way? Hopefully there is no dependency because the other one got questioned by Davinci team

@QilongTang QilongTang added this to the 2.19.5 milestone Mar 19, 2024
@aparajit-pratap aparajit-pratap merged commit 0dc249d into DynamoDS:RC2.19.5_master Mar 19, 2024
21 of 23 checks passed
@aparajit-pratap aparajit-pratap deleted the RC2.19.5_master branch March 19, 2024 14:33
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.

3 participants