Support for other PJON config constants?

[akadlec]

Is possible to add support for other PJON config constants? I'm using TSA strategy and it would be great to be able to configure eg. TSA_RESPONSE_TIME_OUT or TS_RESPONSE_TIME_OUT for TS strategy

[gioblu]

Ciao @akadlec In PJON 13.0 and the new version of PJON-cython we have only ThroughSerial which implements the old ThroughSerialAsync with some minor fixes and optimizations, by the way could you try the new version along with PJON 13.0 ? That would be a great help to verify if all works fine.

[akadlec]

Hi @gioblu yes I know it. I could test v13.0

[xlfe]

@akadlec the challenge will be that - at the moment - (at least from v12) those constants are compile time constants, ie cannot be set at runtime

see

PJON-cython/setup.py

Lines 39 to 43 in eea8ad8

	for s in ['TS', 'TSA', 'GUDP', 'LUDP']:
	strategies.extend([
	'-DPJON_INCLUDE_{}'.format(s),
	'-D{}_RESPONSE_TIME_OUT=100000'.format(s)
	]

One option would be to fork the repo, set them in that file (setup.py) to your needs and then pip install from your forked version of PJON-cython.

Or, as @gioblu suggestes in v13 you may not need to customise them anymore?

Let me know how you go testing v13

cheers