Provide more explicit capture of error from PoolListHandler.get given unknown pool type?

Assessment

This issue has not been assessed yet.

Description

Currently, PoolListHandler.get throws a 500 error if it is passed a pool type that it doesn't recognize. This is as it should be :) However, one of the places this get is used is within an ajax call in sequencing.html:

https://github.com/jdereus/labman/blob/46241e448edd4fc615aeea914e8dd80bf88aad79/labcontrol/gui/templates/sequencing.html#L100-L108

At the moment there is no handling in place for errors from this ajax call, so if the 500 error is encountered, it is not handled and the code just keeps running. This leads to a cryptic error later in which the DataTable cannot be created because no results were returned from the ajax call:

At least the user can't make a garbage output, but clearly we could do better here.

This issue should be addressed before adding any new protocols to the software (as those could lead to new pool types). Related to #509 Centralize and harden identification of pool component types and probably to #510 PoolListHandler.get would identify pools from future new protocols as amplicon sequencing pools . Similar to #405 Silent 500 errors when plating samples.

Getting set up

First steps

1. Read the whole issue, then the project's contributing guide.
2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
3. Fork the repository and make your change on a branch.
4. Open a pull request that references the issue number.

More from biocore/LabControl

• In plating interface, valid sample IDs occasionally "flash" red before turning another color

  Open

  ⋯Report a problem with this issue

  • Is something wrong?

  • Already taken
  • Not beginner-friendly
  • Stale or invalid
  • Something else…
  • Send Cancel
  bug front-end question

  Difficulty 4/5 3-5 days Newbie friendliness 30/100

  biocore/LabControl#594 · Opened Sep 11, 2019

• Adding a "being processed" color/border/... to newly plated wells? Or vice versa.

  Open

  ⋯Report a problem with this issue

  • Is something wrong?

  • Already taken
  • Not beginner-friendly
  • Stale or invalid
  • Something else…
  • Send Cancel
  front-end question

  Difficulty 5/5 Over a week Newbie friendliness 35/100

  biocore/LabControl#593 · Opened Sep 11, 2019

• Cache list of active samples when the active study is changed in the plating interface

  May be free again @fedarko claimed this 2575 days ago, and no pull request is open. Open

  ⋯Report a problem with this issue

  • Is something wrong?

  • Already taken
  • Not beginner-friendly
  • Stale or invalid
  • Something else…
  • Send Cancel
  front-end

  biocore/LabControl#592 · 1 assignee · Opened Sep 11, 2019

• Refactor updateUnknownsAndDuplicates to only check certain wells, making it more efficient

  Open

  ⋯Report a problem with this issue

  • Is something wrong?

  • Already taken
  • Not beginner-friendly
  • Stale or invalid
  • Something else…
  • Send Cancel
  code refactor front-end

  Difficulty 5/5 Over a week Newbie friendliness 25/100

  biocore/LabControl#591 · Opened Sep 11, 2019

• Remove need for hidden section in plate-map-legend

  Open

  ⋯Report a problem with this issue

  • Is something wrong?

  • Already taken
  • Not beginner-friendly
  • Stale or invalid
  • Something else…
  • Send Cancel
  priority:low

  Difficulty 3/5 1-2 days Newbie friendliness 20/100

  biocore/LabControl#590 · Opened Sep 10, 2019

All issues in biocore/LabControl

Similar issues

• Reject out-of-range --seed before the run starts

  Open

  ⋯Report a problem with this issue

  • Is something wrong?

  • Already taken
  • Not beginner-friendly
  • Stale or invalid
  • Something else…
  • Send Cancel

  Difficulty 2/5 1-3 hours Newbie friendliness 84/100

  PedestrianDynamics/pyFDS-Evac#343 · Opened Sep 29, 2026

  Maintainers usually reply within 1 day

• test_run_with_command_flags fails on macOS (BSD printenv doesn't support --version)

  Open

  ⋯Report a problem with this issue

  • Is something wrong?

  • Already taken
  • Not beginner-friendly
  • Stale or invalid
  • Something else…
  • Send Cancel

  Difficulty 2/5 1-3 hours Newbie friendliness 88/100

  theskumar/python-dotenv#708 · Opened Sep 29, 2026

• gdr_decode_packed_bf16 accepts NaN fixed scale and returns NaN output

  Open

  ⋯Report a problem with this issue

  • Is something wrong?

  • Already taken
  • Not beginner-friendly
  • Stale or invalid
  • Something else…
  • Send Cancel

  Difficulty 1/5 Under an hour Newbie friendliness 88/100

  ROCm/aiter#5951 · Opened Sep 29, 2026

  Maintainers usually reply within 2 days

• DOC: document unit-preserving behaviour in timedelta/datetime arithmetic

  Open

  ⋯Report a problem with this issue

  • Is something wrong?

  • Already taken
  • Not beginner-friendly
  • Stale or invalid
  • Something else…
  • Send Cancel
  Docs Timedelta

  Difficulty 2/5 1-3 hours Newbie friendliness 72/100

  pandas-dev/pandas#69919 · Opened Sep 29, 2026

  Maintainers usually reply within 1 day

• Document why west has two ways to access west.configuration.Configuration

  Open

  ⋯Report a problem with this issue

  • Is something wrong?

  • Already taken
  • Not beginner-friendly
  • Stale or invalid
  • Something else…
  • Send Cancel
  API documentation

  Difficulty 2/5 1-3 hours Newbie friendliness 72/100

  zephyrproject-rtos/west#1009 · 2 comments · Opened Sep 29, 2026

  Maintainers usually reply within 3 days

More Python issues