lf-lang / lf-python-support Goto Github PK

• Bring back supporting C code to reactor-c
• Rename the repo to lf-python-support or lf-py-libor something like that

If the target language is Python, it is really easy to share the state variable as a pointer but not as value.
Here is an example of me incorrectly passing the pointer of a list to reactors.
Also, it will be better if the example here from the tutorial video can clarify the state value must not be pointer when share to downstream reactors.

It would be nice if we could simply write

print(f"Current tag is: {lf.tag()}")

and get a properly formatted output.
This would require implementing the __str__ method for the Tag type.

Lingua-Franca PR in question: lf-lang/lingua-franca#1348

It appears the latest version of Reactor-C-Py is breaking lingua-franca CI due to mismatch between const and non-const qualified functions/variables:

• https://github.com/lf-lang/lingua-franca/runs/8166085012?check_suite_focus=true: At first glance, it looks like our integration with reactor-c is causing a redefinition of the function, which is caused by a mismatch between const char and char.

Would be good if someone familiar with this repo and reactor-c has a look at the CI errors and makes the changes to ensure matching signatures between reactor-c-py and reactor-c

The C target has a few useful user-facing APIs such as DEBUG_PRINT() and error_print() that will allow the user to add thread-safe print statements in the body of reactions. The output (or lack thereof) of these APIs can then be controlled using the logging target specification.

This could be a very useful feature in the Python target as well.

To-Do

• Implement log functions
• Update the wiki once this feature is implemented.