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

In web GUI when clicking on tree, getting "Type Error: undefined is not an iterable" #1547

Closed
corneliusroemer opened this issue Nov 12, 2024 · 7 comments
Labels
package: nextclade_web t:bug Type: bug, error, something isn't working

Comments

@corneliusroemer
Copy link
Member

Not sure how to reproduce, but seems worth looking into:

Error message: TypeError: undefined is not iterable (cannot read property Symbol(Symbol.iterator))

Nextclade version 3.10.0 (commit: 59e757f, branch: master)

Memory available: 3586 MBytes

User agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/130.0.0.0 Safari/537.36

Browser details: {"browser":{"name":"Chrome","version":"130.0.0.0"},"os":{"name":"macOS","version":"10.15.7","versionName":"Catalina"},"platform":{"type":"desktop","vendor":"Apple"},"engine":{"name":"Blink"}}

Call stack:

TypeError: undefined is not iterable (cannot read property Symbol(Symbol.iterator))
    at b._setSelectedNodes (https://master.clades.nextstrain.org/_next/static/chunks/412.20e66927d3aaae82.js:1:132108)
    at b.render (https://master.clades.nextstrain.org/_next/static/chunks/412.20e66927d3aaae82.js:1:128897)
    at d.value (https://master.clades.nextstrain.org/_next/static/chunks/412.20e66927d3aaae82.js:1:173407)
    at d.value (https://master.clades.nextstrain.org/_next/static/chunks/412.20e66927d3aaae82.js:1:174951)
    at hg (https://master.clades.nextstrain.org/_next/static/chunks/framework-5774a691a4029c28.js:1:99176)
    at hf (https://master.clades.nextstrain.org/_next/static/chunks/framework-5774a691a4029c28.js:1:98946)
    at hf (https://master.clades.nextstrain.org/_next/static/chunks/framework-5774a691a4029c28.js:1:98841)
    at hf (https://master.clades.nextstrain.org/_next/static/chunks/framework-5774a691a4029c28.js:1:98841)
    at h8 (https://master.clades.nextstrain.org/_next/static/chunks/framework-5774a691a4029c28.js:1:111693)
    at h7 (https://master.clades.nextstrain.org/_next/static/chunks/framework-5774a691a4029c28.js:1:108246)

I ran Nextclade like this:
https://master.clades.nextstrain.org/?dataset-name=nextstrain/sars-cov-2/BA.2.86&input-tree=https://nextstrain.org/charon/getDataset?prefix=staging/nextclade/sars-cov-2/BA.2.86

With data from GISAID shared via Slack

What's strange is that this error only happened after roughly the following sequence:

  1. Run Nextclade
  2. Navigate to tree
  3. Do some filtering/coloring on tree
  4. Navigating back to Nextclade results table
  5. Applying an AA filter like ORF1a:1639 to the results table
  6. Then clicking again on tree, this is when Nextclade crashed
@corneliusroemer corneliusroemer added t:bug Type: bug, error, something isn't working good first issue Good for newcomers help wanted Extra attention is needed needs triage Mark for review and label assignment package: nextclade_web and removed good first issue Good for newcomers help wanted Extra attention is needed labels Nov 12, 2024
@corneliusroemer
Copy link
Member Author

corneliusroemer commented Nov 12, 2024

@corneliusroemer
Copy link
Member Author

corneliusroemer commented Nov 12, 2024

I think this is an upstream bug, as I've encountered it outside of Auspice as well: nextstrain/auspice#1895

We could potentially handle Auspice bugs like this better and not crash all of Nextclade, just say "Auspice crashed" and still allow the user to navigate back to the results table.

@corneliusroemer
Copy link
Member Author

Should we revert #1546 until upstream fixes this? @ivan-aksamentov

@ivan-aksamentov
Copy link
Member

@corneliusroemer Done (should be up soon)

@corneliusroemer
Copy link
Member Author

Great thanks! What do you think about not crashing on Auspice errors? Probably not really that important as Auspice crashes rarely but if it's easy to do :)

@ivan-aksamentov ivan-aksamentov removed the needs triage Mark for review and label assignment label Nov 12, 2024
@ivan-aksamentov
Copy link
Member

ivan-aksamentov commented Nov 12, 2024

@corneliusroemer I don't have an immediate idea of how to do this - there's React components which can probably be isolated with an error boundary. But there is also a more global Redux stuff, which is more difficult to isolate. It's quite tightly integrated actually. Might take some work to figure out. I prefer nothing to be crashing ideally :)

@ivan-aksamentov
Copy link
Member

I'll close this, because there's not much actionable in Nextclade. We'll wait for a new version from dependabot and upgrade as usual.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: nextclade_web t:bug Type: bug, error, something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants