Skip to content

Commit

Permalink
Clean up some Python warnings coming out during builds
Browse files Browse the repository at this point in the history
  • Loading branch information
Myoldmopar committed Feb 12, 2024
1 parent bbf0a32 commit 62e0cab
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions doc/tools/parse_output_variables.py
Original file line number Diff line number Diff line change
Expand Up @@ -213,13 +213,13 @@ def main():
file_contents = f.read()

# first warn about commented lines
p = re.compile('//\s*SetupOutputVariable')
p = re.compile(r'//\s*SetupOutputVariable')
matches = p.findall(file_contents)

if len(matches) > 0:
print("File %s contains commented SetupOutputVariable calls; output may be flawed" % file_name)

p = re.compile('SetupOutputVariable\([^,]*,[^,]*,[^,]*,[^,]*,[^,]*,[^,]*,[^;]*;')
p = re.compile(r'SetupOutputVariable\([^,]*,[^,]*,[^,]*,[^,]*,[^,]*,[^,]*,[^;]*;')
matches = p.findall(file_contents)

for match in matches:
Expand Down
2 changes: 1 addition & 1 deletion idd/schema/idd_parser.py
Original file line number Diff line number Diff line change
Expand Up @@ -805,7 +805,7 @@ def next_token(data):
return TOKEN_UNITS

return TOKEN_NONE
elif c.isalpha() or c in '-:.#/\[]{}_@$%^&*()|+=<>?\'"~':
elif c.isalpha() or c in r'-:.#/\[]{}_@$%^&*()|+=<>?\'"~':
if c == 'A':
if parse_number(data) is not None:
return TOKEN_A
Expand Down

5 comments on commit 62e0cab

@nrel-bot
Copy link

Choose a reason for hiding this comment

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

MiscFixesMay2023 (Myoldmopar) - Win64-Windows-10-VisualStudio-16: OK (2761 of 2761 tests passed, 0 test warnings)

Build Badge Test Badge

@nrel-bot-2c
Copy link

Choose a reason for hiding this comment

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

MiscFixesMay2023 (Myoldmopar) - x86_64-Linux-Ubuntu-22.04-gcc-11.4: OK (3589 of 3589 tests passed, 0 test warnings)

Build Badge Test Badge

@nrel-bot-3
Copy link

Choose a reason for hiding this comment

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

MiscFixesMay2023 (Myoldmopar) - x86_64-MacOS-10.18-clang-15.0.0: OK (3547 of 3548 tests passed, 0 test warnings)

Build Badge Test Badge

@nrel-bot-2
Copy link

Choose a reason for hiding this comment

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

MiscFixesMay2023 (Myoldmopar) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-UnitTestsCoverage-Debug: OK (1974 of 1974 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

@nrel-bot-2c
Copy link

Choose a reason for hiding this comment

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

MiscFixesMay2023 (Myoldmopar) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-IntegrationCoverage-Debug: OK (790 of 790 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

Please sign in to comment.