Exceeding some character count results in a memory related crash

[ycw]

Exceeding some character count results in a memory related crash

[ycw]

increase the budget in asconfig.json:options.memoryBase ( say, 655360 = 640kB )

Babylon.Font/asconfig.json

Line 18 in 0d0c942

"memoryBase": 65536

then rebuild (npm run asbuild)

🥂

[meshantz]

Okay, that makes sense. Thanks for the quick reply!

Is there a way to calculate what values will be a problem for a given font, or is the only way to wrap every call to builder.create with error handling to catch this error?

[meshantz]

Hmm. I can't seem to recover from this error at all. I've even tried reloading the entire complier mechanism (which would be pretty heavy to do every time this error occurs) to no avail.

All subsequent calls to builder.create fail with the same error.

Here is (roughly) the reloading code I attempted:

  let compiler = await Compiler.Build(`${constants.BABYLON_FONT_WASM}`)

  let fonts: { [key in FontName]: Font } = {
    calibri: await Font.Install(fontPaths.calibri, compiler, opentype),
    consola: await Font.Install(fontPaths.consola, compiler, opentype),
    'consola-bold': await Font.Install(
      fontPaths['consola-bold'],
      compiler,
      opentype,
    ),
  }

  let builder = new TextMeshBuilder(BABYLON, earcut)

      try {
        mesh = builder.create(
          // eslint-disable-next-line @typescript-eslint/ban-ts-comment
          // @ts-ignore
          {
            font: fonts.calibri,
            text: failingText,
            size: 12,
          },
          scene,
        )
      } catch (error) {
        Compiler.Build(`${constants.BABYLON_FONT_WASM}`).then(
          async (newCompiler) => {
            compiler = newCompiler
            fonts = {
              calibri: await Font.Install(
                fontPaths.calibri,
                compiler,
                opentype,
              ),
              consola: await Font.Install(
                fontPaths.consola,
                compiler,
                opentype,
              ),
              'consola-bold': await Font.Install(
                fontPaths['consola-bold'],
                compiler,
                opentype,
              ),
            }
            builder = new TextMeshBuilder(BABYLON, earcut)
            builder.create(
              // eslint-disable-next-line @typescript-eslint/ban-ts-comment
              // @ts-ignore
              {
                font: fonts.calibri,
                text: 'small',
                size: 12,
              },
              makeOpts.scene,
            )
          },
        )
      }

[ycw]

a workaround off the top of my head

bytes required for (font, text) can be computed by:

function bytes_required(font, text) {
  let b = 0
  const cmds = font.raw.getPath(text, 0, 0, 100).commands
  for (const cmd of cmds) {
    b += 1
    switch (cmd.type) {
      case 'M': 
      case 'L': b += 16; break
      case 'Q': b += 48; break
      case 'C': b += 64; break 
    }
  }
  return b
}

then, validate before each builder.create:

if ( bytes_required(font, 'hello') <= 65536 ) {
  // builder.create( .. )
} 

where 65536 := asconfig.json:options.memoryBase

🥂

[meshantz]

Awesome. Thanks again! I'll give this a shot.

[ycw]

solved in v4.