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.

Sign up for GitHub

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

Jump to bottom

Filter in Eventlog gives internal error even with bugfix 932 #938

Open
Jeremdu44 opened this issue Aug 24, 2022 · 2 comments
Open

Filter in Eventlog gives internal error even with bugfix 932 #938

Jeremdu44 opened this issue Aug 24, 2022 · 2 comments
Assignees
@levy
Labels
in IDE / sequence chart major A serious issue, reducing the usability new New issue that has not been yet acknowledged

Comments

@Jeremdu44
Copy link

  • OMNeT++ version: [6.0]
  • OS: [latest Linux, using Debian]

Hello everybody

I have some problems with the eventlog filter these days, now that I am in 6.0 I had to patch first my omnet version in order to make the filter window appears.
But now that is done, I get some troubles with its fonctionality :

So, I compiled the latest version of the 6.0.x branch of git. The last commit https ://github.com/omnetpp/omnetpp/commit/fb11f604704afe3e2a6f822b444d5c0a6c15f99c has an id fb11f60.

I compiled the tutorial example named "tictoc", activated the logs in the ini file and made a simple simu tictoc1. I then displayed the elog sequence chart (15-36-10.png).

Capture du 2022-08-23 15-36-10

Then I filtered (15-36-43.png),

Capture du 2022-08-23 15-36-43

which gave me an empty sequence chart (15-34-24) :

Capture du 2022-08-23 15-34-24

It is on this capture that we can read the version number (6.0) and the commit number corresponding to the bugfix 932 (fb11f60). Finally, by removing the filters, we get another error (last capture 15-34-58.png).

Capture du 2022-08-23 15-34-58

All this in Linux version under Debian.
Thanks in advance for any clue for this issue
@Jeremdu44 Jeremdu44 added the new New issue that has not been yet acknowledged label Aug 24, 2022
@levy levy self-assigned this Aug 24, 2022
@levy levy added in IDE / sequence chart major A serious issue, reducing the usability labels Aug 24, 2022
@levy
Copy link
Contributor

levy commented Aug 24, 2022

I can confirm this bug. The fix will be available in the 6.0.x branch soon.

With the fix applied, you may encounter errors during opening the eventlog files in the IDE, because restoring the filter settings will fail. The workaround is simple: reset the stored filter settings by closing the eventlog files and opening them again.

rhornig pushed a commit that referenced this issue Aug 25, 2022
@levy
ui: Fixed issue #938, filtering the eventlog in the sequence chart ma…
502ebba 
…kes it empty.

Several parts were commented out during the porting of the old implementation
from C++ to Java. The missing parts were revived and tested manually.
rhornig pushed a commit that referenced this issue Aug 25, 2022
@levy
ui: Fixed issue #938, filtering the eventlog in the sequence chart ma…
ef65a41 
…kes it empty.

Several parts were commented out during the porting of the old implementation
from C++ to Java. The missing parts were revived and tested manually.
@levy
Copy link
Contributor

levy commented Aug 25, 2022

This should be fixed the master and omnetpp-6.0.x branches.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@levy levy

Labels
in IDE / sequence chart major A serious issue, reducing the usability new New issue that has not been yet acknowledged
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@levy @Jeremdu44