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

OpenGL issue on Run CPU Tests #14052

Closed
anr2me opened this issue Feb 2, 2021 · 3 comments · Fixed by #14061
Closed

OpenGL issue on Run CPU Tests #14052

anr2me opened this issue Feb 2, 2021 · 3 comments · Fixed by #14061
Labels
Milestone

Comments

@anr2me
Copy link
Collaborator

anr2me commented Feb 2, 2021

If i build PPSSPP my self i'm seeing this "Run CPU tests" in Developer Tools
image

But if i extract the latest dev build from https://buildbot.orphis.net/ppsspp/ on a new folder, the "Run CPU tests" is being grayed out.
image

What happens?

If i clicked "Run CPU tests" from my build, i'm getting this popup
image

44:47:997 idle0        E[SYSTEM]: Common\Log.cpp:49 (C:\Users\AdamN\Documents\Projects\ppsspp\Common\GPU\OpenGL\GLRenderManager.cpp:GLRenderManager::BindFramebufferAsRenderTarget:298) Critical: [insideFrame_] *** Assertion ***

Btw how do i enable "Run CPU tests" so it won't be grayed out?

What should happen?

Not sure whether this is a bug or not since i don't know how to enable/disable "Run CPU tests"

What hardware, operating system, and PPSSPP version? On desktop and mobile phone, GPU matters for graphical issues.

Please test newest version in https://buildbot.orphis.net/ppsspp/

PPSSPP Win64 v1.10.3-1720-gd73456e93

@anr2me anr2me changed the title Issue regarding Run CPU Tests OpenGL issue on Run CPU Tests Feb 2, 2021
@sum2012
Copy link
Collaborator

sum2012 commented Feb 2, 2021

Choice *cpuTests = new Choice(dev->T("Run CPU Tests"));
list->Add(cpuTests)->OnClick.Handle(this, &DeveloperToolsScreen::OnRunCPUTests);

cpuTests->SetEnabled(TestsAvailable());

bool TestsAvailable() {
#ifdef IOS
std::string testDirectory = g_Config.flash0Directory + "../";
#else
std::string testDirectory = g_Config.memStickDirectory;
#endif
// Hack to easily run the tests on Windows from the submodule
if (File::IsDirectory("../pspautotests")) {
testDirectory = "../";
}
return File::Exists(testDirectory + "pspautotests/tests/");
}

Seem need clone pspautotests

@anr2me
Copy link
Collaborator Author

anr2me commented Feb 3, 2021

So the test only comes with repo?

@hrydgard
Copy link
Owner

hrydgard commented Feb 3, 2021

yeah there's no use for this button if you haven't got the pspautotests because it uses them.

The crash seems like a problem we should fix though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants