Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Validator Improvements #666

Closed
signedav opened this issue Apr 8, 2022 · 5 comments
Closed

Validator Improvements #666

signedav opened this issue Apr 8, 2022 · 5 comments
Labels

Comments

@signedav
Copy link
Member

signedav commented Apr 8, 2022

  • In Validator make this (maybe instead of Zoom to coordinates)
    See in attributetable.
    image
  • Choose type of error that should be validated or not (e.g. nur topologie oder genau topologie nicht)
  • Evtl. auch filtern des resultat nach typ (mit reiter oder mit checkboxen)
  • Text in Result should be selectable (to copy parts of it (e.g. OID)) - Evtl. auch nur Copy OID
@signedav
Copy link
Member Author

Concerning the type of check I know that at least we can skip errors with:
--skipGeometryErrors, --sqlEnableNull, --skipReferenceErrors

The filter per type I currently don't know where to get the info from, when the result is written to an XTF based on the model https://models.interlis.ch/tools/IliVErrors.ili

@signedav
Copy link
Member Author

signedav commented Apr 22, 2022

So no filtering of the result but being able to pass the Arguments above. Maybe with checkboxes or dropdown.

Ilivalidator offers:
image

@signedav
Copy link
Member Author

signedav commented Apr 29, 2022

Possible Implementations are:

  • Right click "copy text"
  • Filter configuration to decide what checks can be skipped (a little like above)
  • Button to highlight selected
  • Button to zoom to selected
  • Button to center selected

@signedav
Copy link
Member Author

signedav commented Sep 2, 2022

Additional ideas is to support singlePassValidation (claeis/ili2db#483 (not yet done in ili2db)) and maybe disableValidation. Because Claude says:

Export ohne Validierung ist nützlich, weil es mal prüft, ob mit den technischen Aspekten (t_typ's, t_id's usw.) alles ok ist

@signedav
Copy link
Member Author

signedav commented Nov 3, 2022

Closing since most parts are implemented. For the others I'll open new issues.

@signedav signedav closed this as completed Nov 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant