Skip to content

Fix GTK activation and Builder MO rendering - #565

Merged
ohlincha merged 2 commits into
mainfrom
wip/organizer-cleanup-rebased
Oct 26, 2026
Merged

ohlincha merged 2 commits into
mainfrom
wip/organizer-cleanup-rebased

Conversation

@mattasplund

@mattasplund mattasplund commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Prevent hidden child allocation warnings while scrolling the Organizer summary panel.
  • Draw Builder's 3D corner labels with screen-space quads instead of legacy bitmap display lists.
  • Use ewxRaiseWindow() in GTK activation paths to avoid timestamp-zero window-manager warnings.
  • Fix the mismatched array deallocation in isosurface generation.

Validation

  • cmake --build /home/asplund/build/ECCE/build-cmake/9.3.3-rpm --target builder -j 8 passed after rebasing onto main.
  • The user reported the MO viewer issue fixed in local testing. Automated tests were not run (BUILD_TESTING=OFF).
  • The GitHub parser/dialog check reports six localization-fixture and parser-classification failures; the base main commit (c9bd11fa) reports the identical failures, and this PR does not change parser files.

@ohlincha

Copy link
Copy Markdown
Collaborator

Thanks. Tested before merging: summary_test (with the new scroll check), orca_mo_test, ecceqm_mo_large_test, mo_explore_workflow_test, organizer_close_test and the 72-app smoke suite, all passing on a build of this branch (Xvfb, llvmpipe). Lycopene MOs (NWChem, ORCA, G16, ECCE-QM, MOPAC) did not crash on main under llvmpipe either, so the original crash may depend on the students' GL driver; #566 follows up with renderer detection. A glxinfo -B line from an affected machine would help there.

@ohlincha
ohlincha merged commit 49a98d0 into main Oct 26, 2026
9 of 11 checks passed
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.

2 participants