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

ComputationTrace: Fixed argument for @NotNull parameter must not be n… #2342

Conversation

mgronover
Copy link
Member

Addresses issue: 2341

just fixed that exception.

Copy link
Member

@alexanderpann alexanderpann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. If you just add a null check, you can just merge the PR.

@mgronover mgronover merged commit 11de7ba into maintenance/mps20213 Dec 13, 2023
3 checks passed
@mgronover mgronover deleted the bugfix/2341-traceexplorer-has-notnull-parameter-exception branch December 13, 2023 14:34
mgronover added a commit that referenced this pull request Dec 14, 2023
* Fix and improve trace explorer (#2328)

* ComputationTrace: Added popup menu for filtering #2319

- Fixed update and un-/folding of trace tree
- Fixed NPE
- Fixed possible endless loop when a used filter is not avoiding it and did massive cleanups
- No more lazy creation of children in ComputationTrace.

* Automatically remove assessment result entries with invalid references

* Fix typo in changelog

* Update to 2021.3.5

* Bugfix/2329 follows up 2319 further improvements for the trace explorer (#2335)

* ComputationTrace: improved filter interface and refactored handling of options used by the TraceTab #2329

* ComputationTrace: Fixed argument for @NotNull parameter must not be null #2341 (#2342)

* interpreter: generalize the evaluate methods into one method with options

---------

Co-authored-by: Michael Gronover <38247916+mgronover@users.noreply.github.com>
Co-authored-by: Alexander Pann <apann@itemis.com>
Co-authored-by: Michael Gronover <mgronover@itemis.com>
Co-authored-by: Alexander Pann <88385944+alexanderpann@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
mgronover added a commit that referenced this pull request Dec 14, 2023
* Fix and improve trace explorer (#2328)

* ComputationTrace: Added popup menu for filtering #2319

- Fixed update and un-/folding of trace tree
- Fixed NPE
- Fixed possible endless loop when a used filter is not avoiding it and did massive cleanups
- No more lazy creation of children in ComputationTrace.

* Automatically remove assessment result entries with invalid references

* Fix typo in changelog

* Update to 2021.3.5

* Bugfix/2329 follows up 2319 further improvements for the trace explorer (#2335)

* ComputationTrace: improved filter interface and refactored handling of options used by the TraceTab #2329

* ComputationTrace: Fixed argument for @NotNull parameter must not be null #2341 (#2342)

* interpreter: generalize the evaluate methods into one method with options

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Michael Gronover <38247916+mgronover@users.noreply.github.com>
Co-authored-by: Alexander Pann <apann@itemis.com>
Co-authored-by: Michael Gronover <mgronover@itemis.com>
Co-authored-by: Alexander Pann <88385944+alexanderpann@users.noreply.github.com>
mgronover added a commit that referenced this pull request Jan 16, 2024
* Fix and improve trace explorer (#2328)

* ComputationTrace: Added popup menu for filtering #2319

- Fixed update and un-/folding of trace tree
- Fixed NPE
- Fixed possible endless loop when a used filter is not avoiding it and did massive cleanups
- No more lazy creation of children in ComputationTrace.

* Automatically remove assessment result entries with invalid references

* Fix typo in changelog

* Update to 2021.3.5

* Bugfix/2329 follows up 2319 further improvements for the trace explorer (#2335)

* ComputationTrace: improved filter interface and refactored handling of options used by the TraceTab #2329

* ComputationTrace: Fixed argument for @NotNull parameter must not be null #2341 (#2342)

* interpreter: generalize the evaluate methods into one method with options

* core.base: run assessement result entry removal under write acces in EDT

With the previous implementation, the removed node didn't show up on the left side of the diff view.

* mpsutil.contextactions: do not update the tool when the node is null

* Use reusable automerge workflow

* mpsutil.interpreter: add a dummy computation trace when the computation trace is not available

---------

Co-authored-by: Michael Gronover <38247916+mgronover@users.noreply.github.com>
Co-authored-by: Alexander Pann <apann@itemis.com>
Co-authored-by: Michael Gronover <mgronover@itemis.com>
Co-authored-by: Alexander Pann <88385944+alexanderpann@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Sergej Koščejev <sergej@koscejev.cz>
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.

2 participants