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

Sync numerous updates from older repo #4

Merged
merged 8 commits into from
Feb 22, 2024
Merged

Conversation

markwaddle
Copy link
Contributor

So the team can switch to the new public repo and everyone can benefit

  • renames scripts to use hyphens, ex: node-engine-service, instead of
    underscores (more conventional)
  • renames command-line arguments to use hyphens, ex: --log-level,
    instead of underscores (more conventional)
  • various README updates for the above
  • adds colorized logging for nodeengine using rich package
  • adds pytest optional dependency
  • adds launch configuration for "Node Engine Service", which was
    mentioned in the docs and is generally helpful
  • adds pylance settings to format, check types, fix on save
  • enables stream_log on simple-chat-client flows so you can see the logs
    with debug-service.py
  • introduces FlowExecutor protocol
  • flows run flows in-process by default now, instead of making network
    calls back to the http service
  • remote clients, such as apps, can use the NodeEngineClient
  • moved SSE connection management to the service instead of the runtime
  • added ability for executing Runtime in process, and add event
    consumers
  • service registers each SSE connection as an event consumer
  • fixed the notebooks so they both run and display reasonable outputs
  • removed need to have "examples" package, to avoid name collisions with
    such a generic name
  • adds debug_information collection by default to error that results
    from flow execution
  • adds support for loading installed module components by qualified module names
  • avoids infinite loop in moduleloader

So the team can switch to the new public repo and everyone can benefit

- renames scripts to use hyphens, ex: `node-engine-service`, instead of
  underscores (more conventional)
- renames command-line arguments to use hyphens, ex: `--log-level`,
  instead of underscores (more conventional)
- various README updates for the above
- adds colorized logging for nodeengine using `rich` package
- adds pytest optional dependency
- adds launch configuration for "Node Engine Service", which was
  mentioned in the docs and is generally helpful
- adds pylance settings to format, check types, fix on save
- enables stream_log on simple-chat-client flows so you can see the logs
  with debug-service.py
- introduces FlowExecutor protocol
- flows run flows in-process by default now, instead of making network
  calls back to the http service
- remote clients, such as apps, can use the NodeEngineClient
- moved SSE connection management to the service instead of the runtime
- added ability for executing Runtime in process, and add event
  consumers
- service registers each SSE connection as an event consumer
- fixed the notebooks so they both run and display reasonable outputs
- removed need to have "examples" package, to avoid name collisions with
  such a generic name
- adds debug_information collection by default to error that results
  from flow execution
Copy link
Collaborator

@dluc dluc left a comment

Choose a reason for hiding this comment

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

🚀 thanks!

@markwaddle markwaddle merged commit cebe98b into main Feb 22, 2024
2 checks passed
@markwaddle markwaddle deleted the markwaddle/sync-updates branch February 22, 2024 19:59
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