Inputs where Integers are to be used, crash when supplied other data types

[sudo-panda]

Examples,

• Temporal Transition Code
• Timer

[sudo-panda]

I would like to work on this issue.

[sudo-panda]

Can I add support for using parameters in Temporal Transition Code, or should I do that in another pull request.

[okanasik]

I think it is better to add this feature as part of parameterization project. It is not directly related with this issue.

[sudo-panda]

I solved it on the colab repo but I am unable to create a pull request due it not being a fork. Any ideas?

[okanasik]

I have written a method on the hangouts, did you try that?

you should have a fork of the visualstates as your personal repository then you should add this as another remote repository to your local-colab repository then you should push your branch that you would like to create pull request from to that forked remote repo finally through github ui you can create pull request from your personal repo I do not know any other way that you can create a pull request

Also do not forget that before working on any single issue, you should checkout the master branch. pull the latest version from the remote repo. create a new branch for the issue/feature make your changes and push your changes to the remote repo. finally create a pull request.

If you still have problems please let me know.

[sudo-panda]

Sorry I didn't get the notification for the message. I will try that method and if I get merge conflicts while trying pull from the remote repo should I solve it and commit or it is considered a bad practice.

[okanasik]

If you need to merge because of a conflict that is not related to current changes that you have build for the respective issue, it would be bad practice. However, if you need to merge because your changes overwrites the changes done between your last pull and this pull, it is fine. Also do not hesitate to break things, we can always fix things in git :)

[sudo-panda]

Alright, thanks

[okanasik]

fixed by #126