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

[Docs] Document protocol used by Flyte CoPilot #317

Open
2 of 13 tasks
kumare3 opened this issue May 19, 2020 · 5 comments
Open
2 of 13 tasks

[Docs] Document protocol used by Flyte CoPilot #317

kumare3 opened this issue May 19, 2020 · 5 comments
Labels
documentation Improvements or additions to documentation enhancement New feature or request flytecopilot

Comments

@kumare3
Copy link
Contributor

kumare3 commented May 19, 2020

Motivation: Why do you think this is important?
When new users are using CoPilot, the protocol used to communicate should be clearly documented.

Goal: What should the final outcome look like, ideally?
Any user should be able to refer to this documentation and use coPilot

Describe alternatives you've considered
NA

Flyte component

  • Overall
  • Flyte Setup and Installation scripts
  • Flyte Documentation
  • Flyte communication (slack/email etc)
  • FlytePropeller
  • FlyteIDL (Flyte specification language)
  • Flytekit (Python SDK)
  • FlyteAdmin (Control Plane service)
  • FlytePlugins
  • DataCatalog
  • FlyteStdlib (common libraries)
  • FlyteConsole (UI)
  • Other

[Optional] Propose: Link/Inline
NA

Additional context
NA

Is this a blocker for you to adopt Flyte
This is a blocker to use CoPilot

@kumare3 kumare3 added enhancement New feature or request untriaged This issues has not yet been looked at by the Maintainers labels May 19, 2020
@kumare3 kumare3 added documentation Improvements or additions to documentation and removed untriaged This issues has not yet been looked at by the Maintainers labels May 11, 2021
@kumare3
Copy link
Contributor Author

kumare3 commented May 11, 2021

The design for flyte copilot is available here. this should be converted to the documentation

@kumare3
Copy link
Contributor Author

kumare3 commented May 11, 2021

@SandraGH5 FYI

@kumare3 kumare3 self-assigned this Dec 28, 2021
@cosmicBboy cosmicBboy changed the title [Feature]Document protocol used by Flyte CoPilot [Docs] Document protocol used by Flyte CoPilot Apr 21, 2022
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 6, 2022
* Start-Node outputUri copied in input and output result

Signed-off-by: Prafulla Mahindrakar <[email protected]>

* Test fix

Signed-off-by: Prafulla Mahindrakar <[email protected]>

* Linter fix

Signed-off-by: Prafulla Mahindrakar <[email protected]>

* Incorporated the feedback

Signed-off-by: Prafulla Mahindrakar <[email protected]>

* Only adding outputUri in the result for start-node

Signed-off-by: Prafulla Mahindrakar <[email protected]>

* linter issues

Signed-off-by: Prafulla Mahindrakar <[email protected]>
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 6, 2022
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 20, 2022
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Dec 20, 2022
Signed-off-by: Prafulla Mahindrakar <[email protected]>
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Aug 9, 2023
* Start-Node outputUri copied in input and output result

Signed-off-by: Prafulla Mahindrakar <[email protected]>

* Test fix

Signed-off-by: Prafulla Mahindrakar <[email protected]>

* Linter fix

Signed-off-by: Prafulla Mahindrakar <[email protected]>

* Incorporated the feedback

Signed-off-by: Prafulla Mahindrakar <[email protected]>

* Only adding outputUri in the result for start-node

Signed-off-by: Prafulla Mahindrakar <[email protected]>

* linter issues

Signed-off-by: Prafulla Mahindrakar <[email protected]>
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Aug 21, 2023
@github-actions
Copy link

Hello 👋, This issue has been inactive for over 9 months. To help maintain a clean and focused backlog, we'll be marking this issue as stale and will close the issue if we detect no activity in the next 7 days. Thank you for your contribution and understanding! 🙏

@github-actions github-actions bot added the stale label Aug 26, 2023
@github-actions
Copy link

github-actions bot commented Sep 2, 2023

Hello 👋, This issue has been inactive for over 9 months and hasn't received any updates since it was marked as stale. We'll be closing this issue for now, but if you believe this issue is still relevant, please feel free to reopen it. Thank you for your contribution and understanding! 🙏

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Sep 2, 2023
@eapolinario eapolinario reopened this Nov 2, 2023
@github-actions github-actions bot removed the stale label Nov 3, 2023
eapolinario pushed a commit to eapolinario/flyte that referenced this issue Apr 30, 2024
Signed-off-by: Prafulla Mahindrakar <[email protected]>
austin362667 pushed a commit to austin362667/flyte that referenced this issue May 7, 2024
Signed-off-by: Prafulla Mahindrakar <[email protected]>
@davidmirror-ops
Copy link
Contributor

Is this still a need? Also the referenced design doc is not accesible

robert-ulbrich-mercedes-benz pushed a commit to robert-ulbrich-mercedes-benz/flyte that referenced this issue Jul 2, 2024
Signed-off-by: Prafulla Mahindrakar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request flytecopilot
Projects
None yet
Development

No branches or pull requests

4 participants