justingardner / mgl

A suite of mex/m files for displaying psychophysics stimuli
http://justingardner.net/mgl
Other
18 stars 22 forks source link

BugHunt: mglRenderingDemo Issues #45

Closed cjoynes closed 1 year ago

cjoynes commented 2 years ago
benjamin-heasly commented 2 years ago

Unfortunately I wasn't able to reproduce these myself with:

Perhaps we could retest these since some other (possibly related) issues have been resolved in the meantime. If these still are happening, it would be helpful to have more system info, output, reproduction steps, etc.

benjamin-heasly commented 2 years ago

On Gardner lab Lagavulin machine, I found the issue with mglTestDots and mglTestText. Some of the new tests I made for the automated test runner (mglRunRenderingTests) had the same names as existing mgl tests. So there was Matlab path shadowing. On my machine this happened to work in favor of the new tests. But not on Lagavulin. I renamed the new tests to not conflict, and that should clear up those two.

justingardner commented 1 year ago

Closing. This seems to be mostly working now. mglTestDots was modified because you now specify the argument as size of dots in degrees. MglTestText seems to plot the text flipped upside down. Will look into that, but these demos are basically working fine now, so closing.