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.

Sign up for GitHub

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

Jump to bottom

DeprecationWarning: invalid escape sequence '\.' #3789

Closed
njzjz opened this issue May 16, 2024 · 1 comment · Fixed by #3793
Closed

DeprecationWarning: invalid escape sequence '\.' #3789

njzjz opened this issue May 16, 2024 · 1 comment · Fixed by #3793
Assignees
@njzjz
Labels
bug

Comments

@njzjz
Copy link
Member

njzjz commented May 16, 2024 

source/tests/pt/model/test_fitting_net.py:116
  /home/runner/work/deepmd-kit/deepmd-kit/source/tests/pt/model/test_fitting_net.py:116: DeprecationWarning: invalid escape sequence '\.'
    "filter_layers\.networks\.(\d).layers\.(\d)\.([a-z]+)", name
@njzjz njzjz added the bug label May 16, 2024
@njzjz
Copy link
Member Author

njzjz commented May 17, 2024

It looks to me W605 can fix it automatically. No need to manually fix it.

@njzjz njzjz self-assigned this May 17, 2024
njzjz added a commit to njzjz/deepmd-kit that referenced this issue May 17, 2024
@njzjz
style: enable W rules
9deb288 
Fix deepmodeling#3789.

Signed-off-by: Jinzhe Zeng <[email protected]>
@njzjz njzjz linked a pull request May 17, 2024 that will close this issue
style: enable W rules #3793
Merged
@njzjz njzjz mentioned this issue May 17, 2024
Merged
github-merge-queue bot pushed a commit that referenced this issue May 18, 2024
@njzjz @pre-commit-ci
style: enable W rules (#3793)
81b5949 
Fix #3789.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- **Documentation**
- Corrected a minor formatting issue in the quick start guide for
displaying a file path.

- **Chores**
  - Added a new linting rule for warnings to improve code quality.

- **Tests**
- Improved regular expression handling in test cases by using raw string
notation.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Signed-off-by: Jinzhe Zeng <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@njzjz njzjz closed this as completed May 18, 2024
njzjz added a commit to njzjz/deepmd-kit that referenced this issue Jul 2, 2024
@njzjz @pre-commit-ci
style: enable W rules (deepmodeling#3793)
96139dc 
Fix deepmodeling#3789.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

- **Documentation**
- Corrected a minor formatting issue in the quick start guide for
displaying a file path.

- **Chores**
  - Added a new linting rule for warnings to improve code quality.

- **Tests**
- Improved regular expression handling in test cases by using raw string
notation.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Signed-off-by: Jinzhe Zeng <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
(cherry picked from commit 81b5949)
Signed-off-by: Jinzhe Zeng <[email protected]>
njzjz added a commit that referenced this issue Jul 3, 2024
@njzjz @pre-commit-ci
style: enable W rules (#3793)
35ee52c 
Fix #3789.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

- **Documentation**
- Corrected a minor formatting issue in the quick start guide for
displaying a file path.

- **Chores**
  - Added a new linting rule for warnings to improve code quality.

- **Tests**
- Improved regular expression handling in test cases by using raw string
notation.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Signed-off-by: Jinzhe Zeng <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
(cherry picked from commit 81b5949)
Signed-off-by: Jinzhe Zeng <[email protected]>
mtaillefumier pushed a commit to mtaillefumier/deepmd-kit that referenced this issue Sep 18, 2024
@njzjz @pre-commit-ci
style: enable W rules (deepmodeling#3793)
d77521a 
Fix deepmodeling#3789.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

- **Documentation**
- Corrected a minor formatting issue in the quick start guide for
displaying a file path.

- **Chores**
  - Added a new linting rule for warnings to improve code quality.

- **Tests**
- Improved regular expression handling in test cases by using raw string
notation.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Signed-off-by: Jinzhe Zeng <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@njzjz njzjz

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

style: enable W rules njzjz/deepmd-kit
1 participant
@njzjz