danionita / e3tools

e3tool is a Java GUI-based tool for constructing and evaluating e3value models. Includes the e3fraud fraud assessment extension
Other
3 stars 4 forks source link

Improved, explicit model checker #67

Open danionita opened 7 years ago

danionita commented 7 years ago

The checker:

It should run

bobismijnnaam commented 7 years ago

Can you send me the pdf for the manual? I can't find it in my gmail :(

bobismijnnaam commented 7 years ago

I'm thinking about the error "Cannot find end-stimulus when calculating path” (starting at start stimulus [ID])". If this is the case, you have either a flow collision ("cannot connect end stimulus as down/up") or an unconnected signal port (an error we check for). Does the error "cannot find end stimulus" still make sense (do we need it) as a separate error or do the two earlier mentioned errors describe all the cases in which an end stimulus might not be found?

danionita commented 7 years ago

I think the more granular errors make more sense. The only other situation in which an end-stimulus is missing - except if it is not connected or if a start stimulus is connected instead - is if there is a loop. Do we or can we check for this?

On Sun, Oct 16, 2016 at 8:20 PM, Bob Rubbens notifications@github.com wrote:

I'm thinking about the error "Cannot find end-stimulus when calculating path” (starting at start stimulus [ID])". If this is the case, you have either a flow collision ("cannot connect end stimulus as down/up") or an unconnected signal port (an error we check for). Does the error "cannot find end stimulus" still make sense (do we need it) as a separate error or do the two earlier mentioned errors describe all the cases in which an end stimulus might not be found?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/danionita/e3tools/issues/67#issuecomment-254064007, or mute the thread https://github.com/notifications/unsubscribe-auth/AKjEpAflKDQEoZoyZmFjRaThHdmS88rJks5q0mr4gaJpZM4KK6Op .

bobismijnnaam commented 7 years ago

I was planning to write something that detects a loop. So then the error "end stimulus not found" can never appear I guess.

On 18 October 2016 at 11:11, danionita notifications@github.com wrote:

I think the more granular errors make more sense. The only other situation in which an end-stimulus is missing - except if it is not connected or if a start stimulus is connected instead - is if there is a loop. Do we or can we check for this?

On Sun, Oct 16, 2016 at 8:20 PM, Bob Rubbens notifications@github.com wrote:

I'm thinking about the error "Cannot find end-stimulus when calculating path” (starting at start stimulus [ID])". If this is the case, you have either a flow collision ("cannot connect end stimulus as down/up") or an unconnected signal port (an error we check for). Does the error "cannot find end stimulus" still make sense (do we need it) as a separate error or do the two earlier mentioned errors describe all the cases in which an end stimulus might not be found?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/danionita/e3tools/issues/67#issuecomment-254064007, or mute the thread https://github.com/notifications/unsubscribe-auth/ AKjEpAflKDQEoZoyZmFjRaThHdmS88rJks5q0mr4gaJpZM4KK6Op .

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/danionita/e3tools/issues/67#issuecomment-254451214, or mute the thread https://github.com/notifications/unsubscribe-auth/AC8IaMzzvHWhBWlkOoG3C4JEZkN1UYZdks5q1I1bgaJpZM4KK6Op .

danionita commented 7 years ago

What is the status of this? I see the button in the menu but when I press it something is outputted on System.out but nothing happens in the GUI.

bobismijnnaam commented 7 years ago

Implemented as of de2047e0801f8c58afff8ffcdb9f77ca78e4393e.

danionita commented 7 years ago

Issues so far:

bobismijnnaam commented 7 years ago

@danionita Fixed your earlier two points. Is this enough for now?

danionita commented 7 years ago

Looks like it. If further testing reveals other bugs, we can re-open the issue.