cgal icon indicating copy to clipboard operation
cgal copied to clipboard

Polyhedron demo - fix display of protecting balls

Open janetournois opened this issue 3 years ago • 1 comments

Summary of Changes

On some setups (including mine), the protecting balls for a C3t3 could not be displayed anymore in the demo. This PR attempts to fix the display of protecting balls.

The regression was introduced by PR #5809 by the set of commits :

  • e5914cf70fcd231cd0930f7ab1564b05e4013edc (the first bad wrt displaying protecting balls, that compiles and displays the C3t3 properly)
  • 4f9fc89a10651c087083a66026a9c5f4c49c7c17
  • 73eafddd39b595c1cb0cfcd862104551c748eb4f
  • 2c36a1797cbec4dde5f0e08180d4c0e6a593e097
  • last good was : ec41af7e33af779082dadd93bd3816a4c8e9ca52

My patch does not fix the code introduced by these commits, but disables the "c3t3 spheres" shader.

Release Management

  • Affected package(s): Polyhedron demo
  • License and copyright ownership: unchanged

janetournois avatar Aug 02 '22 16:08 janetournois

With commit b98ac1a0bbec, the protecting balls are displayed but they do not take the cut plane into account. All of them are displayed.

janetournois avatar Aug 02 '22 16:08 janetournois

When opening a C3t3 with protecting balls, these errors repeat several times in the log:

GL ERROR : (ShaderCompilerSource : ErrorType)
 SHADER_ID_LINK error has been generated. GLSL link failed for program 24, "": The ♦ shader uses varying subdomain_out, but previous
 shader does not write to it.
Out of resource error.


QOpenGLShader::link: The ♦ shader uses varying subdomain_out, but previous shader does not write to it.
Out of resource error.

GL ERROR : (ShaderCompilerSource : ErrorType)
 SHADER_ID_LINK error has been generated. GLSL link failed for program 24, "": The ♦ shader uses varying subdomain_out, but previous
 shader does not write to it.
Out of resource error.


QOpenGLShader::link: The ♦ shader uses varying subdomain_out, but previous shader does not write to it.
Out of resource error.

QOpenGLShaderProgram::attributeLocation(colors): shader program is not linked
QOpenGLShaderProgram::attributeLocation(vertex): shader program is not linked
QOpenGLShaderProgram::attributeLocation(vertex): shader program is not linked
QOpenGLShaderProgram::attributeLocation(normals): shader program is not linked
QOpenGLShaderProgram::attributeLocation(normals): shader program is not linked
QOpenGLShaderProgram::attributeLocation(colors): shader program is not linked
QOpenGLShaderProgram::attributeLocation(colors): shader program is not linked
QOpenGLShaderProgram::attributeLocation(center): shader program is not linked
QOpenGLShaderProgram::attributeLocation(center): shader program is not linked
QOpenGLShaderProgram::attributeLocation(radius): shader program is not linked
QOpenGLShaderProgram::attributeLocation(radius): shader program is not linked

janetournois avatar Sep 19 '22 07:09 janetournois

@lrineau replaced?

sloriot avatar Sep 20 '22 16:09 sloriot

@lrineau replaced?

Yes, as soon as Jane confirms that the PR https://github.com/CGAL/cgal/pull/6863 is a correct solution.

lrineau avatar Sep 20 '22 16:09 lrineau

I confirm!

janetournois avatar Sep 20 '22 16:09 janetournois

Replaced by https://github.com/CGAL/cgal/pull/6863

lrineau avatar Sep 20 '22 16:09 lrineau