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

Update grafer to v0.5.1 #443

Merged
merged 2 commits into from
Sep 9, 2021
Merged

Update grafer to v0.5.1 #443

merged 2 commits into from
Sep 9, 2021

Conversation

adamocarolli
Copy link
Contributor

@adamocarolli adamocarolli commented Sep 8, 2021

Description

Update grafer.

Changes

  • Update grafer to v0.5.0 in package.
  • Update yarn.lock

Considerations

  • Updating Grafer removes clipping issues experienced when a foreground layer has the same z-index/viewing-frustum as the background layer.

Future Work

  • Zooming around the graph on a Mac's trackpad is difficult on the new Grafer version - Done in v0.5.1
  • Label font on the Rings is difficult to read - Done in v0.5.1

Testing

  • Try out graphs of various sizes. Perform queries etc.

Artifacts

Screen Shot 2021-09-08 at 1 53 00 PM

#442

@adamocarolli adamocarolli changed the title Update grafer to v0.5.0 [WIP] Update grafer to v0.5.0 Sep 8, 2021
@RosaRomeroGomez
Copy link
Contributor

Looks great! However, I found (a probably unrelated issue) when trying to run some random queries in some models. It seems you need to check for undefined ids.

image

@adamocarolli
Copy link
Contributor Author

Looks great! However, I found (a probably unrelated issue) when trying to run some random queries in some models. It seems you need to check for undefined ids.

image

Yeah this seems unrelated

@adamocarolli adamocarolli changed the title [WIP] Update grafer to v0.5.0 Update grafer to v0.5.1 Sep 9, 2021
@adamocarolli adamocarolli merged commit 561303c into main Sep 9, 2021
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.

2 participants