Skip to content
Snippets Groups Projects

Handle polygon index size errors

Merged Erwan Rouchet requested to merge large-polygon-error into master
All threads resolved!

Closes #721 (closed)

This removes the memory size constraint, but keeps a maximum size on serializers set to 300 points.

This catches the index size errors in any SQL query made by Django from Zone.objects.create, Zone.objects.update, Zone.objects.bulk_create, Zone.objects.bulk_update, Zone().save(), etc. by overriding the Zone queryset, similarly to how we overrode the Element queryset to add trash.

This uses an unrealistic polygon; out of 18000 Transkribus polygons, none could be simplified down to 300 points and crash properly, so I could not use any for the unit tests.

Edited by Erwan Rouchet

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Bastien Abadie resolved all threads

    resolved all threads

  • Bastien Abadie resolved all threads

    resolved all threads

  • Bastien Abadie added 1 commit

    added 1 commit

    • da7c96ec - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Bastien Abadie mentioned in commit 70fbebf1

    mentioned in commit 70fbebf1

  • mentioned in issue #1373 (closed)

  • Please register or sign in to reply
    Loading