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

Fix the Array Visualisation #1588

Merged
merged 5 commits into from
Mar 17, 2021
Merged

Fix the Array Visualisation #1588

merged 5 commits into from
Mar 17, 2021

Conversation

iamrecursion
Copy link
Contributor

@iamrecursion iamrecursion commented Mar 17, 2021

Pull Request Description

This PR fixes an issue with the Array visualisation code that was returning null. It also updates the way we do visualisations for Vectors, and improves some error messages in the Json module.

Closes #1584.
Closes #1576.
Closes #1586.
Closes #1585.

Important Notes

N/A

Checklist

Please include the following checklist in your PR:

  • The documentation has been updated if necessary.
  • All code conforms to the Scala, Java, and Rust style guides.
  • All documentation and configuration conforms to the markdown and YAML style guides.
  • All code has been tested where possible.

@iamrecursion iamrecursion merged commit e4e16a3 into main Mar 17, 2021
@iamrecursion iamrecursion deleted the wip/ara/array-vis branch March 17, 2021 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p-highest Should be completed ASAP
Projects
None yet
2 participants