Skip to content
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

Update pysmartdatamodels.py #57

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

fdrobnic
Copy link

Added the ngsi_ld_example_generator_str() function to be able to generate fake examples before the model is published.

Added the ngsi_ld_example_generator_str() function to be able to generate fake examples before the model is published.
@albertoabellagarcia
Copy link
Contributor

albertoabellagarcia commented Jan 24, 2024

We appreciate the contribution but we would need also the changes in the README file showing an example of execution of the new function. Thus we could test the contribution.

Added the ngsi_ld_example_generator_str() function to be able to generate fake examples before the model is published.
@fdrobnic
Copy link
Author

Updated, hope that it is sufficient.

Added the ngsi_ld_example_generator_str() function to be able to generate fake examples before the model is published.
@albertoabellagarcia
Copy link
Contributor

Thank you I am reviewing the code and what happens if the schema is not correct. this error is not controlled and therefore the function just throw an error without much information

@fdrobnic
Copy link
Author

fdrobnic commented May 8, 2024

Added a similar error handling as implemented elsewhere in the package.

@albertoabellagarcia
Copy link
Contributor

We are solving other issues about the package. please be patient with this approval

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants