Concurrency Tools - Python Versionitis

[AdvancedImagingUTSW]

Interestingly enough, it does seem that the concurrency tools dependency for threading.excepthook has some strange Python version-specific availability.

3.10.1 - threading.excepthook available
3.9.9 - threading.excepthook available.
3.8.12 - threading.excepthook available.
3.7.12 - Not available!
3.6.15 - Not available!
3.5.9 - Not available!

I believe we are using Python 3.7. Would love to hear what version of Python everyone else has in their environments.

[AdvancedImagingUTSW]

After firing up the acquisition computer and looking, we have Python 3.7.7 in the main environment that I have been working with.

In a separate environment, I have 3.9.0

Was able to execute past the previous error, but ran into another AttributeError:

(aslm) C:\Users\Spectral\Documents\GitHub\ASLM\src>python main.py
Traceback (most recent call last):
File "C:\Users\Spectral\Documents\GitHub\ASLM\src\main.py", line 36, in
main()
File "C:\Users\Spectral\Documents\GitHub\ASLM\src\main.py", line 32, in main
app = controller(root, configuration_path, experiment_path, etl_constants_path, args)
File "C:\Users\Spectral\Documents\GitHub\ASLM\src\controller\aslm_controller.py", line 27, in init
self.model = Model(args, configuration_path, experiment_path, etl_constants_path)
File "C:\Users\Spectral\Documents\GitHub\ASLM\src\model\aslm_model.py", line 27, in init
self.cam = start_camera(self.configuration, 0, self.verbose)
File "C:\Users\Spectral\Documents\GitHub\ASLM\src\model\aslm_model_functions.py", line 14, in start_camera
cam = ObjectInSubprocess(CameraModel(camera_id, verbose))
File "C:\Users\Spectral\Documents\GitHub\ASLM\src\model\concurrency\concurrency_tools.py", line 497, in init
name=initializer.name,
AttributeError: 'Camera' object has no attribute 'name'

[codeCollision4]

I would try running the camera code without the Concurrency tools and see if you get the same error. I can investigate this after I finish the widgets.

[AdvancedImagingUTSW]

It definitely works without the concurrency tools, but I started to notice the GUI being a bit laggy once the devices were all instantiated so I tried it.

[annie-xd-wang]

When using the ObjectInSubprocess from concurrency_tools.py, you should use like this cam = ObjectInSubprocess(CameraModel, camera_id, verbose). The class name is the the initializer, the args should be as *initargs. I didn't debug it, but you could try this way to see whether it works.

[AdvancedImagingUTSW]

Okay @annie-xd-wang, your approach worked for the camera. And now that I look more closely, I see where the class name is the initializer. I will need a little more help getting the rest of the devices properly set up.

It may be excessive, but I check the OS before trying to use concurrency tools. While I think it will work with Linux, I know it will not work with Mac.

[andrewjUTSW]

Just a comment, this is where the need for CI will become more pronounced (version issues, platforms, etc...).
Out of curiosity, why are you using a different python version than install instructions?

[AdvancedImagingUTSW]

I assume that anaconda adjusted the exact python version as different dependencies were installed. Either way, I think we need to implement an exact environment that all of us are working in. And if we want to use concurrency tools, it will need Python 3.8 3.9, or 3.10.

[andrewjUTSW]

Sorry, should have read the thread more carefully, I see now, you note it does not work for 3.7. My bad ;-P. Got it!
None the less, hopefully we can surface these issues with less pain via some CI. Also, yes, 100% agree on the exact version!

[andrewjUTSW]

Just to follow-up on the original issue, I too am reproducing the py3.7 problem. Looks like it is specifically here for me:
https://github.com/AdvancedImagingUTSW/ASLM/blob/15eb40781db28cdcdaa5adfaa372a1fefaa0b4ea/src/model/concurrency/concurrency_tools.py#L23
https://stackoverflow.com/questions/58279753/cannot-import-name-shared-memory
Looks like 3.7 doesn't include this , starts with 3.8.
Did we decide which version to settle on?

[AdvancedImagingUTSW]

I feel like 3.9, being one release behind the most recent, is a decent bet. But, I have no real good reason for that other than maybe it is more mature than 3.10?