cnr-isti-vclab / TagLab

A CNN based image segmentation tool oriented to marine data analysis
https://taglab.isti.cnr.it/
GNU General Public License v3.0
84 stars 33 forks source link

Bug: Bricking TagLab with deleted / moved default dictionary #151

Closed Jordan-Pierce closed 3 weeks ago

Jordan-Pierce commented 3 months ago

The following will brick TagLab.py

self.default_dictionary is a value being read from a registry, but the events to handle if that value (path to default dictionary) is incorrect (deleted / modified) do not do what is intended (i.e., update the default dictionary to ./dictionaries/scripts.json); instead it just bricks.

Luckily the registry stores the expected value (path) of the self.default_dictionary and outputs it to log, so the user can create a new version in the same location.

Matt-Davies-314 commented 1 month ago

Hi Jordan, one of my colleagues is having a similar issue. Where can I find the log to find the expected path?

Jordan-Pierce commented 1 month ago

@Matt-Davies-314 by "log" I mean the command line console / terminal. It should complain about not finding a self.default_dictionary at the following path; that path is where you should create an updated version of your dictionary, otherwise you cannot unbrick it.

Matt-Davies-314 commented 1 month ago

Thanks Jordan, then I guess it is not the issue. I've just seen the other thread with the 'Name' error, that's the same problem we're having.

Jordan-Pierce commented 1 month ago

Yep, someone else on our team is getting the same issue. The Name issue is that the default dictionary being used when opening TagLab doesn't have a Name key in the JSON file, which is why it's throwing an error. But, I'm not sure why it's happening. Will share any fixes if we find them