Opened 17 years ago

Last modified 11 years ago

#13 assigned enhancement

Add proof object search facilities to IdView

Reported by: David Aspinall <da+pgtrac@…> Owned by: Graham Dutton
Priority: minor Milestone:
Component: 1:pg-eclipse Keywords:
Cc:

Description (last modified by David Aspinall)

Add a proof object search facility to IdView, similar to the Open Type dialog in Java, so we can find names dynamically. A more advanced facility would be to search on object contents.

This may require using prover-side search, and/or interfacing with ProverKnowledge.

Change History (6)

comment:1 Changed 17 years ago by David Aspinall <da+pgtrac@…>

Regular expression name searching has now been added to IdView, and the ProverSyntax datastructures storing names have been overhauled. This works pretty well, although there are a few quirks to sort out:

  • Statements can't be loaded until Isar has a theory context. Might be solved by using PGIP idshow instead (although that probably needs tuning).
  • Keeping qualified names: these clutter the display a bit, so we could strip them on display but recover the full names for passing to Prover Knowledge for lookup. Needs an extra field containing fully qualified name shadow of items, or an index into allitems.

  • Regexp search: this updates on every change which makes it a bit slow. Simplest to change to use another event (focus leaving? return pressed?). More sophisticated would be to set/reset a short timer after the change, and update only after the timer has expired.
  • Possible race condition somewhere seems to sometimes prevent description text from being updated, with response "Could not load statement" when statement was in fact loaded (next view succeeds). Perhaps race is in Prover Knowledge.

comment:2 Changed 17 years ago by David Aspinall <da+pgtrac@…>

Milestone: PG-Eclipse-1.1.0PG-Eclipse-1.0.6
Owner: changed from David Aspinall to anonymous

Further remark on "Statement could not be loaded":

  • Also seen (repeatedly) on theorem lookups that return multiple results/format that confuses ProverKnowledge, e.g. IntDef?.int_distrib. This produces a response but PK reports could not be loaded.

Although this may be an inconsistency in Isabelle's name reporting, perhaps: it uses synthesised names for individual theorems with sets of theorems (int_distrib_1, etc). We don't want to see these, I'm fixing this now.

comment:3 Changed 17 years ago by David Aspinall

Description: modified (diff)

comment:4 Changed 17 years ago by David Aspinall

Most desirable next improvement for next version:

  • Use glob search instead of regexp search on names.

Otherwise support is OK for now.

comment:5 Changed 17 years ago by David Aspinall

Milestone: PG-Eclipse-1.0.6PG-Eclipse-1.1.0
Owner: changed from anonymous to Graham Dutton
Priority: majorminor
Status: newassigned

comment:6 Changed 11 years ago by David Aspinall

Milestone: PG-Eclipse-1.1.0

Milestone PG-Eclipse-1.1.0 deleted

Note: See TracTickets for help on using tickets.