search

Appsilon / mbaza

Save 99% of Your Time Classifying Camera-Trap Footage. Completely Free.
https://appsilon.com/data-for-good/mbaza-ai/
GNU Affero General Public License v3.0
31 stars 7 forks source link

Specify csv file and database with photos locations #348

Closed jchojna closed 2 years ago

jchojna commented 2 years ago

Changes

Closes #179

How to test

jchojna commented 2 years ago

Good job! tada A couple of issues to resolve before merging:

  1. Images don't load in the map popup: Screenshot from 2022-09-05 16-57-38

  2. The absolute_location column is written to the CSV on export, but it is essentially ignored when importing. This inconsistency can be troublesome in the future. I see two clean solutions:

    1. Implement a mechanism to add custom properties on import, but ignore them on export. Might require serious refactoring.
    2. Prepend the image DB path to the (relative) location everywhere where the absolute path is needed. We'll need to pass around the DB path via props / function arguments - not ideal, but simple. I'd prefer this solution.

Bulletpoing 1 - this is strange, it works for me all the time. How can I reproduce?