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

testUsdBugs fails with float valued file truncation #2170

Closed
nvmkuruc opened this issue Jan 17, 2023 · 4 comments
Closed

testUsdBugs fails with float valued file truncation #2170

nvmkuruc opened this issue Jan 17, 2023 · 4 comments

Comments

@nvmkuruc
Copy link
Collaborator

Description of Issue

When the passed value to truncate is a float, the following error message is produced

======================================================================
ERROR: test_USD_5196 (__main__.TestUsdBugs)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/mkuruc/code/inst/USD/tests/testUsdBugs", line 433, in test_USD_5196
    fobj.truncate(size / 2)
TypeError: 'float' object cannot be interpreted as an integer

----------------------------------------------------------------------
Ran 13 tests in 0.992s

Converting to an explicit integer division avoids the issue.

Steps to Reproduce

ctest -C Release -R testUsdBugs --output-on-failure

System Information (OS, Hardware)

Ubuntu 22.04.1 LTS

Package Versions

Python 3.10
Boost 1.78
TBB 2020.3

Build Flags

--no-imaging --tests

@nvmkuruc
Copy link
Collaborator Author

I have a fix.

@sunyab
Copy link
Contributor

sunyab commented Jan 18, 2023

Filed as internal issue #USD-7881

@nvmkuruc
Copy link
Collaborator Author

@jesschimein I think this can be closed. The fix was merged last year.

@jesschimein
Copy link
Contributor

Thanks Matt! Closing this issue related to #2171

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

No branches or pull requests

3 participants