search

opensemanticsearch / open-semantic-search

Open Source research tool to search, browse, analyze and explore large document collections by Semantic Search Engine and Open Source Text Mining & Text Analytics platform (Integrates ETL for document processing, OCR for images & PDF, named entity recognition for persons, organizations & locations, metadata management by thesaurus & ontologies, search user interface & search apps for fulltext search, faceted search & knowledge graph)
https://opensemanticsearch.org
GNU General Public License v3.0
941 stars 164 forks source link

Several minor issues and fixes #458

Open pengchuan-lin-bp opened 1 year ago

pengchuan-lin-bp commented 1 year ago

Hello,

Here are some other minor issues that I encountered while trying out OSS and fixes I used for them.

  1. File system monitoring script uses pyinotify, which doesn't work well with NFS (network file system). I modified src/open-semantic-etl/src/opensemanticetl/etl_filemonitoring.py to use polling observer from watchdog instead, but the drawback is that it will no longer be instantaneous anymore without polling constantly. I also needed to list watchdog as a requirement in src/open-semantic-etl/src/opensemanticetl/requirements.txt.
  2. In the search page, the black tooltip box that shows up while hovering over file path will cut off long file path strings due to max-width configured by .tooltip in src/solr-php-ui/src/css/foundation.css. I added word-break: break-all; so that the tooltip box displays the entire string without any cut off.
  3. In the search page. the sorting option dropdown will have a fourth, repeating option if Newest or Oldest is chosen. I wrapped the last option $sort_other in src/solr-php-ui/src/templates/select_sort.php with <?php if(false): ?> and <?php endif; ?> so that it won't show up anymore.

I hope this can be of help!

Mandalka commented 1 year ago

Please add a Pullrequest for 2 and one for 3 so i can easier review it and one to add to Contributors in README.md. Sorry not having time for deeper look at 1 these days.

pengchuan-lin-bp commented 1 year ago

All three pull requests are created. No worries for 1. Since it's NFS specific, I simply wrote it down to help anyone that might need this information.

Thank you for your help!