Update library requirements.txt

[SkepticRaven]

Updating the requirements.txt to not be 2-3 year old libraries. There were a couple other changes to get compatibility working again.

Tests run at the beginning of this pull:

• Ran unittests: python3 -m unittest discover -s /behavior-classifier/tests/
  • Ran 33 tests in 0.785s OK
• Ran python3 initialize_project.py on a project
• Opened the GUI from an updated gui-vm and trained a classifier

I don't have a bunch of dev environments to test, so these were all conducted inside singularity (running debian). One of the fixed I needed to do (upgrade to shapely change) may address https://github.com/KumarLabJax/JABS-behavior-classifier/issues/5#issuecomment-1621725227 Also address https://github.com/KumarLabJax/JABS-behavior-classifier/issues/24

Would be good to have someone with a mac environment to share their experience.

[SkepticRaven]

Works successfully on windows 10 (python 3.10.11)

[SkepticRaven]

@dahhei tried to install the venv on a mac and discovered shiboken needed to be updated from 5 -> 6

[anshu957]

The app gets installed without any issue on mac M1. I can also train a classifier for single mouse dataset. However, when I try to export the classifier, I get the following error:

Traceback (most recent call last):
  File "/Users/chouda/tmp/jabs_app/JABS-behavior-classifier/src/ui/main_window.py", line 215, in _export_training_data
    out_path = export_training_data(self._project,
  File "/Users/chouda/tmp/jabs_app/JABS-behavior-classifier/src/project/export_training.py", line 92, in export_training_data
    (1,), dtype=np.int)
  File "/Users/chouda/tmp/jabs_app/JABS-behavior-classifier/jabs.venv/lib/python3.9/site-packages/numpy/__init__.py", line 313, in __getattr__
    raise AttributeError(__former_attrs__[attr])
AttributeError: module 'numpy' has no attribute 'int'.
`np.int` was a deprecated alias for the builtin `int`. To avoid this error in existing code, use `int` by itself. Doing this will not modify any behavior and is safe. When replacing `np.int`, you may wish to use e.g. `np.int64` or `np.int32` to specify the precision. If you wish to review your current use, check the release note link for additional information.

[SkepticRaven]

This branch would have particularly benefited from having more coverage on unittests. Given that I missed that one, I should probably click all the buttons on the gui and even run tests on the non-gui version to make sure things don't change before a merge. New list of tests that should probably go through:

• Click all the buttons on gui
• Test multiple project pose versions (2, 3, 4, 5)
• Test command line interface (classify.py, initialize_project.py, and stats.py)

Also I upgraded the singularity gui vm from buster -> bookworm, so should make the non-gui one also be bookworm (not bullseye).

[SkepticRaven]

A couple notes from testing/poking around:

• (Critical) Pause (after play) causes segfault ~50% of the time
  • Currently unclear cause, as threads seem to get closed correctly. It's happening after/during the repaint even after it stops
• (Minor) Text littered around the UI is now lighter compared to older versions
  • This may be linked to OS settings and may be limited to my Ubuntu host machine (not present in CentOS test)

[SkepticRaven]

Latest commit should clear the 2 issues above. Remaining checks before merge:

• Confirm crash fix native on macs
• Test stats.py