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

Add streamlit components and add ability to run via Remote. #206

Merged
merged 20 commits into from
Feb 17, 2023

Conversation

wmeddie
Copy link
Member

@wmeddie wmeddie commented Feb 10, 2023

Description

Add the ability to configure and run Streamlit applications.

image (68)

Streamlit applications are unable to accept command line arguments. We always have a default of experiment_name that interfered with this so I also took the chance to remove this default argument/hyperparameter. It only really makes sense in the case of training models after all.

Pull Request Type

  • Xircuits Core (Jupyterlab Related changes)
  • Xircuits Canvas (Custom RD Related changes)
  • Xircuits Component Library
  • Xircuits Project Template
  • Testing Automation
  • Documentation
  • Others (Please Specify)

Type of Change

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Tests

See the included streamlit_sample.xircuits file.

Tested on?

  • Windows
  • Linux Ubuntu
  • Centos
  • Mac
  • Others (State here -> xxx )

Notes

@MFA-X-AI MFA-X-AI marked this pull request as ready for review February 17, 2023 11:49
Copy link
Member

@MFA-X-AI MFA-X-AI left a comment

Choose a reason for hiding this comment

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

Great addition to our library, we have a frontend framework support now.
Here's a preview of how it works
https://user-images.githubusercontent.com/68586800/219625051-97b75ee6-fe54-4c42-85e8-ebc5abeaf116.mp4
Thanks boss, merging with main.

@MFA-X-AI MFA-X-AI merged commit 35a4b4e into master Feb 17, 2023
@MFA-X-AI MFA-X-AI deleted the add-streamlit branch February 26, 2023 18:53
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.

4 participants