Skip to content

Parser #270

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

Merged
adeas31 merged 12 commits into from
May 3, 2025
Merged

Parser #270

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
103237a
[OMParser] use a single entry point for OMParser
syntron Apr 24, 2025
36a54f0
[OMTypedParser] use a single entry point for OMTypedParser
syntron Apr 24, 2025
7f74995
[OMCSessionBase] fix exception handling for pyparsing.ParseException
syntron Apr 24, 2025
12edd0d
[OMCSessionBase] simplify the two use cases of OMParser.om_parse_basic()
syntron Apr 24, 2025
8ad5efa
Merge remote-tracking branch 'origin' into parser
adeas31 Apr 28, 2025
fbd5c50
Return unchanged result from `_ask_with_fallback`
adeas31 Apr 29, 2025
2f93ad3
[OMParser] use a single entry point for OMParser
syntron Apr 24, 2025
7c5be8c
[OMTypedParser] use a single entry point for OMTypedParser
syntron Apr 24, 2025
e0bc792
[OMCSessionBase] fix exception handling for pyparsing.ParseException
syntron Apr 24, 2025
a906365
[OMCSessionBase] simplify the two use cases of OMParser.om_parse_basic()
syntron Apr 24, 2025
6cfe458
Merge branch 'parser' of https://github.com/syntron/OMPython into parser
adeas31 Apr 30, 2025
6a6b501
Do not strip the result
adeas31 Apr 30, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
[OMCSessionBase] fix exception handling for pyparsing.ParseException 
* fix ex.message => ex.msg
  • Loading branch information
@syntron
syntron committed Apr 29, 2025
commit e0bc792f5ab4692602ab3b38f2b28adc679df07e
8 changes: 4 additions & 4 deletions OMPython/OMCSession.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,7 @@ def getClassComment(self, className):
return self.ask('getClassComment', className)
except pyparsing.ParseException as ex:
logger.warning("Method 'getClassComment' failed for %s", className)
logger.warning('OMTypedParser error: %s', ex.message)
logger.warning('OMTypedParser error: %s', ex.msg)
return 'No description available'

def getNthComponent(self, className, comp_id):
Expand Down Expand Up @@ -229,7 +229,7 @@ def getParameterValue(self, className, parameterName):
try:
return self.ask('getParameterValue', f'{className}, {parameterName}')
except pyparsing.ParseException as ex:
logger.warning('OMTypedParser error: %s', ex.message)
logger.warning('OMTypedParser error: %s', ex.msg)
return ""

def getComponentModifierNames(self, className, componentName):
Expand All @@ -240,7 +240,7 @@ def getComponentModifierValue(self, className, componentName):
# FIXME: OMPython exception UnboundLocalError exception for 'Modelica.Fluid.Machines.ControlledPump'
return self.ask('getComponentModifierValue', f'{className}, {componentName}')
except pyparsing.ParseException as ex:
logger.warning('OMTypedParser error: %s', ex.message)
logger.warning('OMTypedParser error: %s', ex.msg)
result = self.ask('getComponentModifierValue', f'{className}, {componentName}', parsed=False)
try:
answer = om_parser_basic(result)
Expand All @@ -257,7 +257,7 @@ def getExtendsModifierValue(self, className, extendsName, modifierName):
# FIXME: OMPython exception UnboundLocalError exception for 'Modelica.Fluid.Machines.ControlledPump'
return self.ask('getExtendsModifierValue', f'{className}, {extendsName}, {modifierName}')
except pyparsing.ParseException as ex:
logger.warning('OMTypedParser error: %s', ex.message)
logger.warning('OMTypedParser error: %s', ex.msg)
result = self.ask('getExtendsModifierValue', f'{className}, {extendsName}, {modifierName}', parsed=False)
try:
answer = om_parser_basic(result)
Expand Down