-
Notifications
You must be signed in to change notification settings - Fork 4
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
Need to programmatically add XML mappings to existing class (in Shale we programmatically update xml_mappings
)
#70
Comments
The PR is here: |
@kwkwan the internals of lutaml-model is very different from Shale. Can you explain what exactly you want to do in this use case with some code? Thanks. |
For the support of dynamic creation of the classes (e.g. CityGML), The original
After the dynamic creation of the classes, it is expected to have the new keys with the original keys.
|
xml_mappings
)
@HassanAkbar this is blocking @kwkwan 's work on migrating away from Shale. Can you please help? Thanks. |
xml_mappings
)xml_mappings
)
The use case is described as follows.
This is a valid use case and we should make it easy for the user to adopt. i.e., we don't want the user to hack our internal mapping (e.g. hacking |
@ronaldtse I'm looking into it, will let you know if I have any questions. |
Hi @HassanAkbar Two issues are related to this issue:
I guess this issue may able be fixed if those two issues are fixed. |
@kwkwan these two issues have been fixed. Can you check if this issue can be closed? Thanks! |
When I ran
bundle exec rake
in the branchmigrate-lutaml-model
of the repolutaml/xmi
. I got the following error.The problem may due to the methods to access
xml_mappings
. It may due to the changes of the mechanism to handle it.The text was updated successfully, but these errors were encountered: