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

Inference progress and results handling #51

Merged
merged 5 commits into from
Oct 4, 2024
Merged

Inference progress and results handling #51

merged 5 commits into from
Oct 4, 2024

Conversation

ondrej33
Copy link
Member

@ondrej33 ondrej33 commented Oct 4, 2024

This PR mainly focuses on how we track and display computation progress and results.

  • Tracking of update statuses in InferenceSolver is changed, and new struct InferenceStatusReport is used.
  • Progress messages are now displayed in a more readable format, with additional information.
  • Result information is now displayed in a more readable format, with more details.
  • Progress and result information is divided into "summary" (newly in plain text) and "details" (remains in text window).
  • Front-end methods and events for analysis are refactored, and redundant ones are removed.
  • The structure of sub-modules in analysis has been updated.
  • There is a new option to dump BDD with satisfying parametrizations.
  • BDDs and symbolic contexts are sanitized at the end of the computation.
  • Slightly different messages and removed useless options (sampling, ...) for unsatisfiable sketches.

It also introduces smaller bug fixes and modifications:

  • issues with app building (in release mode) are fixed
  • Rust dependencies are updated
  • most of the CSS for the analysis window is moved to the proper .less file (from the .ts component)
  • constructors for dynamic properties are slightly refactored
  • window size of the analysis is changed (the starting size is changed, and the minimum is also newly set)

@ondrej33 ondrej33 merged commit a0615bc into main Oct 4, 2024
6 checks passed
@ondrej33 ondrej33 deleted the analysis-state branch October 4, 2024 19:24
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