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

Mismatching fields in generated trace in case of CHR #147

Open
jwintz opened this issue Jun 5, 2022 · 3 comments
Open

Mismatching fields in generated trace in case of CHR #147

jwintz opened this issue Jun 5, 2022 · 3 comments

Comments

@jwintz
Copy link
Contributor

jwintz commented Jun 5, 2022

Hi,

This issue concerns the trace-browser branch, the elaborator specifically.

"suspend_sibling": { "goal_text": "even X0", "goal_id": 10 },

I see a mismatch here, between the sibling goal_id and the goal_text fields.

It might very well be a misunderstanding of mine, but aren't they supposed to match the definition of the following step:

?

@gares
Copy link
Contributor

gares commented Jun 5, 2022

Goal 10 is resumed because the variable it contains was assigned to s X.... If I understand the question, it is normal that the goal text is not exactly the same, the purpose is indeed to see the updated goal.

@jwintz
Copy link
Contributor Author

jwintz commented Jun 5, 2022

Fine by me ! It was more a matter of understanding ... I just find it strange to click on such a text to end up on a step which is not named the same in the context of the browser (the display ). In the screenshot below, the sibling links to goal id 10, which actually is the one display in the second screenshot.

Screenshot 2022-06-05 at 17 43 04

Screenshot 2022-06-05 at 17 43 54

If only ergonomic questions remain, this issue can very well be closed for me.

Thanks.

@gares
Copy link
Contributor

gares commented Jun 5, 2022

I don't know what is best to display, but I think the trace is correct: 7 (0|5) should have as event the resumption of goal 10, since it sets X0 to (s X1).

This phenomena is true in general, a new goal (p X) may become p (f Y) before it gets the chance to be handled. Maybe one could do a wdiff and color differently f Y, signaling that the goal at creation time was less instantiated. In a few cases it would help I guess. In general, I'm not so sure, since it is very frequent. It probably depends on the display of the differences. If it was "optional/no demand" then I think it would be good.

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

No branches or pull requests

2 participants