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

The bundled distro hangs on drill-down to literal expression #809

Open
barmac opened this issue Dec 6, 2023 · 4 comments
Open

The bundled distro hangs on drill-down to literal expression #809

barmac opened this issue Dec 6, 2023 · 4 comments
Assignees
Labels
backlog Queued in backlog bug Something isn't working spring cleaning Could be cleaned up one day

Comments

@barmac
Copy link
Member

barmac commented Dec 6, 2023

Describe the Bug

When I run the starter example and try to drill-down to literal expression, the code hangs on an infinite loop. The browser tab is effectively dead.

When I pause the execution in debugger, it displays CodeMirror code:

image

Steps to Reproduce

  1. Run the starter example (needs adjustment to run locally, e.g. replace https with http, change path to DMN)
  2. Convert a decision to literal expression
  3. Try to drill-down

Expected Behavior

It should work.

Environment

Please complete the following information:

  • Browser: [e.g. IE 11, Chrome 69]
  • OS: [e.g. Windows 7]
  • Library version: [e.g. 2.0.0]
@barmac barmac added the bug Something isn't working label Dec 6, 2023
@barmac
Copy link
Member Author

barmac commented Dec 6, 2023

This does not occur if you import Viewer from 'dmn-js'. You have to use prebundled distros to reproduce the error.

@barmac
Copy link
Member Author

barmac commented Dec 6, 2023

Note that i got somehow value of NaN. This can be the reason of the failure.

@nikku nikku added the ready Ready to be worked on label Dec 12, 2023
@apmulet
Copy link

apmulet commented Jan 3, 2024

Hello ... I was trying the bundle distro this week , and found the same infinite loop cause i = NAN when trying edit and empty cell on new tables
Thanks for this comments... your doing a great Job !

@nikku nikku added spring cleaning Could be cleaned up one day backlog Queued in backlog and removed ready Ready to be worked on labels Mar 14, 2024
@nikku
Copy link
Member

nikku commented Mar 14, 2024

Moving to backlog and spring cleaning.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Queued in backlog bug Something isn't working spring cleaning Could be cleaned up one day
Development

No branches or pull requests

3 participants