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

Add (py)collimator to the tools list #637

Merged
merged 2 commits into from
Jul 12, 2024

Conversation

jp-andre
Copy link
Contributor

Add Collimator and our companion Python package pycollimator to the list of tools.

As a web platform, interface was set to GUI and platforms to all of them.

Copy link
Collaborator

@chrbertsch chrbertsch left a comment

Choose a reason for hiding this comment

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

It would be great if you could provide provide compatibility information (see https://github.com/modelica/fmi-standard.org/blob/main/CONTRIBUTING.md#compatibility-information), i.e. how you tested your FMI implementation? (Could also be in a separate Pull Request.)
Benefit: your tool will get the "yellow star" and be listed on the top.

@jp-andre
Copy link
Contributor Author

It would be great if you could provide provide compatibility information (see https://github.com/modelica/fmi-standard.org/blob/main/CONTRIBUTING.md#compatibility-information), i.e. how you tested your FMI implementation? (Could also be in a separate Pull Request.) Benefit: your tool will get the "yellow star" and be listed on the top.

This looks like it could be very valuable information indeed. As it might take a bit more time to compile this data, I will take a note to open another PR here. Thanks! (and I assume someone else will come and click the merge button on this PR)

@chrbertsch chrbertsch merged commit 4484b40 into modelica:main Jul 12, 2024
2 checks passed
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