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

Adding share functionality to Graphing Calculator #601

Conversation

DavidShoe
Copy link
Contributor

@DavidShoe DavidShoe commented Jul 25, 2019

Description of the changes:

Use the new GitBitmap feature from the graph engine to get a stream to the current graph state. Then build up a HMLT share with that and the current set of equations as a key.

How changes were validated:

Manual testing

@HowardWolosky HowardWolosky changed the title Adding share functionality. Adding share functionality to Graphing Calculator Jul 25, 2019
@HowardWolosky HowardWolosky added graphing calculator Work items related to the graphing calculator feature. Enhancement and removed graphing calculator Work items related to the graphing calculator feature. labels Jul 25, 2019
@HowardWolosky
Copy link
Member

@DavidShoe - I don't actually see any file changes as part of the PR...

Working -> feature branch so can validate then pull to main project from fork.
Copy link
Contributor

@joseartrivera joseartrivera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

David, thank you for working on the share feature. This is looking great so far! I have left some comments for you to review, let me know your thoughts on them. Thanks again for the contribution!

@ghost ghost removed the needs author feedback label Jul 29, 2019
@sanderl sanderl merged commit c1efa3d into microsoft:feature/GraphingCalculator Aug 13, 2019
@microsoft microsoft deleted a comment from DavidShoe Feb 17, 2020
@microsoft microsoft deleted a comment from DavidShoe Feb 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
graphing calculator Work items related to the graphing calculator feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants