Fix run-unit-tests issue with finding acroname #8444
Conversation
| log.progress( '[' + ' '.join( configuration ) + ']', test.name, '...' ) | ||
| else: | ||
| log.progress( test.name, '...' ) | ||
| if not log.is_debug_on() or log.is_color_on(): |
There was a problem hiding this comment.
Why add the color check?
Not just debug check?
There was a problem hiding this comment.
If color is on, then the terminal supports advanced controls and progress will not stay on-screen. I wanted to avoid the progress since debug includes the same information, for readability on the libci logs. But when I run it locally, I run with debug and I have color, so it still shows progress...
| log.d( 'sys.path=', sys.path ) | ||
| acroname = None | ||
| # | ||
| sys.path = sys.path[:-1] # remove what we added |
There was a problem hiding this comment.
If we want to use a third party lib like numpy in our test it will not work right?
If yes, than we need to think of another way to solve this.
There was a problem hiding this comment.
Right, it won't work (unless numpy isn't in site-packages) unless the module is in lib/.
One thing I can do is remove site-packages before loading librealsense, but it's hard to control when a script imports pyrs.
For now, it'll work.
2 participants
No description provided.