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

[Model Issue]: bamm instead of samm in bom_as_specified #744

Open
gautam-e opened this issue Jun 3, 2024 · 4 comments
Open

[Model Issue]: bamm instead of samm in bom_as_specified #744

gautam-e opened this issue Jun 3, 2024 · 4 comments

Comments

@gautam-e
Copy link

gautam-e commented Jun 3, 2024

URL to Model with Issue

https://github.com/eclipse-tractusx/sldt-semantic-models/blob/main/io.catenax.bom_as_specified/1.0.1/BomAsSpecified.ttl

Issue Description

The model above references only BAMM and not SAMM.
If I understand correctly, BAMM has been renamed / replaces by SAMM. In that case, the model needs to be corrected.

@agg3fe
Copy link
Contributor

agg3fe commented Jun 6, 2024

@gautam-e Hi, Would be great if you or someone from your team can join the weekly meeting on Monday. We can then discuss this issue and provide necessary approval.

@gautam-e
Copy link
Author

@agg3fe I'm not aware of the Monday meeting. I also don't work on Mondays. The request is fairly simple to understand though so I guess it could be done without me(?). Let me know if there's any other way I can help.

@agg3fe
Copy link
Contributor

agg3fe commented Jun 17, 2024

@agg3fe I'm not aware of the Monday meeting. I also don't work on Mondays. The request is fairly simple to understand though so I guess it could be done without me(?). Let me know if there's any other way I can help.

@gautam-e Ok, in that case please create the Pull Request for the new model in Samm and let me know when done. I will review and provide the feedback.
But before the final approval, someone from your team has to join the meeting as this is the governance process that we follow to add new models. This is required so that all the other catena-x members are also aware of new samm model.

@gautam-e gautam-e mentioned this issue Jun 18, 2024
28 tasks
@gautam-e
Copy link
Author

@agg3fe done. Please have a look #760
It looks like I don't have the permission to do this though. But perhaps you can take over the PR or something.

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 a pull request may close this issue.

2 participants