-
Notifications
You must be signed in to change notification settings - Fork 5
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.
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
chore: define yaml classes for remaining elements in MODELS #307
chore: define yaml classes for remaining elements in MODELS #307
Conversation
def to_dto(self): | ||
raise NotImplementedError |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it is a mistake to define this as needed in YamlBase, as I have done currently.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You mean it should be defined in YamlBase instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, it doesnt matter that much. We want to skip dtos
completely, and map directly to domain. dtos are no longer needed, but we will need some clever ways to map to domain. WIP :)
Why is this pull request needed?
Add classes for remaining elements in MODELS, for use with V2.
What does this pull request change?
STAGE
SINGLE_SPEED_COMPRESSOR_TRAIN
VARIABLE_SPEED_COMPRESSOR_TRAIN
SIMPLIFIED_VARIABLE_SPEED_COMPRESSOR_TRAIN
VARIABLE_SPEED_COMPRESSOR_TRAIN_MULTIPLE_STREAMS_AND_PRESSURES
Issues related to this change:
https://equinor-ecalc.atlassian.net/browse/ECALC-505?atlOrigin=eyJpIjoiYzMwMGYwZTgyMDZhNGUwOTkzODAzZTliNWU0ZWNjMDAiLCJwIjoiaiJ9