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

Funny behavior with the multiply note value clamp #2334

Closed
pikurasa opened this issue Jun 8, 2020 · 2 comments
Closed

Funny behavior with the multiply note value clamp #2334

pikurasa opened this issue Jun 8, 2020 · 2 comments

Comments

@pikurasa
Copy link
Collaborator

pikurasa commented Jun 8, 2020

Funny behavior with the multiply note value clamp

Screenshot at 2020-06-08 13:13:56 funny clamp state

I think that the reproducible steps are:

  1. insert notes into multiply note value clamp
  2. Extract a middle note
  3. Repeat 1 and 2, if the issue does not happen the first time

The issue is rather bothersome once encountered as there is no easy remedy to get the clamp working properly again.

Test Files (note that I am on 4e8f2c3 for this testing)
Funny-Clamp-State-2.html.zip
funny-clamp-state.html.zip

I also checked master... same issue: Clamp-issue-on-Master.html.zip

I do not see anything consistent in logs, but at one point in testing, I ran across

block.js:180 Uncaught (in promise) Error: COULD NOT CREATE CACHE
    at checkBounds (block.js:180)
    at checkBounds (block.js:187)
    at checkBounds (block.js:187)
    at checkBounds (block.js:187)
    at checkBounds (block.js:187)
    at block.js:203
    at new Promise (<anonymous>)
    at Block._createCache (block.js:171)
    at __processHighlightBitmap (block.js:906)
    at Image.img.onload (block.js:7753)
@walterbender
Copy link
Member

Does the same thing happen to any other clamps? e.g. tie?

@sksum
Copy link
Member

sksum commented Jun 9, 2020

This seems to happens when using collapsed notes/blocks ( in any type of clamp )

@sksum sksum mentioned this issue Jun 10, 2020
@sksum sksum added the WF6-Needs testing Please test label Jun 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants