Skip to content

rtc: Add methods to build CesionL2, CesionL1, and CesionL0 from other data models #186

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
Merged
merged 3 commits into from
Feb 9, 2021
Merged

rtc: Add methods to build CesionL2, CesionL1, and CesionL0 from other data models #186

merged 3 commits into from
Feb 9, 2021

Conversation

@jtrh
Copy link
Contributor

@jtrh jtrh commented Feb 8, 2021 •
edited
Loading

  • data_models: Add method to create CesionL1 from CesionL2.
  • data_models: Add method to create CesionL0 from CesionL1.
  • data_models_aec: Add method to create CesionL2 from CesionAecXml.

@jtrh jtrh self-assigned this Feb 8, 2021
@jtrh jtrh changed the title Feature/add rtc cesion data model converters rtc: Add methods to build CesionL2, CesionL1, and CesionL0 from other data models Feb 8, 2021
@jtrh jtrh force-pushed the feature/add-rtc-cesion-data-model-converters branch from 2ad8976 to 950e539 Compare February 8, 2021 23:17
@jtrh jtrh marked this pull request as ready for review February 8, 2021 23:21
@jtrh jtrh requested review from glarrain and yaselc February 8, 2021 23:21
@jtrh jtrh merged commit 7fe7c40 into develop Feb 9, 2021
@jtrh jtrh deleted the feature/add-rtc-cesion-data-model-converters branch February 9, 2021 13:06
@jtrh jtrh mentioned this pull request Feb 9, 2021
Merged
yaselc
yaselc reviewed Feb 9, 2021
View reviewed changes
Copy link

@yaselc yaselc left a comment

Choose a reason for hiding this comment

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

✔️

glarrain
glarrain reviewed Feb 9, 2021
View reviewed changes
Copy link
Contributor

@glarrain glarrain left a comment

Choose a reason for hiding this comment

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

👍🏼

@jtrobles-cdd jtrobles-cdd added enhancement New feature or request and removed feature labels Feb 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@glarrain glarrain glarrain left review comments

@yaselc yaselc yaselc left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@jtrh jtrh

Labels

component: rtc enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@jtrh @glarrain @yaselc @jtrobles-cdd