MetaCell / MetaCell/geppetto-meta

GeppettoGraphVisualization requiring number as id for nodes, should be any

Open
#232 1 comment 0 reactions 0 assignees View on GitHub
geppetto-ui good first issue
Dominant language
JavaScript
Stars
6
Forks
4
PR merge metrics
No merged PRs in 30d

Description

https://github.com/MetaCell/geppetto-meta/blob/release/v1.0.0/geppetto.js/geppetto-ui/src/graph-visualization/Graph.js#L402

I don't see any reasons for this to be a number. We usually want there instance ids, which are strings instead.
We are getting an error on console when using strings atm.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start at geppetto.js/geppetto-ui/src/graph-visualization/Graph.js around line 402, where node IDs are handled. Reproduce the console error with a string instance ID and inspect the current number requirement. Done means graph nodes accept string IDs without the reported error; verify the existing graph-visualization behavior afterward.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.