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

[bug] Fix crashing when a SNode is destroyed after taichi resets #8393

Merged
merged 2 commits into from
Oct 27, 2023

Conversation

lin-hitonami
Copy link
Contributor

@lin-hitonami lin-hitonami commented Oct 27, 2023

Issue: fixes #8381

Brief Summary

🤖 Generated by Copilot at bf37155

Improve memory management and stability for multiple programs by adding a prog attribute and a validity check to SNodeTree.

Walkthrough

🤖 Generated by Copilot at bf37155

  • Add prog attribute to SNodeTree class to store the Program object that created the tree (link)
  • Prevent SNodeTree from being destroyed by a different or invalid program by checking the prog attribute in the destroy method (link)

@netlify
Copy link

netlify bot commented Oct 27, 2023

Deploy Preview for docsite-preview ready!

Name Link
🔨 Latest commit 1a49eef
🔍 Latest deploy log https://app.netlify.com/sites/docsite-preview/deploys/653b7dbdcccc8400085b7e01
😎 Deploy Preview https://deploy-preview-8393--docsite-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@lin-hitonami lin-hitonami merged commit ae83197 into master Oct 27, 2023
@lin-hitonami lin-hitonami deleted the fix-stannum branch October 27, 2023 10:02
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.

[v1.7.0][Bug] Fix error with stannum unit tests
2 participants