Skip to content

Conversation

mihow
Copy link
Collaborator

@mihow mihow commented Nov 13, 2024

Previously several counts and lists of occurrences and taxa were filtered by a default threshold of their best classification score. This cause confusion because some counts in the UI were filtered, and others were not, and the threshold is only adjustable by the user in some views.

The decision was made to remove the threshold and show all occurrences & taxa unfiltered. Any filtering should be user configurable and passed to the backend via a query param.

Copy link

netlify bot commented Nov 13, 2024

Deploy Preview for ami-dev ready!

Name Link
🔨 Latest commit d86ea34
🔍 Latest deploy log https://app.netlify.com/sites/ami-dev/deploys/67343719f9a12d00086f2761
😎 Deploy Preview https://deploy-preview-613--ami-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 75
Accessibility: 89
Best Practices: 92
SEO: 100
PWA: 80
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@mihow mihow merged commit 378b9dc into main Nov 14, 2024
6 checks passed
@mihow mihow deleted the feat/no-hidden-threshold branch November 14, 2024 09:51
Copy link

sentry-io bot commented Nov 19, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ TypeError: SourceImageCollection.sample_random() got an unexpected keyword argument 'deployment_id' ami.tasks.populate_collection View Issue
  • ‼️ OperationalError: remaining connection slots are reserved for non-replication superuser connections ami.ml.models.pipeline.save_results View Issue

Did you find this useful? React with a 👍 or 👎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant