Fix 2 Python syntax warnings#3172
Merged
Merged
Conversation
aurghob
approved these changes
Oct 11, 2024
aurghob
added a commit
that referenced
this pull request
Nov 19, 2024
* Fix the issue with parsing sentences expressing the decade with century and number with unit in Spanish (#3154) * Date range without accents - Draft commit * Date range without accents - Fix cannot recognize Spanish written in the 90s * Date range without accents - Revert local nuget config and program.cs change * DateRangeWithoutAccents - Resolved comments --------- Co-authored-by: Michael Wang (Centific Technologies Inc) <v-michwang@microsoft.com> * Bump NuGet.CommandLine in /.NET/Microsoft.Recognizers.Text (#3119) Bumps [NuGet.CommandLine](https://github.com/NuGet/NuGet.Client) from 5.9.3 to 5.11.5. - [Release notes](https://github.com/NuGet/NuGet.Client/releases) - [Commits](https://github.com/NuGet/NuGet.Client/commits) --- updated-dependencies: - dependency-name: NuGet.CommandLine dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [EN DateTimeV2] Support for longer span date time (#3166) * Datetime for longer span - local draft commit * DateTimeForLongerSpan - Implement for from --------- Co-authored-by: Michael Wang (Centific Technologies Inc) <v-michwang@microsoft.com> * Support mixed length unit foot and inch (#3169) * SupportMixedUnitLengthNotRecognized - Implemented * SupportMixedUnitLengthNotRecognized - Update test cases * SupportMixedUnitLengthNotRecognized - Update one test case to trigger rebuild --------- Co-authored-by: Michael Wang (Centific Technologies Inc) <v-michwang@microsoft.com> * FixPythonSyntaxWarnings - Initial commit (#3172) Co-authored-by: Michael Wang (Centific Technologies Inc) <v-michwang@microsoft.com> * Support French number sub type - Initial commit (#3173) Co-authored-by: Michael Wang (Centific Technologies Inc) <v-michwang@microsoft.com> * FixTestDateTimeGermanNRE - Fixed (#3175) Co-authored-by: Michael Wang (Centific Technologies Inc) <v-michwang@microsoft.com> * SimpleConsoleAppSupportMultiLanguage - Initial implemention (#3177) Co-authored-by: Michael Wang (Centific Technologies Inc) <v-michwang@microsoft.com> * [EN DateTimeV2] Add support for time range with duration (#3174) * Datetime for longer span - local draft commit * DateTimeForLongerSpan - Implement for from * DateTimeForLongerSpan - from datetime for duration local commit * DateTimeForLongerSpan - Parsing time period with duration initial commit * DateTimeForLongerSpan - Parsing time period with duration update test cases * DateTimeForLongerSpan - Add support for time range with duration - Update test case to trigger rebuild * DateTimeForLongerSpan - Add support for time range with duration - Update test case to trigger rebuild --------- Co-authored-by: Michael Wang (Centific Technologies Inc) <v-michwang@microsoft.com> * [EN DateTimeV2] Support [duration]+starting+[datetime] pattern as date range or date time range (#3178) * DateTimeForLongerSpanForStarting - Initial implemention * Support duration starting datetime pattern - initial commit * Support duration starting datetime pattern - update test cases not supported attribute --------- Co-authored-by: Michael Wang (Centific Technologies Inc) <v-michwang@microsoft.com> --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Michael <xiaofeng200485@163.com> Co-authored-by: Michael Wang (Centific Technologies Inc) <v-michwang@microsoft.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Aurgho Bhattacharjee <aurghob@microsoft.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix 2 Python syntax warnings as below.
/usr/local/lib/python3.10/site-packages/recognizers_date_time/date_time/utilities.py:552: SyntaxWarning: 'int' object is not subscriptable; perhaps you missed a comma?
bias = 0[0]
/usr/local/lib/python3.10/site-packages/recognizers_number/number/extractors.py:37: SyntaxWarning: "is" with a literal. Did you mean "=="?
if source is None or len(source.strip()) is 0: