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

UsageGraphError: Graph.addNode: the "UBERON:0013702" node already exist in the graph. #30

Open
Tehsurfer opened this issue Jun 14, 2024 · 7 comments

Comments

@Tehsurfer
Copy link

Tehsurfer commented Jun 14, 2024

I am getting this error when running flatmap-viewer.

UsageGraphError: Graph.addNode: the "UBERON:0013702" node already exist in the graph. #30

I don't notice it affecting the functionality of the flatmap at all, but it does break the tests that get run on flatmapvuer

To reproduce:

  1. clone this branch:
    https://github.com/Tehsurfer/mapintegratedvuer/tree/marker-clustering

  2. Put the .env.local file I sent into the root directory of the repo if you don't already have those variables

  3. npm run serve should produce the error, scroll up to the top it should be one of the first 5

@dbrnz
Copy link
Contributor

dbrnz commented Jun 14, 2024

What map GUID is giving this error? On what map server??

@Tehsurfer
Copy link
Author

Tehsurfer commented Jun 14, 2024

flatmap-viewer2.9.3

https://github.com/AnatomicMaps/rat-flatmap/blob/027dc313b3ce6d84818eb47505ad7632143da624/manifest.json

mapcore-demo.org/devel/flatmap/v4/flatmap/

(Although the tests are running on https://mapcore-demo.org/current/flatmap/v3/ and have the same issue)

@Tehsurfer
Copy link
Author

Wait, that isn't the GUID is it, how to do you get the GUID?

@dbrnz
Copy link
Contributor

dbrnz commented Jun 14, 2024

All looks well with the code and the source data (it's a JSON file at here, and it contains just a single UBERON:0013702 node). Running your code doesn't show the error:
Screenshot 2024-06-14 at 2 47 49 PM

What I do get though, as a result of a sidebar search for Heart, is:
Screenshot 2024-06-14 at 2 52 26 PM

@Tehsurfer
Copy link
Author

hmm ok give me a moment to fix the other error messages and then I'll be able to see how to reproduce this one more easily

@Tehsurfer
Copy link
Author

Ah yeah sorry about that you actually need these steps to reproduce it:

  1. clone this branch:
    https://github.com/Tehsurfer/mapintegratedvuer/tree/marker-clustering

  2. Put the .env.local file I sent into the root directory of the repo if you don't already have those variables

  3. npm run serve

  4. Change the species of the map, top right:
    image

@dbrnz
Copy link
Contributor

dbrnz commented Jun 14, 2024

See my question on Slack to @alan-wu:

what exactly happens when a flatmap’s species is changed? Is a new flatmap instance created by calling MapManager.loadMap()? Or does the vuer remember previously loaded map instances and switch back??

There are no issues with flatmap-viewer when loading maps for a different species.

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

No branches or pull requests

2 participants