Skip to content

Stabilizing the build and enabling tests#1001

Merged
khalatepradnya merged 3 commits intoNVIDIA:experimental/pythonfrom
khalatepradnya:exp-python/fix-tests
Dec 29, 2023
Merged

Stabilizing the build and enabling tests#1001
khalatepradnya merged 3 commits intoNVIDIA:experimental/pythonfrom
khalatepradnya:exp-python/fix-tests

Conversation

@khalatepradnya
Copy link
Collaborator

@khalatepradnya khalatepradnya commented Dec 4, 2023

  • Removed left-over tests pertaining to initialize_state functionality
  • Adjusted calls for assertion

@khalatepradnya khalatepradnya self-assigned this Dec 4, 2023
github-actions bot pushed a commit that referenced this pull request Dec 5, 2023
github-actions bot pushed a commit that referenced this pull request Dec 6, 2023
github-actions bot pushed a commit that referenced this pull request Dec 7, 2023
github-actions bot pushed a commit that referenced this pull request Dec 13, 2023
github-actions bot pushed a commit that referenced this pull request Dec 13, 2023
github-actions bot pushed a commit that referenced this pull request Dec 18, 2023
@khalatepradnya
Copy link
Collaborator Author

khalatepradnya commented Dec 27, 2023

To be merged after #1058

* Adjusted calls for assertion
* Addressing the following warning -
  "DeprecationWarning: expectation_z() is deprecated, use expectation() with the same argument structure."
* Renamed tests under 'eager' directory so top-level 'tests' can be run
  with 'pytest'
@khalatepradnya
Copy link
Collaborator Author

khalatepradnya commented Dec 28, 2023

/ok to test

Command Bot: Processing...

github-actions bot pushed a commit that referenced this pull request Dec 28, 2023
@khalatepradnya
Copy link
Collaborator Author

khalatepradnya commented Dec 29, 2023

/ok to test

Command Bot: Processing...

Copy link
Collaborator

@amccaskey amccaskey left a comment

Choose a reason for hiding this comment

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

Thanks for the pybind version fix. Most everything here looks like code formatting. LGTM +1

@github-actions github-actions bot locked and limited conversation to collaborators Dec 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants