Skip to content

Conversation

@tylera-nvidia
Copy link
Collaborator

Fix to support correct specialization for generator printing (switched order of Print Utility and PrintData function). Also added cast to protect from warnings for conflicting size/stride sizes

@tylera-nvidia
Copy link
Collaborator Author

`[----------] Global test environment tear-down
[==========] 1263 tests from 353 test suites ran. (675302 ms total)
[ PASSED ] 1263 tests.

YOU HAVE 20 DISABLED TESTS

docker@ipp1-1946:/home/tallen/scratch/playground/matxExample/MatX/build$ git status
On branch generatorPrintPatch
Your branch is up to date with 'origin/generatorPrintPatch'.

nothing to commit, working tree clean
docker@ipp1-1946:/home/tallen/scratch/playground/matxExample/MatX/build$ git rev-parse HEAD
6382201
docker@ipp1-1946:/home/tallen/scratch/playground/matxExample/MatX/build$ `

@luitjens
Copy link
Collaborator

@tylera-nvidia can you rebase. it looks like we have conflicts.

@cliffburdick
Copy link
Collaborator

/blossom-ci

@tylera-nvidia
Copy link
Collaborator Author

[----------] Global test environment tear-down
[==========] 1282 tests from 353 test suites ran. (684566 ms total)
[ PASSED ] 1282 tests.

YOU HAVE 20 DISABLED TESTS

docker@ipp1-1946:/home/tallen/scratch/MatX/build$ git status
On branch generatorPrintPatch
Your branch is up to date with 'origin/generatorPrintPatch'.

nothing to commit, working tree clean
docker@ipp1-1946:/home/tallen/scratch/MatX/build$ git rev-parse HEAD
fc0d90e

@tylera-nvidia tylera-nvidia merged commit 522c741 into main Mar 28, 2023
@tylera-nvidia tylera-nvidia deleted the generatorPrintPatch branch March 28, 2023 18:32
cliffburdick pushed a commit that referenced this pull request Dec 8, 2023
* fix to support correct specialization for generator printing

* adding casts to fix compile warnings for uint ranks

* updates for tensor_t no longer being a pointer in RadarPipeline
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.

4 participants