Skip to content

Rename misleading test names in ConvTranspose op tests - #788

Merged
hariharans29 merged 1 commit into
masterfrom
hari/fixConvTransposeTests
Apr 7, 2019
Merged

Rename misleading test names in ConvTranspose op tests#788
hariharans29 merged 1 commit into
masterfrom
hari/fixConvTransposeTests

Conversation

@hariharans29

Copy link
Copy Markdown
Member

Resolve #699

Unable to add actual 3D tests for ConvTranspose as they are not yet supported by the implementation

@hariharans29
hariharans29 requested a review from a team as a code owner April 6, 2019 08:24
@hariharans29
hariharans29 merged commit 5cf7203 into master Apr 7, 2019
@hariharans29
hariharans29 deleted the hari/fixConvTransposeTests branch April 7, 2019 00:01
jnagi-intel pushed a commit to jnagi-intel/onnxruntime that referenced this pull request Jan 5, 2026
* Changed fix

* Fix to omit subgraph

* Commit a fix for cluster index len

* Fixing the Warning with size_t on clusters

* Loop Test fix
yuslepukhin pushed a commit that referenced this pull request Mar 17, 2026
)

## Describe your changes
Pydantic v2 had significant breaking changes which blocks migration to
it. So, we pinned the version to v1.

Pydantic v2 has a `v1` namespace inside it. 

This PR introduces `olive.common.pydantic_v1` module that handles the
import of `v1` api so that Olive can be version agnostic wrt pydantic.
This way, we won't have version conflict with client applications such
as that in microsoft/Olive#709.

Note:
The doc builder still needs pydantic v1 and autodoc_pydantic v1 since we
are using the v1 api.

## Checklist before requesting a review
- [ ] Add unit tests for this change.
- [ ] Make sure all tests can pass.
- [ ] Update documents if necessary.
- [ ] Lint and apply fixes to your code by running `lintrunner -a`
- [ ] Is this a user-facing change? If yes, give a description of this
change to be included in the release notes.

## (Optional) Issue link
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.

2 participants