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

[Lang] Remove deprecated sparse_matrix_builder function #7942

Merged
merged 1 commit into from
May 8, 2023

Conversation

lin-hitonami
Copy link
Contributor

@lin-hitonami lin-hitonami commented May 6, 2023

Issue: #

Brief Summary

🤖 Generated by Copilot at 6809c08

Removed deprecated sparse_matrix_builder class and related code from sparse_matrix.py and test_deprecation.py. Simplified the sparse_matrix module.

Walkthrough

🤖 Generated by Copilot at 6809c08

  • Remove deprecated sparse_matrix_builder class and its alias from sparse_matrix.py (link)
  • Remove unused warnings and annotations modules from sparse_matrix.py (link, link)
  • Remove test case for deprecated sparse_matrix_builder class from test_deprecation.py (link)

@netlify
Copy link

netlify bot commented May 6, 2023

Deploy Preview for docsite-preview ready!

Name Link
🔨 Latest commit 6809c08
🔍 Latest deploy log https://app.netlify.com/sites/docsite-preview/deploys/645605407cdb4b000811f099
😎 Deploy Preview https://deploy-preview-7942--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 settings.

Copy link
Contributor

@feisuzhu feisuzhu left a comment

Choose a reason for hiding this comment

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

LGTM!

@lin-hitonami lin-hitonami merged commit 0af5994 into taichi-dev:master May 8, 2023
@lin-hitonami lin-hitonami deleted the remove-sparse branch May 8, 2023 10:35
feisuzhu pushed a commit that referenced this pull request May 9, 2023
Issue: #

### Brief Summary

<!--
copilot:summary
-->
### <samp>🤖 Generated by Copilot at 6809c08</samp>

Removed deprecated `sparse_matrix_builder` class and related code from
`sparse_matrix.py` and `test_deprecation.py`. Simplified the
`sparse_matrix` module.

### Walkthrough

<!--
copilot:walkthrough
-->
### <samp>🤖 Generated by Copilot at 6809c08</samp>

* Remove deprecated `sparse_matrix_builder` class and its alias from
`sparse_matrix.py`
([link](https://github.com/taichi-dev/taichi/pull/7942/files?diff=unified&w=0#diff-e788aae97f9ad8507a4eebc224175b2f9ad6f14359999ca0ea99ad4503fc9dcaL297-R296))
* Remove unused `warnings` and `annotations` modules from
`sparse_matrix.py`
([link](https://github.com/taichi-dev/taichi/pull/7942/files?diff=unified&w=0#diff-e788aae97f9ad8507a4eebc224175b2f9ad6f14359999ca0ea99ad4503fc9dcaL1),
[link](https://github.com/taichi-dev/taichi/pull/7942/files?diff=unified&w=0#diff-e788aae97f9ad8507a4eebc224175b2f9ad6f14359999ca0ea99ad4503fc9dcaL10-R9))
* Remove test case for deprecated `sparse_matrix_builder` class from
`test_deprecation.py`
([link](https://github.com/taichi-dev/taichi/pull/7942/files?diff=unified&w=0#diff-8981be068a363e39524dc2e29d28d4c13a097d0037fc3a1176b249ce5bf35ef8L145-L153))
quadpixels pushed a commit to quadpixels/taichi that referenced this pull request May 13, 2023
)

Issue: #

### Brief Summary

<!--
copilot:summary
-->
### <samp>🤖 Generated by Copilot at 6809c08</samp>

Removed deprecated `sparse_matrix_builder` class and related code from
`sparse_matrix.py` and `test_deprecation.py`. Simplified the
`sparse_matrix` module.

### Walkthrough

<!--
copilot:walkthrough
-->
### <samp>🤖 Generated by Copilot at 6809c08</samp>

* Remove deprecated `sparse_matrix_builder` class and its alias from
`sparse_matrix.py`
([link](https://github.com/taichi-dev/taichi/pull/7942/files?diff=unified&w=0#diff-e788aae97f9ad8507a4eebc224175b2f9ad6f14359999ca0ea99ad4503fc9dcaL297-R296))
* Remove unused `warnings` and `annotations` modules from
`sparse_matrix.py`
([link](https://github.com/taichi-dev/taichi/pull/7942/files?diff=unified&w=0#diff-e788aae97f9ad8507a4eebc224175b2f9ad6f14359999ca0ea99ad4503fc9dcaL1),
[link](https://github.com/taichi-dev/taichi/pull/7942/files?diff=unified&w=0#diff-e788aae97f9ad8507a4eebc224175b2f9ad6f14359999ca0ea99ad4503fc9dcaL10-R9))
* Remove test case for deprecated `sparse_matrix_builder` class from
`test_deprecation.py`
([link](https://github.com/taichi-dev/taichi/pull/7942/files?diff=unified&w=0#diff-8981be068a363e39524dc2e29d28d4c13a097d0037fc3a1176b249ce5bf35ef8L145-L153))
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