Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@GopiGugan
Copy link
Contributor

@GopiGugan GopiGugan commented Sep 27, 2022

@GopiGugan
Copy link
Contributor Author

Search by country seems to be broken again:

d3.js:1497 Uncaught (in promise) TypeError: Cannot read properties of null (reading '__data__')
    at Selection.selection_datum [as datum] (d3.js:1497:20)
    at draw_cluster_box (drawtree.js:51:16)
    at select_cluster (search.js:322:3)
    at main_search (search.js:182:17)
    at async wrap_search (search.js:110:5)
    at async HTMLInputElement.<anonymous> (covizu.js:351:7)

@GopiGugan
Copy link
Contributor Author

Search is not picking up hits from recombinant clusters:
Screen Shot 2022-10-11 at 12 52 06 PM

@GopiGugan
Copy link
Contributor Author

Moving the slider resets the selected cluster to be first "Non Recombinant" cluster in the tree. Need to change this to select search hits from the closest cluster

@GopiGugan
Copy link
Contributor Author

For hits in the recombinant lineages, clicking the 'Previous' button displays the next bead

@GopiGugan
Copy link
Contributor Author

GopiGugan commented Oct 24, 2022

Checking the following:

  • Search
    • Search by accession number
    • Search by lineage
    • Search by country
    • Search by country (with date filter)
  • Tree filtering
  • Download Time-scaled tree (nwk)
  • Download Time-scaled tree (csv)
  • Download Beadplot (svg)
  • Download Beadplot (nwk)
  • Beadplot cutoff slider
  • Expanding Beadplot
  • Translated pages
  • Changing the colour of the tree based on divergence, collection date, etc.
  • Left/Right arrow key to move between beads
  • Tooltips

@GopiGugan
Copy link
Contributor Author

During a search, multiple time scales are being drawn:
Screen Shot 2022-10-24 at 7 41 40 AM

Add 'partial_redraw' to selectively redraw the time scale and tree without having to re-render certain elements
@GopiGugan GopiGugan merged commit 4ddcb65 into master Oct 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DOMException: illegal character?

3 participants