|
1 | | -<!--- Provide a general summary of your changes in the Title above --> |
2 | | -<!--- Please do remember to follow the contributing guidelines --> |
| 1 | +<!-- |
| 2 | +Thank you for contributing to BrainPy! |
| 3 | +Please fill out this template to help us review your pull request. |
| 4 | +See CONTRIBUTING.md for guidelines. |
| 5 | +--> |
| 6 | + |
| 7 | +## Summary |
| 8 | + |
| 9 | +<!-- Provide a clear, concise summary of your changes --> |
| 10 | + |
| 11 | +**Fixes #(issue number)** <!-- If this PR closes an issue, link it here --> |
| 12 | + |
| 13 | +## Type of Change |
| 14 | + |
| 15 | +<!-- Mark the appropriate option with an "x" --> |
| 16 | + |
| 17 | +- [ ] Bug fix (non-breaking change which fixes an issue) |
| 18 | +- [ ] New feature (non-breaking change which adds functionality) |
| 19 | +- [ ] Breaking change (fix or feature that would cause existing functionality to change) |
| 20 | +- [ ] Documentation update |
| 21 | +- [ ] Code refactoring (no functional changes) |
| 22 | +- [ ] Performance improvement |
| 23 | +- [ ] Test coverage improvement |
| 24 | +- [ ] Other (please describe): |
3 | 25 |
|
4 | 26 | ## Description |
5 | | -<!--- Why is this change required? What problem does it solve? --> |
6 | | -<!--- Describe your changes in detail here to communicate to the maintainers why this pull request should be accepted --> |
7 | | -<!--- Describe your technology stack here if not a documentation update --> |
8 | | -<!--- Tasklist format is recommended for all pull requests and is required for all draft pull requests. You can couple your description with the tasklist --> |
9 | | -<!--- If it fixes an open issue, please link to the issue here in the last line. --> |
10 | | - |
11 | | -## How Has This Been Tested |
12 | | -<!--- Please describe in detail how you tested your changes locally --> |
13 | | -<!--- Include details of your testing environment, and the tests you ran to --> |
14 | | -<!--- For example, markdown files should pass markdownlint locally according to the rules --> |
15 | | -<!--- See how your change affects other areas of the code, etc. --> |
16 | | - |
17 | | -## Types of changes |
18 | | -<!--- What types of changes does your code introduce? --> |
19 | | -<!--- Only left the line that best describes this pull request --> |
20 | | -- Bug fix (non-breaking change which fixes an issue) |
21 | | -- New feature (non-breaking change which adds functionality) |
22 | | -- Documentation (non-breaking change which updates documentation) |
23 | | -- Breaking change (fix or feature that would cause existing functionality to change) |
24 | | -- Code style (formatting, renaming) |
25 | | -- Refactoring (no functional changes, no api changes) |
26 | | -- Other (please describe here): |
| 27 | + |
| 28 | +**What does this PR do?** |
| 29 | +<!-- Describe your changes in detail --> |
| 30 | + |
| 31 | +**Why is this change needed?** |
| 32 | +<!-- Explain the motivation for this change --> |
| 33 | + |
| 34 | +**How does it work?** |
| 35 | +<!-- Provide technical details about your implementation --> |
| 36 | + |
| 37 | +## Changes Made |
| 38 | + |
| 39 | +<!-- List the main changes in this PR --> |
| 40 | + |
| 41 | +- |
| 42 | +- |
| 43 | +- |
| 44 | + |
| 45 | +## Testing |
| 46 | + |
| 47 | +**How has this been tested?** |
| 48 | +<!-- Describe the tests you ran to verify your changes --> |
| 49 | + |
| 50 | +```python |
| 51 | +# Example test code or commands used |
| 52 | +``` |
| 53 | + |
| 54 | +**Test environment:** |
| 55 | +- Python version: |
| 56 | +- BrainPy version: |
| 57 | +- JAX version: |
| 58 | +- OS: |
| 59 | + |
| 60 | +**Test coverage:** |
| 61 | +- [ ] Added new tests for this change |
| 62 | +- [ ] All existing tests pass |
| 63 | +- [ ] Manual testing performed |
| 64 | + |
| 65 | +## Performance Impact |
| 66 | + |
| 67 | +<!-- If applicable, describe any performance implications --> |
| 68 | + |
| 69 | +- [ ] No performance impact |
| 70 | +- [ ] Performance improved (provide benchmarks) |
| 71 | +- [ ] Performance may be affected (explain why and how much) |
| 72 | + |
| 73 | +## Documentation |
| 74 | + |
| 75 | +- [ ] Docstrings added/updated |
| 76 | +- [ ] Documentation updated (if needed) |
| 77 | +- [ ] Examples added/updated (if applicable) |
| 78 | +- [ ] CHANGELOG entry added (for notable changes) |
| 79 | + |
| 80 | +## Screenshots (if applicable) |
| 81 | + |
| 82 | +<!-- Add screenshots to help explain your changes --> |
| 83 | + |
| 84 | +## Breaking Changes |
| 85 | + |
| 86 | +<!-- If this is a breaking change, describe the impact and migration path --> |
| 87 | + |
| 88 | +**Breaking change details:** |
| 89 | +<!-- What breaks? How should users update their code? --> |
| 90 | + |
| 91 | +**Migration guide:** |
| 92 | +```python |
| 93 | +# Before (old way) |
| 94 | + |
| 95 | +# After (new way) |
| 96 | +``` |
27 | 97 |
|
28 | 98 | ## Checklist |
29 | | -<!--- Go over all the following points, and put an `x` in all the boxes that apply. --> |
30 | | -<!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> |
31 | | -- [ ] Code follows the code style of this project. |
32 | | -- [ ] Changes follow the **CONTRIBUTING** guidelines. |
33 | | -- [ ] Update necessary documentation accordingly. |
34 | | -- [ ] Lint and tests pass locally with the changes. |
35 | | -- [ ] Check issues and pull requests first. You don't want to duplicate effort. |
36 | | - |
37 | | -## Other information |
| 99 | + |
| 100 | +<!-- Ensure all items are complete before submitting --> |
| 101 | + |
| 102 | +- [ ] My code follows the project's style guidelines |
| 103 | +- [ ] I have performed a self-review of my code |
| 104 | +- [ ] I have commented my code, particularly in hard-to-understand areas |
| 105 | +- [ ] I have made corresponding changes to the documentation |
| 106 | +- [ ] My changes generate no new warnings |
| 107 | +- [ ] I have added tests that prove my fix is effective or that my feature works |
| 108 | +- [ ] New and existing unit tests pass locally with my changes |
| 109 | +- [ ] Any dependent changes have been merged and published |
| 110 | +- [ ] I have checked for duplicate PRs/issues |
| 111 | +- [ ] I have read and followed the [CONTRIBUTING](../CONTRIBUTING.md) guidelines |
| 112 | + |
| 113 | +## Additional Notes |
| 114 | + |
| 115 | +<!-- Any additional information that reviewers should know --> |
| 116 | + |
| 117 | +## Related Issues/PRs |
| 118 | + |
| 119 | +<!-- Link to related issues or pull requests --> |
| 120 | + |
| 121 | +- Related to # |
| 122 | +- Depends on # |
| 123 | +- Blocks # |
0 commit comments