Logger python

Here I am updating the API for python and now use python's stack when printing an error.

changed milestone to %Continuous Simulation Data Stream

added 1 commit

• 4b3ebaa2 - Add docstring

Compare with previous version

changed target branch from logger_reader to master

added 80 commits

• 4b3ebaa2...c78c1aa2 - 76 commits from branch master
• add64e71 - Start implementation of PyObjectReader
• 06e8c26d - Implement new API
• 7d6dd3f5 - Add docstring
• 4d5dd352 - Format

Compare with previous version

unmarked as a Work In Progress

assigned to @matthieu and unassigned @nnrw56

Hey, pedro does not have the time to review it. Can you do it Matthieu? Nothing fancy here, just python related stuff.

Can you summarize in a paragraph what the changes are? This will help me go through the logic of what is going on. Thanks!

Previously the API was a bit too dirty, I am cleaning it in order to slowly get towards the following one https://gitlab.cosma.dur.ac.uk/swift/swiftsim/-/wikis/logger%20reader%20api.

I have created my own error function (error_python) that prints the python stack if we are compiling with python.

I am creating a python class for the logger (struct PyObjectReader), it's function __init__ (Reader_init) along with the destructor (Reader_dealloc) and the two required function for the keyword with (__enter__ and __exit__ in python, pyEnter and pyExit in my code C). The list libloggerReaderMethods will give you details (and docstring) about all the methods owned by my new class.

How familiar are you with python in C? If you wish I can deeply comment the merge request.

added 1 commit

• a26c653d - First correction from review

Compare with previous version

added 1 commit

• 33d63c79 - Update format + change makefile variable name

Compare with previous version

I have done the required changes.

resolved all threads

merged

mentioned in commit ac2362fb