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

Python: Bug: Multiple KernelProcessEventVisibility related issues #10642

Open
drdrew42 opened this issue Feb 22, 2025 · 0 comments · May be fixed by #10643
Open

Python: Bug: Multiple KernelProcessEventVisibility related issues #10642

drdrew42 opened this issue Feb 22, 2025 · 0 comments · May be fixed by #10643
Labels
bug Something isn't working

Comments

@drdrew42
Copy link
Contributor

Describe the bug
Events emitted with visibility=KernelProcessEventVisibility.Public cannot be 'caught'.

There are overlapping causes:

  • KernelProcessEvent is configured to use enum values (but everywhere visibility is checked as an enum)
  • Once the previous is fixed, then when enqueue_step_messages processes events emitted, any Public events are processed and re-emitted before matching any of the step's edges -- however, doing so modifies the event's namespace in-place and in so doing prevents the event from propagating.

There is also a typo in LocalProcess.initialize_process ('factorie=' instead of 'factories=')

To Reproduce
Steps to reproduce the behavior:

  1. run python/samples/getting_started_with_processes/step03/step03b_food_ordering.py
  2. note that fish sandwich ends with fried fish completion (e.g. no bun), similarly for fish and chips, and none of the orders get 'packed' -- i.e. all food processes terminate on a Public event.

Expected behavior
All food orders should end with a 'Food Packed!' statement.

Platform

  • Language: Python
  • Source: main (python ver-1.20.3)
  • AI model: N/A
  • IDE: nvim
  • OS: N/A
@drdrew42 drdrew42 added the bug Something isn't working label Feb 22, 2025
@drdrew42 drdrew42 linked a pull request Feb 22, 2025 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant